[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAH+2xPCa2GNgtJ_5hr3dcaXzA+EB4JLfCsqDJQQY4pbxuw-4JQ@mail.gmail.com>
Date: Thu, 16 Jul 2020 21:32:06 +0200
From: Bruno Thomsen <bruno.thomsen@...il.com>
To: Fabio Estevam <festevam@...il.com>
Cc: linux-kernel <linux-kernel@...r.kernel.org>,
"moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE"
<linux-arm-kernel@...ts.infradead.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>, Shawn Guo <shawnguo@...nel.org>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
NXP Linux Team <linux-imx@....com>,
Bruno Thomsen <bth@...strup.com>
Subject: Re: [PATCH v2 2/2] ARM: dts: imx7: add support for kamstrup flex concentrator
Hi Fabio,
Den tor. 16. jul. 2020 kl. 20.01 skrev Fabio Estevam <festevam@...il.com>:
> On Thu, Jul 16, 2020 at 2:26 PM Bruno Thomsen <bruno.thomsen@...il.com> wrote:
>
> > Limitations: Ethernet PHY type auto detection does not
> > work when using reset-{assert-us,deassert-us,gpios}
> > properties so it's using a fixed PHY type ID for now. Auto
> > detection worked when using the deprecated FEC properties
> > phy-reset-{gpios,duration,post-delay}.
>
> I think we need to understand this better. Why does it fail?
Yes, we need to locate the root cause.
> I gave a test on an imx6q-sabresd and the Ethernet PHY (AR8031) could
> be properly detected using reset-{assert-us,deassert-us,gpios}
> properties inside the mdio node.
Okay, thanks.
> Is this a Micrel KSZ8081 specific issue?
Maybe, I'm actually beginning to suspect that the issue might be related
to polling vs interrupt mode. As all partially related examples I have seen
uses interrupt mode and I have only configured poll mode. But the hardware
does have the interrupt signal connected to the imx7.
> Please report this issue to the Ethernet PHY folks.
I'm going to send an Ethernet PHY mail tomorrow with the details I know
at the moment. Just wanted to get an updated version of the device tree
out in case of any other issues not related to the PHY part.
/Bruno
Powered by blists - more mailing lists