Re: [PATCH] iio: light: veml3328: remove redundant cast in read_avail
From: Jonathan Cameron
Date: Tue Jul 28 2026 - 17:48:17 EST
On Mon, 27 Jul 2026 17:50:23 +0400
Giorgi Tchankvetadze <giorgitchankvetadze1997@xxxxxxxxx> wrote:
> On Mon, Jul 27, 2026 at 5:16 PM David Lechner <dlechner@xxxxxxxxxxxx> wrote:
> >
> > Usually, we would declare the array like:
> >
> > static const int veml3328_scale_vals[4][4][2] ...
> >
> > in which case this cast would be needed.
> >
> >
> I intentionally kept the patch tiny and dealt with the obvious thing,
> but yes, I agree this is how it should be done.
I think it is worth making the more complex change to improve the
readability more generally. So if you don't mind, please do that
for v2.
Jonathan