Re: [PATCH v1 3/3] leds: lm3697: Switch to use fwnode_property_count_uXX()

[Dan Murphy <dmurphy@xxxxxx>]

Andy

On 7/23/19 3:14 PM, Andy Shevchenko wrote:
> Use fwnode_property_count_uXX() directly, that makes code neater.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

Acked-by: Dan Murphy <dmurphy@xxxxxx>

> ---
>   drivers/leds/leds-lm3697.c | 5 +----
>   1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/leds/leds-lm3697.c b/drivers/leds/leds-lm3697.c
> index 54e0e35df824..b71711aff8a3 100644
> --- a/drivers/leds/leds-lm3697.c
> +++ b/drivers/leds/leds-lm3697.c
> @@ -244,10 +244,7 @@ static int lm3697_probe_dt(struct lm3697 *priv)
>   		led->lmu_data.lsb_brightness_reg = LM3697_CTRL_A_BRT_LSB +
>   						   led->control_bank * 2;
>   
> -		led->num_leds = fwnode_property_read_u32_array(child,
> -						       "led-sources",
> -						       NULL, 0);
> -
> +		led->num_leds = fwnode_property_count_u32(child, "led-sources");
>   		if (led->num_leds > LM3697_MAX_LED_STRINGS) {
>   			dev_err(&priv->client->dev, "To many LED strings defined\n");
>   			continue;