Message ID | 20230919095938.70679-3-angelogioacchino.delregno@collabora.com (mailing list archive) |
---|---|
State | Changes Requested |
Delegated to: | Hans Verkuil |
Headers |
Received: from vger.kernel.org ([23.128.96.18]) by www.linuxtv.org with esmtp (Exim 4.92) (envelope-from <linux-media-owner@vger.kernel.org>) id 1qiXWV-008TRO-El; Tue, 19 Sep 2023 10:00:03 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230348AbjISKAE (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Tue, 19 Sep 2023 06:00:04 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47052 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231724AbjISJ7w (ORCPT <rfc822;linux-media@vger.kernel.org>); Tue, 19 Sep 2023 05:59:52 -0400 Received: from madras.collabora.co.uk (madras.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e5ab]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D00E2123; Tue, 19 Sep 2023 02:59:46 -0700 (PDT) Received: from IcarusMOD.eternityproject.eu (2-237-20-237.ip236.fastwebnet.it [2.237.20.237]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kholk11) by madras.collabora.co.uk (Postfix) with ESMTPSA id 95BC26607083; Tue, 19 Sep 2023 10:59:44 +0100 (BST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1695117585; bh=ty1ncJ/gw8hrt51lR2U+TFArnXv5dU7HXDPnYcxw+dw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=StKEpJ45dfYg1hZOjzQqqJo3FpCWv8VqfgMPIQo7ZvtnXXpwzhwgrztPa4NeZ8T+y RVe0zeLcKynvJjpeOEr32ul/Abs0xJAhnwLplgr3pK6paFIQoHo+d8BRTxzH5eiIy/ uoj/dHKTDdFsNciQMeVwNuQWAMxOgCYsNlTmqctJA147Sljyda6bWa5UTZ2qvQSR5l xN1cY5vKssh0CdT8wNDCh2jHjufv/peXcgDi8gzlTvsOt9LjIR/nj2x/ayYlNXIS22 monR6h0q5f0N2s9ApuGN13f4S9sUCAfPMmpcq0VRzQKPJRa937S0tCb35egfWi9ScY VULyrDJyyqzNQ== From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> To: mchehab@kernel.org Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, matthias.bgg@gmail.com, angelogioacchino.delregno@collabora.com, moudy.ho@mediatek.com, hverkuil-cisco@xs4all.nl, sakari.ailus@linux.intel.com, u.kleine-koenig@pengutronix.de, linqiheng@huawei.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, kernel@collabora.com, wenst@chromium.org Subject: [PATCH 2/2] media: platform: mtk-mdp3: Use devicetree phandle to retrieve SCP Date: Tue, 19 Sep 2023 11:59:38 +0200 Message-ID: <20230919095938.70679-3-angelogioacchino.delregno@collabora.com> X-Mailer: git-send-email 2.42.0 In-Reply-To: <20230919095938.70679-1-angelogioacchino.delregno@collabora.com> References: <20230919095938.70679-1-angelogioacchino.delregno@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-LSpam-Score: -2.5 (--) X-LSpam-Report: No, score=-2.5 required=5.0 tests=BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1 autolearn=ham autolearn_force=no |
Series |
MediaTek MDP3: use devicetree to retrieve SCP
|
|
Commit Message
AngeloGioacchino Del Regno
Sept. 19, 2023, 9:59 a.m. UTC
Instead of walking the entire parent node for something that has the
right compatible, use the scp_get() function provided by the MediaTek
SCP remoteproc driver to retrieve a handle to mtk_scp through the
devicetree "mediatek,scp" (phandle) property.
In case of multi-core SCP, this also allows to select a specific core.
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
.../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
Comments
On Tue, Sep 19, 2023 at 6:00 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Instead of walking the entire parent node for something that has the > right compatible, use the scp_get() function provided by the MediaTek > SCP remoteproc driver to retrieve a handle to mtk_scp through the > devicetree "mediatek,scp" (phandle) property. > > In case of multi-core SCP, this also allows to select a specific core. > > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 16 ++++++++++------ > 1 file changed, 10 insertions(+), 6 deletions(-) > > diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > index 8677e7fd5083..d93d3833633e 100644 > --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > @@ -254,13 +254,17 @@ static int mdp_probe(struct platform_device *pdev) > goto err_destroy_job_wq; > } > > - mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); > - if (WARN_ON(!mm_pdev)) { > - dev_err(&pdev->dev, "Could not get scp device\n"); > - ret = -ENODEV; > - goto err_destroy_clock_wq; > + mdp->scp = scp_get(pdev); > + if (!mdp->scp) { > + mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); > + if (WARN_ON(!mm_pdev)) { > + dev_err(&pdev->dev, "Could not get scp device\n"); > + ret = -ENODEV; > + goto err_destroy_clock_wq; > + } > + mdp->scp = platform_get_drvdata(mm_pdev); You need to keep the original code as a fallback for old device trees. ChenYu > } > - mdp->scp = platform_get_drvdata(mm_pdev); > + > mdp->rproc_handle = scp_get_rproc(mdp->scp); > dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle); > > -- > 2.42.0 >
Il 19/09/23 12:21, Chen-Yu Tsai ha scritto: > On Tue, Sep 19, 2023 at 6:00 PM AngeloGioacchino Del Regno > <angelogioacchino.delregno@collabora.com> wrote: >> >> Instead of walking the entire parent node for something that has the >> right compatible, use the scp_get() function provided by the MediaTek >> SCP remoteproc driver to retrieve a handle to mtk_scp through the >> devicetree "mediatek,scp" (phandle) property. >> >> In case of multi-core SCP, this also allows to select a specific core. >> >> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> >> --- >> .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 16 ++++++++++------ >> 1 file changed, 10 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >> index 8677e7fd5083..d93d3833633e 100644 >> --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >> +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >> @@ -254,13 +254,17 @@ static int mdp_probe(struct platform_device *pdev) >> goto err_destroy_job_wq; >> } >> >> - mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); >> - if (WARN_ON(!mm_pdev)) { >> - dev_err(&pdev->dev, "Could not get scp device\n"); >> - ret = -ENODEV; >> - goto err_destroy_clock_wq; >> + mdp->scp = scp_get(pdev); >> + if (!mdp->scp) { >> + mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); >> + if (WARN_ON(!mm_pdev)) { >> + dev_err(&pdev->dev, "Could not get scp device\n"); >> + ret = -ENODEV; >> + goto err_destroy_clock_wq; >> + } >> + mdp->scp = platform_get_drvdata(mm_pdev); > > You need to keep the original code as a fallback for old device trees. > I haven't removed the original code, it *is* there as a fallback :-) mdp->scp = scp_get() <--- new if (!mdp->scp) { fallback } Regards, Angelo > ChenYu > >> } >> - mdp->scp = platform_get_drvdata(mm_pdev); >> + >> mdp->rproc_handle = scp_get_rproc(mdp->scp); >> dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle); >> >> -- >> 2.42.0 >> > _______________________________________________ > Kernel mailing list -- kernel@mailman.collabora.com > To unsubscribe send an email to kernel-leave@mailman.collabora.com
On Tue, Sep 19, 2023 at 6:24 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Il 19/09/23 12:21, Chen-Yu Tsai ha scritto: > > On Tue, Sep 19, 2023 at 6:00 PM AngeloGioacchino Del Regno > > <angelogioacchino.delregno@collabora.com> wrote: > >> > >> Instead of walking the entire parent node for something that has the > >> right compatible, use the scp_get() function provided by the MediaTek > >> SCP remoteproc driver to retrieve a handle to mtk_scp through the > >> devicetree "mediatek,scp" (phandle) property. > >> > >> In case of multi-core SCP, this also allows to select a specific core. > >> > >> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > >> --- > >> .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 16 ++++++++++------ > >> 1 file changed, 10 insertions(+), 6 deletions(-) > >> > >> diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > >> index 8677e7fd5083..d93d3833633e 100644 > >> --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > >> +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c > >> @@ -254,13 +254,17 @@ static int mdp_probe(struct platform_device *pdev) > >> goto err_destroy_job_wq; > >> } > >> > >> - mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); > >> - if (WARN_ON(!mm_pdev)) { > >> - dev_err(&pdev->dev, "Could not get scp device\n"); > >> - ret = -ENODEV; > >> - goto err_destroy_clock_wq; > >> + mdp->scp = scp_get(pdev); > >> + if (!mdp->scp) { > >> + mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); > >> + if (WARN_ON(!mm_pdev)) { > >> + dev_err(&pdev->dev, "Could not get scp device\n"); > >> + ret = -ENODEV; > >> + goto err_destroy_clock_wq; > >> + } > >> + mdp->scp = platform_get_drvdata(mm_pdev); > > > > You need to keep the original code as a fallback for old device trees. > > > > I haven't removed the original code, it *is* there as a fallback :-) > > mdp->scp = scp_get() <--- new > if (!mdp->scp) { fallback } I see it now. I guess it's time to call it a day... I even replied with the wrong email ... Reviewed-by: Chen-Yu Tsai <wenst@chromium.org> > > ChenYu > > > >> } > >> - mdp->scp = platform_get_drvdata(mm_pdev); > >> + > >> mdp->rproc_handle = scp_get_rproc(mdp->scp); > >> dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle); > >> > >> -- > >> 2.42.0 > >> > > _______________________________________________ > > Kernel mailing list -- kernel@mailman.collabora.com > > To unsubscribe send an email to kernel-leave@mailman.collabora.com > >
Il 19/09/23 12:26, Chen-Yu Tsai ha scritto: > On Tue, Sep 19, 2023 at 6:24 PM AngeloGioacchino Del Regno > <angelogioacchino.delregno@collabora.com> wrote: >> >> Il 19/09/23 12:21, Chen-Yu Tsai ha scritto: >>> On Tue, Sep 19, 2023 at 6:00 PM AngeloGioacchino Del Regno >>> <angelogioacchino.delregno@collabora.com> wrote: >>>> >>>> Instead of walking the entire parent node for something that has the >>>> right compatible, use the scp_get() function provided by the MediaTek >>>> SCP remoteproc driver to retrieve a handle to mtk_scp through the >>>> devicetree "mediatek,scp" (phandle) property. >>>> >>>> In case of multi-core SCP, this also allows to select a specific core. >>>> >>>> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> >>>> --- >>>> .../media/platform/mediatek/mdp3/mtk-mdp3-core.c | 16 ++++++++++------ >>>> 1 file changed, 10 insertions(+), 6 deletions(-) >>>> >>>> diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >>>> index 8677e7fd5083..d93d3833633e 100644 >>>> --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >>>> +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c >>>> @@ -254,13 +254,17 @@ static int mdp_probe(struct platform_device *pdev) >>>> goto err_destroy_job_wq; >>>> } >>>> >>>> - mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); >>>> - if (WARN_ON(!mm_pdev)) { >>>> - dev_err(&pdev->dev, "Could not get scp device\n"); >>>> - ret = -ENODEV; >>>> - goto err_destroy_clock_wq; >>>> + mdp->scp = scp_get(pdev); >>>> + if (!mdp->scp) { >>>> + mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); >>>> + if (WARN_ON(!mm_pdev)) { >>>> + dev_err(&pdev->dev, "Could not get scp device\n"); >>>> + ret = -ENODEV; >>>> + goto err_destroy_clock_wq; >>>> + } >>>> + mdp->scp = platform_get_drvdata(mm_pdev); >>> >>> You need to keep the original code as a fallback for old device trees. >>> >> >> I haven't removed the original code, it *is* there as a fallback :-) >> >> mdp->scp = scp_get() <--- new >> if (!mdp->scp) { fallback } > > I see it now. I guess it's time to call it a day... I even replied with > the wrong email ... > > Reviewed-by: Chen-Yu Tsai <wenst@chromium.org> hahaha! no worries, it happens :-) Thanks for the review btw. Cheers! Angelo > >>> ChenYu >>> >>>> } >>>> - mdp->scp = platform_get_drvdata(mm_pdev); >>>> + >>>> mdp->rproc_handle = scp_get_rproc(mdp->scp); >>>> dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle); >>>> >>>> -- >>>> 2.42.0 >>>> >>> _______________________________________________ >>> Kernel mailing list -- kernel@mailman.collabora.com >>> To unsubscribe send an email to kernel-leave@mailman.collabora.com >> >>
diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c index 8677e7fd5083..d93d3833633e 100644 --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-core.c @@ -254,13 +254,17 @@ static int mdp_probe(struct platform_device *pdev) goto err_destroy_job_wq; } - mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); - if (WARN_ON(!mm_pdev)) { - dev_err(&pdev->dev, "Could not get scp device\n"); - ret = -ENODEV; - goto err_destroy_clock_wq; + mdp->scp = scp_get(pdev); + if (!mdp->scp) { + mm_pdev = __get_pdev_by_id(pdev, NULL, MDP_INFRA_SCP); + if (WARN_ON(!mm_pdev)) { + dev_err(&pdev->dev, "Could not get scp device\n"); + ret = -ENODEV; + goto err_destroy_clock_wq; + } + mdp->scp = platform_get_drvdata(mm_pdev); } - mdp->scp = platform_get_drvdata(mm_pdev); + mdp->rproc_handle = scp_get_rproc(mdp->scp); dev_dbg(&pdev->dev, "MDP rproc_handle: %pK", mdp->rproc_handle);