diff options
author | Simon Glass <sjg@chromium.org> | 2019-07-20 12:23:42 -0600 |
---|---|---|
committer | Simon Glass <sjg@chromium.org> | 2019-07-29 09:38:05 -0600 |
commit | 6a3b5b54110980a42284beb05865436652113772 (patch) | |
tree | 2491338717205929d13d20dba2453d13c9d7c28b | |
parent | d5079330f588a1aeedc734197124543bbc4e2d3c (diff) | |
download | u-boot-6a3b5b54110980a42284beb05865436652113772.tar.gz |
binman: Allow state functions to fail to return data
At present these state functions raise an exception if they cannot find
what is requested. But in some cases the information is optional (e.g. an
fdtmap in a coming patch) so it is better to return gracefully.
Update these two functions to return None when the data cannot be found.
Signed-off-by: Simon Glass <sjg@chromium.org>
-rw-r--r-- | tools/binman/state.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/tools/binman/state.py b/tools/binman/state.py index 7c3a987723..8767410066 100644 --- a/tools/binman/state.py +++ b/tools/binman/state.py @@ -49,7 +49,10 @@ def GetFdtForEtype(etype): Returns: Fdt object associated with the entry type """ - return output_fdt_files[etype][0] + value = output_fdt_files.get(etype); + if not value: + return None + return value[0] def GetFdtPath(etype): """Get the full pathname of a particular Fdt object @@ -80,7 +83,9 @@ def GetFdtContents(etype='u-boot-dtb'): pathname to Fdt Fdt data (as bytes) """ - if etype in output_fdt_files and not use_fake_dtb: + if etype not in output_fdt_files: + return None, None + if not use_fake_dtb: pathname = GetFdtPath(etype) data = GetFdtForEtype(etype).GetContents() else: |