Hi Tom, On Fri, Jul 19, 2013 at 1:56 PM, Charles Coldwell <coldw...@gmail.com> wrote: > On Fri, Jul 19, 2013 at 12:55 PM, Tom Rini <tr...@ti.com> wrote: >> On Fri, Jul 19, 2013 at 10:01:34AM -0300, Fabio Estevam wrote: >> >>> From: Fabio Estevam <fabio.este...@freescale.com> >>> >>> Commit de1d786e (add support for Xilinx 1000BASE-X phy (GTX)) introduced the >>> checking for ESTATUS_1000_XHALF, but it incorrectly sets the >>> SUPPORTED_1000baseX_Full flag in this case. >>> >>> Set the SUPPORTED_1000baseX_Half flag instead. >>> >>> Signed-off-by: Fabio Estevam <fabio.este...@freescale.com> >> >> So, do we need both patches to fix the problem? > > One patch fixes "the" problem; the other fixes another problem we > haven't seen yet (but will).
I agree with Charles. Sascha Silbe's patch fixes the regression we noticed on some mx6 boards. This one is about a different issue and it is not related to the previous problem. IMHO we should apply both of them. Thanks, Fabio Estevam _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot