On 04/01/2024 22:15, Javier Carrasco wrote: > On 04.01.24 19:52, Dhruva Gole wrote: > >> >> This series also broke boot on TI SK-AM62x [0]. >> > >>> >>> This looks like a correct fix to me. >>> Could you please send a proper PATCH with Fixes tag? Thanks! >> >> Thanks for reviewing this Roger, the same patch above worked for me to >> fix SK-AM62x as well [1]. >> >> [0] https://storage.kernelci.org/next/master/next-20240103/arm64/defconfig/gcc-10/lab-ti/baseline-nfs-am62xx_sk-fs.txt >> [1] https://gist.github.com/DhruvaG2000/326b5d7fab4be95f20cd0aac4125f577 >> > Hi Dhruva, > > I am glad that you guys found a fix that quickly. > > it seems that you guys work for the device manufacturer (because of your > email addresses), so I was wondering if you could explain (or provide > the documentation) why the tps6598x should not receive the GAID command > and a reset crashes the system. Everything looks exactly the same as for > the tps25750, but in that case there are no complaints from sending a > cold reset. Looking at the kernel logs I don't see any crashes. Looks like the baseline-nfs.login test failed due to some reason [2]. I cannot see why though. [1] https://linux.kernelci.org/soc/ti/job/next/kernel/next-20240103/plan/baseline-nfs/ [2] https://linux.kernelci.org/test/plan/id/659561d445debed180c795fc/ -- cheers, -roger
