regulator: s2mps11: drop redundant sanity checks in s2mpg10_of_parse_cb()
authorAndré Draszik <andre.draszik@linaro.org>
Tue, 10 Feb 2026 17:23:50 +0000 (17:23 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 16 Feb 2026 16:30:46 +0000 (16:30 +0000)
commit5a1256acaa856e6770d74dd786b1b12548969c28
treeefaee39192a61f1807c95e4f46d4502b4db5ba4b
parente243cdd87b911ce9968b62e4ab2b680dfadc4341
regulator: s2mps11: drop redundant sanity checks in s2mpg10_of_parse_cb()

The sanity checks being removed in this commit are useless as earlier
code already checks for all conditions, including all error cases like
out-of-bounds conditions. In other words, the code being removed here
has no effect, as any potential error it could catch will already have
been caught by earlier code.

The checks removed here are also incomplete (as they're off-by-one) -
they should have been checking >= ARRAY_SIZE() to be complete.

Simply remove this redundant and incorrect code.

No functional change.

Reported-by: Dan Carpenter <dan.carpenter@linaro.org>
Closes: https://lore.kernel.org/all/aYmsu8qREppwBESH@stanley.mountain/
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: André Draszik <andre.draszik@linaro.org>
Link: https://patch.msgid.link/20260210-s2mpg1x-regulators-fixes-v2-1-ab3d3457f1ae@linaro.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/s2mps11.c