Skip to contentSkip to conversation

PowerPC 440EPx: Sequoia bootwrapper

PATCH In review Valentine Barshak 4 participants 9 messages Jul 30, 2007 – Aug 3, 2007 PowerPC

lore
Share & tools
lore

Valentine Barshak

author
#0 lore raw

PATCH 4/6 PowerPC 440EPx: Sequoia bootwrapper

A bootwrapper code for Sequoia.

Signed-off-by Valentine Barshak <[email protected]>
---
 arch/powerpc/boot/44x.h              |    1 
 arch/powerpc/boot/Makefile           |    6 
 arch/powerpc/boot/cuboot-sequoia.c   |   31 ++++
 arch/powerpc/boot/sequoia.c          |  234 +++++++++++++++++++++++++++++++++++
 arch/powerpc/boot/treeboot-sequoia.c |   27 ++++
 5 files changed, 297 insertions(+), 2 deletions(-)
arch/powerpc/boot/44x.h +1 -0
diff -ruN linux.orig/arch/powerpc/boot/44x.h linux/arch/powerpc/boot/44x.h
@@ -15,5 +15,6 @@
 
 void ibm44x_dbcr_reset(void);
 void ebony_init(void *mac0, void *mac1);
+void sequoia_init(void *mac0, void *mac1);
 
 #endif /* _PPC_BOOT_44X_H_ */
arch/powerpc/boot/cuboot-sequoia.c +31 -0
diff -ruN linux.orig/arch/powerpc/boot/cuboot-sequoia.c linux/arch/powerpc/boot/cuboot-sequoia.c
@@ -0,0 +1,31 @@
+/*
+ * Old U-boot compatibility for Sequoia
+ *
+ * Based on Ebony code by David Gibson <[email protected]>
+ *
+ * Copyright 2007 David Gibson, IBM Corporatio.
+ *   Based on cuboot-83xx.c, which is:
+ * Copyright (c) 2007 Freescale Semiconductor, Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published
+ * by the Free Software Foundation.
+ */
+
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+#include "cuboot.h"
+
+#define TARGET_4xx
+#define TARGET_44x
+#include "ppcboot.h"
+
+static bd_t bd;
+
+void platform_init(unsigned long r3, unsigned long r4, unsigned long r5,
+                   unsigned long r6, unsigned long r7)
+{
+	CUBOOT_INIT();
+	sequoia_init(&bd.bi_enetaddr, &bd.bi_enet1addr);
+}
arch/powerpc/boot/Makefile +4 -2
diff -ruN linux.orig/arch/powerpc/boot/Makefile linux/arch/powerpc/boot/Makefile
@@ -44,10 +44,11 @@
 src-wlib := string.S crt0.S stdio.c main.c flatdevtree.c flatdevtree_misc.c \
 		ns16550.c serial.c simple_alloc.c div64.S util.S \
 		gunzip_util.c elf_util.c $(zlib) devtree.c oflib.c ofconsole.c \
-		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c
+		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c \
+		sequoia.c
 src-plat := of.c cuboot-83xx.c cuboot-85xx.c holly.c \
 		cuboot-ebony.c treeboot-ebony.c prpmc2800.c \
-		ps3-head.S ps3-hvcall.S ps3.c
+		ps3-head.S ps3-hvcall.S ps3.c treeboot-sequoia.c cuboot-sequoia.c
 src-boot := $(src-wlib) $(src-plat) empty.c
 
 src-boot := $(addprefix $(obj)/, $(src-boot))

@@ -142,6 +143,7 @@
 image-$(CONFIG_PPC_83xx)		+= cuImage.83xx
 image-$(CONFIG_PPC_85xx)		+= cuImage.85xx
 image-$(CONFIG_EBONY)			+= treeImage.ebony cuImage.ebony
+image-$(CONFIG_SEQUOIA)			+= treeImage.sequoia cuImage.sequoia
 endif
 
 # For 32-bit powermacs, build the COFF and miboot images
arch/powerpc/boot/sequoia.c +234 -0

234 lines not on this page · raw message

arch/powerpc/boot/treeboot-sequoia.c +27 -0
diff -ruN linux.orig/arch/powerpc/boot/treeboot-sequoia.c linux/arch/powerpc/boot/treeboot-sequoia.c
@@ -0,0 +1,27 @@
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+
+extern char _end[];
+
+BSS_STACK(4096);
+
+void platform_init(void)
+{
+	unsigned long end_of_ram = 0x10000000;
+	unsigned long avail_ram = end_of_ram - (unsigned long)_end;
+
+	simple_alloc_init(_end, avail_ram, 32, 64);
+	sequoia_init(NULL, NULL);
+}
#1 lore raw
Quoted 10 lines · Valentine Barshak

A bootwrapper code for Sequoia.

Signed-off-by Valentine Barshak <[email protected]>
---
 arch/powerpc/boot/44x.h              |    1
 arch/powerpc/boot/Makefile           |    6
 arch/powerpc/boot/cuboot-sequoia.c   |   31 ++++
 arch/powerpc/boot/sequoia.c          |  234
 +++++++++++++++++++++++++++++++++++ 
 arch/powerpc/boot/treeboot-sequoia.c |   27 ++++

Is treeboot-* really needed on Sequoia? From my understanding this is only needed for platforms using OpenBIOS as bootloader.

Quoted 108 lines
 5 files changed, 297 insertions(+), 2 deletions(-)

diff -ruN linux.orig/arch/powerpc/boot/44x.h linux/arch/powerpc/boot/44x.h

arch/powerpc/boot/44x.h +1 -0
@@ -15,5 +15,6 @@
 
 void ibm44x_dbcr_reset(void);
 void ebony_init(void *mac0, void *mac1);
+void sequoia_init(void *mac0, void *mac1);
 
 #endif /* _PPC_BOOT_44X_H_ */

diff -ruN linux.orig/arch/powerpc/boot/cuboot-sequoia.c linux/arch/powerpc/boot/cuboot-sequoia.c ---
linux.orig/arch/powerpc/boot/cuboot-sequoia.c 1970-01-01 03:00:00.000000000
+0300 +++ linux/arch/powerpc/boot/cuboot-sequoia.c 2007-07-27
20:44:26.000000000 +0400 @@ -0,0 +1,31 @@
+/*
+ * Old U-boot compatibility for Sequoia
+ *
+ * Based on Ebony code by David Gibson <[email protected]>
+ *
+ * Copyright 2007 David Gibson, IBM Corporatio.
+ * Based on cuboot-83xx.c, which is:
+ * Copyright (c) 2007 Freescale Semiconductor, Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published + * by the Free Software Foundation.
+ */

arch/powerpc/boot/44x.h +18 -0
+
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+#include "cuboot.h"
+
+#define TARGET_4xx
+#define TARGET_44x
+#include "ppcboot.h"
+
+static bd_t bd;
+
+void platform_init(unsigned long r3, unsigned long r4, unsigned long r5,
+                   unsigned long r6, unsigned long r7)
+{
+	CUBOOT_INIT();
+	sequoia_init(&bd.bi_enetaddr, &bd.bi_enet1addr);
+}

diff -ruN linux.orig/arch/powerpc/boot/Makefile linux/arch/powerpc/boot/Makefile ---
linux.orig/arch/powerpc/boot/Makefile 2007-07-27 20:37:10.000000000 +0400
+++ linux/arch/powerpc/boot/Makefile 2007-07-27 20:44:26.000000000 +0400 @@
-44,10 +44,11 @@
src-wlib := string.S crt0.S stdio.c main.c flatdevtree.c
flatdevtree_misc.c \ ns16550.c serial.c simple_alloc.c div64.S util.S \

arch/powerpc/boot/44x.h +4 -2
 		gunzip_util.c elf_util.c $(zlib) devtree.c oflib.c ofconsole.c \
