From: Andrew Morton <akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
To: Kevin Hilman <khilman-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org,
dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org,
Cyril Chemparathy <cyril-l0cyMroinI0@public.gmane.org>,
spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
lrg-kDsPt+C1G03kYMGBc/C6ZA@public.gmane.org
Subject: Re: [PATCH v3 00/12] tnetv107x ssp driver stack
Date: Thu, 21 Oct 2010 15:59:02 -0700 [thread overview]
Message-ID: <20101021155902.b9702f72.akpm@linux-foundation.org> (raw)
In-Reply-To: <87mxq7p4hv.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
On Thu, 21 Oct 2010 15:26:04 -0700
Kevin Hilman <khilman-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org> wrote:
> Cyril Chemparathy <cyril-l0cyMroinI0@public.gmane.org> 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 ++++++++++++
------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
http://p.sf.net/sfu/nokia-dev2dev
prev parent reply other threads:[~2010-10-21 22:59 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-21 21:01 [PATCH v3 00/12] tnetv107x ssp driver stack Cyril Chemparathy
[not found] ` <1287694873-12904-1-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-10-21 21:01 ` [PATCH v3 01/12] misc: add driver for sequencer serial port Cyril Chemparathy
[not found] ` <1287694873-12904-2-git-send-email-cyril-l0cyMroinI0@public.gmane.org>
2010-10-21 23:12 ` Andrew Morton
[not found] ` <20101021161224.b4c0b623.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2010-10-22 12:39 ` Cyril Chemparathy
2010-10-22 12:48 ` Arnd Bergmann
[not found] ` <201010221448.17728.arnd-r2nGTMty4D4@public.gmane.org>
2010-10-22 19:33 ` Cyril Chemparathy
2010-10-22 22:46 ` Russell King - ARM Linux
2010-10-22 14:20 ` Alan Cox
2010-10-22 19:03 ` Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 02/12] davinci: add tnetv107x ssp platform device Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 03/12] davinci: add ssp config for tnetv107x evm board Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 04/12] spi: add ti-ssp spi master driver Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 05/12] davinci: add spi devices on tnetv107x evm Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 07/12] davinci: add tnetv107x evm regulators Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 08/12] gpio: add ti-ssp virtual gpio driver Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 09/12] davinci: add tnetv107x evm ti-ssp gpio device Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 10/12] backlight: add support for tps6116x controller Cyril Chemparathy
2010-10-21 21:19 ` Joe Perches
2010-10-21 21:01 ` [PATCH v3 11/12] davinci: add tnetv107x evm backlight device Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 12/12] davinci: add tnetv107x evm i2c eeprom device Cyril Chemparathy
2010-10-21 21:01 ` [PATCH v3 06/12] regulator: add driver for tps6524x regulator Cyril Chemparathy
2010-10-21 22:26 ` [PATCH v3 00/12] tnetv107x ssp driver stack Kevin Hilman
[not found] ` <87mxq7p4hv.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
2010-10-21 22:59 ` Andrew Morton [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20101021155902.b9702f72.akpm@linux-foundation.org \
--to=akpm-de/tnxtf+jlsfhdxvbkv3wd2fqjk+8+b@public.gmane.org \
--cc=broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org \
--cc=cyril-l0cyMroinI0@public.gmane.org \
--cc=davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org \
--cc=dbrownell-Rn4VEauK+AKRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
--cc=khilman-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=lrg-kDsPt+C1G03kYMGBc/C6ZA@public.gmane.org \
--cc=rpurdie-Fm38FmjxZ/leoWH0uzbU5w@public.gmane.org \
--cc=spi-devel-general-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).