On 3/10/23 05:58, Rasmus Villemoes wrote:
> 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.villem...@prevas.dk>
> ---
> drivers/sysinfo/gpio.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/sysinfo/gpio.c b/drivers/sysinfo/gpio.c
> index 1d7f050998..82f90303bb 100644
> --- a/drivers/sysinfo/gpio.c
> +++ b/drivers/sysinfo/gpio.c
> @@ -57,7 +57,7 @@ static int sysinfo_gpio_get_str(struct udevice *dev, int
> id, size_t size, char *
> int i, ret;
> u32 revision;
>
> - for (i = 0; i < priv->gpio_num; i++) {
> + for (i = 0; ; i++) {
> ret = dev_read_u32_index(dev, "revisions", i,
> &revision);
> if (ret) {
> @@ -80,7 +80,8 @@ static int sysinfo_gpio_get_str(struct udevice *dev, int
> id, size_t size, char *
> strncpy(val, name, size);
> val[size - 1] = '\0';
> return 0;
> - } default:
> + }
> + default:
> return -EINVAL;
> };
> }
Reviewed-by: Sean Anderson <sean.ander...@seco.com>
