[PATCH v7 08/11] leds: trigger: netdev: add available mode sysfs attr

From: Christian Marangi
Date: Wed Dec 14 2022 - 19:00:41 EST


Add avaiable_mode sysfs attr to show and give some details about the
supported modes and how they can be handled by the trigger.
This is in preparation for hardware only modes that doesn't support
software fallback.

Signed-off-by: Christian Marangi <ansuelsmth@xxxxxxxxx>
---
drivers/leds/trigger/ledtrig-netdev.c | 32 +++++++++++++++++++++++++++
1 file changed, 32 insertions(+)

diff --git a/drivers/leds/trigger/ledtrig-netdev.c b/drivers/leds/trigger/ledtrig-netdev.c
index a471e0cde836..3a3b77bb41fb 100644
--- a/drivers/leds/trigger/ledtrig-netdev.c
+++ b/drivers/leds/trigger/ledtrig-netdev.c
@@ -33,6 +33,8 @@
* (has carrier) or not
* tx - LED blinks on transmitted data
* rx - LED blinks on receive data
+ * available_mode - Display available mode and how they can be handled
+ * by the LED
*
*/

@@ -370,12 +372,42 @@ static ssize_t interval_store(struct device *dev,

static DEVICE_ATTR_RW(interval);

+static ssize_t available_mode_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ struct led_netdev_data *trigger_data = led_trigger_get_drvdata(dev);
+ struct netdev_led_attr_detail *detail;
+ int i, len = 0;
+
+ for (i = 0; i < ARRAY_SIZE(attr_details); i++) {
+ detail = &attr_details[i];
+
+ if (led_trigger_blink_mode_is_supported(trigger_data->led_cdev, detail->bit)) {
+ if (!trigger_data->net_dev) {
+ if (detail->hardware_only)
+ len += sprintf(buf + len, "%s [hardware]\n",
+ detail->name);
+ else
+ len += sprintf(buf + len, "%s [software-hardware]\n",
+ detail->name);
+ }
+ } else {
+ len += sprintf(buf + len, "%s [software]\n", detail->name);
+ }
+ }
+
+ return len;
+}
+
+static DEVICE_ATTR_RO(available_mode);
+
static struct attribute *netdev_trig_attrs[] = {
&dev_attr_device_name.attr,
&dev_attr_link.attr,
&dev_attr_rx.attr,
&dev_attr_tx.attr,
&dev_attr_interval.attr,
+ &dev_attr_available_mode.attr,
NULL
};
ATTRIBUTE_GROUPS(netdev_trig);
--
2.37.2