From 0c9a29edb8504eb49cee3a25dc88e2b7c63672ee Mon Sep 17 00:00:00 2001 From: Vasant Hegde Date: Mon, 6 Apr 2015 14:00:20 +0530 Subject: FSP/LED: Define macro instead of hardcoding led type Define macors for LED related device tree property and its values. Signed-off-by: Vasant Hegde Signed-off-by: Stewart Smith --- hw/fsp/fsp-leds.c | 13 ++++++++----- include/fsp-leds.h | 20 ++++++++++++++++++++ 2 files changed, 28 insertions(+), 5 deletions(-) diff --git a/hw/fsp/fsp-leds.c b/hw/fsp/fsp-leds.c index 6ac0122..d25376d 100644 --- a/hw/fsp/fsp-leds.c +++ b/hw/fsp/fsp-leds.c @@ -1307,13 +1307,13 @@ void create_led_device_nodes(void) } /* LED parent node */ - pled = dt_new(opal_node, "led"); + pled = dt_new(opal_node, DT_PROPERTY_LED_NODE); if (!pled) { prlog(PR_WARNING, PREFIX "Parent device node creation failed\n"); return; } - dt_add_property_strings(pled, "compatible", "ibm,opal-v3-led"); + dt_add_property_strings(pled, "compatible", DT_PROPERTY_LED_COMPATIBLE); /* LED child nodes */ list_for_each_safe(&cec_ledq, led, next, link) { @@ -1331,11 +1331,14 @@ void create_led_device_nodes(void) continue; } - dt_add_property_strings(cled, "led-types", "identify", "fault"); + dt_add_property_strings(cled, DT_PROPERTY_LED_TYPES, + LED_TYPE_IDENTIFY, LED_TYPE_FAULT); if (is_enclosure_led(led->loc_code)) - dt_add_property_strings(cled, "led-loc", "enclosure"); + dt_add_property_strings(cled, DT_PROPERTY_LED_LOCATION, + LED_LOC_ENCLOSURE); else - dt_add_property_strings(cled, "led-loc", "descendent"); + 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 a411636..2b8ed98 100644 --- a/include/fsp-leds.h +++ b/include/fsp-leds.h @@ -150,4 +150,24 @@ struct led_set_cmd { #define LED_CONTROL_LEN 2 #define FSP_LC_STRUCT_FIXED_SZ 0x0a +/* LED Device tree property names */ +#define DT_PROPERTY_LED_COMPATIBLE "ibm,opal-v3-led" +#define DT_PROPERTY_LED_NODE "led" +#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" +#define LED_MODE_GUIDING_LIGHT "guidinglight" + +/* LED type */ +#define LED_TYPE_IDENTIFY "identify" +#define LED_TYPE_FAULT "fault" +#define LED_TYPE_ATTENTION "attention" + +/* LED location */ +#define LED_LOC_ENCLOSURE "enclosure" +#define LED_LOC_DESCENDENT "descendent" + #endif -- cgit v1.1