Re: [RFT] [PATCH] regulator: tps65219: Fix .bypass_val_on setting

From: jerome Neanne
Date: Mon Sep 12 2022 - 05:49:19 EST


Hi Axel,

On 28/08/2022 14:01, Axel Lin wrote:
The .bypass_val_on setting does not match the .bypass_mask setting, so the
.bypass_mask bit will never get set. Fix it by removing .bypass_val_on
setting, the regulator_set_bypass_regmap and regulator_get_bypass_regmap
helpers will use rdev->desc->bypass_mask as val_on if the val_on is 0.
I think this will result in exact same behavior. val would be assigned to 1 when enable is set and 0 otherwise. Anyway you are right this line is useless.
I don't have this h/w to test, please help to review and test the patch.

I'll double check on the board.