* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
@ 2009-11-14 1:01 Albert Aribaud
2009-11-14 2:21 ` Tom
2009-11-14 6:29 ` Prafulla Wadaskar
0 siblings, 2 replies; 6+ messages in thread
From: Albert Aribaud @ 2009-11-14 1:01 UTC (permalink / raw)
To: u-boot
This patch adds initial u-boot support for the Marvell orion5x
SoC and the LAcie ED Mini V2 board--support is limited to serial,
flash and environment. Further support will be added as devices
common with kirkwood are made useable across SoCs.
Signed-off-by: Albert Aribaud <albert.aribaud@free.fr>
---
Makefile | 3 +
board/Marvell/edminiv2/Makefile | 51 +++++
board/Marvell/edminiv2/config.mk | 25 +++
board/Marvell/edminiv2/edminiv2.c | 165 ++++++++++++++++
board/Marvell/edminiv2/edminiv2.h | 39 ++++
cpu/arm926ejs/orion5x/Makefile | 52 +++++
cpu/arm926ejs/orion5x/cpu.c | 339 ++++++++++++++++++++++++++++++++
cpu/arm926ejs/orion5x/dram.c | 61 ++++++
cpu/arm926ejs/orion5x/mpp.c | 88 ++++++++
cpu/arm926ejs/orion5x/timer.c | 171 ++++++++++++++++
drivers/serial/serial.c | 3 +
include/asm-arm/arch-orion5x/88f5182.h | 40 ++++
include/asm-arm/arch-orion5x/cpu.h | 213 ++++++++++++++++++++
include/asm-arm/arch-orion5x/gpio.h | 53 +++++
include/asm-arm/arch-orion5x/mpp.h | 122 ++++++++++++
include/asm-arm/arch-orion5x/orion5x.h | 67 +++++++
include/configs/edminiv2.h | 158 +++++++++++++++
17 files changed, 1650 insertions(+), 0 deletions(-)
create mode 100644 board/Marvell/edminiv2/Makefile
create mode 100644 board/Marvell/edminiv2/config.mk
create mode 100644 board/Marvell/edminiv2/edminiv2.c
create mode 100644 board/Marvell/edminiv2/edminiv2.h
create mode 100644 cpu/arm926ejs/orion5x/Makefile
create mode 100644 cpu/arm926ejs/orion5x/cpu.c
create mode 100644 cpu/arm926ejs/orion5x/dram.c
create mode 100644 cpu/arm926ejs/orion5x/mpp.c
create mode 100644 cpu/arm926ejs/orion5x/timer.c
create mode 100644 include/asm-arm/arch-orion5x/88f5182.h
create mode 100644 include/asm-arm/arch-orion5x/cpu.h
create mode 100644 include/asm-arm/arch-orion5x/gpio.h
create mode 100644 include/asm-arm/arch-orion5x/mpp.h
create mode 100644 include/asm-arm/arch-orion5x/orion5x.h
create mode 100644 include/configs/edminiv2.h
diff --git a/Makefile b/Makefile
index bcb3fe9..60f9140 100644
--- a/Makefile
+++ b/Makefile
@@ -2951,6 +2951,9 @@ davinci_dm365evm_config : unconfig
davinci_dm6467evm_config : unconfig
@$(MKCONFIG) $(@:_config=) arm arm926ejs dm6467evm davinci davinci
+edminiv2_config: unconfig
+ @$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=) Marvell orion5x
+
imx27lite_config: unconfig
@$(MKCONFIG) $(@:_config=) arm arm926ejs imx27lite logicpd mx27
diff --git a/board/Marvell/edminiv2/Makefile b/board/Marvell/edminiv2/Makefile
new file mode 100644
index 0000000..2e6d305
--- /dev/null
+++ b/board/Marvell/edminiv2/Makefile
@@ -0,0 +1,51 @@
+#
+# (C) Copyright 2009
+# Marvell Semiconductor <www.marvell.com>
+# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# 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; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+# MA 02110-1301 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB = $(obj)lib$(BOARD).a
+
+COBJS := edminiv2.o
+
+SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
+OBJS := $(addprefix $(obj),$(COBJS))
+SOBJS := $(addprefix $(obj),$(SOBJS))
+
+$(LIB): $(obj).depend $(OBJS) $(SOBJS)
+ $(AR) $(ARFLAGS) $@ $(OBJS) $(SOBJS)
+
+clean:
+ rm -f $(SOBJS) $(OBJS)
+
+distclean: clean
+ rm -f $(LIB) core *.bak .depend
+
+#########################################################################
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#########################################################################
diff --git a/board/Marvell/edminiv2/config.mk b/board/Marvell/edminiv2/config.mk
new file mode 100644
index 0000000..a4ea769
--- /dev/null
+++ b/board/Marvell/edminiv2/config.mk
@@ -0,0 +1,25 @@
+#
+# (C) Copyright 2009
+# Marvell Semiconductor <www.marvell.com>
+# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# 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; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+# MA 02110-1301 USA
+#
+
+TEXT_BASE = 0x00600000
diff --git a/board/Marvell/edminiv2/edminiv2.c b/board/Marvell/edminiv2/edminiv2.c
new file mode 100644
index 0000000..f23de0e
--- /dev/null
+++ b/board/Marvell/edminiv2/edminiv2.c
@@ -0,0 +1,165 @@
+/*
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#include <common.h>
+#include <miiphy.h>
+#include <asm/arch/orion5x.h>
+#include <asm/arch/mpp.h>
+#include "edminiv2.h"
+
+DECLARE_GLOBAL_DATA_PTR;
+
+#if defined(CONFIG_FLASH_CFI_LEGACY)
+#include <flash.h>
+ulong board_flash_get_legacy (ulong base, int banknum, flash_info_t * info)
+{
+ int sectsz[] = CONFIG_SYS_FLASH_SECTSZ;
+ int sect;
+
+ if (base != CONFIG_SYS_FLASH_BASE)
+ return 0;
+
+ info->size = 0;
+ info->sector_count = CONFIG_SYS_MAX_FLASH_SECT;
+ for (sect=0; sect<CONFIG_SYS_MAX_FLASH_SECT; sect++) {
+ info->start[sect] = base+info->size;
+ info->size += sectsz[sect];
+ }
+ info->flash_id = 0x01000000;
+ info->portwidth = FLASH_CFI_8BIT;
+ info->chipwidth = FLASH_CFI_BY8;
+ info->buffer_size = 32;
+ info->erase_blk_tout = 16384;
+ info->write_tout = 2;
+ info->buffer_write_tout = 5;
+ info->vendor = CFI_CMDSET_AMD_LEGACY;
+ info->cmd_reset= 0xF0;
+ info->interface= FLASH_CFI_X8;
+ info->legacy_unlock = 0;
+ info->manufacturer_id = 0xC2;
+ info->device_id = 0xBA;
+ info->device_id2= 0;
+
+ info->ext_addr = 0;
+ info->cfi_version = 0x3133;
+ info->cfi_offset = 0x0000;
+ info->addr_unlock1 = 0x00000555;
+ info->addr_unlock2 = 0x000002AA;
+ info->name = "MX29LV400CB";
+
+ return 1;
+}
+#endif /* CONFIG_SYS_FLASH_CFI */
+
+int board_init(void)
+{
+ /*
+ * default gpio configuration
+ * There are maximum 64 gpios controlled through 2 sets of registers
+ * the below configuration configures mainly initial LED status
+ */
+ orion5x_config_gpio(EDMINIV2_OE_VAL_LOW, EDMINIV2_OE_LOW);
+
+ /* Multi-Purpose Pins Functionality configuration */
+ u32 edminiv2_mpp_config[] = {
+ MPP0_GPIO,
+ MPP1_GPIO,
+ MPP2_GPIO,
+ MPP3_GPIO,
+ MPP4_GPIO,
+ MPP5_GPIO,
+ MPP6_GPIO,
+ MPP7_GPIO,
+ MPP8_GPIO,
+ MPP9_GPIO,
+ MPP10_GPIO,
+ MPP11_GPIO,
+ MPP12_SATA0_PRESENTn,
+ MPP13_SATA1_PRESENTn,
+ MPP14_SATA0_ACTn,
+ MPP15_SATA1_ACTn,
+ MPP16_GPIO,
+ MPP17_GPIO,
+ MPP18_GPIO,
+ MPP19_GPIO,
+ 0
+ };
+ orion5x_mpp_conf(edminiv2_mpp_config);
+
+ /*
+ * arch number of board
+ */
+ gd->bd->bi_arch_number = MACH_TYPE_EDMINI_V2;
+
+ /* adress of boot parameters */
+ gd->bd->bi_boot_params = orion5x_sdram_bar(0) + 0x100;
+
+ return 0;
+}
+
+int dram_init(void)
+{
+ int i;
+
+ for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
+ gd->bd->bi_dram[i].start = orion5x_sdram_bar(i);
+ gd->bd->bi_dram[i].size = orion5x_sdram_bs(i);
+ }
+ return 0;
+}
+
+#ifdef CONFIG_RESET_PHY_R
+/* Configure and enable MV88E1116 PHY */
+void reset_phy(void)
+{
+ u16 reg;
+ u16 devadr;
+ char *name = "egiga0";
+
+ if (miiphy_set_current_dev(name))
+ return;
+
+ /* command to read PHY dev address */
+ if (miiphy_read(name, 0xEE, 0xEE, (u16 *) &devadr)) {
+ printf("Err..%s could not read PHY dev address\n",
+ __FUNCTION__);
+ return;
+ }
+
+ /*
+ * Enable RGMII delay on Tx and Rx for CPU port
+ * Ref: sec 4.7.2 of chip datasheet
+ */
+ miiphy_write(name, devadr, MV88E1116_PGADR_REG, 2);
+ miiphy_read(name, devadr, MV88E1116_MAC_CTRL_REG, ®);
+ reg |= (MV88E1116_RGMII_RXTM_CTRL | MV88E1116_RGMII_TXTM_CTRL);
+ miiphy_write(name, devadr, MV88E1116_MAC_CTRL_REG, reg);
+ miiphy_write(name, devadr, MV88E1116_PGADR_REG, 0);
+
+ /* reset the phy */
+ miiphy_reset(name, devadr);
+
+ printf("88E1116 Initialized on %s\n", name);
+}
+#endif /* CONFIG_RESET_PHY_R */
diff --git a/board/Marvell/edminiv2/edminiv2.h b/board/Marvell/edminiv2/edminiv2.h
new file mode 100644
index 0000000..77a108e
--- /dev/null
+++ b/board/Marvell/edminiv2/edminiv2.h
@@ -0,0 +1,39 @@
+/*
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#ifndef __EDMINIV2_H
+#define __EDMINIV2_H
+
+#define EDMINIV2_OE_LOW (~(0))
+#define EDMINIV2_OE_VAL_LOW (1 << 29) /* USB_PWEN low */
+
+/* PHY related */
+#define MV88E1116_LED_FCTRL_REG 10
+#define MV88E1116_CPRSP_CR3_REG 21
+#define MV88E1116_MAC_CTRL_REG 21
+#define MV88E1116_PGADR_REG 22
+#define MV88E1116_RGMII_TXTM_CTRL (1 << 4)
+#define MV88E1116_RGMII_RXTM_CTRL (1 << 5)
+
+#endif /* __EDMINIV2_H */
diff --git a/cpu/arm926ejs/orion5x/Makefile b/cpu/arm926ejs/orion5x/Makefile
new file mode 100644
index 0000000..4b506a7
--- /dev/null
+++ b/cpu/arm926ejs/orion5x/Makefile
@@ -0,0 +1,52 @@
+#
+# (C) Copyright 2009 Albert ARIBAUD <albrt.aribaud@free.fr>
+#
+# Based on original Kirkwood support which is
+# (C) Copyright 2009
+# Marvell Semiconductor <www.marvell.com>
+# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+#
+# See file CREDITS for list of people who contributed to this
+# project.
+#
+# 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; either version 2 of
+# the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+# MA 02110-1301 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB = $(obj)lib$(SOC).a
+
+COBJS-y = dram.o
+COBJS-y += cpu.o
+COBJS-y += mpp.o
+COBJS-y += timer.o
+
+SRCS := $(SOBJS:.o=.S) $(COBJS-y:.o=.c)
+OBJS := $(addprefix $(obj),$(SOBJS) $(COBJS-y))
+
+all: $(obj).depend $(LIB)
+
+$(LIB): $(OBJS)
+ $(AR) $(ARFLAGS) $@ $(OBJS)
+
+#########################################################################
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#########################################################################
diff --git a/cpu/arm926ejs/orion5x/cpu.c b/cpu/arm926ejs/orion5x/cpu.c
new file mode 100644
index 0000000..50ca724
--- /dev/null
+++ b/cpu/arm926ejs/orion5x/cpu.c
@@ -0,0 +1,339 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support which is
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#include <common.h>
+#include <netdev.h>
+#include <asm/cache.h>
+#include <u-boot/md5.h>
+#include <asm/arch/orion5x.h>
+#include <hush.h>
+
+#define BUFLEN 16
+
+void reset_cpu(unsigned long ignored)
+{
+ struct orion5x_cpu_registers *cpureg =
+ (struct orion5x_cpu_registers *)ORION5X_CPU_REG_BASE;
+
+ writel(readl(&cpureg->rstoutn_mask) | (1 << 2),
+ &cpureg->rstoutn_mask);
+ writel(readl(&cpureg->sys_soft_rst) | 1,
+ &cpureg->sys_soft_rst);
+ while (1) ;
+}
+
+/*
+ * Window Size
+ * Used with the Base register to set the address window size and location.
+ * Must be programmed from LSB to MSB as sequence of ones followed by
+ * sequence of zeros. The number of ones specifies the size of the window in
+ * 64 KByte granularity (e.g., a value of 0x00FF specifies 256 = 16 MByte).
+ * NOTE: A value of 0x0 specifies 64-KByte size.
+ */
+unsigned int orion5x_winctrl_calcsize(unsigned int sizeval)
+{
+ int i;
+ unsigned int j = 0;
+ u32 val = sizeval >> 1;
+
+ for (i = 0; val > 0x10000; i++) {
+ j |= (1 << i);
+ val = val >> 1;
+ }
+ return (0x0000ffff & j);
+}
+
+/*
+ * orion5x_config_adr_windows - Configure address Windows
+ *
+ * There are 8 address windows supported by Orion5x Soc to addess different
+ * devices. Each window can be configured for size, BAR and remap addr
+ * Below configuration is standard for most of the cases
+ *
+ * If remap function not used, remap_lo must be set as base
+ *
+ * Reference Documentation:
+ * Mbus-L to Mbus Bridge Registers Configuration.
+ * (Sec 25.1 and 25.3 of Datasheet)
+ */
+int orion5x_config_adr_windows(void)
+{
+ struct orion5x_win_registers *winregs =
+ (struct orion5x_win_registers *)ORION5X_CPU_WIN_BASE;
+
+ /* Window 0: PCIE MEM address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCIE_MEM, ORION5X_TARGET_PCIE,
+ ORION5X_ATTR_PCIE_MEM, ORION5X_WIN_ENABLE), &winregs[0].ctrl);
+ writel(ORION5X_DEFADR_PCIE_MEM, &winregs[0].base);
+ writel(ORION5X_DEFADR_PCIE_MEM_REMAP_LO, &winregs[0].remap_lo);
+ writel(ORION5X_DEFADR_PCIE_MEM_REMAP_HI, &winregs[0].remap_hi);
+
+ /* Window 1: PCIE IO address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCIE_IO, ORION5X_TARGET_PCIE,
+ ORION5X_ATTR_PCIE_IO, ORION5X_WIN_ENABLE), &winregs[1].ctrl);
+ writel(ORION5X_DEFADR_PCIE_IO, &winregs[1].base);
+ writel(ORION5X_DEFADR_PCIE_IO_REMAP_LO, &winregs[1].remap_lo);
+ writel(ORION5X_DEFADR_PCIE_IO_REMAP_HI, &winregs[1].remap_hi);
+
+ /* Window 2: PCI MEM address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCI_MEM, ORION5X_TARGET_PCI,
+ ORION5X_ATTR_PCI_MEM, ORION5X_WIN_ENABLE), &winregs[2].ctrl);
+ writel(ORION5X_DEFADR_PCI_MEM, &winregs[2].base);
+
+ /* Window 3: PCI IO address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCI_IO, ORION5X_TARGET_PCI,
+ ORION5X_ATTR_PCI_IO, ORION5X_WIN_ENABLE), &winregs[3].ctrl);
+ writel(ORION5X_DEFADR_PCI_IO, &winregs[3].base);
+
+ /* Window 4: DEV_CS0 address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS0, ORION5X_TARGET_DEVICE,
+ ORION5X_ATTR_DEV_CS0, ORION5X_WIN_ENABLE), &winregs[4].ctrl);
+ writel(ORION5X_DEFADR_DEV_CS0, &winregs[4].base);
+
+ /* Window 5: DEV_CS1 address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS1, ORION5X_TARGET_DEVICE,
+ ORION5X_ATTR_DEV_CS1, ORION5X_WIN_ENABLE), &winregs[5].ctrl);
+ writel(ORION5X_DEFADR_DEV_CS1, &winregs[5].base);
+
+ /* Window 6: DEV_CS2 address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS2, ORION5X_TARGET_DEVICE,
+ ORION5X_ATTR_DEV_CS2, ORION5X_WIN_ENABLE), &winregs[6].ctrl);
+ writel(ORION5X_DEFADR_DEV_CS2, &winregs[6].base);
+
+ /* Window 7: BOOT Memory address space */
+ writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_BOOTROM, ORION5X_TARGET_DEVICE,
+ ORION5X_ATTR_BOOTROM, ORION5X_WIN_ENABLE), &winregs[7].ctrl);
+ writel(ORION5X_DEFADR_BOOTROM, &winregs[7].base);
+
+ return 0;
+}
+
+/*
+ * orion5x_config_gpio - GPIO configuration
+ */
+void orion5x_config_gpio(u32 gpp_oe_val, u32 gpp_oe)
+{
+ struct orion5x_gpio_registers *gpioreg =
+ (struct orion5x_gpio_registers *)ORION5X_GPIO_BASE;
+
+ /* Init GPIOS to default values as per board requirement */
+ writel(gpp_oe_val, &gpioreg->dout);
+ writel(gpp_oe, &gpioreg->oe);
+}
+
+/*
+ * orion5x_config_mpp - Multi-Purpose Pins Functionality configuration
+ *
+ * Each MPP can be configured to different functionality through
+ * MPP control register, ref (sec 6.1 of Orion5x h/w specification)
+ *
+ * There are maximum 64 Multi-Pourpose Pins on Orion5x
+ * Each MPP functionality can be configuration by a 4bit value
+ * of MPP control reg, the value and associated functionality depends
+ * upon used SoC varient
+ */
+int orion5x_config_mpp(u32 mpp0_7, u32 mpp8_15, u32 mpp16_23)
+{
+ u32 *mppreg = (u32 *) ORION5X_MPP_BASE;
+
+ /* program mpp registers */
+ writel(mpp0_7, &mppreg[0]);
+ writel(mpp8_15, &mppreg[1]);
+ writel(mpp16_23, &mppreg[2]);
+ return 0;
+}
+
+/* Orion5x identification is done through PCIE space.
+ * As long as there is no Orion5x support for PCIE,
+ * We'll define the ID and revision PCIE registers
+ * here.
+ */
+#define PCIE_DEV_ID_OFF (ORION5X_REG_PCIE_BASE + 0x0000)
+#define PCIE_DEV_REV_OFF (ORION5X_REG_PCIE_BASE + 0x0008)
+u32 orion5x_device_id(void)
+{
+ return readl(PCIE_DEV_ID_OFF) >> 16;
+}
+
+u32 orion5x_device_rev(void)
+{
+ return readl(PCIE_DEV_REV_OFF) & 0xff;
+}
+
+#if defined(CONFIG_DISPLAY_CPUINFO)
+
+int print_cpuinfo(void)
+{
+ char dev_str[] = "0x0000";
+ char *dev_name = dev_str;
+ char rev_str[] = "0x00";
+ char *rev_name = rev_str;
+
+ u32 dev = orion5x_device_id();
+ u32 rev = orion5x_device_rev();
+
+ if (dev == MV88F5181_DEV_ID) {
+ dev_name = "MV88F5181";
+ if (rev == MV88F5181_REV_B1) {
+ rev_name = "B1";
+ } else if (rev == MV88F5181L_REV_A1) {
+ dev_name = "MV88F5181L";
+ rev_name = "A1";
+ } else if (rev == MV88F5181L_REV_A0) {
+ dev_name = "MV88F5181L";
+ rev_name = "A0";
+ } else {
+ sprintf(rev_str,"0x%02x", rev);
+ }
+ } else if (dev == MV88F5182_DEV_ID) {
+ dev_name = "MV88F5182";
+ if (rev == MV88F5182_REV_A2) {
+ rev_name = "A2";
+ } else {
+ sprintf(rev_str,"0x%02x", rev);
+ }
+ } else if (dev == MV88F5281_DEV_ID) {
+ dev_name = "MV88F5281";
+ if (rev == MV88F5281_REV_D2) {
+ rev_name = "D2";
+ } else if (rev == MV88F5281_REV_D1) {
+ rev_name = "D1";
+ } else if (rev == MV88F5281_REV_D0) {
+ rev_name = "D0";
+ } else {
+ sprintf(rev_str,"0x%02x", rev);
+ }
+ } else if (dev == MV88F6183_DEV_ID) {
+ dev_name = "MV88F6183";
+ if (rev == MV88F6183_REV_B0) {
+ rev_name = "B0";
+ } else {
+ sprintf(rev_str,"0x%02x", rev);
+ }
+ } else {
+ sprintf(dev_str,"0x%04x", rev);
+ sprintf(rev_str,"0x%02x", rev);
+ }
+
+ printf("SoC: Orion5x %s-%s\n", dev_name, rev_name);
+
+ return 0;
+}
+#endif /* CONFIG_DISPLAY_CPUINFO */
+
+#ifdef CONFIG_ARCH_CPU_INIT
+int arch_cpu_init(void)
+{
+ struct orion5x_cpu_registers *cpureg =
+ (struct orion5x_cpu_registers *)ORION5X_CPU_REG_BASE;
+
+ /* Linux expects` the internal registers to be at 0xf1000000 */
+ writel(ORION5X_REGS_PHY_BASE, ORION5X_OFFSET_REG);
+
+ /* Enable and invalidate L2 cache in write through mode */
+ writel(readl(&cpureg->l2_cfg) | 0x18, &cpureg->l2_cfg);
+ invalidate_l2_cache();
+
+ orion5x_config_adr_windows();
+
+#ifdef CONFIG_ORION5X_RGMII_PAD_1V8
+ /*
+ * Configures the I/O voltage of the pads connected to Egigabit
+ * Ethernet interface to 1.8V
+ * By defult it is set to 3.3V
+ */
+ reg = readl(ORION5X_REG_MPP_OUT_DRV_REG);
+ reg |= (1 << 7);
+ writel(reg, ORION5X_REG_MPP_OUT_DRV_REG);
+#endif
+#ifdef CONFIG_ORION5X_EGIGA_INIT
+ /*
+ * Set egiga port0/1 in normal functional mode
+ * This is required becasue on Orion5x by default ports are in reset mode
+ * OS egiga driver may not have provision to set them in normal mode
+ * and if u-boot is build without network support, network may fail@OS level
+ */
+ reg = readl(ORION5XGBE_PORT_SERIAL_CONTROL1_REG(0));
+ reg &= ~(1 << 4); /* Clear PortReset Bit */
+ writel(reg, (ORION5XGBE_PORT_SERIAL_CONTROL1_REG(0)));
+ reg = readl(ORION5XGBE_PORT_SERIAL_CONTROL1_REG(1));
+ reg &= ~(1 << 4); /* Clear PortReset Bit */
+ writel(reg, (ORION5XGBE_PORT_SERIAL_CONTROL1_REG(1)));
+#endif
+#ifdef CONFIG_ORION5X_PCIE_INIT
+ /*
+ * Enable PCI Express Port0
+ */
+ reg = readl(&cpureg->ctrl_stat);
+ reg |= (1 << 0); /* Set PEX0En Bit */
+ writel(reg, &cpureg->ctrl_stat);
+#endif
+ return 0;
+}
+#endif /* CONFIG_ARCH_CPU_INIT */
+
+/*
+ * SOC specific misc init
+ */
+#if defined(CONFIG_ARCH_MISC_INIT)
+int arch_misc_init(void)
+{
+ volatile u32 temp;
+
+ /*CPU streaming & write allocate */
+ temp = readfr_extra_feature_reg();
+ temp &= ~(1 << 28); /* disable wr alloc */
+ writefr_extra_feature_reg(temp);
+
+ temp = readfr_extra_feature_reg();
+ temp &= ~(1 << 29); /* streaming disabled */
+ writefr_extra_feature_reg(temp);
+
+ /* L2Cache settings */
+ temp = readfr_extra_feature_reg();
+ /* Disable L2C pre fetch - Set bit 24 */
+ temp |= (1 << 24);
+ /* enable L2C - Set bit 22 */
+ temp |= (1 << 22);
+ writefr_extra_feature_reg(temp);
+
+ icache_enable();
+ /* Change reset vector to address 0x0 */
+ temp = get_cr();
+ set_cr(temp & ~CR_V);
+
+ return 0;
+}
+#endif /* CONFIG_ARCH_MISC_INIT */
+
+#ifdef CONFIG_ORION5X_EGIGA
+int cpu_eth_init(bd_t *bis)
+{
+ orion5x_egiga_initialize(bis);
+ return 0;
+}
+#endif
diff --git a/cpu/arm926ejs/orion5x/dram.c b/cpu/arm926ejs/orion5x/dram.c
new file mode 100644
index 0000000..69aec26
--- /dev/null
+++ b/cpu/arm926ejs/orion5x/dram.c
@@ -0,0 +1,61 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support which is
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#include <config.h>
+#include <asm/arch/orion5x.h>
+
+#define ORION5X_REG_CPUCS_WIN_BAR(x) (ORION5X_REGISTER(0x1500) + (x * 0x08))
+#define ORION5X_REG_CPUCS_WIN_SZ(x) (ORION5X_REGISTER(0x1504) + (x * 0x08))
+/*
+ * orion5x_sdram_bar - reads SDRAM Base Address Register
+ */
+u32 orion5x_sdram_bar(enum memory_bank bank)
+{
+ u32 result = 0;
+ u32 enable = 0x01 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
+
+ if ((!enable) || (bank > BANK3))
+ return 0;
+
+ result = readl(ORION5X_REG_CPUCS_WIN_BAR(bank));
+ return result;
+}
+
+/*
+ * orion5x_sdram_bs - reads SDRAM Bank size
+ */
+u32 orion5x_sdram_bs(enum memory_bank bank)
+{
+ u32 result = 0;
+ u32 enable = 0x01 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
+
+ if ((!enable) || (bank > BANK3))
+ return 0;
+ result = 0xff000000 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
+ result += 0x01000000;
+ return result;
+}
diff --git a/cpu/arm926ejs/orion5x/mpp.c b/cpu/arm926ejs/orion5x/mpp.c
new file mode 100644
index 0000000..1b1e7fa
--- /dev/null
+++ b/cpu/arm926ejs/orion5x/mpp.c
@@ -0,0 +1,88 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support
+ *
+ * arch/arm/mach-orion5x/mpp.c
+ *
+ * MPP functions for Marvell Kirorion5xood SoCs
+ * Referenced from Linux kernel source
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+#include <common.h>
+#include <asm/arch/orion5x.h>
+#include <asm/arch/mpp.h>
+
+static u32 orion5x_variant(void)
+{
+ switch (orion5x_device_id()) {
+ case MV88F5181_DEV_ID:
+ return 0;
+ case MV88F5182_DEV_ID:
+ return MPP_F5182_MASK;
+ case MV88F5281_DEV_ID:
+ return MPP_F5281_MASK;
+ case MV88F6183_DEV_ID:
+ return 0;
+ default:
+ debug("MPP setup: unknown Orion5x variant\n");
+ return 0;
+ }
+}
+
+#define MPP_CTRL(i) (ORION5X_MPP_BASE + (i* 4))
+#define MPP_NR_REGS (1 + MPP_MAX/8)
+
+void orion5x_mpp_conf(u32 *mpp_list)
+{
+ u32 mpp_ctrl[MPP_NR_REGS];
+ unsigned int variant_mask;
+ int i;
+
+ variant_mask = orion5x_variant();
+ if (!variant_mask)
+ return;
+
+ debug( "initial MPP regs:");
+ for (i = 0; i < MPP_NR_REGS; i++) {
+ mpp_ctrl[i] = readl(MPP_CTRL(i));
+ debug(" %08x", mpp_ctrl[i]);
+ }
+ debug("\n");
+
+
+ while (*mpp_list) {
+ unsigned int num = MPP_NUM(*mpp_list);
+ unsigned int sel = MPP_SEL(*mpp_list);
+ int shift;
+
+ if (num > MPP_MAX) {
+ debug("orion5x_mpp_conf: invalid MPP "
+ "number (%u)\n", num);
+ continue;
+ }
+ if (!(*mpp_list & variant_mask)) {
+ debug("orion5x_mpp_conf: requested MPP%u config "
+ "unavailable on this hardware\n", num);
+ continue;
+ }
+
+ shift = (num & 7) << 2;
+ mpp_ctrl[num / 8] &= ~(0xf << shift);
+ mpp_ctrl[num / 8] |= sel << shift;
+
+ mpp_list++;
+ }
+
+ debug(" final MPP regs:");
+ for (i = 0; i < MPP_NR_REGS; i++) {
+ writel(mpp_ctrl[i], MPP_CTRL(i));
+ debug(" %08x", mpp_ctrl[i]);
+ }
+ debug("\n");
+
+}
diff --git a/cpu/arm926ejs/orion5x/timer.c b/cpu/arm926ejs/orion5x/timer.c
new file mode 100644
index 0000000..80fee75
--- /dev/null
+++ b/cpu/arm926ejs/orion5x/timer.c
@@ -0,0 +1,171 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support which is
+ * Copyright (C) Marvell International Ltd. and its affiliates
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#include <common.h>
+#include <asm/arch/orion5x.h>
+
+#define UBOOT_CNTR 0 /* counter to use for uboot timer */
+
+/* Timer reload and current value registers */
+struct orion5x_tmr_val {
+ u32 reload; /* Timer reload reg */
+ u32 val; /* Timer value reg */
+};
+
+/* Timer registers */
+struct orion5x_tmr_registers {
+ u32 ctrl; /* Timer control reg */
+ u32 pad[3];
+ struct orion5x_tmr_val tmr[2];
+ u32 wdt_reload;
+ u32 wdt_val;
+};
+
+struct orion5x_tmr_registers *orion5x_tmr_regs = (struct orion5x_tmr_registers *)ORION5X_TIMER_BASE;
+
+/*
+ * ARM Timers Registers Map
+ */
+#define CNTMR_CTRL_REG &orion5x_tmr_regs->ctrl
+#define CNTMR_RELOAD_REG(tmrnum) &orion5x_tmr_regs->tmr[tmrnum].reload
+#define CNTMR_VAL_REG(tmrnum) &orion5x_tmr_regs->tmr[tmrnum].val
+
+/*
+ * ARM Timers Control Register
+ * CPU_TIMERS_CTRL_REG (CTCR)
+ */
+#define CTCR_ARM_TIMER_EN_OFFS(cntr) (cntr * 2)
+#define CTCR_ARM_TIMER_EN_MASK(cntr) (1 << CTCR_ARM_TIMER_EN_OFFS)
+#define CTCR_ARM_TIMER_EN(cntr) (1 << CTCR_ARM_TIMER_EN_OFFS(cntr))
+#define CTCR_ARM_TIMER_DIS(cntr) (0 << CTCR_ARM_TIMER_EN_OFFS(cntr))
+
+#define CTCR_ARM_TIMER_AUTO_OFFS(cntr) ((cntr * 2) + 1)
+#define CTCR_ARM_TIMER_AUTO_MASK(cntr) (1 << 1)
+#define CTCR_ARM_TIMER_AUTO_EN(cntr) (1 << CTCR_ARM_TIMER_AUTO_OFFS(cntr))
+#define CTCR_ARM_TIMER_AUTO_DIS(cntr) (0 << CTCR_ARM_TIMER_AUTO_OFFS(cntr))
+
+/*
+ * ARM Timer\Watchdog Reload Register
+ * CNTMR_RELOAD_REG (TRR)
+ */
+#define TRG_ARM_TIMER_REL_OFFS 0
+#define TRG_ARM_TIMER_REL_MASK 0xffffffff
+
+/*
+ * ARM Timer\Watchdog Register
+ * CNTMR_VAL_REG (TVRG)
+ */
+#define TVR_ARM_TIMER_OFFS 0
+#define TVR_ARM_TIMER_MASK 0xffffffff
+#define TVR_ARM_TIMER_MAX 0xffffffff
+#define TIMER_LOAD_VAL 0xffffffff
+
+#define READ_TIMER (readl(CNTMR_VAL_REG(UBOOT_CNTR)) / \
+ (CONFIG_SYS_TCLK / 1000))
+
+static ulong timestamp;
+static ulong lastdec;
+
+void reset_timer_masked(void)
+{
+ /* reset time */
+ lastdec = READ_TIMER;
+ timestamp = 0;
+}
+
+ulong get_timer_masked(void)
+{
+ ulong now = READ_TIMER;
+
+ if (lastdec >= now) {
+ /* normal mode */
+ timestamp += lastdec - now;
+ } else {
+ /* we have an overflow ... */
+ timestamp += lastdec +
+ (TIMER_LOAD_VAL / (CONFIG_SYS_TCLK / 1000)) - now;
+ }
+ lastdec = now;
+
+ return timestamp;
+}
+
+void reset_timer(void)
+{
+ reset_timer_masked();
+}
+
+ulong get_timer(ulong base)
+{
+ return get_timer_masked() - base;
+}
+
+void set_timer(ulong t)
+{
+ timestamp = t;
+}
+
+void udelay(unsigned long usec)
+{
+ uint current;
+ ulong delayticks;
+
+ current = readl(CNTMR_VAL_REG(UBOOT_CNTR));
+ delayticks = (usec * (CONFIG_SYS_TCLK / 1000000));
+
+ if (current < delayticks) {
+ delayticks -= current;
+ while (readl(CNTMR_VAL_REG(UBOOT_CNTR)) < current) ;
+ while ((TIMER_LOAD_VAL - delayticks) <
+ readl(CNTMR_VAL_REG(UBOOT_CNTR))) ;
+ } else {
+ while (readl(CNTMR_VAL_REG(UBOOT_CNTR)) >
+ (current - delayticks)) ;
+ }
+}
+
+/*
+ * init the counter
+ */
+int timer_init(void)
+{
+ unsigned int cntmrctrl;
+
+ /* load value into timer */
+ writel(TIMER_LOAD_VAL, CNTMR_RELOAD_REG(UBOOT_CNTR));
+ writel(TIMER_LOAD_VAL, CNTMR_VAL_REG(UBOOT_CNTR));
+
+ /* enable timer in auto reload mode */
+ cntmrctrl = readl(CNTMR_CTRL_REG);
+ cntmrctrl |= CTCR_ARM_TIMER_EN(UBOOT_CNTR);
+ cntmrctrl |= CTCR_ARM_TIMER_AUTO_EN(UBOOT_CNTR);
+ writel(cntmrctrl, CNTMR_CTRL_REG);
+
+ /* init the timestamp and lastdec value */
+ reset_timer_masked();
+
+ return 0;
+}
diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index dd5f332..18686a2 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial/serial.c
@@ -30,6 +30,9 @@
#ifdef CONFIG_KIRKWOOD
#include <asm/arch/kirkwood.h>
#endif
+#ifdef CONFIG_ORION5X
+#include <asm/arch/orion5x.h>
+#endif
#if defined (CONFIG_SERIAL_MULTI)
#include <serial.h>
diff --git a/include/asm-arm/arch-orion5x/88f5182.h b/include/asm-arm/arch-orion5x/88f5182.h
new file mode 100644
index 0000000..1dfdb24
--- /dev/null
+++ b/include/asm-arm/arch-orion5x/88f5182.h
@@ -0,0 +1,40 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood 88F6182 support which is
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * Header file for Feroceon CPU core 88F5182 SOC.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#ifndef _CONFIG_88F5182_H
+#define _CONFIG_88F5182_H
+
+/* SOC specific definations */
+#define F88F5182_REGS_PHYS_BASE 0xf1000000
+#define ORION5X_REGS_PHY_BASE F88F5182_REGS_PHYS_BASE
+
+/* TCLK Core Clock defination */
+#define CONFIG_SYS_TCLK 166000000 /* 166MHz */
+
+#endif /* _CONFIG_88F5182_H */
diff --git a/include/asm-arm/arch-orion5x/cpu.h b/include/asm-arm/arch-orion5x/cpu.h
new file mode 100644
index 0000000..7078d2e
--- /dev/null
+++ b/include/asm-arm/arch-orion5x/cpu.h
@@ -0,0 +1,213 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirorion5x_ood support which is
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#ifndef _ORION5X_CPU_H
+#define _ORION5X_CPU_H
+
+#include <asm/system.h>
+
+#ifndef __ASSEMBLY__
+
+#define ORION5X_CPU_WIN_CTRL_DATA(size, target, attr, en) (en | (target << 4) \
+ | (attr << 8) | (orion5x_winctrl_calcsize(size) << 16))
+
+#define ORION5XGBE_PORT_SERIAL_CONTROL1_REG(_x) \
+ ((_x ? ORION5X_EGIGA0_BASE : ORION5X_EGIGA1_BASE) + 0x44c)
+
+#define ORION5X_REG_DEVICE_ID (ORION5X_MPP_BASE + 0x34)
+#define ORION5X_REG_SYSRST_CNT (ORION5X_MPP_BASE + 0x50)
+#define SYSRST_CNT_1SEC_VAL (25*1000000)
+#define ORION5X_REG_MPP_OUT_DRV_REG (ORION5X_MPP_BASE + 0xE0)
+
+enum memory_bank {
+ BANK0,
+ BANK1,
+ BANK2,
+ BANK3
+};
+
+enum orion5x_cpu_winen {
+ ORION5X_WIN_DISABLE,
+ ORION5X_WIN_ENABLE
+};
+
+enum orion5x_cpu_target {
+ ORION5X_TARGET_DRAM = 0,
+ ORION5X_TARGET_DEVICE = 1,
+ ORION5X_TARGET_PCI = 3,
+ ORION5X_TARGET_PCIE = 4,
+ ORION5X_TARGET_SASRAM = 9
+};
+
+enum orion5x_cpu_attrib {
+ ORION5X_ATTR_DRAM_CS0 = 0x0e,
+ ORION5X_ATTR_DRAM_CS1 = 0x0d,
+ ORION5X_ATTR_DRAM_CS2 = 0x0b,
+ ORION5X_ATTR_DRAM_CS3 = 0x07,
+ ORION5X_ATTR_PCI_MEM = 0x59,
+ ORION5X_ATTR_PCI_IO = 0x51,
+ ORION5X_ATTR_PCIE_MEM = 0x59,
+ ORION5X_ATTR_PCIE_IO = 0x51,
+ ORION5X_ATTR_SASRAM = 0x00,
+ ORION5X_ATTR_DEV_CS0 = 0x1e,
+ ORION5X_ATTR_DEV_CS1 = 0x1d,
+ ORION5X_ATTR_DEV_CS2 = 0x1b,
+ ORION5X_ATTR_BOOTROM = 0x0f
+};
+
+/*
+ * Default Device Address MAP BAR values
+ */
+#define ORION5X_DEFADR_PCIE_MEM 0x90000000
+#define ORION5X_DEFADR_PCIE_MEM_REMAP_LO 0x90000000
+#define ORION5X_DEFADR_PCIE_MEM_REMAP_HI 0
+#define ORION5X_DEFSZ_PCIE_MEM (128*1024*1024)
+
+#define ORION5X_DEFADR_PCIE_IO 0xf0000000
+#define ORION5X_DEFADR_PCIE_IO_REMAP_LO 0x90000000
+#define ORION5X_DEFADR_PCIE_IO_REMAP_HI 0
+#define ORION5X_DEFSZ_PCIE_IO (64*1024)
+
+#define ORION5X_DEFADR_PCI_MEM 0x98000000
+#define ORION5X_DEFSZ_PCI_MEM (128*1024*1024)
+
+#define ORION5X_DEFADR_PCI_IO 0xf0100000
+#define ORION5X_DEFSZ_PCI_IO (64*1024)
+
+#define ORION5X_DEFADR_DEV_CS0 0xfa000000
+#define ORION5X_DEFSZ_DEV_CS0 (2*1024*1024)
+
+#define ORION5X_DEFADR_DEV_CS1 0xf8000000
+#define ORION5X_DEFSZ_DEV_CS1 (32*1024*1024)
+
+#define ORION5X_DEFADR_DEV_CS2 0xfa800000
+#define ORION5X_DEFSZ_DEV_CS2 (1*1024*1024)
+
+#define ORION5X_DEFADR_BOOTROM 0xFFF80000
+#define ORION5X_DEFSZ_BOOTROM (512*1024)
+
+/* Constants for known devices and revisions
+ *
+ */
+
+/* Orion-1 (88F5181) and Orion-VoIP (88F5181L) */
+#define MV88F5181_DEV_ID 0x5181
+#define MV88F5181_REV_B1 3
+#define MV88F5181L_REV_A0 8
+#define MV88F5181L_REV_A1 9
+/* Orion-NAS (88F5182) */
+#define MV88F5182_DEV_ID 0x5182
+#define MV88F5182_REV_A2 2
+/* Orion-2 (88F5281) */
+#define MV88F5281_DEV_ID 0x5281
+#define MV88F5281_REV_D0 4
+#define MV88F5281_REV_D1 5
+#define MV88F5281_REV_D2 6
+/* Orion-1-90 (88F6183) */
+#define MV88F6183_DEV_ID 0x6183
+#define MV88F6183_REV_B0 3
+
+/*
+ * read feroceon/sheeva core extra feature register
+ * using co-proc instruction
+ */
+static inline unsigned int readfr_extra_feature_reg(void)
+{
+ unsigned int val;
+ asm volatile ("mrc p15, 1, %0, c15, c1, 0 @ readfr exfr":"=r"
+ (val)::"cc");
+ return val;
+}
+
+/*
+ * write feroceon/sheeva core extra feature register
+ * using co-proc instruction
+ */
+static inline void writefr_extra_feature_reg(unsigned int val)
+{
+ asm volatile ("mcr p15, 1, %0, c15, c1, 0 @ writefr exfr"::"r"
+ (val):"cc");
+ isb();
+}
+
+/*
+ * MBus-L to Mbus Bridge Registers
+ * Ref: Datasheet sec:A.3
+ */
+struct orion5x_win_registers {
+ u32 ctrl;
+ u32 base;
+ u32 remap_lo;
+ u32 remap_hi;
+};
+
+/*
+ * CPU control and status Registers
+ * Ref: Datasheet sec:A.3.2
+ */
+struct orion5x_cpu_registers {
+ u32 config; /*0x20100 */
+ u32 ctrl_stat; /*0x20104 */
+ u32 rstoutn_mask; /* 0x20108 */
+ u32 sys_soft_rst; /* 0x2010C */
+ u32 ahb_mbus_cause_irq; /* 0x20110 */
+ u32 ahb_mbus_mask_irq; /* 0x20114 */
+ u32 pad1[2];
+ u32 ftdll_config; /* 0x20120 */
+ u32 pad2;
+ u32 l2_cfg; /* 0x20128 */
+};
+
+/*
+ * GPIO Registers
+ * Ref: Datasheet sec:A.19
+ */
+struct orion5x_gpio_registers {
+ u32 dout;
+ u32 oe;
+ u32 blink_en;
+ u32 din_pol;
+ u32 din;
+ u32 irq_cause;
+ u32 irq_mask;
+ u32 irq_level;
+};
+
+/*
+ * functions
+ */
+void reset_cpu(unsigned long ignored);
+u32 orion5x_device_id(void);
+u32 orion5x_device_rev(void);
+unsigned int orion5x_sdram_bar(enum memory_bank bank);
+unsigned int orion5x_sdram_bs(enum memory_bank bank);
+int orion5x_config_adr_windows(void);
+void orion5x_config_gpio(unsigned int gpp_oe_val, unsigned int gpp_oe);
+int orion5x_config_mpp(unsigned int mpp0_7, unsigned int mpp8_15, unsigned int mpp16_23);
+unsigned int orion5x_winctrl_calcsize(unsigned int sizeval);
+#endif /* __ASSEMBLY__ */
+#endif /* _ORION5X_CPU_H */
diff --git a/include/asm-arm/arch-orion5x/gpio.h b/include/asm-arm/arch-orion5x/gpio.h
new file mode 100644
index 0000000..40fe2e0
--- /dev/null
+++ b/include/asm-arm/arch-orion5x/gpio.h
@@ -0,0 +1,53 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on arch/asm-arm/mach-kirkwood/include/mach/gpio.h
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+/*
+ * Based on (mostly copied from) plat-orion based Linux 2.6 kernel driver.
+ * Removed kernel level irq handling. Took some macros from kernel to
+ * allow build.
+ *
+ * Dieter Kiermaier dk-arm-linux at gmx.de
+ */
+
+#ifndef __ORION5X_GPIO_H
+#define __ORION5X_GPIO_H
+
+/* got from kernel include/linux/bitops.h */
+#define BITS_PER_BYTE 8
+#define BITS_TO_LONGS(nr) DIV_ROUND_UP(nr, BITS_PER_BYTE * sizeof(long))
+
+#define GPIO_MAX 50
+#define GPIO_OFF(pin) (((pin) >> 5) ? 0x0040 : 0x0000)
+#define GPIO_OUT(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x00)
+#define GPIO_IO_CONF(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x04)
+#define GPIO_BLINK_EN(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x08)
+#define GPIO_IN_POL(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x0c)
+#define GPIO_DATA_IN(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x10)
+#define GPIO_EDGE_CAUSE(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x14)
+#define GPIO_EDGE_MASK(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x18)
+#define GPIO_LEVEL_MASK(pin) (ORION5X_GPIO0_BASE + GPIO_OFF(pin) + 0x1c)
+
+/*
+ * Kirorion5xood-specific GPIO API
+ */
+
+void orion5x_gpio_set_valid(unsigned pin, int mode);
+int orion5x_gpio_is_valid(unsigned pin, int mode);
+int orion5x_gpio_direction_input(unsigned pin);
+int orion5x_gpio_direction_output(unsigned pin, int value);
+int orion5x_gpio_get_value(unsigned pin);
+void orion5x_gpio_set_value(unsigned pin, int value);
+void orion5x_gpio_set_blink(unsigned pin, int blink);
+void orion5x_gpio_set_unused(unsigned pin);
+
+#define GPIO_INPUT_OK (1 << 0)
+#define GPIO_OUTPUT_OK (1 << 1)
+
+#endif
diff --git a/include/asm-arm/arch-orion5x/mpp.h b/include/asm-arm/arch-orion5x/mpp.h
new file mode 100644
index 0000000..a03eb8e
--- /dev/null
+++ b/include/asm-arm/arch-orion5x/mpp.h
@@ -0,0 +1,122 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support which is
+ * Copyright 2009: Marvell Technology Group Ltd.
+ *
+ * This file is licensed under the terms of the GNU General Public
+ * License version 2. This program is licensed "as is" without any
+ * warranty of any kind, whether express or implied.
+ */
+
+#ifndef __ORION5X_MPP_H
+#define __ORION5X_MPP_H
+
+#define MPP(_num, _sel, _in, _out, _F5182, _F5281) ( \
+ /* MPP number */ ((_num) & 0xff) | \
+ /* MPP select value */ (((_sel) & 0xf) << 8) | \
+ /* may be input signal */ ((!!(_in)) << 12) | \
+ /* may be output signal */ ((!!(_out)) << 13) | \
+ /* available on F5182 */ ((!!(_F5182)) << 14) | \
+ /* available on F5281 */ ((!!(_F5182)) << 15))
+
+#define MPP_NUM(x) ((x) & 0xff)
+#define MPP_SEL(x) (((x) >> 8) & 0xf)
+
+ /* num sel i o 5182 5281 */
+
+#define MPP_INPUT_MASK MPP( 0, 0x0, 1, 0, 0, 0 )
+#define MPP_OUTPUT_MASK MPP( 0, 0x0, 0, 1, 0, 0 )
+
+#define MPP_F5182_MASK MPP( 0, 0x0, 0, 0, 1, 0 )
+#define MPP_F5281_MASK MPP( 0, 0x0, 0, 0, 0, 1 )
+
+#define MPP0_PEX_RST_OUTn MPP( 0, 0x0, 0, 1, 1, 1 )
+#define MPP0_PCI_REQ2n MPP( 0, 0x2, 1, 0, 1, 1 )
+#define MPP0_GPIO MPP( 0, 0x3, 1, 1, 1, 1 )
+
+#define MPP1_GPIO MPP( 1, 0x0, 1, 1, 1, 1 )
+#define MPP1_PCI_GNT2n MPP( 1, 0x2, 0, 1, 1, 1 )
+
+#define MPP2_GPIO MPP( 2, 0x0, 1, 1, 1, 1 )
+#define MPP2_PCI_REQ3n MPP( 2, 0x2, 1, 0, 1, 1 )
+#define MPP2_PMEn MPP( 2, 0x3, 0, 1, 1, 1 )
+
+#define MPP3_GPIO MPP( 3, 0x0, 1, 1, 1, 1 )
+#define MPP3_PCI_GNT3nO MPP( 3, 0x2, 0, 1, 1, 1 )
+
+#define MPP4_GPIO MPP( 4, 0x0, 1, 1, 1, 1 )
+#define MPP4_PCI_REQ4n MPP( 4, 0x2, 1, 0, 1, 1 )
+#define MPP4_BOOT_NAND_FLASH_REn MPP( 4, 0x4, 0, 0, 1, 1 )
+#define MPP4_SATA0_PRESENTn MPP( 4, 0x5, 0, 1, 1, 0 )
+
+#define MPP5_GPIO MPP( 5, 0x0, 1, 1, 1, 1 )
+#define MPP5_PCI_GNT4n MPP( 5, 0x2, 0, 1, 1, 1 )
+#define MPP5_BOOT_NAND_FLASH_WEn MPP( 5, 0x4, 0, 0, 1, 1 )
+#define MPP5_SATA1_PRESENTn MPP( 5, 0x5, 0, 1, 1, 0 )
+
+#define MPP6_GPIO MPP( 6, 0x0, 1, 1, 1, 1 )
+#define MPP6_PCI_REQ5n MPP( 6, 0x2, 1, 0, 1, 1 )
+#define MPP6_NAND_FLASH_RE0n MPP( 6, 0x4, 0, 0, 1, 1 )
+#define MPP6_SATA0_ACTn MPP( 6, 0x5, 0, 1, 1, 0 )
+
+#define MPP7_GPIO MPP( 7, 0x0, 1, 1, 1, 1 )
+#define MPP7_PCI_GNT5n MPP( 7, 0x2, 0, 1, 1, 1 )
+#define MPP7_NAND_FLASH_WE0n MPP( 7, 0x4, 0, 0, 1, 1 )
+#define MPP7_SATA1_ACTn MPP( 7, 0x5, 0, 1, 1, 0 )
+
+#define MPP8_GPIO MPP( 8, 0x0, 1, 1, 1, 1 )
+#define MPP8_GE_COL MPP( 8, 0x1, 1, 0, 1, 1 )
+
+#define MPP9_GPIO MPP( 9, 0x0, 1, 1, 1, 1 )
+#define MPP9_GE_RXERR MPP( 9, 0x1, 1, 0, 1, 1 )
+
+#define MPP10_GPIO MPP( 10, 0x0, 1, 1, 1, 1 )
+#define MPP10_GE_CRS MPP( 10, 0x1, 1, 0, 1, 1 )
+
+#define MPP11_GPIO MPP( 11, 0x0, 1, 1, 1, 1 )
+#define MPP11_GE_TXERR MPP( 11, 0x1, 0, 1, 1, 1 )
+
+#define MPP12_GPIO MPP( 12, 0x0, 1, 1, 1, 1 )
+#define MPP12_GE_TXD4 MPP( 12, 0x1, 0, 1, 1, 1 )
+#define MPP12_NAND_FLASH_RE1n MPP( 12, 0x4, 1, 1, 1, 1 )
+#define MPP12_SATA0_PRESENTn MPP( 12, 0x5, 0, 1, 1, 0 )
+
+#define MPP13_GPIO MPP( 13, 0x0, 1, 1, 1, 1 )
+#define MPP13_GE_TXD5 MPP( 13, 0x1, 0, 1, 1, 1 )
+#define MPP13_NAND_FLASH_WE1n MPP( 13, 0x4, 0, 1, 1, 1 )
+#define MPP13_SATA1_PRESENTn MPP( 13, 0x5, 0, 1, 1, 0 )
+
+#define MPP14_GPIO MPP( 14, 0x0, 1, 1, 1, 1 )
+#define MPP14_GE_TXD6 MPP( 14, 0x1, 0, 1, 1, 1 )
+#define MPP14_NAND_FLASH_RE2n MPP( 14, 0x4, 1, 1, 1, 1 )
+#define MPP14_SATA0_ACTn MPP( 14, 0x5, 0, 1, 1, 0 )
+
+#define MPP15_GPIO MPP( 15, 0x0, 1, 1, 1, 1 )
+#define MPP15_GE_TXD7 MPP( 15, 0x1, 0, 1, 1, 1 )
+#define MPP15_NAND_FLASH_WE2n MPP( 15, 0x4, 0, 1, 1, 1 )
+#define MPP15_SATA1_ACTn MPP( 15, 0x5, 0, 1, 1, 0 )
+
+#define MPP16_UART1_RXD MPP( 16, 0x0, 1, 0, 1, 1 )
+#define MPP16_GE_RXD4 MPP( 16, 0x1, 1, 0, 1, 1 )
+#define MPP16_BOOT_NAND_FLASH_REn MPP( 16, 0x4, 0, 0, 1, 1 )
+#define MPP16_GPIO MPP( 16, 0x5, 1, 1, 1, 0 )
+
+#define MPP17_UART1_TXD MPP( 17, 0x0, 0, 1, 1, 1 )
+#define MPP17_GE_RXD5 MPP( 17, 0x1, 1, 0, 1, 1 )
+#define MPP17_BOOT_NAND_FLASH_WEn MPP( 17, 0x4, 0, 0, 1, 1 )
+#define MPP17_GPIO MPP( 17, 0x5, 1, 1, 1, 0 )
+
+#define MPP18_UART1_CTSn MPP( 18, 0x0, 1, 0, 1, 1 )
+#define MPP18_GE_RXD6 MPP( 18, 0x1, 1, 0, 1, 1 )
+#define MPP18_GPIO MPP( 18, 0x5, 1, 1, 1, 0 )
+
+#define MPP19_UART1_RTSn MPP( 19, 0x0, 0, 1, 1, 1 )
+#define MPP19_GE_RXD7 MPP( 19, 0x1, 1, 0, 1, 1 )
+#define MPP19_GPIO MPP( 19, 0x5, 1, 1, 1, 0 )
+
+#define MPP_MAX 19
+
+void orion5x_mpp_conf(unsigned int *mpp_list);
+
+#endif
diff --git a/include/asm-arm/arch-orion5x/orion5x.h b/include/asm-arm/arch-orion5x/orion5x.h
new file mode 100644
index 0000000..76cb927
--- /dev/null
+++ b/include/asm-arm/arch-orion5x/orion5x.h
@@ -0,0 +1,67 @@
+/*
+ * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
+ *
+ * Based on original Kirkwood support which is
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * Header file for Marvell's Orion SoC with Feroceon CPU core.
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#ifndef _ASM_ARCH_ORION5X_H
+#define _ASM_ARCH_ORION5X_H
+
+#ifndef __ASSEMBLY__
+#include <asm/types.h>
+#include <asm/io.h>
+#endif /* __ASSEMBLY__ */
+
+#if defined (CONFIG_FEROCEON)
+#include <asm/arch/cpu.h>
+
+/* SOC specific definations */
+#define INTREG_BASE 0xd0000000
+#define ORION5X_REGISTER(x) (ORION5X_REGS_PHY_BASE + x)
+#define ORION5X_OFFSET_REG (INTREG_BASE + 0x20080)
+
+/* Documented registers */
+#define ORION5X_TWSI_BASE (ORION5X_REGISTER(0x11000))
+#define ORION5X_UART0_BASE (ORION5X_REGISTER(0x12000))
+#define ORION5X_UART1_BASE (ORION5X_REGISTER(0x12100))
+#define ORION5X_MPP_BASE (ORION5X_REGISTER(0x10000))
+#define ORION5X_GPIO_BASE (ORION5X_REGISTER(0x10100))
+#define ORION5X_CPU_WIN_BASE (ORION5X_REGISTER(0x20000))
+#define ORION5X_CPU_REG_BASE (ORION5X_REGISTER(0x20100))
+#define ORION5X_TIMER_BASE (ORION5X_REGISTER(0x20300))
+#define ORION5X_REG_PCI_BASE (ORION5X_REGISTER(0x30000))
+#define ORION5X_REG_PCIE_BASE (ORION5X_REGISTER(0x40000))
+#define ORION5X_USB20_PORT0_BASE (ORION5X_REGISTER(0x50000))
+#define ORION5X_USB20_PORT1_BASE (ORION5X_REGISTER(0xA0000))
+#define ORION5X_EGIGA_BASE (ORION5X_REGISTER(0x72000))
+
+#if defined (CONFIG_88F5182)
+#include <asm/arch/88f5182.h>
+#else
+#error "SOC Name not defined"
+#endif
+#endif /* CONFIG_FEROCEON */
+#endif /* _ASM_ARCH_ORION5X_H */
diff --git a/include/configs/edminiv2.h b/include/configs/edminiv2.h
new file mode 100644
index 0000000..a8036dd
--- /dev/null
+++ b/include/configs/edminiv2.h
@@ -0,0 +1,158 @@
+/*
+ * (C) Copyright 2009
+ * Marvell Semiconductor <www.marvell.com>
+ * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
+ *
+ * See file CREDITS for list of people who contributed to this
+ * project.
+ *
+ * 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; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
+ * MA 02110-1301 USA
+ */
+
+#ifndef _CONFIG_EDMINIV2_H
+#define _CONFIG_EDMINIV2_H
+
+/*
+ * Version number information
+ */
+#define CONFIG_IDENT_STRING " Lacie ED Mini V2"
+
+/*
+ * High Level Configuration Options (easy to change)
+ */
+#define CONFIG_MARVELL 1
+#define CONFIG_ARM926EJS 1 /* Basic Architecture */
+#define CONFIG_FEROCEON 1 /* CPU Core subversion */
+#define CONFIG_ORION5X 1 /* SOC Family Name */
+#define CONFIG_88F5182 1 /* SOC Name */
+#define CONFIG_MACH_EDMINIV2 1 /* Machine type */
+
+#define CONFIG_SKIP_LOWLEVEL_INIT /* disable board lowlevel_init */
+
+/*
+ * CLKs configurations
+ */
+#define CONFIG_SYS_HZ 1000
+
+/*
+ * NS16550 Configuration
+ */
+#define CONFIG_SYS_NS16550
+#define CONFIG_SYS_NS16550_SERIAL
+#define CONFIG_SYS_NS16550_REG_SIZE (-4)
+#define CONFIG_SYS_NS16550_CLK CONFIG_SYS_TCLK
+#define CONFIG_SYS_NS16550_COM1 ORION5X_UART0_BASE
+
+/*
+ * Serial Port configuration
+ * The following definitions let you select what serial you want to use
+ * for your console driver.
+ */
+
+#define CONFIG_CONS_INDEX 1 /*Console on UART0 */
+#define CONFIG_BAUDRATE 115200
+#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, \
+ 115200,230400, 460800, 921600 }
+/*
+ * FLASH configuration
+ */
+
+#define CONFIG_SYS_FLASH_CFI
+#define CONFIG_FLASH_CFI_DRIVER
+#define CONFIG_FLASH_CFI_WIDTH FLASH_CFI_8BIT
+#define CONFIG_FLASH_CFI_LEGACY
+
+//#define CONFIG_SYS_FLASH_LEGACY_512Kx8
+#define CONFIG_SYS_MAX_FLASH_BANKS 1 /* max num of flash banks */
+#define CONFIG_SYS_MAX_FLASH_SECT 11 /* max num of sects on one chip */
+#define CONFIG_SYS_FLASH_BASE 0xfff80000
+#define CONFIG_SYS_FLASH_SECTSZ {16384,8192,8192,32768,65536,65536,65536,65536,65536,65536,65536}
+
+/* auto boot */
+#define CONFIG_BOOTDELAY 3 /* default enable autoboot */
+
+/*
+ * For booting Linux, the board info and command line data
+ * have to be in the first 8 MB of memory, since this is
+ * the maximum mapped by the Linux kernel during initialization.
+ */
+#define CONFIG_CMDLINE_TAG 1 /* enable passing of ATAGs */
+#define CONFIG_INITRD_TAG 1 /* enable INITRD tag */
+#define CONFIG_SETUP_MEMORY_TAGS 1 /* enable memory tag */
+
+#define CONFIG_SYS_PROMPT "Marvell>> " /* Command Prompt */
+#define CONFIG_SYS_CBSIZE 1024 /* Console I/O Buff Size */
+#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE \
+ +sizeof(CONFIG_SYS_PROMPT) + 16) /* Print Buff */
+/*
+ * Commands configuration
+ */
+/*
+ * Disabling many default commands for staggered bring-up
+ */
+#if 0
+#include <config_cmd_default.h>
+#else
+#define CONFIG_CMD_BDI /* bdinfo */
+#define CONFIG_CMD_CONSOLE /* coninfo */
+#define CONFIG_CMD_ECHO /* echo arguments */
+#define CONFIG_CMD_EDITENV /* editenv */
+#define CONFIG_CMD_IMI /* iminfo */
+#define CONFIG_CMD_ITEST /* Integer (and string) test */
+#define CONFIG_CMD_FLASH /* flinfo, erase, protect */
+#define CONFIG_CMD_IMLS /* List all found images */
+#define CONFIG_CMD_LOADB /* loadb */
+#define CONFIG_CMD_LOADS /* loads */
+#define CONFIG_CMD_MEMORY /* md mm nm mw cp cmp crc base loop mtest */
+#define CONFIG_CMD_MISC /* Misc functions like sleep etc*/
+#define CONFIG_CMD_RUN /* run command in env variable */
+#define CONFIG_CMD_SAVEENV /* saveenv */
+#define CONFIG_CMD_SOURCE /* "source" command support */
+#endif
+#define CONFIG_CMD_AUTOSCRIPT
+#define CONFIG_CMD_ENV
+
+/*
+ * Environment variables configurations
+ */
+#define CONFIG_ENV_IS_IN_FLASH 1
+#define CONFIG_ENV_SECT_SIZE 0x2000 /* 16K */
+#define CONFIG_ENV_SIZE 0x2000
+#define CONFIG_ENV_OFFSET 0x4000 /* env starts here */
+
+/*
+ * Size of malloc() pool
+ */
+#define CONFIG_SYS_MALLOC_LEN (1024 * 128) /* 128kB for malloc() */
+/* size in bytes reserved for initial data */
+#define CONFIG_SYS_GBL_DATA_SIZE 128
+
+/*
+ * Other required minimal configurations
+ */
+#define CONFIG_CONSOLE_INFO_QUIET /* some code reduction */
+#define CONFIG_ARCH_CPU_INIT /* call arch_cpu_init() */
+#define CONFIG_ARCH_MISC_INIT /* call arch_misc_init() */
+#define CONFIG_DISPLAY_CPUINFO /* Display cpu info */
+#define CONFIG_NR_DRAM_BANKS 4
+#define CONFIG_STACKSIZE 0x00100000 /* regular stack- 1M */
+#define CONFIG_SYS_LOAD_ADDR 0x00800000 /* default load adr- 8M */
+#define CONFIG_SYS_MEMTEST_START 0x00400000 /* 4M */
+#define CONFIG_SYS_MEMTEST_END 0x007fffff /*(_8M -1) */
+#define CONFIG_SYS_RESET_ADDRESS 0xffff0000 /* Rst Vector Adr */
+#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
+
+#endif /* _CONFIG_EDMINIV2_H */
--
1.6.4.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
2009-11-14 1:01 [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board Albert Aribaud
@ 2009-11-14 2:21 ` Tom
2009-11-14 6:29 ` Prafulla Wadaskar
1 sibling, 0 replies; 6+ messages in thread
From: Tom @ 2009-11-14 2:21 UTC (permalink / raw)
To: u-boot
Albert Aribaud wrote:
> This patch adds initial u-boot support for the Marvell orion5x
> SoC and the LAcie ED Mini V2 board--support is limited to serial,
> flash and environment. Further support will be added as devices
> common with kirkwood are made useable across SoCs.
>
> Signed-off-by: Albert Aribaud <albert.aribaud@free.fr>
> ---
> Makefile | 3 +
> board/Marvell/edminiv2/Makefile | 51 +++++
> board/Marvell/edminiv2/config.mk | 25 +++
> board/Marvell/edminiv2/edminiv2.c | 165 ++++++++++++++++
> board/Marvell/edminiv2/edminiv2.h | 39 ++++
> cpu/arm926ejs/orion5x/Makefile | 52 +++++
> cpu/arm926ejs/orion5x/cpu.c | 339 ++++++++++++++++++++++++++++++++
> cpu/arm926ejs/orion5x/dram.c | 61 ++++++
> cpu/arm926ejs/orion5x/mpp.c | 88 ++++++++
> cpu/arm926ejs/orion5x/timer.c | 171 ++++++++++++++++
> drivers/serial/serial.c | 3 +
> include/asm-arm/arch-orion5x/88f5182.h | 40 ++++
> include/asm-arm/arch-orion5x/cpu.h | 213 ++++++++++++++++++++
> include/asm-arm/arch-orion5x/gpio.h | 53 +++++
> include/asm-arm/arch-orion5x/mpp.h | 122 ++++++++++++
> include/asm-arm/arch-orion5x/orion5x.h | 67 +++++++
> include/configs/edminiv2.h | 158 +++++++++++++++
> 17 files changed, 1650 insertions(+), 0 deletions(-)
> create mode 100644 board/Marvell/edminiv2/Makefile
> create mode 100644 board/Marvell/edminiv2/config.mk
> create mode 100644 board/Marvell/edminiv2/edminiv2.c
> create mode 100644 board/Marvell/edminiv2/edminiv2.h
> create mode 100644 cpu/arm926ejs/orion5x/Makefile
> create mode 100644 cpu/arm926ejs/orion5x/cpu.c
> create mode 100644 cpu/arm926ejs/orion5x/dram.c
> create mode 100644 cpu/arm926ejs/orion5x/mpp.c
> create mode 100644 cpu/arm926ejs/orion5x/timer.c
> create mode 100644 include/asm-arm/arch-orion5x/88f5182.h
> create mode 100644 include/asm-arm/arch-orion5x/cpu.h
> create mode 100644 include/asm-arm/arch-orion5x/gpio.h
> create mode 100644 include/asm-arm/arch-orion5x/mpp.h
> create mode 100644 include/asm-arm/arch-orion5x/orion5x.h
> create mode 100644 include/configs/edminiv2.h
For this pass, I ran the linux kernel checkpatch.pl against
your patch. I get
total: 180 errors, 26 warnings, 1662 lines checked
In general, the patch should have no errors or warnings.
Please fix the checkpatch issues.
Tom
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
2009-11-14 1:01 [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board Albert Aribaud
2009-11-14 2:21 ` Tom
@ 2009-11-14 6:29 ` Prafulla Wadaskar
2009-11-14 9:22 ` Albert ARIBAUD
1 sibling, 1 reply; 6+ messages in thread
From: Prafulla Wadaskar @ 2009-11-14 6:29 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: u-boot-bounces at lists.denx.de
> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
> Sent: Saturday, November 14, 2009 6:32 AM
> To: U-Boot at lists.denx.de
> Subject: [U-Boot] [PATCH] Initial support for Orion5x SoC and
> EDMini board
>
> This patch adds initial u-boot support for the Marvell orion5x
> SoC and the LAcie ED Mini V2 board--support is limited to serial,
> flash and environment. Further support will be added as devices
> common with kirkwood are made useable across SoCs.
>
> Signed-off-by: Albert Aribaud <albert.aribaud@free.fr>
> ---
Hi Albert
Please break this patch in to small pieces
> Makefile | 3 +
> board/Marvell/edminiv2/Makefile | 51 +++++
> board/Marvell/edminiv2/config.mk | 25 +++
> board/Marvell/edminiv2/edminiv2.c | 165 ++++++++++++++++
> board/Marvell/edminiv2/edminiv2.h | 39 ++++
Pls edit MAKEALL and Maintainers files too for new board support
Arm: add Edminv2 board support (orion5x based)
> cpu/arm926ejs/orion5x/Makefile | 52 +++++
> cpu/arm926ejs/orion5x/cpu.c | 339
> ++++++++++++++++++++++++++++++++
> cpu/arm926ejs/orion5x/dram.c | 61 ++++++
> cpu/arm926ejs/orion5x/mpp.c | 88 ++++++++
> cpu/arm926ejs/orion5x/timer.c | 171 ++++++++++++++++
> drivers/serial/serial.c | 3 +
> include/asm-arm/arch-orion5x/88f5182.h | 40 ++++
> include/asm-arm/arch-orion5x/cpu.h | 213 ++++++++++++++++++++
> include/asm-arm/arch-orion5x/gpio.h | 53 +++++
> include/asm-arm/arch-orion5x/mpp.h | 122 ++++++++++++
> include/asm-arm/arch-orion5x/orion5x.h | 67 +++++++
Break this in to basic orion5x Soc support patch
And individual drivers patches like gpio, uart, etc..
> include/configs/edminiv2.h | 158 +++++++++++++++
> 17 files changed, 1650 insertions(+), 0 deletions(-)
> create mode 100644 board/Marvell/edminiv2/Makefile
> create mode 100644 board/Marvell/edminiv2/config.mk
> create mode 100644 board/Marvell/edminiv2/edminiv2.c
> create mode 100644 board/Marvell/edminiv2/edminiv2.h
> create mode 100644 cpu/arm926ejs/orion5x/Makefile
> create mode 100644 cpu/arm926ejs/orion5x/cpu.c
> create mode 100644 cpu/arm926ejs/orion5x/dram.c
> create mode 100644 cpu/arm926ejs/orion5x/mpp.c
> create mode 100644 cpu/arm926ejs/orion5x/timer.c
> create mode 100644 include/asm-arm/arch-orion5x/88f5182.h
> create mode 100644 include/asm-arm/arch-orion5x/cpu.h
> create mode 100644 include/asm-arm/arch-orion5x/gpio.h
> create mode 100644 include/asm-arm/arch-orion5x/mpp.h
> create mode 100644 include/asm-arm/arch-orion5x/orion5x.h
> create mode 100644 include/configs/edminiv2.h
>
> diff --git a/Makefile b/Makefile
> index bcb3fe9..60f9140 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2951,6 +2951,9 @@ davinci_dm365evm_config : unconfig
> davinci_dm6467evm_config : unconfig
> @$(MKCONFIG) $(@:_config=) arm arm926ejs dm6467evm
> davinci davinci
>
> +edminiv2_config: unconfig
> + @$(MKCONFIG) $(@:_config=) arm arm926ejs $(@:_config=)
> Marvell orion5x
> +
> imx27lite_config: unconfig
> @$(MKCONFIG) $(@:_config=) arm arm926ejs imx27lite logicpd mx27
>
> diff --git a/board/Marvell/edminiv2/Makefile
> b/board/Marvell/edminiv2/Makefile
> new file mode 100644
> index 0000000..2e6d305
> --- /dev/null
> +++ b/board/Marvell/edminiv2/Makefile
> @@ -0,0 +1,51 @@
> +#
> +# (C) Copyright 2009
> +# Marvell Semiconductor <www.marvell.com>
> +# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
This is copy paste mistake, pls correct in all files
You can keep my reference, i.e. based on (ref: boards/Marvell/openrd_base/openrd_base.c)
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# 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; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA
> +#
> +
> +include $(TOPDIR)/config.mk
> +
> +LIB = $(obj)lib$(BOARD).a
> +
> +COBJS := edminiv2.o
> +
> +SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> +OBJS := $(addprefix $(obj),$(COBJS))
> +SOBJS := $(addprefix $(obj),$(SOBJS))
> +
> +$(LIB): $(obj).depend $(OBJS) $(SOBJS)
> + $(AR) $(ARFLAGS) $@ $(OBJS) $(SOBJS)
> +
> +clean:
> + rm -f $(SOBJS) $(OBJS)
> +
> +distclean: clean
> + rm -f $(LIB) core *.bak .depend
> +
> +#############################################################
> ############
> +
> +# defines $(obj).depend target
> +include $(SRCTREE)/rules.mk
> +
> +sinclude $(obj).depend
> +
> +#############################################################
> ############
> diff --git a/board/Marvell/edminiv2/config.mk
> b/board/Marvell/edminiv2/config.mk
> new file mode 100644
> index 0000000..a4ea769
> --- /dev/null
> +++ b/board/Marvell/edminiv2/config.mk
> @@ -0,0 +1,25 @@
> +#
> +# (C) Copyright 2009
> +# Marvell Semiconductor <www.marvell.com>
> +# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
Ditto
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# 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; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA
> +#
> +
> +TEXT_BASE = 0x00600000
Is this valid for your board? BTW: how much DRAM you have on this board?
> diff --git a/board/Marvell/edminiv2/edminiv2.c
> b/board/Marvell/edminiv2/edminiv2.c
> new file mode 100644
> index 0000000..f23de0e
> --- /dev/null
> +++ b/board/Marvell/edminiv2/edminiv2.c
> @@ -0,0 +1,165 @@
> +/*
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
Ditto
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#include <common.h>
> +#include <miiphy.h>
> +#include <asm/arch/orion5x.h>
> +#include <asm/arch/mpp.h>
> +#include "edminiv2.h"
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +#if defined(CONFIG_FLASH_CFI_LEGACY)
> +#include <flash.h>
> +ulong board_flash_get_legacy (ulong base, int banknum,
> flash_info_t * info)
> +{
> + int sectsz[] = CONFIG_SYS_FLASH_SECTSZ;
> + int sect;
> +
> + if (base != CONFIG_SYS_FLASH_BASE)
> + return 0;
> +
> + info->size = 0;
> + info->sector_count = CONFIG_SYS_MAX_FLASH_SECT;
> + for (sect=0; sect<CONFIG_SYS_MAX_FLASH_SECT; sect++) {
> + info->start[sect] = base+info->size;
> + info->size += sectsz[sect];
> + }
> + info->flash_id = 0x01000000;
> + info->portwidth = FLASH_CFI_8BIT;
> + info->chipwidth = FLASH_CFI_BY8;
> + info->buffer_size = 32;
> + info->erase_blk_tout = 16384;
> + info->write_tout = 2;
> + info->buffer_write_tout = 5;
> + info->vendor = CFI_CMDSET_AMD_LEGACY;
> + info->cmd_reset= 0xF0;
> + info->interface= FLASH_CFI_X8;
> + info->legacy_unlock = 0;
> + info->manufacturer_id = 0xC2;
> + info->device_id = 0xBA;
> + info->device_id2= 0;
> +
> + info->ext_addr = 0;
> + info->cfi_version = 0x3133;
> + info->cfi_offset = 0x0000;
> + info->addr_unlock1 = 0x00000555;
> + info->addr_unlock2 = 0x000002AA;
> + info->name = "MX29LV400CB";
> +
> + return 1;
> +}
> +#endif /* CONFIG_SYS_FLASH_CFI */
> +
> +int board_init(void)
> +{
> + /*
> + * default gpio configuration
> + * There are maximum 64 gpios controlled through 2 sets
> of registers
I don't think there are 64 GPIOs in orion5x
> + * the below configuration configures mainly initial LED status
> + */
> + orion5x_config_gpio(EDMINIV2_OE_VAL_LOW, EDMINIV2_OE_LOW);
You can eliminate LOW here, since mpps less than 32
> +
> + /* Multi-Purpose Pins Functionality configuration */
> + u32 edminiv2_mpp_config[] = {
> + MPP0_GPIO,
> + MPP1_GPIO,
> + MPP2_GPIO,
> + MPP3_GPIO,
> + MPP4_GPIO,
> + MPP5_GPIO,
> + MPP6_GPIO,
> + MPP7_GPIO,
> + MPP8_GPIO,
> + MPP9_GPIO,
> + MPP10_GPIO,
> + MPP11_GPIO,
> + MPP12_SATA0_PRESENTn,
> + MPP13_SATA1_PRESENTn,
> + MPP14_SATA0_ACTn,
> + MPP15_SATA1_ACTn,
> + MPP16_GPIO,
> + MPP17_GPIO,
> + MPP18_GPIO,
> + MPP19_GPIO,
> + 0
> + };
> + orion5x_mpp_conf(edminiv2_mpp_config);
> +
> + /*
> + * arch number of board
> + */
> + gd->bd->bi_arch_number = MACH_TYPE_EDMINI_V2;
> +
> + /* adress of boot parameters */
> + gd->bd->bi_boot_params = orion5x_sdram_bar(0) + 0x100;
> +
> + return 0;
> +}
> +
> +int dram_init(void)
> +{
> + int i;
> +
> + for (i = 0; i < CONFIG_NR_DRAM_BANKS; i++) {
> + gd->bd->bi_dram[i].start = orion5x_sdram_bar(i);
> + gd->bd->bi_dram[i].size = orion5x_sdram_bs(i);
> + }
> + return 0;
> +}
> +
> +#ifdef CONFIG_RESET_PHY_R
> +/* Configure and enable MV88E1116 PHY */
> +void reset_phy(void)
> +{
> + u16 reg;
> + u16 devadr;
> + char *name = "egiga0";
> +
> + if (miiphy_set_current_dev(name))
> + return;
> +
> + /* command to read PHY dev address */
> + if (miiphy_read(name, 0xEE, 0xEE, (u16 *) &devadr)) {
> + printf("Err..%s could not read PHY dev address\n",
> + __FUNCTION__);
> + return;
> + }
> +
> + /*
> + * Enable RGMII delay on Tx and Rx for CPU port
> + * Ref: sec 4.7.2 of chip datasheet
> + */
> + miiphy_write(name, devadr, MV88E1116_PGADR_REG, 2);
> + miiphy_read(name, devadr, MV88E1116_MAC_CTRL_REG, ®);
> + reg |= (MV88E1116_RGMII_RXTM_CTRL | MV88E1116_RGMII_TXTM_CTRL);
> + miiphy_write(name, devadr, MV88E1116_MAC_CTRL_REG, reg);
> + miiphy_write(name, devadr, MV88E1116_PGADR_REG, 0);
> +
> + /* reset the phy */
> + miiphy_reset(name, devadr);
> +
> + printf("88E1116 Initialized on %s\n", name);
> +}
> +#endif /* CONFIG_RESET_PHY_R */
> diff --git a/board/Marvell/edminiv2/edminiv2.h
> b/board/Marvell/edminiv2/edminiv2.h
> new file mode 100644
> index 0000000..77a108e
> --- /dev/null
> +++ b/board/Marvell/edminiv2/edminiv2.h
> @@ -0,0 +1,39 @@
> +/*
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
Ditto
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#ifndef __EDMINIV2_H
> +#define __EDMINIV2_H
> +
> +#define EDMINIV2_OE_LOW (~(0))
> +#define EDMINIV2_OE_VAL_LOW (1 << 29) /*
> USB_PWEN low */
Pls remove word LOW here
> +
> +/* PHY related */
> +#define MV88E1116_LED_FCTRL_REG 10
> +#define MV88E1116_CPRSP_CR3_REG 21
> +#define MV88E1116_MAC_CTRL_REG 21
> +#define MV88E1116_PGADR_REG 22
> +#define MV88E1116_RGMII_TXTM_CTRL (1 << 4)
> +#define MV88E1116_RGMII_RXTM_CTRL (1 << 5)
> +
> +#endif /* __EDMINIV2_H */
> diff --git a/cpu/arm926ejs/orion5x/Makefile
> b/cpu/arm926ejs/orion5x/Makefile
> new file mode 100644
> index 0000000..4b506a7
> --- /dev/null
> +++ b/cpu/arm926ejs/orion5x/Makefile
> @@ -0,0 +1,52 @@
> +#
> +# (C) Copyright 2009 Albert ARIBAUD <albrt.aribaud@free.fr>
> +#
> +# Based on original Kirkwood support which is
> +# (C) Copyright 2009
> +# Marvell Semiconductor <www.marvell.com>
> +# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> +#
> +# See file CREDITS for list of people who contributed to this
> +# project.
> +#
> +# 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; either version 2 of
> +# the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software
> +# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> +# MA 02110-1301 USA
> +#
> +
> +include $(TOPDIR)/config.mk
> +
> +LIB = $(obj)lib$(SOC).a
> +
> +COBJS-y = dram.o
> +COBJS-y += cpu.o
Pls put these lines in order
> +COBJS-y += mpp.o
> +COBJS-y += timer.o
> +
> +SRCS := $(SOBJS:.o=.S) $(COBJS-y:.o=.c)
> +OBJS := $(addprefix $(obj),$(SOBJS) $(COBJS-y))
> +
> +all: $(obj).depend $(LIB)
> +
> +$(LIB): $(OBJS)
> + $(AR) $(ARFLAGS) $@ $(OBJS)
> +
> +#############################################################
> ############
> +
> +# defines $(obj).depend target
> +include $(SRCTREE)/rules.mk
> +
> +sinclude $(obj).depend
> +
> +#############################################################
> ############
> diff --git a/cpu/arm926ejs/orion5x/cpu.c b/cpu/arm926ejs/orion5x/cpu.c
> new file mode 100644
> index 0000000..50ca724
> --- /dev/null
> +++ b/cpu/arm926ejs/orion5x/cpu.c
> @@ -0,0 +1,339 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support which is
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#include <common.h>
> +#include <netdev.h>
> +#include <asm/cache.h>
> +#include <u-boot/md5.h>
> +#include <asm/arch/orion5x.h>
> +#include <hush.h>
> +
> +#define BUFLEN 16
> +
> +void reset_cpu(unsigned long ignored)
> +{
> + struct orion5x_cpu_registers *cpureg =
> + (struct orion5x_cpu_registers *)ORION5X_CPU_REG_BASE;
> +
> + writel(readl(&cpureg->rstoutn_mask) | (1 << 2),
> + &cpureg->rstoutn_mask);
> + writel(readl(&cpureg->sys_soft_rst) | 1,
> + &cpureg->sys_soft_rst);
> + while (1) ;
> +}
> +
> +/*
> + * Window Size
> + * Used with the Base register to set the address window
> size and location.
> + * Must be programmed from LSB to MSB as sequence of ones followed by
> + * sequence of zeros. The number of ones specifies the size
> of the window in
> + * 64 KByte granularity (e.g., a value of 0x00FF specifies
> 256 = 16 MByte).
> + * NOTE: A value of 0x0 specifies 64-KByte size.
> + */
> +unsigned int orion5x_winctrl_calcsize(unsigned int sizeval)
> +{
> + int i;
> + unsigned int j = 0;
> + u32 val = sizeval >> 1;
> +
> + for (i = 0; val > 0x10000; i++) {
> + j |= (1 << i);
> + val = val >> 1;
> + }
> + return (0x0000ffff & j);
> +}
> +
> +/*
> + * orion5x_config_adr_windows - Configure address Windows
> + *
> + * There are 8 address windows supported by Orion5x Soc to
> addess different
> + * devices. Each window can be configured for size, BAR and
> remap addr
> + * Below configuration is standard for most of the cases
> + *
> + * If remap function not used, remap_lo must be set as base
> + *
> + * Reference Documentation:
> + * Mbus-L to Mbus Bridge Registers Configuration.
> + * (Sec 25.1 and 25.3 of Datasheet)
> + */
> +int orion5x_config_adr_windows(void)
> +{
> + struct orion5x_win_registers *winregs =
> + (struct orion5x_win_registers *)ORION5X_CPU_WIN_BASE;
> +
> + /* Window 0: PCIE MEM address space */
> +
> writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCIE_MEM,
> ORION5X_TARGET_PCIE,
> + ORION5X_ATTR_PCIE_MEM, ORION5X_WIN_ENABLE),
> &winregs[0].ctrl);
> + writel(ORION5X_DEFADR_PCIE_MEM, &winregs[0].base);
> + writel(ORION5X_DEFADR_PCIE_MEM_REMAP_LO, &winregs[0].remap_lo);
> + writel(ORION5X_DEFADR_PCIE_MEM_REMAP_HI, &winregs[0].remap_hi);
> +
> + /* Window 1: PCIE IO address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCIE_IO,
> ORION5X_TARGET_PCIE,
> + ORION5X_ATTR_PCIE_IO, ORION5X_WIN_ENABLE),
> &winregs[1].ctrl);
> + writel(ORION5X_DEFADR_PCIE_IO, &winregs[1].base);
> + writel(ORION5X_DEFADR_PCIE_IO_REMAP_LO, &winregs[1].remap_lo);
> + writel(ORION5X_DEFADR_PCIE_IO_REMAP_HI, &winregs[1].remap_hi);
> +
> + /* Window 2: PCI MEM address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCI_MEM,
> ORION5X_TARGET_PCI,
> + ORION5X_ATTR_PCI_MEM, ORION5X_WIN_ENABLE),
> &winregs[2].ctrl);
> + writel(ORION5X_DEFADR_PCI_MEM, &winregs[2].base);
> +
> + /* Window 3: PCI IO address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_PCI_IO,
> ORION5X_TARGET_PCI,
> + ORION5X_ATTR_PCI_IO, ORION5X_WIN_ENABLE),
> &winregs[3].ctrl);
> + writel(ORION5X_DEFADR_PCI_IO, &winregs[3].base);
> +
> + /* Window 4: DEV_CS0 address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS0,
> ORION5X_TARGET_DEVICE,
> + ORION5X_ATTR_DEV_CS0, ORION5X_WIN_ENABLE),
> &winregs[4].ctrl);
> + writel(ORION5X_DEFADR_DEV_CS0, &winregs[4].base);
> +
> + /* Window 5: DEV_CS1 address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS1,
> ORION5X_TARGET_DEVICE,
> + ORION5X_ATTR_DEV_CS1, ORION5X_WIN_ENABLE),
> &winregs[5].ctrl);
> + writel(ORION5X_DEFADR_DEV_CS1, &winregs[5].base);
> +
> + /* Window 6: DEV_CS2 address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_DEV_CS2,
> ORION5X_TARGET_DEVICE,
> + ORION5X_ATTR_DEV_CS2, ORION5X_WIN_ENABLE),
> &winregs[6].ctrl);
> + writel(ORION5X_DEFADR_DEV_CS2, &winregs[6].base);
> +
> + /* Window 7: BOOT Memory address space */
> + writel(ORION5X_CPU_WIN_CTRL_DATA(ORION5X_DEFSZ_BOOTROM,
> ORION5X_TARGET_DEVICE,
> + ORION5X_ATTR_BOOTROM, ORION5X_WIN_ENABLE),
> &winregs[7].ctrl);
> + writel(ORION5X_DEFADR_BOOTROM, &winregs[7].base);
> +
> + return 0;
> +}
> +
> +/*
> + * orion5x_config_gpio - GPIO configuration
> + */
> +void orion5x_config_gpio(u32 gpp_oe_val, u32 gpp_oe)
> +{
> + struct orion5x_gpio_registers *gpioreg =
> + (struct orion5x_gpio_registers *)ORION5X_GPIO_BASE;
> +
> + /* Init GPIOS to default values as per board requirement */
> + writel(gpp_oe_val, &gpioreg->dout);
> + writel(gpp_oe, &gpioreg->oe);
> +}
> +
> +/*
> + * orion5x_config_mpp - Multi-Purpose Pins Functionality
> configuration
> + *
> + * Each MPP can be configured to different functionality through
> + * MPP control register, ref (sec 6.1 of Orion5x h/w specification)
> + *
> + * There are maximum 64 Multi-Pourpose Pins on Orion5x
Correct this -64
> + * Each MPP functionality can be configuration by a 4bit value
> + * of MPP control reg, the value and associated functionality depends
> + * upon used SoC varient
> + */
> +int orion5x_config_mpp(u32 mpp0_7, u32 mpp8_15, u32 mpp16_23)
> +{
> + u32 *mppreg = (u32 *) ORION5X_MPP_BASE;
> +
> + /* program mpp registers */
> + writel(mpp0_7, &mppreg[0]);
> + writel(mpp8_15, &mppreg[1]);
> + writel(mpp16_23, &mppreg[2]);
> + return 0;
> +}
> +
> +/* Orion5x identification is done through PCIE space.
> + * As long as there is no Orion5x support for PCIE,
> + * We'll define the ID and revision PCIE registers
> + * here.
> + */
> +#define PCIE_DEV_ID_OFF (ORION5X_REG_PCIE_BASE + 0x0000)
> +#define PCIE_DEV_REV_OFF (ORION5X_REG_PCIE_BASE + 0x0008)
Pls move the defination to header file
> +u32 orion5x_device_id(void)
> +{
> + return readl(PCIE_DEV_ID_OFF) >> 16;
> +}
> +
> +u32 orion5x_device_rev(void)
> +{
> + return readl(PCIE_DEV_REV_OFF) & 0xff;
> +}
> +
> +#if defined(CONFIG_DISPLAY_CPUINFO)
> +
> +int print_cpuinfo(void)
> +{
> + char dev_str[] = "0x0000";
> + char *dev_name = dev_str;
> + char rev_str[] = "0x00";
> + char *rev_name = rev_str;
> +
> + u32 dev = orion5x_device_id();
> + u32 rev = orion5x_device_rev();
> +
> + if (dev == MV88F5181_DEV_ID) {
> + dev_name = "MV88F5181";
> + if (rev == MV88F5181_REV_B1) {
> + rev_name = "B1";
> + } else if (rev == MV88F5181L_REV_A1) {
> + dev_name = "MV88F5181L";
> + rev_name = "A1";
> + } else if (rev == MV88F5181L_REV_A0) {
> + dev_name = "MV88F5181L";
> + rev_name = "A0";
> + } else {
> + sprintf(rev_str,"0x%02x", rev);
> + }
> + } else if (dev == MV88F5182_DEV_ID) {
> + dev_name = "MV88F5182";
> + if (rev == MV88F5182_REV_A2) {
> + rev_name = "A2";
> + } else {
> + sprintf(rev_str,"0x%02x", rev);
> + }
> + } else if (dev == MV88F5281_DEV_ID) {
> + dev_name = "MV88F5281";
> + if (rev == MV88F5281_REV_D2) {
> + rev_name = "D2";
> + } else if (rev == MV88F5281_REV_D1) {
> + rev_name = "D1";
> + } else if (rev == MV88F5281_REV_D0) {
> + rev_name = "D0";
> + } else {
> + sprintf(rev_str,"0x%02x", rev);
> + }
> + } else if (dev == MV88F6183_DEV_ID) {
> + dev_name = "MV88F6183";
> + if (rev == MV88F6183_REV_B0) {
> + rev_name = "B0";
> + } else {
> + sprintf(rev_str,"0x%02x", rev);
> + }
> + } else {
> + sprintf(dev_str,"0x%04x", rev);
> + sprintf(rev_str,"0x%02x", rev);
This is common line, pls take out of if-else
> + }
> +
> + printf("SoC: Orion5x %s-%s\n", dev_name, rev_name);
> +
> + return 0;
> +}
> +#endif /* CONFIG_DISPLAY_CPUINFO */
> +
> +#ifdef CONFIG_ARCH_CPU_INIT
> +int arch_cpu_init(void)
> +{
> + struct orion5x_cpu_registers *cpureg =
> + (struct orion5x_cpu_registers *)ORION5X_CPU_REG_BASE;
> +
> + /* Linux expects` the internal registers to be at 0xf1000000 */
> + writel(ORION5X_REGS_PHY_BASE, ORION5X_OFFSET_REG);
> +
> + /* Enable and invalidate L2 cache in write through mode */
> + writel(readl(&cpureg->l2_cfg) | 0x18, &cpureg->l2_cfg);
> + invalidate_l2_cache();
> +
> + orion5x_config_adr_windows();
> +
> +#ifdef CONFIG_ORION5X_RGMII_PAD_1V8
> + /*
> + * Configures the I/O voltage of the pads connected to Egigabit
> + * Ethernet interface to 1.8V
> + * By defult it is set to 3.3V
> + */
> + reg = readl(ORION5X_REG_MPP_OUT_DRV_REG);
> + reg |= (1 << 7);
> + writel(reg, ORION5X_REG_MPP_OUT_DRV_REG);
> +#endif
> +#ifdef CONFIG_ORION5X_EGIGA_INIT
> + /*
> + * Set egiga port0/1 in normal functional mode
> + * This is required becasue on Orion5x by default ports
> are in reset mode
> + * OS egiga driver may not have provision to set them
> in normal mode
> + * and if u-boot is build without network support,
> network may fail at OS level
> + */
> + reg = readl(ORION5XGBE_PORT_SERIAL_CONTROL1_REG(0));
> + reg &= ~(1 << 4); /* Clear PortReset Bit */
> + writel(reg, (ORION5XGBE_PORT_SERIAL_CONTROL1_REG(0)));
> + reg = readl(ORION5XGBE_PORT_SERIAL_CONTROL1_REG(1));
> + reg &= ~(1 << 4); /* Clear PortReset Bit */
> + writel(reg, (ORION5XGBE_PORT_SERIAL_CONTROL1_REG(1)));
> +#endif
> +#ifdef CONFIG_ORION5X_PCIE_INIT
> + /*
> + * Enable PCI Express Port0
> + */
> + reg = readl(&cpureg->ctrl_stat);
> + reg |= (1 << 0); /* Set PEX0En Bit */
> + writel(reg, &cpureg->ctrl_stat);
> +#endif
> + return 0;
> +}
> +#endif /* CONFIG_ARCH_CPU_INIT */
> +
> +/*
> + * SOC specific misc init
> + */
> +#if defined(CONFIG_ARCH_MISC_INIT)
> +int arch_misc_init(void)
> +{
> + volatile u32 temp;
> +
> + /*CPU streaming & write allocate */
> + temp = readfr_extra_feature_reg();
> + temp &= ~(1 << 28); /* disable wr alloc */
> + writefr_extra_feature_reg(temp);
> +
> + temp = readfr_extra_feature_reg();
> + temp &= ~(1 << 29); /* streaming disabled */
> + writefr_extra_feature_reg(temp);
> +
> + /* L2Cache settings */
> + temp = readfr_extra_feature_reg();
> + /* Disable L2C pre fetch - Set bit 24 */
> + temp |= (1 << 24);
> + /* enable L2C - Set bit 22 */
> + temp |= (1 << 22);
> + writefr_extra_feature_reg(temp);
> +
> + icache_enable();
> + /* Change reset vector to address 0x0 */
> + temp = get_cr();
> + set_cr(temp & ~CR_V);
> +
> + return 0;
> +}
> +#endif /* CONFIG_ARCH_MISC_INIT */
> +
> +#ifdef CONFIG_ORION5X_EGIGA
> +int cpu_eth_init(bd_t *bis)
> +{
> + orion5x_egiga_initialize(bis);
> + return 0;
> +}
> +#endif
Pls do not add this by default, make it a part of egiga driver patch
> diff --git a/cpu/arm926ejs/orion5x/dram.c
> b/cpu/arm926ejs/orion5x/dram.c
> new file mode 100644
> index 0000000..69aec26
> --- /dev/null
> +++ b/cpu/arm926ejs/orion5x/dram.c
> @@ -0,0 +1,61 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support which is
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#include <config.h>
> +#include <asm/arch/orion5x.h>
> +
> +#define ORION5X_REG_CPUCS_WIN_BAR(x)
> (ORION5X_REGISTER(0x1500) + (x * 0x08))
> +#define ORION5X_REG_CPUCS_WIN_SZ(x)
> (ORION5X_REGISTER(0x1504) + (x * 0x08))
> +/*
> + * orion5x_sdram_bar - reads SDRAM Base Address Register
> + */
> +u32 orion5x_sdram_bar(enum memory_bank bank)
> +{
> + u32 result = 0;
> + u32 enable = 0x01 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
> +
> + if ((!enable) || (bank > BANK3))
> + return 0;
> +
> + result = readl(ORION5X_REG_CPUCS_WIN_BAR(bank));
> + return result;
> +}
> +
> +/*
> + * orion5x_sdram_bs - reads SDRAM Bank size
> + */
> +u32 orion5x_sdram_bs(enum memory_bank bank)
> +{
> + u32 result = 0;
> + u32 enable = 0x01 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
> +
> + if ((!enable) || (bank > BANK3))
> + return 0;
> + result = 0xff000000 & readl(ORION5X_REG_CPUCS_WIN_SZ(bank));
> + result += 0x01000000;
> + return result;
> +}
> diff --git a/cpu/arm926ejs/orion5x/mpp.c b/cpu/arm926ejs/orion5x/mpp.c
> new file mode 100644
> index 0000000..1b1e7fa
> --- /dev/null
> +++ b/cpu/arm926ejs/orion5x/mpp.c
> @@ -0,0 +1,88 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support
> + *
> + * arch/arm/mach-orion5x/mpp.c
> + *
> + * MPP functions for Marvell Kirorion5xood SoCs
> + * Referenced from Linux kernel source
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#include <common.h>
> +#include <asm/arch/orion5x.h>
> +#include <asm/arch/mpp.h>
> +
> +static u32 orion5x_variant(void)
> +{
> + switch (orion5x_device_id()) {
> + case MV88F5181_DEV_ID:
> + return 0;
> + case MV88F5182_DEV_ID:
> + return MPP_F5182_MASK;
> + case MV88F5281_DEV_ID:
> + return MPP_F5281_MASK;
> + case MV88F6183_DEV_ID:
> + return 0;
> + default:
> + debug("MPP setup: unknown Orion5x variant\n");
> + return 0;
> + }
> +}
> +
> +#define MPP_CTRL(i) (ORION5X_MPP_BASE + (i* 4))
> +#define MPP_NR_REGS (1 + MPP_MAX/8)
> +
> +void orion5x_mpp_conf(u32 *mpp_list)
> +{
> + u32 mpp_ctrl[MPP_NR_REGS];
> + unsigned int variant_mask;
> + int i;
> +
> + variant_mask = orion5x_variant();
> + if (!variant_mask)
> + return;
> +
> + debug( "initial MPP regs:");
> + for (i = 0; i < MPP_NR_REGS; i++) {
> + mpp_ctrl[i] = readl(MPP_CTRL(i));
> + debug(" %08x", mpp_ctrl[i]);
> + }
> + debug("\n");
> +
> +
> + while (*mpp_list) {
> + unsigned int num = MPP_NUM(*mpp_list);
> + unsigned int sel = MPP_SEL(*mpp_list);
> + int shift;
> +
> + if (num > MPP_MAX) {
> + debug("orion5x_mpp_conf: invalid MPP "
> + "number (%u)\n", num);
> + continue;
> + }
> + if (!(*mpp_list & variant_mask)) {
> + debug("orion5x_mpp_conf: requested
> MPP%u config "
> + "unavailable on this hardware\n", num);
> + continue;
> + }
> +
> + shift = (num & 7) << 2;
> + mpp_ctrl[num / 8] &= ~(0xf << shift);
> + mpp_ctrl[num / 8] |= sel << shift;
> +
> + mpp_list++;
> + }
> +
> + debug(" final MPP regs:");
> + for (i = 0; i < MPP_NR_REGS; i++) {
> + writel(mpp_ctrl[i], MPP_CTRL(i));
> + debug(" %08x", mpp_ctrl[i]);
> + }
> + debug("\n");
> +
> +}
> diff --git a/cpu/arm926ejs/orion5x/timer.c
> b/cpu/arm926ejs/orion5x/timer.c
> new file mode 100644
> index 0000000..80fee75
> --- /dev/null
> +++ b/cpu/arm926ejs/orion5x/timer.c
> @@ -0,0 +1,171 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support which is
> + * Copyright (C) Marvell International Ltd. and its affiliates
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#include <common.h>
> +#include <asm/arch/orion5x.h>
> +
> +#define UBOOT_CNTR 0 /* counter to use for uboot timer */
> +
> +/* Timer reload and current value registers */
> +struct orion5x_tmr_val {
> + u32 reload; /* Timer reload reg */
> + u32 val; /* Timer value reg */
> +};
> +
> +/* Timer registers */
> +struct orion5x_tmr_registers {
> + u32 ctrl; /* Timer control reg */
> + u32 pad[3];
> + struct orion5x_tmr_val tmr[2];
> + u32 wdt_reload;
> + u32 wdt_val;
> +};
> +
> +struct orion5x_tmr_registers *orion5x_tmr_regs = (struct
> orion5x_tmr_registers *)ORION5X_TIMER_BASE;
> +
> +/*
> + * ARM Timers Registers Map
> + */
> +#define CNTMR_CTRL_REG &orion5x_tmr_regs->ctrl
> +#define CNTMR_RELOAD_REG(tmrnum)
> &orion5x_tmr_regs->tmr[tmrnum].reload
> +#define CNTMR_VAL_REG(tmrnum)
> &orion5x_tmr_regs->tmr[tmrnum].val
> +
> +/*
> + * ARM Timers Control Register
> + * CPU_TIMERS_CTRL_REG (CTCR)
> + */
> +#define CTCR_ARM_TIMER_EN_OFFS(cntr) (cntr * 2)
> +#define CTCR_ARM_TIMER_EN_MASK(cntr) (1 << CTCR_ARM_TIMER_EN_OFFS)
> +#define CTCR_ARM_TIMER_EN(cntr) (1 <<
> CTCR_ARM_TIMER_EN_OFFS(cntr))
> +#define CTCR_ARM_TIMER_DIS(cntr) (0 <<
> CTCR_ARM_TIMER_EN_OFFS(cntr))
> +
> +#define CTCR_ARM_TIMER_AUTO_OFFS(cntr) ((cntr * 2) + 1)
> +#define CTCR_ARM_TIMER_AUTO_MASK(cntr) (1 << 1)
> +#define CTCR_ARM_TIMER_AUTO_EN(cntr) (1 <<
> CTCR_ARM_TIMER_AUTO_OFFS(cntr))
> +#define CTCR_ARM_TIMER_AUTO_DIS(cntr) (0 <<
> CTCR_ARM_TIMER_AUTO_OFFS(cntr))
> +
> +/*
> + * ARM Timer\Watchdog Reload Register
> + * CNTMR_RELOAD_REG (TRR)
> + */
> +#define TRG_ARM_TIMER_REL_OFFS 0
> +#define TRG_ARM_TIMER_REL_MASK 0xffffffff
> +
> +/*
> + * ARM Timer\Watchdog Register
> + * CNTMR_VAL_REG (TVRG)
> + */
> +#define TVR_ARM_TIMER_OFFS 0
> +#define TVR_ARM_TIMER_MASK 0xffffffff
> +#define TVR_ARM_TIMER_MAX 0xffffffff
> +#define TIMER_LOAD_VAL 0xffffffff
> +
> +#define READ_TIMER
> (readl(CNTMR_VAL_REG(UBOOT_CNTR)) / \
> + (CONFIG_SYS_TCLK / 1000))
> +
> +static ulong timestamp;
> +static ulong lastdec;
> +
> +void reset_timer_masked(void)
> +{
> + /* reset time */
> + lastdec = READ_TIMER;
> + timestamp = 0;
> +}
> +
> +ulong get_timer_masked(void)
> +{
> + ulong now = READ_TIMER;
> +
> + if (lastdec >= now) {
> + /* normal mode */
> + timestamp += lastdec - now;
> + } else {
> + /* we have an overflow ... */
> + timestamp += lastdec +
> + (TIMER_LOAD_VAL / (CONFIG_SYS_TCLK /
> 1000)) - now;
> + }
> + lastdec = now;
> +
> + return timestamp;
> +}
> +
> +void reset_timer(void)
> +{
> + reset_timer_masked();
> +}
> +
> +ulong get_timer(ulong base)
> +{
> + return get_timer_masked() - base;
> +}
> +
> +void set_timer(ulong t)
> +{
> + timestamp = t;
> +}
> +
> +void udelay(unsigned long usec)
> +{
> + uint current;
> + ulong delayticks;
> +
> + current = readl(CNTMR_VAL_REG(UBOOT_CNTR));
> + delayticks = (usec * (CONFIG_SYS_TCLK / 1000000));
> +
> + if (current < delayticks) {
> + delayticks -= current;
> + while (readl(CNTMR_VAL_REG(UBOOT_CNTR)) < current) ;
> + while ((TIMER_LOAD_VAL - delayticks) <
> + readl(CNTMR_VAL_REG(UBOOT_CNTR))) ;
> + } else {
> + while (readl(CNTMR_VAL_REG(UBOOT_CNTR)) >
> + (current - delayticks)) ;
> + }
> +}
> +
> +/*
> + * init the counter
> + */
> +int timer_init(void)
> +{
> + unsigned int cntmrctrl;
> +
> + /* load value into timer */
> + writel(TIMER_LOAD_VAL, CNTMR_RELOAD_REG(UBOOT_CNTR));
> + writel(TIMER_LOAD_VAL, CNTMR_VAL_REG(UBOOT_CNTR));
> +
> + /* enable timer in auto reload mode */
> + cntmrctrl = readl(CNTMR_CTRL_REG);
> + cntmrctrl |= CTCR_ARM_TIMER_EN(UBOOT_CNTR);
> + cntmrctrl |= CTCR_ARM_TIMER_AUTO_EN(UBOOT_CNTR);
> + writel(cntmrctrl, CNTMR_CTRL_REG);
> +
> + /* init the timestamp and lastdec value */
> + reset_timer_masked();
> +
> + return 0;
> +}
> diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
> index dd5f332..18686a2 100644
> --- a/drivers/serial/serial.c
> +++ b/drivers/serial/serial.c
> @@ -30,6 +30,9 @@
> #ifdef CONFIG_KIRKWOOD
> #include <asm/arch/kirkwood.h>
> #endif
> +#ifdef CONFIG_ORION5X
> +#include <asm/arch/orion5x.h>
> +#endif
>
> #if defined (CONFIG_SERIAL_MULTI)
> #include <serial.h>
> diff --git a/include/asm-arm/arch-orion5x/88f5182.h
> b/include/asm-arm/arch-orion5x/88f5182.h
> new file mode 100644
> index 0000000..1dfdb24
> --- /dev/null
> +++ b/include/asm-arm/arch-orion5x/88f5182.h
> @@ -0,0 +1,40 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood 88F6182 support which is
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * Header file for Feroceon CPU core 88F5182 SOC.
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#ifndef _CONFIG_88F5182_H
> +#define _CONFIG_88F5182_H
> +
> +/* SOC specific definations */
> +#define F88F5182_REGS_PHYS_BASE 0xf1000000
> +#define ORION5X_REGS_PHY_BASE F88F5182_REGS_PHYS_BASE
> +
> +/* TCLK Core Clock defination */
> +#define CONFIG_SYS_TCLK 166000000 /* 166MHz */
> +
> +#endif /* _CONFIG_88F5182_H */
> diff --git a/include/asm-arm/arch-orion5x/cpu.h
> b/include/asm-arm/arch-orion5x/cpu.h
> new file mode 100644
> index 0000000..7078d2e
> --- /dev/null
> +++ b/include/asm-arm/arch-orion5x/cpu.h
> @@ -0,0 +1,213 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirorion5x_ood support which is
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#ifndef _ORION5X_CPU_H
> +#define _ORION5X_CPU_H
> +
> +#include <asm/system.h>
> +
> +#ifndef __ASSEMBLY__
> +
> +#define ORION5X_CPU_WIN_CTRL_DATA(size, target, attr, en)
> (en | (target << 4) \
> + | (attr << 8) |
> (orion5x_winctrl_calcsize(size) << 16))
> +
> +#define ORION5XGBE_PORT_SERIAL_CONTROL1_REG(_x) \
> + ((_x ? ORION5X_EGIGA0_BASE :
> ORION5X_EGIGA1_BASE) + 0x44c)
> +
> +#define ORION5X_REG_DEVICE_ID
> (ORION5X_MPP_BASE + 0x34)
> +#define ORION5X_REG_SYSRST_CNT
> (ORION5X_MPP_BASE + 0x50)
> +#define SYSRST_CNT_1SEC_VAL (25*1000000)
> +#define ORION5X_REG_MPP_OUT_DRV_REG
> (ORION5X_MPP_BASE + 0xE0)
> +
> +enum memory_bank {
> + BANK0,
> + BANK1,
> + BANK2,
> + BANK3
> +};
> +
> +enum orion5x_cpu_winen {
> + ORION5X_WIN_DISABLE,
> + ORION5X_WIN_ENABLE
> +};
> +
> +enum orion5x_cpu_target {
> + ORION5X_TARGET_DRAM = 0,
> + ORION5X_TARGET_DEVICE = 1,
> + ORION5X_TARGET_PCI = 3,
> + ORION5X_TARGET_PCIE = 4,
> + ORION5X_TARGET_SASRAM = 9
> +};
> +
> +enum orion5x_cpu_attrib {
> + ORION5X_ATTR_DRAM_CS0 = 0x0e,
> + ORION5X_ATTR_DRAM_CS1 = 0x0d,
> + ORION5X_ATTR_DRAM_CS2 = 0x0b,
> + ORION5X_ATTR_DRAM_CS3 = 0x07,
> + ORION5X_ATTR_PCI_MEM = 0x59,
> + ORION5X_ATTR_PCI_IO = 0x51,
> + ORION5X_ATTR_PCIE_MEM = 0x59,
> + ORION5X_ATTR_PCIE_IO = 0x51,
> + ORION5X_ATTR_SASRAM = 0x00,
> + ORION5X_ATTR_DEV_CS0 = 0x1e,
> + ORION5X_ATTR_DEV_CS1 = 0x1d,
> + ORION5X_ATTR_DEV_CS2 = 0x1b,
> + ORION5X_ATTR_BOOTROM = 0x0f
> +};
> +
> +/*
> + * Default Device Address MAP BAR values
> + */
> +#define ORION5X_DEFADR_PCIE_MEM 0x90000000
> +#define ORION5X_DEFADR_PCIE_MEM_REMAP_LO 0x90000000
> +#define ORION5X_DEFADR_PCIE_MEM_REMAP_HI 0
> +#define ORION5X_DEFSZ_PCIE_MEM (128*1024*1024)
> +
> +#define ORION5X_DEFADR_PCIE_IO 0xf0000000
> +#define ORION5X_DEFADR_PCIE_IO_REMAP_LO 0x90000000
> +#define ORION5X_DEFADR_PCIE_IO_REMAP_HI 0
> +#define ORION5X_DEFSZ_PCIE_IO (64*1024)
> +
> +#define ORION5X_DEFADR_PCI_MEM 0x98000000
> +#define ORION5X_DEFSZ_PCI_MEM (128*1024*1024)
> +
> +#define ORION5X_DEFADR_PCI_IO 0xf0100000
> +#define ORION5X_DEFSZ_PCI_IO (64*1024)
> +
> +#define ORION5X_DEFADR_DEV_CS0 0xfa000000
> +#define ORION5X_DEFSZ_DEV_CS0 (2*1024*1024)
> +
> +#define ORION5X_DEFADR_DEV_CS1 0xf8000000
> +#define ORION5X_DEFSZ_DEV_CS1 (32*1024*1024)
> +
> +#define ORION5X_DEFADR_DEV_CS2 0xfa800000
> +#define ORION5X_DEFSZ_DEV_CS2 (1*1024*1024)
> +
> +#define ORION5X_DEFADR_BOOTROM 0xFFF80000
> +#define ORION5X_DEFSZ_BOOTROM (512*1024)
> +
> +/* Constants for known devices and revisions
> + *
> + */
> +
> +/* Orion-1 (88F5181) and Orion-VoIP (88F5181L) */
> +#define MV88F5181_DEV_ID 0x5181
> +#define MV88F5181_REV_B1 3
> +#define MV88F5181L_REV_A0 8
> +#define MV88F5181L_REV_A1 9
> +/* Orion-NAS (88F5182) */
> +#define MV88F5182_DEV_ID 0x5182
> +#define MV88F5182_REV_A2 2
> +/* Orion-2 (88F5281) */
> +#define MV88F5281_DEV_ID 0x5281
> +#define MV88F5281_REV_D0 4
> +#define MV88F5281_REV_D1 5
> +#define MV88F5281_REV_D2 6
> +/* Orion-1-90 (88F6183) */
> +#define MV88F6183_DEV_ID 0x6183
> +#define MV88F6183_REV_B0 3
> +
> +/*
> + * read feroceon/sheeva core extra feature register
Pls remove name sheeva
> + * using co-proc instruction
> + */
> +static inline unsigned int readfr_extra_feature_reg(void)
> +{
> + unsigned int val;
> + asm volatile ("mrc p15, 1, %0, c15, c1, 0 @ readfr exfr":"=r"
> + (val)::"cc");
> + return val;
> +}
> +
> +/*
> + * write feroceon/sheeva core extra feature register
Same
> + * using co-proc instruction
> + */
> +static inline void writefr_extra_feature_reg(unsigned int val)
> +{
> + asm volatile ("mcr p15, 1, %0, c15, c1, 0 @ writefr exfr"::"r"
> + (val):"cc");
> + isb();
> +}
> +
> +/*
> + * MBus-L to Mbus Bridge Registers
> + * Ref: Datasheet sec:A.3
Are those references correct?
> + */
> +struct orion5x_win_registers {
> + u32 ctrl;
> + u32 base;
> + u32 remap_lo;
> + u32 remap_hi;
> +};
> +
> +/*
> + * CPU control and status Registers
> + * Ref: Datasheet sec:A.3.2
Are those references correct?
> + */
> +struct orion5x_cpu_registers {
> + u32 config; /*0x20100 */
> + u32 ctrl_stat; /*0x20104 */
> + u32 rstoutn_mask; /* 0x20108 */
> + u32 sys_soft_rst; /* 0x2010C */
> + u32 ahb_mbus_cause_irq; /* 0x20110 */
> + u32 ahb_mbus_mask_irq; /* 0x20114 */
> + u32 pad1[2];
> + u32 ftdll_config; /* 0x20120 */
> + u32 pad2;
> + u32 l2_cfg; /* 0x20128 */
> +};
> +
> +/*
> + * GPIO Registers
> + * Ref: Datasheet sec:A.19
Are those references correct?
> + */
> +struct orion5x_gpio_registers {
> + u32 dout;
> + u32 oe;
> + u32 blink_en;
> + u32 din_pol;
> + u32 din;
> + u32 irq_cause;
> + u32 irq_mask;
> + u32 irq_level;
> +};
> +
> +/*
> + * functions
> + */
> +void reset_cpu(unsigned long ignored);
> +u32 orion5x_device_id(void);
> +u32 orion5x_device_rev(void);
> +unsigned int orion5x_sdram_bar(enum memory_bank bank);
> +unsigned int orion5x_sdram_bs(enum memory_bank bank);
> +int orion5x_config_adr_windows(void);
> +void orion5x_config_gpio(unsigned int gpp_oe_val, unsigned
> int gpp_oe);
> +int orion5x_config_mpp(unsigned int mpp0_7, unsigned int
> mpp8_15, unsigned int mpp16_23);
> +unsigned int orion5x_winctrl_calcsize(unsigned int sizeval);
> +#endif /* __ASSEMBLY__ */
> +#endif /* _ORION5X_CPU_H */
> diff --git a/include/asm-arm/arch-orion5x/gpio.h
> b/include/asm-arm/arch-orion5x/gpio.h
> new file mode 100644
> index 0000000..40fe2e0
> --- /dev/null
> +++ b/include/asm-arm/arch-orion5x/gpio.h
> @@ -0,0 +1,53 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on arch/asm-arm/mach-kirkwood/include/mach/gpio.h
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +/*
> + * Based on (mostly copied from) plat-orion based Linux 2.6
> kernel driver.
> + * Removed kernel level irq handling. Took some macros from kernel to
> + * allow build.
> + *
> + * Dieter Kiermaier dk-arm-linux at gmx.de
> + */
> +
> +#ifndef __ORION5X_GPIO_H
> +#define __ORION5X_GPIO_H
> +
> +/* got from kernel include/linux/bitops.h */
> +#define BITS_PER_BYTE 8
> +#define BITS_TO_LONGS(nr) DIV_ROUND_UP(nr, BITS_PER_BYTE
> * sizeof(long))
> +
> +#define GPIO_MAX 50
Is this correct defination for Orion?
> +#define GPIO_OFF(pin) (((pin) >> 5) ? 0x0040 : 0x0000)
> +#define GPIO_OUT(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x00)
> +#define GPIO_IO_CONF(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x04)
> +#define GPIO_BLINK_EN(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x08)
> +#define GPIO_IN_POL(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x0c)
> +#define GPIO_DATA_IN(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x10)
> +#define GPIO_EDGE_CAUSE(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x14)
> +#define GPIO_EDGE_MASK(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x18)
> +#define GPIO_LEVEL_MASK(pin) (ORION5X_GPIO0_BASE +
> GPIO_OFF(pin) + 0x1c)
> +
> +/*
> + * Kirorion5xood-specific GPIO API
Copy paste error
> + */
> +
> +void orion5x_gpio_set_valid(unsigned pin, int mode);
> +int orion5x_gpio_is_valid(unsigned pin, int mode);
> +int orion5x_gpio_direction_input(unsigned pin);
> +int orion5x_gpio_direction_output(unsigned pin, int value);
> +int orion5x_gpio_get_value(unsigned pin);
> +void orion5x_gpio_set_value(unsigned pin, int value);
> +void orion5x_gpio_set_blink(unsigned pin, int blink);
> +void orion5x_gpio_set_unused(unsigned pin);
> +
> +#define GPIO_INPUT_OK (1 << 0)
> +#define GPIO_OUTPUT_OK (1 << 1)
> +
> +#endif
> diff --git a/include/asm-arm/arch-orion5x/mpp.h
> b/include/asm-arm/arch-orion5x/mpp.h
> new file mode 100644
> index 0000000..a03eb8e
> --- /dev/null
> +++ b/include/asm-arm/arch-orion5x/mpp.h
> @@ -0,0 +1,122 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support which is
> + * Copyright 2009: Marvell Technology Group Ltd.
> + *
> + * This file is licensed under the terms of the GNU General Public
> + * License version 2. This program is licensed "as is" without any
> + * warranty of any kind, whether express or implied.
> + */
> +
> +#ifndef __ORION5X_MPP_H
> +#define __ORION5X_MPP_H
> +
> +#define MPP(_num, _sel, _in, _out, _F5182, _F5281) ( \
> + /* MPP number */ ((_num) & 0xff) | \
> + /* MPP select value */ (((_sel) & 0xf) << 8) | \
> + /* may be input signal */ ((!!(_in)) << 12) | \
> + /* may be output signal */ ((!!(_out)) << 13) | \
> + /* available on F5182 */ ((!!(_F5182)) << 14) | \
> + /* available on F5281 */ ((!!(_F5182)) << 15))
> +
> +#define MPP_NUM(x) ((x) & 0xff)
> +#define MPP_SEL(x) (((x) >> 8) & 0xf)
> +
> + /* num sel i o 5182 5281 */
> +
> +#define MPP_INPUT_MASK MPP( 0, 0x0,
> 1, 0, 0, 0 )
> +#define MPP_OUTPUT_MASK MPP( 0, 0x0,
> 0, 1, 0, 0 )
> +
> +#define MPP_F5182_MASK MPP( 0, 0x0,
> 0, 0, 1, 0 )
> +#define MPP_F5281_MASK MPP( 0, 0x0,
> 0, 0, 0, 1 )
> +
> +#define MPP0_PEX_RST_OUTn MPP( 0, 0x0, 0, 1, 1, 1 )
> +#define MPP0_PCI_REQ2n MPP( 0, 0x2,
> 1, 0, 1, 1 )
> +#define MPP0_GPIO MPP( 0, 0x3, 1, 1, 1, 1 )
> +
> +#define MPP1_GPIO MPP( 1, 0x0, 1, 1, 1, 1 )
> +#define MPP1_PCI_GNT2n MPP( 1, 0x2,
> 0, 1, 1, 1 )
> +
> +#define MPP2_GPIO MPP( 2, 0x0, 1, 1, 1, 1 )
> +#define MPP2_PCI_REQ3n MPP( 2, 0x2,
> 1, 0, 1, 1 )
> +#define MPP2_PMEn MPP( 2, 0x3, 0, 1, 1, 1 )
> +
> +#define MPP3_GPIO MPP( 3, 0x0, 1, 1, 1, 1 )
> +#define MPP3_PCI_GNT3nO MPP( 3, 0x2,
> 0, 1, 1, 1 )
> +
> +#define MPP4_GPIO MPP( 4, 0x0, 1, 1, 1, 1 )
> +#define MPP4_PCI_REQ4n MPP( 4, 0x2,
> 1, 0, 1, 1 )
> +#define MPP4_BOOT_NAND_FLASH_REn MPP( 4, 0x4, 0, 0, 1, 1 )
> +#define MPP4_SATA0_PRESENTn MPP( 4, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP5_GPIO MPP( 5, 0x0, 1, 1, 1, 1 )
> +#define MPP5_PCI_GNT4n MPP( 5, 0x2,
> 0, 1, 1, 1 )
> +#define MPP5_BOOT_NAND_FLASH_WEn MPP( 5, 0x4, 0, 0, 1, 1 )
> +#define MPP5_SATA1_PRESENTn MPP( 5, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP6_GPIO MPP( 6, 0x0, 1, 1, 1, 1 )
> +#define MPP6_PCI_REQ5n MPP( 6, 0x2,
> 1, 0, 1, 1 )
> +#define MPP6_NAND_FLASH_RE0n MPP( 6, 0x4, 0, 0, 1, 1 )
> +#define MPP6_SATA0_ACTn MPP( 6, 0x5,
> 0, 1, 1, 0 )
> +
> +#define MPP7_GPIO MPP( 7, 0x0, 1, 1, 1, 1 )
> +#define MPP7_PCI_GNT5n MPP( 7, 0x2,
> 0, 1, 1, 1 )
> +#define MPP7_NAND_FLASH_WE0n MPP( 7, 0x4, 0, 0, 1, 1 )
> +#define MPP7_SATA1_ACTn MPP( 7, 0x5,
> 0, 1, 1, 0 )
> +
> +#define MPP8_GPIO MPP( 8, 0x0, 1, 1, 1, 1 )
> +#define MPP8_GE_COL MPP( 8, 0x1, 1, 0, 1, 1 )
> +
> +#define MPP9_GPIO MPP( 9, 0x0, 1, 1, 1, 1 )
> +#define MPP9_GE_RXERR MPP( 9, 0x1,
> 1, 0, 1, 1 )
> +
> +#define MPP10_GPIO MPP( 10, 0x0, 1, 1, 1, 1 )
> +#define MPP10_GE_CRS MPP( 10, 0x1, 1, 0, 1, 1 )
> +
> +#define MPP11_GPIO MPP( 11, 0x0, 1, 1, 1, 1 )
> +#define MPP11_GE_TXERR MPP( 11, 0x1,
> 0, 1, 1, 1 )
> +
> +#define MPP12_GPIO MPP( 12, 0x0, 1, 1, 1, 1 )
> +#define MPP12_GE_TXD4 MPP( 12, 0x1,
> 0, 1, 1, 1 )
> +#define MPP12_NAND_FLASH_RE1n MPP( 12, 0x4,
> 1, 1, 1, 1 )
> +#define MPP12_SATA0_PRESENTn MPP( 12, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP13_GPIO MPP( 13, 0x0, 1, 1, 1, 1 )
> +#define MPP13_GE_TXD5 MPP( 13, 0x1,
> 0, 1, 1, 1 )
> +#define MPP13_NAND_FLASH_WE1n MPP( 13, 0x4,
> 0, 1, 1, 1 )
> +#define MPP13_SATA1_PRESENTn MPP( 13, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP14_GPIO MPP( 14, 0x0, 1, 1, 1, 1 )
> +#define MPP14_GE_TXD6 MPP( 14, 0x1,
> 0, 1, 1, 1 )
> +#define MPP14_NAND_FLASH_RE2n MPP( 14, 0x4,
> 1, 1, 1, 1 )
> +#define MPP14_SATA0_ACTn MPP( 14, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP15_GPIO MPP( 15, 0x0, 1, 1, 1, 1 )
> +#define MPP15_GE_TXD7 MPP( 15, 0x1,
> 0, 1, 1, 1 )
> +#define MPP15_NAND_FLASH_WE2n MPP( 15, 0x4,
> 0, 1, 1, 1 )
> +#define MPP15_SATA1_ACTn MPP( 15, 0x5, 0, 1, 1, 0 )
> +
> +#define MPP16_UART1_RXD MPP( 16, 0x0,
> 1, 0, 1, 1 )
> +#define MPP16_GE_RXD4 MPP( 16, 0x1,
> 1, 0, 1, 1 )
> +#define MPP16_BOOT_NAND_FLASH_REn MPP( 16, 0x4, 0, 0, 1, 1 )
> +#define MPP16_GPIO MPP( 16, 0x5, 1, 1, 1, 0 )
> +
> +#define MPP17_UART1_TXD MPP( 17, 0x0,
> 0, 1, 1, 1 )
> +#define MPP17_GE_RXD5 MPP( 17, 0x1,
> 1, 0, 1, 1 )
> +#define MPP17_BOOT_NAND_FLASH_WEn MPP( 17, 0x4, 0, 0, 1, 1 )
> +#define MPP17_GPIO MPP( 17, 0x5, 1, 1, 1, 0 )
> +
> +#define MPP18_UART1_CTSn MPP( 18, 0x0, 1, 0, 1, 1 )
> +#define MPP18_GE_RXD6 MPP( 18, 0x1,
> 1, 0, 1, 1 )
> +#define MPP18_GPIO MPP( 18, 0x5, 1, 1, 1, 0 )
> +
> +#define MPP19_UART1_RTSn MPP( 19, 0x0, 0, 1, 1, 1 )
> +#define MPP19_GE_RXD7 MPP( 19, 0x1,
> 1, 0, 1, 1 )
> +#define MPP19_GPIO MPP( 19, 0x5, 1, 1, 1, 0 )
> +
> +#define MPP_MAX 19
> +
> +void orion5x_mpp_conf(unsigned int *mpp_list);
> +
> +#endif
> diff --git a/include/asm-arm/arch-orion5x/orion5x.h
> b/include/asm-arm/arch-orion5x/orion5x.h
> new file mode 100644
> index 0000000..76cb927
> --- /dev/null
> +++ b/include/asm-arm/arch-orion5x/orion5x.h
> @@ -0,0 +1,67 @@
> +/*
> + * Copyright (C) 2009 Albert ARIBAUD <albert.aribaud@free.fr>
> + *
> + * Based on original Kirkwood support which is
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * Header file for Marvell's Orion SoC with Feroceon CPU core.
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#ifndef _ASM_ARCH_ORION5X_H
> +#define _ASM_ARCH_ORION5X_H
> +
> +#ifndef __ASSEMBLY__
> +#include <asm/types.h>
> +#include <asm/io.h>
> +#endif /* __ASSEMBLY__ */
> +
> +#if defined (CONFIG_FEROCEON)
> +#include <asm/arch/cpu.h>
> +
> +/* SOC specific definations */
> +#define INTREG_BASE 0xd0000000
> +#define ORION5X_REGISTER(x)
> (ORION5X_REGS_PHY_BASE + x)
> +#define ORION5X_OFFSET_REG (INTREG_BASE + 0x20080)
> +
> +/* Documented registers */
> +#define ORION5X_TWSI_BASE
> (ORION5X_REGISTER(0x11000))
> +#define ORION5X_UART0_BASE
> (ORION5X_REGISTER(0x12000))
> +#define ORION5X_UART1_BASE
> (ORION5X_REGISTER(0x12100))
> +#define ORION5X_MPP_BASE
> (ORION5X_REGISTER(0x10000))
> +#define ORION5X_GPIO_BASE
> (ORION5X_REGISTER(0x10100))
> +#define ORION5X_CPU_WIN_BASE
> (ORION5X_REGISTER(0x20000))
> +#define ORION5X_CPU_REG_BASE
> (ORION5X_REGISTER(0x20100))
> +#define ORION5X_TIMER_BASE
> (ORION5X_REGISTER(0x20300))
> +#define ORION5X_REG_PCI_BASE
> (ORION5X_REGISTER(0x30000))
> +#define ORION5X_REG_PCIE_BASE
> (ORION5X_REGISTER(0x40000))
> +#define ORION5X_USB20_PORT0_BASE
> (ORION5X_REGISTER(0x50000))
> +#define ORION5X_USB20_PORT1_BASE
> (ORION5X_REGISTER(0xA0000))
> +#define ORION5X_EGIGA_BASE
> (ORION5X_REGISTER(0x72000))
I hope these are inlined with orion5x and not the result of copy-paste, pls check this carefully
> +
> +#if defined (CONFIG_88F5182)
> +#include <asm/arch/88f5182.h>
> +#else
> +#error "SOC Name not defined"
> +#endif
> +#endif /* CONFIG_FEROCEON */
> +#endif /* _ASM_ARCH_ORION5X_H */
> diff --git a/include/configs/edminiv2.h b/include/configs/edminiv2.h
> new file mode 100644
> index 0000000..a8036dd
> --- /dev/null
> +++ b/include/configs/edminiv2.h
> @@ -0,0 +1,158 @@
> +/*
> + * (C) Copyright 2009
> + * Marvell Semiconductor <www.marvell.com>
> + * Written-by: Prafulla Wadaskar <prafulla@marvell.com>
> + *
> + * See file CREDITS for list of people who contributed to this
> + * project.
> + *
> + * 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; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston,
> + * MA 02110-1301 USA
> + */
> +
> +#ifndef _CONFIG_EDMINIV2_H
> +#define _CONFIG_EDMINIV2_H
> +
> +/*
> + * Version number information
> + */
> +#define CONFIG_IDENT_STRING " Lacie ED Mini V2"
Short board name will be prefered
> +
> +/*
> + * High Level Configuration Options (easy to change)
> + */
> +#define CONFIG_MARVELL 1
> +#define CONFIG_ARM926EJS 1 /* Basic Architecture */
> +#define CONFIG_FEROCEON 1 /* CPU Core
> subversion */
> +#define CONFIG_ORION5X 1 /* SOC Family Name */
> +#define CONFIG_88F5182 1 /* SOC Name */
> +#define CONFIG_MACH_EDMINIV2 1 /* Machine type */
> +
> +#define CONFIG_SKIP_LOWLEVEL_INIT /* disable board
> lowlevel_init */
> +
> +/*
> + * CLKs configurations
> + */
> +#define CONFIG_SYS_HZ 1000
> +
> +/*
> + * NS16550 Configuration
> + */
> +#define CONFIG_SYS_NS16550
> +#define CONFIG_SYS_NS16550_SERIAL
> +#define CONFIG_SYS_NS16550_REG_SIZE (-4)
> +#define CONFIG_SYS_NS16550_CLK CONFIG_SYS_TCLK
> +#define CONFIG_SYS_NS16550_COM1 ORION5X_UART0_BASE
> +
> +/*
> + * Serial Port configuration
> + * The following definitions let you select what serial you
> want to use
> + * for your console driver.
> + */
> +
> +#define CONFIG_CONS_INDEX 1 /*Console on UART0 */
> +#define CONFIG_BAUDRATE 115200
> +#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, \
> + 115200,230400,
> 460800, 921600 }
> +/*
> + * FLASH configuration
> + */
> +
> +#define CONFIG_SYS_FLASH_CFI
> +#define CONFIG_FLASH_CFI_DRIVER
> +#define CONFIG_FLASH_CFI_WIDTH FLASH_CFI_8BIT
> +#define CONFIG_FLASH_CFI_LEGACY
> +
> +//#define CONFIG_SYS_FLASH_LEGACY_512Kx8
> +#define CONFIG_SYS_MAX_FLASH_BANKS 1 /* max num
> of flash banks */
> +#define CONFIG_SYS_MAX_FLASH_SECT 11 /* max num
> of sects on one chip */
> +#define CONFIG_SYS_FLASH_BASE 0xfff80000
> +#define CONFIG_SYS_FLASH_SECTSZ
> {16384,8192,8192,32768,65536,65536,65536,65536,65536,65536,65536}
> +
> +/* auto boot */
> +#define CONFIG_BOOTDELAY 3 /* default enable autoboot */
> +
> +/*
> + * For booting Linux, the board info and command line data
> + * have to be in the first 8 MB of memory, since this is
> + * the maximum mapped by the Linux kernel during initialization.
> + */
> +#define CONFIG_CMDLINE_TAG 1 /* enable passing of ATAGs */
> +#define CONFIG_INITRD_TAG 1 /* enable INITRD tag */
> +#define CONFIG_SETUP_MEMORY_TAGS 1 /* enable memory tag */
> +
> +#define CONFIG_SYS_PROMPT "Marvell>> " /*
Is this Marvell custom board ?
If not, even you can choose to keep in in boards instead of boards/Marvell/
> Command Prompt */
> +#define CONFIG_SYS_CBSIZE 1024 /* Console I/O
> Buff Size */
> +#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE \
> + +sizeof(CONFIG_SYS_PROMPT) + 16) /* Print Buff */
> +/*
> + * Commands configuration
> + */
> +/*
> + * Disabling many default commands for staggered bring-up
> + */
> +#if 0
This is not allowed, pls remove it, or use some defination here
I suggest to use config_cmd_default.h folloed by def/undefs for required commands to inline with other arm boards configuration
> +#include <config_cmd_default.h>
> +#else
> +#define CONFIG_CMD_BDI /* bdinfo */
> +#define CONFIG_CMD_CONSOLE /* coninfo */
> +#define CONFIG_CMD_ECHO /* echo arguments */
> +#define CONFIG_CMD_EDITENV /* editenv */
> +#define CONFIG_CMD_IMI /* iminfo */
> +#define CONFIG_CMD_ITEST /* Integer (and string) test */
> +#define CONFIG_CMD_FLASH /* flinfo, erase, protect */
> +#define CONFIG_CMD_IMLS /* List all found images */
> +#define CONFIG_CMD_LOADB /* loadb */
> +#define CONFIG_CMD_LOADS /* loads */
> +#define CONFIG_CMD_MEMORY /* md mm nm mw cp cmp crc
> base loop mtest */
> +#define CONFIG_CMD_MISC /* Misc functions like sleep etc*/
> +#define CONFIG_CMD_RUN /* run command in env variable */
> +#define CONFIG_CMD_SAVEENV /* saveenv */
> +#define CONFIG_CMD_SOURCE /* "source" command support */
> +#endif
> +#define CONFIG_CMD_AUTOSCRIPT
> +#define CONFIG_CMD_ENV
> +
> +/*
> + * Environment variables configurations
> + */
> +#define CONFIG_ENV_IS_IN_FLASH 1
> +#define CONFIG_ENV_SECT_SIZE 0x2000 /* 16K */
> +#define CONFIG_ENV_SIZE 0x2000
> +#define CONFIG_ENV_OFFSET 0x4000 /* env starts here */
> +
> +/*
> + * Size of malloc() pool
> + */
> +#define CONFIG_SYS_MALLOC_LEN (1024 * 128) /* 128kB
> for malloc() */
> +/* size in bytes reserved for initial data */
> +#define CONFIG_SYS_GBL_DATA_SIZE 128
> +
> +/*
> + * Other required minimal configurations
> + */
> +#define CONFIG_CONSOLE_INFO_QUIET /* some code reduction */
> +#define CONFIG_ARCH_CPU_INIT /* call arch_cpu_init() */
> +#define CONFIG_ARCH_MISC_INIT /* call arch_misc_init() */
> +#define CONFIG_DISPLAY_CPUINFO /* Display cpu info */
> +#define CONFIG_NR_DRAM_BANKS 4
> +#define CONFIG_STACKSIZE 0x00100000 /* regular stack- 1M */
> +#define CONFIG_SYS_LOAD_ADDR 0x00800000 /* default load
> adr- 8M */
> +#define CONFIG_SYS_MEMTEST_START 0x00400000 /* 4M */
> +#define CONFIG_SYS_MEMTEST_END 0x007fffff /*(_8M -1) */
> +#define CONFIG_SYS_RESET_ADDRESS 0xffff0000 /* Rst Vector Adr */
> +#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
> +
> +#endif /* _CONFIG_EDMINIV2_H */
> --
> 1.6.4.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
2009-11-14 6:29 ` Prafulla Wadaskar
@ 2009-11-14 9:22 ` Albert ARIBAUD
2009-11-14 11:59 ` Prafulla Wadaskar
0 siblings, 1 reply; 6+ messages in thread
From: Albert ARIBAUD @ 2009-11-14 9:22 UTC (permalink / raw)
To: u-boot
Hi Prafulla and all,
Prafulla Wadaskar a ?crit :
>
>> -----Original Message-----
>> From: u-boot-bounces at lists.denx.de
>> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
>> Sent: Saturday, November 14, 2009 6:32 AM
>> To: U-Boot at lists.denx.de
>> Subject: [U-Boot] [PATCH] Initial support for Orion5x SoC and
>> EDMini board
>>
>> This patch adds initial u-boot support for the Marvell orion5x
>> SoC and the LAcie ED Mini V2 board--support is limited to serial,
>> flash and environment. Further support will be added as devices
>> common with kirkwood are made useable across SoCs.
>>
>> Signed-off-by: Albert Aribaud <albert.aribaud@free.fr>
>> ---
>
> Hi Albert
>
> Please break this patch in to small pieces
> [...]
> Pls edit MAKEALL and Maintainers files too for new board support
> [...]
> Arm: add Edminv2 board support (orion5x based)
Ok.
> Break this in to basic orion5x Soc support patch
> And individual drivers patches like gpio, uart, etc..
Will do. What exact criteria should I use for splitting patches?
Obviously some patches will never be compiled alone, e.g. basic Orion5x
support won't be compiled until board support patches come in; also,
having SoC and board support without uart support, if it even compiles,
will be pretty useless as the resulting u-boot won't even have a console.
>> +# (C) Copyright 2009
>> +# Marvell Semiconductor <www.marvell.com>
>> +# Written-by: Prafulla Wadaskar <prafulla@marvell.com>
>
> This is copy paste mistake, pls correct in all files
> You can keep my reference, i.e. based on (ref: boards/Marvell/openrd_base/openrd_base.c)
Ok.
>> +TEXT_BASE = 0x00600000
>
> Is this valid for your board? BTW: how much DRAM you have on this board?
Yes it works for the board--I'm using this value for testing. However,
in very old and non-reuseable u-boot code, it was 0x00F00000. I'll
switch to this in order to minimize the difference between these older
u-boots and the current mainline one.
The board has 64M RAM.
>> + /*
>> + * default gpio configuration
>> + * There are maximum 64 gpios controlled through 2 sets
>> of registers
>
> I don't think there are 64 GPIOs in orion5x
Right, I'll fix the comment. That's 24 (with only 20 actually referenced
by the board).
>> + orion5x_config_gpio(EDMINIV2_OE_VAL_LOW, EDMINIV2_OE_LOW);
>
> You can eliminate LOW here, since mpps less than 3
> Pls remove word LOW here
Ok.
>> diff --git a/cpu/arm926ejs/orion5x/Makefile
>> [...]
>> +COBJS-y = dram.o
>> +COBJS-y += cpu.o
>
> Pls put these lines in order
Ok. Note this comes directly from kirkwood's Makefile, so the lines are
in the same order (dram, cpu, mpp, timer) there too.
>> + * There are maximum 64 Multi-Pourpose Pins on Orion5x
> Correct this -64
Will do.
>> +#define PCIE_DEV_ID_OFF (ORION5X_REG_PCIE_BASE + 0x0000)
>> +#define PCIE_DEV_REV_OFF (ORION5X_REG_PCIE_BASE + 0x0008)
>
> Pls move the defination to header file
Will do.
>> + if (dev == MV88F5181_DEV_ID) {
>> + dev_name = "MV88F5181";
>> + if (rev == MV88F5181_REV_B1) {
>> + rev_name = "B1";
>> + } else if (rev == MV88F5181L_REV_A1) {
>> + dev_name = "MV88F5181L";
>> + rev_name = "A1";
>> + } else if (rev == MV88F5181L_REV_A0) {
>> + dev_name = "MV88F5181L";
>> + rev_name = "A0";
>> + } else {
>> + sprintf(rev_str,"0x%02x", rev);
>> + }
>> + } else if (dev == MV88F5182_DEV_ID) {
>> + dev_name = "MV88F5182";
>> + if (rev == MV88F5182_REV_A2) {
>> + rev_name = "A2";
>> + } else {
>> + sprintf(rev_str,"0x%02x", rev);
>> + }
>> + } else if (dev == MV88F5281_DEV_ID) {
>> + dev_name = "MV88F5281";
>> + if (rev == MV88F5281_REV_D2) {
>> + rev_name = "D2";
>> + } else if (rev == MV88F5281_REV_D1) {
>> + rev_name = "D1";
>> + } else if (rev == MV88F5281_REV_D0) {
>> + rev_name = "D0";
>> + } else {
>> + sprintf(rev_str,"0x%02x", rev);
>> + }
>> + } else if (dev == MV88F6183_DEV_ID) {
>> + dev_name = "MV88F6183";
>> + if (rev == MV88F6183_REV_B0) {
>> + rev_name = "B0";
>> + } else {
>> + sprintf(rev_str,"0x%02x", rev);
>> + }
>> + } else {
>> + sprintf(dev_str,"0x%04x", rev);
>> + sprintf(rev_str,"0x%02x", rev);
>
> This is common line, pls take out of if-else
Can you be more specific? The outer 'if' sequence deals with device id
and I see no no common line in its then/.../else branches, and the inner
ifs are for revision and again, I cannot pinpoint the common line.
Unless you would want the sprintf's to be done inconditionally before
entering the if's?
Note however that there is a typo:
+ } else {
+ sprintf(dev_str,"0x%04x", rev);
+ sprintf(rev_str,"0x%02x", rev);
Should actually be
+ } else {
+ sprintf(dev_str,"0x%04x", dev); <- this is dev, not rev
+ sprintf(rev_str,"0x%02x", rev);
>> +#ifdef CONFIG_ORION5X_EGIGA
>> +int cpu_eth_init(bd_t *bis)
>> +{
>> + orion5x_egiga_initialize(bis);
>> + return 0;
>> +}
>> +#endif
>
> Pls do not add this by default, make it a part of egiga driver patch
Will do.
>> + * read feroceon/sheeva core extra feature register
>
> Pls remove name sheeva
>
>> + * write feroceon/sheeva core extra feature register
>
> Same
Right.
>> + * MBus-L to Mbus Bridge Registers
>> + * Ref: Datasheet sec:A.3
>
> Are those references correct?
No, they're not, they're kirkwood's. I'll do a pass on these and fix
them wrt orion.
>> +#define GPIO_MAX 50
>
> Is this correct defination for Orion?
No. Should be 26. Will fix and add a reference to the specs.
>> + * Kirorion5xood-specific GPIO API
>
> Copy paste error
Worse, actually: it is a ':%s//' search/replace error. Will fix.
>> +/* Documented registers */
>> +#define ORION5X_TWSI_BASE
>> (ORION5X_REGISTER(0x11000))
>> +#define ORION5X_UART0_BASE
>> (ORION5X_REGISTER(0x12000))
>> +#define ORION5X_UART1_BASE
>> (ORION5X_REGISTER(0x12100))
>> +#define ORION5X_MPP_BASE
>> (ORION5X_REGISTER(0x10000))
>> +#define ORION5X_GPIO_BASE
>> (ORION5X_REGISTER(0x10100))
>> +#define ORION5X_CPU_WIN_BASE
>> (ORION5X_REGISTER(0x20000))
>> +#define ORION5X_CPU_REG_BASE
>> (ORION5X_REGISTER(0x20100))
>> +#define ORION5X_TIMER_BASE
>> (ORION5X_REGISTER(0x20300))
>> +#define ORION5X_REG_PCI_BASE
>> (ORION5X_REGISTER(0x30000))
>> +#define ORION5X_REG_PCIE_BASE
>> (ORION5X_REGISTER(0x40000))
>> +#define ORION5X_USB20_PORT0_BASE
>> (ORION5X_REGISTER(0x50000))
>> +#define ORION5X_USB20_PORT1_BASE
>> (ORION5X_REGISTER(0xA0000))
>> +#define ORION5X_EGIGA_BASE
>> (ORION5X_REGISTER(0x72000))
>
> I hope these are inlined with orion5x and not the result of copy-paste, pls check this carefully
Yes they are in line--while I started off a copy of the kirkwood list, I
took extra care to match each one with Orion5x's spec.
>> +#define CONFIG_IDENT_STRING " Lacie ED Mini V2"
>
> Short board name will be prefered
Understood, will put " EDMiniV2".
>> +#define CONFIG_SYS_PROMPT "Marvell>> " /*
>
> Is this Marvell custom board ?
> If not, even you can choose to keep in in boards instead of boards/Marvell/
No, it's not a Marvell custom board, it's a LaCie product board. I'll
move the board to boards/ directly (or maybe "boards/lacie", to provide
a home for other lacie boards? Any best pratice here to follow?) and
change the prompt to "EDMiniV2".
>> +/*
>> + * Disabling many default commands for staggered bring-up
>> + */
>> +#if 0
>
> This is not allowed, pls remove it, or use some defination here
> I suggest to use config_cmd_default.h folloed by def/undefs for required commands to inline with other arm boards configuration
Will do.
Thanks a lot for the review. I'll fix the issues you pointed out and
resubmit a patchset.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
2009-11-14 9:22 ` Albert ARIBAUD
@ 2009-11-14 11:59 ` Prafulla Wadaskar
2009-11-14 13:03 ` Albert ARIBAUD
0 siblings, 1 reply; 6+ messages in thread
From: Prafulla Wadaskar @ 2009-11-14 11:59 UTC (permalink / raw)
To: u-boot
> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.aribaud at free.fr]
> Sent: Saturday, November 14, 2009 2:53 PM
> To: Prafulla Wadaskar
> Cc: U-Boot at lists.denx.de; Ashish Karkare; Prabhanjan Sarnaik
> Subject: Re: [U-Boot] [PATCH] Initial support for Orion5x SoC
> and EDMini board
>
> Hi Prafulla and all,
>
> Prafulla Wadaskar a ?crit :
> >
> >> -----Original Message-----
> >> From: u-boot-bounces at lists.denx.de
> >> [mailto:u-boot-bounces at lists.denx.de] On Behalf Of Albert Aribaud
> >> Sent: Saturday, November 14, 2009 6:32 AM
> >> To: U-Boot at lists.denx.de
> >> Subject: [U-Boot] [PATCH] Initial support for Orion5x SoC and
> >> EDMini board
> >>
> >> This patch adds initial u-boot support for the Marvell orion5x
> >> SoC and the LAcie ED Mini V2 board--support is limited to serial,
> >> flash and environment. Further support will be added as devices
> >> common with kirkwood are made useable across SoCs.
> >>
> >> Signed-off-by: Albert Aribaud <albert.aribaud@free.fr>
> >> ---
> >
> > Hi Albert
> >
> > Please break this patch in to small pieces
> > [...]
> > Pls edit MAKEALL and Maintainers files too for new board support
> > [...]
> > Arm: add Edminv2 board support (orion5x based)
>
> Ok.
>
> > Break this in to basic orion5x Soc support patch
> > And individual drivers patches like gpio, uart, etc..
>
> Will do. What exact criteria should I use for splitting patches?
> Obviously some patches will never be compiled alone, e.g.
> basic Orion5x
> support won't be compiled until board support patches come in; also,
> having SoC and board support without uart support, if it even
> compiles,
> will be pretty useless as the resulting u-boot won't even
> have a console.
It should be patch series with properly addressed dependencies
i.e.
(1/4)basic orion soc support patch
(2/4)orion gpio driver
(3/4)orion UART support
(4/4) Board support patch
All patches when applied in series should build u-boot for respective board.
The spilt has to be done carefully because individual patch may be applied to different repositories,
like nandf to u-boot-nand.git, basic orion support to u-boot-marvell.git, egiga driver patch will go in u-boot-net.git etc
> >> +TEXT_BASE = 0x00600000
> >
> > Is this valid for your board? BTW: how much DRAM you have
> on this board?
>
> Yes it works for the board--I'm using this value for testing.
> However,
> in very old and non-reuseable u-boot code, it was 0x00F00000. I'll
> switch to this in order to minimize the difference between
> these older
> u-boots and the current mainline one.
I will suggest to use lower part of available ram for this
> >> diff --git a/cpu/arm926ejs/orion5x/Makefile
> >> [...]
> >> +COBJS-y = dram.o
> >> +COBJS-y += cpu.o
> >
> > Pls put these lines in order
>
> Ok. Note this comes directly from kirkwood's Makefile, so the
> lines are
Thanks , I will provide separate patch for this
>
> >> + if (dev == MV88F5181_DEV_ID) {
> >> + dev_name = "MV88F5181";
> >> + if (rev == MV88F5181_REV_B1) {
> >> + rev_name = "B1";
> >> + } else if (rev == MV88F5181L_REV_A1) {
> >> + dev_name = "MV88F5181L";
> >> + rev_name = "A1";
> >> + } else if (rev == MV88F5181L_REV_A0) {
> >> + dev_name = "MV88F5181L";
> >> + rev_name = "A0";
> >> + } else {
> >> + sprintf(rev_str,"0x%02x", rev);
> >> + }
> >> + } else if (dev == MV88F5182_DEV_ID) {
> >> + dev_name = "MV88F5182";
> >> + if (rev == MV88F5182_REV_A2) {
> >> + rev_name = "A2";
> >> + } else {
> >> + sprintf(rev_str,"0x%02x", rev);
> >> + }
> >> + } else if (dev == MV88F5281_DEV_ID) {
> >> + dev_name = "MV88F5281";
> >> + if (rev == MV88F5281_REV_D2) {
> >> + rev_name = "D2";
> >> + } else if (rev == MV88F5281_REV_D1) {
> >> + rev_name = "D1";
> >> + } else if (rev == MV88F5281_REV_D0) {
> >> + rev_name = "D0";
> >> + } else {
> >> + sprintf(rev_str,"0x%02x", rev);
> >> + }
> >> + } else if (dev == MV88F6183_DEV_ID) {
> >> + dev_name = "MV88F6183";
> >> + if (rev == MV88F6183_REV_B0) {
> >> + rev_name = "B0";
> >> + } else {
> >> + sprintf(rev_str,"0x%02x", rev);
> >> + }
> >> + } else {
> >> + sprintf(dev_str,"0x%04x", rev);
> >> + sprintf(rev_str,"0x%02x", rev);
> >
> > This is common line, pls take out of if-else
>
> Can you be more specific? The outer 'if' sequence deals with
> device id
Okay I will take this back
Whereas I found
+ sprintf(rev_str,"0x%02x", rev);
At so many places, can you reduce it?
> > Is this Marvell custom board ?
> > If not, even you can choose to keep in in boards instead of
> boards/Marvell/
>
> No, it's not a Marvell custom board, it's a LaCie product board. I'll
> move the board to boards/ directly (or maybe "boards/lacie",
> to provide
> a home for other lacie boards? Any best pratice here to
> follow?) and
> change the prompt to "EDMiniV2".
That's good approach for this board, if you are planning to put support for more Lacie boards, then its better to have boards/lacie
Regards..
Prafulla . .
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board
2009-11-14 11:59 ` Prafulla Wadaskar
@ 2009-11-14 13:03 ` Albert ARIBAUD
0 siblings, 0 replies; 6+ messages in thread
From: Albert ARIBAUD @ 2009-11-14 13:03 UTC (permalink / raw)
To: u-boot
Hi again Prafulla,
Prafulla Wadaskar a ?crit :
>>> Break this in to basic orion5x Soc support patch
>>> And individual drivers patches like gpio, uart, etc..
>>
>> Will do. What exact criteria should I use for splitting patches?
>> Obviously some patches will never be compiled alone, e.g.
>> basic Orion5x support won't be compiled until board support patches
>> come in; also, having SoC and board support without uart support,
>> if it even compiles, will be pretty useless as the resulting
>> u-boot won't even have a console.
>
> It should be patch series with properly addressed dependencies
> i.e.
> (1/4)basic orion soc support patch
> (2/4)orion gpio driver
> (3/4)orion UART support
> (4/4) Board support patch
>
> All patches when applied in series should build u-boot for respective board.
> The spilt has to be done carefully because individual patch may be applied to different repositories,
> like nandf to u-boot-nand.git, basic orion support to u-boot-marvell.git, egiga driver patch will go in u-boot-net.git etc
Duly noted. Currently, I only add support for SoC, UART and NOT FLASH
(not NAND) so only u-boot-marvell.git needs to be considered; I'll keep
other repos in mind for upcoming patches like egiga and others.
>>>> +TEXT_BASE = 0x00600000
>>> Is this valid for your board? BTW: how much DRAM you have
>> on this board?
>>
>> Yes it works for the board--I'm using this value for testing.
>> However,
>> in very old and non-reuseable u-boot code, it was 0x00F00000. I'll
>> switch to this in order to minimize the difference between
>> these older u-boots and the current mainline one.
>
> I will suggest to use lower part of available ram for this
Understood. Note the 0x00600000 value comes from kirkwood too, in case
this matters.
>>>> + if (dev == MV88F5181_DEV_ID) {
>>>> + dev_name = "MV88F5181";
>>>> + if (rev == MV88F5181_REV_B1) {
>>>> + rev_name = "B1";
>>>> + } else if (rev == MV88F5181L_REV_A1) {
>>>> + dev_name = "MV88F5181L";
>>>> + rev_name = "A1";
>>>> + } else if (rev == MV88F5181L_REV_A0) {
>>>> + dev_name = "MV88F5181L";
>>>> + rev_name = "A0";
>>>> + } else {
>>>> + sprintf(rev_str,"0x%02x", rev);
>>>> + }
>>>> + } else if (dev == MV88F5182_DEV_ID) {
>>>> + dev_name = "MV88F5182";
>>>> + if (rev == MV88F5182_REV_A2) {
>>>> + rev_name = "A2";
>>>> + } else {
>>>> + sprintf(rev_str,"0x%02x", rev);
>>>> + }
>>>> + } else if (dev == MV88F5281_DEV_ID) {
>>>> + dev_name = "MV88F5281";
>>>> + if (rev == MV88F5281_REV_D2) {
>>>> + rev_name = "D2";
>>>> + } else if (rev == MV88F5281_REV_D1) {
>>>> + rev_name = "D1";
>>>> + } else if (rev == MV88F5281_REV_D0) {
>>>> + rev_name = "D0";
>>>> + } else {
>>>> + sprintf(rev_str,"0x%02x", rev);
>>>> + }
>>>> + } else if (dev == MV88F6183_DEV_ID) {
>>>> + dev_name = "MV88F6183";
>>>> + if (rev == MV88F6183_REV_B0) {
>>>> + rev_name = "B0";
>>>> + } else {
>>>> + sprintf(rev_str,"0x%02x", rev);
>>>> + }
>>>> + } else {
>>>> + sprintf(dev_str,"0x%04x", rev);
>>>> + sprintf(rev_str,"0x%02x", rev);
>>> This is common line, pls take out of if-else
>> Can you be more specific? The outer 'if' sequence deals with
>> device id
>
> Okay I will take this back
> Whereas I found
> + sprintf(rev_str,"0x%02x", rev);
> At so many places, can you reduce it?
Hmm yes, I think I can start by setting dev_name/rev_name tu NULL, go
through the ifs and set dev_name/rev_name only if appropriate, and then
do the sprintf()s only if dev_name/rev_name are still NULL after the ifs.
>>> Is this Marvell custom board ?
>>> If not, even you can choose to keep in in boards instead of
>> boards/Marvell/
>>
>> No, it's not a Marvell custom board, it's a LaCie product board. I'll
>> move the board to boards/ directly (or maybe "boards/lacie",
>> to provide
>> a home for other lacie boards? Any best pratice here to
>> follow?) and
>> change the prompt to "EDMiniV2".
>
> That's good approach for this board, if you are planning to put support for more Lacie boards, then its better to have boards/lacie
I guess there's at least a second one to come, so I'll go for the
boards/lacie/* approach.
> Regards..
> Prafulla . .
Thanks again!
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-11-14 13:03 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-14 1:01 [U-Boot] [PATCH] Initial support for Orion5x SoC and EDMini board Albert Aribaud
2009-11-14 2:21 ` Tom
2009-11-14 6:29 ` Prafulla Wadaskar
2009-11-14 9:22 ` Albert ARIBAUD
2009-11-14 11:59 ` Prafulla Wadaskar
2009-11-14 13:03 ` Albert ARIBAUD
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox