2023-05-31 06:50:57

by Chen-Yu Tsai

[permalink] [raw]
Subject: [PATCH] soc: mediatek: SVS: Fix MT8192 GPU node name

Device tree node names should be generic. The planned device node name
for the GPU, according to the bindings and posted DT changes, is "gpu",
not "mali".

Fix the GPU node name in the SVS driver to follow.

Fixes: 0bbb09b2af9d ("soc: mediatek: SVS: add mt8192 SVS GPU driver")
Signed-off-by: Chen-Yu Tsai <[email protected]>
---
drivers/soc/mediatek/mtk-svs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/soc/mediatek/mtk-svs.c b/drivers/soc/mediatek/mtk-svs.c
index 81585733c8a9..3a2f97cd5272 100644
--- a/drivers/soc/mediatek/mtk-svs.c
+++ b/drivers/soc/mediatek/mtk-svs.c
@@ -2061,9 +2061,9 @@ static int svs_mt8192_platform_probe(struct svs_platform *svsp)
svsb = &svsp->banks[idx];

if (svsb->type == SVSB_HIGH)
- svsb->opp_dev = svs_add_device_link(svsp, "mali");
+ svsb->opp_dev = svs_add_device_link(svsp, "gpu");
else if (svsb->type == SVSB_LOW)
- svsb->opp_dev = svs_get_subsys_device(svsp, "mali");
+ svsb->opp_dev = svs_get_subsys_device(svsp, "gpu");

if (IS_ERR(svsb->opp_dev))
return dev_err_probe(svsp->dev, PTR_ERR(svsb->opp_dev),
--
2.41.0.rc0.172.g3f132b7071-goog



Subject: Re: [PATCH] soc: mediatek: SVS: Fix MT8192 GPU node name

Il 31/05/23 08:35, Chen-Yu Tsai ha scritto:
> Device tree node names should be generic. The planned device node name
> for the GPU, according to the bindings and posted DT changes, is "gpu",
> not "mali".
>
> Fix the GPU node name in the SVS driver to follow.
>
> Fixes: 0bbb09b2af9d ("soc: mediatek: SVS: add mt8192 SVS GPU driver")
> Signed-off-by: Chen-Yu Tsai <[email protected]>

Hahaha, I was about to send the same patch! :D

Reviewed-by: AngeloGioacchino Del Regno <[email protected]>

> ---
> drivers/soc/mediatek/mtk-svs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/soc/mediatek/mtk-svs.c b/drivers/soc/mediatek/mtk-svs.c
> index 81585733c8a9..3a2f97cd5272 100644
> --- a/drivers/soc/mediatek/mtk-svs.c
> +++ b/drivers/soc/mediatek/mtk-svs.c
> @@ -2061,9 +2061,9 @@ static int svs_mt8192_platform_probe(struct svs_platform *svsp)
> svsb = &svsp->banks[idx];
>
> if (svsb->type == SVSB_HIGH)
> - svsb->opp_dev = svs_add_device_link(svsp, "mali");
> + svsb->opp_dev = svs_add_device_link(svsp, "gpu");
> else if (svsb->type == SVSB_LOW)
> - svsb->opp_dev = svs_get_subsys_device(svsp, "mali");
> + svsb->opp_dev = svs_get_subsys_device(svsp, "gpu");
>
> if (IS_ERR(svsb->opp_dev))
> return dev_err_probe(svsp->dev, PTR_ERR(svsb->opp_dev),


2023-05-31 09:25:33

by Alexandre Mergnat

[permalink] [raw]
Subject: Re: [PATCH] soc: mediatek: SVS: Fix MT8192 GPU node name

On 31/05/2023 08:35, Chen-Yu Tsai wrote:
> Device tree node names should be generic. The planned device node name
> for the GPU, according to the bindings and posted DT changes, is "gpu",
> not "mali".
>
> Fix the GPU node name in the SVS driver to follow.

Reviewed-by: Alexandre Mergnat <[email protected]>

--
Regards,
Alexandre


2023-06-09 14:56:53

by Matthias Brugger

[permalink] [raw]
Subject: Re: [PATCH] soc: mediatek: SVS: Fix MT8192 GPU node name



On 31/05/2023 08:35, Chen-Yu Tsai wrote:
> Device tree node names should be generic. The planned device node name
> for the GPU, according to the bindings and posted DT changes, is "gpu",
> not "mali".
>
> Fix the GPU node name in the SVS driver to follow.
>
> Fixes: 0bbb09b2af9d ("soc: mediatek: SVS: add mt8192 SVS GPU driver")
> Signed-off-by: Chen-Yu Tsai <[email protected]>

Applied, thanks

> ---
> drivers/soc/mediatek/mtk-svs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/soc/mediatek/mtk-svs.c b/drivers/soc/mediatek/mtk-svs.c
> index 81585733c8a9..3a2f97cd5272 100644
> --- a/drivers/soc/mediatek/mtk-svs.c
> +++ b/drivers/soc/mediatek/mtk-svs.c
> @@ -2061,9 +2061,9 @@ static int svs_mt8192_platform_probe(struct svs_platform *svsp)
> svsb = &svsp->banks[idx];
>
> if (svsb->type == SVSB_HIGH)
> - svsb->opp_dev = svs_add_device_link(svsp, "mali");
> + svsb->opp_dev = svs_add_device_link(svsp, "gpu");
> else if (svsb->type == SVSB_LOW)
> - svsb->opp_dev = svs_get_subsys_device(svsp, "mali");
> + svsb->opp_dev = svs_get_subsys_device(svsp, "gpu");
>
> if (IS_ERR(svsb->opp_dev))
> return dev_err_probe(svsp->dev, PTR_ERR(svsb->opp_dev),