[v2] Mediatek SCP dt-binding tweaks

[PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp

Posted by Nícolas F. R. A. Prado 2 years, 4 months ago

The SCP co-processor can optionally be passed a reserved memory region
to use. Add this property in the dt-binding.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>

---
I left out Angelo's R-b since of the 3 lines in the patch from v1, only
1 is left.

Changes in v2:
- Dropped type and description since it's a well-known property
- Set maxItems to 1

 Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
index ec9ddeb6ca2c..c6a92ac8db18 100644
--- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
@@ -51,6 +51,9 @@ properties:
   interrupts:
     maxItems: 1
 
+  memory-region:
+    maxItems: 1
+
 required:
   - compatible
   - reg
-- 
2.36.0

Re: [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp

Posted by Rob Herring 2 years, 4 months ago

On Mon, 02 May 2022 15:24:20 -0400, Nícolas F. R. A. Prado wrote:
> The SCP co-processor can optionally be passed a reserved memory region
> to use. Add this property in the dt-binding.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> 
> ---
> I left out Angelo's R-b since of the 3 lines in the patch from v1, only
> 1 is left.
> 
> Changes in v2:
> - Dropped type and description since it's a well-known property
> - Set maxItems to 1
> 
>  Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 

Acked-by: Rob Herring <robh@kernel.org>

Re: [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp

Posted by AngeloGioacchino Del Regno 2 years, 4 months ago

Il 03/05/22 16:16, Rob Herring ha scritto:
> On Mon, 02 May 2022 15:24:20 -0400, Nícolas F. R. A. Prado wrote:
>> The SCP co-processor can optionally be passed a reserved memory region
>> to use. Add this property in the dt-binding.
>>
>> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
>>
>> ---
>> I left out Angelo's R-b since of the 3 lines in the patch from v1, only
>> 1 is left.
>>
>> Changes in v2:
>> - Dropped type and description since it's a well-known property
>> - Set maxItems to 1
>>
>>   Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml | 3 +++
>>   1 file changed, 3 insertions(+)
>>
> 
> Acked-by: Rob Herring <robh@kernel.org>


Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

[PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp
[PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp