On Tue, Jun 04, 2024 at 03:47:20PM +0200, Marc Gonzalez wrote: > HDMI TX block embedded in the APQ8098. > > Signed-off-by: Marc Gonzalez <mgonzalez@xxxxxxxxxx> > --- > Documentation/devicetree/bindings/display/msm/hdmi.yaml | 26 ++++++++++++++++++++++++-- > 1 file changed, 24 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/msm/hdmi.yaml b/Documentation/devicetree/bindings/display/msm/hdmi.yaml > index 47e97669821c3..13199f1676f7c 100644 > --- a/Documentation/devicetree/bindings/display/msm/hdmi.yaml > +++ b/Documentation/devicetree/bindings/display/msm/hdmi.yaml > @@ -19,14 +19,15 @@ properties: > - qcom,hdmi-tx-8974 > - qcom,hdmi-tx-8994 > - qcom,hdmi-tx-8996 > + - qcom,hdmi-tx-8998 > > clocks: > minItems: 1 > - maxItems: 5 > + maxItems: 8 > > clock-names: > minItems: 1 > - maxItems: 5 > + maxItems: 8 > > reg: > minItems: 1 > @@ -151,6 +152,27 @@ allOf: > - const: extp > hdmi-mux-supplies: false You need 'maxItems: 5' for clocks in the prior if/then schema. > > + - if: > + properties: > + compatible: > + contains: > + enum: > + - qcom,hdmi-tx-8998 > + then: > + properties: > + clocks: > + minItems: 8 > + clock-names: > + items: > + - const: mdp_core > + - const: mnoc > + - const: iface > + - const: bus > + - const: iface_mmss > + - const: core > + - const: alt_iface > + - const: extp Please match the order of 8996 for the ones that are common. Rob