summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--meta-oe/recipes-extended/upm/upm/0001-Replace-strncpy-with-memcpy.patch49
-rw-r--r--meta-oe/recipes-extended/upm/upm/0001-Use-stdint-types.patch139
-rw-r--r--meta-oe/recipes-extended/upm/upm/0001-include-sys-types.h-for-uint-definition.patch33
-rw-r--r--meta-oe/recipes-extended/upm/upm_git.bb7
4 files changed, 142 insertions, 86 deletions
diff --git a/meta-oe/recipes-extended/upm/upm/0001-Replace-strncpy-with-memcpy.patch b/meta-oe/recipes-extended/upm/upm/0001-Replace-strncpy-with-memcpy.patch
deleted file mode 100644
index 4b17087ef8..0000000000
--- a/meta-oe/recipes-extended/upm/upm/0001-Replace-strncpy-with-memcpy.patch
+++ /dev/null
@@ -1,49 +0,0 @@
1From d314f26e024aaf15bf4ab22ceb98501148d0eac8 Mon Sep 17 00:00:00 2001
2From: Khem Raj <raj.khem@gmail.com>
3Date: Mon, 7 May 2018 19:53:33 -0700
4Subject: [PATCH] Replace strncpy with memcpy
5
6gcc8 detects that strncpy is overwiritng the null terminating character
7the source strings are already initialized to 0 so memcpy would do the same
8job
9
10Fixes
11rn2903.c:153:5: error: 'strncpy' output may be truncated copying 16 bytes from a string of length 511 [-Werror=stringop-truncation]
12 strncpy(dev->hardware_eui, dev->resp_data, RN2903_MAX_HEX_EUI64);
13 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
14
15Signed-off-by: Khem Raj <raj.khem@gmail.com>
16---
17 src/ecezo/ecezo.c | 2 +-
18 src/rn2903/rn2903.c | 2 +-
19 2 files changed, 2 insertions(+), 2 deletions(-)
20
21diff --git a/src/ecezo/ecezo.c b/src/ecezo/ecezo.c
22index 6a195fc1..56c6dab3 100644
23--- a/src/ecezo/ecezo.c
24+++ b/src/ecezo/ecezo.c
25@@ -488,7 +488,7 @@ int ecezo_send_command(const ecezo_context dev, char *cmd, char *buffer,
26 // our write buffer
27 char writeBuffer[ECEZO_MAX_BUFFER_LEN];
28
29- strncpy(writeBuffer, cmd, ECEZO_MAX_BUFFER_LEN);
30+ memcpy(writeBuffer, cmd, ECEZO_MAX_BUFFER_LEN-1);
31 writeBuffer[ECEZO_MAX_BUFFER_LEN - 1] = 0;
32
33 int writelen = strlen(writeBuffer);
34diff --git a/src/rn2903/rn2903.c b/src/rn2903/rn2903.c
35index f30a33ae..01a011da 100644
36--- a/src/rn2903/rn2903.c
37+++ b/src/rn2903/rn2903.c
38@@ -150,7 +150,7 @@ static rn2903_context _rn2903_postinit(rn2903_context dev,
39 rn2903_close(dev);
40 return NULL;
41 }
42- strncpy(dev->hardware_eui, dev->resp_data, RN2903_MAX_HEX_EUI64);
43+ memcpy(dev->hardware_eui, dev->resp_data, RN2903_MAX_HEX_EUI64);
44
45 return dev;
46 }
47--
482.17.0
49
diff --git a/meta-oe/recipes-extended/upm/upm/0001-Use-stdint-types.patch b/meta-oe/recipes-extended/upm/upm/0001-Use-stdint-types.patch
new file mode 100644
index 0000000000..40856fcf0d
--- /dev/null
+++ b/meta-oe/recipes-extended/upm/upm/0001-Use-stdint-types.patch
@@ -0,0 +1,139 @@
1From 5a1e731d71d577f56a2c013e4a75a8e90188e63b Mon Sep 17 00:00:00 2001
2From: Paul Eggleton <paul.eggleton@linux.intel.com>
3Date: Tue, 9 Jul 2019 05:21:59 -0700
4Subject: [PATCH] Use stdint types
5
6Fixes compilation with musl.
7
8Upstream-Status: Pending
9
10Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
11---
12 src/bma250e/bma250e.cxx | 16 ++++++++--------
13 src/bmg160/bmg160.cxx | 10 +++++-----
14 src/bmi160/bosch_bmi160.h | 2 +-
15 src/bmm150/bmm150.cxx | 8 ++++----
16 4 files changed, 18 insertions(+), 18 deletions(-)
17
18diff --git a/src/bma250e/bma250e.cxx b/src/bma250e/bma250e.cxx
19index a8792782..1b3ecdd8 100644
20--- a/src/bma250e/bma250e.cxx
21+++ b/src/bma250e/bma250e.cxx
22@@ -195,35 +195,35 @@ BMA250E::BMA250E(std::string initStr) : mraaIo(initStr)
23 fifoConfig(mode, axes);
24 }
25 if(tok.substr(0, 20) == "setInterruptEnable0:") {
26- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(20), nullptr, 0);
27+ uint8_t bits = (uint8_t)std::stoul(tok.substr(20), nullptr, 0);
28 setInterruptEnable0(bits);
29 }
30 if(tok.substr(0, 20) == "setInterruptEnable1:") {
31- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(20), nullptr, 0);
32+ uint8_t bits = (uint8_t)std::stoul(tok.substr(20), nullptr, 0);
33 setInterruptEnable1(bits);
34 }
35 if(tok.substr(0, 20) == "setInterruptEnable2:") {
36- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(20), nullptr, 0);
37+ uint8_t bits = (uint8_t)std::stoul(tok.substr(20), nullptr, 0);
38 setInterruptEnable2(bits);
39 }
40 if(tok.substr(0, 17) == "setInterruptMap0:") {
41- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
42+ uint8_t bits = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
43 setInterruptMap0(bits);
44 }
45 if(tok.substr(0, 17) == "setInterruptMap1:") {
46- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
47+ uint8_t bits = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
48 setInterruptMap1(bits);
49 }
50 if(tok.substr(0, 17) == "setInterruptMap2:") {
51- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
52+ uint8_t bits = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
53 setInterruptMap2(bits);
54 }
55 if(tok.substr(0, 16) == "setInterruptSrc:") {
56- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(16), nullptr, 0);
57+ uint8_t bits = (uint8_t)std::stoul(tok.substr(16), nullptr, 0);
58 setInterruptSrc(bits);
59 }
60 if(tok.substr(0, 26) == "setInterruptOutputControl:") {
61- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(26), nullptr, 0);
62+ uint8_t bits = (uint8_t)std::stoul(tok.substr(26), nullptr, 0);
63 setInterruptOutputControl(bits);
64 }
65 if(tok.substr(0, 26) == "setInterruptLatchBehavior:") {
66diff --git a/src/bmg160/bmg160.cxx b/src/bmg160/bmg160.cxx
67index 1a0939f8..2438c399 100644
68--- a/src/bmg160/bmg160.cxx
69+++ b/src/bmg160/bmg160.cxx
70@@ -173,23 +173,23 @@ BMG160::BMG160(std::string initStr) : mraaIo(initStr)
71 fifoConfig(mode, axes);
72 }
73 if(tok.substr(0, 20) == "setInterruptEnable0:") {
74- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(20), nullptr, 0);
75+ uint8_t bits = (uint8_t)std::stoul(tok.substr(20), nullptr, 0);
76 setInterruptEnable0(bits);
77 }
78 if(tok.substr(0, 17) == "setInterruptMap0:") {
79- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
80+ uint8_t bits = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
81 setInterruptMap0(bits);
82 }
83 if(tok.substr(0, 17) == "setInterruptMap1:") {
84- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
85+ uint8_t bits = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
86 setInterruptMap1(bits);
87 }
88 if(tok.substr(0, 16) == "setInterruptSrc:") {
89- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(16), nullptr, 0);
90+ uint8_t bits = (uint8_t)std::stoul(tok.substr(16), nullptr, 0);
91 setInterruptSrc(bits);
92 }
93 if(tok.substr(0, 26) == "setInterruptOutputControl:") {
94- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(26), nullptr, 0);
95+ uint8_t bits = (uint8_t)std::stoul(tok.substr(26), nullptr, 0);
96 setInterruptOutputControl(bits);
97 }
98 if(tok.substr(0, 26) == "setInterruptLatchBehavior:") {
99diff --git a/src/bmi160/bosch_bmi160.h b/src/bmi160/bosch_bmi160.h
100index 87ca2249..cd9efe6f 100644
101--- a/src/bmi160/bosch_bmi160.h
102+++ b/src/bmi160/bosch_bmi160.h
103@@ -81,7 +81,7 @@ typedef int16_t s16;/**< used for signed 16bit */
104 typedef int32_t s32;/**< used for signed 32bit */
105 typedef int64_t s64;/**< used for signed 64bit */
106
107-typedef u_int8_t u8;/**< used for unsigned 8bit */
108+typedef uint8_t u8;/**< used for unsigned 8bit */
109 typedef u_int16_t u16;/**< used for unsigned 16bit */
110 typedef u_int32_t u32;/**< used for unsigned 32bit */
111 typedef u_int64_t u64;/**< used for unsigned 64bit */
112diff --git a/src/bmm150/bmm150.cxx b/src/bmm150/bmm150.cxx
113index 234ebf92..02436788 100644
114--- a/src/bmm150/bmm150.cxx
115+++ b/src/bmm150/bmm150.cxx
116@@ -170,19 +170,19 @@ BMM150::BMM150(std::string initStr) : mraaIo(initStr)
117 setOpmode(opmode);
118 }
119 if(tok.substr(0, 19) == "setInterruptEnable:") {
120- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(19), nullptr, 0);
121+ uint8_t bits = (uint8_t)std::stoul(tok.substr(19), nullptr, 0);
122 setInterruptEnable(bits);
123 }
124 if(tok.substr(0, 19) == "setInterruptConfig:") {
125- u_int8_t bits = (u_int8_t)std::stoul(tok.substr(19), nullptr, 0);
126+ uint8_t bits = (uint8_t)std::stoul(tok.substr(19), nullptr, 0);
127 setInterruptConfig(bits);
128 }
129 if(tok.substr(0, 17) == "setRepetitionsXY:") {
130- u_int8_t reps = (u_int8_t)std::stoul(tok.substr(17), nullptr, 0);
131+ uint8_t reps = (uint8_t)std::stoul(tok.substr(17), nullptr, 0);
132 setRepetitionsXY(reps);
133 }
134 if(tok.substr(0, 16) == "setRepetitionsZ:") {
135- u_int8_t reps = (u_int8_t)std::stoul(tok.substr(16), nullptr, 0);
136+ uint8_t reps = (uint8_t)std::stoul(tok.substr(16), nullptr, 0);
137 setRepetitionsZ(reps);
138 }
139 if(tok.substr(0, 14) == "setPresetMode:") {
diff --git a/meta-oe/recipes-extended/upm/upm/0001-include-sys-types.h-for-uint-definition.patch b/meta-oe/recipes-extended/upm/upm/0001-include-sys-types.h-for-uint-definition.patch
deleted file mode 100644
index a0a47411d7..0000000000
--- a/meta-oe/recipes-extended/upm/upm/0001-include-sys-types.h-for-uint-definition.patch
+++ /dev/null
@@ -1,33 +0,0 @@
1From 48a580bd402cf6a3ee9e42013653219bfeb3caf6 Mon Sep 17 00:00:00 2001
2From: Khem Raj <raj.khem@gmail.com>
3Date: Thu, 21 Jun 2018 18:39:16 -0700
4Subject: [PATCH] include sys/types.h for uint definition
5
6uint is defined in sys/types.h, therefore this
7header needs to be included, it gets exposed with
8musl where this header is not getting included indirectly
9as it is happening when building on glibc
10
11Fixes build errors on musl e.g.
12upm/src/kx122/kx122.hpp:456:31: error: 'uint' has not been declared
13| void setBufferThreshold(uint samples);
14| ^~~~
15
16Signed-off-by: Khem Raj <raj.khem@gmail.com>
17---
18Upstream-Status: Submitted [https://github.com/intel-iot-devkit/upm/pull/656]
19 src/kx122/kx122.h | 1 +
20 1 file changed, 1 insertion(+)
21
22diff --git a/src/kx122/kx122.h b/src/kx122/kx122.h
23index 1622ed50..56e5215e 100644
24--- a/src/kx122/kx122.h
25+++ b/src/kx122/kx122.h
26@@ -31,6 +31,7 @@ extern "C"{
27 #include <assert.h>
28 #include <unistd.h>
29 #include <math.h>
30+#include <sys/types.h>
31
32 #include <mraa/i2c.h>
33 #include <mraa/spi.h>
diff --git a/meta-oe/recipes-extended/upm/upm_git.bb b/meta-oe/recipes-extended/upm/upm_git.bb
index 8854a33bcb..a6b379ec36 100644
--- a/meta-oe/recipes-extended/upm/upm_git.bb
+++ b/meta-oe/recipes-extended/upm/upm_git.bb
@@ -7,13 +7,12 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=66493d54e65bfc12c7983ff2e884f37f"
7 7
8DEPENDS = "libjpeg-turbo mraa" 8DEPENDS = "libjpeg-turbo mraa"
9 9
10SRCREV = "dc45cd78595c7c24c8a8574c63bb48b5bb99c5aa" 10SRCREV = "5cf20df96c6b35c19d5b871ba4e319e96b4df72d"
11PV = "1.6.0-git${SRCPV}" 11PV = "2.0.0+git${SRCPV}"
12 12
13SRC_URI = "git://github.com/intel-iot-devkit/${BPN}.git;protocol=http \ 13SRC_URI = "git://github.com/intel-iot-devkit/${BPN}.git;protocol=http \
14 file://0001-Replace-strncpy-with-memcpy.patch \
15 file://0001-include-sys-types.h-for-uint-definition.patch \
16 file://0001-CMakeLists.txt-Use-SWIG_SUPPORT_FILES-to-find-the-li.patch \ 14 file://0001-CMakeLists.txt-Use-SWIG_SUPPORT_FILES-to-find-the-li.patch \
15 file://0001-Use-stdint-types.patch \
17 " 16 "
18 17
19S = "${WORKDIR}/git" 18S = "${WORKDIR}/git"
generated by cgit v1.2.3-54-g00ecf (git 2.39.0) at 2025-10-04 08:27:22 +0000