mbox series

[0/4] Drop support for I2C 10 bit devices from I3C subsystem

Message ID 20190307095808.22540-1-pgaj@cadence.com (mailing list archive)
Headers show
Series Drop support for I2C 10 bit devices from I3C subsystem | expand

Message

Przemysław Gaj March 7, 2019, 9:58 a.m. UTC
I3C MIPI spec does not clearly say that I2C 10bit devices are supported.
Additionally, it isn't possible to pass 10 bit device properly through DEFSLVS
command and reserve address for such device on secondary master side.

Przemyslaw Gaj (4):
  i3c: Drop support for I2C 10 bit addresing
  i3c: master: cdns: Drop support for I2C 10 bit addresing in Cadence
    I3C master
  i3c: master: dw-i3c-master: Remove i2c_funcs hook implementation
  dt-bindings: i3c: Document dropped support for I2C 10 bit devices

 Documentation/devicetree/bindings/i3c/i3c.txt |  4 +++-
 drivers/i3c/master.c                          | 19 +++++++++++--------
 drivers/i3c/master/dw-i3c-master.c            |  6 ------
 drivers/i3c/master/i3c-master-cdns.c          | 10 +---------
 include/linux/i3c/master.h                    |  5 +----
 5 files changed, 16 insertions(+), 28 deletions(-)

Comments

Boris Brezillon March 11, 2019, 7:09 a.m. UTC | #1
Hi Przemek,

Missing v2 in the subject prefix, and there's no changelog in the cover
letter/patches.

On Thu, 7 Mar 2019 09:58:04 +0000
Przemyslaw Gaj <pgaj@cadence.com> wrote:

> I3C MIPI spec does not clearly say that I2C 10bit devices are supported.
> Additionally, it isn't possible to pass 10 bit device properly through DEFSLVS
> command and reserve address for such device on secondary master side.
> 
> Przemyslaw Gaj (4):
>   i3c: Drop support for I2C 10 bit addresing
>   i3c: master: cdns: Drop support for I2C 10 bit addresing in Cadence
>     I3C master
>   i3c: master: dw-i3c-master: Remove i2c_funcs hook implementation
>   dt-bindings: i3c: Document dropped support for I2C 10 bit devices

Looks like this series is still not bisectable. Please make sure things
compile/work correctly after each of your change.

Regards,

Boris

> 
>  Documentation/devicetree/bindings/i3c/i3c.txt |  4 +++-
>  drivers/i3c/master.c                          | 19 +++++++++++--------
>  drivers/i3c/master/dw-i3c-master.c            |  6 ------
>  drivers/i3c/master/i3c-master-cdns.c          | 10 +---------
>  include/linux/i3c/master.h                    |  5 +----
>  5 files changed, 16 insertions(+), 28 deletions(-)
>