platform_MemCheck: Support more ram type when parse mosys result
Currently this test look for string like DDR3-1600 when parsing
the memory spd timing result from mosys. This make the test fail
when mosys return string like LPDDR3-1600. This patch changes
the parser to support modifier for memory type to make it work
with more memory types such as LPDDR3, DDR3L, GDDR5, etc.
BUG=chrome-os-partner:37193
TEST=manual, check the regex with various string
Change-Id: I403f0e4ee22b68369dd01206b6913c609841b75e
Signed-off-by: Puthikorn Voravootivat <puthik@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/255542
Reviewed-by: Gwendal Grignou <gwendal@chromium.org>
(cherry picked from commit c2640286b9b2b9aa7e2b7fc8f491462d0fcac938)
Reviewed-on: https://chromium-review.googlesource.com/265868
Reviewed-by: Matthew Yuan <matthewyuan@chromium.org>
Tested-by: Matthew Yuan <matthewyuan@chromium.org>
diff --git a/client/site_tests/platform_MemCheck/platform_MemCheck.py b/client/site_tests/platform_MemCheck/platform_MemCheck.py
index 9dc70ce..ecf704a 100644
--- a/client/site_tests/platform_MemCheck/platform_MemCheck.py
+++ b/client/site_tests/platform_MemCheck/platform_MemCheck.py
@@ -97,7 +97,7 @@
cmd = 'mosys memory spd print timings -s speeds'
# result example
# DDR3-800, DDR3-1066, DDR3-1333, DDR3-1600
- pattern = 'DDR([3-9]|[1-9]\d+)-(?P<speed>\d+)'
+ pattern = '[A-Z]*DDR([3-9]|[1-9]\d+)[A-Z]*-(?P<speed>\d+)'
timing_run = utils.run(cmd)
keyval['speedref'] = speedref
