[PATCH 02/17] sandbox: Add missing comments for os_alarm()

[Simon Glass]

Hi Heinrich,

On Sat, 5 Nov 2022 at 17:39, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> On 10/30/22 02:47, Simon Glass wrote:
> > Add the documentation to avoid a warning with 'make htmldocs'.
> >
> > Fixes: 10107efedd5 ("sandbox: add SIGALRM-based watchdog device")
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >   include/os.h | 4 ++++
> >   1 file changed, 4 insertions(+)
> >
> > diff --git a/include/os.h b/include/os.h
> > index 54874f5e0e8..0415f0f0e7a 100644
> > --- a/include/os.h
> > +++ b/include/os.h
> > @@ -110,6 +110,10 @@ void os_exit(int exit_code) __attribute__((noreturn));
> >
> >   /**
> >    * os_alarm() - access to the OS alarm() system call
>
> alarm() is not a system call but a POSIX function.

I saw this in the other thread.

I would say that alarm() is a system call which is why it is in
section 2 of the manual. I don't really care but someone else feels
the same way.

Do you feel strongly about this?

Regards,
Simon