Re: [PATCH v7 1/2] dt-bindings: ti: Update audio-refclk binding and j721e system controller

[Moteen Shah]

On 17/07/26 14:02, Moteen Shah wrote:

> Add ti,j721s2-audio-refclk as a supported compatible string in the
> ti,am62-audio-refclk binding. J721S2 uses the same audio reference
> clock IP block first introduced on AM62. Per writing-bindings
> guidelines, the J721S2-specific compatible is added as the primary
> string with ti,am62-audio-refclk as the fallback.
>
> Also extend the ti,j721e-system-controller clock-controller@ child
> pattern to accept audio-refclk schemas alongside ehrpwm-tbclk via a
> oneOf constraint, fixing the alphanumerical ordering of $refs.
>
> Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> Signed-off-by: Moteen Shah <m-shah@xxxxxx>

Hi Stephen, Michael, Brian,

Just checking in to see if you or the DT reviewers have any feedback
on this clock bindings patch of this series, or if it is ok to be picked 
up?

Regards,
Moteen

> ---
>   .../devicetree/bindings/clock/ti,am62-audio-refclk.yaml   | 8 ++++++--
>   .../bindings/soc/ti/ti,j721e-system-controller.yaml       | 6 ++++--
>   2 files changed, 10 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/ti,am62-audio-refclk.yaml b/Documentation/devicetree/bindings/clock/ti,am62-audio-refclk.yaml
> index b2e40bd39a3af..6c8fb0793070d 100644
> --- a/Documentation/devicetree/bindings/clock/ti,am62-audio-refclk.yaml
> +++ b/Documentation/devicetree/bindings/clock/ti,am62-audio-refclk.yaml
> @@ -11,8 +11,12 @@ maintainers:
>   
>   properties:
>     compatible:
> -    items:
> -      - const: ti,am62-audio-refclk
> +    oneOf:
> +      - items:
> +          - const: ti,am62-audio-refclk
> +      - items:
> +          - const: ti,j721s2-audio-refclk
> +          - const: ti,am62-audio-refclk
>   
>     reg:
>       maxItems: 1
> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,j721e-system-controller.yaml b/Documentation/devicetree/bindings/soc/ti/ti,j721e-system-controller.yaml
> index f3bd0be3b279f..364be49f8c7c7 100644
> --- a/Documentation/devicetree/bindings/soc/ti/ti,j721e-system-controller.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/ti,j721e-system-controller.yaml
> @@ -53,9 +53,11 @@ patternProperties:
>   
>     "^clock-controller@[0-9a-f]+$":
>       type: object
> -    $ref: /schemas/clock/ti,am654-ehrpwm-tbclk.yaml#
> +    oneOf:
> +      - $ref: /schemas/clock/ti,am62-audio-refclk.yaml#
> +      - $ref: /schemas/clock/ti,am654-ehrpwm-tbclk.yaml#
>       description:
> -      Clock provider for TI EHRPWM nodes.
> +      Clock provider for TI EHRPWM or Audio Reference Clock nodes.
>   
>     "phy@[0-9a-f]+$":
>       type: object