All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	"Andreas Färber" <afaerber@suse.de>,
	"Baruch Siach" <baruch@tkos.co.il>,
	"Linus Walleij" <linusw@kernel.org>,
	"Imre Kaloz" <kaloz@openwrt.org>,
	"Krzysztof Halasa" <khalasa@piap.pl>,
	"Andy Gross" <agross@kernel.org>,
	"Bjorn Andersson" <bjorn.andersson@linaro.org>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Alexandre Torgue" <alexandre.torgue@st.com>,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Dinh Nguyen" <dinguyen@kernel.org>,
	"Anders Berg" <anders.berg@lsi.com>
Subject: Re: [PATCH] dt-bindings: arm: Add missing root node constraint for board/SoC bindings
Date: Fri, 2 Oct 2020 19:48:53 +0530	[thread overview]
Message-ID: <20201002141853.GA7178@Mani-XPS-13-9360> (raw)
In-Reply-To: <20201001200943.1193870-1-robh@kernel.org>

On Thu, Oct 01, 2020 at 03:09:43PM -0500, Rob Herring wrote:
> Board/SoC top-level compatible bindings should be constrained to the root
> node. Add the missing constraints that the node name must be "/".
> 
> Cc: "Andreas Färber" <afaerber@suse.de>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Baruch Siach <baruch@tkos.co.il>
> Cc: Linus Walleij <linusw@kernel.org>
> Cc: Imre Kaloz <kaloz@openwrt.org>
> Cc: Krzysztof Halasa <khalasa@piap.pl>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Dinh Nguyen <dinguyen@kernel.org>
> Cc: Anders Berg <anders.berg@lsi.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/actions.yaml      | 2 ++

For actions,

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

Thanks,
Mani

>  Documentation/devicetree/bindings/arm/altera.yaml       | 2 ++
>  Documentation/devicetree/bindings/arm/axxia.yaml        | 2 ++
>  Documentation/devicetree/bindings/arm/bitmain.yaml      | 2 ++
>  Documentation/devicetree/bindings/arm/digicolor.yaml    | 2 ++
>  Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml | 2 ++
>  Documentation/devicetree/bindings/arm/qcom.yaml         | 2 ++
>  Documentation/devicetree/bindings/arm/rda.yaml          | 2 ++
>  Documentation/devicetree/bindings/arm/stm32/stm32.yaml  | 2 ++
>  Documentation/devicetree/bindings/arm/tegra.yaml        | 2 ++
>  10 files changed, 20 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/actions.yaml b/Documentation/devicetree/bindings/arm/actions.yaml
> index ace3fdaa8396..14023f0a8552 100644
> --- a/Documentation/devicetree/bindings/arm/actions.yaml
> +++ b/Documentation/devicetree/bindings/arm/actions.yaml
> @@ -11,6 +11,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        # The Actions Semi S500 is a quad-core ARM Cortex-A9 SoC.
> diff --git a/Documentation/devicetree/bindings/arm/altera.yaml b/Documentation/devicetree/bindings/arm/altera.yaml
> index b388c5aa7984..0bc5020b7539 100644
> --- a/Documentation/devicetree/bindings/arm/altera.yaml
> +++ b/Documentation/devicetree/bindings/arm/altera.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Dinh Nguyen <dinguyen@kernel.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/axxia.yaml b/Documentation/devicetree/bindings/arm/axxia.yaml
> index 98780a569f22..3ea5f2fdcd96 100644
> --- a/Documentation/devicetree/bindings/arm/axxia.yaml
> +++ b/Documentation/devicetree/bindings/arm/axxia.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Anders Berg <anders.berg@lsi.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      description: LSI AXM5516 Validation board (Amarillo)
>      items:
> diff --git a/Documentation/devicetree/bindings/arm/bitmain.yaml b/Documentation/devicetree/bindings/arm/bitmain.yaml
> index 5cd5b36cff2d..5880083ab8d0 100644
> --- a/Documentation/devicetree/bindings/arm/bitmain.yaml
> +++ b/Documentation/devicetree/bindings/arm/bitmain.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/digicolor.yaml b/Documentation/devicetree/bindings/arm/digicolor.yaml
> index d9c80b827e9b..849e20518339 100644
> --- a/Documentation/devicetree/bindings/arm/digicolor.yaml
> +++ b/Documentation/devicetree/bindings/arm/digicolor.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Baruch Siach <baruch@tkos.co.il>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      const: cnxt,cx92755
>  
> diff --git a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> index f4f7451e5e8a..f18302efb90e 100644
> --- a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> +++ b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Linus Walleij <linus.walleij@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 6031aee0f5a8..ae6284be9fef 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -73,6 +73,8 @@ description: |
>    foundry 2.
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/rda.yaml b/Documentation/devicetree/bindings/arm/rda.yaml
> index 51cec2b63b04..9672aa0c760d 100644
> --- a/Documentation/devicetree/bindings/arm/rda.yaml
> +++ b/Documentation/devicetree/bindings/arm/rda.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> index 790e6dd48e34..696a0101ebcc 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Alexandre Torgue <alexandre.torgue@st.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
> index e0b3debaee9e..b4d53290c5f0 100644
> --- a/Documentation/devicetree/bindings/arm/tegra.yaml
> +++ b/Documentation/devicetree/bindings/arm/tegra.yaml
> @@ -11,6 +11,8 @@ maintainers:
>    - Jonathan Hunter <jonathanh@nvidia.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> -- 
> 2.25.1
> 

