[PATCH v2 1/5] dt-bindings: soc: cix,sky1-system-control: add audss system control

[Joakim Zhang]

Hi Krzysztof,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk at kernel.org>
> Sent: Friday, June 5, 2026 5:21 PM
> To: Joakim Zhang <joakim.zhang at cixtech.com>; mturquette at baylibre.com;
> sboyd at kernel.org; bmasney at redhat.com; robh at kernel.org;
> krzk+dt at kernel.org; conor+dt at kernel.org; p.zabel at pengutronix.de; Gary Yang
> <gary.yang at cixtech.com>
> Cc: cix-kernel-upstream <cix-kernel-upstream at cixtech.com>; linux-
> clk at vger.kernel.org; devicetree at vger.kernel.org; linux-kernel at vger.kernel.org;
> linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH v2 1/5] dt-bindings: soc: cix,sky1-system-control: add audss
> system control
> 
> EXTERNAL EMAIL
> 
> On 05/06/2026 11:18, Krzysztof Kozlowski wrote:
> > On 05/06/2026 05:22, joakim.zhang at cixtech.com wrote:
> >>    compatible:
> >> -    items:
> >> -      - enum:
> >> -          - cix,sky1-system-control
> >> -          - cix,sky1-s5-system-control
> >> -      - const: syscon
> >> +    oneOf:
> >> +      - items:
> >> +          - enum:
> >> +              - cix,sky1-system-control
> >> +              - cix,sky1-s5-system-control
> >> +          - const: syscon
> >> +      - items:
> >> +          - const: cix,sky1-audss-system-control
> >> +          - const: simple-mfd
> >> +          - const: syscon
> >>
> >>    reg:
> >>      maxItems: 1
> >> @@ -27,6 +32,11 @@ properties:
> >>    '#reset-cells':
> >>      const: 1
> >>
> >> +  clock-controller:
> >> +    $ref: /schemas/clock/cix,sky1-audss-clock.yaml#
> >> +    description:
> >> +      AUDSS internal clock provider (cix,sky1-audss-system-control only).
> >
> > Are you sure this patch builds? Your cover letter should explain
> > merging
> 
> I am sure it does not...
> 
> I recommend switching to compatible-style of defining subnodes in parent
> schema which would decouple patches.
> 
> example:
> https://elixir.bootlin.com/linux/v7.1-
> rc6/source/Documentation/devicetree/bindings/display/msm/qcom,sm8750-
> mdss.yaml#L41

Thank you for your guidance. I will study it further.

Joakim