From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mugunthan V N Date: Tue, 15 Sep 2015 13:27:45 +0530 Subject: [U-Boot] [PATCH] net: phy: on phy device create do not initialize link to 1 In-Reply-To: References: <1441275621-6929-1-git-send-email-mugunthanvnm@ti.com> Message-ID: <55F7CF79.3070900@ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Friday 04 September 2015 09:01 PM, Joe Hershberger wrote: > On Thu, Sep 3, 2015 at 5:20 AM, Mugunthan V N 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 >> Signed-off-by: Mugunthan V N > > Acked-by: Joe Hershberger > Joe, A gentle ping on this patch. Regards Mugunthan V N