lkml.org 
[lkml]   [2021]   [Nov]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v1 1/2] modpost: file2alias: fixup mdio alias garbled code in modules.alias
From
Date

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

\
 
 \ /
  Last update: 2021-11-23 03:22    [W:0.118 / U:0.148 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site