Re: [PATCH v1 01/11] i2c: Introduce common module to instantiate CCGx UCSI

From: Andy Shevchenko
Date: Mon Dec 13 2021 - 13:05:40 EST


On Wed, Dec 08, 2021 at 02:29:04PM +0200, Jarkko Nikula wrote:
> On 12/7/21 21:21, Andy Shevchenko wrote:
> > Introduce a common module to provide an API to instantiate UCSI device
> > for Cypress CCGx Type-C controller. Individual bus drivers need to select
> > this one on demand.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > ---
> > drivers/i2c/busses/Kconfig | 7 +++++++
> > drivers/i2c/busses/Makefile | 3 +++
> > drivers/i2c/busses/i2c-ccgx-ucsi.c | 27 +++++++++++++++++++++++++++
> > drivers/i2c/busses/i2c-ccgx-ucsi.h | 11 +++++++++++
> > 4 files changed, 48 insertions(+)
> > create mode 100644 drivers/i2c/busses/i2c-ccgx-ucsi.c
> > create mode 100644 drivers/i2c/busses/i2c-ccgx-ucsi.h
> >
> I've mixed feelings about this set. I'd either put patches 3-8 first since
> e.g. 6/11 and 8/11 are fixing existing issues or even better to split CCGx
> UCSI stuff into another set.

I have sent v2 with DesignWare patches only and no conversion part included.

--
With Best Regards,
Andy Shevchenko