forked from mirrors/linux
		
	net: phy: aquantia: add support for PHY LEDs
Aquantia Ethernet PHYs got 3 LED output pins which are typically used to indicate link status and activity. Add a minimal LED controller driver supporting the most common uses with the 'netdev' trigger as well as software-driven forced control of the LEDs. Signed-off-by: Daniel Golle <daniel@makrotopia.org> [ rework indentation, fix checkpatch error and improve some functions ] Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									c11d5dbbe7
								
							
						
					
					
						commit
						61578f6793
					
				
					 4 changed files with 252 additions and 3 deletions
				
			
		|  | @ -1,5 +1,5 @@ | |||
| # SPDX-License-Identifier: GPL-2.0
 | ||||
| aquantia-objs			+= aquantia_main.o aquantia_firmware.o | ||||
| aquantia-objs			+= aquantia_main.o aquantia_firmware.o aquantia_leds.o | ||||
| ifdef CONFIG_HWMON | ||||
| aquantia-objs			+= aquantia_hwmon.o | ||||
| endif | ||||
|  |  | |||
|  | @ -63,6 +63,28 @@ | |||
| #define VEND1_GLOBAL_CONTROL2_UP_RUN_STALL_OVD	BIT(6) | ||||
| #define VEND1_GLOBAL_CONTROL2_UP_RUN_STALL	BIT(0) | ||||
| 
 | ||||
| #define VEND1_GLOBAL_LED_PROV			0xc430 | ||||
| #define AQR_LED_PROV(x)				(VEND1_GLOBAL_LED_PROV + (x)) | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK2500		BIT(14) | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK5000		BIT(15) | ||||
| #define VEND1_GLOBAL_LED_PROV_FORCE_ON		BIT(8) | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK10000		BIT(7) | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK1000		BIT(6) | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK100		BIT(5) | ||||
| #define VEND1_GLOBAL_LED_PROV_RX_ACT		BIT(3) | ||||
| #define VEND1_GLOBAL_LED_PROV_TX_ACT		BIT(2) | ||||
| #define VEND1_GLOBAL_LED_PROV_ACT_STRETCH	GENMASK(0, 1) | ||||
| 
 | ||||
| #define VEND1_GLOBAL_LED_PROV_LINK_MASK		(VEND1_GLOBAL_LED_PROV_LINK100 | \ | ||||
| 						 VEND1_GLOBAL_LED_PROV_LINK1000 | \ | ||||
| 						 VEND1_GLOBAL_LED_PROV_LINK10000 | \ | ||||
| 						 VEND1_GLOBAL_LED_PROV_LINK5000 | \ | ||||
| 						 VEND1_GLOBAL_LED_PROV_LINK2500) | ||||
| 
 | ||||
| #define VEND1_GLOBAL_LED_DRIVE			0xc438 | ||||
| #define VEND1_GLOBAL_LED_DRIVE_VDD		BIT(1) | ||||
| #define AQR_LED_DRIVE(x)			(VEND1_GLOBAL_LED_DRIVE + (x)) | ||||
| 
 | ||||
| #define VEND1_THERMAL_PROV_HIGH_TEMP_FAIL	0xc421 | ||||
| #define VEND1_THERMAL_PROV_LOW_TEMP_FAIL	0xc422 | ||||
| #define VEND1_THERMAL_PROV_HIGH_TEMP_WARN	0xc423 | ||||
|  | @ -125,6 +147,8 @@ | |||
| #define VEND1_GLOBAL_INT_VEND_MASK_GLOBAL2	BIT(1) | ||||
| #define VEND1_GLOBAL_INT_VEND_MASK_GLOBAL3	BIT(0) | ||||
| 
 | ||||
| #define AQR_MAX_LEDS				3 | ||||
| 
 | ||||
