diff options
| author | Ryan Harkin <ryan.harkin@linaro.org> | 2017-03-02 17:45:16 +0000 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2017-03-09 19:52:14 -0500 |
| commit | 072c8c4cedad87c4be4edca259e1760d7db42e1b (patch) | |
| tree | 587431b2c9f407bb892d8c8b07049318abad2815 /include | |
| parent | 285226785ee178c0bbe8a67185c21e461cf4bc9f (diff) | |
| download | u-boot-072c8c4cedad87c4be4edca259e1760d7db42e1b.tar.gz u-boot-072c8c4cedad87c4be4edca259e1760d7db42e1b.tar.xz u-boot-072c8c4cedad87c4be4edca259e1760d7db42e1b.zip | |
do_smhload: fix return code
do_smhload was using a ulong to store the return value from
smh_load_file. That returns an int, where -1 indicates an error. As a
ulong will never be negative, smh_load_file errors were not detected and
so_smhload always returned zero.
Also, when errors were spotted, do_smhload was returning 1, rather than
the enumeration CMD_RET_FAILURE (which is also 1).
Signed-off-by: Ryan Harkin <ryan.harkin@linaro.org>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions