Re: [PATCH RFC 5/6] riscv: dts: thead: Add PVT node

Michal Wilczynski/Kernel (PLT) /SRPOL/Engineer/Samsung Electronics
2025-05-27T08:54:53Z

W dniu 27.05.2025 o 10:00, Drew Fustini pisze:
> On Sat, May 24, 2025 at 11:14:59PM +0200, Michal Wilczynski wrote:
>> Add PVT DT node for thermal sensor.
>>
>> Signed-off-by: Michal Wilczynski <m.wilczynski@samsung.com>
>> ---
>> arch/riscv/boot/dts/thead/th1520.dtsi | 11 +++++++++++
>> 1 file changed, 11 insertions(+)
>>
>> diff --git a/arch/riscv/boot/dts/thead/th1520.dtsi b/arch/riscv/boot/dts/thead/th1520.dtsi
>> index f24e12d7259fabcfbdc2dfa966d759db06684ab4..faf5c3aaf209b24cd99ddc377a88e08a8cce24fe 100644
>> --- a/arch/riscv/boot/dts/thead/th1520.dtsi
>> +++ b/arch/riscv/boot/dts/thead/th1520.dtsi
>> @@ -648,6 +648,17 @@ padctrl_aosys: pinctrl@fffff4a000 {
>> thead,pad-group = <1>;
>> };
>>
>> + pvt: pvt@fffff4e000 {
>> + compatible = "moortec,mr75203";
>> + reg = <0xff 0xfff4e000 0x0 0x80>,
>> + <0xff 0xfff4e080 0x0 0x100>,
>> + <0xff 0xfff4e180 0x0 0x680>,
>> + <0xff 0xfff4e800 0x0 0x600>;
>> + reg-names = "common", "ts", "pd", "vm";
>> + clocks = <&aonsys_clk>;
>> + #thermal-sensor-cells = <1>;
>> + };
>> +
>> gpio@fffff52000 {
>> compatible = "snps,dw-apb-gpio";
>> reg = <0xff 0xfff52000 0x0 0x1000>;
>>
>> --
>> 2.34.1
>>
> I found that on my lpi4a that boot while hang after applying this patch.
> I think that it is related to clocks as boot finished okay when using
> clk_ignore_unused on the kernel cmdline. Do you happen have that in your
> kernel cmdline?

Right I had that option enabled, that's why I've missed this. Thanks for

letting me know ! I'll remove this option for future testing.

>
> I need to investigate further to understand which clocks are causing the
> problem.

No problem I can look into that as well, most likely towards the end

of the week as I'm travelling and don't have access to the board.

>
> Thanks,
> Drew
>