| struct aqr107_hw_stat { | ||||
| 	const char *name; | ||||
| 	int reg; | ||||
|  | @ -149,6 +173,7 @@ static const struct aqr107_hw_stat aqr107_hw_stats[] = { | |||
| 
 | ||||
| struct aqr107_priv { | ||||
| 	u64 sgmii_stats[AQR107_SGMII_STAT_SZ]; | ||||
| 	unsigned long leds_active_low; | ||||
| }; | ||||
| 
 | ||||
| #if IS_REACHABLE(CONFIG_HWMON) | ||||
|  | @ -158,3 +183,18 @@ static inline int aqr_hwmon_probe(struct phy_device *phydev) { return 0; } | |||
| #endif | ||||
| 
 | ||||
| int aqr_firmware_load(struct phy_device *phydev); | ||||
| 
 | ||||
| int aqr_phy_led_blink_set(struct phy_device *phydev, u8 index, | ||||
| 			  unsigned long *delay_on, | ||||
| 			  unsigned long *delay_off); | ||||
| int aqr_phy_led_brightness_set(struct phy_device *phydev, | ||||
| 			       u8 index, enum led_brightness value); | ||||
| int aqr_phy_led_hw_is_supported(struct phy_device *phydev, u8 index, | ||||
| 				unsigned long rules); | ||||
| int aqr_phy_led_hw_control_get(struct phy_device *phydev, u8 index, | ||||
| 			       unsigned long *rules); | ||||
| int aqr_phy_led_hw_control_set(struct phy_device *phydev, u8 index, | ||||
| 			       unsigned long rules); | ||||
| int aqr_phy_led_active_low_set(struct phy_device *phydev, int index, bool enable); | ||||
| int aqr_phy_led_polarity_set(struct phy_device *phydev, int index, | ||||
| 			     unsigned long modes); | ||||
|  |  | |||
							
								
								
									
										150
									
								
								drivers/net/phy/aquantia/aquantia_leds.c
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										150
									
								
								drivers/net/phy/aquantia/aquantia_leds.c
									
									
									
									
									
										Normal file
									
								
							|  | @ -0,0 +1,150 @@ | |||
| // SPDX-License-Identifier: GPL-2.0
 | ||||
| /* LED driver for Aquantia PHY
 | ||||
|  * | ||||
|  * Author: Daniel Golle <daniel@makrotopia.org> | ||||
|  */ | ||||
| 
 | ||||
| #include <linux/phy.h> | ||||
| 
 | ||||
| #include "aquantia.h" | ||||
| 
 | ||||
| int aqr_phy_led_brightness_set(struct phy_device *phydev, | ||||
| 			       u8 index, enum led_brightness value) | ||||
| { | ||||
| 	if (index >= AQR_MAX_LEDS) | ||||
| 		return -EINVAL; | ||||
| 
 | ||||
| 	return phy_modify_mmd(phydev, MDIO_MMD_VEND1, AQR_LED_PROV(index), | ||||
| 			      VEND1_GLOBAL_LED_PROV_LINK_MASK | | ||||
| 			      VEND1_GLOBAL_LED_PROV_FORCE_ON | | ||||
| 			      VEND1_GLOBAL_LED_PROV_RX_ACT | | ||||
| 			      VEND1_GLOBAL_LED_PROV_TX_ACT, | ||||
| 			      value ? VEND1_GLOBAL_LED_PROV_FORCE_ON : 0); | ||||
| } | ||||
| 
 | ||||
| static const unsigned long supported_triggers = (BIT(TRIGGER_NETDEV_LINK) | | ||||
| 						 BIT(TRIGGER_NETDEV_LINK_100) | | ||||
| 						 BIT(TRIGGER_NETDEV_LINK_1000) | | ||||
| 						 BIT(TRIGGER_NETDEV_LINK_2500) | | ||||
| 						 BIT(TRIGGER_NETDEV_LINK_5000) | | ||||
| 						 BIT(TRIGGER_NETDEV_LINK_10000)  | | ||||
| 						 BIT(TRIGGER_NETDEV_RX) | | ||||
| 						 BIT(TRIGGER_NETDEV_TX)); | ||||
| 
 | ||||
| int aqr_phy_led_hw_is_supported(struct phy_device *phydev, u8 index, | ||||
| 				unsigned long rules) | ||||
| { | ||||
| 	if (index >= AQR_MAX_LEDS) | ||||
| 		return -EINVAL; | ||||
| 
 | ||||
| 	/* All combinations of the supported triggers are allowed */ | ||||
| 	if (rules & ~supported_triggers) | ||||
| 		return -EOPNOTSUPP; | ||||
| 
 | ||||
| 	return 0; | ||||
| } | ||||
| 
 | ||||
| int aqr_phy_led_hw_control_get(struct phy_device *phydev, u8 index, | ||||
| 			       unsigned long *rules) | ||||
| { | ||||
| 	int val; | ||||
| 
 | ||||
| 	if (index >= AQR_MAX_LEDS) | ||||
| 		return -EINVAL; | ||||
| 
 | ||||
| 	val = phy_read_mmd(phydev, MDIO_MMD_VEND1, AQR_LED_PROV(index)); | ||||
| 	if (val < 0) | ||||
| 		return val; | ||||
| 
 | ||||
| 	*rules = 0; | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_LINK100) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_LINK_100); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_LINK1000) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_LINK_1000); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_LINK2500) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_LINK_2500); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_LINK5000) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_LINK_5000); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_LINK10000) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_LINK_10000); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_RX_ACT) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_RX); | ||||
| 
 | ||||
