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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Thu, 05 Jun 2008 22:19:46 +0200
From:	Felix Fietkau <nbd@...nwrt.org>
To:	Trent Piepho <tpiepho@...escale.com>
CC:	netdev@...r.kernel.org
Subject: Re: RFC: Custom PHY device detection functions

Trent Piepho wrote:
> On Wed, 4 Jun 2008, Felix Fietkau wrote:
>> I'm working on Linux on small consumer grade wireless routers. Many of these 
>> have switches that connect directly to the Ethernet controller through 
>> MII/MDIO. These typically support 6-8 ports, one of which connects to the 
>> CPU. I am working on supporting some of these with PHY drivers, which I will 
>> submit for review when they're ready.
>> Most of these devices can be detected with the regular PHY ID and mask, but 
>> unfortunately there are some that lay out their registers a bit differently 
>> and thus need custom detection.
>> I worked around this with the following patch, that allows drivers to do 
>> their own detection, but I'm not sure if this is suitable for inclusion in 
>> the PHY layer or if there is a different way to solve this.
>> Any ideas?
> 
> If the phy uses non-standard registers, isn't it too late after the phy
> layer has already read the phy_id?
Reading phy_id does not cause any problems, it just doesn't give us 
anything base detection on.

> If not, could you use the existing probe function?  Set phy_id_mask to 0 and
> have the probe function return ENODEV if it's the wrong device.  I think the
> driver layer will go through all matching drivers, calling each one's probe
> function, until one of them claims the device.
Thanks, I'll try that.

- Felix
--
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