-		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c
+		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c \
+		sequoia.c
 src-plat := of.c cuboot-83xx.c cuboot-85xx.c holly.c \
 		cuboot-ebony.c treeboot-ebony.c prpmc2800.c \
-		ps3-head.S ps3-hvcall.S ps3.c
+		ps3-head.S ps3-hvcall.S ps3.c treeboot-sequoia.c cuboot-sequoia.c
 src-boot := $(src-wlib) $(src-plat) empty.c
 
 src-boot := $(addprefix $(obj)/, $(src-boot))

@@ -142,6 +143,7 @@
 image-$(CONFIG_PPC_83xx)		+= cuImage.83xx
 image-$(CONFIG_PPC_85xx)		+= cuImage.85xx
 image-$(CONFIG_EBONY)			+= treeImage.ebony cuImage.ebony
+image-$(CONFIG_SEQUOIA)			+= treeImage.sequoia cuImage.sequoia
 endif
 
 # For 32-bit powermacs, build the COFF and miboot images

diff -ruN linux.orig/arch/powerpc/boot/sequoia.c
linux/arch/powerpc/boot/sequoia.c ---
linux.orig/arch/powerpc/boot/sequoia.c 1970-01-01 03:00:00.000000000 +0300
+++ linux/arch/powerpc/boot/sequoia.c 2007-07-27 20:59:09.000000000 +0400

arch/powerpc/boot/44x.h +29 -0
@@ -0,0 +1,234 @@
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * Clocking code based on code by:
+ * Stefan Roese <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include <stdarg.h>
+#include <stddef.h>
+#include "types.h"
+#include "elf.h"
+#include "string.h"
+#include "stdio.h"
+#include "page.h"
+#include "ops.h"
+#include "dcr.h"
+#include "44x.h"
+
+extern char _dtb_start[];
+extern char _dtb_end[];
+
+static u8 *sequoia_mac0, *sequoia_mac1;

arch/powerpc/boot/44x.h line 39

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+	u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+	u32 reg;
+	u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+	reg = mfdcr(DCRN_CPR0_DATA);

How about adding mfcpr functions to access the CPR regs. Makes the source better readable:

static inline u32 mfcpr(u32 reg)

{
	mtdcr(DCRN_CPR0_ADDR, reg);
	return mfdcr(DCRN_CPR0_DATA);
}
Quoted 106 lines
arch/powerpc/boot/44x.h +106 -0
+	tmp = (reg & 0x000F0000) >> 16;
+	fwdva = tmp ? tmp : 16;
+	tmp = (reg & 0x00000700) >> 8;
+	fwdvb = tmp ? tmp : 8;
+	tmp = (reg & 0x1F000000) >> 24;
+	fbdv = tmp ? tmp : 32;
+	lfbdv = (reg & 0x0000007F);
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	opbdv0 = tmp ? tmp : 4;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	perdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	prbdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	spcid0 = tmp ? tmp : 4;
+
+	/* Calculate M */
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	if (tmp == 0) { /* PLL output */
+		tmp = (reg & 0x20000000) >> 29;
+		if (!tmp) /* PLLOUTA */
+			m = fbdv * lfbdv * fwdva;
+		else
+			m = fbdv * lfbdv * fwdvb;
+	}
+	else if (tmp == 1) /* CPU output */
+		m = fbdv * fwdva;
+	else
+		m = perdv0 * opbdv0 * fwdvb;
+
+	vco = (m * sysclk) + (m >> 1);
+	cpu = vco / fwdva;
+	plb = vco / fwdvb / prbdv0;
+	opb = plb / opbdv0;
+	ebc = plb / perdv0;
+
+	/* FIXME */
+	uart0 = ser_clk;
+
+	/* Figure out timebase.  Either CPU or default TmrClk */
+	asm volatile (
+			"mfspr	%0,%1\n"
+			:
+			"=&r"(reg) : "i"(SPRN_CCR1));
+	if (reg & 0x0080)
+		tb = 25000000; /* TmrClk is 25MHz */
+	else
+		tb = cpu;
+
+	dt_fixup_cpu_clocks(cpu, tb, 0);
+	dt_fixup_clock("/plb", plb);
+	dt_fixup_clock("/plb/opb", opb);
+	dt_fixup_clock("/plb/opb/ebc", ebc);
+	dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}
+
+
+/*
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START		0x1
+#define DDR_START_SHIFT		0
+#define DDR_MAX_CS_REG		0x3
+#define DDR_MAX_CS_REG_SHIFT	24
+#define DDR_MAX_COL_REG		0xf
+#define DDR_MAX_COL_REG_SHIFT	16
+#define DDR_MAX_ROW_REG		0xf
+#define DDR_MAX_ROW_REG_SHIFT	8
+/* DDR0_08 */
+#define DDR_DDR2_MODE		0x1
+#define DDR_DDR2_MODE_SHIFT	0
+/* DDR0_10 */
+#define DDR_CS_MAP		0x3
+#define DDR_CS_MAP_SHIFT	8
+/* DDR0_14 */
+#define DDR_REDUC		0x1
+#define DDR_REDUC_SHIFT		16
+/* DDR0_42 */
+#define DDR_APIN		0x7
+#define DDR_APIN_SHIFT		24
+/* DDR0_43 */
+#define DDR_COL_SZ		0x7
+#define DDR_COL_SZ_SHIFT	8
+#define DDR_BANK8		0x1
+#define DDR_BANK8_SHIFT		0
+

arch/powerpc/boot/44x.h line 155

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+#define DDR_GET_VAL(val, mask, shift)	(((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)
+{
+	unsigned long val, max_cs, max_col, max_row;
+	unsigned long cs, col, row, bank, dpath;
+	unsigned long memsize;
+
+	mtdcr(DCRN_SDRAM0_CFGADDR, 2);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);

Again, mfsdram() functions would be handy here.

Quoted 54 lines
arch/powerpc/boot/44x.h +54 -0
+	if (!DDR_GET_VAL(val, DDR_START, DDR_START_SHIFT))
+		fatal("DDR controller is not initialized\n");
+
+	/* get maximum cs col and row values */
+	max_cs  = DDR_GET_VAL(val, DDR_MAX_CS_REG, DDR_MAX_CS_REG_SHIFT);
+	max_col = DDR_GET_VAL(val, DDR_MAX_COL_REG, DDR_MAX_COL_REG_SHIFT);
+	max_row = DDR_GET_VAL(val, DDR_MAX_ROW_REG, DDR_MAX_ROW_REG_SHIFT);
+
+	/* get CS value */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 10);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	val = DDR_GET_VAL(val, DDR_CS_MAP, DDR_CS_MAP_SHIFT);
+	cs = 0;
+	while (val) {
+		if (val && 0x1)
+			cs++;
+		val = val >> 1;
+	}
+
+	if (!cs)
+		fatal("No memory installed\n");
+	if (cs > max_cs)
+		fatal("DDR wrong CS configuration\n");
+
+	/* get data path bytes */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 14);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	if (DDR_GET_VAL(val, DDR_REDUC, DDR_REDUC_SHIFT))
+		dpath = 8; /* 64 bits */
+	else
+		dpath = 4; /* 32 bits */
+
+	/* get adress pins (rows) */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 42);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	row = DDR_GET_VAL(val, DDR_APIN, DDR_APIN_SHIFT);
+	if (row > max_row)
+		fatal("DDR wrong APIN configuration\n");
+	row = max_row - row;
+
+	/* get collomn size and banks */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 43);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	col = DDR_GET_VAL(val, DDR_COL_SZ, DDR_COL_SZ_SHIFT);
+	if (col > max_col)
+		fatal("DDR wrong COL configuration\n");
+	col = max_col - col;
+
+	if (DDR_GET_VAL(val, DDR_BANK8, DDR_BANK8_SHIFT))
+		bank = 8; /* 8 banks */

