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

[Sughosh Ganu]

On Thu, 19 Sept 2024 at 12:35, Ilias Apalodimas
<ilias.apalodimas at linaro.org> wrote:
>
> 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?

True, 'can't be overwritten' might be a more accurate description.
Maybe, 'cannot be overwritten or resized' should also be fine.

-sughosh

>
> Also, I'd drop the Fixes tag for non code issues.
>
> Thanks
> /Ilias
> >   */
> >  enum lmb_flags {
> >         LMB_NONE                = 0,
> > --
> > 2.45.2
> >