[REGRESSION] Re: [PATCH v2 16/24] clk: imx: Convert clock-osc-* back to osc_*

[Christoph Niedermaier]

From: Francesco Dolcini <francesco at dolcini.it>
Sent: Wednesday, April 16, 2025 11:27 AM
> On Tue, Apr 15, 2025 at 02:13:30PM -0300, Fabio Estevam wrote:
>> On Tue, Apr 15, 2025 at 1:55 PM Marek Vasut <marex at denx.de> wrote:
>>> Do you have af9cdd1ccd2d ("Revert "arm64: dts: imx8mn: Include 32kHz
>>> oscillator clock in SPL DTs"") in place ? If so, try and revert it, does
>>> it help ?
>>
>> It seems I have not pushed the "Revert "arm64: dts: imx8mn: Include 32kHz
>> oscillator clock in SPL DTs" fix. Sorry about that.
>>
>> As Francesco is using imx8mp, then I guess we also need to do the same
>> for imx8mp:
>>
>> diff --git a/arch/arm/dts/imx8mp-u-boot.dtsi b/arch/arm/dts/imx8mp-u-boot.dtsi
>> index 6de9ab5d37cf..35af412f01b0 100644
>> --- a/arch/arm/dts/imx8mp-u-boot.dtsi
>> +++ b/arch/arm/dts/imx8mp-u-boot.dtsi
>> @@ -37,11 +37,6 @@
>>         /delete-property/ assigned-clock-rates;
>>  };
>>
>> -&osc_32k {
>> -       bootph-pre-ram;
>> -       bootph-all;
>> -};
>> -
>>  &osc_24m {
>>         bootph-pre-ram;
>>         bootph-all;
>>
>> Does this fix the boot problem?
> 
> My issue is not the boot. The issue is that USB is not working.
> 
> With this patch applied nothing changes, the issue is not solved.
> 
> Francesco

Hello,

I don't know if it is related or not, I noticed that Adam Ford's patches
commit 6d33ca36e3b ("clk: imx8mm: register UART clocks") and
commit 8999b76f238 ("clk: imx8mn: register UART clocks")
use the former name "clock-osc-24m". But Marek's patch
commit b4734c9c333 ("clk: imx: Convert clock-osc-* back to osc_*")
converted clock-osc-* to osc_*. So I think the UART clocks should also
use the current name osc_*.

Regards
Christoph