diff options
| author | Michal Simek <michal.simek@xilinx.com> | 2018-04-19 15:15:25 +0200 |
|---|---|---|
| committer | Michal Simek <michal.simek@xilinx.com> | 2018-04-23 13:15:32 +0200 |
| commit | ebc675b98d92f7b8264ca84e65cc896d95f9868b (patch) | |
| tree | 4efa0d9f6e36beca918e2ec609636b5b657632d2 /cmd | |
| parent | 811c7bdebe7bc5ffd7709169b105d3d9edb47f30 (diff) | |
cmd: clk: Check return value from soc_clk_dump
In case of error in soc_clk_dump function are returned different values
then CMD return values (-1, 0, 1).
For example:
ZynqMP> clk dump
exit not allowed from main input shel
The patch is checking all negative return values and return
CMD_RET_FAILURE which is proper reaction for these cases.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Diffstat (limited to 'cmd')
| -rw-r--r-- | cmd/clk.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/cmd/clk.c b/cmd/clk.c index 6d3d46a1844..52b25405ce6 100644 --- a/cmd/clk.c +++ b/cmd/clk.c @@ -16,7 +16,15 @@ int __weak soc_clk_dump(void) static int do_clk_dump(cmd_tbl_t *cmdtp, int flag, int argc, char *const argv[]) { - return soc_clk_dump(); + int ret; + + ret = soc_clk_dump(); + if (ret < 0) { + printf("Clock dump error %d\n", ret); + ret = CMD_RET_FAILURE; + } + + return ret; } static cmd_tbl_t cmd_clk_sub[] = { |