Re: [PATCH] regulator: TPS6287X: Use min/max uV to get VRANGE

From: Jonas Andreasson
Date: Fri Jan 17 2025 - 07:41:35 EST


On 1/16/25 16:58, Mark Brown wrote:
On Wed, Jan 15, 2025 at 10:55:22AM +0100, Jonas Andreasson wrote:

+ if (!data || data->range == -1)
+ goto fallback;

...

+ return selector;
+
+fallback:
+ return regulator_map_voltage_pickable_linear_range(rdev, min_uV, max_uV);

You could just have the return statement directly and avoid the goto
which would make this easier to follow.

Hello.
Thanks for looking at my code!
That is a good point which I will fix.

Kind Regards,
Jonas