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:	Tue, 18 Feb 2014 17:39:20 +0000
From:	Ben Dooks <>
To:	Sergei Shtylyov <>
Subject: Re: [PATCH] sh_eth: call of_mdiobus_register() to register phys

On 18/02/14 18:11, Sergei Shtylyov wrote:
> Hello.
> On 02/18/2014 07:00 PM, Ben Dooks wrote:
>>>>>> If the sh_eth device is registered using OF, then the driver
>>>>>     Which is not supported yet as my DT patch hasn't been merged.
>>>>> This patch seems somewhat premature.
>>>> I've got your OF patches in my local tree to test with, this
>>>> is what I found during that testing.
>>>     The issue is that I didn't post my v3 patch to netdev due to
>>> net-next.git repo being closed at this moment and DaveM not wanting to
>>> see any patch targeted to it during this time. I've now posted v4 of my
>>> Ether DT patch to netdev.
>> Ok, I will look for these tomorrow.
>     No significant changes there...
>>>>>> should call of_mdiobus_register() to register any PHYs connected
>>>>>> to the system.
>>>>>     That's not necessary (but good to have).
>>>> Well, it is necessary if you then want any PHYS bound to
>>>> the device to have their OF information to hand,
>>>     Ether DT support worked for me without this fragment, at least.
>> Yes, it just that the PHY is not being linked to the relevant
>> OF node. The PHY gets bound, it will not be able to find the
>> DT info passed.
>     With no DT support in the PHY driver, I don't see how it matters.
> Perhaps it has to do with your "init-regs" prop patch though...

Yes, or if we add some other properties to the PHY node to say
how to initialise the registers, such as if the PHY node had.

I am going to look in to adding led<name> initialisers to the
PHY node in case people do not like my init-regs patch.

Ben Dooks
Senior Engineer				Codethink - Providing Genius
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to
More majordomo info at

Powered by blists - more mailing lists