From mboxrd@z Thu Jan  1 00:00:00 1970
From: Troy Kisky <troy.kisky@boundarydevices.com>
Date: Mon, 4 Jan 2016 15:06:27 -0700
Subject: [U-Boot] [PATCH 2/2] mx6cuboxi: Make Ethernet functional
In-Reply-To: <CAOMZO5AUvPWPj20RxWwg=-cZoh6g6uDSBKx1gYw9ir1uGD0-vA@mail.gmail.com>
References: <1451925378-14121-1-git-send-email-festevam@gmail.com>
 <1451925378-14121-2-git-send-email-festevam@gmail.com>
 <568AD8C2.4070602@boundarydevices.com>
 <CAOMZO5AUvPWPj20RxWwg=-cZoh6g6uDSBKx1gYw9ir1uGD0-vA@mail.gmail.com>
Message-ID: <568AECE3.1020004@boundarydevices.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 1/4/2016 2:50 PM, Fabio Estevam wrote:
> Hi Troy,
> 
> On Mon, Jan 4, 2016 at 6:40 PM, Troy Kisky
> <troy.kisky@boundarydevices.com> wrote:
> 
>> Except for clearing bit 2 (which is undocumented), this looks just like the default
>> implementation, which you are still calling below.
>>
>> So, it looks like this subroutine can be replaced with an appropriate delay.
>> If bit 2 does matter, perhaps the default ar8035_config should be changed instead.
> 
> Just realized that ar8035_config() is never called in my case, so doing:
> 
> int board_phy_config(struct phy_device *phydev)
> {
>     mx6_rgmii_rework(phydev);
> 
>     return 0;
> }
> 
> , is enough here.
> 
> Any idea as to why ar8035_config() is never called?
> 

=> mii read 6 2
004D
=> mii read 6 3
D072
=>

Do you get a different value?