openwrtv4/target
Felix Fietkau 60b81acd60 ath79: dev-eth: Don't advertise 1gbit in link code word on ar9331
While the AR9331 has a gigabit MAC towards the internal switch, the
integrated PHYs however are only 100-base-tx capable.  The existing code
however advertieses gigabit capability in the link status word.  If you
attach such a PHY to a gigabit capable switch on the remote end, with
some probability it attempts to negotiate gigabit and fails, falling
baco to the AR9331 assuming a 10mbit half-duplex link.  This has been
observed quite frequently with the Carambola2 and gigabit capable
switches.

In ath79_register_eth(), "pdata->has_gbit = 1;" is set unconditionally
for both AR9331 ethernet ports. This is most likely wrong. Despite the
two MAC IP cores being gigabit MACs, the MAC for eth1 is connected to a
100base-T PHY via MII. The has_gbit attribute is used in the ethernet
driver to determine the supported link modes.

So either pdata->has_gbit is not set to 1 anymore, or the ethernet
driver needs to be modified to determine the advertised link code word
on another criteria than pdata->has_gbit.  This patch implements the
former solution.

Signed-off-by: Harald Welte <laforge@gnumonks.org>

SVN-Revision: 42432
2014-09-07 09:45:32 +00:00
..
imagebuilder imagebuilder: remove postinst files before generating image 2014-08-31 11:06:31 +00:00
linux ath79: dev-eth: Don't advertise 1gbit in link code word on ar9331 2014-09-07 09:45:32 +00:00
sdk sdk: keep $(STAGING_DIR)/usr/bin, at least curl needs it 2014-09-01 22:34:37 +00:00
toolchain target/toolchain: strip files, leave out the initial gcc 2014-08-26 12:32:51 +00:00
Config.in kernel: add a NAND_SUPPORT symbol 2014-06-11 12:59:22 +00:00
Makefile target: do not make target/*/install depend on target/*/compile - removes one redundant kernel build dir call on target/install 2012-06-06 17:24:05 +00:00