diff mbox series

gpio: dwapb: Don't print error on -EPROBE_DEFER

Message ID 20220610075152.10214-1-Sergey.Semin@baikalelectronics.ru
State Superseded
Headers show
Series gpio: dwapb: Don't print error on -EPROBE_DEFER | expand

Commit Message

Serge Semin June 10, 2022, 7:51 a.m. UTC
Currently if the APB or Debounce clocks aren't yet ready to be requested
the DW GPIO driver will correctly handle that by deferring the probe
procedure, but the error is still printed to the system log. It needlessly
pollutes the log since there was no real error but a request to postpone
the clock request procedure since the clocks subsystem hasn't been fully
initialized yet. Let's fix that by using the dev_err_probe method to print
the APB/clock request error status. It will correctly handle the deferred
probe situation and print the error if it actually happens.

Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru>
---
 drivers/gpio/gpio-dwapb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/drivers/gpio/gpio-dwapb.c b/drivers/gpio/gpio-dwapb.c
index b0f3aca61974..a51458be34a9 100644
--- a/drivers/gpio/gpio-dwapb.c
+++ b/drivers/gpio/gpio-dwapb.c
@@ -653,7 +653,7 @@  static int dwapb_get_clks(struct dwapb_gpio *gpio)
 	err = devm_clk_bulk_get_optional(gpio->dev, DWAPB_NR_CLOCKS,
 					 gpio->clks);
 	if (err) {
-		dev_err(gpio->dev, "Cannot get APB/Debounce clocks\n");
+		dev_err_probe(gpio->dev, err, "Cannot get APB/Debounce clocks\n");
 		return err;
 	}