Re: [PATCH v1 1/2] modpost: file2alias: fixup mdio alias garbled code in modules.alias

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




在 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.




[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux