package/devmem2: remove package

It was here just for backwards compatibility and is not used by any
package or board after commits

  59856480b6 packages/devmem2: remove from all board configurations
  2f6e3eae55 package/ti-gfx: do not use devmem2 in script

There are address casting issues with 32bit access on a 64bit bus (like
on the SoC FPGA ARM processors) with this tool. The Busybox version of
devmem is the most up to date and supports quadword access.

Fixes: https://bugs.busybox.net/show_bug.cgi?id=10171
Signed-off-by: Carlos Santos <unixmania@gmail.com>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

diff --git a/Config.in.legacy b/Config.in.legacy
index b8b550e5ee63f84f27e9c931e159e3acc51a2ebf..aeeeba04ddd86ab4dea3e3cc732806a04cecdcb7 100644 (file)
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -146,6 +146,13 @@ endif
 
 comment "Legacy options removed in 2019.11"
 
+config BR2_PACKAGE_DEVMEM2
+       bool "devmem2 package was removed"
+       select BR2_LEGACY
+       help
+         Use the the Busybox devmem utility, instead, which provides
+         the same functionality.
+
 config BR2_PACKAGE_USTR
        bool "ustr package removed"
        select BR2_LEGACY

diff --git a/DEVELOPERS b/DEVELOPERS
index b427fd92ae030844d497d08ba1e1084206943b63..9e68a153a5777564e5eb4913cb28bb50cbec371d 100644 (file)
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -1516,7 +1516,6 @@ F:        package/cgroupfs-mount/
 F:     package/crda/
 F:     package/cunit/
 F:     package/dacapo/
-F:     package/devmem2/
 F:     package/dnsmasq/
 F:     package/dosfstools/
 F:     package/eigen/

diff --git a/package/devmem2/0001-be-coherent-in-type-usage.patch b/package/devmem2/0001-be-coherent-in-type-usage.patch
deleted file mode 100644 (file)
index 1777387..0000000
--- a/package/devmem2/0001-be-coherent-in-type-usage.patch
+++ /dev/null
@@ -1,52 +0,0 @@
-From 6570e7afe1eebd64022b871b9ebd9a76be267f57 Mon Sep 17 00:00:00 2001
-From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
-Date: Sun, 29 Jun 2014 16:27:49 +0200
-Subject: [PATCH] Be coherent in type usage
-
-- no need to use off_t because it is read from strtoul()
-- print unsigned int type correctly
-
-When built with _FILE_OFFSET_BITS=64, off_t becomes an long long int
-and was printed as an unsigned int which lead to a garbled output.
-
-Signed-off-by: Colin Didier <colin.didier@devialet.com>
-Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
----
- devmem2.c | 7 +++----
- 1 file changed, 3 insertions(+), 4 deletions(-)
-
-diff --git a/devmem2.c b/devmem2.c
-index 7732ecb..7aa5c79 100644
---- a/devmem2.c
-+++ b/devmem2.c
-@@ -58,8 +58,7 @@
- int main(int argc, char **argv) {
-     int fd;
-     void *map_base, *virt_addr; 
--      unsigned long read_result, writeval;
--      off_t target;
-+      unsigned long target, read_result, writeval;
-       int access_type = 'w';
-       
-       if(argc < 2) {
-@@ -101,7 +100,7 @@ int main(int argc, char **argv) {
-                       fprintf(stderr, "Illegal data type '%c'.\n", access_type);
-                       exit(2);
-       }
--    printf("Value at address 0x%X (%p): 0x%X\n", target, virt_addr, read_result); 
-+    printf("Value at address 0x%lX (%p): 0x%X\n", target, virt_addr, read_result); 
-     fflush(stdout);
- 
-       if(argc > 3) {
-@@ -120,7 +119,7 @@ int main(int argc, char **argv) {
-                               read_result = *((unsigned long *) virt_addr);
-                               break;
-               }
--              printf("Written 0x%X; readback 0x%X\n", writeval, read_result); 
-+              printf("Written 0x%lX; readback 0x%X\n", writeval, read_result); 
-               fflush(stdout);
-       }
-       
--- 
-2.0.0
-

diff --git a/package/devmem2/Config.in b/package/devmem2/Config.in
deleted file mode 100644 (file)
index 1ccb055..0000000
--- a/package/devmem2/Config.in
+++ /dev/null
@@ -1,7 +0,0 @@
-config BR2_PACKAGE_DEVMEM2
-       bool "devmem2"
-       depends on BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
-       help
-         Simple program to read/write from/to any location in memory.
-
-         http://bootlin.com/pub/mirror/devmem2.c

diff --git a/package/devmem2/devmem2.hash b/package/devmem2/devmem2.hash
deleted file mode 100644 (file)
index aa638d6..0000000
--- a/package/devmem2/devmem2.hash
+++ /dev/null
@@ -1,3 +0,0 @@
-# Locally calculated
-sha256 3b15515693bae1ebd14d914e46d388edfec2175829ea1576a7a0c8606ebbe639  devmem2.c
-sha256 560913d09310d446920c16bd0aca1ff21d3b6fedd527195bacfc832ce343e0ae  devmem2.c.license

diff --git a/package/devmem2/devmem2.mk b/package/devmem2/devmem2.mk
deleted file mode 100644 (file)
index e1bbee3..0000000
--- a/package/devmem2/devmem2.mk
+++ /dev/null
@@ -1,30 +0,0 @@
-################################################################################
-#
-# devmem2
-#
-################################################################################
-
-DEVMEM2_SITE = http://bootlin.com/pub/mirror
-DEVMEM2_SOURCE = devmem2.c
-DEVMEM2_VERSION = 1
-DEVMEM2_LICENSE = GPL-2.0+
-DEVMEM2_LICENSE_FILES = devmem2.c.license
-
-define DEVMEM2_EXTRACT_CMDS
-       cp $(DEVMEM2_DL_DIR)/$($(PKG)_SOURCE) $(@D)/
-endef
-
-define DEVMEM2_EXTRACT_LICENSE
-       head -n 38 $(@D)/devmem2.c >$(@D)/devmem2.c.license
-endef
-DEVMEM2_PRE_PATCH_HOOKS += DEVMEM2_EXTRACT_LICENSE
-
-define DEVMEM2_BUILD_CMDS
-       $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) devmem2
-endef
-
-define DEVMEM2_INSTALL_TARGET_CMDS
-       $(INSTALL) -D $(@D)/devmem2 $(TARGET_DIR)/sbin/devmem2
-endef
-
-$(eval $(generic-package))