public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 0/4] RDC321x southbridge and GPIO support
@ 2010-03-11  8:42 Florian Fainelli
  2010-03-14 12:44 ` Florian Fainelli
  2010-03-17 14:15 ` Wim Van Sebroeck
  0 siblings, 2 replies; 3+ messages in thread
From: Florian Fainelli @ 2010-03-11  8:42 UTC (permalink / raw)
  To: linux-kernel, Samuel Ortiz, Wim Van Sebroeck, Ingo Molnar

Hi Wim, Samuel, Ingo,

Here are 4 patches relative to adding support for the RDC321x southbridge.
This southbridge is present on all x86-based RDC321x System-on-a-Chip and
provides access to GPIOs and Wathchdog. The watchdog driver is already merged.

Samuel, this is rebased on top of your mfd-2.6 master branch after commit 
522dba7 and I think the 4 patches should go via your tree to make sure the 
merge is consistent.

Thank you.

Florian Fainelli (4):
  x86: remove rdc321x_defs.h
  MFD: add support for the RDC321x southbridge
  GPIO: add support for RDC321x GPIO controller
  WATCHDOG: convert rdc321x_wdt to use southbridge accessors


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2 0/4] RDC321x southbridge and GPIO support
  2010-03-11  8:42 [PATCH v2 0/4] RDC321x southbridge and GPIO support Florian Fainelli
@ 2010-03-14 12:44 ` Florian Fainelli
  2010-03-17 14:15 ` Wim Van Sebroeck
  1 sibling, 0 replies; 3+ messages in thread
From: Florian Fainelli @ 2010-03-14 12:44 UTC (permalink / raw)
  To: linux-kernel; +Cc: Samuel Ortiz, Wim Van Sebroeck, Ingo Molnar

Samuel,

Any comments on this? Thanks.

Le jeudi 11 mars 2010 09:42:03, Florian Fainelli a écrit :
> Hi Wim, Samuel, Ingo,
> 
> Here are 4 patches relative to adding support for the RDC321x southbridge.
> This southbridge is present on all x86-based RDC321x System-on-a-Chip and
> provides access to GPIOs and Wathchdog. The watchdog driver is already
>  merged.
> 
> Samuel, this is rebased on top of your mfd-2.6 master branch after commit
> 522dba7 and I think the 4 patches should go via your tree to make sure the
> merge is consistent.
> 
> Thank you.
> 
> Florian Fainelli (4):
>   x86: remove rdc321x_defs.h
>   MFD: add support for the RDC321x southbridge
>   GPIO: add support for RDC321x GPIO controller
>   WATCHDOG: convert rdc321x_wdt to use southbridge accessors
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2 0/4] RDC321x southbridge and GPIO support
  2010-03-11  8:42 [PATCH v2 0/4] RDC321x southbridge and GPIO support Florian Fainelli
  2010-03-14 12:44 ` Florian Fainelli
@ 2010-03-17 14:15 ` Wim Van Sebroeck
  1 sibling, 0 replies; 3+ messages in thread
From: Wim Van Sebroeck @ 2010-03-17 14:15 UTC (permalink / raw)
  To: Florian Fainelli; +Cc: linux-kernel, Samuel Ortiz, Ingo Molnar

Hi Florian,

> Here are 4 patches relative to adding support for the RDC321x southbridge.
> This southbridge is present on all x86-based RDC321x System-on-a-Chip and
> provides access to GPIOs and Wathchdog. The watchdog driver is already merged.
> 
> Samuel, this is rebased on top of your mfd-2.6 master branch after commit 
> 522dba7 and I think the 4 patches should go via your tree to make sure the 
> merge is consistent.
> 
> Thank you.
> 
> Florian Fainelli (4):
>   x86: remove rdc321x_defs.h
>   MFD: add support for the RDC321x southbridge
>   GPIO: add support for RDC321x GPIO controller
>   WATCHDOG: convert rdc321x_wdt to use southbridge accessors

Watchdog part is OK for me. So Acked-by me can be added.
Personally I don't like the fact that you remove the rdc321x_defs.h file as the first patch.
If someone bisects the kernel, he will get a broken driver between the first and the fourth patch.

Kind regards,
Wim.


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-03-17 14:15 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-11  8:42 [PATCH v2 0/4] RDC321x southbridge and GPIO support Florian Fainelli
2010-03-14 12:44 ` Florian Fainelli
2010-03-17 14:15 ` Wim Van Sebroeck

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox