leds: Standardise LED naming scheme

As discussed on LKML some notion of 'function' is needed in
LED naming. This patch adds this to the documentation and
standardises existing LED drivers.

Signed-off-by: Richard Purdie <rpurdie@rpsys.net>

12 files changed, 29 insertions, 27 deletions

diff --git a/drivers/hwmon/applesmc.c b/drivers/hwmon/applesmc.c
index 86c66c345f8b..0c94770b7f83 100644
--- a/drivers/hwmon/applesmc.c
+++ b/drivers/hwmon/applesmc.c
@@ -905,7 +905,7 @@ static ssize_t applesmc_key_at_index_store(struct device *dev,
 }
 
 static struct led_classdev applesmc_backlight = {
-        .name                   = "smc:kbd_backlight",
+        .name                   = "smc::kbd_backlight",
         .default_trigger        = "nand-disk",
         .brightness_set         = applesmc_brightness_set,
 };
diff --git a/drivers/input/misc/wistron_btns.c b/drivers/input/misc/wistron_btns.c
index b438d998625c..72176f3d49cb 100644
--- a/drivers/input/misc/wistron_btns.c
+++ b/drivers/input/misc/wistron_btns.c
@@ -998,12 +998,12 @@ static void wistron_wifi_led_set(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev wistron_mail_led = {
-        .name                   = "mail:green",
+        .name                   = "wistron:green:mail",
         .brightness_set         = wistron_mail_led_set,
 };
 
 static struct led_classdev wistron_wifi_led = {
-        .name                   = "wifi:red",
+        .name                   = "wistron:red:wifi",
         .brightness_set         = wistron_wifi_led_set,
 };
 
diff --git a/drivers/leds/leds-ams-delta.c b/drivers/leds/leds-ams-delta.c
index 599878c8e714..9e3077463d84 100644
--- a/drivers/leds/leds-ams-delta.c
+++ b/drivers/leds/leds-ams-delta.c
@@ -37,42 +37,42 @@ static void ams_delta_led_set(struct led_classdev *led_cdev,
 static struct ams_delta_led ams_delta_leds[] = {
         {
                 .cdev           = {
-                        .name           = "ams-delta:camera",
+                        .name           = "ams-delta::camera",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_CAMERA,
         },
         {
                 .cdev           = {
-                        .name           = "ams-delta:advert",
+                        .name           = "ams-delta::advert",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_ADVERT,
         },
         {
                 .cdev           = {
-                        .name           = "ams-delta:email",
+                        .name           = "ams-delta::email",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_EMAIL,
         },
         {
                 .cdev           = {
-                        .name           = "ams-delta:handsfree",
+                        .name           = "ams-delta::handsfree",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_HANDSFREE,
         },
         {
                 .cdev           = {
-                        .name           = "ams-delta:voicemail",
+                        .name           = "ams-delta::voicemail",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_VOICEMAIL,
         },
         {
                 .cdev           = {
-                        .name           = "ams-delta:voice",
+                        .name           = "ams-delta::voice",
                         .brightness_set = ams_delta_led_set,
                 },
                 .bitmask        = AMS_DELTA_LATCH1_LED_VOICE,
diff --git a/drivers/leds/leds-clevo-mail.c b/drivers/leds/leds-clevo-mail.c
index 50aaa0b33810..49a7972459c9 100644
--- a/drivers/leds/leds-clevo-mail.c
+++ b/drivers/leds/leds-clevo-mail.c
@@ -93,7 +93,7 @@ static void clevo_mail_led_set(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev clevo_mail_led = {
-        .name                   = "clevo",
+        .name                   = "clevo::mail",
         .brightness_set         = clevo_mail_led_set,
 };
 
diff --git a/drivers/leds/leds-corgi.c b/drivers/leds/leds-corgi.c
index cf1dcd719a28..e45f6c4b59ba 100644
--- a/drivers/leds/leds-corgi.c
+++ b/drivers/leds/leds-corgi.c
@@ -38,13 +38,13 @@ static void corgiled_green_set(struct led_classdev *led_cdev, enum led_brightnes
 }
 
 static struct led_classdev corgi_amber_led = {
-        .name                   = "corgi:amber",
+        .name                   = "corgi:amber:charge",
         .default_trigger        = "sharpsl-charge",
         .brightness_set         = corgiled_amber_set,
 };
 
 static struct led_classdev corgi_green_led = {
-        .name                   = "corgi:green",
+        .name                   = "corgi:green:mail",
         .default_trigger        = "nand-disk",
         .brightness_set         = corgiled_green_set,
 };
diff --git a/drivers/leds/leds-locomo.c b/drivers/leds/leds-locomo.c
index 2207335e9212..7295f7f52185 100644
--- a/drivers/leds/leds-locomo.c
+++ b/drivers/leds/leds-locomo.c
@@ -43,13 +43,13 @@ static void locomoled_brightness_set1(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev locomo_led0 = {
-        .name                   = "locomo:amber",
+        .name                   = "locomo:amber:charge",
         .default_trigger        = "sharpsl-charge",
         .brightness_set         = locomoled_brightness_set0,
 };
 
 static struct led_classdev locomo_led1 = {
-        .name                   = "locomo:green",
+        .name                   = "locomo:green:mail",
         .default_trigger        = "nand-disk",
         .brightness_set         = locomoled_brightness_set1,
 };
diff --git a/drivers/leds/leds-net48xx.c b/drivers/leds/leds-net48xx.c
index 45ba3d45bcb8..054360473c94 100644
--- a/drivers/leds/leds-net48xx.c
+++ b/drivers/leds/leds-net48xx.c
@@ -31,7 +31,7 @@ static void net48xx_error_led_set(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev net48xx_error_led = {
-        .name           = "net48xx:error",
+        .name           = "net48xx::error",
         .brightness_set = net48xx_error_led_set,
 };
 
diff --git a/drivers/leds/leds-spitz.c b/drivers/leds/leds-spitz.c
index 126d09cc96ec..93e1012b17e6 100644
--- a/drivers/leds/leds-spitz.c
+++ b/drivers/leds/leds-spitz.c
@@ -38,13 +38,13 @@ static void spitzled_green_set(struct led_classdev *led_cdev, enum led_brightnes
 }
 
 static struct led_classdev spitz_amber_led = {
-        .name                   = "spitz:amber",
+        .name                   = "spitz:amber:charge",
         .default_trigger        = "sharpsl-charge",
         .brightness_set         = spitzled_amber_set,
 };
 
 static struct led_classdev spitz_green_led = {
-        .name                   = "spitz:green",
+        .name                   = "spitz:green:hddactivity",
         .default_trigger        = "ide-disk",
         .brightness_set         = spitzled_green_set,
 };
@@ -72,8 +72,10 @@ static int spitzled_probe(struct platform_device *pdev)
 {
         int ret;
 
-        if (machine_is_akita())
+        if (machine_is_akita()) {
+                spitz_green_led.name = "spitz:green:mail";
                 spitz_green_led.default_trigger = "nand-disk";
+        }
 
         ret = led_classdev_register(&pdev->dev, &spitz_amber_led);
         if (ret < 0)
diff --git a/drivers/leds/leds-tosa.c b/drivers/leds/leds-tosa.c
index fb2416a38303..9e0a188fbb0a 100644
--- a/drivers/leds/leds-tosa.c
+++ b/drivers/leds/leds-tosa.c
@@ -45,13 +45,13 @@ static void tosaled_green_set(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev tosa_amber_led = {
-        .name                   = "tosa:amber",
+        .name                   = "tosa:amber:charge",
         .default_trigger        = "sharpsl-charge",
         .brightness_set         = tosaled_amber_set,
 };
 
 static struct led_classdev tosa_green_led = {
-        .name                   = "tosa:green",
+        .name                   = "tosa:green:mail",
         .default_trigger        = "nand-disk",
         .brightness_set         = tosaled_green_set,
 };
diff --git a/drivers/leds/leds-wrap.c b/drivers/leds/leds-wrap.c
index 27fb2d8e991f..5770cc36ec20 100644
--- a/drivers/leds/leds-wrap.c
+++ b/drivers/leds/leds-wrap.c
@@ -43,12 +43,12 @@ static void wrap_extra_led_set(struct led_classdev *led_cdev,
 }
 
 static struct led_classdev wrap_error_led = {
-        .name           = "wrap:error",
+        .name           = "wrap::error",
         .brightness_set = wrap_error_led_set,
 };
 
 static struct led_classdev wrap_extra_led = {
-        .name           = "wrap:extra",
+        .name           = "wrap::extra",
         .brightness_set = wrap_extra_led_set,
 };
 
diff --git a/drivers/misc/asus-laptop.c b/drivers/misc/asus-laptop.c
index 0846c33296bc..3a36a294e91b 100644
--- a/drivers/misc/asus-laptop.c
+++ b/drivers/misc/asus-laptop.c
@@ -239,7 +239,7 @@ static struct workqueue_struct *led_workqueue;
         static int object##_led_wk;                                     \
         static DECLARE_WORK(object##_led_work, object##_led_update);    \
         static struct led_classdev object##_led = {                     \
-                .name           = "asus:" ledname,                      \
+                .name           = "asus::" ledname,                     \
                 .brightness_set = object##_led_set,                     \
         }
 
diff --git a/drivers/net/wireless/b43/leds.c b/drivers/net/wireless/b43/leds.c
index 0908335892db..0aac1ff511df 100644
--- a/drivers/net/wireless/b43/leds.c
+++ b/drivers/net/wireless/b43/leds.c
@@ -147,12 +147,12 @@ static void b43_map_led(struct b43_wldev *dev,
         case B43_LED_TRANSFER:
         case B43_LED_APTRANSFER:
                 snprintf(name, sizeof(name),
-                         "b43-%s:tx", wiphy_name(hw->wiphy));
+                         "b43-%s::tx", wiphy_name(hw->wiphy));
                 b43_register_led(dev, &dev->led_tx, name,
                                  ieee80211_get_tx_led_name(hw),
                                  led_index, activelow);
                 snprintf(name, sizeof(name),
-                         "b43-%s:rx", wiphy_name(hw->wiphy));
+                         "b43-%s::rx", wiphy_name(hw->wiphy));
                 b43_register_led(dev, &dev->led_rx, name,
                                  ieee80211_get_rx_led_name(hw),
                                  led_index, activelow);
@@ -162,7 +162,7 @@ static void b43_map_led(struct b43_wldev *dev,
         case B43_LED_RADIO_B:
         case B43_LED_MODE_BG:
                 snprintf(name, sizeof(name),
-                         "b43-%s:radio", wiphy_name(hw->wiphy));
+                         "b43-%s::radio", wiphy_name(hw->wiphy));
                 b43_register_led(dev, &dev->led_radio, name,
                                  b43_rfkill_led_name(dev),
                                  led_index, activelow);
@@ -173,7 +173,7 @@ static void b43_map_led(struct b43_wldev *dev,
         case B43_LED_WEIRD:
         case B43_LED_ASSOC:
                 snprintf(name, sizeof(name),
-                         "b43-%s:assoc", wiphy_name(hw->wiphy));
+                         "b43-%s::assoc", wiphy_name(hw->wiphy));
                 b43_register_led(dev, &dev->led_assoc, name,
                                  ieee80211_get_assoc_led_name(hw),
                                  led_index, activelow);