Patchwork [GIT,PULL] imx: gpio interrupt cleanup

login
register
mail settings
Submitter Shawn Guo
Date Dec. 8, 2011, 2:32 p.m.
Message ID <20111208143204.GA8139@S2100-06.ap.freescale.net>
Download mbox
Permalink /patch/130183/
State New
Headers show

Pull-request

git://git.linaro.org/people/shawnguo/linux-2.6.git imx/clean-gpio-irq

Comments

Shawn Guo - Dec. 8, 2011, 2:32 p.m.
Hi Arnd and  Olof,

Please consider to pull the series below.  I really should have
Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
in gpio driver'.  So this pull request is also asking comments for the
path of these patches.

Thomas, Grant,

Please speak up if you do not want "your" patches to go through arm-soc
tree.  Thanks.

Shawn

The following changes since commit fec607aab5dca019cfc76768fa0b5908d47a55b1:

  Merge branch 'imx/fix-irqdomain' of git://git.linaro.org/people/shawnguo/linux-2.6 into fixes (2011-12-06 14:15:02 +0000)

are available in the git repository at:

  git://git.linaro.org/people/shawnguo/linux-2.6.git imx/clean-gpio-irq

Jamie Iles (1):
      irqdomain: export irq_domain_simple_ops for !CONFIG_OF

Shawn Guo (5):
      arm/imx: eliminate macro IMX_GPIO_TO_IRQ()
      arm/imx: eliminate macro IOMUX_TO_IRQ()
      arm/imx: eliminate macro IRQ_GPIOx()
      gpio/mxc: adopt irq_domain in gpio driver
      arm/imx: remove macro MXC_GPIO_IRQ_START

 arch/arm/mach-imx/eukrea_mbimx27-baseboard.c |    3 +-
 arch/arm/mach-imx/mach-apf9328.c             |    6 ++-
 arch/arm/mach-imx/mach-armadillo5x0.c        |   12 ++++--
 arch/arm/mach-imx/mach-cpuimx27.c            |   12 ++++--
 arch/arm/mach-imx/mach-cpuimx35.c            |    3 +-
 arch/arm/mach-imx/mach-imx27_visstrim_m10.c  |    8 ++--
 arch/arm/mach-imx/mach-imx6q.c               |   13 ------
 arch/arm/mach-imx/mach-kzm_arm11_01.c        |   20 +++++++--
 arch/arm/mach-imx/mach-mx21ads.c             |    5 +-
 arch/arm/mach-imx/mach-mx27_3ds.c            |    3 +-
 arch/arm/mach-imx/mach-mx27ads.c             |   12 +++---
 arch/arm/mach-imx/mach-mx31_3ds.c            |   13 +++---
 arch/arm/mach-imx/mach-mx31ads.c             |   14 ++++--
 arch/arm/mach-imx/mach-mx31lilly.c           |   10 +++-
 arch/arm/mach-imx/mach-mx31lite.c            |   10 +++-
 arch/arm/mach-imx/mach-mx31moboard.c         |    4 +-
 arch/arm/mach-imx/mach-mxt_td60.c            |    6 +-
 arch/arm/mach-imx/mach-pca100.c              |    4 +-
 arch/arm/mach-imx/mach-pcm037.c              |   18 +++++---
 arch/arm/mach-imx/mach-pcm038.c              |    4 +-
 arch/arm/mach-imx/mach-qong.c                |    9 ++--
 arch/arm/mach-imx/mach-scb9328.c             |    6 ++-
 arch/arm/mach-imx/mach-vpr200.c              |    3 +-
 arch/arm/mach-imx/mx31lilly-db.c             |    5 +-
 arch/arm/mach-imx/mx31lite-db.c              |    5 +-
 arch/arm/mach-imx/pcm970-baseboard.c         |   13 +++---
 arch/arm/mach-mx5/board-cpuimx51.c           |   12 ++++--
 arch/arm/mach-mx5/board-cpuimx51sd.c         |    6 ++-
 arch/arm/mach-mx5/board-mx53_ard.c           |    5 +-
 arch/arm/mach-mx5/eukrea_mbimx51-baseboard.c |    3 +-
 arch/arm/mach-mx5/imx51-dt.c                 |   12 -----
 arch/arm/mach-mx5/imx53-dt.c                 |   12 -----
 arch/arm/mach-mx5/mx51_efika.c               |    3 +-
 arch/arm/plat-mxc/include/mach/hardware.h    |    2 -
 arch/arm/plat-mxc/include/mach/iomux-mx3.h   |    3 -
 arch/arm/plat-mxc/include/mach/iomux-v1.h    |   12 +++---
 arch/arm/plat-mxc/include/mach/irqs.h        |    2 -
 drivers/gpio/Kconfig                         |    1 +
 drivers/gpio/gpio-mxc.c                      |   57 ++++++++++++++------------
 include/linux/irqdomain.h                    |    3 +-
 kernel/irq/irqdomain.c                       |   12 +++--
 41 files changed, 195 insertions(+), 171 deletions(-)
