<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <style type="text/css" style="display:none"></style> </head> <body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;"> <div>Applied on drm-misc-next.<br> </div> <div>Many thanks Raphaël & Yannick for your patch.<br> </div> <div>Note: I have updated the "From:" field to yannick.fertre@foss.st.com for more consistency.<br> </div> <div>Philippe :-)<br> </div> <p><br> </p> <hr tabindex="-1" style="color: rgb(33, 33, 33); font-size: 12pt; display: inline-block; width: 98%;"> <div style="color: rgb(33, 33, 33);"> <div> <div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>De :</b> Yannick FERTRE - foss<br> <b>Envoyé :</b> lundi 8 mars 2021 10:10<br> <b>À :</b> Raphael GALLAIS-POU - foss; Yannick FERTRE; Philippe CORNU; Benjamin Gaignard; Vincent ABRIOU; Sam Ravnborg; Joe Perches<br> <b>Cc :</b> David Airlie; Daniel Vetter; Maxime Coquelin; Alexandre TORGUE; dri-devel@lists.freedesktop.org; linux-stm32@st-md-mailman.stormreply.com; linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Philippe CORNU - foss; Raphael GALLAIS-POU; Alexandre TORGUE - foss<br> <b>Objet :</b> Re: [PATCH 1/2] drm/stm: dsi: Avoid printing errors for -EPROBE_DEFER</font> <div> </div> </div> </div> <font size="2"><span style="font-size:10pt;"> <div class="PlainText">Tested-by: Yannick Fertre <yannick.fertre@foss.st.com><br> <br> On 2/22/21 10:22 AM, Raphael GALLAIS-POU - foss wrote:<br> > From: Yannick Fertre <yannick.fertre@st.com><br> > <br> > Don't print error when probe deferred error is returned.<br> > <br> > Signed-off-by: Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com><br> > Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com><br> > ---<br> > drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 9 +++------<br> > 1 file changed, 3 insertions(+), 6 deletions(-)<br> > <br> > diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c<br> > index 2e1f2664495d..8399d337589d 100644<br> > --- a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c<br> > +++ b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c<br> > @@ -363,8 +363,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_device *pdev)<br> > dsi->vdd_supply = devm_regulator_get(dev, "phy-dsi");<br> > if (IS_ERR(dsi->vdd_supply)) {<br> > ret = PTR_ERR(dsi->vdd_supply);<br> > - if (ret != -EPROBE_DEFER)<br> > - DRM_ERROR("Failed to request regulator: %d\n", ret);<br> > + dev_err_probe(dev, ret, "Failed to request regulator\n");<br> > return ret;<br> > }<br> > <br> > @@ -377,9 +376,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_device *pdev)<br> > dsi->pllref_clk = devm_clk_get(dev, "ref");<br> > if (IS_ERR(dsi->pllref_clk)) {<br> > ret = PTR_ERR(dsi->pllref_clk);<br> > - if (ret != -EPROBE_DEFER)<br> > - DRM_ERROR("Unable to get pll reference clock: %d\n",<br> > - ret);<br> > + dev_err_probe(dev, ret, "Unable to get pll reference clock\n");<br> > goto err_clk_get;<br> > }<br> > <br> > @@ -419,7 +416,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_device *pdev)<br> > dsi->dsi = dw_mipi_dsi_probe(pdev, &dw_mipi_dsi_stm_plat_data);<br> > if (IS_ERR(dsi->dsi)) {<br> > ret = PTR_ERR(dsi->dsi);<br> > - DRM_ERROR("Failed to initialize mipi dsi host: %d\n", ret);<br> > + dev_err_probe(dev, ret, "Failed to initialize mipi dsi host\n");<br> > goto err_dsi_probe;<br> > }<br> > <br> > <br> </div> </span></font></div> </body> </html>