[U-Boot] [PATCH] net: phy: on phy device create do not initialize link to 1

Joe Hershberger joe.hershberger at gmail.com
Wed Sep 30 19:27:33 CEST 2015


On Thu, Sep 24, 2015 at 4:31 PM, Joe Hershberger
<joe.hershberger at gmail.com> wrote:
> On Tue, Sep 15, 2015 at 2:57 AM, Mugunthan V N <mugunthanvnm at ti.com> wrote:
>> On Friday 04 September 2015 09:01 PM, Joe Hershberger wrote:
>>> On Thu, Sep 3, 2015 at 5:20 AM, Mugunthan V N <mugunthanvnm at ti.com> wrote:
>>>> Currently when phy device is created the link variable is
>>>> initialized to 1 which denoted phy link is already up. On a power
>>>> reset there is no issue as phy status register link status will
>>>> not be set, so phy auto negotiate will be started. But when a cpu
>>>> reset is issued (ex: dra72x-evm) phy's link status bit is already
>>>> set which leads to assume that link is already setup in
>>>> genphy_update_link() initial check which results in ehternet not
>>>> working. So do not assume that link is already up and on phy
>>>> device create set link to zero. This is verified on dra72x-evm.
>>>>
>>>> Reported-by: Franklin S Cooper Jr <fcooper at ti.com>
>>>> Signed-off-by: Mugunthan V N <mugunthanvnm at ti.com>
>>>
>>> Acked-by: Joe Hershberger <joe.hershberger at ni.com>
>>>
>>
>> Joe,
>>
>> A gentle ping on this patch.
>
> This will go in for next release, since it was outside the merge window.

Decided to pull this in for this release.

Applied to u-boot-net/master, thanks!

-Joe


More information about the U-Boot mailing list