* [PATCH v2 0/2] Mediatek SCP dt-binding tweaks
@ 2022-05-02 19:24 Nícolas F. R. A. Prado
2022-05-02 19:24 ` [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp Nícolas F. R. A. Prado
2022-05-02 19:24 ` [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp Nícolas F. R. A. Prado
0 siblings, 2 replies; 6+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-05-02 19:24 UTC (permalink / raw)
To: Bjorn Andersson
Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
Krzysztof Kozlowski, Mathieu Poirier, Matthias Brugger,
Rob Herring, Tinghan Shen, devicetree, linux-arm-kernel,
linux-kernel, linux-mediatek, linux-remoteproc
Two simple patches for the Mediatek SCP dt-binding. The first fixes the
reg/reg-names property while the second adds a new optional
memory-region property.
v1: https://lore.kernel.org/all/20220429211111.2214119-1-nfraprado@collabora.com
Changes in v2:
- Dropped type and description from memory-region since it's a
well-known property
- Set memory-region maxItems to 1
Nícolas F. R. A. Prado (2):
dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp
dt-bindings: remoteproc: mediatek: Add optional memory-region to
mtk,scp
.../devicetree/bindings/remoteproc/mtk,scp.yaml | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
--
2.36.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp
2022-05-02 19:24 [PATCH v2 0/2] Mediatek SCP dt-binding tweaks Nícolas F. R. A. Prado
@ 2022-05-02 19:24 ` Nícolas F. R. A. Prado
2022-05-03 12:20 ` Krzysztof Kozlowski
2022-05-02 19:24 ` [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp Nícolas F. R. A. Prado
1 sibling, 1 reply; 6+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-05-02 19:24 UTC (permalink / raw)
To: Bjorn Andersson
Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
Krzysztof Kozlowski, Mathieu Poirier, Matthias Brugger,
Rob Herring, Tinghan Shen, devicetree, linux-arm-kernel,
linux-kernel, linux-mediatek, linux-remoteproc
The SCP has three memory regions: sram, l1tcm and cfg. Only sram is
required, the other two are optional. Fix the dt-binding so that the
optional regions can be omitted and passed in any order.
Also add the missing minItems to the reg property and update the
description.
Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
---
(no changes since v1)
.../devicetree/bindings/remoteproc/mtk,scp.yaml | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
index 823a236242de..ec9ddeb6ca2c 100644
--- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
@@ -24,14 +24,20 @@ properties:
reg:
description:
Should contain the address ranges for memory regions SRAM, CFG, and
- L1TCM.
+ L1TCM. Only SRAM is required, while CFG and L1TCM are optional.
+ minItems: 1
maxItems: 3
reg-names:
+ minItems: 1
items:
- const: sram
- - const: cfg
- - const: l1tcm
+ - enum:
+ - l1tcm
+ - cfg
+ - enum:
+ - l1tcm
+ - cfg
clocks:
description:
--
2.36.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp
2022-05-02 19:24 [PATCH v2 0/2] Mediatek SCP dt-binding tweaks Nícolas F. R. A. Prado
2022-05-02 19:24 ` [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp Nícolas F. R. A. Prado
@ 2022-05-02 19:24 ` Nícolas F. R. A. Prado
2022-05-03 14:16 ` Rob Herring
1 sibling, 1 reply; 6+ messages in thread
From: Nícolas F. R. A. Prado @ 2022-05-02 19:24 UTC (permalink / raw)
To: Bjorn Andersson
Cc: AngeloGioacchino Del Regno, kernel, Nícolas F. R. A. Prado,
Krzysztof Kozlowski, Mathieu Poirier, Matthias Brugger,
Rob Herring, Tinghan Shen, devicetree, linux-arm-kernel,
linux-kernel, linux-mediatek, linux-remoteproc
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
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp
2022-05-02 19:24 ` [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp Nícolas F. R. A. Prado
@ 2022-05-03 12:20 ` Krzysztof Kozlowski
0 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2022-05-03 12:20 UTC (permalink / raw)
To: Nícolas F. R. A. Prado, Bjorn Andersson
Cc: AngeloGioacchino Del Regno, kernel, Krzysztof Kozlowski,
Mathieu Poirier, Matthias Brugger, Rob Herring, Tinghan Shen,
devicetree, linux-arm-kernel, linux-kernel, linux-mediatek,
linux-remoteproc
On 02/05/2022 21:24, Nícolas F. R. A. Prado wrote:
> The SCP has three memory regions: sram, l1tcm and cfg. Only sram is
> required, the other two are optional. Fix the dt-binding so that the
> optional regions can be omitted and passed in any order.
I commented for v1 - order is strict, so no for this change.
>
> Also add the missing minItems to the reg property and update the
> description.
>
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
>
> ---
>
> (no changes since v1)
>
> .../devicetree/bindings/remoteproc/mtk,scp.yaml | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
> index 823a236242de..ec9ddeb6ca2c 100644
> --- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
> @@ -24,14 +24,20 @@ properties:
> reg:
> description:
> Should contain the address ranges for memory regions SRAM, CFG, and
> - L1TCM.
> + L1TCM. Only SRAM is required, while CFG and L1TCM are optional.
> + minItems: 1
> maxItems: 3
>
> reg-names:
> + minItems: 1
> items:
> - const: sram
> - - const: cfg
> - - const: l1tcm
> + - enum:
> + - l1tcm
> + - cfg
> + - enum:
> + - l1tcm
> + - cfg
Nope.
>
> clocks:
> description:
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp
2022-05-02 19:24 ` [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp Nícolas F. R. A. Prado
@ 2022-05-03 14:16 ` Rob Herring
2022-05-03 14:18 ` AngeloGioacchino Del Regno
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2022-05-03 14:16 UTC (permalink / raw)
To: Nícolas F. R. A. Prado
Cc: Rob Herring, linux-mediatek, Mathieu Poirier, linux-kernel,
devicetree, Matthias Brugger, linux-arm-kernel, Bjorn Andersson,
linux-remoteproc, Tinghan Shen, kernel,
AngeloGioacchino Del Regno, Krzysztof Kozlowski
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>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp
2022-05-03 14:16 ` Rob Herring
@ 2022-05-03 14:18 ` AngeloGioacchino Del Regno
0 siblings, 0 replies; 6+ messages in thread
From: AngeloGioacchino Del Regno @ 2022-05-03 14:18 UTC (permalink / raw)
To: Rob Herring, Nícolas F. R. A. Prado
Cc: Rob Herring, linux-mediatek, Mathieu Poirier, linux-kernel,
devicetree, Matthias Brugger, linux-arm-kernel, Bjorn Andersson,
linux-remoteproc, Tinghan Shen, kernel, Krzysztof Kozlowski
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>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-05-03 14:18 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-02 19:24 [PATCH v2 0/2] Mediatek SCP dt-binding tweaks Nícolas F. R. A. Prado
2022-05-02 19:24 ` [PATCH v2 1/2] dt-bindings: remoteproc: mediatek: Fix optional reg-names for mtk,scp Nícolas F. R. A. Prado
2022-05-03 12:20 ` Krzysztof Kozlowski
2022-05-02 19:24 ` [PATCH v2 2/2] dt-bindings: remoteproc: mediatek: Add optional memory-region to mtk,scp Nícolas F. R. A. Prado
2022-05-03 14:16 ` Rob Herring
2022-05-03 14:18 ` AngeloGioacchino Del Regno
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).