diff mbox series

dt-bindings: remoteproc: qcom: Fix sm6115 pas documentation

Message ID 20230313202655.1719587-1-bhupesh.sharma@linaro.org
State New
Headers
Series dt-bindings: remoteproc: qcom: Fix sm6115 pas documentation |

Commit Message

Bhupesh Sharma March 13, 2023, 8:26 p.m. UTC 
  Fix the sm6115 pas yaml documentation to note that 'memory-region'
is a required property.

While at it also mark that memory-region property supports
only one item.

Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
---
 .../devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Krzysztof Kozlowski March 14, 2023, 8:37 a.m. UTC | #1 
On 13/03/2023 21:26, Bhupesh Sharma wrote:
> Fix the sm6115 pas yaml documentation to note that 'memory-region'
> is a required property.
> 
> While at it also mark that memory-region property supports
> only one item.
> 
> Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof
Bhupesh Sharma March 27, 2023, 8:44 p.m. UTC | #2 
On Tue, 14 Mar 2023 at 14:07, Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 13/03/2023 21:26, Bhupesh Sharma wrote:
> > Fix the sm6115 pas yaml documentation to note that 'memory-region'
> > is a required property.
> >
> > While at it also mark that memory-region property supports
> > only one item.
> >
> > Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > Signed-off-by: Bhupesh Sharma <bhupesh.sharma@linaro.org>
>
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Thanks Krzysztof for the review,

And a gentle ping to maintainers for merging this change.

Thanks,
Bhupesh
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml
index f5d1fa9f45f1..cd5d5d068912 100644
--- a/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml
@@ -32,7 +32,7 @@  properties:
       - const: xo
 
   memory-region:
-    minItems: 1
+    maxItems: 1
     description: Reference to the reserved-memory for the Hexagon core
 
   smd-edge: false
@@ -43,6 +43,7 @@  properties:
 
 required:
   - compatible
+  - memory-region
   - reg
 
 allOf: