Re: [PATCH 1/6] powerpc/kconfig: Move NR_IRQS into "Kernel Options"

[Arnd Bergmann <arnd@arndb.de>]

On Monday 12 October 2015 22:07:45 Michael Ellerman wrote:
> Yeah, this builds and boots at least on pseries KVM.
> 
> diff --git a/arch/powerpc/include/asm/irq.h b/arch/powerpc/include/asm/irq.h
> index e8e3a0a04eb0..35fba282b7f9 100644
> --- a/arch/powerpc/include/asm/irq.h
> +++ b/arch/powerpc/include/asm/irq.h
> @@ -23,11 +23,8 @@ extern atomic_t ppc_n_lost_interrupts;
>  /* This number is used when no interrupt has been assigned */
>  #define NO_IRQ                 (0)
>  
> -/* Total number of virq in the platform */
> -#define NR_IRQS                CONFIG_NR_IRQS
> -
> -/* Same thing, used by the generic IRQ code */
>  #define NR_IRQS_LEGACY         NUM_ISA_INTERRUPTS
> +#define NR_IRQS                        NR_IRQS_LEGACY
>  
>  extern irq_hw_number_t virq_to_hw(unsigned int virq);
>  

This looks like the way it's intended. I've also checked that no powerpc
platform defines a static IRQ larger than NUM_ISA_INTERRUPTS, and only
two files do it at all (arch/powerpc/platforms/chrp/pegasos_eth.c and arch/powerpc/sysdev/rtc_cmos_setup.c).

	Arnd