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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20101021155902.b9702f72.akpm@linux-foundation.org>
Date:	Thu, 21 Oct 2010 15:59:02 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Kevin Hilman <khilman@...prootsystems.com>
Cc:	Cyril Chemparathy <cyril@...com>,
	davinci-linux-open-source@...ux.davincidsp.com,
	spi-devel-general@...ts.sourceforge.net,
	broonie@...nsource.wolfsonmicro.com, lrg@...mlogic.co.uk,
	dbrownell@...rs.sourceforge.net, grant.likely@...retlab.ca,
	linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
	rpurdie@...ys.net
Subject: Re: [PATCH v3 00/12] tnetv107x ssp driver stack

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

> Cyril Chemparathy <cyril@...com> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/