]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
core: fdtaddr: use map_sysmem() as cast for the return
authorJohan Jonker <jbx6244@gmail.com>
Sun, 23 Apr 2023 09:19:41 +0000 (11:19 +0200)
committerSimon Glass <sjg@chromium.org>
Fri, 28 Apr 2023 17:52:30 +0000 (11:52 -0600)
For the devfdt_get_addr_index_ptr() and devfdt_get_addr_size_index_ptr()
function use map_sysmem() function as cast for the return for use in
sandbox. Also fix sandbox test.

Signed-off-by: Johan Jonker <jbx6244@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Drop second hunk:
Signed-off-by: Simon Glass <sjg@chromium.org>
drivers/core/fdtaddr.c
test/dm/test-fdt.c

index 91bcd1a2c21557e4c77a530f0492cb0f9c2c3e9d..b9b0c28852f96231970e023ade7e7fb7f16974b1 100644 (file)
@@ -12,6 +12,7 @@
 #include <dm.h>
 #include <fdt_support.h>
 #include <log.h>
+#include <mapmem.h>
 #include <asm/global_data.h>
 #include <asm/io.h>
 #include <dm/device-internal.h>
@@ -97,7 +98,10 @@ void *devfdt_get_addr_index_ptr(const struct udevice *dev, int index)
 {
        fdt_addr_t addr = devfdt_get_addr_index(dev, index);
 
-       return (addr == FDT_ADDR_T_NONE) ? NULL : (void *)(uintptr_t)addr;
+       if (addr == FDT_ADDR_T_NONE)
+               return NULL;
+
+       return map_sysmem(addr, 0);
 }
 
 fdt_addr_t devfdt_get_addr_size_index(const struct udevice *dev, int index,
index 8e6e42e46b479d8ba2c4948488167ad09ad4ee93..eeecd1dc2dec654efc9cfc74ef89287c0c2f72bd 100644 (file)
@@ -617,6 +617,7 @@ static int dm_test_fdt_get_addr_ptr_flat(struct unit_test_state *uts)
 {
        struct udevice *gpio, *dev;
        void *ptr;
+       void *paddr;
 
        /* Test for missing reg property */
        ut_assertok(uclass_first_device_err(UCLASS_GPIO, &gpio));
@@ -624,7 +625,9 @@ static int dm_test_fdt_get_addr_ptr_flat(struct unit_test_state *uts)
 
        ut_assertok(uclass_find_device_by_seq(UCLASS_TEST_DUMMY, 0, &dev));
        ptr = devfdt_get_addr_ptr(dev);
-       ut_asserteq_ptr((void *)0x8000, ptr);
+
+       paddr = map_sysmem(0x8000, 0);
+       ut_asserteq_ptr(paddr, ptr);
 
        return 0;
 }