diff mbox

[i-g-t] list-workarounds: Don't add an already present platform

Message ID 1368193213-23926-1-git-send-email-damien.lespiau@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lespiau, Damien May 10, 2013, 1:40 p.m. UTC
Currently if we come across several sites that say that a specific
workaround is implemented for a platform, we just add the platform
several times to the list. eg.

WaFbcDisableDpfcClockGating: ivb, hsw, ivb, hsw

This patch prevent that by only adding the plaform if it's not already
there.

Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
---
 scripts/list-workarounds | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
diff mbox

Patch

diff --git a/scripts/list-workarounds b/scripts/list-workarounds
index 7bfd82d..6c8c636 100755
--- a/scripts/list-workarounds
+++ b/scripts/list-workarounds
@@ -47,7 +47,10 @@  def parse(me):
 		platforms = match.group('platforms')
 
 		if wa_name in workarounds:
-			workarounds[wa_name] += parse_platforms(platforms)
+			platforms = parse_platforms(platforms)
+			for p in platforms:
+				if not p in workarounds[wa_name]:
+					workarounds[wa_name].append(p)
 		else:
 			workarounds[wa_name] = parse_platforms(platforms)