Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 13 Jun 2023 12:35:12 +0300
From:   Matti Vaittinen <mazziesaccount@gmail.com>
To:     Matti Vaittinen <mazziesaccount@gmail.com>,
        Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Cc:     Matti Vaittinen <mazziesaccount@gmail.com>,
        Jonathan Cameron <jic23@kernel.org>,
        Lars-Peter Clausen <lars@metafoo.de>,
        linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH 3/3] iio: light: bu27008: Fix intensity data type
Message-ID: <240a7ca5fc1b76da20d81f930d00f31a54b1fdf8.1686648422.git.mazziesaccount@gmail.com>
References: <cover.1686648422.git.mazziesaccount@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="VHU0M3xPK1b1nyOO"
Content-Disposition: inline
In-Reply-To: <cover.1686648422.git.mazziesaccount@gmail.com>
Precedence: bulk


--VHU0M3xPK1b1nyOO
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

The intensity data from bu27008 is unsigned. The type of the scan data
was incorrectly marked as signed resulting large intensity values to be
interpreted as negative ones.

Fix the scan data type.

Signed-off-by: Matti Vaittinen <mazziesaccount@gmail.com>

---
Fixes tag not added because AFACS the bu27008 is not yet in any releases.
---
 drivers/iio/light/rohm-bu27008.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/light/rohm-bu27008.c b/drivers/iio/light/rohm-bu27=
008.c
index 80eb14ea8193..b50bf8973d9a 100644
--- a/drivers/iio/light/rohm-bu27008.c
+++ b/drivers/iio/light/rohm-bu27008.c
@@ -190,7 +190,7 @@ static const struct iio_itime_sel_mul bu27008_itimes[] =
=3D {
 	.address =3D BU27008_REG_##data##_LO,					\
 	.scan_index =3D BU27008_##color,						\
 	.scan_type =3D {								\
-		.sign =3D 's',							\
+		.sign =3D 'u',							\
 		.realbits =3D 16,							\
 		.storagebits =3D 16,						\
 		.endianness =3D IIO_LE,						\
--=20
2.40.1


--=20
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =3D]=20

--VHU0M3xPK1b1nyOO
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEEIx+f8wZb28fLKEhTeFA3/03aocUFAmSIOFAACgkQeFA3/03a
ocW2JAf/WHNOhKsmgJ7tYo98tiAwIYkU5Hd2wzWzUi8iGqowDg7fF7tJOgTFTx4K
abHmA3hEMLpt6KR4kGOtDw2qS8z8QsclOx6vlft39MHgJGbWqFpMwB0USqhj4QD1
r/Cm+uTfCUN0aO4aRpj+nvHLD7CFKtRG9G0PfpilhB8aqxhqb0h6OW5ifYidy82g
JSsFy4q5knXtRahjQlNC1Rd00ciJSWUmSU+1dRcwa9bwIHFRWCFRB6eHd4LSddkY
xytHJrEip3nx4wbhjHMB5lEv0kbd1CfyIBPg1JzZXXrxpYnfYdltPtNmsN45u6lx
XnOWkKbNK0yoq0AlGMjC2R/a7SJzYg==
=UxJ2
-----END PGP SIGNATURE-----

--VHU0M3xPK1b1nyOO--