Re: [PATCH v3 00/12] tnetv107x ssp driver stack

From: Andrew Morton
Date: Thu Oct 21 2010 - 19:00:20 EST


On Thu, 21 Oct 2010 15:26:04 -0700
Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> wrote:

> Cyril Chemparathy <cyril@xxxxxx> writes:
>
> > TI's sequencer serial port (TI-SSP) is a jack-of-all-trades type of serial port
> > device. It has a built-in programmable execution engine that can be programmed
> > to operate as almost any serial bus (I2C, SPI, EasyScale, and others).
>
> Andrew, looking for some advice here...
>
> This is a piece of davinci hardware, but introduces drivers in various
> subsystems. I'm willing to merge this series via the davinci tree after
> getting acks from the various subsystem maintainers. Is this an OK
> approach? It seems best to me to merge this all together.

Yes, that's the best approach.

Some of the desired acks may not be forthcoming. In which case the
best you can do is to review the code yourself, make the maintainer(s)
aware of what's going on and just go ahead and merge it. If people are
too busy to look at the code then so be it - we can't really permit
that problem to permanently block new drivers.

> We already have acks for the regulator and gpio driver parts, and the
> backlight driver has a clear owner in MAINTAINERS. However, who should
> be doing the final review/ack of the drivers/misc and drivers/gpio
> changes is less clear to me.
>

I'll go and have a look at drivers/misc/ti_ssp.c.

>
> Thanks,
>
> Kevin
>
> > arch/arm/mach-davinci/board-tnetv107x-evm.c | 199 +++++++
> > arch/arm/mach-davinci/devices-tnetv107x.c | 25 +
> > arch/arm/mach-davinci/include/mach/ti_ssp.h | 98 ++++
> > arch/arm/mach-davinci/include/mach/tnetv107x.h | 2 +
> > arch/arm/mach-davinci/tnetv107x.c | 2 +-
> > drivers/gpio/Kconfig | 10 +
> > drivers/gpio/Makefile | 1 +
> > drivers/gpio/ti-ssp-gpio.c | 200 +++++++
> > drivers/misc/Kconfig | 11 +
> > drivers/misc/Makefile | 1 +
> > drivers/misc/ti_ssp.c | 436 +++++++++++++++
> > drivers/regulator/Kconfig | 10 +
> > drivers/regulator/Makefile | 1 +
> > drivers/regulator/tps6524x-regulator.c | 692 ++++++++++++++++++++++++
> > drivers/spi/Kconfig | 7 +
> > drivers/spi/Makefile | 1 +
> > drivers/spi/spi_ti_ssp.c | 397 ++++++++++++++
> > drivers/video/backlight/Kconfig | 7 +
> > drivers/video/backlight/Makefile | 2 +-
> > drivers/video/backlight/tps6116x.c | 340 ++++++++++++
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/