[PATCH 1/1] lmb: add missing LMB_NOOVERWRITE description

Ilias Apalodimas ilias.apalodimas at linaro.org
Thu Sep 19 09:05:11 CEST 2024


On Thu, 19 Sept 2024 at 00:39, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> Without a LMB_NOOVERWRITE building the HTML documentation leads to a
> warning which should be treated as an error.
>
> Fixes: 5e9553cc72b8 ("lmb: allow for resizing lmb regions")
> Reported-by: Tom Rini <trini at konsulko.com>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  include/lmb.h | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/include/lmb.h b/include/lmb.h
> index fc2daaa7bfc..8ed5ac0a4fc 100644
> --- a/include/lmb.h
> +++ b/include/lmb.h
> @@ -18,6 +18,7 @@
>   * enum lmb_flags - definition of memory region attributes
>   * @LMB_NONE: no special request
>   * @LMB_NOMAP: don't add to mmu configuration
> + * @LMB_NOOVERWRITE: region cannot be resized

I think the description is 'can't be overwritten. Sughosh?

Also, I'd drop the Fixes tag for non code issues.

Thanks
/Ilias
>   */
>  enum lmb_flags {
>         LMB_NONE                = 0,
> --
> 2.45.2
>


More information about the U-Boot mailing list