| 	if (val & VEND1_GLOBAL_LED_PROV_TX_ACT) | ||||
| 		*rules |= BIT(TRIGGER_NETDEV_TX); | ||||
| 
 | ||||
| 	return 0; | ||||
| } | ||||
| 
 | ||||
| int aqr_phy_led_hw_control_set(struct phy_device *phydev, u8 index, | ||||
| 			       unsigned long rules) | ||||
| { | ||||
| 	u16 val = 0; | ||||
| 
 | ||||
| 	if (index >= AQR_MAX_LEDS) | ||||
| 		return -EINVAL; | ||||
| 
 | ||||
| 	if (rules & (BIT(TRIGGER_NETDEV_LINK_100) | BIT(TRIGGER_NETDEV_LINK))) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_LINK100; | ||||
| 
 | ||||
| 	if (rules & (BIT(TRIGGER_NETDEV_LINK_1000) | BIT(TRIGGER_NETDEV_LINK))) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_LINK1000; | ||||
| 
 | ||||
| 	if (rules & (BIT(TRIGGER_NETDEV_LINK_2500) | BIT(TRIGGER_NETDEV_LINK))) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_LINK2500; | ||||
| 
 | ||||
| 	if (rules & (BIT(TRIGGER_NETDEV_LINK_5000) | BIT(TRIGGER_NETDEV_LINK))) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_LINK5000; | ||||
| 
 | ||||
| 	if (rules & (BIT(TRIGGER_NETDEV_LINK_10000) | BIT(TRIGGER_NETDEV_LINK))) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_LINK10000; | ||||
| 
 | ||||
| 	if (rules & BIT(TRIGGER_NETDEV_RX)) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_RX_ACT; | ||||
| 
 | ||||
| 	if (rules & BIT(TRIGGER_NETDEV_TX)) | ||||
| 		val |= VEND1_GLOBAL_LED_PROV_TX_ACT; | ||||
| 
 | ||||
| 	return phy_modify_mmd(phydev, MDIO_MMD_VEND1, AQR_LED_PROV(index), | ||||
| 			      VEND1_GLOBAL_LED_PROV_LINK_MASK | | ||||
| 			      VEND1_GLOBAL_LED_PROV_FORCE_ON | | ||||
| 			      VEND1_GLOBAL_LED_PROV_RX_ACT | | ||||
| 			      VEND1_GLOBAL_LED_PROV_TX_ACT, val); | ||||
| } | ||||
| 
 | ||||
| int aqr_phy_led_active_low_set(struct phy_device *phydev, int index, bool enable) | ||||
| { | ||||
| 	return phy_modify_mmd(phydev, MDIO_MMD_VEND1, AQR_LED_DRIVE(index), | ||||
| 			      VEND1_GLOBAL_LED_DRIVE_VDD, enable); | ||||
| } | ||||
| 
 | ||||
