__late_set_fixmap() and __late_clear_fixmap() are only used after
early_ioremap_reset() has been called, but the comment above them does
not say anything about that. So arm64, riscv and powerpc, whose
__set_fixmap() works before and after paging_init(), describe the same
situation in three different ways:

                 calls reset    defines the macros
        arm64    yes            yes
        riscv    no             yes
        powerpc  no             no

Patch 1 documents when early_ioremap_reset() needs to be called and
that only architectures calling it need to define the macros.

Patches 2 and 3 remove the unneeded riscv macros, which are
unreachable, and the arm64 reset call and macros, which change
nothing.

No functional change.

Sang-Heon Jeon (3):
  mm/early_ioremap: clarify early_ioremap_reset() semantics
  riscv: remove unused __late_set_fixmap() and __late_clear_fixmap()
  arm64: remove early_ioremap_reset() call and __late_* macros

 arch/arm64/include/asm/fixmap.h |  3 ---
 arch/arm64/kernel/setup.c       |  2 --
 arch/riscv/include/asm/fixmap.h |  3 ---
 mm/early_ioremap.c              | 10 +++++++---
 4 files changed, 7 insertions(+), 11 deletions(-)

-- 
2.43.0


Reply via email to