On Sat, May 30, 2020 at 10:49 PM Christophe JAILLET
<christophe.jaillet@xxxxxxxxxx> wrote:
> 'pinctrl_unregister()' should not be called to undo
> 'devm_pinctrl_register_and_init()', it is already handled by the framework.
>
> This simplifies the error handling paths of the probe function.
> The 'imx_free_resources()' can be removed as well.
>
> Fixes: a51c158bf0f7 ("pinctrl: imx: use radix trees for groups and functions")
> Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
Patch applied!
Yours,
Linus Walleij
Re: [PATCH] pinctrl: freescale: imx: Fix an error handling path in 'imx_pinctrl_probe()'
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] pinctrl: freescale: imx: Fix an error handling path in 'imx_pinctrl_probe()'
- From: Linus Walleij <linus.walleij@xxxxxxxxxx>
- Date: Wed, 3 Jun 2020 14:34:44 +0200
- In-reply-to: <20200530204955.588962-1-christophe.jaillet@wanadoo.fr>
- References: <20200530204955.588962-1-christophe.jaillet@wanadoo.fr>
- References:
- [PATCH] pinctrl: freescale: imx: Fix an error handling path in 'imx_pinctrl_probe()'
- From: Christophe JAILLET
- [PATCH] pinctrl: freescale: imx: Fix an error handling path in 'imx_pinctrl_probe()'
- Prev by Date: Re: [PATCH V2] pinctrl: sirf: add missing put_device() call in sirfsoc_gpio_probe()
- Next by Date: Re: [PATCH] pinctrl: freescale: imx: Use 'devm_of_iomap()' to avoid a resource leak in case of error in 'imx_pinctrl_probe()'
- Previous by thread: Re: [PATCH] pinctrl: freescale: imx: Fix an error handling path in 'imx_pinctrl_probe()'
- Next by thread: [PATCH v3 0/4] pinctrl: bcm2835: Add support for wake-up interrupts
- Index(es):
 |