On Fri, May 15, 2015 at 09:06:55AM +0800, Qiao Zhou wrote: > >>@@ -360,8 +366,14 @@ static inline int pm80x_dev_resume(struct device *dev) > >> struct pm80x_chip *chip = dev_get_drvdata(pdev->dev.parent); > >> int irq = platform_get_irq(pdev, 0); > >> > >>+ if ((irq < 0) || (irq >= 24)) { > >>+ dev_err(dev, "pm80x: wrong irq 0x%x\n", irq); > >>+ > > > >Superfluous '\n'. > Will refine it. The \n is ok isn't it? regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html