mirror of
				git://git.openwrt.org/openwrt/openwrt.git
				synced 2025-10-31 05:54:26 -04:00 
			
		
		
		
	Don't mask bit 4 of the AT8022 phy id. If bit 4 of the AT8022 phy id (0x004dd023) is masked, it will match the phy id of the AR8327 switch (0x004dd033) as well. It results in applied at803x driver settings/callbacks, which will at least limit the AR8327 phys to 100MBit operation instead of the possible 1000MBit. Signed-off-by: Mathias Kresin <dev@kresin.me>
		
			
				
	
	
		
			74 lines
		
	
	
		
			2.4 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
			
		
		
	
	
			74 lines
		
	
	
		
			2.4 KiB
		
	
	
	
		
			Diff
		
	
	
	
	
	
| From: Felix Fietkau <nbd@nbd.name>
 | |
| Subject: net: phy: at803x: add support for AT8032
 | |
| 
 | |
| Like AT8030, this PHY needs the GPIO reset workaround
 | |
| 
 | |
| Signed-off-by: Felix Fietkau <nbd@nbd.name>
 | |
| ---
 | |
| 
 | |
| --- a/drivers/net/phy/at803x.c
 | |
| +++ b/drivers/net/phy/at803x.c
 | |
| @@ -62,8 +62,10 @@
 | |
|  
 | |
|  #define ATH8030_PHY_ID 0x004dd076
 | |
|  #define ATH8031_PHY_ID 0x004dd074
 | |
| +#define ATH8032_PHY_ID 0x004dd023
 | |
|  #define ATH8035_PHY_ID 0x004dd072
 | |
|  #define AT803X_PHY_ID_MASK			0xffffffef
 | |
| +#define AT8032_PHY_ID_MASK			0xffffffff
 | |
|  
 | |
|  MODULE_DESCRIPTION("Atheros 803x PHY driver");
 | |
|  MODULE_AUTHOR("Matus Ujhelyi");
 | |
| @@ -256,7 +258,8 @@ static int at803x_probe(struct phy_devic
 | |
|  	if (!priv)
 | |
|  		return -ENOMEM;
 | |
|  
 | |
| -	if (phydev->drv->phy_id != ATH8030_PHY_ID)
 | |
| +	if (phydev->drv->phy_id != ATH8030_PHY_ID &&
 | |
| +	    phydev->drv->phy_id != ATH8032_PHY_ID)
 | |
|  		goto does_not_require_reset_workaround;
 | |
|  
 | |
|  	gpiod_reset = devm_gpiod_get_optional(dev, "reset", GPIOD_OUT_LOW);
 | |
| @@ -332,7 +335,7 @@ static void at803x_link_change_notify(st
 | |
|  	struct at803x_priv *priv = phydev->priv;
 | |
|  
 | |
|  	/*
 | |
| -	 * Conduct a hardware reset for AT8030 every time a link loss is
 | |
| +	 * Conduct a hardware reset for AT8030/2 every time a link loss is
 | |
|  	 * signalled. This is necessary to circumvent a hardware bug that
 | |
|  	 * occurs when the cable is unplugged while TX packets are pending
 | |
|  	 * in the FIFO. In such cases, the FIFO enters an error mode it
 | |
| @@ -444,6 +447,24 @@ static struct phy_driver at803x_driver[]
 | |
|  	.aneg_done		= at803x_aneg_done,
 | |
|  	.ack_interrupt		= &at803x_ack_interrupt,
 | |
|  	.config_intr		= &at803x_config_intr,
 | |
| +}, {
 | |
| +	/* ATHEROS 8032 */
 | |
| +	.phy_id			= ATH8032_PHY_ID,
 | |
| +	.name			= "Atheros 8032 ethernet",
 | |
| +	.phy_id_mask		= AT8032_PHY_ID_MASK,
 | |
| +	.probe			= at803x_probe,
 | |
| +	.config_init		= at803x_config_init,
 | |
| +	.link_change_notify	= at803x_link_change_notify,
 | |
| +	.set_wol		= at803x_set_wol,
 | |
| +	.get_wol		= at803x_get_wol,
 | |
| +	.suspend		= at803x_suspend,
 | |
| +	.resume			= at803x_resume,
 | |
| +	.features		= PHY_BASIC_FEATURES,
 | |
| +	.flags			= PHY_HAS_INTERRUPT,
 | |
| +	.config_aneg		= genphy_config_aneg,
 | |
| +	.read_status		= genphy_read_status,
 | |
| +	.ack_interrupt		= at803x_ack_interrupt,
 | |
| +	.config_intr		= at803x_config_intr,
 | |
|  } };
 | |
|  
 | |
|  module_phy_driver(at803x_driver);
 | |
| @@ -451,6 +472,7 @@ module_phy_driver(at803x_driver);
 | |
|  static struct mdio_device_id __maybe_unused atheros_tbl[] = {
 | |
|  	{ ATH8030_PHY_ID, AT803X_PHY_ID_MASK },
 | |
|  	{ ATH8031_PHY_ID, AT803X_PHY_ID_MASK },
 | |
| +	{ ATH8032_PHY_ID, AT8032_PHY_ID_MASK },
 | |
|  	{ ATH8035_PHY_ID, AT803X_PHY_ID_MASK },
 | |
|  	{ }
 | |
|  };
 |