Hi Tom, On Fri, Jul 19, 2013 at 1:56 PM, Charles Coldwell <[email protected]> wrote: > On Fri, Jul 19, 2013 at 12:55 PM, Tom Rini <[email protected]> wrote: >> On Fri, Jul 19, 2013 at 10:01:34AM -0300, Fabio Estevam wrote: >> >>> From: Fabio Estevam <[email protected]> >>> >>> 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 <[email protected]> >> >> 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 [email protected] http://lists.denx.de/mailman/listinfo/u-boot
