Even though the MT8188 SoC's Dual-Core SCP IP is practically the
same as the one found on MT8195, it doesn't have a dedicated L1
TCM and relies only on SRAM.
Set reg/reg-names minItems to 1 globally and override it in all of
the conditionals for the SoCs that require more, and then split
the mt8195/8188 conditionals to allow specifying only the cfg MMIO
on MT8188.
Fixes: 91e0d560b9fd ("dt-bindings: remoteproc: mediatek: Support MT8188
dual-core SCP")
Signed-off-by: AngeloGioacchino Del Regno
<[email protected]>
---
Changes in v2:
- Fix dtc warnings and validate again
CHKDT ./Documentation/devicetree/bindings
LINT ./Documentation/devicetree/bindings
DTEX Documentation/devicetree/bindings/remoteproc/mtk,scp.example.dts
DTC [C] Documentation/devicetree/bindings/remoteproc/mtk,scp.example.dtb
Done.
.../bindings/remoteproc/mtk,scp.yaml | 19 +++++++++++++++----
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
index 179c98b33b4d..f0cae3e4bc78 100644
--- a/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
+++ b/Documentation/devicetree/bindings/remoteproc/mtk,scp.yaml
@@ -28,11 +28,11 @@ properties:
description:
Should contain the address ranges for memory regions SRAM, CFG, and,
on some platforms, L1TCM.
- minItems: 2
+ minItems: 1
maxItems: 3
reg-names:
- minItems: 2
+ minItems: 1
maxItems: 3
clocks:
@@ -185,7 +185,7 @@ allOf:
then:
properties:
reg:
- maxItems: 3
+ minItems: 3
reg-names:
items:
- const: sram
@@ -196,11 +196,22 @@ allOf:
compatible:
enum:
- mediatek,mt8188-scp-dual
+ then:
+ properties:
+ reg:
+ maxItems: 1
+ reg-names:
+ items:
+ - const: cfg
+ - if:
+ properties:
+ compatible:
+ enum:
- mediatek,mt8195-scp-dual
then:
properties:
reg:
- maxItems: 2
+ minItems: 2
reg-names:
items:
- const: cfg
--
2.52.0
