Re: [PATCH v9 5/6] iio: adc: ltc2378: Add support for LTC2338-18

[Jonathan Cameron <jic23@kernel.org>]

On Tue, 28 Jul 2026 10:12:34 -0300
Marcelo Schmitt <marcelo.schmitt1@gmail.com> wrote:

> On 07/27, sashiko-bot@kernel.org wrote:
> > Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
> > - [Low] Declaring a variable immediately after a switch case label without a block statement causes a compilation error.
> > --
> > 
> > --- Patch [5]: [PATCH v9 5/6] iio: adc: ltc2378: Add support for LTC2338-18 ---
> > commit 29728b1d34f34b2970657e2d83536de22940ee2e
> > Author: Marcelo Schmitt <marcelo.schmitt@analog.com>
> > 
> > iio: adc: ltc2378: Add support for LTC2338-18
> > 
> > This commit adds support for the LTC2338-18 ADC to the existing ltc2378
> > driver. It extends the driver to handle the internal voltage reference and
> > input signal attenuation specific to this part.
> >   
> > > diff --git a/drivers/iio/adc/ltc2378.c b/drivers/iio/adc/ltc2378.c
> > > --- a/drivers/iio/adc/ltc2378.c
> > > +++ b/drivers/iio/adc/ltc2378.c  
> > 
> > [ ... ]
> >   
> > > @@ -383,7 +395,10 @@ static int ltc2378_read_raw(struct iio_dev *indio_dev,
> > >  		return IIO_VAL_INT;
> > >  	}
> > >  	case IIO_CHAN_INFO_SCALE:
> > > +		struct u32_fract fract = st->info->internal_div;  
> > 
> > [Severity: Low]
> > Does this declaration cause a build error?  
> No, at least not when I built with clang/llvm-22.
Maybe but given we still stick to declarations at top of scope (if not
strictly required) makes me want add scope here anyway.

So I tweaked it to add the {}

Series applied to the testing branch of iio.git.

Thanks,

Jonathan


> 
> > Declaring a variable immediately
> > after a switch case label without a block statement is not a valid statement
> > in C and usually causes a compiler error. Could this case be wrapped in a
> > block to scope the variable, or the declaration be moved to the top of the
> > function?
> >   
> > >  		*val = st->ref_uV / MILLI;
> > > +		if (fract.numerator && fract.denominator)
> > > +			*val = mult_frac(*val, fract.numerator, fract.denominator);
> > >  		/*
> > >  		 * For all LTC2378-like devices, the amount of bits that express
> > >  		 * voltage magnitude depend on the polarity / output code format:  
> > 
> > -- 
> > Sashiko AI review · https://sashiko.dev/#/patchset/cover.1785186980.git.marcelo.schmitt@analog.com?part=5  
>