Re: [PATCH 1/3] dt-bindings: media: i2c: max9286: Add support for per-port supplies
From: Rob Herring <robh@kernel.org>
Date: 2021-12-17 14:18:09
Also in:
linux-media, linux-renesas-soc
On Fri, Dec 17, 2021 at 12:09:44AM +0200, Laurent Pinchart wrote:
quoted hunk ↗ jump to hunk
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Power supplies for the ports can be controlled per port depending on the hardware design. Support per-port supplies in the DT bindings, mutually exclusive with the global supply. Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- .../bindings/media/i2c/maxim,max9286.yaml | 25 ++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-)diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml index 02f656e78700..33aa307e8ee5 100644 --- a/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max9286.yaml@@ -39,7 +39,7 @@ properties: maxItems: 1 poc-supply: - description: Regulator providing Power over Coax to the cameras + description: Regulator providing Power over Coax to all the ports enable-gpios: description: GPIO connected to the \#PWDN pin with inverted polarity@@ -160,6 +160,10 @@ properties: additionalProperties: false +patternProperties: + "^port[0-3]-poc-supply$": + description: Regulator providing Power over Coax for a particular port + required: - compatible - reg@@ -167,6 +171,25 @@ required: - i2c-mux - gpio-controller +allOf: + - if: + required: + - poc-supply + then: + allOf: + - not: + required: + - port0-poc-supply + - not: + required: + - port1-poc-supply + - not: + required: + - port2-poc-supply + - not: + required: + - port3-poc-supply
I think you can invert the if and move patternProperties to the 'then' and...
+ additionalProperties: false
then use unevaluatedProperties here. Rob