]> git.karo-electronics.de Git - linux-beck.git/commitdiff
sh: fixed PMB mode refactoring.
authorPaul Mundt <lethal@linux-sh.org>
Wed, 13 Jan 2010 09:31:48 +0000 (18:31 +0900)
committerPaul Mundt <lethal@linux-sh.org>
Wed, 13 Jan 2010 09:31:48 +0000 (18:31 +0900)
This introduces some much overdue chainsawing of the fixed PMB support.
fixed PMB was introduced initially to work around the fact that dynamic
PMB mode was relatively broken, though they were never intended to
converge. The main areas where there are differences are whether the
system is booted in 29-bit mode or 32-bit mode, and whether legacy
mappings are to be preserved. Any system booting in true 32-bit mode will
not care about legacy mappings, so these are roughly decoupled.

Regardless of the entry point, PMB and 32BIT are directly related as far
as the kernel is concerned, so we also switch back to having one select
the other.

With legacy mappings iterated through and applied in the initialization
path it's now possible to finally merge the two implementations and
permit dynamic remapping overtop of remaining entries regardless of
whether boot mappings are crafted by hand or inherited from the boot
loader.

Signed-off-by: Paul Mundt <lethal@linux-sh.org>
arch/sh/boot/Makefile
arch/sh/include/asm/addrspace.h
arch/sh/include/asm/io.h
arch/sh/include/asm/mmu.h
arch/sh/include/asm/page.h
arch/sh/kernel/head_32.S
arch/sh/kernel/setup.c
arch/sh/kernel/vmlinux.lds.S
arch/sh/mm/Kconfig
arch/sh/mm/Makefile
arch/sh/mm/pmb.c

index 743ce0c8d98d212bb3a746e55e8e0a8c11ed750b..1ce63624c9b9c8140a1eb9d348442b7fa2d560e8 100644 (file)
@@ -46,15 +46,8 @@ $(obj)/romImage: $(obj)/romimage/vmlinux FORCE
 $(obj)/romimage/vmlinux: $(obj)/zImage FORCE
        $(Q)$(MAKE) $(build)=$(obj)/romimage $@
 
