diff options
| author | Liao Yuanhong <liaoyuanhong@vivo.com> | 2025-08-28 20:27:37 +0800 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2025-08-28 16:02:51 +0200 |
| commit | 899fb38dd76dd3ede425bbaf8a96d390180a5d1c (patch) | |
| tree | 6deed8a64e43db827a5766a58e2feca2eeb2d5cb | |
| parent | e2ab5f600bb01d3625d667d97b3eb7538e388336 (diff) | |
regulator: core: Remove redundant ternary operators
For ternary operators in the form of "a ? true : false", if 'a' itself
returns a boolean result, the ternary operator can be omitted. Remove
redundant ternary operators to clean up the code.
Signed-off-by: Liao Yuanhong <liaoyuanhong@vivo.com>
Message-ID: <20250828122737.43488-1-liaoyuanhong@vivo.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/core.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 554d83c4af0c..dd7b10e768c0 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -1586,8 +1586,8 @@ static int set_machine_constraints(struct regulator_dev *rdev) } if (rdev->constraints->active_discharge && ops->set_active_discharge) { - bool ad_state = (rdev->constraints->active_discharge == - REGULATOR_ACTIVE_DISCHARGE_ENABLE) ? true : false; + bool ad_state = rdev->constraints->active_discharge == + REGULATOR_ACTIVE_DISCHARGE_ENABLE; ret = ops->set_active_discharge(rdev, ad_state); if (ret < 0) { |