Re: [PATCH v5 10/11] iio: adc: hx711: split hx711_reset variable assignment

From: Andy Shevchenko

Date: Wed Apr 29 2026 - 14:55:59 EST


On Wed, Apr 29, 2026 at 11:15:43AM +0530, Piyush Patle wrote:
> Separate the hx711_wait_for_ready() assignment from the declaration in
> hx711_reset().
>
> This is a small preparatory cleanup before the later variant-specific
> changes add another local variable in the same function.
>
> No functional change.

As I pointed out this also should be done for val assignment in other case.
And it should go before loop iterator patch.

...

> static int hx711_reset(struct hx711_data *hx711_data)
> {
> - int val = hx711_wait_for_ready(hx711_data);
> + int val;
> +
> + val = hx711_wait_for_ready(hx711_data);

>

This blank line has to be removed now.

> if (val) {

--
With Best Regards,
Andy Shevchenko