summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorQuentin Schulz <[email protected]>2018-07-09 19:16:26 +0200
committerTom Rini <[email protected]>2018-07-20 15:55:08 -0400
commitabba76354a049c48fcdf4a252383ccd5c119b8ae (patch)
treefbbfd95f2c1a5bf5da88ce9a34bd974776bf9687
parent6c90f62338226040513742921e718058f3273afb (diff)
test/py: return a RAM address different from 0 as it can be interpreted as NULL
Some functions test that the given address is not NULL (0) and fail or have a different behaviour if that's the case (e.g. hexport_r). Let's make the RAM base address to be not zero by setting it to 2MiB if that's the case. 2MiB is chosen because it represents the size of an ARM LPAE/v8 section. Suggested-by: Stephen Warren <[email protected]> Signed-off-by: Quentin Schulz <[email protected]> Reviewed-by: Simon Glass <[email protected]> Reviewed-by: Stephen Warren <[email protected]> Tested-by: Stephen Warren <[email protected]>
-rw-r--r--test/py/u_boot_utils.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/test/py/u_boot_utils.py b/test/py/u_boot_utils.py
index d68d1dd1054..bf2a0fc0e24 100644
--- a/test/py/u_boot_utils.py
+++ b/test/py/u_boot_utils.py
@@ -236,6 +236,12 @@ def find_ram_base(u_boot_console):
ram_base = -1
raise Exception('Failed to find RAM bank start in `bdinfo`')
+ # We don't want ram_base to be zero as some functions test if the given
+ # address is NULL (0). Let's add 2MiB then (size of an ARM LPAE/v8 section).
+
+ if ram_base == 0:
+ ram_base += 1024 * 1024 * 2
+
return ram_base
class PersistentFileHelperCtxMgr(object):