netdev - Re: RFC: Custom PHY device detection functions

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] [day] [month] [year] [list]

Message-ID: <48484A62.8010208@openwrt.org>
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