[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

I have lost count how many times I have sent
these patches. Will someone either take them
in or NAK them so I stop sending ??

Felipe Balbi (2):
  arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
  arm: Kconfig: allow OMAP to use sparse IRQ numbering

 arch/arm/Kconfig |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

-- 
1.7.6.rc1

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

I have lost count how many times I have sent
these patches. Will someone either take them
in or NAK them so I stop sending ??

Felipe Balbi (2):
  arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
  arm: Kconfig: allow OMAP to use sparse IRQ numbering

 arch/arm/Kconfig |    5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

-- 
1.7.6.rc1

[RESEND PATCH 1/2] arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol

[Felipe Balbi]

commit e2a93ecc7fc469dab52323c11a2d8ceaa62aac7b
(ARM: Use genirq definitions from kernel/irq/Kconfig)
made ARM Kconfig use the generic Kconfig symbols from
kernel/irq/Kconfig but did not fix the boards which
were using the older symbols, fix them.

Signed-off-by: Felipe Balbi <balbi@ti.com>
---
 arch/arm/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 9adc278..a061525 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -545,7 +545,7 @@ config ARCH_MMP
 	select HAVE_SCHED_CLOCK
 	select TICK_ONESHOT
 	select PLAT_PXA
-	select SPARSE_IRQ
+	select HAVE_SPARSE_IRQ
 	help
 	  Support for Marvell's PXA168/PXA910(MMP) and MMP2 processor line.
 
@@ -617,7 +617,7 @@ config ARCH_PXA
 	select HAVE_SCHED_CLOCK
 	select TICK_ONESHOT
 	select PLAT_PXA
-	select SPARSE_IRQ
+	select HAVE_SPARSE_IRQ
 	help
 	  Support for Intel/Marvell's PXA2xx/PXA3xx processor line.
 
-- 
1.7.6.rc1

[RESEND PATCH 1/2] arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol

[Felipe Balbi]

commit e2a93ecc7fc469dab52323c11a2d8ceaa62aac7b
(ARM: Use genirq definitions from kernel/irq/Kconfig)
made ARM Kconfig use the generic Kconfig symbols from
kernel/irq/Kconfig but did not fix the boards which
were using the older symbols, fix them.

Signed-off-by: Felipe Balbi <balbi@ti.com>
---
 arch/arm/Kconfig |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 9adc278..a061525 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -545,7 +545,7 @@ config ARCH_MMP
 	select HAVE_SCHED_CLOCK
 	select TICK_ONESHOT
 	select PLAT_PXA
-	select SPARSE_IRQ
+	select HAVE_SPARSE_IRQ
 	help
 	  Support for Marvell's PXA168/PXA910(MMP) and MMP2 processor line.
 
@@ -617,7 +617,7 @@ config ARCH_PXA
 	select HAVE_SCHED_CLOCK
 	select TICK_ONESHOT
 	select PLAT_PXA
-	select SPARSE_IRQ
+	select HAVE_SPARSE_IRQ
 	help
 	  Support for Intel/Marvell's PXA2xx/PXA3xx processor line.
 
-- 
1.7.6.rc1

[RESEND PATCH 2/2] arm: Kconfig: allow OMAP to use sparse IRQ numbering

[Felipe Balbi]

Select HAVE_SPARSE_IRQ to allow OMAP to use
sparse IRQ numbering scheme. The main difference
is that irq_descs will be added to a radix tree
instead of a static array.

Signed-off-by: Felipe Balbi <balbi@ti.com>
---
 arch/arm/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index a061525..bdb6244 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -850,6 +850,7 @@ config ARCH_DAVINCI
 config ARCH_OMAP
 	bool "TI OMAP"
 	select HAVE_CLK
+	select HAVE_SPARSE_IRQ
 	select ARCH_REQUIRE_GPIOLIB
 	select ARCH_HAS_CPUFREQ
 	select GENERIC_CLOCKEVENTS
-- 
1.7.6.rc1

[RESEND PATCH 2/2] arm: Kconfig: allow OMAP to use sparse IRQ numbering

[Felipe Balbi]

Select HAVE_SPARSE_IRQ to allow OMAP to use
sparse IRQ numbering scheme. The main difference
is that irq_descs will be added to a radix tree
instead of a static array.

Signed-off-by: Felipe Balbi <balbi@ti.com>
---
 arch/arm/Kconfig |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index a061525..bdb6244 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -850,6 +850,7 @@ config ARCH_DAVINCI
 config ARCH_OMAP
 	bool "TI OMAP"
 	select HAVE_CLK
+	select HAVE_SPARSE_IRQ
 	select ARCH_REQUIRE_GPIOLIB
 	select ARCH_HAS_CPUFREQ
 	select GENERIC_CLOCKEVENTS
-- 
1.7.6.rc1

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Russell King - ARM Linux]

On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??

It might help to copy the MMP and PXA people so that we get to hear
their side of the story.  It may be that they always require sparse
IRQ support to be enabled.

'cause at the moment I've no idea whether your patch is right or not.

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Russell King - ARM Linux]

On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??

It might help to copy the MMP and PXA people so that we get to hear
their side of the story.  It may be that they always require sparse
IRQ support to be enabled.

'cause at the moment I've no idea whether your patch is right or not.

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 596 bytes --]

On Wed, Jun 15, 2011 at 04:14:22PM +0100, Russell King - ARM Linux wrote:
> On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> > I have lost count how many times I have sent
> > these patches. Will someone either take them
> > in or NAK them so I stop sending ??
> 
> It might help to copy the MMP and PXA people so that we get to hear
> their side of the story.  It may be that they always require sparse
> IRQ support to be enabled.
> 
> 'cause at the moment I've no idea whether your patch is right or not.

