Re: [PATCH v2 1/1] iio: chemical: bme680: use BME680_NUM_CHANNELS for scan buffer

[Jonathan Cameron <jic23@kernel.org>]

On Mon, 20 Apr 2026 13:54:36 +0530
Nikhil Gautam <nikhilgtr@gmail.com> wrote:

> Use BME680_NUM_CHANNELS instead of the hardcoded channel count
> in the scan buffer.
> 
> This avoids use of a magic number and improves code readability
> and maintainability.
> 
> Suggested-by: Jonathan Cameron <jic23@kernel.org>
> Signed-off-by: Nikhil Gautam <nikhilgtr@gmail.com>
I think I sat on this to see if anyone else wanted to review then
lost it (patchwork kept track though!)

Anyhow, applied to the testing branch of iio.git.

Thanks,

Jonathan

> ---
>  drivers/iio/chemical/bme680_core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/iio/chemical/bme680_core.c b/drivers/iio/chemical/bme680_core.c
> index 3e850562ab00..42c1a9de175a 100644
> --- a/drivers/iio/chemical/bme680_core.c
> +++ b/drivers/iio/chemical/bme680_core.c
> @@ -128,7 +128,7 @@ struct bme680_data {
>  	u16 heater_temp;
>  
>  	struct {
> -		s32 chan[4];
> +		s32 chan[BME680_NUM_CHANNELS];
>  		aligned_s64 ts;
>  	} scan;
>