On 05.01.24 09:12, Jai Luthra wrote: > I added some prints and can see that the probe fails once at > fwnode_usb_role_switch_get() because the other endpoint (of USB device) > is not yet probed. It then re-probes later where it passes. > > The GAID reset being done unconditionally in your series seems to cause > the board to get stuck in the boot process when it hits the above error > due to probe-order between USB subsystem and this IC. My guess would be > SoC stops getting power because we reset the PD chip? > > Anyway, I will send below change as a separate "Fixes:" patch for now, > to keep how things as they were before your series. > My biggest concern is that we are sending GAID for the tps25750 under the same circumstances. Could we not have the same problem with that device? We would be resetting the PD controller and the SoC would stop getting power as well, right? Or is there anything device-specific that would avoid that? > If you have a better architecture in mind that can reset only when PTCH > has been applied and not for other probe defers, feel free to send it on > top of it. > I added the cold reset to have the same behavior upon probe failures for both devices, given that they use the same command. But if that can cause problems, let's leave the reset alone... Best regards, Javier Carrasco
