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

Reply via email to