[Skiboot] [PATCH 10/15] FSP/LED: Use LED mode information

Vasant Hegde hegdevasant at linux.vnet.ibm.com
Fri Mar 20 23:09:16 AEDT 2015


Lets use the LED mode information populated by previous patch.

We don't support fault indicator in 'Guiding Light' mode. So
don't expose fault indicator to playload in Guiding Light mode.

Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
---
 hw/fsp/fsp-leds.c |   17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/hw/fsp/fsp-leds.c b/hw/fsp/fsp-leds.c
index e2a0ac1..3b2bcf1 100644
--- a/hw/fsp/fsp-leds.c
+++ b/hw/fsp/fsp-leds.c
@@ -1285,6 +1285,7 @@ success:
  */
 void create_led_device_nodes(void)
 {
+	const char *led_mode = NULL;
 	struct fsp_led_data *led, *next;
 	struct dt_node *pled, *cled;
 
@@ -1317,6 +1318,12 @@ void create_led_device_nodes(void)
 
 	dt_add_property_strings(pled, "compatible", DT_PROPERTY_LED_COMPATIBLE);
 
+	led_mode = dt_prop_get(pled, DT_PROPERTY_LED_MODE);
+	if (!led_mode) {
+		prlog(PR_WARNING, PREFIX "Unknown operating mode\n");
+		return;
+	}
+
 	/* LED child nodes */
 	list_for_each_safe(&cec_ledq, led, next, link) {
 		/* Duplicate LED location code */
@@ -1330,8 +1337,14 @@ void create_led_device_nodes(void)
 			continue;
 		}
 
-		dt_add_property_strings(cled, DT_PROPERTY_LED_TYPE,
-					LED_TYPE_IDENTIFY, LED_TYPE_FAULT);
+		if (!strcmp(led_mode, LED_MODE_LIGHT_PATH))
+			dt_add_property_strings(cled, DT_PROPERTY_LED_TYPE,
+						LED_TYPE_IDENTIFY,
+						LED_TYPE_FAULT);
+		else
+			dt_add_property_strings(cled, DT_PROPERTY_LED_TYPE,
+						LED_TYPE_IDENTIFY);
+
 		if (is_enclosure_led(led->loc_code))
 			dt_add_property_strings(cled, DT_PROPERTY_LED_LOCATION,
 						LED_LOC_ENCLOSURE);



More information about the Skiboot mailing list