Re: [PATCH RFC 2/6] gpio: mvebu: honour EPROBE_DEFER for devm_clk_get()
From: Uwe Kleine-König <hidden>
Date: 2020-03-29 13:17:12
Also in:
linux-devicetree, linux-gpio, linux-pwm
On Sun, Mar 29, 2020 at 11:48:09AM +0100, Russell King wrote:
quoted hunk ↗ jump to hunk
Honour deferred probing for devm_clk_get() so that we can get the clock for PWM. Signed-off-by: Russell King <redacted> --- drivers/gpio/gpio-mvebu.c | 3 +++ 1 file changed, 3 insertions(+)diff --git a/drivers/gpio/gpio-mvebu.c b/drivers/gpio/gpio-mvebu.c index fa5641615db6..ee13b11c5298 100644 --- a/drivers/gpio/gpio-mvebu.c +++ b/drivers/gpio/gpio-mvebu.c@@ -1132,6 +1132,9 @@ static int mvebu_gpio_probe(struct platform_device *pdev) } mvchip->clk = devm_clk_get(&pdev->dev, NULL); + if (mvchip->clk == ERR_PTR(-EPROBE_DEFER)) + return -EPROBE_DEFER; + /* Not all SoCs require a clock.*/ if (!IS_ERR(mvchip->clk)) clk_prepare_enable(mvchip->clk);
I'd say the following is the right thing to do here: mvchip->clk = devm_clk_get_optional(...); if (IS_ERR(mvchip->clk)) return ... Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ | _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel