atom feed43 messages in org.freebsd.freebsd-archRe: Extending sys/dev/mii
FromSent OnAttachments
Stefan BethkeJan 4, 2012 8:03 am 
Adrian ChaddJan 4, 2012 9:07 am 
Marius StroblJan 4, 2012 1:59 pm 
Stefan BethkeJan 5, 2012 12:52 pm 
Stefan BethkeJan 6, 2012 4:56 am 
Stefan BethkeJan 6, 2012 5:02 am.diff
Marius StroblJan 6, 2012 10:27 am 
Stefan BethkeJan 6, 2012 12:35 pm 
Marius StroblJan 6, 2012 1:47 pm 
Stefan BethkeJan 6, 2012 1:52 pm 
Marius StroblJan 8, 2012 5:00 am 
Stefan BethkeJan 8, 2012 2:27 pm 
Adrian ChaddJan 10, 2012 5:18 pm 
Marius StroblJan 11, 2012 11:37 am 
Adrian ChaddJan 14, 2012 8:15 pm 
Warner LoshJan 14, 2012 9:13 pm 
Stefan BethkeJan 20, 2012 3:08 pm.patch
Warner LoshJan 20, 2012 5:43 pm 
Oleksandr TymoshenkoJan 20, 2012 8:12 pm 
Stefan BethkeJan 21, 2012 4:02 am 
Aleksandr RybalkoJan 21, 2012 5:12 am 
Oleksandr TymoshenkoJan 23, 2012 11:45 pm 
Aleksandr RybalkoJan 24, 2012 5:53 am 
Marius StroblJan 25, 2012 2:17 pm 
Warner LoshJan 25, 2012 2:28 pm 
Marius StroblJan 25, 2012 3:21 pm 
Hans Petter SelaskyJan 26, 2012 8:24 am 
Stefan BethkeJan 26, 2012 8:24 am 
Stefan BethkeJan 26, 2012 8:30 am 
Marius StroblJan 27, 2012 6:15 am 
Adrian ChaddFeb 10, 2012 9:22 pm 
Marius StroblFeb 11, 2012 3:17 am 
Aleksandr RybalkoFeb 11, 2012 4:45 am 
Adrian ChaddFeb 11, 2012 4:59 pm 
Warner LoshFeb 11, 2012 5:15 pm 
Adrian ChaddFeb 11, 2012 8:48 pm 
Warner LoshFeb 11, 2012 8:58 pm 
Juli MallettFeb 11, 2012 9:05 pm 
Adrian ChaddFeb 11, 2012 9:19 pm 
Aleksandr RybalkoFeb 12, 2012 9:51 am 
Aleksandr RybalkoFeb 12, 2012 10:04 am 
Juli MallettFeb 12, 2012 10:30 am 
Aleksandr RybalkoFeb 12, 2012 10:38 am 
Subject:Re: Extending sys/dev/mii
From:Warner Losh (im@bsdimp.com)
Date:Feb 11, 2012 8:58:45 pm
List:org.freebsd.freebsd-arch

On Feb 11, 2012, at 9:49 PM, Adrian Chadd wrote:

On 11 February 2012 17:15, Warner Losh <im@bsdimp.com> wrote:

On Feb 11, 2012, at 6:00 PM, Adrian Chadd wrote:

I'd like to try and finally bring some sanity to the hardcoded PHY mask handling in if_arge (and make it actually work for AR71xx and AR724x - where AR71xx has one shared MDIO bus between both MACs, but AR724x has two independent MDIO busses..)

FDT would do that...

.. how would it bring sanity to the device driver?

FDT would encode the links between the MAC and the PHY. FDT is a directed
graph, not a tree, so you can have multiple paths to a device node that follow
different domains.

Right now the driver assumes that both arge0 and arge1 mdiobus are the same and uses the phymask setting to determine how/when to access registers (ie, trying to read/write from phy registers not in the phymask of argeX are denied.) It is one of the annoying issues with the AR7241 internal switch support as that switch hangs off of arge1's MII bus.

It would no longer need to do this, or make any assumptions at all. FDT would
tell it all.

It may make it easier to specify the configuration but it doesn't fix the fundamentally wrong assumption.

Nope. Bad assumptions would still need to be fixed.

The trouble in this whole mess (where FDT may help) is that phy's for arge0 may actually sit on arge1. So you can't probe/attach the miibus on arge0 until you've probe/attached arge1, so the arge1 MII registers can be tickled.

The PHYs don't sit on arge1. They sit on another device that the driver bogusly
assumes is tightly coupled to arge1, when in fact it isn't.

Ray/Stefan/others: if anything, I'd like to try and bring sanity to this particular thorny issue in -HEAD before we worry about switch PHY devices.

Warner