* [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property
@ 2021-10-18 11:51 Tinghan Shen
2021-10-18 11:55 ` AngeloGioacchino Del Regno
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Tinghan Shen @ 2021-10-18 11:51 UTC (permalink / raw)
To: Lee Jones, Rob Herring, Benson Leung, Enric Balletbo i Serra,
Guenter Roeck, Matthias Brugger, AngeloGioacchino Del Regno
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Project_Global_Chrome_Upstream_Group, Tinghan Shen
Change from "mtk,rpmsg-name" to "mediatek,rpmsg-name" to sync with the
vendor name defined in vendor-prefixes.yaml.
Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
---
Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
index d793dd0316b7..0faa4da6c7c8 100644
--- a/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
+++ b/Documentation/devicetree/bindings/mfd/google,cros-ec.yaml
@@ -59,7 +59,7 @@ properties:
whether this nvram is present or not.
type: boolean
- mtk,rpmsg-name:
+ mediatek,rpmsg-name:
description:
Must be defined if the cros-ec is a rpmsg device for a Mediatek
ARM Cortex M4 Co-processor. Contains the name pf the rpmsg
--
2.18.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property
2021-10-18 11:51 [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property Tinghan Shen
@ 2021-10-18 11:55 ` AngeloGioacchino Del Regno
2021-10-18 17:32 ` Rob Herring
2021-10-27 2:15 ` Rob Herring
2 siblings, 0 replies; 5+ messages in thread
From: AngeloGioacchino Del Regno @ 2021-10-18 11:55 UTC (permalink / raw)
To: Tinghan Shen, Lee Jones, Rob Herring, Benson Leung,
Enric Balletbo i Serra, Guenter Roeck, Matthias Brugger
Cc: devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Project_Global_Chrome_Upstream_Group
> Change from "mtk,rpmsg-name" to "mediatek,rpmsg-name" to sync with the
> vendor name defined in vendor-prefixes.yaml.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> ---
> Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Reviewed-By: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property
2021-10-18 11:51 [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property Tinghan Shen
2021-10-18 11:55 ` AngeloGioacchino Del Regno
@ 2021-10-18 17:32 ` Rob Herring
2021-10-27 2:27 ` Tinghan Shen
2021-10-27 2:15 ` Rob Herring
2 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2021-10-18 17:32 UTC (permalink / raw)
To: Tinghan Shen
Cc: AngeloGioacchino Del Regno, linux-arm-kernel, linux-kernel,
Benson Leung, Project_Global_Chrome_Upstream_Group,
linux-mediatek, Lee Jones, Enric Balletbo i Serra, Rob Herring,
Matthias Brugger, Guenter Roeck, devicetree
On Mon, 18 Oct 2021 19:51:11 +0800, Tinghan Shen wrote:
> Change from "mtk,rpmsg-name" to "mediatek,rpmsg-name" to sync with the
> vendor name defined in vendor-prefixes.yaml.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> ---
> Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Running 'make dtbs_check' with the schema in this patch gives the
following warnings. Consider if they are expected or the schema is
incorrect. These may not be new warnings.
Note that it is not yet a requirement to have 0 warnings for dtbs_check.
This will change in the future.
Full log is available here: https://patchwork.ozlabs.org/patch/1542563
cros_ec: 'mtk,rpmsg-name' does not match any of the regexes: '^extcon[0-9]*$', '^i2c-tunnel[0-9]*$', '^regulator@[0-9]+$', 'pinctrl-[0-9]+'
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-burnet.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel14.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku1.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-sku6.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku0.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-sku1.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml
arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property
2021-10-18 11:51 [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property Tinghan Shen
2021-10-18 11:55 ` AngeloGioacchino Del Regno
2021-10-18 17:32 ` Rob Herring
@ 2021-10-27 2:15 ` Rob Herring
2 siblings, 0 replies; 5+ messages in thread
From: Rob Herring @ 2021-10-27 2:15 UTC (permalink / raw)
To: Tinghan Shen
Cc: Lee Jones, linux-arm-kernel, linux-mediatek, Benson Leung,
devicetree, Matthias Brugger, linux-kernel,
Project_Global_Chrome_Upstream_Group, AngeloGioacchino Del Regno,
Guenter Roeck, Enric Balletbo i Serra, Rob Herring
On Mon, 18 Oct 2021 19:51:11 +0800, Tinghan Shen wrote:
> Change from "mtk,rpmsg-name" to "mediatek,rpmsg-name" to sync with the
> vendor name defined in vendor-prefixes.yaml.
>
> Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> ---
> Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property
2021-10-18 17:32 ` Rob Herring
@ 2021-10-27 2:27 ` Tinghan Shen
0 siblings, 0 replies; 5+ messages in thread
From: Tinghan Shen @ 2021-10-27 2:27 UTC (permalink / raw)
To: Rob Herring
Cc: AngeloGioacchino Del Regno, linux-arm-kernel, linux-kernel,
Benson Leung, Project_Global_Chrome_Upstream_Group,
linux-mediatek, Lee Jones, Enric Balletbo i Serra, Rob Herring,
Matthias Brugger, Guenter Roeck, devicetree
On Mon, 2021-10-18 at 12:32 -0500, Rob Herring wrote:
> On Mon, 18 Oct 2021 19:51:11 +0800, Tinghan Shen wrote:
> > Change from "mtk,rpmsg-name" to "mediatek,rpmsg-name" to sync with
> > the
> > vendor name defined in vendor-prefixes.yaml.
> >
> > Signed-off-by: Tinghan Shen <tinghan.shen@mediatek.com>
> > ---
> > Documentation/devicetree/bindings/mfd/google,cros-ec.yaml | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
>
> Running 'make dtbs_check' with the schema in this patch gives the
> following warnings. Consider if they are expected or the schema is
> incorrect. These may not be new warnings.
>
> Note that it is not yet a requirement to have 0 warnings for
> dtbs_check.
> This will change in the future.
>
> Full log is available here:
> https://urldefense.com/v3/__https://patchwork.ozlabs.org/patch/1542563__;!!CTRNKA9wMg0ARbw!yQGrJgf_q0cFKm4GiKGqWF4eFuMbBE3w_p4r9ipndPFZMOYDBfs0oAwGpZa6mKdIggI$
>
>
>
> cros_ec: 'mtk,rpmsg-name' does not match any of the regexes:
> '^extcon[0-9]*$', '^i2c-tunnel[0-9]*$', '^regulator@[0-9]+$',
> 'pinctrl-[0-9]+'
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-
> burnet.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-damu.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-
> fennel14.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-
> sku1.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-fennel-
> sku6.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-
> sku16.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kappa.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-kenzo.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-
> sku0.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-willow-
> sku1.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-kakadu.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku16.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku272.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku288.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-kodama-sku32.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku0.dt.yaml
> arch/arm64/boot/dts/mediatek/mt8183-kukui-krane-sku176.dt.yaml
>
Hi Rob,
Thanks for your reviewing. I'm sorry that forget to mention the
dependency of this patch.
This patch is introduced because of this change,
https://lore.kernel.org/all/20210924033935.2127-7-tinghan.shen@mediatek.com/
I will resend this patch to mention the dependency.
Best regards,
TingHan
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-10-27 2:27 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-18 11:51 [PATCH] dt-bindings: mfd: change naming of mediatek rpmsg property Tinghan Shen
2021-10-18 11:55 ` AngeloGioacchino Del Regno
2021-10-18 17:32 ` Rob Herring
2021-10-27 2:27 ` Tinghan Shen
2021-10-27 2:15 ` Rob Herring
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).