[U-Boot] [PATCH v3 0/4] arm: mvebu: Allow DB-88F6820-GP to boot Linux

[U-Boot] [PATCH v3 0/4] arm: mvebu: Allow DB-88F6820-GP to boot Linux

[Kevin Smith]

Changes needed to allow the DB-88F6820-GP development board to successfully
boot a Linux kernel.

Changes in v3:
- Correct coding-style errors
- Correct config macro style error
- Add acked-by lines

Kevin Smith (3):
  arm: mvebu: Update CBAR with SOC regs base
  mv-common.h: Include support for device trees
  db-88f6820-gp.h: Load data blobs into lower memory

Stefan Roese (1):
  arm: mvebu: Disable L2 cache before enabling d-cache

 arch/arm/mach-mvebu/cpu.c       | 15 +++++++++++++++
 include/configs/db-88f6820-gp.h |  5 +++++
 include/configs/mv-common.h     |  2 ++
 3 files changed, 22 insertions(+)

-- 
2.3.2

[U-Boot] [PATCH v3 1/4] arm: mvebu: Disable L2 cache before enabling d-cache

[Kevin Smith]

From: Stefan Roese <sr@denx.de>

L2 cache may still be enabled by the BootROM. We need to first disable
it before enabling d-cache support.

Signed-off-by: Stefan Roese <sr@denx.de>
Tested-by: Kevin Smith <kevin.smith@elecsyscorp.com>
---
 arch/arm/mach-mvebu/cpu.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm/mach-mvebu/cpu.c b/arch/arm/mach-mvebu/cpu.c
index 04681fc..417fc35 100644
--- a/arch/arm/mach-mvebu/cpu.c
+++ b/arch/arm/mach-mvebu/cpu.c
@@ -7,6 +7,7 @@
 #include <common.h>
 #include <netdev.h>
 #include <asm/io.h>
+#include <asm/pl310.h>
 #include <asm/arch/cpu.h>
 #include <asm/arch/soc.h>
 
@@ -240,6 +241,13 @@ int cpu_eth_init(bd_t *bis)
 #ifndef CONFIG_SYS_DCACHE_OFF
 void enable_caches(void)
 {
+	struct pl310_regs *const pl310 =
+		(struct pl310_regs *)CONFIG_SYS_PL310_BASE;
+
+	/* First disable L2 cache - may still be enable from BootROM */
+	if (mvebu_soc_family() == MVEBU_SOC_A38X)
+		clrbits_le32(&pl310->pl310_ctrl, L2X0_CTRL_EN);
+
 	/* Avoid problem with e.g. neta ethernet driver */
 	invalidate_dcache_all();
 
-- 
2.3.2

[U-Boot] [PATCH v3 2/4] arm: mvebu: Update CBAR with SOC regs base

[Kevin Smith]

SMP-enabled Linux kernels read the CBAR register in CP15 to find
the address of the SCU registers.  After remapping internal
registers, also update the CBAR so the kernel can find them.

Signed-off-by: Kevin Smith <kevin.smith@elecsyscorp.com>
Acked-by: Stefan Roese <sr@denx.de>
---
 arch/arm/mach-mvebu/cpu.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/arch/arm/mach-mvebu/cpu.c b/arch/arm/mach-mvebu/cpu.c
index 417fc35..0121db8 100644
--- a/arch/arm/mach-mvebu/cpu.c
+++ b/arch/arm/mach-mvebu/cpu.c
@@ -161,10 +161,17 @@ static void update_sdram_window_sizes(void)
 }
 
 #ifdef CONFIG_ARCH_CPU_INIT
