mbox series

[0/2] Add I2C driver for ZTE ZX2967 family SoCs

Message ID 1495947576-11037-1-git-send-email-shawnguo@kernel.org
Headers show
Series Add I2C driver for ZTE ZX2967 family SoCs | expand

Message

Shawn Guo May 28, 2017, 4:59 a.m. UTC 
From: Shawn Guo <shawn.guo@linaro.org>


Hi Wolfram,

The ZX2967 I2C driver submission had gone through quite a number of
review iterations [1], and seems close to be accepted.  But for some
reason, the ball got dropped from there.  I'm here to pick it up and
hopefully move it forward.

This is basically a resend of the latest posting from Baoyou with
Wolfram's comments on v7 fully addressed.

Changes since v7:
 - Rebase and test on v4.12-rc2.
 - Drop the dev_err() and hardware reset on timeout, which is not really
   necessary.
 - Return -ETIMEDOUT instead of -EIO on timeout.
 - Unify zx2967_i2c_xfer_write_bytes() and zx2967_i2c_xfer_read_bytes()
   into one function zx2967_i2c_xfer_bytes(), and zx2967_i2c_xfer_write()
   and zx2967_i2c_xfer_read() into zx2967_i2c_xfer_msg().

Shawn

[1] https://lkml.org/lkml/2017/2/22/101

Baoyou Xie (2):
  dt: bindings: add documentation for zx2967 family i2c controller
  i2c: zx2967: add i2c controller driver for ZTE's zx2967 family

 .../devicetree/bindings/i2c/i2c-zx2967.txt         |  22 +
 drivers/i2c/busses/Kconfig                         |   9 +
 drivers/i2c/busses/Makefile                        |   1 +
 drivers/i2c/busses/i2c-zx2967.c                    | 610 +++++++++++++++++++++
 4 files changed, 642 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/i2c/i2c-zx2967.txt
 create mode 100644 drivers/i2c/busses/i2c-zx2967.c

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Shawn Guo June 10, 2017, 2:32 p.m. UTC | #1 
Hi Wolfram,

On Sun, May 28, 2017 at 12:59:34PM +0800, Shawn Guo wrote:
> From: Shawn Guo <shawn.guo@linaro.org>

> 

> Hi Wolfram,

> 

> The ZX2967 I2C driver submission had gone through quite a number of

> review iterations [1], and seems close to be accepted.  But for some

> reason, the ball got dropped from there.  I'm here to pick it up and

> hopefully move it forward.

> 

> This is basically a resend of the latest posting from Baoyou with

> Wolfram's comments on v7 fully addressed.

> 

> Changes since v7:

>  - Rebase and test on v4.12-rc2.

>  - Drop the dev_err() and hardware reset on timeout, which is not really

>    necessary.

>  - Return -ETIMEDOUT instead of -EIO on timeout.

>  - Unify zx2967_i2c_xfer_write_bytes() and zx2967_i2c_xfer_read_bytes()

>    into one function zx2967_i2c_xfer_bytes(), and zx2967_i2c_xfer_write()

>    and zx2967_i2c_xfer_read() into zx2967_i2c_xfer_msg().


Any comments on the series?  We hope it can get into v4.13 merge window.

Shawn
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html