arch/powerpc/boot/44x.h line 219

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+	else
+		bank = 4; /* 4 banks */
+
+	memsize = cs * (1 << (col+row)) * bank * dpath;
+	dt_fixup_memory(0, memsize);
+}
+
+static void sequoia_fixups(void)
+{
+	unsigned long sysclk = 33333333;

Blank line after variable declaration please.

Quoted 48 lines
arch/powerpc/boot/44x.h +15 -0
+	ibm440ep_fixup_clocks(sysclk, 11059200);
+	ibm4xx_fixup_ebc_ranges("/plb/opb/ebc");
+	ibm440epx_fixup_memsize();
+	dt_fixup_mac_addresses(sequoia_mac0, sequoia_mac1);
+}
+
+void sequoia_init(void *mac0, void *mac1)
+{
+	platform_ops.fixups = sequoia_fixups;
+	platform_ops.exit = ibm44x_dbcr_reset;
+	sequoia_mac0 = mac0;
+	sequoia_mac1 = mac1;
+	ft_init(_dtb_start, 0, 32);
+	serial_console_init();
+}

diff -ruN linux.orig/arch/powerpc/boot/treeboot-sequoia.c linux/arch/powerpc/boot/treeboot-sequoia.c ---
linux.orig/arch/powerpc/boot/treeboot-sequoia.c 1970-01-01
03:00:00.000000000 +0300 +++
linux/arch/powerpc/boot/treeboot-sequoia.c 2007-07-27 20:44:26.000000000
+0400 @@ -0,0 +1,27 @@

arch/powerpc/boot/44x.h +27 -0
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+
+extern char _end[];
+
+BSS_STACK(4096);
+
+void platform_init(void)
+{
+	unsigned long end_of_ram = 0x10000000;
+	unsigned long avail_ram = end_of_ram - (unsigned long)_end;
+
+	simple_alloc_init(_end, avail_ram, 32, 64);
+	sequoia_init(NULL, NULL);
+}
Quoted 4 lines

_______________________________________________
Linuxppc-dev mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [email protected]
=====================================================================

author
#2 lore raw
Quoted 12 lines · Stefan Roese
Quoted 10 lines · Valentine Barshak

A bootwrapper code for Sequoia.

Signed-off-by Valentine Barshak <[email protected]>
---
 arch/powerpc/boot/44x.h              |    1
 arch/powerpc/boot/Makefile           |    6
 arch/powerpc/boot/cuboot-sequoia.c   |   31 ++++
 arch/powerpc/boot/sequoia.c          |  234
 +++++++++++++++++++++++++++++++++++ 
 arch/powerpc/boot/treeboot-sequoia.c |   27 ++++
Quoted 1 line

Is treeboot-* really needed on Sequoia? From my understanding this is only needed for platforms using OpenBIOS as bootloader.

Right, it's not used for u-boot. I've included it just in case.

Quoted 375 lines
Quoted 109 lines
 5 files changed, 297 insertions(+), 2 deletions(-)

diff -ruN linux.orig/arch/powerpc/boot/44x.h linux/arch/powerpc/boot/44x.h

arch/powerpc/boot/44x.h +1 -0
@@ -15,5 +15,6 @@
 
 void ibm44x_dbcr_reset(void);
 void ebony_init(void *mac0, void *mac1);
+void sequoia_init(void *mac0, void *mac1);
 
 #endif /* _PPC_BOOT_44X_H_ */

diff -ruN linux.orig/arch/powerpc/boot/cuboot-sequoia.c linux/arch/powerpc/boot/cuboot-sequoia.c ---
linux.orig/arch/powerpc/boot/cuboot-sequoia.c 1970-01-01 03:00:00.000000000
+0300 +++ linux/arch/powerpc/boot/cuboot-sequoia.c 2007-07-27
20:44:26.000000000 +0400 @@ -0,0 +1,31 @@
+/*
+ * Old U-boot compatibility for Sequoia
+ *
+ * Based on Ebony code by David Gibson <[email protected]>
+ *
+ * Copyright 2007 David Gibson, IBM Corporatio.
+ * Based on cuboot-83xx.c, which is:
+ * Copyright (c) 2007 Freescale Semiconductor, Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 as published + * by the Free Software Foundation.
+ */

arch/powerpc/boot/44x.h +18 -0
+
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+#include "cuboot.h"
+
+#define TARGET_4xx
+#define TARGET_44x
+#include "ppcboot.h"
+
+static bd_t bd;
+
+void platform_init(unsigned long r3, unsigned long r4, unsigned long r5,
+                   unsigned long r6, unsigned long r7)
+{
+	CUBOOT_INIT();
+	sequoia_init(&bd.bi_enetaddr, &bd.bi_enet1addr);
+}

diff -ruN linux.orig/arch/powerpc/boot/Makefile linux/arch/powerpc/boot/Makefile ---
linux.orig/arch/powerpc/boot/Makefile 2007-07-27 20:37:10.000000000 +0400
+++ linux/arch/powerpc/boot/Makefile 2007-07-27 20:44:26.000000000 +0400 @@
-44,10 +44,11 @@
src-wlib := string.S crt0.S stdio.c main.c flatdevtree.c
flatdevtree_misc.c \ ns16550.c serial.c simple_alloc.c div64.S util.S \

arch/powerpc/boot/44x.h +4 -2
 		gunzip_util.c elf_util.c $(zlib) devtree.c oflib.c ofconsole.c \
-		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c
+		44x.c ebony.c mv64x60.c mpsc.c mv64x60_i2c.c cuboot.c \
+		sequoia.c
 src-plat := of.c cuboot-83xx.c cuboot-85xx.c holly.c \
 		cuboot-ebony.c treeboot-ebony.c prpmc2800.c \
-		ps3-head.S ps3-hvcall.S ps3.c
+		ps3-head.S ps3-hvcall.S ps3.c treeboot-sequoia.c cuboot-sequoia.c
 src-boot := $(src-wlib) $(src-plat) empty.c
 
 src-boot := $(addprefix $(obj)/, $(src-boot))

@@ -142,6 +143,7 @@
 image-$(CONFIG_PPC_83xx)		+= cuImage.83xx
 image-$(CONFIG_PPC_85xx)		+= cuImage.85xx
 image-$(CONFIG_EBONY)			+= treeImage.ebony cuImage.ebony
+image-$(CONFIG_SEQUOIA)			+= treeImage.sequoia cuImage.sequoia
 endif
 
 # For 32-bit powermacs, build the COFF and miboot images

diff -ruN linux.orig/arch/powerpc/boot/sequoia.c
linux/arch/powerpc/boot/sequoia.c ---
linux.orig/arch/powerpc/boot/sequoia.c 1970-01-01 03:00:00.000000000 +0300
+++ linux/arch/powerpc/boot/sequoia.c 2007-07-27 20:59:09.000000000 +0400

arch/powerpc/boot/44x.h +29 -0
@@ -0,0 +1,234 @@
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * Clocking code based on code by:
+ * Stefan Roese <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include <stdarg.h>
+#include <stddef.h>
+#include "types.h"
+#include "elf.h"
+#include "string.h"
+#include "stdio.h"
+#include "page.h"
+#include "ops.h"
+#include "dcr.h"
+#include "44x.h"
+
+extern char _dtb_start[];
+extern char _dtb_end[];
+
+static u8 *sequoia_mac0, *sequoia_mac1;

arch/powerpc/boot/44x.h line 39

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+	u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+	u32 reg;
+	u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+	reg = mfdcr(DCRN_CPR0_DATA);

