netdev - Re: [PATCH] I.MX6: Fix Ethernet PHY mode on Ventana boards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20151208144138.GB13328@lunn.ch>
Date:	Tue, 8 Dec 2015 15:41:38 +0100
From:	Andrew Lunn <andrew@...n.ch>
To:	Tim Harvey <tharvey@...eworks.com>
Cc:	Krzysztof Ha??asa <khalasa@...p.pl>,
	netdev <netdev@...r.kernel.org>, Shawn Guo <shawnguo@...nel.org>,
	Fabio Estevam <Fabio.Estevam@...escale.com>
Subject: Re: [PATCH] I.MX6: Fix Ethernet PHY mode on Ventana boards

> This may be a stupid question but what does the Marvell phy driver
> offer that the generic phy driver does not?

It reliably configures the RGMII delays :-)

No need to depend on your bootloader doing it.

It allows interrupts to be used instead of polling the PHY all the
time, assuming you have the interrupt wired up.  There are
peculiarities for enabling autoneg which is knows about. It might put
the LEDs into a well known state, but that depends on the model. It
might enable auto-crossover on some PHYs. It has some errata
workarounds, support for WOL. You can put a list of register writes in
DT which will be performed when auto neg is enabled. This could be
used for example to configure the LEDs.

Not yet posted, but i have a patch to add HWMON support to read the
temperature sensor in some of the PHYs.

	    Andrew
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html