diff mbox series

[-next] dt-bindings: hwlock: omap: Fix warnings with k3.yaml

Message ID 20200928225155.12432-1-s-anna@ti.com
State Accepted
Headers show
Series [-next] dt-bindings: hwlock: omap: Fix warnings with k3.yaml | expand

Checks

Context Check Description
robh/checkpatch warning total: 0 errors, 2 warnings, 16 lines checked

Commit Message

Suman Anna Sept. 28, 2020, 10:51 p.m. UTC
Update the AM65x HwSpinlock example to fix couple of warnings
that started showing up after the conversion of K3 bindings to
YAML format in commit 66e06509aa37 ("dt-bindings: arm: ti:
Convert K3 board/soc bindings to DT schema").

 compatible: ['ti,am654'] is not valid under any of the given schemas (Possible causes of the failure):
 compatible: ['ti,am654'] is too short
 compatible:0: 'ti,am654' is not one of ['ti,am654-evm']

Also, fix one of the node names while at this.

Signed-off-by: Suman Anna <s-anna@ti.com>
---
 .../devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml        | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Rob Herring Sept. 30, 2020, 2:55 p.m. UTC | #1
On Mon, Sep 28, 2020 at 05:51:55PM -0500, Suman Anna wrote:
> Update the AM65x HwSpinlock example to fix couple of warnings
> that started showing up after the conversion of K3 bindings to
> YAML format in commit 66e06509aa37 ("dt-bindings: arm: ti:
> Convert K3 board/soc bindings to DT schema").
> 
>  compatible: ['ti,am654'] is not valid under any of the given schemas (Possible causes of the failure):
>  compatible: ['ti,am654'] is too short
>  compatible:0: 'ti,am654' is not one of ['ti,am654-evm']
> 
> Also, fix one of the node names while at this.
> 
> Signed-off-by: Suman Anna <s-anna@ti.com>
> ---
>  .../devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml        | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>
Rob Herring Oct. 6, 2020, 3:52 p.m. UTC | #2
On Wed, Sep 30, 2020 at 09:55:37AM -0500, Rob Herring wrote:
> On Mon, Sep 28, 2020 at 05:51:55PM -0500, Suman Anna wrote:
> > Update the AM65x HwSpinlock example to fix couple of warnings
> > that started showing up after the conversion of K3 bindings to
> > YAML format in commit 66e06509aa37 ("dt-bindings: arm: ti:
> > Convert K3 board/soc bindings to DT schema").
> > 
> >  compatible: ['ti,am654'] is not valid under any of the given schemas (Possible causes of the failure):
> >  compatible: ['ti,am654'] is too short
> >  compatible:0: 'ti,am654' is not one of ['ti,am654-evm']
> > 
> > Also, fix one of the node names while at this.
> > 
> > Signed-off-by: Suman Anna <s-anna@ti.com>
> > ---
> >  .../devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml        | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> Reviewed-by: Rob Herring <robh@kernel.org>

I guess this isn't dependent on k3.yaml, so I've applied it.

Rob
Suman Anna Oct. 6, 2020, 4:16 p.m. UTC | #3
On 10/6/20 10:52 AM, Rob Herring wrote:
> On Wed, Sep 30, 2020 at 09:55:37AM -0500, Rob Herring wrote:
>> On Mon, Sep 28, 2020 at 05:51:55PM -0500, Suman Anna wrote:
>>> Update the AM65x HwSpinlock example to fix couple of warnings
>>> that started showing up after the conversion of K3 bindings to
>>> YAML format in commit 66e06509aa37 ("dt-bindings: arm: ti:
>>> Convert K3 board/soc bindings to DT schema").
>>>
>>>  compatible: ['ti,am654'] is not valid under any of the given schemas (Possible causes of the failure):
>>>  compatible: ['ti,am654'] is too short
>>>  compatible:0: 'ti,am654' is not one of ['ti,am654-evm']
>>>
>>> Also, fix one of the node names while at this.
>>>
>>> Signed-off-by: Suman Anna <s-anna@ti.com>
>>> ---
>>>  .../devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml        | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> Reviewed-by: Rob Herring <robh@kernel.org>
> 
> I guess this isn't dependent on k3.yaml, so I've applied it.

Yes, it is independent. Thanks Rob.

regards
Suman
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml b/Documentation/devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml
index 2765758d95e1..ac35491a6f65 100644
--- a/Documentation/devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml
+++ b/Documentation/devicetree/bindings/hwlock/ti,omap-hwspinlock.yaml
@@ -49,7 +49,7 @@  examples:
     / {
         /* K3 AM65x SoCs */
         model = "Texas Instruments K3 AM654 SoC";
-        compatible = "ti,am654";
+        compatible = "ti,am654-evm", "ti,am654";
         #address-cells = <2>;
         #size-cells = <2>;
 
@@ -60,7 +60,7 @@  examples:
             ranges = <0x00 0x00100000 0x00 0x00100000 0x00 0x00020000>, /* ctrl mmr */
                      <0x00 0x30800000 0x00 0x30800000 0x00 0x0bc00000>; /* Main NavSS */
 
-            main_navss@30800000 {
+            bus@30800000 {
                 compatible = "simple-mfd";
                 #address-cells = <2>;
                 #size-cells = <2>;