[PATCH 1/3] ARM: dts: sun7i: Fix A20-OLinuXino-MICRO dts for use with LAN8710

[Maxime Ripard]

Hi Stefan,

Please keep all the recipients in Cc.

On Thu, Aug 24, 2017 at 08:24:05AM +0300, Stefan Mavrodiev wrote:
> On 08/23/2017 05:30 PM, Maxime Ripard wrote:
> > On Tue, Aug 22, 2017 at 11:58:19PM +0300, Stefan Mavrodiev wrote:
> > > As I can see, if GMAC is used, this function is GNULL - same as PA14
> > > and PA9. I don't think that it's controlled by GMAC.
> >
> > Do you know if in such a case the level is properly asserted?
>
> I've checked the level with oscilloscope.

With the pin function set to GMAC? What was the outcome?

> However, I've fixed the patch replacing GMAC node with EMAC and
> setting PA17 as "emac" function. This way the PHY is working
> correctly. Since it's 100T can I do this or there is particular
> reason for using GMAC over EMAC?

In general, the GMAC driver has more features and is better maintained
than the EMAC one, so even it wouldn't change much in our case, it'd
be better to stick to the GMAC.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170825/541a6f3d/attachment.sig>