[PATCH v2 2/5] dt-bindings: misc: qcom,fastrpc: allow FastRPC node without reg entry

Dylan Van Assche posted 5 patches 2 years, 10 months ago
There is a newer version of this series
[PATCH v2 2/5] dt-bindings: misc: qcom,fastrpc: allow FastRPC node without reg entry
Posted by Dylan Van Assche 2 years, 10 months ago
FastRPC nodes may exist without a registry property and actual
Compute Banks  in some cases such as the SLPI on the Qualcomm SDM845 SoC.
Relax the regex to allow such FastRPC nodes and remove 'reg' as a
required property.

Signed-off-by: Dylan Van Assche <me@dylanvanassche.be>
---
 Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml b/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
index 1ab9588cdd89..0ccd3296d4f2 100644
--- a/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
+++ b/Documentation/devicetree/bindings/misc/qcom,fastrpc.yaml
@@ -64,7 +64,7 @@ properties:
     const: 0
 
 patternProperties:
-  "(compute-)?cb@[0-9]*$":
+  "(compute-)?cb(@[0-9]*)?$":
     type: object
 
     description: >
@@ -89,7 +89,6 @@ patternProperties:
 
     required:
       - compatible
-      - reg
 
     additionalProperties: false
 
-- 
2.39.2
Re: [PATCH v2 2/5] dt-bindings: misc: qcom,fastrpc: allow FastRPC node without reg entry
Posted by Krzysztof Kozlowski 2 years, 10 months ago
On 27/03/2023 20:48, Dylan Van Assche wrote:
> FastRPC nodes may exist without a registry property and actual

Why would they exist without unit address?

> Compute Banks  in some cases such as the SLPI on the Qualcomm SDM845 SoC.

I do not see there anything disallowing unit address yet...


> Relax the regex to allow such FastRPC nodes and remove 'reg' as a
> required property.

No, at least not without proper explanation why unit address is not
allowed for these.


Best regards,
Krzysztof