[Skiboot] [PATCH] FSP/LEDS: Remove led-loc property

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Thu Jul 2 18:31:15 AEST 2015


We added led-loc property.. which contains LED location information.
But this property was never used. Also we can make out LED location
(enclosure/component) based on location code (if location code doesn't
contain "-" means its enclosure location code).

As Ben suggested [1], removing this property.

Present code is included in skiboot skiboot-5.0 release..But we don't
have any consumer yet. Hence I think its fine to make this changes.

[1] https://lists.ozlabs.org/pipermail/linuxppc-dev/2015-June/130433.html

Suggested-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
---
 doc/device-tree/ibm,opal/led.txt | 3 ---
 hw/fsp/fsp-leds.c                | 7 -------
 include/fsp-leds.h               | 5 -----
 3 files changed, 15 deletions(-)

diff --git a/doc/device-tree/ibm,opal/led.txt b/doc/device-tree/ibm,opal/led.txt
index 00cd32c..5e48b85 100644
--- a/doc/device-tree/ibm,opal/led.txt
+++ b/doc/device-tree/ibm,opal/led.txt
@@ -12,7 +12,6 @@ leds {
 
 	U78C9.001.RST0027-P1-C1 {
 		led-types = "identify", "fault";
-		led-loc = "descendent";
 		phandle = <0x1000006f>;
 		linux,phandle = <0x1000006f>;
 	};
@@ -30,7 +29,5 @@ The properties under each node:
 
   led-types : Supported indicators (attention/identify/fault).
 
-  led-loc   : enclosure/descendent(FRU) location code.
-
 These LEDs can be accessed through OPAL_LEDS_{GET/SET}_INDICATOR interfaces.
 Refer to doc/opal-api/opal-led-get-set-114-115.txt for interface details.
diff --git a/hw/fsp/fsp-leds.c b/hw/fsp/fsp-leds.c
index 877aca1..52d6c90 100644
--- a/hw/fsp/fsp-leds.c
+++ b/hw/fsp/fsp-leds.c
@@ -1558,13 +1558,6 @@ void create_led_device_nodes(void)
 		else
 			dt_add_property_strings(cled, DT_PROPERTY_LED_TYPES,
 						LED_TYPE_IDENTIFY);
-
-		if (is_enclosure_led(led->loc_code))
-			dt_add_property_strings(cled, DT_PROPERTY_LED_LOCATION,
-						LED_LOC_ENCLOSURE);
-		else
-			dt_add_property_strings(cled, DT_PROPERTY_LED_LOCATION,
-						LED_LOC_DESCENDENT);
 	}
 }
 
diff --git a/include/fsp-leds.h b/include/fsp-leds.h
index dcf30b1..d7abb6d 100644
--- a/include/fsp-leds.h
+++ b/include/fsp-leds.h
@@ -161,7 +161,6 @@ struct sai_data {
 #define DT_PROPERTY_LED_NODE		"leds"
 #define DT_PROPERTY_LED_MODE		"led-mode"
 #define DT_PROPERTY_LED_TYPES		"led-types"
-#define DT_PROPERTY_LED_LOCATION	"led-loc"
 
 /* LED Mode */
 #define LED_MODE_LIGHT_PATH		"lightpath"
@@ -172,8 +171,4 @@ struct sai_data {
 #define LED_TYPE_FAULT			"fault"
 #define LED_TYPE_ATTENTION		"attention"
 
-/* LED location */
-#define LED_LOC_ENCLOSURE		"enclosure"
-#define LED_LOC_DESCENDENT		"descendent"
-
 #endif
-- 
2.1.0



More information about the Skiboot mailing list