| Message ID | 20230427195538.2718661-1-hugo@hugovil.com |
|---|---|
| State | New |
| Headers | show |
| Series | imx8mn-var-som: dts: fix PHY detection bug by adding deassert delay | expand |
Hi Hugo, On Thu, Apr 27, 2023 at 4:56 PM Hugo Villeneuve <hugo@hugovil.com> wrote: > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > While testing the ethernet interface on a symphony carrier > board using an imx8mn SOM with an onboard PHY (EC hardware > configuration), the ethernet PHY is not detected. > > The device tree in Variscite custom linux git repository uses the > following property: > > phy-reset-post-delay = <20>; > > Add a new property 'reset-deassert-us' of 20ms to have the same delay > inside the ethphy handle. Which Ethernet PHY does this board use? What does its datasheet recommend?
On Thu, Apr 27, 2023 at 5:06 PM Hugo Villeneuve <hugo@hugovil.com> wrote: > Hi Fabio, > it uses a ADIN1300 PHY. > > The datasheet indicate that the "Management interface active (t4)" state is reached at most 5ms after the reset signal is deasserted. Please add this information to the commit log and please add a Fixes: tag. Thanks
On 27.04.23 22:12, Hugo Villeneuve wrote: > On Thu, 27 Apr 2023 17:07:59 -0300 > Fabio Estevam <festevam@gmail.com> wrote: > >> On Thu, Apr 27, 2023 at 5:06 PM Hugo Villeneuve <hugo@hugovil.com> wrote: >> >>> Hi Fabio, >>> it uses a ADIN1300 PHY. >>> >>> The datasheet indicate that the "Management interface active (t4)" state is reached at most 5ms after the reset signal is deasserted. >> >> Please add this information to the commit log and please add a Fixes: tag. > > Good idea, will do. Please also add the PHY name into the DT, e.g.: ethphy: ethernet-phy@4 { /* ADIN1300 */ I find this very useful when bringing up a new board and looking for similar DTs. Thanks, Ahmad > > Hugo. > >
Hello Hugo, On 28.04.23 00:18, Hugo Villeneuve wrote: > On Thu, 27 Apr 2023 22:16:45 +0200 > Ahmad Fatoum <a.fatoum@pengutronix.de> wrote: >> Please also add the PHY name into the DT, e.g.: >> >> ethphy: ethernet-phy@4 { /* ADIN1300 */ >> >> I find this very useful when bringing up a new board and looking >> for similar DTs. > > Hi Ahmad, > altough I agree it is a good idea, I prefer to not add this information for now, because there is probably some (old) versions of this SOM which use other PHY devices (maybe AR8033), but I am not 100% sure as this is not clear from the SOM manufacturer infos/website. /* ADIN1300 on new revisions */ would work too. Cheers, Ahmad > > Hugo. >
diff --git a/arch/arm64/boot/dts/freescale/imx8mn-var-som.dtsi b/arch/arm64/boot/dts/freescale/imx8mn-var-som.dtsi index 67072e6c77d5..9052b0d4b5b4 100644 --- a/arch/arm64/boot/dts/freescale/imx8mn-var-som.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mn-var-som.dtsi @@ -103,6 +103,7 @@ ethphy: ethernet-phy@4 { reg = <4>; reset-gpios = <&gpio1 9 GPIO_ACTIVE_LOW>; reset-assert-us = <10000>; + reset-deassert-us = <20000>; }; }; };