From mboxrd@z Thu Jan  1 00:00:00 1970
From: Joe Hershberger <joe.hershberger@gmail.com>
Date: Fri, 9 Dec 2016 12:35:59 -0600
Subject: [U-Boot] Pull request: u-boot-net.git master
In-Reply-To: <CAHTX3d+zoG5aiD-mMRBXvV+4otGaw9-mX9-aRjKf_oJUf+g6mA@mail.gmail.com>
References: <20161208163755.23AD6FED0F@linux-xvxi.amer.corp.natinst.com>
 <20161209121259.GD4248@bill-the-cat>
 <CANr=Z=b1XoV+3dR1uXrZ62Fu3y6VEYryret-7OzmcxvgJQAbrg@mail.gmail.com>
 <CAHTX3d+zoG5aiD-mMRBXvV+4otGaw9-mX9-aRjKf_oJUf+g6mA@mail.gmail.com>
Message-ID: <CANr=Z=ZQymOxyxUqeiLz5kkDyXNULYNFNH+3iA+-uFe8dNoNLA@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 Fri, Dec 9, 2016 at 11:43 AM, Michal Simek <monstr@monstr.eu> wrote:
> Hi Joe,
>
> 2016-12-09 17:22 GMT+01:00 Joe Hershberger <joe.hershberger@gmail.com>:
>>
>> Hi Tom,
>>
>> On Fri, Dec 9, 2016 at 6:12 AM, Tom Rini <trini@konsulko.com> wrote:
>> > On Thu, Dec 08, 2016 at 10:37:26AM -0600, Joe Hershberger wrote:
>> >
>> >> Hi Tom,
>> >>
>> >> The following changes since commit
>> >> 388019f1e2166638453bc4e0cc5d138c2a19e0c9:
>> >>
>> >>   Merge branch 'master' of git://git.denx.de/u-boot-usb (2016-12-06
>> >> 08:07:20 -0500)
>> >>
>> >> are available in the git repository at:
>> >>
>> >>
>> >>   git://git.denx.de/u-boot-net.git master
>> >>
>> >> for you to fetch changes up to
>> >> bb135a0180c31fbd7456021fb9700b49bba7f533:
>> >>
>> >>   net/phy/vitesse: Rework RGMII skew configuration for VSC8601
>> >> (2016-12-08 10:36:22 -0600)
>> >
>> > This is a NAK and is what is breaking microblaze-generic and
>> > imgtec_xilfpga at least:

Uhh... so it's not a NAK? This PR seems to be in your master.

I've rebased and have no commits left on top of your master. I've
pushed it for travis as another test anyway.
https://travis-ci.org/jhershbe/u-boot/builds/182680490

>> > +(imgtec_xilfpga,microblaze-generic)   ret =
>> > mdio_register_seq(emaclite->bus,
>> > dev->seq);+(imgtec_xilfpga,microblaze-generic)   ^
>> > +(imgtec_xilfpga,microblaze-generic) drivers/net/built-in.o: In function
>> > `emaclite_probe':
>> > +(imgtec_xilfpga)
>> > ../drivers/net/xilinx_emaclite.c:(.text.emaclite_probe+0x64): undefined
>> > reference to `mdio_register_seq'
>> > +(imgtec_xilfpga,microblaze-generic) make[1]: *** [u-boot] Error 1
>> > +(imgtec_xilfpga,microblaze-generic) make: *** [sub-make] Error 2
>> > +(microblaze-generic)   ret = mdio_register_seq(priv->bus, dev->seq);
>> > +(microblaze-generic) drivers/net/built-in.o: In function
>> > `axi_emac_probe':             +(microblaze-generic)
>> > build/../drivers/net/xilinx_axi_emac.c:652: undefined reference to
>> > `mdio_register_seq'
>> > +(microblaze-generic) build/../drivers/net/xilinx_emaclite.c:570:
>> > undefined reference to `mdio_register_seq'
>>
>> This is strange... I build tested this on Travis.
>>
>> https://travis-ci.org/jhershbe/u-boot/builds/180270329
>>
>> Are we missing microblaze coverage on Travis or something?
>
>
>
> This patches went to the tree by accident. Tom reverted that incorrect
> merge.
> Adding mdio_register_seq was sent recently for review and it shouldn't have
> these compilation issue.
> I have no idea what's wrong with imgtec_xilfpga.
> And microblaze should be covered by travis already.
>
> Thanks,
> Michal
>
> --
> Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
> w: www.monstr.eu p: +42-0-721842854
> Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
> Maintainer of Linux kernel - Xilinx Zynq ARM architecture
> Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform