You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by je...@apache.org on 2023/11/17 08:43:00 UTC
(nuttx) 02/03: openamp/libmetal: aligned io read/write when size=1,2,3,4
This is an automated email from the ASF dual-hosted git repository.
jerpelea pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/nuttx.git
commit 8ed836f05eacc9790cacfc205943010ad2fb3daa
Author: Bowen Wang <wa...@xiaomi.com>
AuthorDate: Thu Nov 16 15:26:57 2023 +0800
openamp/libmetal: aligned io read/write when size=1,2,3,4
Make NuttX implemented metal io read/write to aligned access
Signed-off-by: Bowen Wang <wa...@xiaomi.com>
---
...ttx-io.c-align-access-when-read-write-siz.patch | 57 ++++++++++++++++++++++
openamp/libmetal.defs | 1 +
2 files changed, 58 insertions(+)
diff --git a/openamp/0002-libmetal-nuttx-io.c-align-access-when-read-write-siz.patch b/openamp/0002-libmetal-nuttx-io.c-align-access-when-read-write-siz.patch
new file mode 100644
index 0000000000..8903c94bd9
--- /dev/null
+++ b/openamp/0002-libmetal-nuttx-io.c-align-access-when-read-write-siz.patch
@@ -0,0 +1,57 @@
+From af3de6cd042c89ae2e8d07d503e0a07129e1296c Mon Sep 17 00:00:00 2001
+From: Bowen Wang <wa...@xiaomi.com>
+Date: Thu, 16 Nov 2023 14:52:48 +0800
+Subject: [PATCH] libmetal/nuttx/io.c: align access when read/write size = 1,
+ 2, 4, 8
+
+Signed-off-by: Bowen Wang <wa...@xiaomi.com>
+---
+ lib/system/nuttx/io.c | 25 +++++++++++++++++++++++--
+ 1 file changed, 23 insertions(+), 2 deletions(-)
+
+diff --git a/lib/system/nuttx/io.c libmetal/lib/system/nuttx/io.c
+index 4fa4727..3ce9cbe 100644
+--- a/lib/system/nuttx/io.c
++++ libmetal/lib/system/nuttx/io.c
+@@ -37,7 +37,17 @@ static int metal_io_block_read_(struct metal_io_region *io,
+ void *va = metal_io_virt(io, offset);
+
+ metal_cache_invalidate(va, len);
+- memcpy(dst, va, len);
++ if (len == 1)
++ *(uint8_t *)dst = *(uint8_t *)va;
++ else if (len == 2)
++ *(uint16_t *)dst = *(uint16_t *)va;
++ else if (len == 4)
++ *(uint32_t *)dst = *(uint32_t *)va;
++ else if (len == 8) {
++ *(uint32_t *)dst = *(uint32_t *)va;
++ *(uint32_t *)(dst + 4) = *(uint32_t *)(va + 4);
++ } else
++ memcpy(dst, va, len);
+
+ return len;
+ }
+@@ -50,7 +60,18 @@ static int metal_io_block_write_(struct metal_io_region *io,
+ {
+ void *va = metal_io_virt(io, offset);
+
+- memcpy(va, src, len);
++ if (len == 1)
++ *(uint8_t *)va = *(uint8_t *)src;
++ else if (len == 2)
++ *(uint16_t *)va = *(uint16_t *)src;
++ else if (len == 4)
++ *(uint32_t *)va = *(uint32_t *)src;
++ else if (len == 8) {
++ *(uint32_t *)va = *(uint32_t *)src;
++ *(uint32_t *)(va + 4) = *(uint32_t *)(src + 4);
++ } else
++ memcpy(va, src, len);
++
+ metal_cache_flush(va, len);
+
+ return len;
+--
+2.34.1
+
diff --git a/openamp/libmetal.defs b/openamp/libmetal.defs
index a4184d55d1..b3369496ab 100644
--- a/openamp/libmetal.defs
+++ b/openamp/libmetal.defs
@@ -78,6 +78,7 @@ libmetal.zip:
$(Q) unzip -o libmetal.zip
$(Q) mv libmetal-$(VERSION) libmetal
$(Q) patch -p0 < 0001-libmetal-add-metal_list_for_each_safe-support.patch
+ $(Q) patch -p0 < 0002-libmetal-nuttx-io.c-align-access-when-read-write-siz.patch
.libmetal_headers: libmetal.zip
else