On 03/10/2016 22:57, Christian Lamparter wrote:
>> > why do you want to remove this check ? imho it does no harm
> The C-60 doesn't have a PHY at 3. This caused the check in ar8xxx_is_possible
> to fail and the ethernet ports on the C-60. Also, it doesn't look like the
> qca8k.c (DSA) driver checks for it. So I removed it.
your patch failed to mention this. remving the code because of a buggy
switch seems odd. please modify the code to just skip missing phy ids.
John
_______________________________________________
Lede-dev mailing list
Lede-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/lede-dev