Shawn Guo - Dec. 12, 2011, 10:07 a.m.
Hi Arnd, Olof,

Please hold this request.  I'm trying to make this series cope with
the Rob's series adding irqdomain support for irq_chip_generic, since
both imx tzic and gpio interrupt controllers are irq_chip_generic users.

Regards,
Shawn

On Thu, Dec 08, 2011 at 10:32:05PM +0800, Shawn Guo wrote:
> Hi Arnd and  Olof,
> 
> Please consider to pull the series below.  I really should have
> Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> in gpio driver'.  So this pull request is also asking comments for the
> path of these patches.
> 
> Thomas, Grant,
> 
> Please speak up if you do not want "your" patches to go through arm-soc
> tree.  Thanks.
> 
> Shawn
> 
> The following changes since commit fec607aab5dca019cfc76768fa0b5908d47a55b1:
> 
>   Merge branch 'imx/fix-irqdomain' of git://git.linaro.org/people/shawnguo/linux-2.6 into fixes (2011-12-06 14:15:02 +0000)
> 
> are available in the git repository at:
> 
>   git://git.linaro.org/people/shawnguo/linux-2.6.git imx/clean-gpio-irq
> 
> Jamie Iles (1):
>       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> 
> Shawn Guo (5):
>       arm/imx: eliminate macro IMX_GPIO_TO_IRQ()
>       arm/imx: eliminate macro IOMUX_TO_IRQ()
>       arm/imx: eliminate macro IRQ_GPIOx()
>       gpio/mxc: adopt irq_domain in gpio driver
>       arm/imx: remove macro MXC_GPIO_IRQ_START
Stephen Warren - Dec. 19, 2011, 5:20 p.m.
Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> Hi Arnd and  Olof,
> 
> Please consider to pull the series below.  I really should have
> Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> in gpio driver'.  So this pull request is also asking comments for the
> path of these patches.
> 
> Thomas, Grant,
> 
> Please speak up if you do not want "your" patches to go through arm-soc
> tree.  Thanks.

Thomas,

I'm waiting on the following patch, since I need it to complete a patch
I'm working on:

> Jamie Iles (1):
>       irqdomain: export irq_domain_simple_ops for !CONFIG_OF

Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
it's OK to rely on this patch in my work?

Thanks.
Thomas Gleixner - Dec. 19, 2011, 5:22 p.m.
On Mon, 19 Dec 2011, Stephen Warren wrote:

> Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> > Hi Arnd and  Olof,
> > 
> > Please consider to pull the series below.  I really should have
> > Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> > for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> > in gpio driver'.  So this pull request is also asking comments for the
> > path of these patches.
> > 
> > Thomas, Grant,
> > 
> > Please speak up if you do not want "your" patches to go through arm-soc
> > tree.  Thanks.
> 
> Thomas,
> 
> I'm waiting on the following patch, since I need it to complete a patch
> I'm working on:
> 
> > Jamie Iles (1):
> >       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> 
> Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
> it's OK to rely on this patch in my work?

If there is nothing else depending on those patches than the arm soc
stuff, them they can go through it with my acked by. Otherwise let me
know, then I pick them up into a branch to pull for you.

Thanks,

	tglx
Jamie Iles - Dec. 19, 2011, 5:29 p.m.
Hi Stephen,

On Mon, Dec 19, 2011 at 09:20:34AM -0800, Stephen Warren wrote:
> Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> > Hi Arnd and  Olof,
> > 
> > Please consider to pull the series below.  I really should have
> > Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> > for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> > in gpio driver'.  So this pull request is also asking comments for the
> > path of these patches.
> > 
> > Thomas, Grant,
> > 
> > Please speak up if you do not want "your" patches to go through arm-soc
> > tree.  Thanks.
> 
> Thomas,
> 
> I'm waiting on the following patch, since I need it to complete a patch
> I'm working on:
> 
> > Jamie Iles (1):
> >       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> 
> Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
> it's OK to rely on this patch in my work?

Thomas has acked this and it's in Russell's tracker as 7235/1 as I have 
a VIC patch that uses it too.

