Message ID | 1366796211-7783-1-git-send-email-luis.henriques@canonical.com |
---|---|
State | New |
Headers | show |
On 04/24/2013 11:36 AM, Luis Henriques wrote: > This is a note to let you know that I have just added a patch titled > > ARM: S3C24XX: Correct NR_IRQS definition for s3c2440 > > to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree > which can be found at: > > http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/linux-3.5.y-queue > > If you, or anyone else, feels it should not be added to this tree, please > reply to this email. I think this patch is not even needed in 3.9, only 3.10-rc1+. Heiko, can you confirm ? I've tested 3.9-rc1 on s3c2440 SoC based board and all interrupts are registered properly without this patch. I believe this patch is needed only for trees containing patch series from Heiko [1], starting with commit: 6f8d7ea275eb2a27fd62211e93921a82f367f939 ARM: S3C24XX: move s3c244x irq init to common irq code [1] https://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git/log/arch/arm/mach-s3c24xx?h=for-next Regards, Sylwester
Am Freitag, 26. April 2013, 12:06:29 schrieb Sylwester Nawrocki: > On 04/24/2013 11:36 AM, Luis Henriques wrote: > > This is a note to let you know that I have just added a patch titled > > > > ARM: S3C24XX: Correct NR_IRQS definition for s3c2440 > > > > to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree > > > > which can be found at: > > http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/ > > linux-3.5.y-queue > > > > If you, or anyone else, feels it should not be added to this tree, please > > reply to this email. > > I think this patch is not even needed in 3.9, only 3.10-rc1+. > > Heiko, can you confirm ? Correct, including the patch in 3.5 should not be necessary, as the interrupt handling was still untouched then. > I've tested 3.9-rc1 on s3c2440 SoC based board and all interrupts are > registered properly without this patch. Very strange ... While the real rework of the s3c244x into the new irq structure will only enter mainline during the next merge window, the irq code in 3.9 already uses full irq domains. s3c24xx_init_irq which is still called at this state from the s3c244x boards, is creating the base irq-domains with default values (and the full 29 irqs in the subint domain) and as before the s3c244x irq initcall does then replace the demuxed interrupts So in theory 3.9 should not work with the previous NR_IRQS value (and rightfully got you fix applied [2] ... or I'm overlooking something ;-) > I believe this patch is needed only for trees containing patch series > from Heiko [1], starting with commit: > > 6f8d7ea275eb2a27fd62211e93921a82f367f939 > ARM: S3C24XX: move s3c244x irq init to common irq code > > [1] > https://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git/log/arch/arm/m > ach-s3c24xx?h=for-next Heiko [2] https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/mach- s3c24xx?id=b530f742ac27460d41d35b638ad6aad92044a982
On Fri, Apr 26, 2013 at 12:33:33PM +0200, Heiko Stübner wrote: > Am Freitag, 26. April 2013, 12:06:29 schrieb Sylwester Nawrocki: > > On 04/24/2013 11:36 AM, Luis Henriques wrote: > > > This is a note to let you know that I have just added a patch titled > > > > > > ARM: S3C24XX: Correct NR_IRQS definition for s3c2440 > > > > > > to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree > > > > > > which can be found at: > > > http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/ > > > linux-3.5.y-queue > > > > > > If you, or anyone else, feels it should not be added to this tree, please > > > reply to this email. > > > > I think this patch is not even needed in 3.9, only 3.10-rc1+. > > > > Heiko, can you confirm ? > > Correct, including the patch in 3.5 should not be necessary, as the interrupt > handling was still untouched then. > Great, thanks for your feedback. I've picked this patch after a request in the stable mailing list to include it in a stable kernel (3.8, if I remember correctly). Anyway, I'll drop it from the 3.5.y queue. Cheers, -- Luis > > > I've tested 3.9-rc1 on s3c2440 SoC based board and all interrupts are > > registered properly without this patch. > > Very strange ... > > While the real rework of the s3c244x into the new irq structure will only > enter mainline during the next merge window, the irq code in 3.9 already uses > full irq domains. > > s3c24xx_init_irq which is still called at this state from the s3c244x boards, > is creating the base irq-domains with default values (and the full 29 irqs in > the subint domain) and as before the s3c244x irq initcall does then replace > the demuxed interrupts > > So in theory 3.9 should not work with the previous NR_IRQS value (and > rightfully got you fix applied [2] ... or I'm overlooking something ;-) > > > > I believe this patch is needed only for trees containing patch series > > from Heiko [1], starting with commit: > > > > 6f8d7ea275eb2a27fd62211e93921a82f367f939 > > ARM: S3C24XX: move s3c244x irq init to common irq code > > > > [1] > > https://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git/log/arch/arm/m > > ach-s3c24xx?h=for-next > > > Heiko > > [2] > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/mach- > s3c24xx?id=b530f742ac27460d41d35b638ad6aad92044a982
On 04/26/2013 12:33 PM, Heiko Stübner wrote: > Am Freitag, 26. April 2013, 12:06:29 schrieb Sylwester Nawrocki: >> On 04/24/2013 11:36 AM, Luis Henriques wrote: >>> This is a note to let you know that I have just added a patch titled >>> >>> ARM: S3C24XX: Correct NR_IRQS definition for s3c2440 >>> >>> to the linux-3.5.y-queue branch of the 3.5.y.z extended stable tree >>> >>> which can be found at: >>> http://kernel.ubuntu.com/git?p=ubuntu/linux.git;a=shortlog;h=refs/heads/ >>> linux-3.5.y-queue >>> >>> If you, or anyone else, feels it should not be added to this tree, please >>> reply to this email. >> >> I think this patch is not even needed in 3.9, only 3.10-rc1+. >> >> Heiko, can you confirm ? > > Correct, including the patch in 3.5 should not be necessary, as the interrupt > handling was still untouched then. > > >> I've tested 3.9-rc1 on s3c2440 SoC based board and all interrupts are >> registered properly without this patch. > > Very strange ... > > While the real rework of the s3c244x into the new irq structure will only > enter mainline during the next merge window, the irq code in 3.9 already uses > full irq domains. > > s3c24xx_init_irq which is still called at this state from the s3c244x boards, > is creating the base irq-domains with default values (and the full 29 irqs in > the subint domain) and as before the s3c244x irq initcall does then replace > the demuxed interrupts > > So in theory 3.9 should not work with the previous NR_IRQS value (and > rightfully got you fix applied [2] ... or I'm overlooking something ;-) Yes, it's already applied. Let me test it again this week. It was late night when I tried it with 3.9-rc1 and I might have mixed something up. >> I believe this patch is needed only for trees containing patch series >> from Heiko [1], starting with commit: >> >> 6f8d7ea275eb2a27fd62211e93921a82f367f939 >> ARM: S3C24XX: move s3c244x irq init to common irq code >> >> [1] >> https://git.kernel.org/cgit/linux/kernel/git/arm/arm-soc.git/log/arch/arm/m >> ach-s3c24xx?h=for-next > > > Heiko > > [2] > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/arch/arm/mach- > s3c24xx?id=b530f742ac27460d41d35b638ad6aad92044a982 Thanks, Sylwester
diff --git a/arch/arm/mach-s3c24xx/include/mach/irqs.h b/arch/arm/mach-s3c24xx/include/mach/irqs.h index b7a9f4d..1e73f5f 100644 --- a/arch/arm/mach-s3c24xx/include/mach/irqs.h +++ b/arch/arm/mach-s3c24xx/include/mach/irqs.h @@ -188,10 +188,8 @@ #if defined(CONFIG_CPU_S3C2416) #define NR_IRQS (IRQ_S3C2416_I2S1 + 1) -#elif defined(CONFIG_CPU_S3C2443) -#define NR_IRQS (IRQ_S3C2443_AC97+1) #else -#define NR_IRQS (IRQ_S3C2440_AC97+1) +#define NR_IRQS (IRQ_S3C2443_AC97 + 1) #endif /* compatibility define. */