diff options
| author | Damien Lespiau <damien.lespiau@intel.com> | 2013-05-10 14:37:10 +0100 |
|---|---|---|
| committer | Damien Lespiau <damien.lespiau@intel.com> | 2013-05-10 15:35:26 +0100 |
| commit | 397dcb745ec517eb116694575ac159242bc179bd (patch) | |
| tree | 39daefbb56089a954e5f046123b39d602cfbde9d /scripts | |
| parent | eab0823e0e58c4a4ca09410a1aa22689b0de85d8 (diff) | |
list-workarounds: Don't add an already present platform
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>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/list-workarounds | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/scripts/list-workarounds b/scripts/list-workarounds index 7bfd82dd..6c8c636a 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) |