WARNING: multiple messages have this Message-ID (diff)
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org, "Baruch Siach" <baruch@tkos.co.il>,
	"Alexandre Torgue" <alexandre.torgue@st.com>,
	"Anders Berg" <anders.berg@lsi.com>,
	"Linus Walleij" <linusw@kernel.org>,
	"Jonathan Hunter" <jonathanh@nvidia.com>,
	"Bjorn Andersson" <bjorn.andersson@linaro.org>,
	"Dinh Nguyen" <dinguyen@kernel.org>,
	"Andy Gross" <agross@kernel.org>,
	"Krzysztof Halasa" <khalasa@piap.pl>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Thierry Reding" <thierry.reding@gmail.com>,
	"Imre Kaloz" <kaloz@openwrt.org>,
	"Andreas Färber" <afaerber@suse.de>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] dt-bindings: arm: Add missing root node constraint for board/SoC bindings
Date: Fri, 2 Oct 2020 19:48:53 +0530	[thread overview]
Message-ID: <20201002141853.GA7178@Mani-XPS-13-9360> (raw)
In-Reply-To: <20201001200943.1193870-1-robh@kernel.org>

On Thu, Oct 01, 2020 at 03:09:43PM -0500, Rob Herring wrote:
> Board/SoC top-level compatible bindings should be constrained to the root
> node. Add the missing constraints that the node name must be "/".
> 
> Cc: "Andreas Färber" <afaerber@suse.de>
> Cc: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Cc: Baruch Siach <baruch@tkos.co.il>
> Cc: Linus Walleij <linusw@kernel.org>
> Cc: Imre Kaloz <kaloz@openwrt.org>
> Cc: Krzysztof Halasa <khalasa@piap.pl>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com>
> Cc: Alexandre Torgue <alexandre.torgue@st.com>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Jonathan Hunter <jonathanh@nvidia.com>
> Cc: Dinh Nguyen <dinguyen@kernel.org>
> Cc: Anders Berg <anders.berg@lsi.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/arm/actions.yaml      | 2 ++

For actions,

Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>

Thanks,
Mani

