diff mbox

[U-Boot,v2,05/10] MIPS: qemu-malta: setup GT64120 registers as done by YAMON

Message ID 1359821166-32352-6-git-send-email-juhosg@openwrt.org
State Superseded
Delegated to: Daniel Schwierzeck
Headers show

Commit Message

Gabor Juhos Feb. 2, 2013, 4:06 p.m. UTC
Move the GT64120 register base to 0x1be00000
and setup PCI BAR registers as done by the
original YAMON bootloader.

This is needed for running Linux kernel.

Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
Cc: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
---
Changes since v1:
 - rebased against mips/testing

Changes since RFC: ---
---
 arch/mips/include/asm/malta.h    |    4 ++-
 board/qemu-malta/lowlevel_init.S |   51 ++++++++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 1 deletion(-)

Comments

Daniel Schwierzeck Feb. 2, 2013, 7:39 p.m. UTC | #1
2013/2/2 Gabor Juhos <juhosg@openwrt.org>:
> Move the GT64120 register base to 0x1be00000
> and setup PCI BAR registers as done by the
> original YAMON bootloader.
>
> This is needed for running Linux kernel.
>
> Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
> ---
> Changes since v1:
>  - rebased against mips/testing
>
> Changes since RFC: ---
> ---
>  arch/mips/include/asm/malta.h    |    4 ++-
>  board/qemu-malta/lowlevel_init.S |   51 ++++++++++++++++++++++++++++++++++++++
>  2 files changed, 54 insertions(+), 1 deletion(-)
>
> diff --git a/arch/mips/include/asm/malta.h b/arch/mips/include/asm/malta.h
> index ab951e6..d4d44a2 100644
> --- a/arch/mips/include/asm/malta.h
> +++ b/arch/mips/include/asm/malta.h
> @@ -9,10 +9,12 @@
>  #ifndef _MIPS_ASM_MALTA_H
>  #define _MIPS_ASM_MALTA_H
>
> -#define MALTA_IO_PORT_BASE     0x10000000
> +#define MALTA_IO_PORT_BASE     0x18000000
>
>  #define MALTA_UART_BASE                (MALTA_IO_PORT_BASE + 0x3f8)
>
> +#define MALTA_GT_BASE          0x1be00000
> +
>  #define MALTA_RESET_BASE       0x1f000500
>  #define GORESET                        0x42
>
> diff --git a/board/qemu-malta/lowlevel_init.S b/board/qemu-malta/lowlevel_init.S
> index c5c5bd9..11544a8 100644
> --- a/board/qemu-malta/lowlevel_init.S
> +++ b/board/qemu-malta/lowlevel_init.S
> @@ -6,7 +6,19 @@
>   * by the Free Software Foundation.
>   */
>
> +#include <config.h>
> +#include <asm/addrspace.h>
>  #include <asm/regdef.h>
> +#include <asm/gt64120.h>
> +#include <asm/malta.h>
> +
> +#ifdef CONFIG_SYS_BIG_ENDIAN
> +#define CPU_TO_GT32(_x)                ((_x))
> +#else
> +#define CPU_TO_GT32(_x) (                                      \
> +       (((_x) & 0xff) << 24) | (((_x) & 0xff00) << 8) |        \
> +       (((_x) & 0xff0000) >> 8) | (((_x) & 0xff000000) >> 24))
> +#endif
>
>         .text
>         .set noreorder
> @@ -15,5 +27,44 @@
>         .globl  lowlevel_init
>  lowlevel_init:
>
> +       /*
> +        * Load BAR registers of GT64120 as done by YAMON
> +        *
> +        * based on a patch sent by Antony Pavlov <antonynpavlov@gmail.com>
> +        * to the barebox mailing list.
> +        * The subject of the original patch:
> +        *   'MIPS: qemu-malta: add YAMON-style GT64120 memory map'
> +        * URL:
> +        * http://www.mail-archive.com/barebox@lists.infradead.org/msg06128.html
> +        *
> +        * based on write_bootloader() in qemu.git/hw/mips_malta.c
> +        * see GT64120 manual and qemu.git/hw/gt64xxx.c for details
> +        */
> +
> +       /* move GT64120 registers from 0x14000000 to 0x1be00000 */
> +       li      t1, KSEG1ADDR(GT_DEF_BASE)
> +       li      t0, CPU_TO_GT32(0xdf000000)
> +       sw      t0, GT_ISD_OFS(t1)
> +
> +       /* setup MEM-to-PCI0 mapping */
> +       li      t1, KSEG1ADDR(MALTA_GT_BASE)
> +
> +       /* setup PCI0 io window to 0x18000000-0x181fffff */
> +       li      t0, CPU_TO_GT32(0xc0000000)
> +       sw      t0, GT_PCI0IOLD_OFS(t1)
> +       li      t0, CPU_TO_GT32(0x40000000)
> +       sw      t0, GT_PCI0IOHD_OFS(t1)
> +
> +       /* setup PCI0 mem windows */
> +       li      t0, CPU_TO_GT32(0x80000000)
> +       sw      t0, GT_PCI0M0LD_OFS(t1)
> +       li      t0, CPU_TO_GT32(0x3f000000)
> +       sw      t0, GT_PCI0M0HD_OFS(t1)
> +
> +       li      t0, CPU_TO_GT32(0xc1000000)
> +       sw      t0, GT_PCI0M1LD_OFS(t1)
> +       li      t0, CPU_TO_GT32(0x5e000000)
> +       sw      t0, GT_PCI0M1HD_OFS(t1)
> +
>         jr      ra
>          nop
> --
> 1.7.10
>