How about adding mfcpr functions to access the CPR regs. Makes the source better readable:

static inline u32 mfcpr(u32 reg)

{
	mtdcr(DCRN_CPR0_ADDR, reg);
	return mfdcr(DCRN_CPR0_DATA);
}
Quoted 106 lines
arch/powerpc/boot/44x.h +106 -0
+	tmp = (reg & 0x000F0000) >> 16;
+	fwdva = tmp ? tmp : 16;
+	tmp = (reg & 0x00000700) >> 8;
+	fwdvb = tmp ? tmp : 8;
+	tmp = (reg & 0x1F000000) >> 24;
+	fbdv = tmp ? tmp : 32;
+	lfbdv = (reg & 0x0000007F);
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	opbdv0 = tmp ? tmp : 4;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	perdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	prbdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	spcid0 = tmp ? tmp : 4;
+
+	/* Calculate M */
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	if (tmp == 0) { /* PLL output */
+		tmp = (reg & 0x20000000) >> 29;
+		if (!tmp) /* PLLOUTA */
+			m = fbdv * lfbdv * fwdva;
+		else
+			m = fbdv * lfbdv * fwdvb;
+	}
+	else if (tmp == 1) /* CPU output */
+		m = fbdv * fwdva;
+	else
+		m = perdv0 * opbdv0 * fwdvb;
+
+	vco = (m * sysclk) + (m >> 1);
+	cpu = vco / fwdva;
+	plb = vco / fwdvb / prbdv0;
+	opb = plb / opbdv0;
+	ebc = plb / perdv0;
+
+	/* FIXME */
+	uart0 = ser_clk;
+
+	/* Figure out timebase.  Either CPU or default TmrClk */
+	asm volatile (
+			"mfspr	%0,%1\n"
+			:
+			"=&r"(reg) : "i"(SPRN_CCR1));
+	if (reg & 0x0080)
+		tb = 25000000; /* TmrClk is 25MHz */
+	else
+		tb = cpu;
+
+	dt_fixup_cpu_clocks(cpu, tb, 0);
+	dt_fixup_clock("/plb", plb);
+	dt_fixup_clock("/plb/opb", opb);
+	dt_fixup_clock("/plb/opb/ebc", ebc);
+	dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}
+
+
+/*
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START		0x1
+#define DDR_START_SHIFT		0
+#define DDR_MAX_CS_REG		0x3
+#define DDR_MAX_CS_REG_SHIFT	24
+#define DDR_MAX_COL_REG		0xf
+#define DDR_MAX_COL_REG_SHIFT	16
+#define DDR_MAX_ROW_REG		0xf
+#define DDR_MAX_ROW_REG_SHIFT	8
+/* DDR0_08 */
+#define DDR_DDR2_MODE		0x1
+#define DDR_DDR2_MODE_SHIFT	0
+/* DDR0_10 */
+#define DDR_CS_MAP		0x3
+#define DDR_CS_MAP_SHIFT	8
+/* DDR0_14 */
+#define DDR_REDUC		0x1
+#define DDR_REDUC_SHIFT		16
+/* DDR0_42 */
+#define DDR_APIN		0x7
+#define DDR_APIN_SHIFT		24
+/* DDR0_43 */
+#define DDR_COL_SZ		0x7
+#define DDR_COL_SZ_SHIFT	8
+#define DDR_BANK8		0x1
+#define DDR_BANK8_SHIFT		0
+

arch/powerpc/boot/44x.h line 155

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+#define DDR_GET_VAL(val, mask, shift)	(((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)
+{
+	unsigned long val, max_cs, max_col, max_row;
+	unsigned long cs, col, row, bank, dpath;
+	unsigned long memsize;
+
+	mtdcr(DCRN_SDRAM0_CFGADDR, 2);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);

Again, mfsdram() functions would be handy here.

Quoted 54 lines
arch/powerpc/boot/44x.h +54 -0
+	if (!DDR_GET_VAL(val, DDR_START, DDR_START_SHIFT))
+		fatal("DDR controller is not initialized\n");
+
+	/* get maximum cs col and row values */
+	max_cs  = DDR_GET_VAL(val, DDR_MAX_CS_REG, DDR_MAX_CS_REG_SHIFT);
+	max_col = DDR_GET_VAL(val, DDR_MAX_COL_REG, DDR_MAX_COL_REG_SHIFT);
+	max_row = DDR_GET_VAL(val, DDR_MAX_ROW_REG, DDR_MAX_ROW_REG_SHIFT);
+
+	/* get CS value */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 10);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	val = DDR_GET_VAL(val, DDR_CS_MAP, DDR_CS_MAP_SHIFT);
+	cs = 0;
+	while (val) {
+		if (val && 0x1)
+			cs++;
+		val = val >> 1;
+	}
+
+	if (!cs)
+		fatal("No memory installed\n");
+	if (cs > max_cs)
+		fatal("DDR wrong CS configuration\n");
+
+	/* get data path bytes */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 14);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	if (DDR_GET_VAL(val, DDR_REDUC, DDR_REDUC_SHIFT))
+		dpath = 8; /* 64 bits */
+	else
+		dpath = 4; /* 32 bits */
+
+	/* get adress pins (rows) */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 42);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	row = DDR_GET_VAL(val, DDR_APIN, DDR_APIN_SHIFT);
+	if (row > max_row)
+		fatal("DDR wrong APIN configuration\n");
+	row = max_row - row;
+
+	/* get collomn size and banks */
+	mtdcr(DCRN_SDRAM0_CFGADDR, 43);
+	val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+	col = DDR_GET_VAL(val, DDR_COL_SZ, DDR_COL_SZ_SHIFT);
+	if (col > max_col)
+		fatal("DDR wrong COL configuration\n");
+	col = max_col - col;
+
+	if (DDR_GET_VAL(val, DDR_BANK8, DDR_BANK8_SHIFT))
+		bank = 8; /* 8 banks */

arch/powerpc/boot/44x.h line 219

Quoted 10 lines
arch/powerpc/boot/44x.h +10 -0
+	else
+		bank = 4; /* 4 banks */
+
+	memsize = cs * (1 << (col+row)) * bank * dpath;
+	dt_fixup_memory(0, memsize);
+}
+
+static void sequoia_fixups(void)
+{
+	unsigned long sysclk = 33333333;

Blank line after variable declaration please.

Quoted 48 lines
arch/powerpc/boot/44x.h +15 -0
+	ibm440ep_fixup_clocks(sysclk, 11059200);
+	ibm4xx_fixup_ebc_ranges("/plb/opb/ebc");
+	ibm440epx_fixup_memsize();
+	dt_fixup_mac_addresses(sequoia_mac0, sequoia_mac1);
+}
+
+void sequoia_init(void *mac0, void *mac1)
+{
+	platform_ops.fixups = sequoia_fixups;
+	platform_ops.exit = ibm44x_dbcr_reset;
+	sequoia_mac0 = mac0;
+	sequoia_mac1 = mac1;
+	ft_init(_dtb_start, 0, 32);
+	serial_console_init();
+}

diff -ruN linux.orig/arch/powerpc/boot/treeboot-sequoia.c linux/arch/powerpc/boot/treeboot-sequoia.c ---
linux.orig/arch/powerpc/boot/treeboot-sequoia.c 1970-01-01
03:00:00.000000000 +0300 +++
linux/arch/powerpc/boot/treeboot-sequoia.c 2007-07-27 20:44:26.000000000
+0400 @@ -0,0 +1,27 @@

arch/powerpc/boot/44x.h +27 -0
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+
+extern char _end[];
+
+BSS_STACK(4096);
+
+void platform_init(void)
+{
+	unsigned long end_of_ram = 0x10000000;
+	unsigned long avail_ram = end_of_ram - (unsigned long)_end;
+
+	simple_alloc_init(_end, avail_ram, 32, 64);
+	sequoia_init(NULL, NULL);
+}
Quoted 4 lines

