| Message ID | 217c8197efaee7d803b22d433abb0ea8e33b84c6.1619700314.git.geert+renesas@glider.be |
|---|---|
| State | Accepted |
| Commit | 7935bb56e21b2add81149f4def8e59b4133fe57c |
| Headers | show |
| Series | dt-bindings: media: renesas,vin: Make resets optional on R-Car Gen1 | expand |
Hi Geert, On 2021-04-29 14:45:52 +0200, Geert Uytterhoeven wrote: > The "resets" property is not present on R-Car Gen1 SoCs. > Supporting it would require migrating from renesas,cpg-clocks to > renesas,cpg-mssr. > > Fixes: 905fc6b1bfb4a631 ("dt-bindings: rcar-vin: Convert bindings to json-schema") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > Note that VIN is not yet enabled in r8a777[89].dtsi. > --- > .../bindings/media/renesas,vin.yaml | 46 ++++++++++++------- > 1 file changed, 29 insertions(+), 17 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml > index 79c936b313ea91ae..19fccc2fd8ece120 100644 > --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml > +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml > @@ -194,23 +194,35 @@ required: > - interrupts > - clocks > - power-domains > - - resets > - > -if: > - properties: > - compatible: > - contains: > - enum: > - - renesas,vin-r8a7778 > - - renesas,vin-r8a7779 > - - renesas,rcar-gen2-vin > -then: > - required: > - - port > -else: > - required: > - - renesas,id > - - ports > + > +allOf: > + - if: > + not: > + properties: > + compatible: > + contains: > + enum: > + - renesas,vin-r8a7778 > + - renesas,vin-r8a7779 > + then: > + required: > + - resets > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - renesas,vin-r8a7778 > + - renesas,vin-r8a7779 > + - renesas,rcar-gen2-vin > + then: > + required: > + - port > + else: > + required: > + - renesas,id > + - ports > > additionalProperties: false > > -- > 2.25.1 >
On Thu, 29 Apr 2021 14:45:52 +0200, Geert Uytterhoeven wrote: > The "resets" property is not present on R-Car Gen1 SoCs. > Supporting it would require migrating from renesas,cpg-clocks to > renesas,cpg-mssr. > > Fixes: 905fc6b1bfb4a631 ("dt-bindings: rcar-vin: Convert bindings to json-schema") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- > Note that VIN is not yet enabled in r8a777[89].dtsi. > --- > .../bindings/media/renesas,vin.yaml | 46 ++++++++++++------- > 1 file changed, 29 insertions(+), 17 deletions(-) > Applied, thanks!
diff --git a/Documentation/devicetree/bindings/media/renesas,vin.yaml b/Documentation/devicetree/bindings/media/renesas,vin.yaml index 79c936b313ea91ae..19fccc2fd8ece120 100644 --- a/Documentation/devicetree/bindings/media/renesas,vin.yaml +++ b/Documentation/devicetree/bindings/media/renesas,vin.yaml @@ -194,23 +194,35 @@ required: - interrupts - clocks - power-domains - - resets - -if: - properties: - compatible: - contains: - enum: - - renesas,vin-r8a7778 - - renesas,vin-r8a7779 - - renesas,rcar-gen2-vin -then: - required: - - port -else: - required: - - renesas,id - - ports + +allOf: + - if: + not: + properties: + compatible: + contains: + enum: + - renesas,vin-r8a7778 + - renesas,vin-r8a7779 + then: + required: + - resets + + - if: + properties: + compatible: + contains: + enum: + - renesas,vin-r8a7778 + - renesas,vin-r8a7779 + - renesas,rcar-gen2-vin + then: + required: + - port + else: + required: + - renesas,id + - ports additionalProperties: false
The "resets" property is not present on R-Car Gen1 SoCs. Supporting it would require migrating from renesas,cpg-clocks to renesas,cpg-mssr. Fixes: 905fc6b1bfb4a631 ("dt-bindings: rcar-vin: Convert bindings to json-schema") Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- Note that VIN is not yet enabled in r8a777[89].dtsi. --- .../bindings/media/renesas,vin.yaml | 46 ++++++++++++------- 1 file changed, 29 insertions(+), 17 deletions(-)