diff options
| author | André Draszik <andre.draszik@linaro.org> | 2026-01-22 15:43:38 +0000 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2026-02-04 13:35:33 +0000 |
| commit | 0809d3dcc0dd8f597adbcd4d881063eb1b437987 (patch) | |
| tree | 9b9e279c10288be2d4c948460d5080e743e7b6a5 | |
| parent | 7d33c0a4c6a3356db2b2f599820baf75d3753d44 (diff) | |
regulator: s2mps11: drop two needless variable initialisations
The initialisations being removed are needless, as both variables are
being assigned values unconditionally further down. Additionally, doing
this eager init here might lead to preventing the compiler from issuing
a warning if a future code change actually forgets to assign a useful
value in some code path.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: André Draszik <andre.draszik@linaro.org>
Link: https://patch.msgid.link/20260122-s2mpg1x-regulators-v7-11-3b1f9831fffd@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | drivers/regulator/s2mps11.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/regulator/s2mps11.c b/drivers/regulator/s2mps11.c index 04ae9c6150bd..1f51fbc6c7b6 100644 --- a/drivers/regulator/s2mps11.c +++ b/drivers/regulator/s2mps11.c @@ -1207,8 +1207,8 @@ static int s2mps11_pmic_probe(struct platform_device *pdev) struct sec_pmic_dev *iodev = dev_get_drvdata(pdev->dev.parent); struct regulator_config config = { }; struct s2mps11_info *s2mps11; - unsigned int rdev_num = 0; - int i, ret = 0; + unsigned int rdev_num; + int i, ret; const struct regulator_desc *regulators; s2mps11 = devm_kzalloc(&pdev->dev, sizeof(struct s2mps11_info), |