Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Marek Vasut
On 3/16/24 10:55 PM, Christophe Chapuis wrote: It could be conditional eventually; I just took the line from lvgl-demo-fb, where it wasn't. Ideally, both should be moved in lv-conf.inc, where the lvgl drm setting is activated. But I didn't want to expand again the scope of this patchset,

Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Tofe
It could be conditional eventually; I just took the line from lvgl-demo-fb, where it wasn't. Ideally, both should be moved in lv-conf.inc, where the lvgl drm setting is activated. But I didn't want to expand again the scope of this patchset, initially focused simply on adding some variables to

Re: [oe] [meta-oe][PATCH v4 2/7] lvgl: install lv_conf.h

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: Add an install append to copy the generated lv_conf.h as part of the lvgl package, so that it will be found and used by the recipes that want to use lvgl. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 5

Re: [oe] [meta-oe][PATCH v4 5/7] lvgl: add more variables to lv-conf.inc

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: As it can be usefull to customize these other variables, let's add them in lv-conf.inc. Signed-off-by: Christophe Chapuis Reviewed-by: Marek Vasut -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online

Re: [oe] [meta-oe][PATCH v3 1/5] lvgl: fix typo in lv-conf.inc

2024-03-16 Thread Marek Vasut
On 3/16/24 9:21 AM, Christophe Chapuis wrote: On Sat, Mar 16, 2024 at 3:59 AM Marek Vasut wrote: On 3/15/24 7:03 PM, Christophe Chapuis wrote: Fix a typo in the first sed instruction, where the #if 0 is never changed to #if 1 at the beginning of lv_conf.h. Signed-off-by: Christophe Chapuis

Re: [oe] [meta-oe][PATCH v4 4/7] lvgl: cleanup sed instructions in lv-conf.inc

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: Use back reference instead of repeating the search pattern. Later on this could lead to a more generic way of replacing values in the .h file. Signed-off-by: Christophe Chapuis Reviewed-by: Marek Vasut -=-=-=-=-=-=-=-=-=-=-=- Links: You

Re: [oe] [meta-oe][PATCH v4 7/7] lvgl: lv-conf.inc: generalize sed instructions

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: Use [ \t] to detect optionnal spaces at the beginning of each line, and after the define. Signed-off-by: Christophe Chapuis You could've squashed this into 5/7, but either way is fine: Reviewed-by: Marek Vasut -=-=-=-=-=-=-=-=-=-=-=- Links:

Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: When DRM is activated for LVGL, it adds a dependency on drm.h. As for lvgl-demo-fb, add an include path to fix this usecase. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 2 ++ 1 file changed, 2

Re: [oe] [meta-oe][PATCH v4 1/7] lvgl: fix typo in lv-conf.inc

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: Fix a typo in the first sed instruction, where the #if 0 is never changed to #if 1 at the beginning of lv_conf.h. Signed-off-by: Christophe Chapuis Reviewed-by: Marek Vasut Please retain the RB tags. -=-=-=-=-=-=-=-=-=-=-=- Links: You receive

Re: [oe] [meta-oe][PATCH v4 3/7] lvgl: remove useless FILES include

2024-03-16 Thread Marek Vasut
On 3/16/24 11:00 AM, Christophe Chapuis wrote: As ${includedir}${PN} is already listed in FILES, there is no need to add one of its subdirectories. Signed-off-by: Christophe Chapuis Reviewed-by: Marek Vasut -=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group.

Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Khem Raj
Ok thx On Sat, Mar 16, 2024 at 9:18 AM Christophe Chapuis wrote: > Yes, it fixes exactly this issue. > > On Sat, Mar 16, 2024 at 5:09 PM Khem Raj wrote: > >> does this fix >> >> https://errors.yoctoproject.org/Errors/Details/758522/ >> >> On Sat, Mar 16, 2024 at 3:01 AM Christophe Chapuis >>

Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Tofe
Yes, it fixes exactly this issue. On Sat, Mar 16, 2024 at 5:09 PM Khem Raj wrote: > does this fix > > https://errors.yoctoproject.org/Errors/Details/758522/ > > On Sat, Mar 16, 2024 at 3:01 AM Christophe Chapuis > wrote: > > > > When DRM is activated for LVGL, it adds a dependency on drm.h. >

Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-16 Thread Khem Raj
On Sat, Mar 16, 2024 at 1:23 AM Jörg Sommer wrote: > I've found this bug report upstream > https://github.com/freeswitch/spandsp/issues/67. It builds fine for me > with the older version df1282eb9af538ab1aadb6d66146e258451d4fe4. > > right. I have commented on the commit which is causing the

