On Fri, Apr 29, 2022 at 10:31:05AM -0300, Fabio Estevam wrote:
Hi Tom,
On Fri, Apr 29, 2022 at 9:44 AM Tom Rini trini@konsulko.com wrote:
Thanks for the hint, Tim! Fabio, the dts needs to be re-synced with Linux as upstream it says: /* * The PHY seems to require a long-enough reset duration to avoid * some rare issues where the PHY gets stuck in an inconsistent and * non-functional state at boot-up. 10ms proved to be fine . */ phy-reset-duration = <10>;
But we have this set to 2, and indeed bumping it up to 10 to match fixes the network tests for me.
Great, care to send a patch to U-Boot that aligns this cubox dts with upstream?
I do have a preliminary patch that syncs all imx6dl/q dts with upstream.
I haven't submitted it yet, as it needs lots of careful manual inspection, especially with the alias nodes.
Still want to submit it, but it is not a trivial one due to the high chances of boot regressions.
I'd rather test your overall sync because I assume based on doing other resyncs before it'll end up pulling in a lot of other dts files.