_______________________________________________
Linuxppc-dev mailing list
[email protected]
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [email protected]
=====================================================================

#3 lore raw
Quoted 4 lines · Valentine Barshak
Quoted 2 lines

Is treeboot-* really needed on Sequoia? From my understanding this is only needed for platforms using OpenBIOS as bootloader.

Right, it's not used for u-boot. I've included it just in case.

I think we should drop it, when it's not used at all.

You didn't comment on my review down below in the code. Perhaps you didn't see it. Here again the original review with 2 small comments.

Thanks for the good work.

Quoted 31 lines

+++ linux/arch/powerpc/boot/sequoia.c 2007-07-27 20:59:09.000000000 +0400

arch/powerpc/boot/sequoia.c +29 -0
@@ -0,0 +1,234 @@
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * Clocking code based on code by:
+ * Stefan Roese <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include <stdarg.h>
+#include <stddef.h>
+#include "types.h"
+#include "elf.h"
+#include "string.h"
+#include "stdio.h"
+#include "page.h"
+#include "ops.h"
+#include "dcr.h"
+#include "44x.h"
+
+extern char _dtb_start[];
+extern char _dtb_end[];
+
+static u8 *sequoia_mac0, *sequoia_mac1;

arch/powerpc/boot/sequoia.c line 39

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+     u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+     u32 reg;
+     u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+     reg = mfdcr(DCRN_CPR0_DATA);

How about adding mfcpr functions to access the CPR regs. Makes the source better readable:

static inline u32 mfcpr(u32 reg)

{
        mtdcr(DCRN_CPR0_ADDR, reg);
        return mfdcr(DCRN_CPR0_DATA);
}
Quoted 106 lines
arch/powerpc/boot/sequoia.c +106 -0
+     tmp = (reg & 0x000F0000) >> 16;
+     fwdva = tmp ? tmp : 16;
+     tmp = (reg & 0x00000700) >> 8;
+     fwdvb = tmp ? tmp : 8;
+     tmp = (reg & 0x1F000000) >> 24;
+     fbdv = tmp ? tmp : 32;
+     lfbdv = (reg & 0x0000007F);
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     opbdv0 = tmp ? tmp : 4;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x07000000) >> 24;
+     perdv0 = tmp ? tmp : 8;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x07000000) >> 24;
+     prbdv0 = tmp ? tmp : 8;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     spcid0 = tmp ? tmp : 4;
+
+     /* Calculate M */
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     if (tmp == 0) { /* PLL output */
+             tmp = (reg & 0x20000000) >> 29;
+             if (!tmp) /* PLLOUTA */
+                     m = fbdv * lfbdv * fwdva;
+             else
+                     m = fbdv * lfbdv * fwdvb;
+     }
+     else if (tmp == 1) /* CPU output */
+             m = fbdv * fwdva;
+     else
+             m = perdv0 * opbdv0 * fwdvb;
+
+     vco = (m * sysclk) + (m >> 1);
+     cpu = vco / fwdva;
+     plb = vco / fwdvb / prbdv0;
+     opb = plb / opbdv0;
+     ebc = plb / perdv0;
+
+     /* FIXME */
+     uart0 = ser_clk;
+
+     /* Figure out timebase.  Either CPU or default TmrClk */
+     asm volatile (
+                     "mfspr  %0,%1\n"
+                     :
+                     "=&r"(reg) : "i"(SPRN_CCR1));
+     if (reg & 0x0080)
+             tb = 25000000; /* TmrClk is 25MHz */
+     else
+             tb = cpu;
+
+     dt_fixup_cpu_clocks(cpu, tb, 0);
+     dt_fixup_clock("/plb", plb);
+     dt_fixup_clock("/plb/opb", opb);
+     dt_fixup_clock("/plb/opb/ebc", ebc);
+     dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}
+
+
+/*
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START            0x1
+#define DDR_START_SHIFT              0
+#define DDR_MAX_CS_REG               0x3
+#define DDR_MAX_CS_REG_SHIFT 24
+#define DDR_MAX_COL_REG              0xf
+#define DDR_MAX_COL_REG_SHIFT        16
+#define DDR_MAX_ROW_REG              0xf
+#define DDR_MAX_ROW_REG_SHIFT        8
+/* DDR0_08 */
+#define DDR_DDR2_MODE                0x1
+#define DDR_DDR2_MODE_SHIFT  0
+/* DDR0_10 */
+#define DDR_CS_MAP           0x3
+#define DDR_CS_MAP_SHIFT     8
+/* DDR0_14 */
+#define DDR_REDUC            0x1
+#define DDR_REDUC_SHIFT              16
+/* DDR0_42 */
+#define DDR_APIN             0x7
+#define DDR_APIN_SHIFT               24
+/* DDR0_43 */
+#define DDR_COL_SZ           0x7
+#define DDR_COL_SZ_SHIFT     8
+#define DDR_BANK8            0x1
+#define DDR_BANK8_SHIFT              0
+

arch/powerpc/boot/sequoia.c line 155

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+#define DDR_GET_VAL(val, mask, shift)        (((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)
+{
+     unsigned long val, max_cs, max_col, max_row;
+     unsigned long cs, col, row, bank, dpath;
+     unsigned long memsize;
+
+     mtdcr(DCRN_SDRAM0_CFGADDR, 2);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);

Again, mfsdram() functions would be handy here.

Quoted 54 lines
arch/powerpc/boot/sequoia.c +54 -0
+     if (!DDR_GET_VAL(val, DDR_START, DDR_START_SHIFT))
+             fatal("DDR controller is not initialized\n");
+
+     /* get maximum cs col and row values */
+     max_cs  = DDR_GET_VAL(val, DDR_MAX_CS_REG, DDR_MAX_CS_REG_SHIFT);
+     max_col = DDR_GET_VAL(val, DDR_MAX_COL_REG, DDR_MAX_COL_REG_SHIFT);
+     max_row = DDR_GET_VAL(val, DDR_MAX_ROW_REG, DDR_MAX_ROW_REG_SHIFT);
+
+     /* get CS value */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 10);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     val = DDR_GET_VAL(val, DDR_CS_MAP, DDR_CS_MAP_SHIFT);
+     cs = 0;
+     while (val) {
+             if (val && 0x1)
+                     cs++;
+             val = val >> 1;
+     }
+
+     if (!cs)
+             fatal("No memory installed\n");
+     if (cs > max_cs)
+             fatal("DDR wrong CS configuration\n");
+
+     /* get data path bytes */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 14);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     if (DDR_GET_VAL(val, DDR_REDUC, DDR_REDUC_SHIFT))
+             dpath = 8; /* 64 bits */
+     else
+             dpath = 4; /* 32 bits */
+
+     /* get adress pins (rows) */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 42);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     row = DDR_GET_VAL(val, DDR_APIN, DDR_APIN_SHIFT);
+     if (row > max_row)
+             fatal("DDR wrong APIN configuration\n");
+     row = max_row - row;
+
+     /* get collomn size and banks */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 43);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     col = DDR_GET_VAL(val, DDR_COL_SZ, DDR_COL_SZ_SHIFT);
+     if (col > max_col)
+             fatal("DDR wrong COL configuration\n");
+     col = max_col - col;
+
+     if (DDR_GET_VAL(val, DDR_BANK8, DDR_BANK8_SHIFT))
+             bank = 8; /* 8 banks */

