diff options
author | Jonathan Cameron <jic23@kernel.org> | 2015-05-02 11:25:48 +0100 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2015-05-03 19:59:25 +0100 |
commit | 2fdaf3f4f8c718a5023db69e2d391d978e94703e (patch) | |
tree | 1ea00d4054bf36830ba62f6a7a1ddc204b908bf2 /drivers/staging/iio | |
parent | 6920ccf65d0964f7f6c6c36c551151e0fcd62327 (diff) | |
download | op-kernel-dev-2fdaf3f4f8c718a5023db69e2d391d978e94703e.zip op-kernel-dev-2fdaf3f4f8c718a5023db69e2d391d978e94703e.tar.gz |
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>
Diffstat (limited to 'drivers/staging/iio')
0 files changed, 0 insertions, 0 deletions