| Message ID | 20210730031901.26243-1-s-anna@ti.com |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | dt-bindings: soc: ti: pruss: Add dma-coherent property | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success |
On 30/07/2021 06:19, Suman Anna wrote: > Update the PRUSS schema file to include the dma-coherent property > that indicates the coherency of the IP. The PRUSS IPs on 66AK2G > SoCs do use this property. > > Signed-off-by: Suman Anna <s-anna@ti.com> > --- > Hi Santosh, > > This patch updates the PRUSS binding in preparation for adding the > PRUSS nodes for 66AK2G SoCs. Without this, the dtbs_check would > complain about the undefined dma-coherent property. Patch is top > of the AM64 ICSSG binding update patch [1]. > > regards > Suman > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210623165032.31223-2-s-anna@ti.com/ > > .../devicetree/bindings/soc/ti/ti,pruss.yaml | 37 +++++++++++++------ > 1 file changed, 25 insertions(+), 12 deletions(-) > Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
On Thu, Jul 29, 2021 at 10:19:01PM -0500, Suman Anna wrote: > Update the PRUSS schema file to include the dma-coherent property > that indicates the coherency of the IP. The PRUSS IPs on 66AK2G > SoCs do use this property. > > Signed-off-by: Suman Anna <s-anna@ti.com> > --- > Hi Santosh, > > This patch updates the PRUSS binding in preparation for adding the > PRUSS nodes for 66AK2G SoCs. Without this, the dtbs_check would > complain about the undefined dma-coherent property. Patch is top > of the AM64 ICSSG binding update patch [1]. New required properties are not backwards compatible. You are kind of saying that here (not used yet?), but make that clear in the commit msg. With that fixed, Reviewed-by: Rob Herring <robh@kernel.org> > > regards > Suman > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210623165032.31223-2-s-anna@ti.com/ > > .../devicetree/bindings/soc/ti/ti,pruss.yaml | 37 +++++++++++++------ > 1 file changed, 25 insertions(+), 12 deletions(-) > > diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml > index 47d7fd24bc56..9d128b9e7deb 100644 > --- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml > +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml > @@ -85,6 +85,8 @@ properties: > dma-ranges: > maxItems: 1 > > + dma-coherent: true > + > power-domains: > description: | > This property is as per sci-pm-domain.txt. > @@ -324,18 +326,29 @@ additionalProperties: false > # - interrupt-controller > # - pru > > -if: > - properties: > - compatible: > - contains: > - enum: > - - ti,k2g-pruss > - - ti,am654-icssg > - - ti,j721e-icssg > - - ti,am642-icssg > -then: > - required: > - - power-domains > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - ti,k2g-pruss > + - ti,am654-icssg > + - ti,j721e-icssg > + - ti,am642-icssg > + then: > + required: > + - power-domains > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - ti,k2g-pruss > + then: > + required: > + - dma-coherent > > examples: > - | > -- > 2.32.0 > >
On 8/3/21 2:28 PM, Rob Herring wrote: > On Thu, Jul 29, 2021 at 10:19:01PM -0500, Suman Anna wrote: >> Update the PRUSS schema file to include the dma-coherent property >> that indicates the coherency of the IP. The PRUSS IPs on 66AK2G >> SoCs do use this property. >> >> Signed-off-by: Suman Anna <s-anna@ti.com> >> --- >> Hi Santosh, >> >> This patch updates the PRUSS binding in preparation for adding the >> PRUSS nodes for 66AK2G SoCs. Without this, the dtbs_check would >> complain about the undefined dma-coherent property. Patch is top >> of the AM64 ICSSG binding update patch [1]. > > New required properties are not backwards compatible. You are kind of > saying that here (not used yet?), but make that clear in the commit msg. OK. Yeah, we haven't added the K2G dts nodes yet, and the new required property is only applicable for that SoC and not others. regards Suman > > With that fixed, > > Reviewed-by: Rob Herring <robh@kernel.org> > >> >> regards >> Suman >> >> [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210623165032.31223-2-s-anna@ti.com/ >> >> .../devicetree/bindings/soc/ti/ti,pruss.yaml | 37 +++++++++++++------ >> 1 file changed, 25 insertions(+), 12 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml >> index 47d7fd24bc56..9d128b9e7deb 100644 >> --- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml >> +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml >> @@ -85,6 +85,8 @@ properties: >> dma-ranges: >> maxItems: 1 >> >> + dma-coherent: true >> + >> power-domains: >> description: | >> This property is as per sci-pm-domain.txt. >> @@ -324,18 +326,29 @@ additionalProperties: false >> # - interrupt-controller >> # - pru >> >> -if: >> - properties: >> - compatible: >> - contains: >> - enum: >> - - ti,k2g-pruss >> - - ti,am654-icssg >> - - ti,j721e-icssg >> - - ti,am642-icssg >> -then: >> - required: >> - - power-domains >> +allOf: >> + - if: >> + properties: >> + compatible: >> + contains: >> + enum: >> + - ti,k2g-pruss >> + - ti,am654-icssg >> + - ti,j721e-icssg >> + - ti,am642-icssg >> + then: >> + required: >> + - power-domains >> + >> + - if: >> + properties: >> + compatible: >> + contains: >> + enum: >> + - ti,k2g-pruss >> + then: >> + required: >> + - dma-coherent >> >> examples: >> - | >> -- >> 2.32.0 >> >>
diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml index 47d7fd24bc56..9d128b9e7deb 100644 --- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml @@ -85,6 +85,8 @@ properties: dma-ranges: maxItems: 1 + dma-coherent: true + power-domains: description: | This property is as per sci-pm-domain.txt. @@ -324,18 +326,29 @@ additionalProperties: false # - interrupt-controller # - pru -if: - properties: - compatible: - contains: - enum: - - ti,k2g-pruss - - ti,am654-icssg - - ti,j721e-icssg - - ti,am642-icssg -then: - required: - - power-domains +allOf: + - if: + properties: + compatible: + contains: + enum: + - ti,k2g-pruss + - ti,am654-icssg + - ti,j721e-icssg + - ti,am642-icssg + then: + required: + - power-domains + + - if: + properties: + compatible: + contains: + enum: + - ti,k2g-pruss + then: + required: + - dma-coherent examples: - |
Update the PRUSS schema file to include the dma-coherent property that indicates the coherency of the IP. The PRUSS IPs on 66AK2G SoCs do use this property. Signed-off-by: Suman Anna <s-anna@ti.com> --- Hi Santosh, This patch updates the PRUSS binding in preparation for adding the PRUSS nodes for 66AK2G SoCs. Without this, the dtbs_check would complain about the undefined dma-coherent property. Patch is top of the AM64 ICSSG binding update patch [1]. regards Suman [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20210623165032.31223-2-s-anna@ti.com/ .../devicetree/bindings/soc/ti/ti,pruss.yaml | 37 +++++++++++++------ 1 file changed, 25 insertions(+), 12 deletions(-)