On Sun, May 05, 2024 at 09:07:49PM +0800, Weifeng Liu wrote: > This is an attempt to alleviate race conditions in the SAM driver where > essential resources like serial device and GPIO pins are not ready at > the time ssam_serial_hub_probe() is called. Instead of giving up > probing, a better way would be to defer the probing by returning > -EPROBE_DEFER, allowing the kernel try again later. > > However, there is no way of identifying all such cases from other real > errors in a few days. So let's take a gradual approach identify and > address these cases as they arise. This commit marks the initial step > in this process. It's a bit pointless to send a new version while we haven't settled yet down on the previous one. Moreover, there is no added details as I asked in the previous round of review. The decision of moving this part to serdev is up to Hans, but I think we also can at least put TODO line here with explanations you gave in the reply to v2 that this is currently the only driver needs this and there is still a chance that more might need it. While writing the above paragraph I realised that this might be due to non-standard appearance of the device in DSDT, that it gets enumerated before the controller. Do you have a DSDT excerpt for the controller and device parts in the order of appearance? -- With Best Regards, Andy Shevchenko