>  Documentation/devicetree/bindings/arm/altera.yaml       | 2 ++
>  Documentation/devicetree/bindings/arm/axxia.yaml        | 2 ++
>  Documentation/devicetree/bindings/arm/bitmain.yaml      | 2 ++
>  Documentation/devicetree/bindings/arm/digicolor.yaml    | 2 ++
>  Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml | 2 ++
>  Documentation/devicetree/bindings/arm/qcom.yaml         | 2 ++
>  Documentation/devicetree/bindings/arm/rda.yaml          | 2 ++
>  Documentation/devicetree/bindings/arm/stm32/stm32.yaml  | 2 ++
>  Documentation/devicetree/bindings/arm/tegra.yaml        | 2 ++
>  10 files changed, 20 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/actions.yaml b/Documentation/devicetree/bindings/arm/actions.yaml
> index ace3fdaa8396..14023f0a8552 100644
> --- a/Documentation/devicetree/bindings/arm/actions.yaml
> +++ b/Documentation/devicetree/bindings/arm/actions.yaml
> @@ -11,6 +11,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        # The Actions Semi S500 is a quad-core ARM Cortex-A9 SoC.
> diff --git a/Documentation/devicetree/bindings/arm/altera.yaml b/Documentation/devicetree/bindings/arm/altera.yaml
> index b388c5aa7984..0bc5020b7539 100644
> --- a/Documentation/devicetree/bindings/arm/altera.yaml
> +++ b/Documentation/devicetree/bindings/arm/altera.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Dinh Nguyen <dinguyen@kernel.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/axxia.yaml b/Documentation/devicetree/bindings/arm/axxia.yaml
> index 98780a569f22..3ea5f2fdcd96 100644
> --- a/Documentation/devicetree/bindings/arm/axxia.yaml
> +++ b/Documentation/devicetree/bindings/arm/axxia.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Anders Berg <anders.berg@lsi.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      description: LSI AXM5516 Validation board (Amarillo)
>      items:
> diff --git a/Documentation/devicetree/bindings/arm/bitmain.yaml b/Documentation/devicetree/bindings/arm/bitmain.yaml
> index 5cd5b36cff2d..5880083ab8d0 100644
> --- a/Documentation/devicetree/bindings/arm/bitmain.yaml
> +++ b/Documentation/devicetree/bindings/arm/bitmain.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/digicolor.yaml b/Documentation/devicetree/bindings/arm/digicolor.yaml
> index d9c80b827e9b..849e20518339 100644
> --- a/Documentation/devicetree/bindings/arm/digicolor.yaml
> +++ b/Documentation/devicetree/bindings/arm/digicolor.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Baruch Siach <baruch@tkos.co.il>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      const: cnxt,cx92755
>  
> diff --git a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> index f4f7451e5e8a..f18302efb90e 100644
> --- a/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> +++ b/Documentation/devicetree/bindings/arm/intel-ixp4xx.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Linus Walleij <linus.walleij@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 6031aee0f5a8..ae6284be9fef 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -73,6 +73,8 @@ description: |
>    foundry 2.
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/rda.yaml b/Documentation/devicetree/bindings/arm/rda.yaml
> index 51cec2b63b04..9672aa0c760d 100644
> --- a/Documentation/devicetree/bindings/arm/rda.yaml
> +++ b/Documentation/devicetree/bindings/arm/rda.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      items:
>        - enum:
> diff --git a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> index 790e6dd48e34..696a0101ebcc 100644
> --- a/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> +++ b/Documentation/devicetree/bindings/arm/stm32/stm32.yaml
> @@ -10,6 +10,8 @@ maintainers:
>    - Alexandre Torgue <alexandre.torgue@st.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> diff --git a/Documentation/devicetree/bindings/arm/tegra.yaml b/Documentation/devicetree/bindings/arm/tegra.yaml
> index e0b3debaee9e..b4d53290c5f0 100644
> --- a/Documentation/devicetree/bindings/arm/tegra.yaml
> +++ b/Documentation/devicetree/bindings/arm/tegra.yaml
> @@ -11,6 +11,8 @@ maintainers:
>    - Jonathan Hunter <jonathanh@nvidia.com>
>  
>  properties:
> +  $nodename:
> +    const: "/"
>    compatible:
>      oneOf:
>        - items:
> -- 
> 2.25.1
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2020-10-02 14:19 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-01 20:09 [PATCH] dt-bindings: arm: Add missing root node constraint for board/SoC bindings Rob Herring
2020-10-01 20:09 ` Rob Herring
2020-10-01 20:55 ` Linus Walleij
2020-10-01 20:55   ` Linus Walleij
2020-10-02 10:58 ` Andreas Färber
2020-10-02 10:58   ` Andreas Färber
2020-10-02 14:18 ` Manivannan Sadhasivam [this message]
2020-10-02 14:18   ` Manivannan Sadhasivam
2020-10-05  9:33 ` Thierry Reding
2020-10-05  9:33   ` Thierry Reding
2020-10-05 10:16 ` Alexandre Torgue
2020-10-05 10:16   ` Alexandre Torgue

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201002141853.GA7178@Mani-XPS-13-9360 \
    --to=manivannan.sadhasivam@linaro.org \
    --cc=afaerber@suse.de \
    --cc=agross@kernel.org \
    --cc=alexandre.torgue@st.com \
    --cc=anders.berg@lsi.com \
    --cc=baruch@tkos.co.il \
    --cc=bjorn.andersson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dinguyen@kernel.org \
    --cc=jonathanh@nvidia.com \
    --cc=kaloz@openwrt.org \
    --cc=khalasa@piap.pl \
    --cc=linusw@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=robh@kernel.org \
    --cc=thierry.reding@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.