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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 14CBFC433FE for ; Fri, 7 Jan 2022 09:23:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346321AbiAGJXi (ORCPT ); Fri, 7 Jan 2022 04:23:38 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59522 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231910AbiAGJXg (ORCPT ); Fri, 7 Jan 2022 04:23:36 -0500 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e3e3]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 82FB4C061245 for ; Fri, 7 Jan 2022 01:23:36 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DD0E51F40A55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1641547414; bh=wj1ZDK4lkn/iSMxOoGczIJTSg6wOxjiARPaZ2r2OUWw=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c7TocaBKZAHN+HhlzCoPRaklI+aiWQMse7sjTJ3DD/6JaFoxtqSb0Y2088RnnaH+1 qGzWIPCWvRqUwaKIxDUIMGG/+RRw2vcdj5w2TAGVlWYjdb5UL5gQuhC4bsR76YevUE qcMPJQCdnQs/owVWkwDW/Ho5IzM/W26Vhr+gydewcZtoP6z/9EWyAiNnOc8mBRducK AiIqupciZFfnW+cJVeoB5ewBy+ibo1+k38gKmZWv51xuAylIQvj9KENrXjw4YISQ0z HoeVrMnHircsrRPFaXXEs3kjaClm73RQwu5zSPcefzAapyeG/yyldEDraJqoFz8cLR 0iqNsArtWAg9g== Subject: Re: [PATCH 3/3] phy: mediatek: phy-mtk-mipi-dsi: Simplify with dev_err_probe() To: Chunfeng Yun , chunkuang.hu@kernel.org Cc: p.zabel@pengutronix.de, kishon@ti.com, vkoul@kernel.org, matthias.bgg@gmail.com, dri-devel@lists.freedesktop.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org References: <20220103145324.48008-1-angelogioacchino.delregno@collabora.com> <20220103145324.48008-3-angelogioacchino.delregno@collabora.com> <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> From: AngeloGioacchino Del Regno Message-ID: Date: Fri, 7 Jan 2022 10:23:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Il 06/01/22 10:13, Chunfeng Yun ha scritto: > On Mon, 2022-01-03 at 15:53 +0100, AngeloGioacchino Del Regno wrote: >> Use the dev_err_probe() helper to simplify error handling during >> probe. >> >> Signed-off-by: AngeloGioacchino Del Regno < >> angelogioacchino.delregno@collabora.com> >> --- >> drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 29 +++++++++------------ >> ---- >> 1 file changed, 10 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> index 6f7425b0bf5b..4b77508f5241 100644 >> --- a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> +++ b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> @@ -148,11 +148,9 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> return PTR_ERR(mipi_tx->regmap); >> >> ref_clk = devm_clk_get(dev, NULL); >> - if (IS_ERR(ref_clk)) { >> - ret = PTR_ERR(ref_clk); >> - dev_err(dev, "Failed to get reference clock: %d\n", >> ret); >> - return ret; >> - } >> + if (IS_ERR(ref_clk)) >> + return dev_err_probe(dev, PTR_ERR(ref_clk), >> + "Failed to get reference >> clock\n"); >> >> ret = of_property_read_u32(dev->of_node, "drive-strength- >> microamp", >> &mipi_tx->mipitx_drive); >> @@ -172,27 +170,20 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> >> ret = of_property_read_string(dev->of_node, "clock-output- >> names", >> &clk_init.name); >> - if (ret < 0) { >> - dev_err(dev, "Failed to read clock-output-names: %d\n", >> ret); >> - return ret; >> - } >> + if (ret < 0) >> + return dev_err_probe(dev, ret, "Failed to read clock- >> output-names\n"); > Seems no need change it here. (no EPROBE_DEFER error) > Hello Chunfeng, pasting from kernel driver-api infrastructure guidelines: [...]Note that it is deemed acceptable to use this function for error prints during probe even if the err is known to never be -EPROBE_DEFER. The benefit compared to a normal dev_err() is the standardized format of the error code and the fact that the error code is returned. https://www.kernel.org/doc/html/latest/driver-api/infrastructure.html Regards, - Angelo > Thanks >> >> clk_init.ops = mipi_tx->driver_data->mipi_tx_clk_ops; >> >> mipi_tx->pll_hw.init = &clk_init; >> mipi_tx->pll = devm_clk_register(dev, &mipi_tx->pll_hw); >> - if (IS_ERR(mipi_tx->pll)) { >> - ret = PTR_ERR(mipi_tx->pll); >> - dev_err(dev, "Failed to register PLL: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(mipi_tx->pll)) >> + return dev_err_probe(dev, PTR_ERR(mipi_tx->pll), >> "Cannot register PLL\n"); >> >> phy = devm_phy_create(dev, NULL, &mtk_mipi_tx_ops); >> - if (IS_ERR(phy)) { >> - ret = PTR_ERR(phy); >> - dev_err(dev, "Failed to create MIPI D-PHY: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(phy)) >> + return dev_err_probe(dev, PTR_ERR(phy), "Failed to >> create MIPI D-PHY\n"); >> + >> phy_set_drvdata(phy, mipi_tx); >> >> phy_provider = devm_of_phy_provider_register(dev, >> of_phy_simple_xlate); > 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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 53BBEC433F5 for ; Fri, 7 Jan 2022 09:23:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=ifVYb48v2BrfMKcniq1EcU8e4fiIaKq4LmnShxAVmDM=; b=Zld/AJgrAeZUryoXtai9r99A1q ku1N3ad869llBG88FxblzirV/FAkNe05Gvh3zX9uLYshouufjGU4nw1/AV+4nJDpcCOGcrDorpFJV 0lbxYWxDFzxsmZ2vvdGJLuoB9Snq5NQFkyUuoR4omTzGcJsrghNIXlqTeRruz4X1VXErMoK08Ze9i zos1qrn2E+u2cFeC0qAb3s5Gtvl8x5J65vZHYcNGBg34L7fLeVMuteI4Asxj2HfNZel0zMnqut5yD 6RoQe7ou/aRgItC8Ahg13kY6+x+48xOi0veD02DLtRCGXw9xtJRY9DodjQSk+P5c+XTQycIKC3NHJ yrrGiM8A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTS-00333d-AR; Fri, 07 Jan 2022 09:23:50 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTF-003306-GP; Fri, 07 Jan 2022 09:23:39 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DD0E51F40A55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1641547414; bh=wj1ZDK4lkn/iSMxOoGczIJTSg6wOxjiARPaZ2r2OUWw=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c7TocaBKZAHN+HhlzCoPRaklI+aiWQMse7sjTJ3DD/6JaFoxtqSb0Y2088RnnaH+1 qGzWIPCWvRqUwaKIxDUIMGG/+RRw2vcdj5w2TAGVlWYjdb5UL5gQuhC4bsR76YevUE qcMPJQCdnQs/owVWkwDW/Ho5IzM/W26Vhr+gydewcZtoP6z/9EWyAiNnOc8mBRducK AiIqupciZFfnW+cJVeoB5ewBy+ibo1+k38gKmZWv51xuAylIQvj9KENrXjw4YISQ0z HoeVrMnHircsrRPFaXXEs3kjaClm73RQwu5zSPcefzAapyeG/yyldEDraJqoFz8cLR 0iqNsArtWAg9g== Subject: Re: [PATCH 3/3] phy: mediatek: phy-mtk-mipi-dsi: Simplify with dev_err_probe() To: Chunfeng Yun , chunkuang.hu@kernel.org Cc: p.zabel@pengutronix.de, kishon@ti.com, vkoul@kernel.org, matthias.bgg@gmail.com, dri-devel@lists.freedesktop.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org References: <20220103145324.48008-1-angelogioacchino.delregno@collabora.com> <20220103145324.48008-3-angelogioacchino.delregno@collabora.com> <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> From: AngeloGioacchino Del Regno Message-ID: Date: Fri, 7 Jan 2022 10:23:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220107_012337_736663_04DDE1EA X-CRM114-Status: GOOD ( 17.24 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 06/01/22 10:13, Chunfeng Yun ha scritto: > On Mon, 2022-01-03 at 15:53 +0100, AngeloGioacchino Del Regno wrote: >> Use the dev_err_probe() helper to simplify error handling during >> probe. >> >> Signed-off-by: AngeloGioacchino Del Regno < >> angelogioacchino.delregno@collabora.com> >> --- >> drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 29 +++++++++------------ >> ---- >> 1 file changed, 10 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> index 6f7425b0bf5b..4b77508f5241 100644 >> --- a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> +++ b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> @@ -148,11 +148,9 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> return PTR_ERR(mipi_tx->regmap); >> >> ref_clk = devm_clk_get(dev, NULL); >> - if (IS_ERR(ref_clk)) { >> - ret = PTR_ERR(ref_clk); >> - dev_err(dev, "Failed to get reference clock: %d\n", >> ret); >> - return ret; >> - } >> + if (IS_ERR(ref_clk)) >> + return dev_err_probe(dev, PTR_ERR(ref_clk), >> + "Failed to get reference >> clock\n"); >> >> ret = of_property_read_u32(dev->of_node, "drive-strength- >> microamp", >> &mipi_tx->mipitx_drive); >> @@ -172,27 +170,20 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> >> ret = of_property_read_string(dev->of_node, "clock-output- >> names", >> &clk_init.name); >> - if (ret < 0) { >> - dev_err(dev, "Failed to read clock-output-names: %d\n", >> ret); >> - return ret; >> - } >> + if (ret < 0) >> + return dev_err_probe(dev, ret, "Failed to read clock- >> output-names\n"); > Seems no need change it here. (no EPROBE_DEFER error) > Hello Chunfeng, pasting from kernel driver-api infrastructure guidelines: [...]Note that it is deemed acceptable to use this function for error prints during probe even if the err is known to never be -EPROBE_DEFER. The benefit compared to a normal dev_err() is the standardized format of the error code and the fact that the error code is returned. https://www.kernel.org/doc/html/latest/driver-api/infrastructure.html Regards, - Angelo > Thanks >> >> clk_init.ops = mipi_tx->driver_data->mipi_tx_clk_ops; >> >> mipi_tx->pll_hw.init = &clk_init; >> mipi_tx->pll = devm_clk_register(dev, &mipi_tx->pll_hw); >> - if (IS_ERR(mipi_tx->pll)) { >> - ret = PTR_ERR(mipi_tx->pll); >> - dev_err(dev, "Failed to register PLL: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(mipi_tx->pll)) >> + return dev_err_probe(dev, PTR_ERR(mipi_tx->pll), >> "Cannot register PLL\n"); >> >> phy = devm_phy_create(dev, NULL, &mtk_mipi_tx_ops); >> - if (IS_ERR(phy)) { >> - ret = PTR_ERR(phy); >> - dev_err(dev, "Failed to create MIPI D-PHY: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(phy)) >> + return dev_err_probe(dev, PTR_ERR(phy), "Failed to >> create MIPI D-PHY\n"); >> + >> phy_set_drvdata(phy, mipi_tx); >> >> phy_provider = devm_of_phy_provider_register(dev, >> of_phy_simple_xlate); > _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek 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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 38275C433EF for ; Fri, 7 Jan 2022 09:23:38 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 61D7C11B38F; Fri, 7 Jan 2022 09:23:37 +0000 (UTC) Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [46.235.227.227]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8A00611B397 for ; Fri, 7 Jan 2022 09:23:35 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DD0E51F40A55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1641547414; bh=wj1ZDK4lkn/iSMxOoGczIJTSg6wOxjiARPaZ2r2OUWw=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c7TocaBKZAHN+HhlzCoPRaklI+aiWQMse7sjTJ3DD/6JaFoxtqSb0Y2088RnnaH+1 qGzWIPCWvRqUwaKIxDUIMGG/+RRw2vcdj5w2TAGVlWYjdb5UL5gQuhC4bsR76YevUE qcMPJQCdnQs/owVWkwDW/Ho5IzM/W26Vhr+gydewcZtoP6z/9EWyAiNnOc8mBRducK AiIqupciZFfnW+cJVeoB5ewBy+ibo1+k38gKmZWv51xuAylIQvj9KENrXjw4YISQ0z HoeVrMnHircsrRPFaXXEs3kjaClm73RQwu5zSPcefzAapyeG/yyldEDraJqoFz8cLR 0iqNsArtWAg9g== Subject: Re: [PATCH 3/3] phy: mediatek: phy-mtk-mipi-dsi: Simplify with dev_err_probe() To: Chunfeng Yun , chunkuang.hu@kernel.org References: <20220103145324.48008-1-angelogioacchino.delregno@collabora.com> <20220103145324.48008-3-angelogioacchino.delregno@collabora.com> <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> From: AngeloGioacchino Del Regno Message-ID: Date: Fri, 7 Jan 2022 10:23:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit 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: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, kishon@ti.com, linux-phy@lists.infradead.org, vkoul@kernel.org, linux-mediatek@lists.infradead.org, matthias.bgg@gmail.com, linux-arm-kernel@lists.infradead.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Il 06/01/22 10:13, Chunfeng Yun ha scritto: > On Mon, 2022-01-03 at 15:53 +0100, AngeloGioacchino Del Regno wrote: >> Use the dev_err_probe() helper to simplify error handling during >> probe. >> >> Signed-off-by: AngeloGioacchino Del Regno < >> angelogioacchino.delregno@collabora.com> >> --- >> drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 29 +++++++++------------ >> ---- >> 1 file changed, 10 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> index 6f7425b0bf5b..4b77508f5241 100644 >> --- a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> +++ b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> @@ -148,11 +148,9 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> return PTR_ERR(mipi_tx->regmap); >> >> ref_clk = devm_clk_get(dev, NULL); >> - if (IS_ERR(ref_clk)) { >> - ret = PTR_ERR(ref_clk); >> - dev_err(dev, "Failed to get reference clock: %d\n", >> ret); >> - return ret; >> - } >> + if (IS_ERR(ref_clk)) >> + return dev_err_probe(dev, PTR_ERR(ref_clk), >> + "Failed to get reference >> clock\n"); >> >> ret = of_property_read_u32(dev->of_node, "drive-strength- >> microamp", >> &mipi_tx->mipitx_drive); >> @@ -172,27 +170,20 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> >> ret = of_property_read_string(dev->of_node, "clock-output- >> names", >> &clk_init.name); >> - if (ret < 0) { >> - dev_err(dev, "Failed to read clock-output-names: %d\n", >> ret); >> - return ret; >> - } >> + if (ret < 0) >> + return dev_err_probe(dev, ret, "Failed to read clock- >> output-names\n"); > Seems no need change it here. (no EPROBE_DEFER error) > Hello Chunfeng, pasting from kernel driver-api infrastructure guidelines: [...]Note that it is deemed acceptable to use this function for error prints during probe even if the err is known to never be -EPROBE_DEFER. The benefit compared to a normal dev_err() is the standardized format of the error code and the fact that the error code is returned. https://www.kernel.org/doc/html/latest/driver-api/infrastructure.html Regards, - Angelo > Thanks >> >> clk_init.ops = mipi_tx->driver_data->mipi_tx_clk_ops; >> >> mipi_tx->pll_hw.init = &clk_init; >> mipi_tx->pll = devm_clk_register(dev, &mipi_tx->pll_hw); >> - if (IS_ERR(mipi_tx->pll)) { >> - ret = PTR_ERR(mipi_tx->pll); >> - dev_err(dev, "Failed to register PLL: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(mipi_tx->pll)) >> + return dev_err_probe(dev, PTR_ERR(mipi_tx->pll), >> "Cannot register PLL\n"); >> >> phy = devm_phy_create(dev, NULL, &mtk_mipi_tx_ops); >> - if (IS_ERR(phy)) { >> - ret = PTR_ERR(phy); >> - dev_err(dev, "Failed to create MIPI D-PHY: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(phy)) >> + return dev_err_probe(dev, PTR_ERR(phy), "Failed to >> create MIPI D-PHY\n"); >> + >> phy_set_drvdata(phy, mipi_tx); >> >> phy_provider = devm_of_phy_provider_register(dev, >> of_phy_simple_xlate); > 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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A7052C433EF for ; Fri, 7 Jan 2022 09:23:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=phCH5gCejUIbwuYIq/WgYYeNmSEFbV44BUgHIoQ+e7s=; b=esk6ruFd8YdSJLyvoptPWGgRJD v6NzsQbtCjtpVbC0sgujW6iTjbysjofCtz0DKdbhyMOle3fVzplLpSE0P+v16BufQh0wDVbhcJrWD 185jQkSTa5AJ92dUhdL3dYol0a2omXOQ5ADsltVmgnC5L6jviWc/24xG2JXACgGXQj5UQvk0XqrAR LAF7BSNmHivy73P0TxdI361tOd2ezcKv1gq34VT0RqXpyfFdLljoDg7zppzg64BXC6YW1pR3e7lEr EDiqmVRxKxTZdQVsTprBn8ZiFKcRR6oLeUkOg4H23GqZZ1kGqA3vXq7BHrFDTXce7EksYVOS74o+R OGb64bRw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTT-00333m-2F; Fri, 07 Jan 2022 09:23:51 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTF-003306-GP; Fri, 07 Jan 2022 09:23:39 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DD0E51F40A55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1641547414; bh=wj1ZDK4lkn/iSMxOoGczIJTSg6wOxjiARPaZ2r2OUWw=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c7TocaBKZAHN+HhlzCoPRaklI+aiWQMse7sjTJ3DD/6JaFoxtqSb0Y2088RnnaH+1 qGzWIPCWvRqUwaKIxDUIMGG/+RRw2vcdj5w2TAGVlWYjdb5UL5gQuhC4bsR76YevUE qcMPJQCdnQs/owVWkwDW/Ho5IzM/W26Vhr+gydewcZtoP6z/9EWyAiNnOc8mBRducK AiIqupciZFfnW+cJVeoB5ewBy+ibo1+k38gKmZWv51xuAylIQvj9KENrXjw4YISQ0z HoeVrMnHircsrRPFaXXEs3kjaClm73RQwu5zSPcefzAapyeG/yyldEDraJqoFz8cLR 0iqNsArtWAg9g== Subject: Re: [PATCH 3/3] phy: mediatek: phy-mtk-mipi-dsi: Simplify with dev_err_probe() To: Chunfeng Yun , chunkuang.hu@kernel.org Cc: p.zabel@pengutronix.de, kishon@ti.com, vkoul@kernel.org, matthias.bgg@gmail.com, dri-devel@lists.freedesktop.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org References: <20220103145324.48008-1-angelogioacchino.delregno@collabora.com> <20220103145324.48008-3-angelogioacchino.delregno@collabora.com> <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> From: AngeloGioacchino Del Regno Message-ID: Date: Fri, 7 Jan 2022 10:23:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220107_012337_736663_04DDE1EA X-CRM114-Status: GOOD ( 17.24 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org Il 06/01/22 10:13, Chunfeng Yun ha scritto: > On Mon, 2022-01-03 at 15:53 +0100, AngeloGioacchino Del Regno wrote: >> Use the dev_err_probe() helper to simplify error handling during >> probe. >> >> Signed-off-by: AngeloGioacchino Del Regno < >> angelogioacchino.delregno@collabora.com> >> --- >> drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 29 +++++++++------------ >> ---- >> 1 file changed, 10 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> index 6f7425b0bf5b..4b77508f5241 100644 >> --- a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> +++ b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> @@ -148,11 +148,9 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> return PTR_ERR(mipi_tx->regmap); >> >> ref_clk = devm_clk_get(dev, NULL); >> - if (IS_ERR(ref_clk)) { >> - ret = PTR_ERR(ref_clk); >> - dev_err(dev, "Failed to get reference clock: %d\n", >> ret); >> - return ret; >> - } >> + if (IS_ERR(ref_clk)) >> + return dev_err_probe(dev, PTR_ERR(ref_clk), >> + "Failed to get reference >> clock\n"); >> >> ret = of_property_read_u32(dev->of_node, "drive-strength- >> microamp", >> &mipi_tx->mipitx_drive); >> @@ -172,27 +170,20 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> >> ret = of_property_read_string(dev->of_node, "clock-output- >> names", >> &clk_init.name); >> - if (ret < 0) { >> - dev_err(dev, "Failed to read clock-output-names: %d\n", >> ret); >> - return ret; >> - } >> + if (ret < 0) >> + return dev_err_probe(dev, ret, "Failed to read clock- >> output-names\n"); > Seems no need change it here. (no EPROBE_DEFER error) > Hello Chunfeng, pasting from kernel driver-api infrastructure guidelines: [...]Note that it is deemed acceptable to use this function for error prints during probe even if the err is known to never be -EPROBE_DEFER. The benefit compared to a normal dev_err() is the standardized format of the error code and the fact that the error code is returned. https://www.kernel.org/doc/html/latest/driver-api/infrastructure.html Regards, - Angelo > Thanks >> >> clk_init.ops = mipi_tx->driver_data->mipi_tx_clk_ops; >> >> mipi_tx->pll_hw.init = &clk_init; >> mipi_tx->pll = devm_clk_register(dev, &mipi_tx->pll_hw); >> - if (IS_ERR(mipi_tx->pll)) { >> - ret = PTR_ERR(mipi_tx->pll); >> - dev_err(dev, "Failed to register PLL: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(mipi_tx->pll)) >> + return dev_err_probe(dev, PTR_ERR(mipi_tx->pll), >> "Cannot register PLL\n"); >> >> phy = devm_phy_create(dev, NULL, &mtk_mipi_tx_ops); >> - if (IS_ERR(phy)) { >> - ret = PTR_ERR(phy); >> - dev_err(dev, "Failed to create MIPI D-PHY: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(phy)) >> + return dev_err_probe(dev, PTR_ERR(phy), "Failed to >> create MIPI D-PHY\n"); >> + >> phy_set_drvdata(phy, mipi_tx); >> >> phy_provider = devm_of_phy_provider_register(dev, >> of_phy_simple_xlate); > -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy 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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DFEABC4332F for ; Fri, 7 Jan 2022 09:25:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=GwP5VochJiEyE6o794DCtQpxYkFg4uJlAS3/F2oywK8=; b=VaUSZmi+poffalDe8XPJnuDwEK dAE/r7V4XUOSuEX4c71VOTGFrw9evbbL5KOsw/mt2lCfi0WYgr3q+kb9bJSnBeU7PdL9tNBH4G/G4 qFYi0OEGYxQWKEVXXYLugKOMUuDZH3R4CxqH2FlVFr37fu6OWJeeAJlL+EGSfBFdgVA5lxTvYXYyI 8GVnD39KcwcrNVFZsdA34564wECl7qLFuj4NrKiL2sJ42KcSCyfo9oR0YRBOPKVs48VD7rAolO+Te eM67vfxTLSgETXe+DWBj/RYSWtqM9rEYbXVMJNaEdwQ0cyBiFHISqTxnjapr2GuDDHjciFpWSjtMv Qj2JfKqg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTJ-003324-R3; Fri, 07 Jan 2022 09:23:41 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1n5lTF-003306-GP; Fri, 07 Jan 2022 09:23:39 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DD0E51F40A55 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1641547414; bh=wj1ZDK4lkn/iSMxOoGczIJTSg6wOxjiARPaZ2r2OUWw=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=c7TocaBKZAHN+HhlzCoPRaklI+aiWQMse7sjTJ3DD/6JaFoxtqSb0Y2088RnnaH+1 qGzWIPCWvRqUwaKIxDUIMGG/+RRw2vcdj5w2TAGVlWYjdb5UL5gQuhC4bsR76YevUE qcMPJQCdnQs/owVWkwDW/Ho5IzM/W26Vhr+gydewcZtoP6z/9EWyAiNnOc8mBRducK AiIqupciZFfnW+cJVeoB5ewBy+ibo1+k38gKmZWv51xuAylIQvj9KENrXjw4YISQ0z HoeVrMnHircsrRPFaXXEs3kjaClm73RQwu5zSPcefzAapyeG/yyldEDraJqoFz8cLR 0iqNsArtWAg9g== Subject: Re: [PATCH 3/3] phy: mediatek: phy-mtk-mipi-dsi: Simplify with dev_err_probe() To: Chunfeng Yun , chunkuang.hu@kernel.org Cc: p.zabel@pengutronix.de, kishon@ti.com, vkoul@kernel.org, matthias.bgg@gmail.com, dri-devel@lists.freedesktop.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, linux-kernel@vger.kernel.org References: <20220103145324.48008-1-angelogioacchino.delregno@collabora.com> <20220103145324.48008-3-angelogioacchino.delregno@collabora.com> <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> From: AngeloGioacchino Del Regno Message-ID: Date: Fri, 7 Jan 2022 10:23:32 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <4d7195c3ac9bc63a5f980548f0c880484403346d.camel@mediatek.com> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220107_012337_736663_04DDE1EA X-CRM114-Status: GOOD ( 17.24 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Il 06/01/22 10:13, Chunfeng Yun ha scritto: > On Mon, 2022-01-03 at 15:53 +0100, AngeloGioacchino Del Regno wrote: >> Use the dev_err_probe() helper to simplify error handling during >> probe. >> >> Signed-off-by: AngeloGioacchino Del Regno < >> angelogioacchino.delregno@collabora.com> >> --- >> drivers/phy/mediatek/phy-mtk-mipi-dsi.c | 29 +++++++++------------ >> ---- >> 1 file changed, 10 insertions(+), 19 deletions(-) >> >> diff --git a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> index 6f7425b0bf5b..4b77508f5241 100644 >> --- a/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> +++ b/drivers/phy/mediatek/phy-mtk-mipi-dsi.c >> @@ -148,11 +148,9 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> return PTR_ERR(mipi_tx->regmap); >> >> ref_clk = devm_clk_get(dev, NULL); >> - if (IS_ERR(ref_clk)) { >> - ret = PTR_ERR(ref_clk); >> - dev_err(dev, "Failed to get reference clock: %d\n", >> ret); >> - return ret; >> - } >> + if (IS_ERR(ref_clk)) >> + return dev_err_probe(dev, PTR_ERR(ref_clk), >> + "Failed to get reference >> clock\n"); >> >> ret = of_property_read_u32(dev->of_node, "drive-strength- >> microamp", >> &mipi_tx->mipitx_drive); >> @@ -172,27 +170,20 @@ static int mtk_mipi_tx_probe(struct >> platform_device *pdev) >> >> ret = of_property_read_string(dev->of_node, "clock-output- >> names", >> &clk_init.name); >> - if (ret < 0) { >> - dev_err(dev, "Failed to read clock-output-names: %d\n", >> ret); >> - return ret; >> - } >> + if (ret < 0) >> + return dev_err_probe(dev, ret, "Failed to read clock- >> output-names\n"); > Seems no need change it here. (no EPROBE_DEFER error) > Hello Chunfeng, pasting from kernel driver-api infrastructure guidelines: [...]Note that it is deemed acceptable to use this function for error prints during probe even if the err is known to never be -EPROBE_DEFER. The benefit compared to a normal dev_err() is the standardized format of the error code and the fact that the error code is returned. https://www.kernel.org/doc/html/latest/driver-api/infrastructure.html Regards, - Angelo > Thanks >> >> clk_init.ops = mipi_tx->driver_data->mipi_tx_clk_ops; >> >> mipi_tx->pll_hw.init = &clk_init; >> mipi_tx->pll = devm_clk_register(dev, &mipi_tx->pll_hw); >> - if (IS_ERR(mipi_tx->pll)) { >> - ret = PTR_ERR(mipi_tx->pll); >> - dev_err(dev, "Failed to register PLL: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(mipi_tx->pll)) >> + return dev_err_probe(dev, PTR_ERR(mipi_tx->pll), >> "Cannot register PLL\n"); >> >> phy = devm_phy_create(dev, NULL, &mtk_mipi_tx_ops); >> - if (IS_ERR(phy)) { >> - ret = PTR_ERR(phy); >> - dev_err(dev, "Failed to create MIPI D-PHY: %d\n", ret); >> - return ret; >> - } >> + if (IS_ERR(phy)) >> + return dev_err_probe(dev, PTR_ERR(phy), "Failed to >> create MIPI D-PHY\n"); >> + >> phy_set_drvdata(phy, mipi_tx); >> >> phy_provider = devm_of_phy_provider_register(dev, >> of_phy_simple_xlate); > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel