On Mon, Nov 18, 2024 at 01:02:00PM +0000, Matt Coster wrote: > Like the existing AXE-1-16M integration, BXS-4-64 uses the single clock > integration in the TI k3-j721s2. > > Signed-off-by: Matt Coster <matt.coster@xxxxxxxxxx> > --- > Changes in v2: > - Use normal reg syntax for 64-bit values > - Link to v1: https://lore.kernel.org/r/20241105-sets-bxs-4-64-patch-v1-v1-8-4ed30e865892@xxxxxxxxxx > --- > .../devicetree/bindings/gpu/img,powervr-rogue.yaml | 41 ++++++++++++++++++++++ > 1 file changed, 41 insertions(+) > > diff --git a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml > index 9dc55a6d0d4023983a3fc480340351f3fa974ce5..b620baa56a4caa41246f7b53064d0e3309bdda8e 100644 > --- a/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml > +++ b/Documentation/devicetree/bindings/gpu/img,powervr-rogue.yaml > @@ -18,6 +18,11 @@ properties: > - ti,am62-gpu > - const: img,img-axe-1-16m > - const: img,img-rogue > + - items: > + - enum: > + - ti,j721s2-gpu > + - const: img,img-bxs-4-64 > + - const: img,img-rogue > > # This legacy combination of compatible strings was introduced early on before the more > # specific GPU identifiers were used. Keep it around here for compatibility, but never use > @@ -78,6 +83,18 @@ allOf: > properties: > power-domains: > maxItems: 1 > + # Cores with two power domains > + - if: > + properties: > + compatible: > + contains: > + anyOf: Nope > + - const: img,img-bxs-4-64 > + then: > + properties: > + power-domains: > + minItems: 2 > + maxItems: 2 You need to constrain power-domain-names. Same for other variants. > # Vendor integrations using a single clock domain > - if: > properties: > @@ -85,6 +102,7 @@ allOf: > contains: > anyOf: > - const: ti,am62-gpu > + - const: ti,j721s2-gpu > then: > properties: > clocks: > @@ -105,3 +123,26 @@ examples: > power-domains = <&k3_pds 187 TI_SCI_PD_EXCLUSIVE>; > power-domain-names = "a"; > }; > + - | > + #include <dt-bindings/interrupt-controller/irq.h> > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/soc/ti,sci_pm_domain.h> Drop. No difference in this example. Best regards, Krzysztof