is this initialization really needed before relocation? If not this should be
moved to pci_init_board()
Gabor Juhos Feb. 3, 2013, 8:15 a.m. UTC | #2
2013.02.02. 20:39 keltezéssel, Daniel Schwierzeck írta:
> 2013/2/2 Gabor Juhos <juhosg@openwrt.org>:
>> Move the GT64120 register base to 0x1be00000
>> and setup PCI BAR registers as done by the
>> original YAMON bootloader.
>>
>> This is needed for running Linux kernel.
>>
>> Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
>> Cc: Daniel Schwierzeck <daniel.schwierzeck@googlemail.com>
>> ---
>> Changes since v1:
>>  - rebased against mips/testing
>>
>> Changes since RFC: ---
>> ---
>>  arch/mips/include/asm/malta.h    |    4 ++-
>>  board/qemu-malta/lowlevel_init.S |   51 ++++++++++++++++++++++++++++++++++++++
>>  2 files changed, 54 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/mips/include/asm/malta.h b/arch/mips/include/asm/malta.h
>> index ab951e6..d4d44a2 100644
>> --- a/arch/mips/include/asm/malta.h
>> +++ b/arch/mips/include/asm/malta.h
>> @@ -9,10 +9,12 @@
>>  #ifndef _MIPS_ASM_MALTA_H
>>  #define _MIPS_ASM_MALTA_H
>>
>> -#define MALTA_IO_PORT_BASE     0x10000000
>> +#define MALTA_IO_PORT_BASE     0x18000000
>>
>>  #define MALTA_UART_BASE                (MALTA_IO_PORT_BASE + 0x3f8)
>>
>> +#define MALTA_GT_BASE          0x1be00000
>> +
>>  #define MALTA_RESET_BASE       0x1f000500
>>  #define GORESET                        0x42
>>
>> diff --git a/board/qemu-malta/lowlevel_init.S b/board/qemu-malta/lowlevel_init.S
>> index c5c5bd9..11544a8 100644
>> --- a/board/qemu-malta/lowlevel_init.S
>> +++ b/board/qemu-malta/lowlevel_init.S
>> @@ -6,7 +6,19 @@
>>   * by the Free Software Foundation.
>>   */
>>
>> +#include <config.h>
>> +#include <asm/addrspace.h>
>>  #include <asm/regdef.h>
>> +#include <asm/gt64120.h>
>> +#include <asm/malta.h>
>> +
>> +#ifdef CONFIG_SYS_BIG_ENDIAN
>> +#define CPU_TO_GT32(_x)                ((_x))
>> +#else
>> +#define CPU_TO_GT32(_x) (                                      \
>> +       (((_x) & 0xff) << 24) | (((_x) & 0xff00) << 8) |        \
>> +       (((_x) & 0xff0000) >> 8) | (((_x) & 0xff000000) >> 24))
>> +#endif
>>
>>         .text
>>         .set noreorder
>> @@ -15,5 +27,44 @@
>>         .globl  lowlevel_init
>>  lowlevel_init:
>>
>> +       /*
>> +        * Load BAR registers of GT64120 as done by YAMON
>> +        *
>> +        * based on a patch sent by Antony Pavlov <antonynpavlov@gmail.com>
>> +        * to the barebox mailing list.
>> +        * The subject of the original patch:
>> +        *   'MIPS: qemu-malta: add YAMON-style GT64120 memory map'
>> +        * URL:
>> +        * http://www.mail-archive.com/barebox@lists.infradead.org/msg06128.html
>> +        *
>> +        * based on write_bootloader() in qemu.git/hw/mips_malta.c
>> +        * see GT64120 manual and qemu.git/hw/gt64xxx.c for details
>> +        */
>> +
>> +       /* move GT64120 registers from 0x14000000 to 0x1be00000 */
>> +       li      t1, KSEG1ADDR(GT_DEF_BASE)
>> +       li      t0, CPU_TO_GT32(0xdf000000)
>> +       sw      t0, GT_ISD_OFS(t1)
>> +
>> +       /* setup MEM-to-PCI0 mapping */
>> +       li      t1, KSEG1ADDR(MALTA_GT_BASE)
>> +
>> +       /* setup PCI0 io window to 0x18000000-0x181fffff */
>> +       li      t0, CPU_TO_GT32(0xc0000000)
>> +       sw      t0, GT_PCI0IOLD_OFS(t1)
>> +       li      t0, CPU_TO_GT32(0x40000000)
>> +       sw      t0, GT_PCI0IOHD_OFS(t1)
>> +
>> +       /* setup PCI0 mem windows */
>> +       li      t0, CPU_TO_GT32(0x80000000)
>> +       sw      t0, GT_PCI0M0LD_OFS(t1)
>> +       li      t0, CPU_TO_GT32(0x3f000000)
>> +       sw      t0, GT_PCI0M0HD_OFS(t1)
>> +
>> +       li      t0, CPU_TO_GT32(0xc1000000)
>> +       sw      t0, GT_PCI0M1LD_OFS(t1)
>> +       li      t0, CPU_TO_GT32(0x5e000000)
>> +       sw      t0, GT_PCI0M1HD_OFS(t1)
>> +
>>         jr      ra
>>          nop
>> --
>> 1.7.10
>>
> 
> is this initialization really needed before relocation? If not this should be
> moved to pci_init_board()

