mbox

[00/14] GPIO: OMAP: first round of cleanup

Message ID 87r56trc8k.fsf@ti.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.1/gpio-cleanup

Message

Kevin Hilman June 16, 2011, 4:07 p.m. UTC
Grant Likely <grant.likely@secretlab.ca> writes:

> On Fri, May 20, 2011 at 05:14:43PM +0200, Kevin Hilman wrote:
>> Start moving SoC-specific register handling out of the driver by passing
>> in register offsets in via platform data.
>> 
>> Also, move OMAP1 MPUIO IRQ handling over to genric IRQ chip.
>> 
>> Applies on top of Tony's for-next branch (which include's the generic
>> IRQ chip work from tglx) and the OMAP GPIO driver move to
>> drivers/gpio.
>
> Hey Kevin,
>
> If you've got a branch with this series in it and tested, then I'd be
> happy to pull that instead of applying the patches myself.
>

OK, I rebased this on v3.0-rc2 and now it's been through quite a bit
more testing.

Here's a branch for you.

Also FYI, we'll probably have 2nd round of GPIO cleanups/updates and
runtime PM conversion for the upcoming merge window too.

Kevin

The following changes since commit 2c53b436a30867eb6b47dd7bab23ba638d1fb0d2:

  Linux 3.0-rc3 (2011-06-13 15:29:59 -0700)

are available in the git repository at:
  git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.1/gpio-cleanup

Charulatha V (1):
      GPIO: OMAP: remove get_gpio_bank()

Kevin Hilman (13):
      GPIO: OMAP: _clear_gpio_irqbank: fix flushing of posted write
      GPIO: OMAP: remove MPUIO handling from _clear_gpio_irqbank()
      GPIO: OMAP: move bank width into struct gpio_bank
      GPIO: OMAP: _get_gpio_irqbank_mask: replace hard-coded mask with bank->width
      GPIO: OMAP: replace get_gpio_index() by using bank width
      GPIO: OMAP: consolidate direction, input, output, remove #ifdefs
      GPIO: OMAP: consolidate IRQ status handling, remove #ifdefs
      GPIO: OMAP: conslidate enable/disable of GPIO IRQs, remove ifdefs
      GPIO: OMAP: convert MPUIO IRQ over to generic irq_chip
      GPIO: OMAP: remove useless gpio_valid() & check_gpio() checks
      GPIO: OMAP: cleanup _set_gpio_wakeup(), remove ifdefs
      GPIO: OMAP: debounce remove SoC specific registers, use pdata
      GPIO: OMAP: cleanup show revision, remove cpu_is checks, display only once

 arch/arm/mach-omap1/gpio15xx.c         |   22 +
 arch/arm/mach-omap1/gpio16xx.c         |   28 ++
 arch/arm/mach-omap1/gpio7xx.c          |   27 ++
 arch/arm/mach-omap2/gpio.c             |   32 ++
 arch/arm/plat-omap/include/plat/gpio.h |   20 +
 drivers/gpio/gpio-omap.c               |  723 +++++++-------------------------
 6 files changed, 282 insertions(+), 570 deletions(-)

Comments

Grant Likely June 16, 2011, 4:26 p.m. UTC | #1
On Thu, Jun 16, 2011 at 09:07:39AM -0700, Kevin Hilman wrote:
> Grant Likely <grant.likely@secretlab.ca> writes:
> 
> > On Fri, May 20, 2011 at 05:14:43PM +0200, Kevin Hilman wrote:
> >> Start moving SoC-specific register handling out of the driver by passing
> >> in register offsets in via platform data.
> >> 
> >> Also, move OMAP1 MPUIO IRQ handling over to genric IRQ chip.
> >> 
> >> Applies on top of Tony's for-next branch (which include's the generic
> >> IRQ chip work from tglx) and the OMAP GPIO driver move to
> >> drivers/gpio.
> >
> > Hey Kevin,
> >
> > If you've got a branch with this series in it and tested, then I'd be
> > happy to pull that instead of applying the patches myself.
> >
> 
> OK, I rebased this on v3.0-rc2 and now it's been through quite a bit
> more testing.
> 
> Here's a branch for you.

Thanks.  A minor comment though...

> 
> Also FYI, we'll probably have 2nd round of GPIO cleanups/updates and
> runtime PM conversion for the upcoming merge window too.

Okay.

> 
> Kevin
> 
> The following changes since commit 2c53b436a30867eb6b47dd7bab23ba638d1fb0d2:
> 
>   Linux 3.0-rc3 (2011-06-13 15:29:59 -0700)
> 
> are available in the git repository at:
>   git://git.kernel.org/pub/scm/linux/kernel/git/khilman/linux-omap-pm.git for_3.1/gpio-cleanup
> 
> Charulatha V (1):
>       GPIO: OMAP: remove get_gpio_bank()
> 
> Kevin Hilman (13):
>       GPIO: OMAP: _clear_gpio_irqbank: fix flushing of posted write
>       GPIO: OMAP: remove MPUIO handling from _clear_gpio_irqbank()
>       GPIO: OMAP: move bank width into struct gpio_bank
>       GPIO: OMAP: _get_gpio_irqbank_mask: replace hard-coded mask with bank->width
>       GPIO: OMAP: replace get_gpio_index() by using bank width
>       GPIO: OMAP: consolidate direction, input, output, remove #ifdefs
>       GPIO: OMAP: consolidate IRQ status handling, remove #ifdefs
>       GPIO: OMAP: conslidate enable/disable of GPIO IRQs, remove ifdefs
>       GPIO: OMAP: convert MPUIO IRQ over to generic irq_chip
>       GPIO: OMAP: remove useless gpio_valid() & check_gpio() checks
>       GPIO: OMAP: cleanup _set_gpio_wakeup(), remove ifdefs
>       GPIO: OMAP: debounce remove SoC specific registers, use pdata
>       GPIO: OMAP: cleanup show revision, remove cpu_is checks, display only once

Use this form for the patch summaries please: "gpio/omap: ...."

It's better for the pull req to Linus when all the gpio patches look the same.

Thanks,
g.