]> git.openfabrics.org - ~emulex/infiniband.git/commit
staging:iio:lis3l02dq: Do not return a error in remove function
authorLars-Peter Clausen <lars@metafoo.de>
Sat, 22 Sep 2012 08:56:00 +0000 (09:56 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sat, 22 Sep 2012 09:20:11 +0000 (10:20 +0100)
commite71a837c810587e12620c957f1464c0674220d51
tree1676c717583c7611b8993f4f53288afb51409651
parente854bcc938e734e2f556bc7ada9552669b8f58b0
staging:iio:lis3l02dq: Do not return a error in remove function

In the Linux device driver model the remove callback is not allowed to fail and
the device will be removed regardless of the return value of the remove
callback. So if we abort in the remove function and do not free all resources we
will create a resource leak. Also all kinds of undefined behaviour are expected
to happen since the IIO device is still there while its parent is already gone.

The errors which the driver tries to handle in the remove function are
non-critical, so we can just ignore them and continue to free all resources and
remove the IIO device.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/accel/lis3l02dq_core.c