+static void set_cbar(u32 addr)
+{
+	asm("mcr p15, 4, %0, c15, c0" : : "r" (addr));
+}
+
+
 int arch_cpu_init(void)
 {
 	/* Linux expects the internal registers to be at 0xf1000000 */
 	writel(SOC_REGS_PHY_BASE, INTREG_BASE_ADDR_REG);
+	set_cbar(SOC_REGS_PHY_BASE + 0xC000);
 
 	/*
 	 * We need to call mvebu_mbus_probe() before calling
-- 
2.3.2

[U-Boot] [PATCH v3 3/4] mv-common.h: Include support for device trees

[Kevin Smith]

Signed-off-by: Kevin Smith <kevin.smith@elecsyscorp.com>
Acked-by: Stefan Roese <sr@denx.de>
---
 include/configs/mv-common.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h
index 51436da..4039b71 100644
--- a/include/configs/mv-common.h
+++ b/include/configs/mv-common.h
@@ -59,6 +59,8 @@
 #define CONFIG_BOOTDELAY	3	/* default enable autoboot */
 #define CONFIG_PREBOOT
 
+#define CONFIG_OF_LIBFDT		/* Device tree support */
+
 /*
  * For booting Linux, the board info and command line data
  * have to be in the first 8 MB of memory, since this is
-- 
2.3.2

[U-Boot] [PATCH v3 3/4] mv-common.h: Include support for device trees

[Stefan Roese]

Hi Luka,

On 18.05.2015 18:09, Kevin Smith wrote:
> Signed-off-by: Kevin Smith <kevin.smith@elecsyscorp.com>
> Acked-by: Stefan Roese <sr@denx.de>
> ---
>   include/configs/mv-common.h | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h
> index 51436da..4039b71 100644
> --- a/include/configs/mv-common.h
> +++ b/include/configs/mv-common.h
> @@ -59,6 +59,8 @@
>   #define CONFIG_BOOTDELAY	3	/* default enable autoboot */
>   #define CONFIG_PREBOOT
>
> +#define CONFIG_OF_LIBFDT		/* Device tree support */
> +
>   /*
>    * For booting Linux, the board info and command line data
>    * have to be in the first 8 MB of memory, since this is
>

I just noticed that you didn't add this patch (and "[PATCH v3 4/4] 
db-88f6820-gp.h: Load data blobs into lower memory") in your last pull 
request. Could you please queue those 2 patches for your next pull request?

Thanks,
Stefan

[U-Boot] [PATCH v3 3/4] mv-common.h: Include support for device trees

[Luka Perkov]

Hi Stefan,

On Fri, Jun 26, 2015 at 01:56:24PM +0200, Stefan Roese wrote:
> On 18.05.2015 18:09, Kevin Smith wrote:
> >Signed-off-by: Kevin Smith <kevin.smith@elecsyscorp.com>
> >Acked-by: Stefan Roese <sr@denx.de>
> >---
> >  include/configs/mv-common.h | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> >diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h
> >index 51436da..4039b71 100644
> >--- a/include/configs/mv-common.h
> >+++ b/include/configs/mv-common.h
> >@@ -59,6 +59,8 @@
> >  #define CONFIG_BOOTDELAY	3	/* default enable autoboot */
> >  #define CONFIG_PREBOOT
> >
> >+#define CONFIG_OF_LIBFDT		/* Device tree support */
> >+
> >  /*
> >   * For booting Linux, the board info and command line data
> >   * have to be in the first 8 MB of memory, since this is
> >
> 
> I just noticed that you didn't add this patch (and "[PATCH v3 4/4]
> db-88f6820-gp.h: Load data blobs into lower memory") in your last pull
> request. Could you please queue those 2 patches for your next pull request?

I just sent a pull request for those. I must have missed them for some
reason. Thank you for the pointer.

When this gets merged we can start dropping CONFIG_OF_LIBFDT from
Marvell boards which include mv-common.h.

Regards,
Luka

[U-Boot] [PATCH v3 4/4] db-88f6820-gp.h: Load data blobs into lower memory

[Kevin Smith]

By default on this platform, u-boot loads data into high memory
in the range of 0x7Fxxxxxx, which generates a data abort when the
kernel tries to read it.  Config the u-boot environment to load
the device tree and initrd image into lower memory to make them
accessible to the kernel.

Signed-off-by: Kevin Smith <kevin.smith@elecsyscorp.com>
Acked-by: Stefan Roese <sr@denx.de>
---
 include/configs/db-88f6820-gp.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/include/configs/db-88f6820-gp.h b/include/configs/db-88f6820-gp.h
index 12a24ce..95a05e6 100644
--- a/include/configs/db-88f6820-gp.h
+++ b/include/configs/db-88f6820-gp.h
@@ -63,6 +63,11 @@
 #define CONFIG_SYS_CONSOLE_INFO_QUIET	/* don't print console @ startup */
 #define CONFIG_SYS_ALT_MEMTEST
 
+/* Keep device tree and initrd in lower memory so the kernel can access them */
+#define CONFIG_EXTRA_ENV_SETTINGS	\
+	"fdt_high=0x10000000\0"		\
+	"initrd_high=0x10000000\0"
+
 /*
  * mv-common.h should be defined after CMD configs since it used them
  * to enable certain macros
-- 
2.3.2