Hi Joe, 2016-12-09 19:35 GMT+01:00 Joe Hershberger <joe.hershber...@gmail.com>:
> On Fri, Dec 9, 2016 at 11:43 AM, Michal Simek <mon...@monstr.eu> wrote: > > Hi Joe, > > > > 2016-12-09 17:22 GMT+01:00 Joe Hershberger <joe.hershber...@gmail.com>: > >> > >> Hi Tom, > >> > >> On Fri, Dec 9, 2016 at 6:12 AM, Tom Rini <tr...@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. > I have some patches in master but nothing what is ready for merge. And Tom lost one email that he merged something what I didn't ask for. Anyway I think it is good reason start to use pull request based on tags not on branches. 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot