MIRRORS/meta-freescale
1
0
Fork 0
mirror of git://git.yoctoproject.org/meta-freescale.git synced 2025-10-22 14:52:19 +02:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

imx-vpu-hantro: fix build with gcc 15

Browse Source 
Change from deprecated K&R function declaration and definitions
to ANSI C style.

GCC 15 by default no longer accepts it:
| decoder_sw/software/test/common/md5_sink.c:68:3: error: too many
| arguments to function 'MD5Init'; expected 0, have 1
|   68 |   MD5Init(&inst->ctx);

With C23 bool, true and false are built in keywords and cannot be
redefined.

Fixes build error:
| ../inc/basetype.h:71:9: error: cannot use keyword 'false' as enumeration constant
|    71 |         false   = 0,

Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
This commit is contained in:
Max Krummenacher 2025-05-12 09:37:31 +02:00
parent dd0b82ccbc
commit 693c297a19
3 changed files with 133 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0001-basetype.h-make-header-compatible-with-c23.patch Normal file
View File

@ -0,0 +1,38 @@
From e39a6602817e4221ae09ac3d520ca6a0d94814dc Mon Sep 17 00:00:00 2001
From: Max Krummenacher <max.krummenacher@toradex.com>
Date: Wed, 7 May 2025 16:13:38 +0000
Subject: [PATCH] basetype.h: make header compatible with c23
With C23 bool, true and false are built in keywords and cannot be
redefined.
Upstream-Status: Pending
Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
---
h1_encoder/software/inc/basetype.h | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/h1_encoder/software/inc/basetype.h b/h1_encoder/software/inc/basetype.h
index 55e46a195b3b..6655ec54481b 100755
--- a/h1_encoder/software/inc/basetype.h
+++ b/h1_encoder/software/inc/basetype.h
@@ -66,12 +66,16 @@ typedef size_t ptr_t;
#define PRT_PTR "x"
#endif
+#if defined __STDC_VERSION__ && __STDC_VERSION__ > 201710L
+/* bool, true and false are keywords. */
+#else
#ifndef __cplusplus
typedef enum {
false = 0,
true = 1
} bool;
#endif
+#endif
#else /* __symbian__ or __win__ or whatever, customize it to suit well */
--
2.42.0

90
recipes-bsp/imx-vpu-hantro/imx-vpu-hantro/0001-test-md5-convert-to-ansi-c.patch Normal file
View File

@ -0,0 +1,90 @@
From 0c5bdd12a6f3ba73e605656828bf429966a997ef Mon Sep 17 00:00:00 2001
From: Max Krummenacher <max.krummenacher@toradex.com>
Date: Wed, 7 May 2025 13:25:26 +0000
Subject: [PATCH] test: md5: convert to ansi c
GCC 15 no longer likes this K&R style function declarations.
Upstream-Status: Pending
Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com>
---
decoder_sw/software/test/common/swhw/md5.c | 15 +++++----------
decoder_sw/software/test/common/swhw/md5.h | 8 ++++----
2 files changed, 9 insertions(+), 14 deletions(-)
diff --git a/decoder_sw/software/test/common/swhw/md5.c b/decoder_sw/software/test/common/swhw/md5.c
index c3334bf751fd..20014fbdf268 100755
--- a/decoder_sw/software/test/common/swhw/md5.c
+++ b/decoder_sw/software/test/common/swhw/md5.c
@@ -42,8 +42,7 @@
/*
* Note: this code is harmless on little-endian machines.
*/
-void ByteReverse(buf, longs) unsigned char *buf;
-unsigned longs;
+void ByteReverse(unsigned char *buf, unsigned longs)
{
uint32 t;
do {
@@ -59,7 +58,7 @@ unsigned longs;
* Start MD5 accumulation. Set bit count to 0 and buffer to mysterious
* initialization constants.
*/
-void MD5Init(ctx) struct MD5Context *ctx;
+void MD5Init(struct MD5Context *ctx)
{
ctx->buf[0] = 0x67452301;
ctx->buf[1] = 0xefcdab89;
@@ -74,9 +73,7 @@ void MD5Init(ctx) struct MD5Context *ctx;
* Update context to reflect the concatenation of another buffer full
* of bytes.
*/
-void MD5Update(ctx, buf, len) struct MD5Context *ctx;
-unsigned char *buf;
-unsigned len;
+void MD5Update(struct MD5Context *ctx, unsigned char *buf, unsigned len)
{
uint32 t;
@@ -124,8 +121,7 @@ unsigned len;
* Final wrapup - pad to 64-byte boundary with the bit pattern
* 1 0* (64-bit count of bits processed, MSB-first)
*/
-void MD5Final(digest, ctx) unsigned char digest[16];
-struct MD5Context *ctx;
+void MD5Final(unsigned char digest[16], struct MD5Context *ctx)
{
unsigned count;
unsigned char *p;
@@ -186,8 +182,7 @@ struct MD5Context *ctx;
* reflect the addition of 16 longwords of new data. MD5Update blocks
* the data and converts bytes into longwords for this routine.
*/
-void MD5Transform(buf, in) uint32 buf[4];
-uint32 in[16];
+void MD5Transform(uint32 buf[4], uint32 in[16])
{
register uint32 a, b, c, d;
diff --git a/decoder_sw/software/test/common/swhw/md5.h b/decoder_sw/software/test/common/swhw/md5.h
index 516400236606..c1d773093b66 100755
--- a/decoder_sw/software/test/common/swhw/md5.h
+++ b/decoder_sw/software/test/common/swhw/md5.h
@@ -76,10 +76,10 @@ struct MD5Context {
unsigned char in[64];
};
-extern void MD5Init();
-extern void MD5Update();
-extern void MD5Final();
-extern void MD5Transform();
+extern void MD5Init(struct MD5Context *ctx);
+extern void MD5Update(struct MD5Context *ctx, unsigned char *buf, unsigned len);
+extern void MD5Final(unsigned char digest[16], struct MD5Context *ctx);
+extern void MD5Transform(uint32 buf[4], uint32 in[16]);
/*
* This is needed to make RSAREF happy on some MS-DOS compilers.
--
2.42.0

6
recipes-bsp/imx-vpu-hantro/imx-vpu-hantro_1.36.0.bb
View File

@ -6,7 +6,11 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=c0fb372b5d7f12181de23ef480f225f3"
PROVIDES = "virtual/imxvpu"
SRC_URI = "${FSL_MIRROR}/${BP}-${IMX_SRCREV_ABBREV}.bin;fsl-eula=true"
SRC_URI = " \
${FSL_MIRROR}/${BP}-${IMX_SRCREV_ABBREV}.bin;fsl-eula=true \
file://0001-test-md5-convert-to-ansi-c.patch \
file://0001-basetype.h-make-header-compatible-with-c23.patch \
"
IMX_SRCREV_ABBREV = "194a305"
SRC_URI[sha256sum] = "0ef1fb5c6653c08f2d2812c72dedf3e8beb091dd5b3d70d6e26f41bac4ebffa7"