arch/powerpc/boot/sequoia.c line 219

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+     else
+             bank = 4; /* 4 banks */
+
+     memsize = cs * (1 << (col+row)) * bank * dpath;
+     dt_fixup_memory(0, memsize);
+}
+
+static void sequoia_fixups(void)
+{
+     unsigned long sysclk = 33333333;

Blank line after variable declaration please.

Quoted 38 lines
arch/powerpc/boot/sequoia.c +15 -0
+     ibm440ep_fixup_clocks(sysclk, 11059200);
+     ibm4xx_fixup_ebc_ranges("/plb/opb/ebc");
+     ibm440epx_fixup_memsize();
+     dt_fixup_mac_addresses(sequoia_mac0, sequoia_mac1);
+}
+
+void sequoia_init(void *mac0, void *mac1)
+{
+     platform_ops.fixups = sequoia_fixups;
+     platform_ops.exit = ibm44x_dbcr_reset;
+     sequoia_mac0 = mac0;
+     sequoia_mac1 = mac1;
+     ft_init(_dtb_start, 0, 32);
+     serial_console_init();
+}

diff -ruN linux.orig/arch/powerpc/boot/treeboot-sequoia.c linux/arch/powerpc/boot/treeboot-sequoia.c ---
linux.orig/arch/powerpc/boot/treeboot-sequoia.c 1970-01-01
03:00:00.000000000 +0300 +++
linux/arch/powerpc/boot/treeboot-sequoia.c 2007-07-27 20:44:26.000000000
+0400 @@ -0,0 +1,27 @@

arch/powerpc/boot/treeboot-sequoia.c +17 -0
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+
+extern char _end[];
+

arch/powerpc/boot/treeboot-sequoia.c line 27

Quoted 10 lines
arch/powerpc/boot/treeboot-sequoia.c +10 -0
+BSS_STACK(4096);
+
+void platform_init(void)
+{
+     unsigned long end_of_ram = 0x10000000;
+     unsigned long avail_ram = end_of_ram - (unsigned long)_end;
+
+     simple_alloc_init(_end, avail_ram, 32, 64);
+     sequoia_init(NULL, NULL);
+}

Best regards,
Stefan

author
#4 lore raw
Quoted 9 lines · Stefan Roese
Quoted 3 lines · Valentine Barshak
Quoted 2 lines

Is treeboot-* really needed on Sequoia? From my understanding this is only needed for platforms using OpenBIOS as bootloader.

Right, it's not used for u-boot. I've included it just in case.

I think we should drop it, when it's not used at all.

You didn't comment on my review down below in the code. Perhaps you didn't see it. Here again the original review with 2 small comments.

Oops, sorry, I've missed it :) Thanks for review. I'll modify the code according to your comments. Thanks,
Valentine.

Quoted 289 lines

Thanks for the good work.

Quoted 31 lines

+++ linux/arch/powerpc/boot/sequoia.c 2007-07-27 20:59:09.000000000 +0400

arch/powerpc/boot/sequoia.c +29 -0
@@ -0,0 +1,234 @@
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * Clocking code based on code by:
+ * Stefan Roese <[email protected]>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include <stdarg.h>
+#include <stddef.h>
+#include "types.h"
+#include "elf.h"
+#include "string.h"
+#include "stdio.h"
+#include "page.h"
+#include "ops.h"
+#include "dcr.h"
+#include "44x.h"
+
+extern char _dtb_start[];
+extern char _dtb_end[];
+
+static u8 *sequoia_mac0, *sequoia_mac1;

arch/powerpc/boot/sequoia.c line 39

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+     u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+     u32 reg;
+     u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+     reg = mfdcr(DCRN_CPR0_DATA);

How about adding mfcpr functions to access the CPR regs. Makes the source better readable:

static inline u32 mfcpr(u32 reg)

{
        mtdcr(DCRN_CPR0_ADDR, reg);
        return mfdcr(DCRN_CPR0_DATA);
}
Quoted 106 lines
arch/powerpc/boot/sequoia.c +106 -0
+     tmp = (reg & 0x000F0000) >> 16;
+     fwdva = tmp ? tmp : 16;
+     tmp = (reg & 0x00000700) >> 8;
+     fwdvb = tmp ? tmp : 8;
+     tmp = (reg & 0x1F000000) >> 24;
+     fbdv = tmp ? tmp : 32;
+     lfbdv = (reg & 0x0000007F);
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     opbdv0 = tmp ? tmp : 4;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x07000000) >> 24;
+     perdv0 = tmp ? tmp : 8;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x07000000) >> 24;
+     prbdv0 = tmp ? tmp : 8;
+
+     mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     spcid0 = tmp ? tmp : 4;
+
+     /* Calculate M */
+     mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+     reg = mfdcr(DCRN_CPR0_DATA);
+     tmp = (reg & 0x03000000) >> 24;
+     if (tmp == 0) { /* PLL output */
+             tmp = (reg & 0x20000000) >> 29;
+             if (!tmp) /* PLLOUTA */
+                     m = fbdv * lfbdv * fwdva;
+             else
+                     m = fbdv * lfbdv * fwdvb;
+     }
+     else if (tmp == 1) /* CPU output */
+             m = fbdv * fwdva;
+     else
+             m = perdv0 * opbdv0 * fwdvb;
+
+     vco = (m * sysclk) + (m >> 1);
+     cpu = vco / fwdva;
+     plb = vco / fwdvb / prbdv0;
+     opb = plb / opbdv0;
+     ebc = plb / perdv0;
+
+     /* FIXME */
+     uart0 = ser_clk;
+
+     /* Figure out timebase.  Either CPU or default TmrClk */
+     asm volatile (
+                     "mfspr  %0,%1\n"
+                     :
+                     "=&r"(reg) : "i"(SPRN_CCR1));
+     if (reg & 0x0080)
+             tb = 25000000; /* TmrClk is 25MHz */
+     else
+             tb = cpu;
+
+     dt_fixup_cpu_clocks(cpu, tb, 0);
+     dt_fixup_clock("/plb", plb);
+     dt_fixup_clock("/plb/opb", opb);
+     dt_fixup_clock("/plb/opb/ebc", ebc);
+     dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+     dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}
+
+
+/*
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START            0x1
+#define DDR_START_SHIFT              0
+#define DDR_MAX_CS_REG               0x3
+#define DDR_MAX_CS_REG_SHIFT 24
+#define DDR_MAX_COL_REG              0xf
+#define DDR_MAX_COL_REG_SHIFT        16
+#define DDR_MAX_ROW_REG              0xf
+#define DDR_MAX_ROW_REG_SHIFT        8
+/* DDR0_08 */
+#define DDR_DDR2_MODE                0x1
+#define DDR_DDR2_MODE_SHIFT  0
+/* DDR0_10 */
+#define DDR_CS_MAP           0x3
+#define DDR_CS_MAP_SHIFT     8
+/* DDR0_14 */
+#define DDR_REDUC            0x1
+#define DDR_REDUC_SHIFT              16
+/* DDR0_42 */
+#define DDR_APIN             0x7
+#define DDR_APIN_SHIFT               24
+/* DDR0_43 */
+#define DDR_COL_SZ           0x7
+#define DDR_COL_SZ_SHIFT     8
+#define DDR_BANK8            0x1
+#define DDR_BANK8_SHIFT              0
+

arch/powerpc/boot/sequoia.c line 155

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+#define DDR_GET_VAL(val, mask, shift)        (((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)
+{
+     unsigned long val, max_cs, max_col, max_row;
+     unsigned long cs, col, row, bank, dpath;
+     unsigned long memsize;
+
+     mtdcr(DCRN_SDRAM0_CFGADDR, 2);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);

Again, mfsdram() functions would be handy here.

