Re: [PATCH] sysinfo: gpio: fix loop over DT "revisions" array

[Sean Anderson <sean.anderson@seco.com>]

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.villemoes@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.anderson@seco.com>