Re: [PATCH v14 0/3] i2c: npcm7xx: add NPCM i2c controller driver

Joel Stanley
2020-06-04T08:32:38Z

On Wed, 3 Jun 2020 at 20:26, Wolfram Sang <wsa@the-dreams.de> wrote:
>
> On Wed, May 27, 2020 at 11:08:17PM +0300, Tali Perry wrote:
> > This patch set adds i2c controller support
> > for the Nuvoton NPCM Baseboard Management Controller (BMC).
> >
> > NPCM7xx includes 16 I2C controllers. This driver operates the controller.
> > This module also includes a slave mode.
> >
> > ---
> > v14 -> v13:
> > - Fix yaml example: add missing include.
> > - Replace all udelay to usleep_range, except one which is called from
> > irq.
> > - Fix compilation error (module_platfrom_init conflict).
> > - debugfs counters always updated. Counting till max value,
> > then stop counting.
> > - Rename bus-frequency to clock-frequency.
> > - Remove unused variables.
>
> I don't have time for a deeper review, but from what I can tell this
> driver is good to go and we can fix things incrementally from now on.
>
> Applied to for-next (will go into 5.8), thanks!

Thanks Wolfram. I encourage this approach to working with patches, and
especially for our vendors who are trying to do the correct thing in
mainlining their code.

Congrats Tali on getting the driver in the tree. This has a been a long journey!

Cheers,

Joel