linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name
@ 2020-04-14  3:08 Hsin-Yi Wang
  2020-04-14  3:08 ` [PATCH 2/2] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Hsin-Yi Wang @ 2020-04-14  3:08 UTC (permalink / raw)
  To: linux-arm-kernel, linux-mediatek
  Cc: Minghsiu Tsai, Houlong Wei, Andrew-CT Chen,
	Mauro Carvalho Chehab, Matthias Brugger, linux-media,
	linux-kernel, Rob Herring, devicetree

Fix warning:
Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-'

Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
---
 arch/arm64/boot/dts/mediatek/mt8173.dtsi | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
index a212bf124e81..d1e9c41004b4 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
@@ -42,14 +42,14 @@ aliases {
 		dpi0 = &dpi0;
 		dsi0 = &dsi0;
 		dsi1 = &dsi1;
-		mdp_rdma0 = &mdp_rdma0;
-		mdp_rdma1 = &mdp_rdma1;
-		mdp_rsz0 = &mdp_rsz0;
-		mdp_rsz1 = &mdp_rsz1;
-		mdp_rsz2 = &mdp_rsz2;
-		mdp_wdma0 = &mdp_wdma0;
-		mdp_wrot0 = &mdp_wrot0;
-		mdp_wrot1 = &mdp_wrot1;
+		mdp-rdma0 = &mdp_rdma0;
+		mdp-rdma1 = &mdp_rdma1;
+		mdp-rsz0 = &mdp_rsz0;
+		mdp-rsz1 = &mdp_rsz1;
+		mdp-rsz2 = &mdp_rsz2;
+		mdp-wdma0 = &mdp_wdma0;
+		mdp-wrot0 = &mdp_wrot0;
+		mdp-wrot1 = &mdp_wrot1;
 		serial0 = &uart0;
 		serial1 = &uart1;
 		serial2 = &uart2;
-- 
2.26.0.110.g2183baf09c-goog


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/2] media: mtk-mdp: Use correct aliases name
  2020-04-14  3:08 [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Hsin-Yi Wang
@ 2020-04-14  3:08 ` Hsin-Yi Wang
  2020-04-14 23:18   ` Chun-Kuang Hu
  2020-04-14 23:17 ` [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Chun-Kuang Hu
  2020-04-16  8:41 ` Hans Verkuil
  2 siblings, 1 reply; 6+ messages in thread
From: Hsin-Yi Wang @ 2020-04-14  3:08 UTC (permalink / raw)
  To: linux-arm-kernel, linux-mediatek
  Cc: Minghsiu Tsai, Houlong Wei, Andrew-CT Chen,
	Mauro Carvalho Chehab, Matthias Brugger, linux-media,
	linux-kernel, Rob Herring, devicetree

aliases property name must include only lowercase and '-'. Fix in dts
and driver.

Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
---
 drivers/media/platform/mtk-mdp/mtk_mdp_comp.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c b/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
index 14991685adb7..58abfbdfb82d 100644
--- a/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
+++ b/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
@@ -15,10 +15,10 @@
 
 
 static const char * const mtk_mdp_comp_stem[MTK_MDP_COMP_TYPE_MAX] = {
-	"mdp_rdma",
-	"mdp_rsz",
-	"mdp_wdma",
-	"mdp_wrot",
+	"mdp-rdma",
+	"mdp-rsz",
+	"mdp-wdma",
+	"mdp-wrot",
 };
 
 struct mtk_mdp_comp_match {
-- 
2.26.0.110.g2183baf09c-goog


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name
  2020-04-14  3:08 [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Hsin-Yi Wang
  2020-04-14  3:08 ` [PATCH 2/2] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
@ 2020-04-14 23:17 ` Chun-Kuang Hu
  2020-04-16  8:41 ` Hans Verkuil
  2 siblings, 0 replies; 6+ messages in thread
From: Chun-Kuang Hu @ 2020-04-14 23:17 UTC (permalink / raw)
  To: Hsin-Yi Wang
  Cc: Linux ARM, moderated list:ARM/Mediatek SoC support,
	Andrew-CT Chen, Minghsiu Tsai, devicetree, linux-kernel,
	Houlong Wei, Rob Herring, Matthias Brugger,
	Mauro Carvalho Chehab, linux-media

Hi, Hsin-Yi

Hsin-Yi Wang <hsinyi@chromium.org> 於 2020年4月14日 週二 上午11:08寫道:
>
> Fix warning:
> Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-'

Reviewed-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>

>
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> ---
>  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> index a212bf124e81..d1e9c41004b4 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> @@ -42,14 +42,14 @@ aliases {
>                 dpi0 = &dpi0;
>                 dsi0 = &dsi0;
>                 dsi1 = &dsi1;
> -               mdp_rdma0 = &mdp_rdma0;
> -               mdp_rdma1 = &mdp_rdma1;
> -               mdp_rsz0 = &mdp_rsz0;
> -               mdp_rsz1 = &mdp_rsz1;
> -               mdp_rsz2 = &mdp_rsz2;
> -               mdp_wdma0 = &mdp_wdma0;
> -               mdp_wrot0 = &mdp_wrot0;
> -               mdp_wrot1 = &mdp_wrot1;
> +               mdp-rdma0 = &mdp_rdma0;
> +               mdp-rdma1 = &mdp_rdma1;
> +               mdp-rsz0 = &mdp_rsz0;
> +               mdp-rsz1 = &mdp_rsz1;
> +               mdp-rsz2 = &mdp_rsz2;
> +               mdp-wdma0 = &mdp_wdma0;
> +               mdp-wrot0 = &mdp_wrot0;
> +               mdp-wrot1 = &mdp_wrot1;
>                 serial0 = &uart0;
>                 serial1 = &uart1;
>                 serial2 = &uart2;
> --
> 2.26.0.110.g2183baf09c-goog
>
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 2/2] media: mtk-mdp: Use correct aliases name
  2020-04-14  3:08 ` [PATCH 2/2] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
@ 2020-04-14 23:18   ` Chun-Kuang Hu
  0 siblings, 0 replies; 6+ messages in thread
From: Chun-Kuang Hu @ 2020-04-14 23:18 UTC (permalink / raw)
  To: Hsin-Yi Wang
  Cc: Linux ARM, moderated list:ARM/Mediatek SoC support,
	Andrew-CT Chen, Minghsiu Tsai, devicetree, linux-kernel,
	Houlong Wei, Rob Herring, Matthias Brugger,
	Mauro Carvalho Chehab, linux-media

Hi, Hsin-Yi:

Hsin-Yi Wang <hsinyi@chromium.org> 於 2020年4月14日 週二 上午11:08寫道:
>
> aliases property name must include only lowercase and '-'. Fix in dts
> and driver.
>

Reviewed-by: Chun-Kuang Hu <chunkuang.hu@kernel.org>

> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
> ---
>  drivers/media/platform/mtk-mdp/mtk_mdp_comp.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c b/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
> index 14991685adb7..58abfbdfb82d 100644
> --- a/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
> +++ b/drivers/media/platform/mtk-mdp/mtk_mdp_comp.c
> @@ -15,10 +15,10 @@
>
>
>  static const char * const mtk_mdp_comp_stem[MTK_MDP_COMP_TYPE_MAX] = {
> -       "mdp_rdma",
> -       "mdp_rsz",
> -       "mdp_wdma",
> -       "mdp_wrot",
> +       "mdp-rdma",
> +       "mdp-rsz",
> +       "mdp-wdma",
> +       "mdp-wrot",
>  };
>
>  struct mtk_mdp_comp_match {
> --
> 2.26.0.110.g2183baf09c-goog
>
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name
  2020-04-14  3:08 [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Hsin-Yi Wang
  2020-04-14  3:08 ` [PATCH 2/2] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
  2020-04-14 23:17 ` [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Chun-Kuang Hu
@ 2020-04-16  8:41 ` Hans Verkuil
  2020-05-16 15:09   ` Matthias Brugger
  2 siblings, 1 reply; 6+ messages in thread
From: Hans Verkuil @ 2020-04-16  8:41 UTC (permalink / raw)
  To: Hsin-Yi Wang, linux-arm-kernel, linux-mediatek
  Cc: Minghsiu Tsai, Houlong Wei, Andrew-CT Chen,
	Mauro Carvalho Chehab, Matthias Brugger, linux-media,
	linux-kernel, Rob Herring, devicetree

On 14/04/2020 05:08, Hsin-Yi Wang wrote:
> Fix warning:
> Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-'
> 
> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
> ---
>  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)

I'll merge patch 2/2 for 5.8. I assume that this dtsi patch is merged through
a mediatek subsystem?

Regards,

	Hans

> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> index a212bf124e81..d1e9c41004b4 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> @@ -42,14 +42,14 @@ aliases {
>  		dpi0 = &dpi0;
>  		dsi0 = &dsi0;
>  		dsi1 = &dsi1;
> -		mdp_rdma0 = &mdp_rdma0;
> -		mdp_rdma1 = &mdp_rdma1;
> -		mdp_rsz0 = &mdp_rsz0;
> -		mdp_rsz1 = &mdp_rsz1;
> -		mdp_rsz2 = &mdp_rsz2;
> -		mdp_wdma0 = &mdp_wdma0;
> -		mdp_wrot0 = &mdp_wrot0;
> -		mdp_wrot1 = &mdp_wrot1;
> +		mdp-rdma0 = &mdp_rdma0;
> +		mdp-rdma1 = &mdp_rdma1;
> +		mdp-rsz0 = &mdp_rsz0;
> +		mdp-rsz1 = &mdp_rsz1;
> +		mdp-rsz2 = &mdp_rsz2;
> +		mdp-wdma0 = &mdp_wdma0;
> +		mdp-wrot0 = &mdp_wrot0;
> +		mdp-wrot1 = &mdp_wrot1;
>  		serial0 = &uart0;
>  		serial1 = &uart1;
>  		serial2 = &uart2;
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name
  2020-04-16  8:41 ` Hans Verkuil
@ 2020-05-16 15:09   ` Matthias Brugger
  0 siblings, 0 replies; 6+ messages in thread
From: Matthias Brugger @ 2020-05-16 15:09 UTC (permalink / raw)
  To: Hans Verkuil, Hsin-Yi Wang, linux-arm-kernel, linux-mediatek
  Cc: Minghsiu Tsai, Houlong Wei, Andrew-CT Chen,
	Mauro Carvalho Chehab, linux-media, linux-kernel, Rob Herring,
	devicetree



On 16/04/2020 10:41, Hans Verkuil wrote:
> On 14/04/2020 05:08, Hsin-Yi Wang wrote:
>> Fix warning:
>> Warning (alias_paths): /aliases: aliases property name must include only lowercase and '-'
>>
>> Signed-off-by: Hsin-Yi Wang <hsinyi@chromium.org>
>> ---
>>  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 16 ++++++++--------
>>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> I'll merge patch 2/2 for 5.8. I assume that this dtsi patch is merged through
> a mediatek subsystem?
> 

Correct. Now queued in v5.7-next/dts64

Thanks!

> Regards,
> 
> 	Hans
> 
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
>> index a212bf124e81..d1e9c41004b4 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
>> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
>> @@ -42,14 +42,14 @@ aliases {
>>  		dpi0 = &dpi0;
>>  		dsi0 = &dsi0;
>>  		dsi1 = &dsi1;
>> -		mdp_rdma0 = &mdp_rdma0;
>> -		mdp_rdma1 = &mdp_rdma1;
>> -		mdp_rsz0 = &mdp_rsz0;
>> -		mdp_rsz1 = &mdp_rsz1;
>> -		mdp_rsz2 = &mdp_rsz2;
>> -		mdp_wdma0 = &mdp_wdma0;
>> -		mdp_wrot0 = &mdp_wrot0;
>> -		mdp_wrot1 = &mdp_wrot1;
>> +		mdp-rdma0 = &mdp_rdma0;
>> +		mdp-rdma1 = &mdp_rdma1;
>> +		mdp-rsz0 = &mdp_rsz0;
>> +		mdp-rsz1 = &mdp_rsz1;
>> +		mdp-rsz2 = &mdp_rsz2;
>> +		mdp-wdma0 = &mdp_wdma0;
>> +		mdp-wrot0 = &mdp_wrot0;
>> +		mdp-wrot1 = &mdp_wrot1;
>>  		serial0 = &uart0;
>>  		serial1 = &uart1;
>>  		serial2 = &uart2;
>>
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-05-16 15:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-14  3:08 [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Hsin-Yi Wang
2020-04-14  3:08 ` [PATCH 2/2] media: mtk-mdp: Use correct aliases name Hsin-Yi Wang
2020-04-14 23:18   ` Chun-Kuang Hu
2020-04-14 23:17 ` [PATCH 1/2] arm64: dts: mt8173: fix mdp aliases property name Chun-Kuang Hu
2020-04-16  8:41 ` Hans Verkuil
2020-05-16 15:09   ` Matthias Brugger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).