在 2021/11/22 下午10:07, Andrew Lunn 写道:
On Mon, Nov 22, 2021 at 08:14:57PM +0800, Yinbo Zhu wrote:
After module compilation, module alias mechanism will generate a ugly
mdio modules alias configure if ethernet phy was selected, this patch
is to fixup mdio alias garbled code.
In addition, that ugly alias configure will cause ethernet phy module
doens't match udev, phy module auto-load is fail, but add this patch
that it is well mdio driver alias configure match phy device uevent.
What PHY do you have problems with? What is the PHY id and which
driver should be loaded.
about that phy id, phy dev read it from PHY Identifier 1 and
Identifier 2 register, phy driver will call MODULE_DEVICE_TABLE to
configure
phy id to mdio_device_id, phy id was used to do a match phy driver
with phy device. that phy problems is phy driver was select 'M' then it
doesn't be auto load.
This code has existed a long time, so suddenly saying it is wrong and
changing it needs a good explanation why it is wrong. Being ugly is
not a good reason.
Andrew
Hi Andrew,
Use default mdio configure, After module compilation, mdio alias
configure is "alias mdio:0000000101000001000011111001???? marvell"
and it doesn't match the match phy dev(mdio dev) uevent, because
the mdio alias configure "0000000101000001000011111001????"
include "?" and "binary number", as general, uevent it include
one string or some string that string consist of one character and one
hexadecimal digit ,
which uevent is reported by mdio when mdio register a device for
ethernet phy device, only uevent from phy dev match alias configure from
phy driver that phy driver will can be auto-load when phy driver
was selected 'M'.
BRs
Yinbo Zhu.
