[Skiboot] [PATCH 04/15] FSP/LED: Validate before creating LED nodes

[Vasant Hegde]

If FSP messes up SLCA data then attention location code points
to some other location code and its possible that we may endup
in duplicate LED location code :-(

Better to continue with partial LED list than crashing OPAL because
of duplicate node in device tree.

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

diff --git a/hw/fsp/fsp-leds.c b/hw/fsp/fsp-leds.c
index 6852706..c4e7e39 100644
--- a/hw/fsp/fsp-leds.c
+++ b/hw/fsp/fsp-leds.c
@@ -1318,6 +1318,10 @@ void create_led_device_nodes(void)
 
 	/* LED child nodes */
 	list_for_each_safe(&cec_ledq, led, next, link) {
+		/* Duplicate LED location code */
+		if (dt_find_by_path(pled, led->loc_code))
+			continue;
+
 		cled = dt_new(pled, led->loc_code);
 		if (!cled) {
 			prlog(PR_WARNING, PREFIX