* [PATCH] uio-module-drv-test: Fix build with clang
@ 2022-05-01 16:49 Khem Raj
2022-05-02 17:59 ` [meta-ti] " Ryan Eatmon
0 siblings, 1 reply; 5+ messages in thread
From: Khem Raj @ 2022-05-01 16:49 UTC (permalink / raw)
To: meta-ti; +Cc: Khem Raj
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-ti] [PATCH] uio-module-drv-test: Fix build with clang
2022-05-01 16:49 [PATCH] uio-module-drv-test: Fix build with clang Khem Raj
@ 2022-05-02 17:59 ` Ryan Eatmon
2022-05-02 18:24 ` Denys Dmytriyenko
0 siblings, 1 reply; 5+ messages in thread
From: Ryan Eatmon @ 2022-05-02 17:59 UTC (permalink / raw)
To: Khem Raj, meta-ti
Was this patch sent to the upstream repository maintainer? This seems
like a very basic change that should work for all compilers and so
should be pushed upstream.
On 5/1/2022 11:49, Khem Raj wrote:
> 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"
>
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#14638): https://lists.yoctoproject.org/g/meta-ti/message/14638
> Mute This Topic: https://lists.yoctoproject.org/mt/90818149/6551054
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
--
Ryan Eatmon reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti] [PATCH] uio-module-drv-test: Fix build with clang
2022-05-02 17:59 ` [meta-ti] " Ryan Eatmon
@ 2022-05-02 18:24 ` Denys Dmytriyenko
2022-05-02 18:29 ` Ryan Eatmon
0 siblings, 1 reply; 5+ messages in thread
From: Denys Dmytriyenko @ 2022-05-02 18:24 UTC (permalink / raw)
To: reatmon; +Cc: Khem Raj, meta-ti
On Mon, May 02, 2022 at 12:59:59PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>
> Was this patch sent to the upstream repository maintainer? This
> seems like a very basic change that should work for all compilers
> and so should be pushed upstream.
If only the upstream project was still alive:
https://git.ti.com/cgit/keystone-linux/uio-module-drv/
This code is used for PRU using UIO on AM3/BBB, AM4, AM5 and DRA7.
> On 5/1/2022 11:49, Khem Raj wrote:
> >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"
> >
> >
> >
> >
> >
>
> --
> Ryan Eatmon reatmon@ti.com
> -----------------------------------------
> Texas Instruments, Inc. - LCPD - MGTS
--
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186 6D76 4209 0272 9A92 C964
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti] [PATCH] uio-module-drv-test: Fix build with clang
2022-05-02 18:24 ` Denys Dmytriyenko
@ 2022-05-02 18:29 ` Ryan Eatmon
2022-05-02 19:12 ` Khem Raj
0 siblings, 1 reply; 5+ messages in thread
From: Ryan Eatmon @ 2022-05-02 18:29 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: Khem Raj, meta-ti
On 5/2/2022 13:24, Denys Dmytriyenko wrote:
> On Mon, May 02, 2022 at 12:59:59PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
>>
>> Was this patch sent to the upstream repository maintainer? This
>> seems like a very basic change that should work for all compilers
>> and so should be pushed upstream.
>
> If only the upstream project was still alive:
>
> https://git.ti.com/cgit/keystone-linux/uio-module-drv/
>
> This code is used for PRU using UIO on AM3/BBB, AM4, AM5 and DRA7.
Yeah, I'm realizing that now and attempting to manually make the same
changes and commit them. Then we can change the SRCREV to point to the
new version.
>
>> On 5/1/2022 11:49, Khem Raj wrote:
>>> 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"
>>>
>>>
>>>
>>>
>>>
>>
>> --
>> Ryan Eatmon reatmon@ti.com
>> -----------------------------------------
>> Texas Instruments, Inc. - LCPD - MGTS
>
--
Ryan Eatmon reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti] [PATCH] uio-module-drv-test: Fix build with clang
2022-05-02 18:29 ` Ryan Eatmon
@ 2022-05-02 19:12 ` Khem Raj
0 siblings, 0 replies; 5+ messages in thread
From: Khem Raj @ 2022-05-02 19:12 UTC (permalink / raw)
To: Ryan Eatmon; +Cc: Denys Dmytriyenko, meta-ti
On Mon, May 2, 2022 at 11:29 AM Ryan Eatmon <reatmon@ti.com> wrote:
>
>
>
> On 5/2/2022 13:24, Denys Dmytriyenko wrote:
> > On Mon, May 02, 2022 at 12:59:59PM -0500, Ryan Eatmon via lists.yoctoproject.org wrote:
> >>
> >> Was this patch sent to the upstream repository maintainer? This
> >> seems like a very basic change that should work for all compilers
> >> and so should be pushed upstream.
> >
> > If only the upstream project was still alive:
> >
> > https://git.ti.com/cgit/keystone-linux/uio-module-drv/
> >
> > This code is used for PRU using UIO on AM3/BBB, AM4, AM5 and DRA7.
>
> Yeah, I'm realizing that now and attempting to manually make the same
> changes and commit them. Then we can change the SRCREV to point to the
> new version.
Thanks
>
> >
> >> On 5/1/2022 11:49, Khem Raj wrote:
> >>> 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"
> >>>
> >>>
> >>>
> >>>
> >>>
> >>
> >> --
> >> Ryan Eatmon reatmon@ti.com
> >> -----------------------------------------
> >> Texas Instruments, Inc. - LCPD - MGTS
> >
>
> --
> Ryan Eatmon reatmon@ti.com
> -----------------------------------------
> Texas Instruments, Inc. - LCPD - MGTS
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-05-02 19:12 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-01 16:49 [PATCH] uio-module-drv-test: Fix build with clang Khem Raj
2022-05-02 17:59 ` [meta-ti] " Ryan Eatmon
2022-05-02 18:24 ` Denys Dmytriyenko
2022-05-02 18:29 ` Ryan Eatmon
2022-05-02 19:12 ` Khem Raj
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).