On Tue, Jun 04, 2019 at 05:31:50PM +0200, Thierry Reding wrote: > From: Thierry Reding <[email protected]> > > When deferring probe, avoid logging a confusing error message. While at > it, make the error message more informational. > > Signed-off-by: Thierry Reding <[email protected]> > --- > drivers/gpu/host1x/dev.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/host1x/dev.c b/drivers/gpu/host1x/dev.c > index c55e2d634887..5a3f797240d4 100644 > --- a/drivers/gpu/host1x/dev.c > +++ b/drivers/gpu/host1x/dev.c > @@ -247,8 +247,11 @@ static int host1x_probe(struct platform_device *pdev) > > host->clk = devm_clk_get(&pdev->dev, NULL); > if (IS_ERR(host->clk)) { > - dev_err(&pdev->dev, "failed to get clock\n"); > err = PTR_ERR(host->clk); > + > + if (err != -EPROBE_DEFER) > + dev_err(&pdev->dev, "failed to get clock: %d\n", err);
Reviewed-by: Daniel Vetter <[email protected]> > + > return err; > } > > -- > 2.21.0 > > _______________________________________________ > dri-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
