tree cd28b046cfbc8869c74b654a8498b3310b8784c3
parent 732b0825475c1a2466a6abf6e223b2a77af011f2
author Rasmus Villemoes <rasmus.villemoes@prevas.dk> 1678445883 +0100
committer Tom Rini <trini@konsulko.com> 1680203399 -0400

sysinfo: gpio: fix loop over DT "revisions" array

There can certainly be a lot more elements in the "revisions" (and
"names") arrays than there are gpios used to form the trinary number
we're searching for; we simply don't know the array size up-front.

Nor do we need to, because the loop body already knows to recognize
-EOVERFLOW as "not that many elements present" (and we have a test
that specifically ensures that dev_read_u32_index() returns exactly
that). So just drop the i < priv->gpio_num condition.

While in here, fix the weird placement of the default: keyword.

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Sean Anderson <sean.anderson@seco.com>