]> git.ipfire.org Git - people/arne_f/kernel.git/commit
iio:light:ltr501 bug in parameter sanity check.
authorJonathan Cameron <jic23@kernel.org>
Sat, 2 May 2015 10:25:48 +0000 (11:25 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sun, 3 May 2015 18:59:25 +0000 (19:59 +0100)
commit2fdaf3f4f8c718a5023db69e2d391d978e94703e
tree1ea00d4054bf36830ba62f6a7a1ddc204b908bf2
parent6920ccf65d0964f7f6c6c36c551151e0fcd62327
iio:light:ltr501 bug in parameter sanity check.

Clearly the intent was to error if the value was not 0 or 1.
As implemented we have (A != 0 || A != 1) which is always true
as A is never both 0 and 1 at the same time.

As the autobuilder suggested, && makes more sense for this error
check.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Acked-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
Cc: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/light/ltr501.c