On Wed, 2010-07-21 at 11:22 +0200, ext-phil.2.carmody@xxxxxxxxx wrote: > A tiny tiny nit... > > From: Dmitry Torokhov [dmitry.torokhov@xxxxxxxxx] > ... > +void input_alloc_absinfo(struct input_dev *dev) > +{ > + if (!dev->absinfo) > + dev->absinfo = kcalloc(ABS_CNT, sizeof(struct input_absinfo), > + GFP_KERNEL); > + > + WARN(!dev->absinfo, "%s(): kzalloc() failed?\n", __func__); > > kcalloc failed, not kzalloc. kmalloc and friends already print a warning with a stack dump when they fail, unless this is overrided with __GFP_NOWARN, which is not the case here. So in usually we do not print messages/warnigns when we fail to allocate. -- Best Regards, Artem Bityutskiy (Артём Битюцкий) -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html