Messages in this thread | | | Subject | Re: [PATCH] leds: lm3532: Fix optional led-max-microamp prop error handling | From | Jacek Anaszewski <> | Date | Thu, 29 Aug 2019 22:04:18 +0200 |
| |
Hi Dan,
Thanks for the update.
On 8/29/19 9:18 PM, Dan Murphy wrote: > Fix the error handling for the led-max-microamp property. > Need to check if the property is present and then if it is > retrieve the setting and its max boundary > > Reported-by: Pavel Machek <pavel@ucw.cz> > Signed-off-by: Dan Murphy <dmurphy@ti.com> > --- > drivers/leds/leds-lm3532.c | 14 +++++++++----- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git a/drivers/leds/leds-lm3532.c b/drivers/leds/leds-lm3532.c > index c5cfd8e3f15f..13b4265fb85a 100644 > --- a/drivers/leds/leds-lm3532.c > +++ b/drivers/leds/leds-lm3532.c > @@ -601,11 +601,15 @@ static int lm3532_parse_node(struct lm3532_data *priv) > goto child_out; > } > > - ret = fwnode_property_read_u32(child, "led-max-microamp", > - &led->full_scale_current); > - > - if (led->full_scale_current > LM3532_FS_CURR_MAX) > - led->full_scale_current = LM3532_FS_CURR_MAX; > + if (fwnode_property_present(child, "led-max-microamp")) { > + if (fwnode_property_read_u32(child, "led-max-microamp", > + &led->full_scale_current)) > + dev_err(&priv->client->dev, > + "Failed getting led-max-microamp\n"); > + > + if (led->full_scale_current > LM3532_FS_CURR_MAX) > + led->full_scale_current = LM3532_FS_CURR_MAX;
One more nit: we have min() macro in kernel.h for such things.
> + } > > if (led->mode == LM3532_BL_MODE_ALS) { > led->mode = LM3532_ALS_CTRL; >
-- Best regards, Jacek Anaszewski
| |