Commit 4e901018 authored by Andrew Lunn's avatar Andrew Lunn Committed by David S. Miller
Browse files

net: phy: phy_device: Call into the PHY driver to set LED blinking 



Linux LEDs can be requested to perform hardware accelerated
blinking. Pass this to the PHY driver, if it implements the op.

Signed-off-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarChristian Marangi <ansuelsmth@gmail.com>
Reviewed-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2d3960e5

drivers/net/phy/phy_device.c

+18 −0
Original line number Diff line number Diff line
@@ -3005,6 +3005,22 @@ static int phy_led_set_brightness(struct led_classdev *led_cdev, 
	return err;

}



static int phy_led_blink_set(struct led_classdev *led_cdev,

			     unsigned long *delay_on,

			     unsigned long *delay_off)

{

	struct phy_led *phyled = to_phy_led(led_cdev);

	struct phy_device *phydev = phyled->phydev;

	int err;



	mutex_lock(&phydev->lock);

	err = phydev->drv->led_blink_set(phydev, phyled->index,

					 delay_on, delay_off);

	mutex_unlock(&phydev->lock);



	return err;

}



static int of_phy_led(struct phy_device *phydev,

		      struct device_node *led)

{
@@ -3027,6 +3043,8 @@ static int of_phy_led(struct phy_device *phydev, 


	if (phydev->drv->led_brightness_set)

		cdev->brightness_set_blocking = phy_led_set_brightness;

	if (phydev->drv->led_blink_set)

		cdev->blink_set = phy_led_blink_set;

	cdev->max_brightness = 1;

	init_data.devicename = dev_name(&phydev->mdio.dev);

	init_data.fwnode = of_fwnode_handle(led);

include/linux/phy.h

+12 −0
Original line number Diff line number Diff line
@@ -1085,6 +1085,18 @@ struct phy_driver { 
	 */

	int (*led_brightness_set)(struct phy_device *dev,

				  u8 index, enum led_brightness value);



	/**

	 * @led_blink_set: Set a PHY LED brightness.  Index indicates

	 * which of the PHYs led should be configured to blink. Delays

	 * are in milliseconds and if both are zero then a sensible

	 * default should be chosen.  The call should adjust the

	 * timings in that case and if it can't match the values

	 * specified exactly.

	 */

	int (*led_blink_set)(struct phy_device *dev, u8 index,

			     unsigned long *delay_on,

			     unsigned long *delay_off);

};

#define to_phy_driver(d) container_of(to_mdio_common_driver(d),		\

				      struct phy_driver, mdiodrv)