Quoted 54 lines
arch/powerpc/boot/sequoia.c +54 -0
+     if (!DDR_GET_VAL(val, DDR_START, DDR_START_SHIFT))
+             fatal("DDR controller is not initialized\n");
+
+     /* get maximum cs col and row values */
+     max_cs  = DDR_GET_VAL(val, DDR_MAX_CS_REG, DDR_MAX_CS_REG_SHIFT);
+     max_col = DDR_GET_VAL(val, DDR_MAX_COL_REG, DDR_MAX_COL_REG_SHIFT);
+     max_row = DDR_GET_VAL(val, DDR_MAX_ROW_REG, DDR_MAX_ROW_REG_SHIFT);
+
+     /* get CS value */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 10);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     val = DDR_GET_VAL(val, DDR_CS_MAP, DDR_CS_MAP_SHIFT);
+     cs = 0;
+     while (val) {
+             if (val && 0x1)
+                     cs++;
+             val = val >> 1;
+     }
+
+     if (!cs)
+             fatal("No memory installed\n");
+     if (cs > max_cs)
+             fatal("DDR wrong CS configuration\n");
+
+     /* get data path bytes */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 14);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     if (DDR_GET_VAL(val, DDR_REDUC, DDR_REDUC_SHIFT))
+             dpath = 8; /* 64 bits */
+     else
+             dpath = 4; /* 32 bits */
+
+     /* get adress pins (rows) */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 42);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     row = DDR_GET_VAL(val, DDR_APIN, DDR_APIN_SHIFT);
+     if (row > max_row)
+             fatal("DDR wrong APIN configuration\n");
+     row = max_row - row;
+
+     /* get collomn size and banks */
+     mtdcr(DCRN_SDRAM0_CFGADDR, 43);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);
+
+     col = DDR_GET_VAL(val, DDR_COL_SZ, DDR_COL_SZ_SHIFT);
+     if (col > max_col)
+             fatal("DDR wrong COL configuration\n");
+     col = max_col - col;
+
+     if (DDR_GET_VAL(val, DDR_BANK8, DDR_BANK8_SHIFT))
+             bank = 8; /* 8 banks */

arch/powerpc/boot/sequoia.c line 219

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+     else
+             bank = 4; /* 4 banks */
+
+     memsize = cs * (1 << (col+row)) * bank * dpath;
+     dt_fixup_memory(0, memsize);
+}
+
+static void sequoia_fixups(void)
+{
+     unsigned long sysclk = 33333333;

Blank line after variable declaration please.

Quoted 38 lines
arch/powerpc/boot/sequoia.c +15 -0
+     ibm440ep_fixup_clocks(sysclk, 11059200);
+     ibm4xx_fixup_ebc_ranges("/plb/opb/ebc");
+     ibm440epx_fixup_memsize();
+     dt_fixup_mac_addresses(sequoia_mac0, sequoia_mac1);
+}
+
+void sequoia_init(void *mac0, void *mac1)
+{
+     platform_ops.fixups = sequoia_fixups;
+     platform_ops.exit = ibm44x_dbcr_reset;
+     sequoia_mac0 = mac0;
+     sequoia_mac1 = mac1;
+     ft_init(_dtb_start, 0, 32);
+     serial_console_init();
+}

diff -ruN linux.orig/arch/powerpc/boot/treeboot-sequoia.c linux/arch/powerpc/boot/treeboot-sequoia.c ---
linux.orig/arch/powerpc/boot/treeboot-sequoia.c 1970-01-01
03:00:00.000000000 +0300 +++
linux/arch/powerpc/boot/treeboot-sequoia.c 2007-07-27 20:44:26.000000000
+0400 @@ -0,0 +1,27 @@

arch/powerpc/boot/treeboot-sequoia.c +17 -0
+/*
+ * Based on Bamboo code by Josh Boyer <[email protected]>
+ * Copyright IBM Corporation, 2007
+ *
+ * Based on ebony wrapper:
+ * Copyright 2007 David Gibson, IBM Corporation.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as published by the Free Software Foundation; version 2 of the License
+ */
+#include "ops.h"
+#include "stdio.h"
+#include "44x.h"
+
+extern char _end[];
+

arch/powerpc/boot/treeboot-sequoia.c line 27

Quoted 10 lines
arch/powerpc/boot/treeboot-sequoia.c +10 -0
+BSS_STACK(4096);
+
+void platform_init(void)
+{
+     unsigned long end_of_ram = 0x10000000;
+     unsigned long avail_ram = end_of_ram - (unsigned long)_end;
+
+     simple_alloc_init(_end, avail_ram, 32, 64);
+     sequoia_init(NULL, NULL);
+}

Best regards,
Stefan

#5 lore raw
In reply to Stefan Roese
Quoted 15 lines · Stefan Roese
Quoted 4 lines · Valentine Barshak
Quoted 2 lines

Is treeboot-* really needed on Sequoia? From my understanding this is only needed for platforms using OpenBIOS as bootloader.

Right, it's not used for u-boot. I've included it just in case.

I think we should drop it, when it's not used at all.

You didn't comment on my review down below in the code. Perhaps you didn't see it. Here again the original review with 2 small comments.

Thanks for the good work.

Quoted 1 line

+++ linux/arch/powerpc/boot/sequoia.c 2007-07-27

20:59:09.000000000 +0400

[snip]

Quoted 10 lines

arch/powerpc/boot/sequoia.c line 155

Quoted 8 lines
arch/powerpc/boot/sequoia.c +8 -0
+static void ibm440epx_fixup_memsize(void)
+{
+     unsigned long val, max_cs, max_col, max_row;
+     unsigned long cs, col, row, bank, dpath;
+     unsigned long memsize;
+
+     mtdcr(DCRN_SDRAM0_CFGADDR, 2);
+     val = mfdcr(DCRN_SDRAM0_CFGDATA);

Again, mfsdram() functions would be handy here.

I concur, also add a #define for the value written to CFGADDR.

Signature · 4 lines
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
#6 lore raw
In reply to Valentine Barshak

On Mon, 30 Jul 2007 19:14:45 +0400

Quoted 71 lines
arch/powerpc/boot/sequoia.c +71 -0
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+	u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+	u32 reg;
+	u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x000F0000) >> 16;
+	fwdva = tmp ? tmp : 16;
+	tmp = (reg & 0x00000700) >> 8;
+	fwdvb = tmp ? tmp : 8;
+	tmp = (reg & 0x1F000000) >> 24;
+	fbdv = tmp ? tmp : 32;
+	lfbdv = (reg & 0x0000007F);
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	opbdv0 = tmp ? tmp : 4;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	perdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	prbdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	spcid0 = tmp ? tmp : 4;
+
+	/* Calculate M */
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	if (tmp == 0) { /* PLL output */
+		tmp = (reg & 0x20000000) >> 29;
+		if (!tmp) /* PLLOUTA */
+			m = fbdv * lfbdv * fwdva;
+		else
+			m = fbdv * lfbdv * fwdvb;
+	}
+	else if (tmp == 1) /* CPU output */
+		m = fbdv * fwdva;
+	else
+		m = perdv0 * opbdv0 * fwdvb;
+
+	vco = (m * sysclk) + (m >> 1);
+	cpu = vco / fwdva;
+	plb = vco / fwdvb / prbdv0;
+	opb = plb / opbdv0;
+	ebc = plb / perdv0;
+
+	/* FIXME */
+	uart0 = ser_clk;
+
+	/* Figure out timebase.  Either CPU or default TmrClk */
+	asm volatile (
+			"mfspr	%0,%1\n"
+			:
+			"=&r"(reg) : "i"(SPRN_CCR1));
+	if (reg & 0x0080)
+		tb = 25000000; /* TmrClk is 25MHz */
+	else
+		tb = cpu;

