On Fri, Jan 22, 2021 at 01:16:49AM +0300, Dmitry Osipenko wrote:
> + /* balance only if there are regulators coupled */
> + if (rdev->coupling_desc.n_coupled > 1) {
> + ret = regulator_balance_voltage(rdev, PM_SUSPEND_ON);
> + goto out;
> + }
> +
> ret = _regulator_do_set_voltage(rdev, min_uV, max_uV);
>
> out:
This would be clearer written as an if/else rather than using the goto.
Attachment:
signature.asc
Description: PGP signature
