linux-kernel - Re: [PATCH] iio: adc: ti-ads1015: Use correct pga upper bound

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240107161524.7dc72a3f@jic23-huawei>
Date: Sun, 7 Jan 2024 16:15:24 +0000
From: Jonathan Cameron <jic23@...nel.org>
To: Mohammed Billoo <mab.kernel@...il.com>
Cc: Lars-Peter Clausen <lars@...afoo.de>, Marek Vasut <marex@...x.de>, Uwe
 Kleine-König <u.kleine-koenig@...gutronix.de>, Biju Das
 <biju.das.jz@...renesas.com>, linux-iio@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] iio: adc: ti-ads1015: Use correct pga upper bound

On Sat,  6 Jan 2024 12:48:35 -0500
Mohammed Billoo <mab.kernel@...il.com> wrote:

> The devicetree binding and datasheets (for both the ADS1015 and
> ADS1115) show that the PGA index should have a maximum value of 5,
> and not 6.
> 
> Signed-off-by: Mohammed Billoo <mab.kernel@...il.com>

Hi Mohammed,

Seems correct, but I'm curious - did you see an actual problem with
setting the value to 6?  From my reading of the ads1115 datasheet
6 and 7 should be functionally identical to 5.

Anyhow, good to clean it up anyway so applied to the togreg branch of
iio.git but that tree will be rebased on rc1 once available so for
now I'll just push it as testing for 0-day to get started on testing it.

Thanks,

Jonathan

> ---
>  drivers/iio/adc/ti-ads1015.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/adc/ti-ads1015.c b/drivers/iio/adc/ti-ads1015.c
> index 6799ea49dbc7..6ae967e4d8fa 100644
> --- a/drivers/iio/adc/ti-ads1015.c
> +++ b/drivers/iio/adc/ti-ads1015.c
> @@ -925,7 +925,7 @@ static int ads1015_client_get_channels_config(struct i2c_client *client)
>  
>  		if (!fwnode_property_read_u32(node, "ti,gain", &pval)) {
>  			pga = pval;
> -			if (pga > 6) {
> +			if (pga > 5) {
>  				dev_err(dev, "invalid gain on %pfw\n", node);
>  				fwnode_handle_put(node);
>  				return -EINVAL;