Hi Thomas, Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxxxxxxxxx> writes: >> I built a first kernel with the 'else' branch above removed (rationale: >> SERDES config was not set for RGMII prior to the patch). The issue >> remained. >> >> - mvneta_gmac_rgmii_set(pp, 1); >> + val = mvreg_read(pp, MVNETA_GMAC_CTRL_2); >> + >> + val |= MVNETA_GMAC2_PCS_ENABLE | MVNETA_GMAC2_PORT_RGMII; >> >> I removed MVNETA_GMAC2_PCS_ENABLE flag setting (rationale: it was only >> done for SGMII prior to the patch and all my platforms have RGMII >> connected PHY). *The issue disappeared on all 3 NAS with that simple >> change*. > > Yes, issue known. See https://bugzilla.kernel.org/show_bug.cgi?id=73401. > > Unfortunately, at this point I don't have enough informations to fix > both issues: the issue you're having, and the issue of using mvneta as > a module. Since the issue you're having is much more problematic, I'll > send a patch to revert my patch, until I have enough informations to > fix the problem in a proper way. > > Thanks for your report! Thanks for your work and the fast feedback. Do not hesitate to bug me if you want me to do some tests at some point. Cheers, a+ -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html