Re: [PATCH 1/2] i2c: i2c-riic: add support for Renesas RIIC

Jean Delvare
2011-07-11T06:44:17Z

Hi Paul,

On Mon, 11 Jul 2011 15:07:06 +0900, Paul Mundt wrote:
> On Mon, Jul 11, 2011 at 08:03:35AM +0200, Jean Delvare wrote:
> > On Mon, 11 Jul 2011 14:52:21 +0900, Paul Mundt wrote:
> > > On Fri, Jul 01, 2011 at 10:00:42AM +0900, Yoshihiro Shimoda wrote:
> > > > This driver supports the RIIC module. The SH7757 has it.
> > > > The driver doesn't use any IRQ handler.
> > > >
> > > > Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> > > > ---
> > > > drivers/i2c/busses/Kconfig | 9 +
> > > > drivers/i2c/busses/Makefile | 1 +
> > > > drivers/i2c/busses/i2c-riic.c | 589 +++++++++++++++++++++++++++++++++++++++++
> > > > include/linux/i2c/riic.h | 29 ++
> > > > 4 files changed, 628 insertions(+), 0 deletions(-)
> > > > create mode 100644 drivers/i2c/busses/i2c-riic.c
> > > > create mode 100644 include/linux/i2c/riic.h
> > >
> > > Ping?
> >
> > Embedded platform -> ping Ben, not me.
>
> It was addressed to Ben and there has been no response. Ben has also been
> included in the Cc for the ping in question. If there continues to be no
> response, how exactly do you propose making progress on this?

You could do a public review of this patch, for a start. Ben would
certainly be more willing to pick it if he knows that it has already
been reviewed by a well-known kernel developer so the most obvious
issues have already been addressed.

--
Jean Delvare