Hi Geert, Thank you for the patch. On Thu, Dec 16, 2021 at 03:17:33PM +0100, Geert Uytterhoeven wrote: > Use the dev_err_probe() helper to streamline error handling. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/tty/serial/sh-sci.c | 11 ++++------- > 1 file changed, 4 insertions(+), 7 deletions(-) > > diff --git a/drivers/tty/serial/sh-sci.c b/drivers/tty/serial/sh-sci.c > index 686ca1777222b1d4..5f6d85b8e3dd4173 100644 > --- a/drivers/tty/serial/sh-sci.c > +++ b/drivers/tty/serial/sh-sci.c > @@ -2790,15 +2790,12 @@ static int sci_init_clocks(struct sci_port *sci_port, struct device *dev) > * global "peripheral_clk" clock. > */ > clk = devm_clk_get(dev, "peripheral_clk"); > - if (!IS_ERR(clk)) > - goto found; > - > - dev_err(dev, "failed to get %s (%ld)\n", clk_names[i], > - PTR_ERR(clk)); > - return PTR_ERR(clk); > + if (IS_ERR(clk)) > + return dev_err_probe(dev, PTR_ERR(clk), > + "failed to get %s\n", > + clk_names[i]); > } > > -found: > if (IS_ERR(clk)) > dev_dbg(dev, "failed to get %s (%ld)\n", clk_names[i], > PTR_ERR(clk)); -- Regards, Laurent Pinchart