Re: [oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Khem Raj
does this fix https://errors.yoctoproject.org/Errors/Details/758522/ On Sat, Mar 16, 2024 at 3:01 AM Christophe Chapuis wrote: > > When DRM is activated for LVGL, it adds a dependency on drm.h. > As for lvgl-demo-fb, add an include path to fix this usecase. > > Signed-off-by: Christophe Chapuis

[oe] [meta-oe][PATCH v4 7/7] lvgl: lv-conf.inc: generalize sed instructions

2024-03-16 Thread Tofe
Use [ \t] to detect optionnal spaces at the beginning of each line, and after the define. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lv-conf.inc | 46 +++ 1 file changed, 23 insertions(+), 23 deletions(-) diff --git

[oe] [meta-oe][PATCH v4 4/7] lvgl: cleanup sed instructions in lv-conf.inc

2024-03-16 Thread Tofe
Use back reference instead of repeating the search pattern. Later on this could lead to a more generic way of replacing values in the .h file. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lv-conf.inc | 28 +++ 1 file changed, 14 insertions(+), 14

[oe] [meta-oe][PATCH v4 6/7] lvgl: fix libdrm include

2024-03-16 Thread Tofe
When DRM is activated for LVGL, it adds a dependency on drm.h. As for lvgl-demo-fb, add an include path to fix this usecase. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 2 ++ 1 file changed, 2 insertions(+) diff --git

[oe] [meta-oe][PATCH v4 3/7] lvgl: remove useless FILES include

2024-03-16 Thread Tofe
As ${includedir}${PN} is already listed in FILES, there is no need to add one of its subdirectories. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 1 - 1 file changed, 1 deletion(-) diff --git a/meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb

[oe] [meta-oe][PATCH v4 5/7] lvgl: add more variables to lv-conf.inc

2024-03-16 Thread Tofe
As it can be usefull to customize these other variables, let's add them in lv-conf.inc. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lv-conf.inc | 17 + 1 file changed, 17 insertions(+) diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc

[oe] [meta-oe][PATCH v4 2/7] lvgl: install lv_conf.h

2024-03-16 Thread Tofe
Add an install append to copy the generated lv_conf.h as part of the lvgl package, so that it will be found and used by the recipes that want to use lvgl. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 5 + 1 file changed, 5 insertions(+) diff --git

[oe] [meta-oe][PATCH v4 1/7] lvgl: fix typo in lv-conf.inc

2024-03-16 Thread Tofe
Fix a typo in the first sed instruction, where the #if 0 is never changed to #if 1 at the beginning of lv_conf.h. Signed-off-by: Christophe Chapuis --- meta-oe/recipes-graphics/lvgl/lv-conf.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-16 Thread Jörg Sommer via lists . openembedded . org
I've found this bug report upstreamhttps://github.com/freeswitch/spandsp/issues/67. It builds fine for me with the older version df1282eb9af538ab1aadb6d66146e258451d4fe4. Mit freundlichen Grüßen Jörg Sommer Software Developer / Programmierer -- Navimatix GmbH Tatzendpromenade 2 07745

[oe] [meta-multimedia][PATCH v3] spandsp: new telephony DSP library

2024-03-16 Thread Jörg Sommer via lists . openembedded . org
From: Jörg Sommer Signed-off-by: Jörg Sommer --- .../spandsp/spandsp/configure.patch | 56 ++ .../spandsp/spandsp/makefile.patch| 161 ++ .../recipes-multimedia/spandsp/spandsp_git.bb | 50 ++ 3 files changed, 267 insertions(+) create mode

Re: [oe] [meta-oe][PATCH v3 1/5] lvgl: fix typo in lv-conf.inc

2024-03-16 Thread Tofe
On Sat, Mar 16, 2024 at 3:59 AM Marek Vasut wrote: > On 3/15/24 7:03 PM, Christophe Chapuis wrote: > > Fix a typo in the first sed instruction, where the #if 0 is > > never changed to #if 1 at the beginning of lv_conf.h. > > > > Signed-off-by: Christophe Chapuis > > --- > >

Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-16 Thread Martin Jansa
I'm seeing the same errors with gcc :) libtool: compile: ccache arm-oe-linux-gnueabi-gcc -mthumb -mfpu=neon -mfloat-abi=softfp -mcpu=cortex-a9 -mtune=cortex-a9 -fstack-protector-strong -O2 -D_FORTIFY_SOURCE=2 -Wformat -Wformat-security -Werror=format-security -Werror=return-type -funwind-tables