Message ID | 20131001190344.GA3006@katana (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Anatolij Gustschin |
Headers | show |
On 10/01/2013 09:03 PM, Wolfram Sang wrote: > > Yup. But I just remembered a better solution: > > From: Wolfram Sang <wsa@the-dreams.de> Subject: [PATCH] ppc: > mpc52xx: silence false positive from old GCC > > So people can compile with -Werror. > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de> --- > arch/powerpc/platforms/52xx/mpc52xx_pic.c | 2 +- 1 file changed, > 1 insertion(+), 1 deletion(-) > > diff --git a/arch/powerpc/platforms/52xx/mpc52xx_pic.c > b/arch/powerpc/platforms/52xx/mpc52xx_pic.c index b89ef65..2898b73 > 100644 --- a/arch/powerpc/platforms/52xx/mpc52xx_pic.c +++ > b/arch/powerpc/platforms/52xx/mpc52xx_pic.c @@ -340,7 +340,7 @@ > static int mpc52xx_irqhost_map(struct irq_domain *h, unsigned int > virq, { int l1irq; int l2irq; - struct irq_chip *irqchip; + struct > irq_chip *uninitialized_var(irqchip); void *hndlr; int type; u32 > reg; > > > uninitialized_var was created for exactly that purpose IIRC. Yup, looks good, thanks. > > Thanks, > > Wolfram > Sebastian
diff --git a/arch/powerpc/platforms/52xx/mpc52xx_pic.c b/arch/powerpc/platforms/52xx/mpc52xx_pic.c index b89ef65..2898b73 100644 --- a/arch/powerpc/platforms/52xx/mpc52xx_pic.c +++ b/arch/powerpc/platforms/52xx/mpc52xx_pic.c @@ -340,7 +340,7 @@ static int mpc52xx_irqhost_map(struct irq_domain *h, unsigned int virq, { int l1irq; int l2irq; - struct irq_chip *irqchip; + struct irq_chip *uninitialized_var(irqchip); void *hndlr; int type; u32 reg;