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

Damien Lespiau damien.lespiau at intel.com
Fri May 10 15:40:13 CEST 2013

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

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

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)
 			workarounds[wa_name] = parse_platforms(platforms)

More information about the Intel-gfx mailing list