Jamie
Stephen Warren - Dec. 19, 2011, 5:40 p.m.
Jamie Iles wrote at Monday, December 19, 2011 10:29 AM:
> On Mon, Dec 19, 2011 at 09:20:34AM -0800, Stephen Warren wrote:
> > Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> > > Hi Arnd and  Olof,
> > >
> > > Please consider to pull the series below.  I really should have
> > > Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> > > for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> > > in gpio driver'.  So this pull request is also asking comments for the
> > > path of these patches.
> > >
> > > Thomas, Grant,
> > >
> > > Please speak up if you do not want "your" patches to go through arm-soc
> > > tree.  Thanks.
> >
> > Thomas,
> >
> > I'm waiting on the following patch, since I need it to complete a patch
> > I'm working on:
> >
> > > Jamie Iles (1):
> > >       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> >
> > Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
> > it's OK to rely on this patch in my work?
> 
> Thomas has acked this and it's in Russell's tracker as 7235/1 as I have
> a VIC patch that uses it too.

Ah, I missed that.

Is it in a stable git tree anywhere, so that it can be used as a
dependency for my patch in the arm-soc tree? It certainly hasn't shown
up in today's next.

Thanks.
Jamie Iles - Dec. 19, 2011, 6:16 p.m.
On Mon, Dec 19, 2011 at 09:40:41AM -0800, Stephen Warren wrote:
> Jamie Iles wrote at Monday, December 19, 2011 10:29 AM:
> > On Mon, Dec 19, 2011 at 09:20:34AM -0800, Stephen Warren wrote:
> > > Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> > > > Hi Arnd and  Olof,
> > > >
> > > > Please consider to pull the series below.  I really should have
> > > > Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> > > > for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> > > > in gpio driver'.  So this pull request is also asking comments for the
> > > > path of these patches.
> > > >
> > > > Thomas, Grant,
> > > >
> > > > Please speak up if you do not want "your" patches to go through arm-soc
> > > > tree.  Thanks.
> > >
> > > Thomas,
> > >
> > > I'm waiting on the following patch, since I need it to complete a patch
> > > I'm working on:
> > >
> > > > Jamie Iles (1):
> > > >       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> > >
> > > Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
> > > it's OK to rely on this patch in my work?
> > 
> > Thomas has acked this and it's in Russell's tracker as 7235/1 as I have
> > a VIC patch that uses it too.
> 
> Ah, I missed that.
> 
> Is it in a stable git tree anywhere, so that it can be used as a
> dependency for my patch in the arm-soc tree? It certainly hasn't shown
> up in today's next.

No, I'm not aware of it having been merged anywhere yet.

Jamie
Stephen Warren - Dec. 19, 2011, 9:03 p.m.
Jamie Iles wrote at Monday, December 19, 2011 11:16 AM:
> On Mon, Dec 19, 2011 at 09:40:41AM -0800, Stephen Warren wrote:
> > Jamie Iles wrote at Monday, December 19, 2011 10:29 AM:
> > > On Mon, Dec 19, 2011 at 09:20:34AM -0800, Stephen Warren wrote:
> > > > Behalf Of Shawn Guo wrote at Thursday, December 08, 2011 7:32 AM:
> > > > > Hi Arnd and  Olof,
> > > > >
> > > > > Please consider to pull the series below.  I really should have
> > > > > Thomas' ack on Jamie's patch 'irqdomain: export irq_domain_simple_ops
> > > > > for !CONFIG_OF', and Grant's ack on patch 'gpio/mxc: adopt irq_domain
> > > > > in gpio driver'.  So this pull request is also asking comments for the
> > > > > path of these patches.
> > > > >
> > > > > Thomas, Grant,
> > > > >
> > > > > Please speak up if you do not want "your" patches to go through arm-soc
> > > > > tree.  Thanks.
> > > >
> > > > Thomas,
> > > >
> > > > I'm waiting on the following patch, since I need it to complete a patch
> > > > I'm working on:
> > > >
> > > > > Jamie Iles (1):
> > > > >       irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> > > >
> > > > Can you ack it for inclusion in the arm-soc tree, or apply it, so I know
> > > > it's OK to rely on this patch in my work?
> > >
> > > Thomas has acked this and it's in Russell's tracker as 7235/1 as I have
> > > a VIC patch that uses it too.
> >
> > Ah, I missed that.
> >
> > Is it in a stable git tree anywhere, so that it can be used as a
> > dependency for my patch in the arm-soc tree? It certainly hasn't shown
> > up in today's next.
> 
> No, I'm not aware of it having been merged anywhere yet.

Russell, do you know when this might be put into git? A few people depend
on this.

There's also a note in the patch tracker that says "For the devel / 
irqchip-consolidation branch". Is that branch stable and/or does it
feed into your devel-stable branch? Olof pointed out that devel-stable
would be the best place for it in terms of usability for an arm-soc
dependency.

Thanks.