[PATCH 1/1] cmd: inconsistent return type of command_process()

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Mon Aug 1 15:17:49 CEST 2022


The declarations in the header and in the implementation must match.

Reported-by: Sergei Antonov <saproj at gmail.com>
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
---
 include/command.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/command.h b/include/command.h
index 44c91f655d..8917da8b21 100644
--- a/include/command.h
+++ b/include/command.h
@@ -229,10 +229,10 @@ enum command_ret_t {
  *			is left unchanged.
  * @param ticks		If ticks is not null, this function set it to the
  *			number of ticks the command took to complete.
- * Return: 0 if the command succeeded, 1 if it failed
+ * Return: 0 if command succeeded, else non-zero (CMD_RET_...)
  */
-int cmd_process(int flag, int argc, char *const argv[], int *repeatable,
-		unsigned long *ticks);
+enum command_ret_t cmd_process(int flag, int argc, char *const argv[],
+			       int *repeatable, unsigned long *ticks);
 
 void fixup_cmdtable(struct cmd_tbl *cmdtp, int size);
 
-- 
2.36.1



More information about the U-Boot mailing list