Re: [PATCH v5 2/2] arm64: dts: ti: Add audio overlay for k3-j721s2-evm

[Nishanth Menon <nm@ti.com>]

On 22:09-20260528, Wang, Sen wrote:
> On 5/20/2026 6:56 AM, Moteen Shah wrote:
> > From: Jayesh Choudhary <j-choudhary@ti.com>
> > 
> > Add device tree overlay to enable analog audio support on J721S2-EVM
> > using PCM3168A codec connected to McASP4 serializers.
> (snip)> +&scm_conf {
> > +	#address-cells = <1>;
> > +	#size-cells = <1>;
> > +
> > +	audio_refclk1: clock-controller@42e4 {
> > +		compatible = "ti,j721s2-audio-refclk", "ti,am62-audio-refclk";
> > +		reg = <0x42e4 0x4>;
> > +		clocks = <&k3_clks 157 299>;
> > +		assigned-clocks = <&k3_clks 157 299>;
> > +		assigned-clock-parents = <&k3_clks 157 328>;
> > +		#clock-cells = <0>;
> > +	};
> > +};
> 
> Hi Moteen,
> 
> Should the bulk of this scm_conf block go to the SoC-dtsi instead? It looks
> like it could be a part of J721 SoC dtsi, which currently doesn't seem to
> have any audio refclks defined.

See 6111ac92f9341 as an example? audio refclk is the primary function,
why make it as board dependent function?


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource