* Jeroen Hofstee <jhofstee@xxxxxxxxxxxxxxxxx> [161021 00:37]:
Hello Tony,Oh OK thanks for explaining it :) As it's already am3517 specific:
On 21-10-16 08:38, Tony Lindgren wrote:
* Jeroen Hofstee <jhofstee@xxxxxxxxxxxxxxxxx> [161020 12:57]:Yes, well, they get the same mac address at least. But does it matter?
Commit b6745f6e4e63 ("drivers: net: cpsw: davinci_emac: move reading macHmm doesn't this now break it for cpsw with two instances?
id to common file") did not only move the code for an am3517, it also
added the slave parameter, resulting in a invalid (all zero) mac address
being returned. So change it back to always read from slave zero, so it
This changes davinci_emac_3517_get_macid, the only way to get there
if (of_device_is_compatible(dev->of_node, "ti,am3517-emac"))
return davinci_emac_3517_get_macid(dev, 0x110, slave, mac_addr)
and the only user of ti,am3517-emac is arch/arm/boot/dts/am3517.dtsi,
which only has one emac. So the change is already am3517 specific.
We may need am3517 specific quirk flag instead?Given above, it is already am3517 specific. Let me know if you prefer this
route then I will have a look at it.
Acked-by: Tony Lindgren <tony@xxxxxxxxxxx>