arch/powerpc/boot/sequoia.c line 111

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+
+	dt_fixup_cpu_clocks(cpu, tb, 0);
+	dt_fixup_clock("/plb", plb);
+	dt_fixup_clock("/plb/opb", opb);
+	dt_fixup_clock("/plb/opb/ebc", ebc);
+	dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}

We don't need to duplicate this function in two different wrappers. I'll move it into 4xx.c in my next round of patches, so that bamboo and sequoia can just call a common function.

Quoted 25 lines
arch/powerpc/boot/sequoia.c +25 -0
+/* 
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START		0x1
+#define DDR_START_SHIFT		0
+#define DDR_MAX_CS_REG		0x3
+#define DDR_MAX_CS_REG_SHIFT	24
+#define DDR_MAX_COL_REG		0xf
+#define DDR_MAX_COL_REG_SHIFT	16
+#define DDR_MAX_ROW_REG		0xf
+#define DDR_MAX_ROW_REG_SHIFT	8
+/* DDR0_08 */
+#define DDR_DDR2_MODE		0x1
+#define DDR_DDR2_MODE_SHIFT	0
+/* DDR0_10 */
+#define DDR_CS_MAP		0x3
+#define DDR_CS_MAP_SHIFT	8
+/* DDR0_14 */
+#define DDR_REDUC		0x1
+#define DDR_REDUC_SHIFT		16
+/* DDR0_42 */
+#define DDR_APIN		0x7

arch/powerpc/boot/sequoia.c line 148

Quoted 10 lines
arch/powerpc/boot/sequoia.c +10 -0
+#define DDR_APIN_SHIFT		24
+/* DDR0_43 */
+#define DDR_COL_SZ		0x7
+#define DDR_COL_SZ_SHIFT	8
+#define DDR_BANK8		0x1
+#define DDR_BANK8_SHIFT		0
+
+#define DDR_GET_VAL(val, mask, shift)	(((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)

And we should move this at the same time. Isn't denali used by another CPU as well?

josh

#7 lore raw
Quoted 78 lines · Josh Boyer

On Mon, 30 Jul 2007 19:14:45 +0400

Quoted 77 lines
arch/powerpc/boot/sequoia.c +77 -0
+#define SPRN_CCR1 0x378
+void ibm440ep_fixup_clocks(unsigned int sysclk, unsigned int ser_clk)
+{
+	u32 cpu, plb, opb, ebc, tb, uart0, m, vco;
+	u32 reg;
+	u32 fwdva, fwdvb, fbdv, lfbdv, opbdv0, perdv0, spcid0, prbdv0, tmp;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x000F0000) >> 16;
+	fwdva = tmp ? tmp : 16;
+	tmp = (reg & 0x00000700) >> 8;
+	fwdvb = tmp ? tmp : 8;
+	tmp = (reg & 0x1F000000) >> 24;
+	fbdv = tmp ? tmp : 32;
+	lfbdv = (reg & 0x0000007F);
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_OPBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	opbdv0 = tmp ? tmp : 4;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PERD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	perdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PRIMBD0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x07000000) >> 24;
+	prbdv0 = tmp ? tmp : 8;
+
+	mtdcr(DCRN_CPR0_ADDR, CPR0_SCPID);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	spcid0 = tmp ? tmp : 4;
+
+	/* Calculate M */
+	mtdcr(DCRN_CPR0_ADDR, CPR0_PLLC0);
+	reg = mfdcr(DCRN_CPR0_DATA);
+	tmp = (reg & 0x03000000) >> 24;
+	if (tmp == 0) { /* PLL output */
+		tmp = (reg & 0x20000000) >> 29;
+		if (!tmp) /* PLLOUTA */
+			m = fbdv * lfbdv * fwdva;
+		else
+			m = fbdv * lfbdv * fwdvb;
+	}
+	else if (tmp == 1) /* CPU output */
+		m = fbdv * fwdva;
+	else
+		m = perdv0 * opbdv0 * fwdvb;
+
+	vco = (m * sysclk) + (m >> 1);
+	cpu = vco / fwdva;
+	plb = vco / fwdvb / prbdv0;
+	opb = plb / opbdv0;
+	ebc = plb / perdv0;
+
+	/* FIXME */
+	uart0 = ser_clk;
+
+	/* Figure out timebase.  Either CPU or default TmrClk */
+	asm volatile (
+			"mfspr	%0,%1\n"
+			:
+			"=&r"(reg) : "i"(SPRN_CCR1));
+	if (reg & 0x0080)
+		tb = 25000000; /* TmrClk is 25MHz */
+	else
+		tb = cpu;
+
+	dt_fixup_cpu_clocks(cpu, tb, 0);
+	dt_fixup_clock("/plb", plb);
+	dt_fixup_clock("/plb/opb", opb);
+	dt_fixup_clock("/plb/opb/ebc", ebc);
+	dt_fixup_clock("/plb/opb/serial@ef600300", uart0);
Quoted 8 lines

arch/powerpc/boot/sequoia.c line 111

Quoted 4 lines
arch/powerpc/boot/sequoia.c +4 -0
+	dt_fixup_clock("/plb/opb/serial@ef600400", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600500", uart0);
+	dt_fixup_clock("/plb/opb/serial@ef600600", uart0);
+}

We don't need to duplicate this function in two different wrappers. I'll move it into 4xx.c in my next round of patches, so that bamboo and sequoia can just call a common function.

Good idea.

Quoted 30 lines
Quoted 30 lines
arch/powerpc/boot/sequoia.c +30 -0
+/*
+ * 440EPx DDR1/2 memory controller code
+ * TODO: move to generic 44x code
+ */
+
+/* DDR0_02 */
+#define DDR_START		0x1
+#define DDR_START_SHIFT		0
+#define DDR_MAX_CS_REG		0x3
+#define DDR_MAX_CS_REG_SHIFT	24
+#define DDR_MAX_COL_REG		0xf
+#define DDR_MAX_COL_REG_SHIFT	16
+#define DDR_MAX_ROW_REG		0xf
+#define DDR_MAX_ROW_REG_SHIFT	8
+/* DDR0_08 */
+#define DDR_DDR2_MODE		0x1
+#define DDR_DDR2_MODE_SHIFT	0
+/* DDR0_10 */
+#define DDR_CS_MAP		0x3
+#define DDR_CS_MAP_SHIFT	8
+/* DDR0_14 */
+#define DDR_REDUC		0x1
+#define DDR_REDUC_SHIFT		16
+/* DDR0_42 */
+#define DDR_APIN		0x7
+#define DDR_APIN_SHIFT		24
+/* DDR0_43 */
+#define DDR_COL_SZ		0x7
+#define DDR_COL_SZ_SHIFT	8
+#define DDR_BANK8		0x1
Quoted 8 lines

arch/powerpc/boot/sequoia.c line 148

Quoted 5 lines
arch/powerpc/boot/sequoia.c +5 -0
+#define DDR_BANK8_SHIFT		0
+
+#define DDR_GET_VAL(val, mask, shift)	(((val) >> (shift)) & (mask))
+
+static void ibm440epx_fixup_memsize(void)

And we should move this at the same time. Isn't denali used by another CPU as well?

Right now the Denali DDR(2) core is only used by the 440EPx/440GRx. But other AMCC PPC's using it will follow soon.

Best regards,
Stefan

#8 lore raw
Quoted 7 lines · On Fri, 3 Aug 2007 08:38:25 +0200 Stefan Roese <[email protected]> wrote:
Quoted 4 lines
Quoted 1 line

+static void ibm440epx_fixup_memsize(void)

And we should move this at the same time. Isn't denali used by another CPU as well?

Right now the Denali DDR(2) core is only used by the 440EPx/440GRx. But other AMCC PPC's using it will follow soon.

So that would be a yes :) Good. Then we should add the function to 4xx.c and call it ibm4xx_denali_fixup_memsize.

josh