[PATCH rtems-littlevgl 4/4] lv_drv_conf: Enable USE_BSD_EVDEV option

[Vijay Kumar Banerjee]

On Fri, Jun 5, 2020 at 1:59 PM Christian Mauderer
<christian.mauderer at embedded-brains.de> wrote:
>
> Hello Vijay,
>
> On 04/06/2020 20:44, Vijay Kumar Banerjee wrote:
> > ---
> >  lv_drv_conf.h | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/lv_drv_conf.h b/lv_drv_conf.h
> > index 697ceaf..e8d2c40 100644
> > --- a/lv_drv_conf.h
> > +++ b/lv_drv_conf.h
> > @@ -322,7 +322,11 @@
> >  #  define USE_EVDEV           0
> >  #endif
> >
> > -#if USE_EVDEV
> > +#ifndef USE_BSD_EVDEV
> > +#  define USE_BSD_EVDEV           1
> > +#endif
>
Hi!

> I haven't tested it and haven't had a detailed look. But first question
> that springs to my mind: Can we still use no evdev on targets without it
> with this change? Or becomes some evdev module for libbsd mandatory with
> it if you want to use lvgl?
>
With this change in the configuration, the evdev module will be
required to compile as it includes the evdev header files if this
option is set, just like in the case of fbdev. The build options still
remain same, so we can add --no-drivers and these files won't be
touched during the build. If it needs to be further broken down in
such a manner that the fbdev is required but not evdev, then I think
we can keep this option enabled and check it during waf configuration
if evdev is available. If not, we can skill building the evdev files.

Do you think such checking would be necessary or would it be
sufficient to have the --no-drivers option? I'll work on it
accordingly if more options are required.

Thanks for the review.

Best regards,
Vijay
> Best regards
>
> Christian
>
> > +
> > +#if USE_EVDEV || USE_BSD_EVDEV
> >  #  define EVDEV_NAME   "/dev/input/event0"        /*You can use the "evtest" Linux tool to get the list of devices and test them*/
> >  #  define EVDEV_SWAP_AXES         0               /*Swap the x and y axes of the touchscreen*/
> >
> >
>
> --
> --------------------------------------------
> embedded brains GmbH
> Herr Christian Mauderer
> Dornierstr. 4
> D-82178 Puchheim
> Germany
> email: christian.mauderer at embedded-brains.de
> Phone: +49-89-18 94 741 - 18
> Fax:   +49-89-18 94 741 - 08
> PGP: Public key available on request.
>
> Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.