| int aqr_phy_led_polarity_set(struct phy_device *phydev, int index, unsigned long modes) | ||||
| { | ||||
| 	struct aqr107_priv *priv = phydev->priv; | ||||
| 	bool active_low = false; | ||||
| 	u32 mode; | ||||
| 
 | ||||
| 	if (index >= AQR_MAX_LEDS) | ||||
| 		return -EINVAL; | ||||
| 
 | ||||
| 	for_each_set_bit(mode, &modes, __PHY_LED_MODES_NUM) { | ||||
| 		switch (mode) { | ||||
| 		case PHY_LED_ACTIVE_LOW: | ||||
| 			active_low = true; | ||||
| 			break; | ||||
| 		default: | ||||
| 			return -EINVAL; | ||||
| 		} | ||||
| 	} | ||||
| 
 | ||||
| 	/* Save LED driver vdd state to restore on SW reset */ | ||||
| 	if (active_low) | ||||
| 		priv->leds_active_low |= BIT(index); | ||||
| 
 | ||||
| 	return aqr_phy_led_active_low_set(phydev, index, active_low); | ||||
| } | ||||
|  | @ -475,7 +475,9 @@ static void aqr107_chip_info(struct phy_device *phydev) | |||
| 
 | ||||
| static int aqr107_config_init(struct phy_device *phydev) | ||||
| { | ||||
| 	int ret; | ||||
| 	struct aqr107_priv *priv = phydev->priv; | ||||
| 	u32 led_active_low; | ||||
| 	int ret, index = 0; | ||||
| 
 | ||||
| 	/* Check that the PHY interface type is compatible */ | ||||
| 	if (phydev->interface != PHY_INTERFACE_MODE_SGMII && | ||||
|  | @ -496,7 +498,19 @@ static int aqr107_config_init(struct phy_device *phydev) | |||
| 	if (!ret) | ||||
| 		aqr107_chip_info(phydev); | ||||
| 
 | ||||
| 	return aqr107_set_downshift(phydev, MDIO_AN_VEND_PROV_DOWNSHIFT_DFLT); | ||||
| 	ret = aqr107_set_downshift(phydev, MDIO_AN_VEND_PROV_DOWNSHIFT_DFLT); | ||||
| 	if (ret) | ||||
| 		return ret; | ||||
| 
 | ||||
| 	/* Restore LED polarity state after reset */ | ||||
| 	for_each_set_bit(led_active_low, &priv->leds_active_low, AQR_MAX_LEDS) { | ||||
| 		ret = aqr_phy_led_active_low_set(phydev, index, led_active_low); | ||||
| 		if (ret) | ||||
| 			return ret; | ||||
| 		index++; | ||||
| 	} | ||||
| 
 | ||||
| 	return 0; | ||||
| } | ||||
| 
 | ||||
| static int aqcs109_config_init(struct phy_device *phydev) | ||||
|  | @ -786,6 +800,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQCS109), | ||||
|  | @ -805,6 +824,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR111), | ||||
|  | @ -824,6 +848,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR111B0), | ||||
|  | @ -843,6 +872,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR405), | ||||
|  | @ -869,6 +903,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR412), | ||||
|  | @ -906,6 +945,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings    = aqr107_get_strings, | ||||
| 	.get_stats      = aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR113C), | ||||
|  | @ -925,6 +969,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings    = aqr107_get_strings, | ||||
| 	.get_stats      = aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR114C), | ||||
|  | @ -944,6 +993,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings    = aqr107_get_strings, | ||||
| 	.get_stats      = aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| { | ||||
| 	PHY_ID_MATCH_MODEL(PHY_ID_AQR813), | ||||
|  | @ -963,6 +1017,11 @@ static struct phy_driver aqr_driver[] = { | |||
| 	.get_strings	= aqr107_get_strings, | ||||
| 	.get_stats	= aqr107_get_stats, | ||||
| 	.link_change_notify = aqr107_link_change_notify, | ||||
| 	.led_brightness_set = aqr_phy_led_brightness_set, | ||||
| 	.led_hw_is_supported = aqr_phy_led_hw_is_supported, | ||||
| 	.led_hw_control_set = aqr_phy_led_hw_control_set, | ||||
| 	.led_hw_control_get = aqr_phy_led_hw_control_get, | ||||
| 	.led_polarity_set = aqr_phy_led_polarity_set, | ||||
| }, | ||||
| }; | ||||
| 
 | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Daniel Golle
						Daniel Golle