On Mon, May 13, 2013 at 01:43:17PM +0200, Laurent Navet wrote: > The same is done almost everywhere in kernel tree : > > e.g : > http://lxr.free-electrons.com/source/drivers/staging/iio/adc/mxs-lradc.c#L928 > http://lxr.free-electrons.com/source/drivers/gpio/gpio-mxs.c#L292 > http://lxr.free-electrons.com/source/drivers/iommu/tegra-smmu.c#L1181 > http://lxr.free-electrons.com/source/drivers/rtc/rtc-spear.c#L388 > http://lxr.free-electrons.com/source/drivers/i2c/busses/i2c-imx.c#L514 > http://lxr.free-electrons.com/source/sound/soc/kirkwood/kirkwood-i2s.c#L475 > > and so on... all theses functions return an int. I was confusing PTR_ERR() and ERR_PTR(), so path looks ok. Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html