Thread (1 message) 1 message, 1 author, 2020-06-03

Re: [PATCH] pinctrl: sirf: Add missing put_device() call in sirfsoc_gpio_probe()

From: yukuai (C) <hidden>
Date: 2020-06-03 01:12:34
Also in: kernel-janitors, linux-gpio, lkml

On 2020/6/3 2:56, Markus Elfring wrote:
quoted
in sirfsoc_gpio_probe(), if of_find_device_by_node() succeed,
put_device() is missing in the error handling patch.
How do you think about another wording variant?

    A coccicheck run provided information like the following.

    drivers/pinctrl/sirf/pinctrl-sirf.c:798:2-8: ERROR: missing put_device;
    call of_find_device_by_node on line 792, but without a corresponding
    object release within this function.

    Generated by: scripts/coccinelle/free/put_device.cocci

    Thus add a jump target to fix the exception handling for this
    function implementation.


Would you like to add the tag “Fixes” to the commit message?
Will do, thanks for your advise!

Yu Kuai


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help