On 6.3.2017 23:13, Joe Hershberger wrote: > On Sun, Mar 5, 2017 at 8:36 AM, Nathan Rossi <nat...@nathanrossi.com> wrote: >> When the zynq_gem driver initializes the phy it sets the supported >> features that the phy can support and advertise. However instead of >> masking the supported features such that it limits the available >> features it sets the phy to have the exact supported features of the >> zynq_gem. This is problematic as it will enable features that a phy does >> not have or cannot advertise. >> >> Specifically this appears as an issue when using a phy that is only >> capable of 10/100, but the zynq_gem driver will override this and try to >> enable and advertise 10/100/1000. >> >> Reported-by: Arno Steffens <s...@gmx.li> >> Fixes: 80243528ef ("net: gem: Fix gem driver on 1Gbps LAN") >> Signed-off-by: Nathan Rossi <nat...@nathanrossi.com> >> Tested-by: Arno Steffens <s...@gmx.li> >> Cc: Joe Hershberger <joe.hershber...@ni.com> >> Cc: Michal Simek <michal.si...@xilinx.com> > > Acked-by: Joe Hershberger <joe.hershber...@ni.com> >
Acked-by: Michal Simek <michal.si...@xilinx.com> Thanks, Michal _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot