[PATCH 3/3] doc: seama: Reword return value section
Quentin Schulz
quentin.schulz at cherry.de
Wed Oct 22 11:33:30 CEST 2025
Hi Tom,
On 10/21/25 10:25 PM, Tom Rini wrote:
> With the addition of general text about how the return value is handled,
> reference that while retaining the additional information about setting
> $seama_image_size
>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> doc/usage/cmd/seama.rst | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/doc/usage/cmd/seama.rst b/doc/usage/cmd/seama.rst
> index 17fd559f4856..a6b00f32902e 100644
> --- a/doc/usage/cmd/seama.rst
> +++ b/doc/usage/cmd/seama.rst
> @@ -56,8 +56,5 @@ The command is available if CONFIG_CMD_SEAMA=y.
> Return value
> ------------
>
> -The return value $? is set 0 (true) if the loading is succefull, and
> -is set to 1 (false) in case of error.
> -
> -The environment variable $seama_image_size is set to the size of the
> -loaded SEAMA image.
> +Along with the general rules for setting $?, the environment variable
> +$seama_image_size is set to the size of the loaded SEAMA image.
Maybe add "as reported by the SEAMA image header" as that seems to be
what is reported in this variable reading cmd/seama.c?
Also wondering if this shouldn't be in a separate section like "Side
effects" since it isn't really the return value. I'm thinking
doc/usage/cmd/fatload.rst (and a few others) could benefit from
something like that to explain filesize env variable is set automatically.
Cheers,
Quentin
More information about the U-Boot
mailing list