Revert "of/fdt: Make sure no-map does not remove already reserved regions"
authorQuentin Perret <qperret@google.com>
Wed, 12 May 2021 12:30:22 +0000 (12:30 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 22 May 2021 08:57:35 +0000 (10:57 +0200)
This reverts commit 6793433fc8f263eaba1621d3724b6aeba511c6c5.
It is not really a fix, and the backport misses dependencies, which
breaks existing platforms.

Reported-by: Alexandre TORGUE <alexandre.torgue@foss.st.com>
Signed-off-by: Quentin Perret <qperret@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/of/fdt.c

index 3f58812d02d9f6fbf434a84071cfb55bddeeb3cc..6df66fcefbb404a985ff2e413deb7f0dedbfdf0f 100644 (file)
@@ -1212,16 +1212,8 @@ int __init __weak early_init_dt_mark_hotplug_memory_arch(u64 base, u64 size)
 int __init __weak early_init_dt_reserve_memory_arch(phys_addr_t base,
                                        phys_addr_t size, bool nomap)
 {
-       if (nomap) {
-               /*
-                * If the memory is already reserved (by another region), we
-                * should not allow it to be marked nomap.
-                */
-               if (memblock_is_region_reserved(base, size))
-                       return -EBUSY;
-
+       if (nomap)
                return memblock_mark_nomap(base, size);
-       }
        return memblock_reserve(base, size);
 }