meta-ti.lists.yoctoproject.org archive mirror
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: meta-ti@lists.yoctoproject.org
Cc: Khem Raj <raj.khem@gmail.com>
Subject: [PATCH] uio-module-drv-test: Fix build with clang
Date: Sun,  1 May 2022 09:49:19 -0700	[thread overview]
Message-ID: <20220501164920.2828087-1-raj.khem@gmail.com> (raw)

Signed-off-by: Khem Raj <raj.khem@gmail.com>
---
 ...-instead-of-int8_t-in-main-signature.patch | 89 +++++++++++++++++++
 .../uio-module-drv/uio-module-drv-test_git.bb |  4 +
 2 files changed, 93 insertions(+)
 create mode 100644 meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test/0001-tests-Use-char-instead-of-int8_t-in-main-signature.patch

diff --git a/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test/0001-tests-Use-char-instead-of-int8_t-in-main-signature.patch b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test/0001-tests-Use-char-instead-of-int8_t-in-main-signature.patch
new file mode 100644
index 00000000..172ed7cb
--- /dev/null
+++ b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test/0001-tests-Use-char-instead-of-int8_t-in-main-signature.patch
@@ -0,0 +1,89 @@
+From 972b8514cc99e76508363d2304bb13c27df99cd6 Mon Sep 17 00:00:00 2001
+From: Khem Raj <raj.khem@gmail.com>
+Date: Sun, 1 May 2022 09:39:42 -0700
+Subject: [PATCH] tests: Use char instead of int8_t in main signature
+
+int8_t is a define in C99 which is flagged by C compiler ( clang )
+
+Fixes errors like below
+
+pruss_uio_test.c:264:6: error: second parameter of 'main' (argument array) must be of type 'char **'
+void main (int32_t argc, int8_t **argv)
+
+Signed-off-by: Khem Raj <raj.khem@gmail.com>
+---
+ test/gen_ipc_int.c                       | 2 +-
+ test/prussdrv_test/test/pruss_uio_test.c | 2 +-
+ test/uio_cic2_int_multithread_test.c     | 2 +-
+ test/uio_int_test.c                      | 2 +-
+ test/uio_mem_test.c                      | 2 +-
+ 5 files changed, 5 insertions(+), 5 deletions(-)
+
+diff --git a/test/gen_ipc_int.c b/test/gen_ipc_int.c
+index 2be3a9d..313c76a 100644
+--- a/test/gen_ipc_int.c
++++ b/test/gen_ipc_int.c
+@@ -26,7 +26,7 @@ static int64_t clock_diff (struct timespec *start, struct timespec *end)
+              + end->tv_nsec - start->tv_nsec;
+ }
+ 
+-void main(int32_t argc, int8_t **argv)
++void main(int32_t argc, char **argv)
+ {
+ 	int size; 
+ 	char *map ;
+diff --git a/test/prussdrv_test/test/pruss_uio_test.c b/test/prussdrv_test/test/pruss_uio_test.c
+index 5af4a2c..7391fbd 100644
+--- a/test/prussdrv_test/test/pruss_uio_test.c
++++ b/test/prussdrv_test/test/pruss_uio_test.c
+@@ -261,7 +261,7 @@ int bsp_init()
+ 
+ }
+ 
+-void main (int32_t argc, int8_t **argv)
++void main (int32_t argc, char **argv)
+ {
+     int ret_val;
+     
+diff --git a/test/uio_cic2_int_multithread_test.c b/test/uio_cic2_int_multithread_test.c
+index 0d9f126..15db958 100755
+--- a/test/uio_cic2_int_multithread_test.c
++++ b/test/uio_cic2_int_multithread_test.c
+@@ -287,7 +287,7 @@ void *wait_inthread_for_interrupt(void *arg)
+ 	}
+ }
+ 
+-void main(int32_t argc, int8_t **argv)
++void main(int32_t argc, char **argv)
+ {
+ 	thread_arg_t thread_arg[MAX_NUM_THREADS];
+ 	int ret, err;
+diff --git a/test/uio_int_test.c b/test/uio_int_test.c
+index 7095a0a..e0ba5a9 100644
+--- a/test/uio_int_test.c
++++ b/test/uio_int_test.c
+@@ -161,7 +161,7 @@ static int64_t clock_diff (struct timespec *start, struct timespec *end)
+              + end->tv_nsec - start->tv_nsec;
+ }
+ 
+-void main(int32_t argc, int8_t **argv)
++void main(int32_t argc, char **argv)
+ {
+ 	int  fd_uio;
+ 	int ret;
+diff --git a/test/uio_mem_test.c b/test/uio_mem_test.c
+index aad13ac..3cdba37 100644
+--- a/test/uio_mem_test.c
++++ b/test/uio_mem_test.c
+@@ -121,7 +121,7 @@ err_exit1:
+ 	return ret;
+ }
+ 
+-void main(int32_t argc, int8_t **argv)
++void main(int32_t argc, char **argv)
+ {
+ 	int  fd_map;
+ 	int ret;
+-- 
+2.36.0
+
diff --git a/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test_git.bb b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test_git.bb
index dc17a1e9..41270042 100644
--- a/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test_git.bb
+++ b/meta-ti-bsp/recipes-bsp/uio-module-drv/uio-module-drv-test_git.bb
@@ -2,6 +2,8 @@ DESCRIPTION = "Test code for user space IO (UIO) driver"
 
 include uio-module-drv.inc
 
+SRC_URI += "file://0001-tests-Use-char-instead-of-int8_t-in-main-signature.patch"
+
 PR = "r1"
 
 COMPATIBLE_MACHINE = "omap-a15|ti33x|ti43x"
@@ -33,3 +35,5 @@ do_install:append:ti33x () {
 do_install:append:ti43x () {
         install -c -m 755 ${S}/test/prussdrv_test/test/pruss_uio_test ${D}${bindir}/.
 }
+
+CLEANBROKEN = "1"
-- 
2.36.0



             reply	other threads:[~2022-05-01 16:49 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-01 16:49 Khem Raj [this message]
2022-05-02 17:59 ` [meta-ti] [PATCH] uio-module-drv-test: Fix build with clang Ryan Eatmon
2022-05-02 18:24   ` Denys Dmytriyenko
2022-05-02 18:29     ` Ryan Eatmon
2022-05-02 19:12       ` Khem Raj

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220501164920.2828087-1-raj.khem@gmail.com \
    --to=raj.khem@gmail.com \
    --cc=meta-ti@lists.yoctoproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).