[PATCH 05/12] ARM64: mmu: Merge create_sections() and map_region() together

[Andrey Smirnov]

Since map_region() is never called without being followed by
tlb_invalidate(), merge it with create_sections() to simplify the
code.

Signed-off-by: Andrey Smirnov <andrew.smir...@gmail.com>
---
 arch/arm/cpu/mmu_64.c | 12 ++++--------
 1 file changed, 4 insertions(+), 8 deletions(-)

diff --git a/arch/arm/cpu/mmu_64.c b/arch/arm/cpu/mmu_64.c
index d7c2542d1..ad94ff14d 100644
--- a/arch/arm/cpu/mmu_64.c
+++ b/arch/arm/cpu/mmu_64.c
@@ -119,7 +119,8 @@ static void split_block(uint64_t *pte, int level)
        set_table(pte, new_table);
 }
 
-static void map_region(uint64_t virt, uint64_t phys, uint64_t size, uint64_t 
attr)
+static void create_sections(uint64_t virt, uint64_t phys, uint64_t size,
+                           uint64_t attr)
 {
        uint64_t block_size;
        uint64_t block_shift;
@@ -162,11 +163,7 @@ static void map_region(uint64_t virt, uint64_t phys, 
uint64_t size, uint64_t att
                }
 
        }
-}
 
-static void create_sections(uint64_t virt, uint64_t phys, uint64_t size, 
uint64_t flags)
-{
-       map_region(virt, phys, size, flags);
        tlb_invalidate();
 }
 
@@ -183,9 +180,8 @@ int arch_remap_range(void *_start, size_t size, unsigned 
flags)
                return -EINVAL;
        }
 
-       map_region((uint64_t)_start, (uint64_t)_start, (uint64_t)size, flags);
-       tlb_invalidate();
-
+       create_sections((uint64_t)_start, (uint64_t)_start, (uint64_t)size,
+                       flags);
        return 0;
 }
 
-- 
2.20.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox