From mboxrd@z Thu Jan  1 00:00:00 1970
From: Joe Hershberger <joe.hershberger@ni.com>
Date: Tue, 30 Apr 2019 21:15:32 +0000
Subject: [U-Boot] Pull request: u-boot-net.git master
In-Reply-To: <20190319224131.GC18421@bill-the-cat>
References: <20190312181625.2ED12FF225@linux-xvxi.amer.corp.natinst.com>
 <20190319224131.GC18421@bill-the-cat>
Message-ID: <CANr=Z=bdCCHGVzYUynwvVAbXb=8pK2F+3gRwVhu9=G+5sHsRBw@mail.gmail.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 Tue, Mar 19, 2019 at 5:41 PM Tom Rini <trini@konsulko.com> wrote:
>
> On Tue, Mar 12, 2019 at 01:15:46PM -0500, Joe Hershberger wrote:
>
> > Hi Tom,
> >
> > These patches passed the CI build here: https://travis-ci.org/jhershbe/u-boot/builds/501807294
> >
> > The following changes since commit 2e8092d94f40a5692baf3ec768ce3216a7bf032a:
> >
> >   Merge branch 'master' of git://git.denx.de/u-boot-sunxi (2019-03-11 15:48:57 -0400)
> >
> > are available in the git repository at:
> >
> >
> >   git://git.denx.de/u-boot-net.git master
> >
> > for you to fetch changes up to 85f05f72bacc2d047731fc648055551e4f6b34cf:
> >
> >   net: phy: aquantia: Set only autoneg on in register 4.c441 (2019-03-12 13:13:37 -0500)
> >
>
> NAK.  One of:
> The first bad commit could be any of:
> 30b2ca2e0fa274b875bb56f541b7dd33ce93c1d1
> 8860e1563f38d16f7ae29053018cd445c0fa111d
> ebb5027d69196dd83fd0fa5bd91fca07acfd77be
> 09e0a36497c84273e5b22488d5af01bf0ba17469
> 841b9df209e37fe1bfefa5f44e837a0ad497443f
> 15e67d1cdc7258c0c07ad1fd6c2818f7e9f52447
> 7aadf5134f2f5771689d0657b69875d0a464859d
> d35488518f3c16d305092c816a5129f45a0b62d7
> Breaks am335x_evm ethernet:
> 18:39:52 => => dhcp
> 18:39:52 ethernet at 4a100000 Waiting for PHY auto negotiation to
> complete... done
> 18:39:52 link up on port 0, speed 1000, full duplex
> 18:39:52 BOOTP broadcast 1
> 18:39:52 BOOTP broadcast 2
> 18:39:52 BOOTP broadcast 3
> 18:39:52 BOOTP broadcast 4
> 18:39:52 BOOTP broadcast 5
> 18:39:52 BOOTP broadcast 6
> 18:39:52 BOOTP broadcast 7
> 18:39:52 BOOTP broadcast 8
> 18:39:52 BOOTP broadcast 9
> 18:39:52 BOOTP broadcast 10
> 18:39:52 BOOTP broadcast 11
> 18:39:52 BOOTP broadcast 12
> 18:39:52 BOOTP broadcast 13
> 18:39:52 BOOTP broadcast 14
> 18:39:52 BOOTP broadcast 15
> 18:39:52 BOOTP broadcast 16
> 18:39:52 BOOTP broadcast 17

I rebased the series on the current master and I can't reproduce this
dhcp issue. On the original series I saw broken DHCP only with "net:
phy: micrel: Use correct skew values on KSZ9021" which doesn't make
any sense because that phy is not used on BBB and isn't even compiled
in. Also, the issue doesn't reproduce when the next patch is applied.
Even that oddity doesn't happen after the rebase.

Also the SPL for boneblack is too big to build with some of the
patches in this series now, so I'm not sure how that should be
handled.

Thanks,
-Joe