[PATCH] video: backlight: sky81452-backlight: Fix reference count imbalance on error
Markus Elfring
Markus.Elfring at web.de
Wed Aug 19 18:33:50 UTC 2020
>>> +++ b/drivers/video/backlight/sky81452-backlight.c
>>> @@ -217,6 +217,7 @@ static struct sky81452_bl_platform_data *sky81452_bl_parse_dt(
>>> num_entry);
>>> if (ret < 0) {
>>> dev_err(dev, "led-sources node is invalid.\n");
>>> + of_node_put(np);
>>> return ERR_PTR(-EINVAL);
>>> }
>>
>> I propose to add the jump target “put_node” so that a bit of common exception
>> handling code can be better reused at the end of this function implementation.
…
> You can safely ignore any review comments from Markus!
How does this feedback fit to the Linux coding style?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?id=18445bf405cb331117bc98427b1ba6f12418ad17#n475
Regards,
Markus
More information about the dri-devel
mailing list