Re: [PATCH v2 1/1] arm64: dts: ti: k3-j722s-evm: Switch audio codec clock from audio_refclk1 to external clock

[Jared McArthur <j-mcarthur@ti.com>]

On 9/3/26 15:00, Nishanth Menon wrote:
> On 16:05-20260804, Jared McArthur wrote:
>> In revision PROC170E1 of the J722SXH01EVM [0], the audio codec's
>> (TLV320AIC3106IRGZT [1]) MCLK was provided by the AUDIO_EXT_REFCLK1
>> (96MHz). Revisions PROC170E2 and later switched the MCLK's source
>> to an external LMK6CE012288CDLFT [2] oscillator (12.288MHz). The
>> PROC170E1 revision of the board is not publicly available.
>>
>> Fix the audio codec's clock by creating a new fixed clock node with
>> a frequency of 12.288MHz, and switch the audio codec's clock from
>> the audio_refclk1 to the new fixed clock. Remove pinmuxing and clock
>> muxing for the audio_refclk1.
>>
>> [0] https://www.ti.com/lit/zip/SPRR495
>> [1] https://www.ti.com/lit/gpn/TLV320AIC3106
>> [2] https://www.ti.com/lit/gpn/lmk6c
>>
> 
> Do we need:
> Fixes: 3a36c535dfd2 ("arm64: dts: ti: k3-j722s-evm: Enable analog audio support")
> 
> ?
> 

Thank you for the comment.

I can add the fixes tag. I was unsure about adding it originally,
because the commit did work for the original revision. To me,
it's debatable whether the commit introduced a bug, because the
bug is due to a hardware change.

-- 
Best,
Jared McArthur