[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20121104101212.33449081@skate>
Date: Sun, 4 Nov 2012 10:12:12 +0100
From: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>
To: Jason Gunthorpe <jgunthorpe@...idianresearch.com>
Cc: Lior Amsalem <alior@...vell.com>, Andrew Lunn <andrew@...n.ch>,
Jason Cooper <jason@...edaemon.net>, netdev@...r.kernel.org,
Maen Suleiman <maen@...vell.com>,
Gregory Clement <gregory.clement@...e-electrons.com>,
linux-arm-kernel@...ts.infradead.org,
Ian Molton <ian.molton@...ethink.co.uk>
Subject: Re: [4/4] arm: mvebu: enable Ethernet controllers on Armada 370/XP
eval boards
Hello Jason,
On Sat, 3 Nov 2012 20:03:11 -0600, Jason Gunthorpe wrote:
> We've been using the patches from Ian Molton for the mv643xx driver
> that make it use the more standard, separately described MDIO
> bus..
[...]
> Where the MDIO bus is explicit, the PHY, its address and parameters
> are explicit - the PHY has an of_node pointer - and phy-handle is used
> to connect them.
>
> I'm not sure having the MDIO bus as a distinct top level item is
> great, considering how the registers overlap.. It might be better
> to put it under the egiga0 block? But that is a minor nit :)
Thanks for your comment. I'll see what I can come up with to make the
phy a separate DT entity.
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
--
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
Powered by blists - more mailing lists