Re: [PATCH 1/9] arm: dts: renesas: lager: Specify ethernet PHY reset timings
Geert Uytterhoeven <[email protected]>
| Newsgroups | org.kernel.vger.linux-renesas-soc,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <CAMuHMdXv8rqGGdFjXqBw4RbzHdoy=CAWY0WdRfsvjSccm=kvxA@mail.gmail.com> |
Hi Marek, On Tue, 14 Jul 2026 at 15:04, Marek Vasut <[email protected]> wrote: > The KSZ8041RNL reference manual [1] DS00002245C page 47 TABLE 7-10: > POWER-UP/RESET TIMING PARAMETERS does not indicate how long should > the system wait after deassertion of the PHY reset and before start > of communication with the PHY via MDIO. Opt for the same value as > used for KSZ9031RNX, which is 300 us. > > The KSZ8041RNL reference manual [1] DS00002245C page 47 TABLE 7-10: > POWER-UP/RESET TIMING PARAMETERS row tSR Stable supply voltages to > reset high is at minimum 10 ms. Set the DT property reset-assert-us > to 10ms because the KSZ8041RNL RM does not explicitly spell out how > long the reset has to be asserted, but this at least covers the worst > case scenario. Thanks for your patch! > [1] https://ww1.microchip.com/downloads/aemDocuments/documents/UNG/ProductDocuments/DataSheets/00002841D.pdf That's the URL for KSZ9131RNX. The correct URL is: https://ww1.microchip.com/downloads/aemDocuments/documents/UNG/ProductDocuments/DataSheets/KSZ8041NL-RNL-Data-Sheet-DS00002245.pdf > Signed-off-by: Marek Vasut <[email protected]> Reviewed-by: Geert Uytterhoeven <[email protected]> i.e. will queue in renesas-devel for v7.4, with the URL corrected. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds