| Message ID | 20200928225155.12432-1-s-anna@ti.com |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [-next] dt-bindings: hwlock: omap: Fix warnings with k3.yaml | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 2 warnings, 16 lines checked |
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>
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
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 --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>;
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(-)