-KERNEL_MEMORY := 0x00000000
-ifeq ($(CONFIG_PMB_FIXED),y)
-KERNEL_MEMORY := $(shell /bin/bash -c 'printf "0x%08x" \
+KERNEL_MEMORY  := $(shell /bin/bash -c 'printf "0x%08x" \
                     $$[$(CONFIG_MEMORY_START) & 0x1fffffff]')
-endif
-ifeq ($(CONFIG_29BIT),y)
-KERNEL_MEMORY := $(shell /bin/bash -c 'printf "0x%08x" \
-                    $$[$(CONFIG_MEMORY_START)]')
-endif
 
 KERNEL_LOAD    := $(shell /bin/bash -c 'printf "0x%08x" \
                     $$[$(CONFIG_PAGE_OFFSET)  + \
index 99d6b3ecbe22d41167d2f4221d8d6223250f7017..bcd7d4d78f6bdb2d96e4ec4354782c03cee3dc18 100644 (file)
@@ -28,7 +28,7 @@
 /* Returns the privileged segment base of a given address  */
 #define PXSEG(a)       (((unsigned long)(a)) & 0xe0000000)
 
-#if defined(CONFIG_29BIT) || defined(CONFIG_PMB_FIXED)
+#ifdef CONFIG_29BIT
 /*
  * Map an address to a certain privileged segment
  */
@@ -40,7 +40,7 @@
        ((__typeof__(a))(((unsigned long)(a) & 0x1fffffff) | P3SEG))
 #define P4SEGADDR(a)   \
        ((__typeof__(a))(((unsigned long)(a) & 0x1fffffff) | P4SEG))
-#endif /* 29BIT || PMB_FIXED */
+#endif /* 29BIT */
 #endif /* P1SEG */
 
 /* Check if an address can be reached in 29 bits */
index 026dd659a6400cf70f6105167cfa927e4670f978..f4314d8b05b8f4a7dab0df47e87acd8e52727461 100644 (file)
@@ -244,18 +244,11 @@ __ioremap(unsigned long offset, unsigned long size, unsigned long flags)
 }
 
 static inline void __iomem *
-__ioremap_mode(unsigned long offset, unsigned long size, unsigned long flags)
+__ioremap_29bit(unsigned long offset, unsigned long size, unsigned long flags)
 {
-#if defined(CONFIG_SUPERH32) && !defined(CONFIG_PMB_FIXED) && !defined(CONFIG_PMB)
+#ifdef CONFIG_29BIT
        unsigned long last_addr = offset + size - 1;
-#endif
-       void __iomem *ret;
 
-       ret = __ioremap_trapped(offset, size);
-       if (ret)
-               return ret;
-
-#if defined(CONFIG_SUPERH32) && !defined(CONFIG_PMB_FIXED) && !defined(CONFIG_PMB)
        /*
         * For P1 and P2 space this is trivial, as everything is already
         * mapped. Uncached access for P1 addresses are done through P2.
@@ -274,6 +267,22 @@ __ioremap_mode(unsigned long offset, unsigned long size, unsigned long flags)
                return (void __iomem *)P4SEGADDR(offset);
 #endif
 
+       return NULL;
+}
+
+static inline void __iomem *
+__ioremap_mode(unsigned long offset, unsigned long size, unsigned long flags)
+{
+       void __iomem *ret;
+
+       ret = __ioremap_trapped(offset, size);
+       if (ret)
+               return ret;
+
+       ret = __ioremap_29bit(offset, size, flags);
+       if (ret)
+               return ret;
+
        return __ioremap(offset, size, flags);
 }
 #else
index c7426ad9926e7c03bda9d717ec79108c345f63d1..4b0882bf5183c96101773f6985b6aa6dc99f0d0d 100644 (file)
@@ -65,11 +65,29 @@ struct pmb_entry {
        struct pmb_entry *link;
 };
 
+#ifdef CONFIG_PMB
 /* arch/sh/mm/pmb.c */
 long pmb_remap(unsigned long virt, unsigned long phys,
               unsigned long size, unsigned long flags);
 void pmb_unmap(unsigned long addr);
 int pmb_init(void);
+#else
+static inline long pmb_remap(unsigned long virt, unsigned long phys,
+                            unsigned long size, unsigned long flags)
+{
+       return -EINVAL
+}
+
+static inline void pmb_unmap(unsigned long addr)
+{
+}
+
+static inline int pmb_init(void)
+{
+       return -ENODEV;
+}
+#endif /* CONFIG_PMB */
+
 #endif /* __ASSEMBLY__ */
 
 #endif /* __MMU_H */
index 81bffc0d6860972953ed2beef9be548fee7b40fe..a86c0f1d05d4549d4af7fb5955aa1491fbd52a87 100644 (file)
@@ -127,7 +127,7 @@ typedef struct page *pgtable_t;
  * is not visible (it is part of the PMB mapping) and so needs to be
  * added or subtracted as required.
  */
-#if defined(CONFIG_PMB_FIXED)
+#if defined(CONFIG_PMB_LEGACY)
 /* phys = virt - PAGE_OFFSET - (__MEMORY_START & 0xe0000000) */
 #define PMB_OFFSET     (PAGE_OFFSET - PXSEG(__MEMORY_START))
 #define __pa(x)        ((unsigned long)(x) - PMB_OFFSET)
index 1151ecdffa719e4283b32d78cd82adfc97fa97a8..e5d421db4c833846a4fc98e60b800247590d18c1 100644 (file)
@@ -13,6 +13,8 @@
 #include <linux/init.h>
 #include <linux/linkage.h>
 #include <asm/thread_info.h>
+#include <asm/mmu.h>
+#include <cpu/mmu_context.h>
 
 #ifdef CONFIG_CPU_SH4A
 #define SYNCO()                synco
@@ -33,7 +35,7 @@ ENTRY(empty_zero_page)
        .long   1               /* LOADER_TYPE */
        .long   0x00000000      /* INITRD_START */
        .long   0x00000000      /* INITRD_SIZE */
-#if defined(CONFIG_32BIT) && defined(CONFIG_PMB_FIXED)
+#ifdef CONFIG_32BIT
        .long   0x53453f00 + 32 /* "SE?" = 32 bit */
 #else
        .long   0x53453f00 + 29 /* "SE?" = 29 bit */
index 8b0e69792cf47faec445adc8f784d8debf4b2899..f79ebe32a24aa2d7f4ae7c718dc9096997b6a351 100644 (file)
@@ -455,10 +455,7 @@ void __init setup_arch(char **cmdline_p)
                sh_mv.mv_setup(cmdline_p);
 
        paging_init();
-
-#ifdef CONFIG_PMB_ENABLE
        pmb_init();
-#endif
 
 #ifdef CONFIG_SMP
        plat_smp_setup();
index a1e4ec24f1f5de82450606dc8ff7cc20e8756424..9e5a5878eeaee4dec1865331fd85b0a54509a778 100644 (file)
@@ -14,17 +14,16 @@ OUTPUT_ARCH(sh)
 #include <asm/cache.h>
 #include <asm/vmlinux.lds.h>
 
+#if defined(CONFIG_32BIT) && !defined(CONFIG_PMB_LEGACY)
+#define MEMORY_OFFSET  0
+#else
+#define MEMORY_OFFSET  (CONFIG_MEMORY_START & 0x1fffffff)
+#endif
+
 ENTRY(_start)
 SECTIONS
 {
-#ifdef CONFIG_PMB_FIXED
-       . = CONFIG_PAGE_OFFSET + (CONFIG_MEMORY_START & 0x1fffffff) +
-           CONFIG_ZERO_PAGE_OFFSET;
-#elif defined(CONFIG_32BIT)
-       . = CONFIG_PAGE_OFFSET + CONFIG_ZERO_PAGE_OFFSET;
-#else
-       . = CONFIG_PAGE_OFFSET + CONFIG_MEMORY_START + CONFIG_ZERO_PAGE_OFFSET;
-#endif
+       . = CONFIG_PAGE_OFFSET + MEMORY_OFFSET + CONFIG_ZERO_PAGE_OFFSET;
 
        _text = .;              /* Text and read-only data */
 
index 358c860aeb9b1cf47176a8fc2cdc444c7fb40ba1..860cd24b4205e7910fe76e0378ab4c66a8e9b09b 100644 (file)
@@ -80,30 +80,18 @@ config 32BIT
        bool
        default y if CPU_SH5
 
-config PMB_ENABLE
-       bool "Support 32-bit physical addressing through PMB"
-       depends on MMU && EXPERIMENTAL && CPU_SH4A && !CPU_SH4AL_DSP
-       help
-         If you say Y here, physical addressing will be extended to
-         32-bits through the SH-4A PMB. If this is not set, legacy
-         29-bit physical addressing will be used.
-
-choice
-       prompt "PMB handling type"
-       depends on PMB_ENABLE
-       default PMB_FIXED
-
 config PMB
-       bool "PMB"
+       bool "Support 32-bit physical addressing through PMB"
        depends on MMU && EXPERIMENTAL && CPU_SH4A && !CPU_SH4AL_DSP
+       select 32BIT
        help
          If you say Y here, physical addressing will be extended to
          32-bits through the SH-4A PMB. If this is not set, legacy
          29-bit physical addressing will be used.
 
-config PMB_FIXED
-       bool "fixed PMB"
-       depends on MMU && EXPERIMENTAL && CPU_SH4A && !CPU_SH4AL_DSP
+config PMB_LEGACY
+       bool "Support legacy boot mappings for PMB"
+       depends on PMB
        select 32BIT
        help
          If this option is enabled, fixed PMB mappings are inherited
@@ -111,8 +99,6 @@ config PMB_FIXED
          management. This is the closest to legacy 29-bit physical mode,
          and allows systems to support up to 512MiB of system memory.
 
-endchoice
-
 config X2TLB
        bool "Enable extended TLB mode"
        depends on (CPU_SHX2 || CPU_SHX3) && MMU && EXPERIMENTAL
index 9fa11d6550447e79f93c517f2bbcc6c500382878..edde8bdd681d474aef5cc3293a34477e3492e9b2 100644 (file)
@@ -33,7 +33,7 @@ obj-y                         += $(tlb-y)
 endif
 
 obj-$(CONFIG_HUGETLB_PAGE)     += hugetlbpage.o
-obj-$(CONFIG_PMB_ENABLE)       += pmb.o
+obj-$(CONFIG_PMB)              += pmb.o
 obj-$(CONFIG_NUMA)             += numa.o
 
 # Special flags for fault_64.o.  This puts restrictions on the number of
index 280f6a166035fe9c418dd831e1f188eb6a923e6b..8f7dbf183fb01f04508519629fbf9040b89ec41e 100644 (file)
@@ -3,7 +3,7 @@
  *
  * Privileged Space Mapping Buffer (PMB) Support.
  *
- * Copyright (C) 2005, 2006, 2007 Paul Mundt
+ * Copyright (C) 2005 - 2010 Paul Mundt
  *
  * P1/P2 Section mapping definitions from map32.h, which was:
  *
@@ -279,51 +279,12 @@ static void __pmb_unmap(struct pmb_entry *pmbe)
        } while (pmbe);
 }
 
-#ifdef CONFIG_PMB
-int __uses_jump_to_uncached pmb_init(void)
-{
-       unsigned int i;
-       long size, ret;
-
-       jump_to_uncached();
-
-       /*
-        * Insert PMB entries for the P1 and P2 areas so that, after
-        * we've switched the MMU to 32-bit mode, the semantics of P1
-        * and P2 are the same as in 29-bit mode, e.g.
-        *
-        *      P1 - provides a cached window onto physical memory
-        *      P2 - provides an uncached window onto physical memory
-        */
-       size = __MEMORY_START + __MEMORY_SIZE;
-
-       ret = pmb_remap(P1SEG, 0x00000000, size, PMB_C);
-       BUG_ON(ret != size);
-
-       ret = pmb_remap(P2SEG, 0x00000000, size, PMB_WT | PMB_UB);
-       BUG_ON(ret != size);
-
-       ctrl_outl(0, PMB_IRMCR);
-
-       /* PMB.SE and UB[7] */
-       ctrl_outl(PASCR_SE | (1 << 7), PMB_PASCR);
-
-       /* Flush out the TLB */
-       i =  ctrl_inl(MMUCR);
-       i |= MMUCR_TI;
-       ctrl_outl(i, MMUCR);
-
-       back_to_cached();
-
-       return 0;
-}
-#else
-int __uses_jump_to_uncached pmb_init(void)
+#ifdef CONFIG_PMB_LEGACY
+static int pmb_apply_legacy_mappings(void)
 {
        int i;
        unsigned long addr, data;
-
-       jump_to_uncached();
+       unsigned int applied = 0;
 
        for (i = 0; i < PMB_ENTRY_MAX; i++) {
                struct pmb_entry *pmbe;
@@ -357,13 +318,69 @@ int __uses_jump_to_uncached pmb_init(void)
 
                pmbe = pmb_alloc(vpn, ppn, flags, i);
                WARN_ON(IS_ERR(pmbe));
+
+               applied++;
+       }
+
+       return (applied == 0);
+}
+#else
+static inline int pmb_apply_legacy_mappings(void)
+{
+       return 1;
+}
+#endif
+
+int __uses_jump_to_uncached pmb_init(void)
+{
+       unsigned int i;
+       unsigned long size, ret;
+
+       jump_to_uncached();
+
+       /*
+        * Attempt to apply the legacy boot mappings if configured. If
+        * this is successful then we simply carry on with those and
+        * don't bother establishing additional memory mappings. Dynamic
+        * device mappings through pmb_remap() can still be bolted on
+        * after this.
+        */
+       ret = pmb_apply_legacy_mappings();
+       if (ret == 0) {
+               back_to_cached();
+               return 0;
        }
 
+       /*
+        * Insert PMB entries for the P1 and P2 areas so that, after
+        * we've switched the MMU to 32-bit mode, the semantics of P1
+        * and P2 are the same as in 29-bit mode, e.g.
+        *
+        *      P1 - provides a cached window onto physical memory
+        *      P2 - provides an uncached window onto physical memory
+        */
+       size = (unsigned long)__MEMORY_START + __MEMORY_SIZE;
+
+       ret = pmb_remap(P1SEG, 0x00000000, size, PMB_C);
+       BUG_ON(ret != size);
+
+       ret = pmb_remap(P2SEG, 0x00000000, size, PMB_WT | PMB_UB);
+       BUG_ON(ret != size);
+
+       ctrl_outl(0, PMB_IRMCR);
+
+       /* PMB.SE and UB[7] */
+       ctrl_outl(PASCR_SE | (1 << 7), PMB_PASCR);
+
+       /* Flush out the TLB */
+       i =  ctrl_inl(MMUCR);
+       i |= MMUCR_TI;
+       ctrl_outl(i, MMUCR);
+
        back_to_cached();
 
        return 0;
 }
-#endif /* CONFIG_PMB */
 
 static int pmb_seq_show(struct seq_file *file, void *iter)
 {
@@ -462,6 +479,5 @@ static int __init pmb_sysdev_init(void)
 {
        return sysdev_driver_register(&cpu_sysdev_class, &pmb_sysdev_driver);
 }
-
 subsys_initcall(pmb_sysdev_init);
 #endif