On Tue, 2014-03-04 at 15:15 -0800, tip-bot for Thomas Gleixner wrote: > Commit-ID: 57310c3c99eb6fab2ecbd63aa3f7c323341ca77e > Gitweb: http://git.kernel.org/tip/57310c3c99eb6fab2ecbd63aa3f7c323341ca77e > Author: Thomas Gleixner <[email protected]> > AuthorDate: Wed, 5 Mar 2014 00:06:11 +0100 > Committer: Thomas Gleixner <[email protected]> > CommitDate: Wed, 5 Mar 2014 00:13:33 +0100 > > powerpc: eeh: Fixup the brown paperbag fallout of the "cleanup" > > Commit b8a9a11b9 (powerpc: eeh: Kill another abuse of irq_desc) is > missing some brackets ..... > > It's not a good idea to write patches in grumpy mode and then forget > to at least compile test them or rely on the few eyeballs discussing > that patch to spot it.....
Ouch :-) Next time you have a series like that, if you want I'll throw it at my build tester. Cheers, Ben. > Reported-by: [email protected] > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Peter Zijlstra <[email protected]> > Cc: Gavin Shan <[email protected]> > Cc: Benjamin Herrenschmidt <[email protected]> > Cc: ppc <[email protected]> > --- > arch/powerpc/kernel/eeh_driver.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/powerpc/kernel/eeh_driver.c > b/arch/powerpc/kernel/eeh_driver.c > index 3e1d7de..bb61ca5 100644 > --- a/arch/powerpc/kernel/eeh_driver.c > +++ b/arch/powerpc/kernel/eeh_driver.c > @@ -166,8 +166,9 @@ static void eeh_enable_irq(struct pci_dev *dev) > * > * tglx > */ > - if (irqd_irq_disabled(irq_get_irq_data(dev->irq)) > + if (irqd_irq_disabled(irq_get_irq_data(dev->irq))) > enable_irq(dev->irq); > + } > } > > /** -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

