Re: [PATCH v2 4/4] arm64: dts: imx8mp: change AUDIO_AXI_CLK_ROOT freq. to 800MHz

[Mihalcea Laurentiu <laurentiumihalcea111@gmail.com>]

On 21.02.2025 21:37, Adam Ford wrote:
> On Fri, Feb 21, 2025 at 1:11 PM Laurentiu Mihalcea
> <laurentiumihalcea111@gmail.com> wrote:
>> From: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
>>
>> AUDIO_AXI_CLK_ROOT can't run at currently requested 600MHz w/ its parent
>> SYS_PLL1 configured at 800MHz. Configure it to run at 800MHz as some
>> applications running on the DSP expect the core to run at this frequency
>> anyways. This change also affects the AUDIOMIX NoC.
> Unless I am missing something, the i.MX 8M Plus Applications Processor
> Datasheet (rev 2.1)  has a table of frequencies, and
> AUDIO_AXI_CLK_ROOT is shown to be 600MHz nominal and 800MHz for
> overdrive.  I agree that it's likely not running at 600MHz now, but
> 800MHz may be out of spec for people who are using the nominal voltage
> instead of the overdrive, since overdrive requires higher voltages
> than the nominal.
>
> adam


You're right, this would not work if someone was using

nominal voltage. Do you have an upstream board in mind/usecase which

works with nominal voltage instead of overdrive? As far as I understood from

Lucas's patch ([1]) there's no upstream board using nominal voltage?


CC-ing Lucas on this as his patch ([1]) is similar to this in that they're both

trying to use overdrive clock frequencies in the DTSI and I'd very much like

to have some sort of consistency in the DTSI if possible.



[1]: https://lore.kernel.org/imx/20250204182737.3361431-1-l.stach@pengutronix.de/