After this initialization, the UART uses a different base address. If we want
UART output before the relocation then we have to initialize it here.

-Gabor
diff mbox

Patch

diff --git a/arch/mips/include/asm/malta.h b/arch/mips/include/asm/malta.h
index ab951e6..d4d44a2 100644
--- a/arch/mips/include/asm/malta.h
+++ b/arch/mips/include/asm/malta.h
@@ -9,10 +9,12 @@ 
 #ifndef _MIPS_ASM_MALTA_H
 #define _MIPS_ASM_MALTA_H
 
-#define MALTA_IO_PORT_BASE	0x10000000
+#define MALTA_IO_PORT_BASE	0x18000000
 
 #define MALTA_UART_BASE		(MALTA_IO_PORT_BASE + 0x3f8)
 
+#define MALTA_GT_BASE		0x1be00000
+
 #define MALTA_RESET_BASE	0x1f000500
 #define GORESET			0x42
 
diff --git a/board/qemu-malta/lowlevel_init.S b/board/qemu-malta/lowlevel_init.S
index c5c5bd9..11544a8 100644
--- a/board/qemu-malta/lowlevel_init.S
+++ b/board/qemu-malta/lowlevel_init.S
@@ -6,7 +6,19 @@ 
  * by the Free Software Foundation.
  */
 
+#include <config.h>
+#include <asm/addrspace.h>
 #include <asm/regdef.h>
+#include <asm/gt64120.h>
+#include <asm/malta.h>
+
+#ifdef CONFIG_SYS_BIG_ENDIAN
+#define CPU_TO_GT32(_x)		((_x))
+#else
+#define CPU_TO_GT32(_x) (					\
+	(((_x) & 0xff) << 24) | (((_x) & 0xff00) << 8) |	\
+	(((_x) & 0xff0000) >> 8) | (((_x) & 0xff000000) >> 24))
+#endif
 
 	.text
 	.set noreorder
@@ -15,5 +27,44 @@ 
 	.globl	lowlevel_init
 lowlevel_init:
 
+	/*
+	 * Load BAR registers of GT64120 as done by YAMON
+	 *
+	 * based on a patch sent by Antony Pavlov <antonynpavlov@gmail.com>
+	 * to the barebox mailing list.
+	 * The subject of the original patch:
+	 *   'MIPS: qemu-malta: add YAMON-style GT64120 memory map'
+	 * URL:
+	 * http://www.mail-archive.com/barebox@lists.infradead.org/msg06128.html
+	 *
+	 * based on write_bootloader() in qemu.git/hw/mips_malta.c
+	 * see GT64120 manual and qemu.git/hw/gt64xxx.c for details
+	 */
+
+	/* move GT64120 registers from 0x14000000 to 0x1be00000 */
+	li	t1, KSEG1ADDR(GT_DEF_BASE)
+	li	t0, CPU_TO_GT32(0xdf000000)
+	sw	t0, GT_ISD_OFS(t1)
+
+	/* setup MEM-to-PCI0 mapping */
+	li	t1, KSEG1ADDR(MALTA_GT_BASE)
+
+	/* setup PCI0 io window to 0x18000000-0x181fffff */
+	li	t0, CPU_TO_GT32(0xc0000000)
+	sw	t0, GT_PCI0IOLD_OFS(t1)
+	li	t0, CPU_TO_GT32(0x40000000)
+	sw	t0, GT_PCI0IOHD_OFS(t1)
+
+	/* setup PCI0 mem windows */
+	li	t0, CPU_TO_GT32(0x80000000)
+	sw	t0, GT_PCI0M0LD_OFS(t1)
+	li	t0, CPU_TO_GT32(0x3f000000)
+	sw	t0, GT_PCI0M0HD_OFS(t1)
+
+	li	t0, CPU_TO_GT32(0xc1000000)
+	sw	t0, GT_PCI0M1LD_OFS(t1)
+	li	t0, CPU_TO_GT32(0x5e000000)
+	sw	t0, GT_PCI0M1HD_OFS(t1)
+
 	jr	ra
 	 nop