Re: [PATCH v2 3/7] iio: temperature: ltc2983: Fix inconsistent channel wording in messages

[Liviu Stan]

On Sat, 16 May 2026 17:48:11 +0100 Jonathan Cameron <jic23@xxxxxxxxxx> wrote:

> On Thu, 14 May 2026 17:46:50 +0300
> Liviu Stan <liviu.stan@xxxxxxxxxx> wrote:
> 
> > Replace all occurrences of the abbreviated 'chann' and 'chan' with
> > 'channel' in error and debug messages throughout the driver.
> > 
> > Fixes: f110f3188e5639c81c457b2b831d40dfe3891bdb ("iio: temperature: Add support for LTC2983")
> > Fixes: a00838cae079b9b9b90969c2b7b031b1bfd9ab3a ("iio: temperature: ltc2983: convert to dev_err_probe()")
> 
> Wrong format and also what is it fixing?
> 
> I think you have misunderstood what that tag means.
> It almost always means we should backport. Which is not something we'll
> do for consistency in error messages.
> 

Sorry for this, I will change in v3.

> > Signed-off-by: Liviu Stan <liviu.stan@xxxxxxxxxx>
> 
> 
> Sashiko did call out a few more cases, but I think they are fine as they
> are using chan: %d to tell us which channel a message is referring to.
> However you should relax your 'all occurrences' to make it clear it's
> not 'all'.
> 

I can modify them too to "channel" so that they're all consistent.

Thanks,
Liviu