net: sun8i_emac: Drop redundant and incorrect setting of syscon register
authorChen-Yu Tsai <wens@csie.org>
Fri, 22 Jul 2016 10:16:10 +0000 (18:16 +0800)
committerHans de Goede <hdegoede@redhat.com>
Tue, 26 Jul 2016 19:56:03 +0000 (21:56 +0200)
commita85ba87dbe24816f2119e7475e255cc08b30934b
treea540e0fd0f3cb8d2809e6927320e7c7772a0555b
parent687284483c15b569da25f4727b3449e1e1d0dc17
net: sun8i_emac: Drop redundant and incorrect setting of syscon register

In sun8i_emac_board_setup, the driver partially configures the syscon
register for H3 EPHY. However, the settings are incomplete, and
completely unusable. The correct settings are later set in
sun8i_emac_set_syscon, but the incorrect CLK_SEL setting persists.

It is incorrect to use CLK_SEL to select 25 MHz, as the SoC does not
have a 25 MHz clock the EPHY can use.

This patch removes the setting of the syscon register in board_setup,
and also moves set_syscon above mdio_init. While mdio_init does not
access the PHY, it is better to have the PHY parameters setup before
the MDIO bus is registered.

Fixes: a29710c525ff ("net: Add EMAC driver for H3/A83T/A64 SoCs.")
Signed-off-by: Chen-Yu Tsai <wens@csie.org>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/net/sun8i_emac.c