I'll do so... thanks, at least one reply :-D

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

On Wed, Jun 15, 2011 at 04:14:22PM +0100, Russell King - ARM Linux wrote:
> On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> > I have lost count how many times I have sent
> > these patches. Will someone either take them
> > in or NAK them so I stop sending ??
> 
> It might help to copy the MMP and PXA people so that we get to hear
> their side of the story.  It may be that they always require sparse
> IRQ support to be enabled.
> 
> 'cause at the moment I've no idea whether your patch is right or not.

I'll do so... thanks, at least one reply :-D

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20110615/d868a602/attachment.sig>

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Kevin Hilman]

Felipe Balbi <balbi@ti.com> writes:

> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??
>
> Felipe Balbi (2):
>   arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>   arm: Kconfig: allow OMAP to use sparse IRQ numbering
>

Tested-by: Kevin Hilman <khilman@ti.com>

Looks good to me.  Built with CONFIG_SPARSE_IRQ=y and boot tested on
OMAP3430/n900 & OMAP4430/Blaze.

Kevin

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Kevin Hilman]

Felipe Balbi <balbi@ti.com> writes:

> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??
>
> Felipe Balbi (2):
>   arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>   arm: Kconfig: allow OMAP to use sparse IRQ numbering
>

Tested-by: Kevin Hilman <khilman@ti.com>

Looks good to me.  Built with CONFIG_SPARSE_IRQ=y and boot tested on
OMAP3430/n900 & OMAP4430/Blaze.

Kevin

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

[-- Attachment #1: Type: text/plain, Size: 818 bytes --]

Hi,

On Wed, Jun 15, 2011 at 06:24:45PM +0300, Felipe Balbi wrote:
> On Wed, Jun 15, 2011 at 04:14:22PM +0100, Russell King - ARM Linux wrote:
> > On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> > > I have lost count how many times I have sent
> > > these patches. Will someone either take them
> > > in or NAK them so I stop sending ??
> > 
> > It might help to copy the MMP and PXA people so that we get to hear
> > their side of the story.  It may be that they always require sparse
> > IRQ support to be enabled.
> > 
> > 'cause at the moment I've no idea whether your patch is right or not.
> 
> I'll do so... thanks, at least one reply :-D

seems like these aren't needed anymore. HAVE_SPARSE_IRQ is selected for
the entire ARM architecture. Sorry for the noise.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 490 bytes --]

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Felipe Balbi]

Hi,

On Wed, Jun 15, 2011 at 06:24:45PM +0300, Felipe Balbi wrote:
> On Wed, Jun 15, 2011 at 04:14:22PM +0100, Russell King - ARM Linux wrote:
> > On Wed, Jun 15, 2011 at 04:02:38PM +0300, Felipe Balbi wrote:
> > > I have lost count how many times I have sent
> > > these patches. Will someone either take them
> > > in or NAK them so I stop sending ??
> > 
> > It might help to copy the MMP and PXA people so that we get to hear
> > their side of the story.  It may be that they always require sparse
> > IRQ support to be enabled.
> > 
> > 'cause at the moment I've no idea whether your patch is right or not.
> 
> I'll do so... thanks, at least one reply :-D

seems like these aren't needed anymore. HAVE_SPARSE_IRQ is selected for
the entire ARM architecture. Sorry for the noise.

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20110616/392e47ae/attachment.sig>

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Sergei Shtylyov]

Hello.

On 15-06-2011 17:02, Felipe Balbi wrote:

> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??

> Felipe Balbi (2):
>    arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>    arm: Kconfig: allow OMAP to use sparse IRQ numbering

    Why not put it into Russell's patch system already?

WBR, Sergei

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Sergei Shtylyov]

Hello.

On 15-06-2011 17:02, Felipe Balbi wrote:

> I have lost count how many times I have sent
> these patches. Will someone either take them
> in or NAK them so I stop sending ??

> Felipe Balbi (2):
>    arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>    arm: Kconfig: allow OMAP to use sparse IRQ numbering

    Why not put it into Russell's patch system already?

WBR, Sergei

Re: [RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Russell King - ARM Linux]

On Thu, Jun 16, 2011 at 03:49:05PM +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 15-06-2011 17:02, Felipe Balbi wrote:
>
>> I have lost count how many times I have sent
>> these patches. Will someone either take them
>> in or NAK them so I stop sending ??
>
>> Felipe Balbi (2):
>>    arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>>    arm: Kconfig: allow OMAP to use sparse IRQ numbering
>
>    Why not put it into Russell's patch system already?

I would like _someone_ (such as those who introduced the ARM arch to
the Sparse IRQ stuff) at least to ack them first so I don't have to
wonder whether the patches are correct or not.

[RESEND PATCH 0/2] ARM IRQ Kconfig patches

[Russell King - ARM Linux]

On Thu, Jun 16, 2011 at 03:49:05PM +0400, Sergei Shtylyov wrote:
> Hello.
>
> On 15-06-2011 17:02, Felipe Balbi wrote:
>
>> I have lost count how many times I have sent
>> these patches. Will someone either take them
>> in or NAK them so I stop sending ??
>
>> Felipe Balbi (2):
>>    arm: Kconfig: select correct HAVE_SPARSE_IRQ symbol
>>    arm: Kconfig: allow OMAP to use sparse IRQ numbering
>
>    Why not put it into Russell's patch system already?

I would like _someone_ (such as those who introduced the ARM arch to
the Sparse IRQ stuff) at least to ack them first so I don't have to
wonder whether the patches are correct or not.