> Take a look at how the 'device_irq' local variable is used in > mv88e6xxx_g2_irq_setup. You assign it to 'err' in an error > path before it is ever set to anything. > > I think you meant to use the structure's 'device_irq' member > instead.
Hi David Agreed. Thanks for the review. Andrew