From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,HTML_MESSAGE,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,PDS_BAD_THREAD_QP_64,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 82A22C433DB for ; Mon, 8 Mar 2021 14:22:37 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1C960651CD for ; Mon, 8 Mar 2021 14:22:37 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1C960651CD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=foss.st.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 815DE89B68; Mon, 8 Mar 2021 14:22:36 +0000 (UTC) Received: from mx07-00178001.pphosted.com (mx07-00178001.pphosted.com [185.132.182.106]) by gabe.freedesktop.org (Postfix) with ESMTPS id 26CBD89B68 for ; Mon, 8 Mar 2021 14:22:34 +0000 (UTC) Received: from pps.filterd (m0241204.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 128ELpMf030911; Mon, 8 Mar 2021 15:22:31 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=foss.st.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : mime-version; s=selector1; bh=rlricdE70O7GaP5H/fDILnTjjifnjoiAsY/5cN+eJZs=; b=iMS7rb9/23O7p7wbK2kMLiG3T5WHiHR76IapC+OHgtUdhzKISlSLvj9bVejy/TrMn3Ms wh8WiuEc4gTauHGVk1PnKVUavGVB2f+VZ7bY3tiaxZUV2LG2+rLAsVaWurzdScMNZyJw 0QxH5juL8GDCQK76XwmtOUFAobSIN5M1Pv0fs5W122Za1NSjj68xahg0mERgeW78FFXA e+brdhKVPtg/3TjBHMclFNbUh92N4OlNiUmV2PnkhaIX5tDAAHATYAdcVbUlq/mBgzXA uvOffkJEyqac/TR+cVhjbRqDo2WOIAl/NYreciyFNpOgry7tT4d1f98vXgAp5Y9axal7 jg== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com with ESMTP id 3741y6k04e-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 08 Mar 2021 15:22:31 +0100 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id 1F3F410002A; Mon, 8 Mar 2021 15:22:30 +0100 (CET) Received: from Webmail-eu.st.com (sfhdag2node3.st.com [10.75.127.6]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id EF32424AF08; Mon, 8 Mar 2021 15:22:29 +0100 (CET) Received: from SFHDAG2NODE3.st.com (10.75.127.6) by SFHDAG2NODE3.st.com (10.75.127.6) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 8 Mar 2021 15:22:29 +0100 Received: from SFHDAG2NODE3.st.com ([fe80::31b3:13bf:2dbe:f64c]) by SFHDAG2NODE3.st.com ([fe80::31b3:13bf:2dbe:f64c%20]) with mapi id 15.00.1473.003; Mon, 8 Mar 2021 15:22:29 +0100 From: Philippe CORNU - foss To: Yannick FERTRE - foss , "Raphael GALLAIS-POU - foss" , Yannick FERTRE , Philippe CORNU , "Benjamin Gaignard" , Vincent ABRIOU , Sam Ravnborg , Joe Perches Subject: RE: [PATCH 1/2] drm/stm: dsi: Avoid printing errors for -EPROBE_DEFER Thread-Topic: [PATCH 1/2] drm/stm: dsi: Avoid printing errors for -EPROBE_DEFER Thread-Index: AQHXCPxP1JW0jPBz9kimszP//+ain6p50u6AgABm/FU= Date: Mon, 8 Mar 2021 14:22:29 +0000 Message-ID: <1615213345469.50782@foss.st.com> References: <20210222092205.32086-1-raphael.gallais-pou@foss.st.com> <20210222092205.32086-2-raphael.gallais-pou@foss.st.com>, <7db3bebd-2cfa-d8d8-40e5-81702295e151@foss.st.com> In-Reply-To: <7db3bebd-2cfa-d8d8-40e5-81702295e151@foss.st.com> Accept-Language: fr-FR, en-US Content-Language: fr-FR X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.47] MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.369, 18.0.761 definitions=2021-03-08_08:2021-03-08, 2021-03-08 signatures=0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Raphael GALLAIS-POU , David Airlie , "linux-kernel@vger.kernel.org" , "dri-devel@lists.freedesktop.org" , Alexandre TORGUE - foss , Maxime Coquelin , "linux-stm32@st-md-mailman.stormreply.com" , "linux-arm-kernel@lists.infradead.org" , Alexandre TORGUE Content-Type: multipart/mixed; boundary="===============1138515426==" Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" --===============1138515426== Content-Language: fr-FR Content-Type: multipart/alternative; boundary="_000_161521334546950782fossstcom_" --_000_161521334546950782fossstcom_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Applied on drm-misc-next. Many thanks Rapha=EBl & Yannick for your patch. Note: I have updated the "From:" field to yannick.fertre@foss.st.com for mo= re consistency. Philippe :-) ________________________________ De : Yannick FERTRE - foss Envoy=E9 : lundi 8 mars 2021 10:10 =C0 : Raphael GALLAIS-POU - foss; Yannick FERTRE; Philippe CORNU; Benjamin = Gaignard; Vincent ABRIOU; Sam Ravnborg; Joe Perches Cc : David Airlie; Daniel Vetter; Maxime Coquelin; Alexandre TORGUE; dri-de= vel@lists.freedesktop.org; linux-stm32@st-md-mailman.stormreply.com; linux-= arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Philippe CORN= U - foss; Raphael GALLAIS-POU; Alexandre TORGUE - foss Objet : Re: [PATCH 1/2] drm/stm: dsi: Avoid printing errors for -EPROBE_DEF= ER Tested-by: Yannick Fertre On 2/22/21 10:22 AM, Raphael GALLAIS-POU - foss wrote: > From: Yannick Fertre > > Don't print error when probe deferred error is returned. > > Signed-off-by: Raphael Gallais-Pou > Signed-off-by: Yannick Fertre > --- > drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c b/drivers/gpu/drm/stm/= dw_mipi_dsi-stm.c > index 2e1f2664495d..8399d337589d 100644 > --- a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c > +++ b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c > @@ -363,8 +363,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_devi= ce *pdev) > dsi->vdd_supply =3D devm_regulator_get(dev, "phy-dsi"); > if (IS_ERR(dsi->vdd_supply)) { > ret =3D PTR_ERR(dsi->vdd_supply); > - if (ret !=3D -EPROBE_DEFER) > - DRM_ERROR("Failed to request regulator: %d\n", ret)= ; > + dev_err_probe(dev, ret, "Failed to request regulator\n"); > return ret; > } > > @@ -377,9 +376,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_devi= ce *pdev) > dsi->pllref_clk =3D devm_clk_get(dev, "ref"); > if (IS_ERR(dsi->pllref_clk)) { > ret =3D PTR_ERR(dsi->pllref_clk); > - if (ret !=3D -EPROBE_DEFER) > - DRM_ERROR("Unable to get pll reference clock: %d\n"= , > - ret); > + dev_err_probe(dev, ret, "Unable to get pll reference clock\= n"); > goto err_clk_get; > } > > @@ -419,7 +416,7 @@ static int dw_mipi_dsi_stm_probe(struct platform_devi= ce *pdev) > dsi->dsi =3D dw_mipi_dsi_probe(pdev, &dw_mipi_dsi_stm_plat_data); > if (IS_ERR(dsi->dsi)) { > ret =3D PTR_ERR(dsi->dsi); > - DRM_ERROR("Failed to initialize mipi dsi host: %d\n", ret); > + dev_err_probe(dev, ret, "Failed to initialize mipi dsi host= \n"); > goto err_dsi_probe; > } > > --_000_161521334546950782fossstcom_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable
Applied on drm-misc-next.
Many thanks Rapha=EBl & Yannick for your patch.
Note: I have updated the "From:" field to yannick.= fertre@foss.st.com for more consistency.
Philippe :-)


De : Yannick FERTRE - fos= s
Envoy=E9 : lundi 8 mars 2021 10:10
=C0 : Raphael GALLAIS-POU - foss; Yannick FERTRE; Philippe CORNU; Be= njamin Gaignard; Vincent ABRIOU; Sam Ravnborg; Joe Perches
Cc : David Airlie; Daniel Vetter; Maxime Coquelin; Alexandre TO= RGUE; dri-devel@lists.freedesktop.org; linux-stm32@st-md-mailman.stormreply= .com; linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; P= hilippe CORNU - foss; Raphael GALLAIS-POU; Alexandre TORGUE - foss
Objet : Re: [PATCH 1/2] drm/stm: dsi: Avoid printing errors for -EPR= OBE_DEFER
 
Tested-by: Yannick Fertre <yannick.fertre@foss.= st.com>

On 2/22/21 10:22 AM, Raphael GALLAIS-POU - foss wrote:
> From: Yannick Fertre <yannick.fertre@st.com>
>
> Don't print error when probe deferred error is returned.
>
> Signed-off-by: Raphael Gallais-Pou <raphael.gallais-pou@foss.st.com= >
> Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com>
> ---
>   drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 9 +++-= -----
>   1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c b/drivers/gpu/drm/s= tm/dw_mipi_dsi-stm.c
> index 2e1f2664495d..8399d337589d 100644
> --- a/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
> +++ b/drivers/gpu/drm/stm/dw_mipi_dsi-stm.c
> @@ -363,8 +363,7 @@ static int dw_mipi_dsi_stm_probe(struct platfo= rm_device *pdev)
>        dsi->vdd_supply =3D devm_= regulator_get(dev, "phy-dsi");
>        if (IS_ERR(dsi->vdd_suppl= y)) {
>            = ;    ret =3D PTR_ERR(dsi->vdd_supply);
> -           &nb= sp; if (ret !=3D -EPROBE_DEFER)
> -           &nb= sp;         DRM_ERROR("Failed = to request regulator: %d\n", ret);
> +           = ;  dev_err_probe(dev, ret, "Failed to request regulator\n");=
>            = ;    return ret;
>        }
>  
> @@ -377,9 +376,7 @@ static int dw_mipi_dsi_stm_probe(struct platfo= rm_device *pdev)
>        dsi->pllref_clk =3D devm_= clk_get(dev, "ref");
>        if (IS_ERR(dsi->pllref_cl= k)) {
>            = ;    ret =3D PTR_ERR(dsi->pllref_clk);
> -           &nb= sp; if (ret !=3D -EPROBE_DEFER)
> -           &nb= sp;         DRM_ERROR("Unable = to get pll reference clock: %d\n",
> -           &nb= sp;            =        ret);
> +           = ;  dev_err_probe(dev, ret, "Unable to get pll reference clock\n&q= uot;);
>            = ;    goto err_clk_get;
>        }
>  
> @@ -419,7 +416,7 @@ static int dw_mipi_dsi_stm_probe(struct platfo= rm_device *pdev)
>        dsi->dsi =3D dw_mipi_dsi_= probe(pdev, &dw_mipi_dsi_stm_plat_data);
>        if (IS_ERR(dsi->dsi)) { >            = ;    ret =3D PTR_ERR(dsi->dsi);
> -           &nb= sp; DRM_ERROR("Failed to initialize mipi dsi host: %d\n", ret); > +           = ;  dev_err_probe(dev, ret, "Failed to initialize mipi dsi host\n&= quot;);
>            = ;    goto err_dsi_probe;
>        }
>  
>
--_000_161521334546950782fossstcom_-- --===============1138515426== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel --===============1138515426==--