* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
@ 2011-10-01 0:12 Marek Vasut
2011-10-01 0:12 ` [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board Marek Vasut
2011-10-05 14:10 ` [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Simon Glass
0 siblings, 2 replies; 14+ messages in thread
From: Marek Vasut @ 2011-10-01 0:12 UTC (permalink / raw)
To: u-boot
The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed configurable.
Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Ben Warren <biggerbadderben@gmail.com>
Cc: Tom Warren <twarren.nvidia@gmail.com>
Cc: Stephen Warren <swarren@nvidia.com>
---
arch/arm/cpu/armv7/tegra2/ap20.c | 9 +++++++--
1 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c b/arch/arm/cpu/armv7/tegra2/ap20.c
index dc5f984..71d9f26 100644
--- a/arch/arm/cpu/armv7/tegra2/ap20.c
+++ b/arch/arm/cpu/armv7/tegra2/ap20.c
@@ -31,6 +31,11 @@
#include <asm/arch/scu.h>
#include <common.h>
+/* The default XTal is 12MHz, some boards might use 13MHz one though */
+#ifndef CONFIG_SYS_TEGRA2_XTAL_MHZ
+#define CONFIG_SYS_TEGRA2_XTAL_MHZ 12
+#endif
+
u32 s_first_boot = 1;
void init_pllx(void)
@@ -46,8 +51,8 @@ void init_pllx(void)
/* Set PLLX_MISC */
writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
- /* Use 12MHz clock here */
- reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
+ /* Use 13MHz clock here */
+ reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ << PLL_DIVM_SHIFT);
reg |= 1000 << PLL_DIVN_SHIFT;
writel(reg, &pll->pll_base);
--
1.7.5.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board
2011-10-01 0:12 [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Marek Vasut
@ 2011-10-01 0:12 ` Marek Vasut
2011-10-03 3:52 ` Simon Glass
2011-10-05 14:10 ` [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Simon Glass
1 sibling, 1 reply; 14+ messages in thread
From: Marek Vasut @ 2011-10-01 0:12 UTC (permalink / raw)
To: u-boot
Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Ben Warren <biggerbadderben@gmail.com>
Cc: Tom Warren <twarren.nvidia@gmail.com>
Cc: Stephen Warren <swarren@nvidia.com>
---
MAINTAINERS | 1 +
board/toradex/colibri_t20/Makefile | 49 +++++++++
board/toradex/colibri_t20/colibri_t20.c | 119 ++++++++++++++++++++++
boards.cfg | 1 +
include/configs/colibri_t20.h | 162 +++++++++++++++++++++++++++++++
5 files changed, 332 insertions(+), 0 deletions(-)
create mode 100644 board/toradex/colibri_t20/Makefile
create mode 100644 board/toradex/colibri_t20/colibri_t20.c
create mode 100644 include/configs/colibri_t20.h
Btw. is that a pure coincidence that there are three Warrens maintaining Tegras?
Or are you guys some kind of Warren-brothers ? ;-)
diff --git a/MAINTAINERS b/MAINTAINERS
index c10996c..db7d85a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -853,6 +853,7 @@ Marek Vasut <marek.vasut@gmail.com>
zipitz2 xscale/pxa
efikamx i.MX51
efikasb i.MX51
+ colibri_t20 Tegra2 (ARM7 & A9 Dual Core)
Hugo Villeneuve <hugo.villeneuve@lyrtech.com>
diff --git a/board/toradex/colibri_t20/Makefile b/board/toradex/colibri_t20/Makefile
new file mode 100644
index 0000000..5002dd5
--- /dev/null
+++ b/board/toradex/colibri_t20/Makefile
@@ -0,0 +1,49 @@
+#
+# (C) Copyright 2000, 2001, 2002
+# Wolfgang Denk, DENX Software Engineering, wd at denx.de.
+#
+# 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., 59 Temple Place, Suite 330, Boston,
+# MA 02111-1307 USA
+#
+
+include $(TOPDIR)/config.mk
+
+LIB = $(obj)lib$(BOARD).o
+
+COBJS := colibri_t20.o
+
+SRCS := $(COBJS:.o=.c)
+OBJS := $(addprefix $(obj),$(COBJS))
+
+$(LIB): $(obj).depend $(OBJS)
+ $(call cmd_link_o_target, $(OBJS))
+
+clean:
+ rm -f $(OBJS)
+
+distclean: clean
+ rm -f $(LIB) core *.bak $(obj).depend
+
+#########################################################################
+
+# defines $(obj).depend target
+include $(SRCTREE)/rules.mk
+
+sinclude $(obj).depend
+
+#########################################################################
diff --git a/board/toradex/colibri_t20/colibri_t20.c b/board/toradex/colibri_t20/colibri_t20.c
new file mode 100644
index 0000000..4945888
--- /dev/null
+++ b/board/toradex/colibri_t20/colibri_t20.c
@@ -0,0 +1,119 @@
+/*
+ * Toradex Colibri T20 Board Support
+ *
+ * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
+ *
+ * Based on code:
+ *
+ * Copyright (C) 2011 Ant Micro <www.antmicro.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., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#include <common.h>
+#include <ns16550.h>
+#include <asm/io.h>
+#include <asm/arch/tegra2.h>
+#include <asm/arch/sys_proto.h>
+
+#include <asm/arch/clk_rst.h>
+#include <asm/arch/clock.h>
+#include <asm/arch/pinmux.h>
+#include <asm/arch/uart.h>
+
+DECLARE_GLOBAL_DATA_PTR;
+
+const struct tegra2_sysinfo sysinfo = {
+ CONFIG_TEGRA2_BOARD_STRING
+};
+
+int timer_init(void)
+{
+ return 0;
+}
+
+static void uart_init(void)
+{
+ struct clk_rst_ctlr *clkrst = (struct clk_rst_ctlr *)NV_PA_CLK_RST_BASE;
+ struct clk_pll *pll = &clkrst->crc_pll[CLOCK_PLL_ID_PERIPH];
+ uint32_t reg;
+
+ reg = readl(&pll->pll_base);
+ if (!(reg & PLL_BASE_OVRRIDE_MASK)) {
+ /* Override pllp setup for 216MHz operation. */
+ reg = PLL_BYPASS_MASK | PLL_BASE_OVRRIDE_MASK |
+ (1 << PLL_DIVP_SHIFT) | (0xd << PLL_DIVM_SHIFT);
+ reg |= (NVRM_PLLP_FIXED_FREQ_KHZ / 500) << PLL_DIVN_SHIFT;
+ writel(reg, &pll->pll_base);
+
+ reg |= PLL_ENABLE_MASK;
+ writel(reg, &pll->pll_base);
+
+ reg &= ~PLL_BYPASS_MASK;
+ writel(reg, &pll->pll_base);
+ }
+
+#ifdef CONFIG_TEGRA2_ENABLE_UARTA
+ /* Assert UART reset and enable clock */
+ reset_set_enable(PERIPH_ID_UART1, 1);
+ clock_enable(PERIPH_ID_UART1);
+
+ /* Enable pllp_out0 to UART */
+ clrbits_le32(&clkrst->crc_clk_src_uarta, 3 << 30);
+
+ /* wait for 2us */
+ udelay(2);
+
+ /* De-assert reset to UART */
+ reset_set_enable(PERIPH_ID_UART1, 0);
+#endif
+}
+
+static void pinmux_init(void)
+{
+ struct pmux_tri_ctlr *pmt = (struct pmux_tri_ctlr *)NV_PA_APB_MISC_BASE;
+
+ /* SDIO 1 */
+ clrbits_le32(&pmt->pmt_tri[TRISTATE_REG(PIN_ATA)], 1 << 30);
+
+ /* Mux SDB/SDC to PWM */
+ clrsetbits_le32(&pmt->pmt_ctl_d, 0xcc00, 0x4400);
+
+ /* Mux SDIO1 to UARTA */
+ setbits_le32(&pmt->pmt_ctl_a, 3 << 30);
+
+ return 0;
+}
+
+#ifdef CONFIG_BOARD_EARLY_INIT_F
+extern int tegra2_start(void);
+int board_early_init_f(void)
+{
+ pinmux_init();
+ uart_init();
+ tegra2_start();
+ return 0;
+}
+#endif
+
+int board_init(void)
+{
+ gd->bd->bi_boot_params = NV_PA_SDRAM_BASE + 0x100;
+ return 0;
+}
diff --git a/boards.cfg b/boards.cfg
index c485cd7..6cc2855 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -188,6 +188,7 @@ s5pc210_universal arm armv7 universal_c210 samsung
smdkv310 arm armv7 smdkv310 samsung s5pc2xx
harmony arm armv7 harmony nvidia tegra2
seaboard arm armv7 seaboard nvidia tegra2
+colibri_t20 arm armv7 colibri_t20 toradex tegra2
u8500_href arm armv7 u8500 st-ericsson u8500
actux1_4_16 arm ixp actux1 - - actux1:FLASH2X2
actux1_8_16 arm ixp actux1 - - actux1:FLASH1X8
diff --git a/include/configs/colibri_t20.h b/include/configs/colibri_t20.h
new file mode 100644
index 0000000..8fc0898
--- /dev/null
+++ b/include/configs/colibri_t20.h
@@ -0,0 +1,162 @@
+/*
+ * Toradex Colibri T20 Configuration File
+ *
+ * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
+ *
+ * Based on code:
+ *
+ * Copyright (C) 2011 Ant Micro <www.antmicro.com>
+ * Copyright (C) 2010, 2011 NVIDIA Corporation <www.nvidia.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., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ */
+
+#ifndef __CONFIG_H
+#define __CONFIG_H
+
+#include <asm/sizes.h>
+#include <asm/arch/tegra2.h>
+
+/*
+ * High Level Configuration Options
+ */
+#define CONFIG_ARMCORTEXA9 /* This is an ARM V7 CPU core */
+#define CONFIG_TEGRA2 /* in a NVidia Tegra2 core */
+#define CONFIG_MACH_TEGRA_GENERIC /* which is a Tegra generic machine */
+#define CONFIG_SYS_L2CACHE_OFF /* No L2 cache */
+
+#define CONFIG_ENABLE_CORTEXA9 /* enable CPU (A9 complex) */
+
+/*
+ * Board-specific config
+ */
+#define CONFIG_SYS_BOARD_ODMDATA 0x100d8011 /* lp1, 256MB */
+#define CONFIG_SYS_CPU_OSC_FREQUENCY 1000000 /* Set CPU clock to 1GHz */
+#define CONFIG_SYS_TEGRA2_XTAL_MHZ 13
+
+#define CONFIG_BOARD_EARLY_INIT_F
+#define CONFIG_MACH_TYPE MACH_TYPE_COLIBRI_TEGRA2
+#define CONFIG_ENV_IS_NOWHERE
+#define CONFIG_ENV_SIZE 0x20000
+#define CONFIG_ENV_OVERWRITE
+
+#define CONFIG_TEGRA2_BOARD_STRING "TORADEX Colibri T20"
+
+/*
+ * Display CPU and Board information
+ */
+#define CONFIG_DISPLAY_CPUINFO
+#define CONFIG_DISPLAY_BOARDINFO
+
+#define CONFIG_SKIP_LOWLEVEL_INIT
+
+#define CONFIG_CMDLINE_TAG
+#define CONFIG_OF_LIBFDT
+
+#define CONFIG_BOOTDELAY 3
+
+#define CONFIG_SYS_MALLOC_LEN (4 << 20)
+
+/*
+ * Default commands
+ */
+#include <config_cmd_default.h>
+#undef CONFIG_CMD_FLASH
+#undef CONFIG_CMD_FPGA
+#undef CONFIG_CMD_IMI
+#undef CONFIG_CMD_IMLS
+#undef CONFIG_CMD_NFS
+#define CONFIG_CMD_GPIO
+#define CONFIG_NET_MULTI
+
+/*
+ * Command line setup
+ */
+#define CONFIG_CMDLINE_EDITING
+#define CONFIG_COMMAND_HISTORY
+#define CONFIG_AUTOCOMPLETE
+
+#define CONFIG_SYS_NO_FLASH
+
+/*
+ * Environment information
+ */
+#define CONFIG_EXTRA_ENV_SETTINGS \
+ "console=ttyS0,115200n8\0" \
+ "mem=256M at 0x0\0" \
+ "smpflag=smp\0" \
+
+/*
+ * Miscellaneous configurable options
+ */
+#define CONFIG_SYS_LONGHELP /* undef to save memory */
+#define CONFIG_SYS_HUSH_PARSER /* use "hush" command parser */
+#define CONFIG_SYS_PROMPT_HUSH_PS2 "> "
+#define CONFIG_SYS_PROMPT "=> "
+/*
+ * Increasing the size of the IO buffer as default nfsargs size is more
+ * than 256 and so it is not possible to edit it
+ */
+#define CONFIG_SYS_CBSIZE (256 * 2) /* Console I/O Buffer Size */
+/* Print Buffer Size */
+#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + \
+ sizeof(CONFIG_SYS_PROMPT) + 16)
+#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
+/* Boot Argument Buffer Size */
+#define CONFIG_SYS_BARGSIZE (CONFIG_SYS_CBSIZE)
+
+/*
+ * Physical Memory Map
+ */
+#define CONFIG_NR_DRAM_BANKS 1
+#define PHYS_SDRAM_1 TEGRA2_SDRC_CS0
+#define PHYS_SDRAM_1_SIZE 0x10000000 /* 256 MB */
+#define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1
+
+#define CONFIG_SYS_MEMTEST_START (PHYS_SDRAM_1 + 0x600000)
+#define CONFIG_SYS_MEMTEST_END (CONFIG_SYS_MEMTEST_START + 0x100000)
+
+#define CONFIG_SYS_LOAD_ADDR 0xA00800
+#define CONFIG_SYS_TEXT_BASE 0x108000
+
+#define CONFIG_SYS_HZ 1000
+#define CONFIG_SYS_INIT_SP_ADDR 0x80000
+
+#define CONFIG_LOADADDR 0x408000
+
+/*
+ * GPIO
+ */
+#define CONFIG_TEGRA2_GPIO
+
+/*
+ * UART
+ */
+#define CONFIG_SERIAL_MULTI
+#define CONFIG_TEGRA2_ENABLE_UARTA
+#define CONFIG_SYS_NS16550
+#define CONFIG_SYS_NS16550_SERIAL
+#define CONFIG_SYS_NS16550_REG_SIZE (-4)
+#define CONFIG_SYS_NS16550_CLK 216000000
+#define CONFIG_SYS_NS16550_COM1 NV_PA_APB_UARTA_BASE
+#define CONFIG_CONS_INDEX 1
+#define CONFIG_BAUDRATE 115200
+#define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 }
+
+#endif /* __CONFIG_H */
--
1.7.5.4
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board
2011-10-01 0:12 ` [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board Marek Vasut
@ 2011-10-03 3:52 ` Simon Glass
2011-10-03 10:35 ` Marek Vasut
0 siblings, 1 reply; 14+ messages in thread
From: Simon Glass @ 2011-10-03 3:52 UTC (permalink / raw)
To: u-boot
Hi Marek,
On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Ben Warren <biggerbadderben@gmail.com>
> Cc: Tom Warren <twarren.nvidia@gmail.com>
> Cc: Stephen Warren <swarren@nvidia.com>
> ---
> ?MAINTAINERS ? ? ? ? ? ? ? ? ? ? ? ? ? ? | ? ?1 +
> ?board/toradex/colibri_t20/Makefile ? ? ?| ? 49 +++++++++
> ?board/toradex/colibri_t20/colibri_t20.c | ?119 ++++++++++++++++++++++
> ?boards.cfg ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?| ? ?1 +
> ?include/configs/colibri_t20.h ? ? ? ? ? | ?162 +++++++++++++++++++++++++++++++
> ?5 files changed, 332 insertions(+), 0 deletions(-)
> ?create mode 100644 board/toradex/colibri_t20/Makefile
> ?create mode 100644 board/toradex/colibri_t20/colibri_t20.c
> ?create mode 100644 include/configs/colibri_t20.h
>
>
> Btw. is that a pure coincidence that there are three Warrens maintaining Tegras?
> Or are you guys some kind of Warren-brothers ? ;-)
>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c10996c..db7d85a 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -853,6 +853,7 @@ Marek Vasut <marek.vasut@gmail.com>
> ? ? ? ?zipitz2 ? ? ? ? xscale/pxa
> ? ? ? ?efikamx ? ? ? ? i.MX51
> ? ? ? ?efikasb ? ? ? ? i.MX51
> + ? ? ? colibri_t20 ? ? Tegra2 (ARM7 & A9 Dual Core)
>
> ?Hugo Villeneuve <hugo.villeneuve@lyrtech.com>
>
> diff --git a/board/toradex/colibri_t20/Makefile b/board/toradex/colibri_t20/Makefile
> new file mode 100644
> index 0000000..5002dd5
> --- /dev/null
> +++ b/board/toradex/colibri_t20/Makefile
> @@ -0,0 +1,49 @@
> +#
> +# (C) Copyright 2000, 2001, 2002
> +# Wolfgang Denk, DENX Software Engineering, wd at denx.de.
> +#
> +# 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., 59 Temple Place, Suite 330, Boston,
> +# MA 02111-1307 USA
> +#
> +
> +include $(TOPDIR)/config.mk
> +
> +LIB ? ?= $(obj)lib$(BOARD).o
> +
> +COBJS ?:= colibri_t20.o
> +
> +SRCS ? := $(COBJS:.o=.c)
> +OBJS ? := $(addprefix $(obj),$(COBJS))
> +
> +$(LIB): ? ? ? ?$(obj).depend $(OBJS)
> + ? ? ? $(call cmd_link_o_target, $(OBJS))
> +
> +clean:
> + ? ? ? rm -f $(OBJS)
> +
> +distclean: ? ? clean
> + ? ? ? rm -f $(LIB) core *.bak $(obj).depend
> +
> +#########################################################################
> +
> +# defines $(obj).depend target
> +include $(SRCTREE)/rules.mk
> +
> +sinclude $(obj).depend
> +
> +#########################################################################
> diff --git a/board/toradex/colibri_t20/colibri_t20.c b/board/toradex/colibri_t20/colibri_t20.c
> new file mode 100644
> index 0000000..4945888
> --- /dev/null
> +++ b/board/toradex/colibri_t20/colibri_t20.c
> @@ -0,0 +1,119 @@
> +/*
> + * Toradex Colibri T20 Board Support
> + *
> + * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
> + *
> + * Based on code:
> + *
> + * Copyright (C) 2011 Ant Micro <www.antmicro.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., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +
> +#include <common.h>
> +#include <ns16550.h>
> +#include <asm/io.h>
> +#include <asm/arch/tegra2.h>
> +#include <asm/arch/sys_proto.h>
> +
> +#include <asm/arch/clk_rst.h>
> +#include <asm/arch/clock.h>
> +#include <asm/arch/pinmux.h>
> +#include <asm/arch/uart.h>
> +
> +DECLARE_GLOBAL_DATA_PTR;
> +
> +const struct tegra2_sysinfo sysinfo = {
> + ? ? ? CONFIG_TEGRA2_BOARD_STRING
> +};
> +
> +int timer_init(void)
> +{
> + ? ? ? return 0;
> +}
> +
> +static void uart_init(void)
> +{
> + ? ? ? struct clk_rst_ctlr *clkrst = (struct clk_rst_ctlr *)NV_PA_CLK_RST_BASE;
> + ? ? ? struct clk_pll *pll = &clkrst->crc_pll[CLOCK_PLL_ID_PERIPH];
> + ? ? ? uint32_t reg;
> +
> + ? ? ? reg = readl(&pll->pll_base);
> + ? ? ? if (!(reg & PLL_BASE_OVRRIDE_MASK)) {
> + ? ? ? ? ? ? ? /* Override pllp setup for 216MHz operation. */
> + ? ? ? ? ? ? ? reg = PLL_BYPASS_MASK | PLL_BASE_OVRRIDE_MASK |
> + ? ? ? ? ? ? ? ? ? ? ? (1 << PLL_DIVP_SHIFT) | (0xd << PLL_DIVM_SHIFT);
> + ? ? ? ? ? ? ? reg |= (NVRM_PLLP_FIXED_FREQ_KHZ / 500) << PLL_DIVN_SHIFT;
> + ? ? ? ? ? ? ? writel(reg, &pll->pll_base);
> +
> + ? ? ? ? ? ? ? reg |= PLL_ENABLE_MASK;
> + ? ? ? ? ? ? ? writel(reg, &pll->pll_base);
> +
> + ? ? ? ? ? ? ? reg &= ~PLL_BYPASS_MASK;
> + ? ? ? ? ? ? ? writel(reg, &pll->pll_base);
> + ? ? ? }
This block of code should already be in clock.c
> +
> +#ifdef CONFIG_TEGRA2_ENABLE_UARTA
> + ? ? ? /* Assert UART reset and enable clock */
> + ? ? ? reset_set_enable(PERIPH_ID_UART1, 1);
> + ? ? ? clock_enable(PERIPH_ID_UART1);
> +
> + ? ? ? /* Enable pllp_out0 to UART */
> + ? ? ? clrbits_le32(&clkrst->crc_clk_src_uarta, 3 << 30);
> +
> + ? ? ? /* wait for 2us */
> + ? ? ? udelay(2);
> +
> + ? ? ? /* De-assert reset to UART */
> + ? ? ? reset_set_enable(PERIPH_ID_UART1, 0);
> +#endif
and this should really be in board.c along with the other UARTs.
I think there is a new set of patches which Albert was going to apply
to the 'next' branch of ARM. Can you please track these down and take
a look, because the above can now be simplified.
> +}
> +
> +static void pinmux_init(void)
> +{
> + ? ? ? struct pmux_tri_ctlr *pmt = (struct pmux_tri_ctlr *)NV_PA_APB_MISC_BASE;
> +
> + ? ? ? /* SDIO 1 */
> + ? ? ? clrbits_le32(&pmt->pmt_tri[TRISTATE_REG(PIN_ATA)], 1 << 30);
> +
> + ? ? ? /* Mux SDB/SDC to PWM */
> + ? ? ? clrsetbits_le32(&pmt->pmt_ctl_d, 0xcc00, 0x4400);
> +
> + ? ? ? /* Mux SDIO1 to UARTA */
> + ? ? ? setbits_le32(&pmt->pmt_ctl_a, 3 << 30);
> +
> + ? ? ? return 0;
> +}
There are pinmux functions also :-)
> +
> +#ifdef CONFIG_BOARD_EARLY_INIT_F
> +extern int tegra2_start(void);
> +int board_early_init_f(void)
> +{
> + ? ? ? pinmux_init();
> + ? ? ? uart_init();
> + ? ? ? tegra2_start();
> + ? ? ? return 0;
> +}
> +#endif
> +
> +int board_init(void)
> +{
> + ? ? ? gd->bd->bi_boot_params = NV_PA_SDRAM_BASE + 0x100;
> + ? ? ? return 0;
> +}
> diff --git a/boards.cfg b/boards.cfg
> index c485cd7..6cc2855 100644
> --- a/boards.cfg
> +++ b/boards.cfg
> @@ -188,6 +188,7 @@ s5pc210_universal ? ? ? ? ? ?arm ? ? ? ? armv7 ? ? ? universal_c210 ? ? ?samsung
> ?smdkv310 ? ? ? ? ? ? ? ? ? ?arm ? ? ? ? armv7 ? ? ? smdkv310 ? ? ? ? ? ?samsung ? ? ? ?s5pc2xx
> ?harmony ? ? ? ? ? ? ? ? ? ? ?arm ? ? ? ? armv7 ? ? ? harmony ? ? ? ? ? ? nvidia ? ? ? ? tegra2
> ?seaboard ? ? ? ? ? ? ? ? ? ? arm ? ? ? ? armv7 ? ? ? seaboard ? ? ? ? ? ?nvidia ? ? ? ? tegra2
> +colibri_t20 ? ? ? ? ? ? ? ? arm ? ? ? ? armv7 ? ? ? colibri_t20 ? ? ? ? toradex ? ? ? ?tegra2
> ?u8500_href ? ? ? ? ? ? ? ? ? arm ? ? ? ? armv7 ? ? ? u8500 ? ? ? ? ? ? ? st-ericsson ? ?u8500
> ?actux1_4_16 ? ? ? ? ? ? ? ? ?arm ? ? ? ? ixp ? ? ? ? actux1 ? ? ? ? ? ? ?- ? ? ? ? ? ? ?- ? ? ? ? ? actux1:FLASH2X2
> ?actux1_8_16 ? ? ? ? ? ? ? ? ?arm ? ? ? ? ixp ? ? ? ? actux1 ? ? ? ? ? ? ?- ? ? ? ? ? ? ?- ? ? ? ? ? actux1:FLASH1X8
> diff --git a/include/configs/colibri_t20.h b/include/configs/colibri_t20.h
> new file mode 100644
> index 0000000..8fc0898
> --- /dev/null
> +++ b/include/configs/colibri_t20.h
> @@ -0,0 +1,162 @@
> +/*
> + * Toradex Colibri T20 Configuration File
> + *
> + * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
> + *
> + * Based on code:
> + *
> + * Copyright (C) 2011 Ant Micro <www.antmicro.com>
> + * Copyright (C) 2010, 2011 NVIDIA Corporation <www.nvidia.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., 59 Temple Place, Suite 330, Boston,
> + * MA 02111-1307 USA
> + */
> +
> +#ifndef ? ? ? ?__CONFIG_H
> +#define ? ? ? ?__CONFIG_H
> +
> +#include <asm/sizes.h>
> +#include <asm/arch/tegra2.h>
Are you ok with including tegra2-common.h here?
> +
> +/*
> + * High Level Configuration Options
> + */
> +#define ? ? ? ?CONFIG_ARMCORTEXA9 ? ? ? ? ? ? ?/* This is an ARM V7 CPU core */
> +#define ? ? ? ?CONFIG_TEGRA2 ? ? ? ? ? ? ? ? ? /* in a NVidia Tegra2 core */
> +#define ? ? ? ?CONFIG_MACH_TEGRA_GENERIC ? ? ? /* which is a Tegra generic machine */
> +#define ? ? ? ?CONFIG_SYS_L2CACHE_OFF ? ? ? ? ?/* No L2 cache */
> +
> +#define ? ? ? ?CONFIG_ENABLE_CORTEXA9 ? ? ? ? ?/* enable CPU (A9 complex) */
> +
> +/*
> + * Board-specific config
> + */
> +#define ? ? ? ?CONFIG_SYS_BOARD_ODMDATA ? ? ? ?0x100d8011 /* lp1, 256MB */
> +#define ? ? ? ?CONFIG_SYS_CPU_OSC_FREQUENCY ? ?1000000 /* Set CPU clock to 1GHz */
> +#define ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ ? ? ?13
> +
> +#define ? ? ? ?CONFIG_BOARD_EARLY_INIT_F
> +#define ? ? ? ?CONFIG_MACH_TYPE ? ? ? ? ? ? ? ?MACH_TYPE_COLIBRI_TEGRA2
> +#define ? ? ? ?CONFIG_ENV_IS_NOWHERE
> +#define ? ? ? ?CONFIG_ENV_SIZE ? ? ? ? ? ? ? ? 0x20000
> +#define ? ? ? ?CONFIG_ENV_OVERWRITE
> +
> +#define ? ? ? ?CONFIG_TEGRA2_BOARD_STRING ? ? ?"TORADEX Colibri T20"
> +
> +/*
> + * Display CPU and Board information
> + */
> +#define ? ? ? ?CONFIG_DISPLAY_CPUINFO
> +#define ? ? ? ?CONFIG_DISPLAY_BOARDINFO
> +
> +#define ? ? ? ?CONFIG_SKIP_LOWLEVEL_INIT
> +
> +#define ? ? ? ?CONFIG_CMDLINE_TAG
> +#define ? ? ? ?CONFIG_OF_LIBFDT
> +
> +#define ? ? ? ?CONFIG_BOOTDELAY ? ? ? ? ? ? ? ?3
> +
> +#define ? ? ? ?CONFIG_SYS_MALLOC_LEN ? ? ? ? ? (4 << 20)
> +
> +/*
> + * Default commands
> + */
> +#include <config_cmd_default.h>
> +#undef CONFIG_CMD_FLASH
> +#undef CONFIG_CMD_FPGA
> +#undef CONFIG_CMD_IMI
> +#undef CONFIG_CMD_IMLS
> +#undef CONFIG_CMD_NFS
> +#define ? ? ? ?CONFIG_CMD_GPIO
> +#define ? ? ? ?CONFIG_NET_MULTI
> +
> +/*
> + * Command line setup
> + */
> +#define ? ? ? ?CONFIG_CMDLINE_EDITING
> +#define ? ? ? ?CONFIG_COMMAND_HISTORY
> +#define ? ? ? ?CONFIG_AUTOCOMPLETE
> +
> +#define ? ? ? ?CONFIG_SYS_NO_FLASH
> +
> +/*
> + * Environment information
> + */
> +#define ? ? ? ?CONFIG_EXTRA_ENV_SETTINGS \
> + ? ? ? "console=ttyS0,115200n8\0" \
> + ? ? ? "mem=256M at 0x0\0" \
> + ? ? ? "smpflag=smp\0" \
Is that smpflag needed, or automatic?
> +
> +/*
> + * Miscellaneous configurable options
> + */
> +#define ? ? ? ?CONFIG_SYS_LONGHELP ? ? ? ? ? ? /* undef to save memory */
> +#define ? ? ? ?CONFIG_SYS_HUSH_PARSER ? ? ? ? ?/* use "hush" command parser */
> +#define ? ? ? ?CONFIG_SYS_PROMPT_HUSH_PS2 ? ? ?"> "
> +#define ? ? ? ?CONFIG_SYS_PROMPT ? ? ? ? ? ? ? "=> "
> +/*
> + * Increasing the size of the IO buffer as default nfsargs size is more
> + * ?than 256 and so it is not possible to edit it
> + */
> +#define ? ? ? ?CONFIG_SYS_CBSIZE ? ? ? ? ? ? ? (256 * 2) /* Console I/O Buffer Size */
> +/* Print Buffer Size */
> +#define ? ? ? ?CONFIG_SYS_PBSIZE ? ? ? ? ? ? ? (CONFIG_SYS_CBSIZE + \
> + ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? sizeof(CONFIG_SYS_PROMPT) + 16)
> +#define ? ? ? ?CONFIG_SYS_MAXARGS ? ? ? ? ? ? ?16 ? ? ?/* max number of command args */
> +/* Boot Argument Buffer Size */
> +#define ? ? ? ?CONFIG_SYS_BARGSIZE ? ? ? ? ? ? (CONFIG_SYS_CBSIZE)
> +
> +/*
> + * Physical Memory Map
> + */
> +#define ? ? ? ?CONFIG_NR_DRAM_BANKS ? ? ? ? ? ?1
> +#define ? ? ? ?PHYS_SDRAM_1 ? ? ? ? ? ? ? ? ? ?TEGRA2_SDRC_CS0
> +#define ? ? ? ?PHYS_SDRAM_1_SIZE ? ? ? ? ? ? ? 0x10000000 ? ? ?/* 256 MB */
> +#define ? ? ? ?CONFIG_SYS_SDRAM_BASE ? ? ? ? ? PHYS_SDRAM_1
> +
> +#define ? ? ? ?CONFIG_SYS_MEMTEST_START ? ? ? ?(PHYS_SDRAM_1 + 0x600000)
> +#define ? ? ? ?CONFIG_SYS_MEMTEST_END ? ? ? ? ?(CONFIG_SYS_MEMTEST_START + 0x100000)
> +
> +#define ? ? ? ?CONFIG_SYS_LOAD_ADDR ? ? ? ? ? ?0xA00800
> +#define ? ? ? ?CONFIG_SYS_TEXT_BASE ? ? ? ? ? ?0x108000
> +
> +#define ? ? ? ?CONFIG_SYS_HZ ? ? ? ? ? ? ? ? ? 1000
> +#define ? ? ? ?CONFIG_SYS_INIT_SP_ADDR ? ? ? ? 0x80000
> +
> +#define ? ? ? ?CONFIG_LOADADDR ? ? ? ? ? ? ? ? 0x408000
> +
> +/*
> + * GPIO
> + */
> +#define ? ? ? ?CONFIG_TEGRA2_GPIO
> +
> +/*
> + * UART
> + */
> +#define ? ? ? ?CONFIG_SERIAL_MULTI
> +#define ? ? ? ?CONFIG_TEGRA2_ENABLE_UARTA
> +#define ? ? ? ?CONFIG_SYS_NS16550
> +#define ? ? ? ?CONFIG_SYS_NS16550_SERIAL
> +#define ? ? ? ?CONFIG_SYS_NS16550_REG_SIZE ? ? (-4)
> +#define ? ? ? ?CONFIG_SYS_NS16550_CLK ? ? ? ? ?216000000
> +#define ? ? ? ?CONFIG_SYS_NS16550_COM1 ? ? ? ? NV_PA_APB_UARTA_BASE
> +#define ? ? ? ?CONFIG_CONS_INDEX ? ? ? ? ? ? ? 1
> +#define ? ? ? ?CONFIG_BAUDRATE ? ? ? ? ? ? ? ? 115200
> +#define ? ? ? ?CONFIG_SYS_BAUDRATE_TABLE ? ? ? { 9600, 19200, 38400, 57600, 115200 }
> +
> +#endif /* __CONFIG_H */
> --
> 1.7.5.4
>
>
Regards,
Simon (Warren?)
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board
2011-10-03 3:52 ` Simon Glass
@ 2011-10-03 10:35 ` Marek Vasut
2011-10-03 16:09 ` Stephen Warren
2011-10-03 23:42 ` Simon Glass
0 siblings, 2 replies; 14+ messages in thread
From: Marek Vasut @ 2011-10-03 10:35 UTC (permalink / raw)
To: u-boot
On Monday, October 03, 2011 05:52:17 AM Simon Glass wrote:
> Hi Marek,
Hi Simon,
[...]
> > +static void uart_init(void)
> > +{
> > + struct clk_rst_ctlr *clkrst = (struct clk_rst_ctlr
> > *)NV_PA_CLK_RST_BASE; + struct clk_pll *pll =
> > &clkrst->crc_pll[CLOCK_PLL_ID_PERIPH]; + uint32_t reg;
> > +
> > + reg = readl(&pll->pll_base);
> > + if (!(reg & PLL_BASE_OVRRIDE_MASK)) {
> > + /* Override pllp setup for 216MHz operation. */
> > + reg = PLL_BYPASS_MASK | PLL_BASE_OVRRIDE_MASK |
> > + (1 << PLL_DIVP_SHIFT) | (0xd << PLL_DIVM_SHIFT);
Here, read below ...
> > + reg |= (NVRM_PLLP_FIXED_FREQ_KHZ / 500) <<
> > PLL_DIVN_SHIFT; + writel(reg, &pll->pll_base);
> > +
> > + reg |= PLL_ENABLE_MASK;
> > + writel(reg, &pll->pll_base);
> > +
> > + reg &= ~PLL_BYPASS_MASK;
> > + writel(reg, &pll->pll_base);
> > + }
>
> This block of code should already be in clock.c
Well ... the 0xd is different from the common code. So maybe that should be
converted like in the PATCH 1/2 ?
>
> > +
> > +#ifdef CONFIG_TEGRA2_ENABLE_UARTA
> > + /* Assert UART reset and enable clock */
> > + reset_set_enable(PERIPH_ID_UART1, 1);
> > + clock_enable(PERIPH_ID_UART1);
> > +
> > + /* Enable pllp_out0 to UART */
> > + clrbits_le32(&clkrst->crc_clk_src_uarta, 3 << 30);
> > +
> > + /* wait for 2us */
> > + udelay(2);
> > +
> > + /* De-assert reset to UART */
> > + reset_set_enable(PERIPH_ID_UART1, 0);
> > +#endif
>
> and this should really be in board.c along with the other UARTs.
You mean nvidia/board.c ?
>
> I think there is a new set of patches which Albert was going to apply
> to the 'next' branch of ARM. Can you please track these down and take
> a look, because the above can now be simplified.
Will check.
>
> > +}
> > +
> > +static void pinmux_init(void)
> > +{
> > + struct pmux_tri_ctlr *pmt = (struct pmux_tri_ctlr
> > *)NV_PA_APB_MISC_BASE; +
> > + /* SDIO 1 */
> > + clrbits_le32(&pmt->pmt_tri[TRISTATE_REG(PIN_ATA)], 1 << 30);
> > +
> > + /* Mux SDB/SDC to PWM */
> > + clrsetbits_le32(&pmt->pmt_ctl_d, 0xcc00, 0x4400);
> > +
> > + /* Mux SDIO1 to UARTA */
> > + setbits_le32(&pmt->pmt_ctl_a, 3 << 30);
> > +
> > + return 0;
> > +}
>
> There are pinmux functions also :-)
Really ? Where are these ?!
>
> > +
> > +#ifdef CONFIG_BOARD_EARLY_INIT_F
> > +extern int tegra2_start(void);
> > +int board_early_init_f(void)
> > +{
> > + pinmux_init();
> > + uart_init();
> > + tegra2_start();
> > + return 0;
> > +}
> > +#endif
> > +
> > +int board_init(void)
> > +{
> > + gd->bd->bi_boot_params = NV_PA_SDRAM_BASE + 0x100;
> > + return 0;
> > +}
> > diff --git a/boards.cfg b/boards.cfg
> > index c485cd7..6cc2855 100644
> > --- a/boards.cfg
> > +++ b/boards.cfg
> > @@ -188,6 +188,7 @@ s5pc210_universal arm armv7
> > universal_c210 samsung smdkv310 arm
> > armv7 smdkv310 samsung s5pc2xx harmony
> > arm armv7 harmony nvidia
> > tegra2 seaboard arm armv7 seaboard
> > nvidia tegra2 +colibri_t20 arm
> > armv7 colibri_t20 toradex tegra2 u8500_href
> > arm armv7 u8500 st-ericsson
> > u8500 actux1_4_16 arm ixp actux1
> > - - actux1:FLASH2X2 actux1_8_16
> > arm ixp actux1 - -
> > actux1:FLASH1X8 diff --git a/include/configs/colibri_t20.h
> > b/include/configs/colibri_t20.h new file mode 100644
> > index 0000000..8fc0898
> > --- /dev/null
> > +++ b/include/configs/colibri_t20.h
> > @@ -0,0 +1,162 @@
> > +/*
> > + * Toradex Colibri T20 Configuration File
> > + *
> > + * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
> > + *
> > + * Based on code:
> > + *
> > + * Copyright (C) 2011 Ant Micro <www.antmicro.com>
> > + * Copyright (C) 2010, 2011 NVIDIA Corporation <www.nvidia.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., 59 Temple Place, Suite 330, Boston,
> > + * MA 02111-1307 USA
> > + */
> > +
> > +#ifndef __CONFIG_H
> > +#define __CONFIG_H
> > +
> > +#include <asm/sizes.h>
> > +#include <asm/arch/tegra2.h>
>
> Are you ok with including tegra2-common.h here?
No, I had to do multiple changes (like different stack position etc).
[...]
>
> Regards,
> Simon (Warren?)
Cheers
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board
2011-10-03 10:35 ` Marek Vasut
@ 2011-10-03 16:09 ` Stephen Warren
2011-10-03 23:42 ` Simon Glass
1 sibling, 0 replies; 14+ messages in thread
From: Stephen Warren @ 2011-10-03 16:09 UTC (permalink / raw)
To: u-boot
Marek Vasut wrote at Monday, October 03, 2011 4:36 AM:
...
> > > +static void pinmux_init(void)
> > > +{
> > > + struct pmux_tri_ctlr *pmt = (struct pmux_tri_ctlr *)NV_PA_APB_MISC_BASE; +
> > > + /* SDIO 1 */
> > > + clrbits_le32(&pmt->pmt_tri[TRISTATE_REG(PIN_ATA)], 1 << 30);
> > > +
> > > + /* Mux SDB/SDC to PWM */
> > > + clrsetbits_le32(&pmt->pmt_ctl_d, 0xcc00, 0x4400);
> > > +
> > > + /* Mux SDIO1 to UARTA */
> > > + setbits_le32(&pmt->pmt_ctl_a, 3 << 30);
> > > +
> > > + return 0;
> > > +}
> >
> > There are pinmux functions also :-)
>
> Really ? Where are these ?!
Simon sent them to the list, but I don't think they've been checked in
yet; see:
http://patchwork.ozlabs.org/project/uboot/list/?submitter=6170&state=1&q=tegra2
> > > b/include/configs/colibri_t20.h new file mode 100644
...
> > > +#include <asm/sizes.h>
> > > +#include <asm/arch/tegra2.h>
> >
> > Are you ok with including tegra2-common.h here?
>
> No, I had to do multiple changes (like different stack position etc).
Can't tegra2-common.h be parameterized or overridden? It'd be nice not
to have to duplicate its content, and to have one place that affects
all Tegra systems.
--
nvpublic
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board
2011-10-03 10:35 ` Marek Vasut
2011-10-03 16:09 ` Stephen Warren
@ 2011-10-03 23:42 ` Simon Glass
1 sibling, 0 replies; 14+ messages in thread
From: Simon Glass @ 2011-10-03 23:42 UTC (permalink / raw)
To: u-boot
Hi Marek,
On Mon, Oct 3, 2011 at 3:35 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On Monday, October 03, 2011 05:52:17 AM Simon Glass wrote:
> > Hi Marek,
>
> Hi Simon,
>
> [...]
>
> > > +static void uart_init(void)
> > > +{
> > > + struct clk_rst_ctlr *clkrst = (struct clk_rst_ctlr
> > > *)NV_PA_CLK_RST_BASE; + struct clk_pll *pll =
> > > &clkrst->crc_pll[CLOCK_PLL_ID_PERIPH]; + uint32_t reg;
> > > +
> > > + reg = readl(&pll->pll_base);
> > > + if (!(reg & PLL_BASE_OVRRIDE_MASK)) {
> > > + /* Override pllp setup for 216MHz operation. */
> > > + reg = PLL_BYPASS_MASK | PLL_BASE_OVRRIDE_MASK |
> > > + (1 << PLL_DIVP_SHIFT) | (0xd <<
> PLL_DIVM_SHIFT);
>
> Here, read below ...
>
> > > + reg |= (NVRM_PLLP_FIXED_FREQ_KHZ / 500) <<
> > > PLL_DIVN_SHIFT; + writel(reg, &pll->pll_base);
> > > +
> > > + reg |= PLL_ENABLE_MASK;
> > > + writel(reg, &pll->pll_base);
> > > +
> > > + reg &= ~PLL_BYPASS_MASK;
> > > + writel(reg, &pll->pll_base);
> > > + }
> >
> > This block of code should already be in clock.c
>
> Well ... the 0xd is different from the common code. So maybe that should be
> converted like in the PATCH 1/2 ?
>
Yes that would be best.
>
> >
> > > +
> > > +#ifdef CONFIG_TEGRA2_ENABLE_UARTA
> > > + /* Assert UART reset and enable clock */
> > > + reset_set_enable(PERIPH_ID_UART1, 1);
> > > + clock_enable(PERIPH_ID_UART1);
> > > +
> > > + /* Enable pllp_out0 to UART */
> > > + clrbits_le32(&clkrst->crc_clk_src_uarta, 3 << 30);
> > > +
> > > + /* wait for 2us */
> > > + udelay(2);
> > > +
> > > + /* De-assert reset to UART */
> > > + reset_set_enable(PERIPH_ID_UART1, 0);
> > > +#endif
> >
> > and this should really be in board.c along with the other UARTs.
>
> You mean nvidia/board.c ?
>
Yes that's right.
> >
> > I think there is a new set of patches which Albert was going to apply
> > to the 'next' branch of ARM. Can you please track these down and take
> > a look, because the above can now be simplified.
>
> Will check.
>
> >
> > > +}
> > > +
> > > +static void pinmux_init(void)
> > > +{
> > > + struct pmux_tri_ctlr *pmt = (struct pmux_tri_ctlr
> > > *)NV_PA_APB_MISC_BASE; +
> > > + /* SDIO 1 */
> > > + clrbits_le32(&pmt->pmt_tri[TRISTATE_REG(PIN_ATA)], 1 << 30);
> > > +
> > > + /* Mux SDB/SDC to PWM */
> > > + clrsetbits_le32(&pmt->pmt_ctl_d, 0xcc00, 0x4400);
> > > +
> > > + /* Mux SDIO1 to UARTA */
> > > + setbits_le32(&pmt->pmt_ctl_a, 3 << 30);
> > > +
> > > + return 0;
> > > +}
> >
> > There are pinmux functions also :-)
>
> Really ? Where are these ?!
>
See Stephen's message, but yes it seems they are not yet committed. Have
added Albert on cc.
>
> > > +
> > > +#ifdef CONFIG_BOARD_EARLY_INIT_F
> > > +extern int tegra2_start(void);
> > > +int board_early_init_f(void)
> > > +{
> > > + pinmux_init();
> > > + uart_init();
> > > + tegra2_start();
> > > + return 0;
> > > +}
> > > +#endif
> > > +
> > > +int board_init(void)
> > > +{
> > > + gd->bd->bi_boot_params = NV_PA_SDRAM_BASE + 0x100;
> > > + return 0;
> > > +}
> > > diff --git a/boards.cfg b/boards.cfg
> > > index c485cd7..6cc2855 100644
> > > --- a/boards.cfg
> > > +++ b/boards.cfg
> > > @@ -188,6 +188,7 @@ s5pc210_universal arm armv7
> > > universal_c210 samsung smdkv310 arm
> > > armv7 smdkv310 samsung s5pc2xx harmony
> > > arm armv7 harmony nvidia
> > > tegra2 seaboard arm armv7 seaboard
> > > nvidia tegra2 +colibri_t20 arm
> > > armv7 colibri_t20 toradex tegra2 u8500_href
> > > arm armv7 u8500 st-ericsson
> > > u8500 actux1_4_16 arm ixp actux1
> > > - - actux1:FLASH2X2 actux1_8_16
> > > arm ixp actux1 - -
> > > actux1:FLASH1X8 diff --git a/include/configs/colibri_t20.h
> > > b/include/configs/colibri_t20.h new file mode 100644
> > > index 0000000..8fc0898
> > > --- /dev/null
> > > +++ b/include/configs/colibri_t20.h
> > > @@ -0,0 +1,162 @@
> > > +/*
> > > + * Toradex Colibri T20 Configuration File
> > > + *
> > > + * Copyright (C) 2011 Marek Vasut <marek.vasut@gmail.com>
> > > + *
> > > + * Based on code:
> > > + *
> > > + * Copyright (C) 2011 Ant Micro <www.antmicro.com>
> > > + * Copyright (C) 2010, 2011 NVIDIA Corporation <www.nvidia.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., 59 Temple Place, Suite 330, Boston,
> > > + * MA 02111-1307 USA
> > > + */
> > > +
> > > +#ifndef __CONFIG_H
> > > +#define __CONFIG_H
> > > +
> > > +#include <asm/sizes.h>
> > > +#include <asm/arch/tegra2.h>
> >
> > Are you ok with including tegra2-common.h here?
>
> No, I had to do multiple changes (like different stack position etc).
>
As Stephen says, there is still a lot in common - it would be nice with
Tegra to avoid duplicating all the stuff in every board file. If you are not
sure please let me know and I'll try to send a patch.
Regards,
Simon
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-01 0:12 [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Marek Vasut
2011-10-01 0:12 ` [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board Marek Vasut
@ 2011-10-05 14:10 ` Simon Glass
2011-10-05 17:05 ` Marek Vasut
1 sibling, 1 reply; 14+ messages in thread
From: Simon Glass @ 2011-10-05 14:10 UTC (permalink / raw)
To: u-boot
Hi Marek,
On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed configurable.
>
> Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Ben Warren <biggerbadderben@gmail.com>
> Cc: Tom Warren <twarren.nvidia@gmail.com>
> Cc: Stephen Warren <swarren@nvidia.com>
> ---
> ?arch/arm/cpu/armv7/tegra2/ap20.c | ? ?9 +++++++--
> ?1 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c b/arch/arm/cpu/armv7/tegra2/ap20.c
> index dc5f984..71d9f26 100644
> --- a/arch/arm/cpu/armv7/tegra2/ap20.c
> +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
> @@ -31,6 +31,11 @@
> ?#include <asm/arch/scu.h>
> ?#include <common.h>
>
> +/* The default XTal is 12MHz, some boards might use 13MHz one though */
> +#ifndef ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ
> +#define ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ ? ? ?12
> +#endif
> +
> ?u32 s_first_boot = 1;
>
> ?void init_pllx(void)
> @@ -46,8 +51,8 @@ void init_pllx(void)
> ? ? ? ?/* Set PLLX_MISC */
> ? ? ? ?writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
>
> - ? ? ? /* Use 12MHz clock here */
> - ? ? ? reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
> + ? ? ? /* Use 13MHz clock here */
Change comment?
> + ? ? ? reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ << PLL_DIVM_SHIFT);
> ? ? ? ?reg |= 1000 << PLL_DIVN_SHIFT;
> ? ? ? ?writel(reg, &pll->pll_base);
>
> --
> 1.7.5.4
>
>
Regards,
Simon
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-05 14:10 ` [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Simon Glass
@ 2011-10-05 17:05 ` Marek Vasut
2011-10-12 0:30 ` Simon Glass
0 siblings, 1 reply; 14+ messages in thread
From: Marek Vasut @ 2011-10-05 17:05 UTC (permalink / raw)
To: u-boot
On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
> Hi Marek,
>
> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
> > configurable.
> >
> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> > Cc: Simon Glass <sjg@chromium.org>
> > Cc: Ben Warren <biggerbadderben@gmail.com>
> > Cc: Tom Warren <twarren.nvidia@gmail.com>
> > Cc: Stephen Warren <swarren@nvidia.com>
> > ---
> > arch/arm/cpu/armv7/tegra2/ap20.c | 9 +++++++--
> > 1 files changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
> > @@ -31,6 +31,11 @@
> > #include <asm/arch/scu.h>
> > #include <common.h>
> >
> > +/* The default XTal is 12MHz, some boards might use 13MHz one though */
> > +#ifndef CONFIG_SYS_TEGRA2_XTAL_MHZ
> > +#define CONFIG_SYS_TEGRA2_XTAL_MHZ 12
> > +#endif
> > +
> > u32 s_first_boot = 1;
> >
> > void init_pllx(void)
> > @@ -46,8 +51,8 @@ void init_pllx(void)
> > /* Set PLLX_MISC */
> > writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
> >
> > - /* Use 12MHz clock here */
> > - reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
> > + /* Use 13MHz clock here */
>
> Change comment?
>
> > + reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
> > writel(reg, &pll->pll_base);
> >
> > --
> > 1.7.5.4
>
> Regards,
> Simon
Hi Simon, I'll probably also change the board.c to support this. But then, can't
the board.c be moved to arch/arm/cpu/armv7/tegra... and it's functions exported
via standard header ? Cheers
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-05 17:05 ` Marek Vasut
@ 2011-10-12 0:30 ` Simon Glass
2011-10-12 11:19 ` Marek Vasut
0 siblings, 1 reply; 14+ messages in thread
From: Simon Glass @ 2011-10-12 0:30 UTC (permalink / raw)
To: u-boot
Hi Marek,
On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
>> Hi Marek,
>>
>> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
>> > configurable.
>> >
>> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
>> > Cc: Simon Glass <sjg@chromium.org>
>> > Cc: Ben Warren <biggerbadderben@gmail.com>
>> > Cc: Tom Warren <twarren.nvidia@gmail.com>
>> > Cc: Stephen Warren <swarren@nvidia.com>
>> > ---
>> > ?arch/arm/cpu/armv7/tegra2/ap20.c | ? ?9 +++++++--
>> > ?1 files changed, 7 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
>> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
>> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
>> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
>> > @@ -31,6 +31,11 @@
>> > ?#include <asm/arch/scu.h>
>> > ?#include <common.h>
>> >
>> > +/* The default XTal is 12MHz, some boards might use 13MHz one though */
>> > +#ifndef ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ
>> > +#define ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ ? ? ?12
>> > +#endif
>> > +
>> > ?u32 s_first_boot = 1;
>> >
>> > ?void init_pllx(void)
>> > @@ -46,8 +51,8 @@ void init_pllx(void)
>> > ? ? ? ?/* Set PLLX_MISC */
>> > ? ? ? ?writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
>> >
>> > - ? ? ? /* Use 12MHz clock here */
>> > - ? ? ? reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
>> > + ? ? ? /* Use 13MHz clock here */
>>
>> Change comment?
>>
>> > + ? ? ? reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
>> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
>> > ? ? ? ?writel(reg, &pll->pll_base);
>> >
>> > --
>> > 1.7.5.4
>>
>> Regards,
>> Simon
>
> Hi Simon, I'll probably also change the board.c to support this. But then, can't
> the board.c be moved to arch/arm/cpu/armv7/tegra... and it's functions exported
> via standard header ? Cheers
>
Which board.c do you mean and which change do you need to make?
Regards,
Simon
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-12 0:30 ` Simon Glass
@ 2011-10-12 11:19 ` Marek Vasut
2011-10-12 21:17 ` Simon Glass
0 siblings, 1 reply; 14+ messages in thread
From: Marek Vasut @ 2011-10-12 11:19 UTC (permalink / raw)
To: u-boot
On Wednesday, October 12, 2011 02:30:57 AM Simon Glass wrote:
> Hi Marek,
>
> On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
> >> Hi Marek,
> >>
> >> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> >> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
> >> > configurable.
> >> >
> >> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> >> > Cc: Simon Glass <sjg@chromium.org>
> >> > Cc: Ben Warren <biggerbadderben@gmail.com>
> >> > Cc: Tom Warren <twarren.nvidia@gmail.com>
> >> > Cc: Stephen Warren <swarren@nvidia.com>
> >> > ---
> >> > arch/arm/cpu/armv7/tegra2/ap20.c | 9 +++++++--
> >> > 1 files changed, 7 insertions(+), 2 deletions(-)
> >> >
> >> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
> >> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
> >> > @@ -31,6 +31,11 @@
> >> > #include <asm/arch/scu.h>
> >> > #include <common.h>
> >> >
> >> > +/* The default XTal is 12MHz, some boards might use 13MHz one though
> >> > */ +#ifndef CONFIG_SYS_TEGRA2_XTAL_MHZ
> >> > +#define CONFIG_SYS_TEGRA2_XTAL_MHZ 12
> >> > +#endif
> >> > +
> >> > u32 s_first_boot = 1;
> >> >
> >> > void init_pllx(void)
> >> > @@ -46,8 +51,8 @@ void init_pllx(void)
> >> > /* Set PLLX_MISC */
> >> > writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
> >> >
> >> > - /* Use 12MHz clock here */
> >> > - reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
> >> > + /* Use 13MHz clock here */
> >>
> >> Change comment?
> >>
> >> > + reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
> >> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
> >> > writel(reg, &pll->pll_base);
> >> >
> >> > --
> >> > 1.7.5.4
> >>
> >> Regards,
> >> Simon
> >
> > Hi Simon, I'll probably also change the board.c to support this. But
> > then, can't the board.c be moved to arch/arm/cpu/armv7/tegra... and it's
> > functions exported via standard header ? Cheers
>
> Which board.c do you mean and which change do you need to make?
Hi Simon,
board/nvidia/common/board.c
same as this patch does .
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-12 11:19 ` Marek Vasut
@ 2011-10-12 21:17 ` Simon Glass
2011-10-13 0:27 ` Marek Vasut
0 siblings, 1 reply; 14+ messages in thread
From: Simon Glass @ 2011-10-12 21:17 UTC (permalink / raw)
To: u-boot
Hi Marek,
On Wed, Oct 12, 2011 at 4:19 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On Wednesday, October 12, 2011 02:30:57 AM Simon Glass wrote:
>> Hi Marek,
>>
>> On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> > On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
>> >> Hi Marek,
>> >>
>> >> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> >> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
>> >> > configurable.
>> >> >
>> >> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
>> >> > Cc: Simon Glass <sjg@chromium.org>
>> >> > Cc: Ben Warren <biggerbadderben@gmail.com>
>> >> > Cc: Tom Warren <twarren.nvidia@gmail.com>
>> >> > Cc: Stephen Warren <swarren@nvidia.com>
>> >> > ---
>> >> > ?arch/arm/cpu/armv7/tegra2/ap20.c | ? ?9 +++++++--
>> >> > ?1 files changed, 7 insertions(+), 2 deletions(-)
>> >> >
>> >> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
>> >> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> > @@ -31,6 +31,11 @@
>> >> > ?#include <asm/arch/scu.h>
>> >> > ?#include <common.h>
>> >> >
>> >> > +/* The default XTal is 12MHz, some boards might use 13MHz one though
>> >> > */ +#ifndef ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ
>> >> > +#define ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ ? ? ?12
>> >> > +#endif
>> >> > +
>> >> > ?u32 s_first_boot = 1;
>> >> >
>> >> > ?void init_pllx(void)
>> >> > @@ -46,8 +51,8 @@ void init_pllx(void)
>> >> > ? ? ? ?/* Set PLLX_MISC */
>> >> > ? ? ? ?writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
>> >> >
>> >> > - ? ? ? /* Use 12MHz clock here */
>> >> > - ? ? ? reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
>> >> > + ? ? ? /* Use 13MHz clock here */
>> >>
>> >> Change comment?
>> >>
>> >> > + ? ? ? reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
>> >> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
>> >> > ? ? ? ?writel(reg, &pll->pll_base);
>> >> >
>> >> > --
>> >> > 1.7.5.4
>> >>
>> >> Regards,
>> >> Simon
>> >
>> > Hi Simon, I'll probably also change the board.c to support this. But
>> > then, can't the board.c be moved to arch/arm/cpu/armv7/tegra... and it's
>> > functions exported via standard header ? Cheers
>>
>> Which board.c do you mean and which change do you need to make?
>
> Hi Simon,
>
> board/nvidia/common/board.c
>
> same as this patch does .
>
Well there are several files called board.c:
board/nvidia/common/board.c - Common things for Nvidia boards
arch/arm/lib/board.c - Generic ARM things
arch/arm/cpu/armv7/tegra2/board.c - Generic Tegra things
Since this is to do with Tegra, I think it should stay where it is in
ap20.c and you should just add a config. For now
arch/arm/cpu/armv7/tegra2/board.c just deals with DRAM.
Regards,
Simon
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-12 21:17 ` Simon Glass
@ 2011-10-13 0:27 ` Marek Vasut
2011-10-13 4:31 ` Simon Glass
0 siblings, 1 reply; 14+ messages in thread
From: Marek Vasut @ 2011-10-13 0:27 UTC (permalink / raw)
To: u-boot
On Wednesday, October 12, 2011 11:17:07 PM Simon Glass wrote:
> Hi Marek,
>
> On Wed, Oct 12, 2011 at 4:19 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > On Wednesday, October 12, 2011 02:30:57 AM Simon Glass wrote:
> >> Hi Marek,
> >>
> >> On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> >> > On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
> >> >> Hi Marek,
> >> >>
> >> >> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com>
wrote:
> >> >> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
> >> >> > configurable.
> >> >> >
> >> >> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> >> >> > Cc: Simon Glass <sjg@chromium.org>
> >> >> > Cc: Ben Warren <biggerbadderben@gmail.com>
> >> >> > Cc: Tom Warren <twarren.nvidia@gmail.com>
> >> >> > Cc: Stephen Warren <swarren@nvidia.com>
> >> >> > ---
> >> >> > arch/arm/cpu/armv7/tegra2/ap20.c | 9 +++++++--
> >> >> > 1 files changed, 7 insertions(+), 2 deletions(-)
> >> >> >
> >> >> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
> >> >> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> > @@ -31,6 +31,11 @@
> >> >> > #include <asm/arch/scu.h>
> >> >> > #include <common.h>
> >> >> >
> >> >> > +/* The default XTal is 12MHz, some boards might use 13MHz one
> >> >> > though */ +#ifndef CONFIG_SYS_TEGRA2_XTAL_MHZ
> >> >> > +#define CONFIG_SYS_TEGRA2_XTAL_MHZ 12
> >> >> > +#endif
> >> >> > +
> >> >> > u32 s_first_boot = 1;
> >> >> >
> >> >> > void init_pllx(void)
> >> >> > @@ -46,8 +51,8 @@ void init_pllx(void)
> >> >> > /* Set PLLX_MISC */
> >> >> > writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
> >> >> >
> >> >> > - /* Use 12MHz clock here */
> >> >> > - reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
> >> >> > + /* Use 13MHz clock here */
> >> >>
> >> >> Change comment?
> >> >>
> >> >> > + reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
> >> >> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
> >> >> > writel(reg, &pll->pll_base);
> >> >> >
> >> >> > --
> >> >> > 1.7.5.4
> >> >>
> >> >> Regards,
> >> >> Simon
> >> >
> >> > Hi Simon, I'll probably also change the board.c to support this. But
> >> > then, can't the board.c be moved to arch/arm/cpu/armv7/tegra... and
> >> > it's functions exported via standard header ? Cheers
> >>
> >> Which board.c do you mean and which change do you need to make?
> >
> > Hi Simon,
> >
> > board/nvidia/common/board.c
> >
> > same as this patch does .
>
> Well there are several files called board.c:
>
> board/nvidia/common/board.c - Common things for Nvidia boards
> arch/arm/lib/board.c - Generic ARM things
> arch/arm/cpu/armv7/tegra2/board.c - Generic Tegra things
>
> Since this is to do with Tegra, I think it should stay where it is in
> ap20.c and you should just add a config. For now
> arch/arm/cpu/armv7/tegra2/board.c just deals with DRAM.
>
Hi Simon,
the same register gets set in both boards/.../board.c and ap20.c ... which
itself is a bit suspicious.
Cheers
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-13 0:27 ` Marek Vasut
@ 2011-10-13 4:31 ` Simon Glass
2011-10-13 14:04 ` Marek Vasut
0 siblings, 1 reply; 14+ messages in thread
From: Simon Glass @ 2011-10-13 4:31 UTC (permalink / raw)
To: u-boot
On Wed, Oct 12, 2011 at 5:27 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> On Wednesday, October 12, 2011 11:17:07 PM Simon Glass wrote:
>> Hi Marek,
>>
>> On Wed, Oct 12, 2011 at 4:19 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> > On Wednesday, October 12, 2011 02:30:57 AM Simon Glass wrote:
>> >> Hi Marek,
>> >>
>> >> On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
>> >> > On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
>> >> >> Hi Marek,
>> >> >>
>> >> >> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut <marek.vasut@gmail.com>
> wrote:
>> >> >> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
>> >> >> > configurable.
>> >> >> >
>> >> >> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
>> >> >> > Cc: Simon Glass <sjg@chromium.org>
>> >> >> > Cc: Ben Warren <biggerbadderben@gmail.com>
>> >> >> > Cc: Tom Warren <twarren.nvidia@gmail.com>
>> >> >> > Cc: Stephen Warren <swarren@nvidia.com>
>> >> >> > ---
>> >> >> > ?arch/arm/cpu/armv7/tegra2/ap20.c | ? ?9 +++++++--
>> >> >> > ?1 files changed, 7 insertions(+), 2 deletions(-)
>> >> >> >
>> >> >> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> >> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
>> >> >> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> >> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
>> >> >> > @@ -31,6 +31,11 @@
>> >> >> > ?#include <asm/arch/scu.h>
>> >> >> > ?#include <common.h>
>> >> >> >
>> >> >> > +/* The default XTal is 12MHz, some boards might use 13MHz one
>> >> >> > though */ +#ifndef ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ
>> >> >> > +#define ? ? ? ?CONFIG_SYS_TEGRA2_XTAL_MHZ ? ? ?12
>> >> >> > +#endif
>> >> >> > +
>> >> >> > ?u32 s_first_boot = 1;
>> >> >> >
>> >> >> > ?void init_pllx(void)
>> >> >> > @@ -46,8 +51,8 @@ void init_pllx(void)
>> >> >> > ? ? ? ?/* Set PLLX_MISC */
>> >> >> > ? ? ? ?writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
>> >> >> >
>> >> >> > - ? ? ? /* Use 12MHz clock here */
>> >> >> > - ? ? ? reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
>> >> >> > + ? ? ? /* Use 13MHz clock here */
>> >> >>
>> >> >> Change comment?
>> >> >>
>> >> >> > + ? ? ? reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
>> >> >> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
>> >> >> > ? ? ? ?writel(reg, &pll->pll_base);
>> >> >> >
>> >> >> > --
>> >> >> > 1.7.5.4
>> >> >>
>> >> >> Regards,
>> >> >> Simon
>> >> >
>> >> > Hi Simon, I'll probably also change the board.c to support this. But
>> >> > then, can't the board.c be moved to arch/arm/cpu/armv7/tegra... and
>> >> > it's functions exported via standard header ? Cheers
>> >>
>> >> Which board.c do you mean and which change do you need to make?
>> >
>> > Hi Simon,
>> >
>> > board/nvidia/common/board.c
>> >
>> > same as this patch does .
>>
>> Well there are several files called board.c:
>>
>> board/nvidia/common/board.c - Common things for Nvidia boards
>> arch/arm/lib/board.c - Generic ARM things
>> arch/arm/cpu/armv7/tegra2/board.c - Generic Tegra things
>>
>> Since this is to do with Tegra, I think it should stay where it is in
>> ap20.c and you should just add a config. For now
>> arch/arm/cpu/armv7/tegra2/board.c just deals with DRAM.
>>
>
> Hi Simon,
>
> the same register gets set in both boards/.../board.c and ap20.c ... which
> itself is a bit suspicious.
>
> Cheers
>
Hi Marek,
Yes but I think you are still looking at the old code. The patches are
applied now to u-boot-arm/master - can you please take a look? The
clock/pinmux framework is basically complete now so most of the poking
of registers elsewhere is gone.
Regards,
Simon
^ permalink raw reply [flat|nested] 14+ messages in thread
* [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable
2011-10-13 4:31 ` Simon Glass
@ 2011-10-13 14:04 ` Marek Vasut
0 siblings, 0 replies; 14+ messages in thread
From: Marek Vasut @ 2011-10-13 14:04 UTC (permalink / raw)
To: u-boot
On Thursday, October 13, 2011 06:31:09 AM Simon Glass wrote:
> On Wed, Oct 12, 2011 at 5:27 PM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > On Wednesday, October 12, 2011 11:17:07 PM Simon Glass wrote:
> >> Hi Marek,
> >>
> >> On Wed, Oct 12, 2011 at 4:19 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> >> > On Wednesday, October 12, 2011 02:30:57 AM Simon Glass wrote:
> >> >> Hi Marek,
> >> >>
> >> >> On Wed, Oct 5, 2011 at 10:05 AM, Marek Vasut <marek.vasut@gmail.com>
wrote:
> >> >> > On Wednesday, October 05, 2011 04:10:56 PM Simon Glass wrote:
> >> >> >> Hi Marek,
> >> >> >>
> >> >> >> On Fri, Sep 30, 2011 at 5:12 PM, Marek Vasut
> >> >> >> <marek.vasut@gmail.com>
> >
> > wrote:
> >> >> >> > The Toradex T20 module uses 13MHz Xtal, so make the Xtal speed
> >> >> >> > configurable.
> >> >> >> >
> >> >> >> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> >> >> >> > Cc: Simon Glass <sjg@chromium.org>
> >> >> >> > Cc: Ben Warren <biggerbadderben@gmail.com>
> >> >> >> > Cc: Tom Warren <twarren.nvidia@gmail.com>
> >> >> >> > Cc: Stephen Warren <swarren@nvidia.com>
> >> >> >> > ---
> >> >> >> > arch/arm/cpu/armv7/tegra2/ap20.c | 9 +++++++--
> >> >> >> > 1 files changed, 7 insertions(+), 2 deletions(-)
> >> >> >> >
> >> >> >> > diff --git a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> >> > b/arch/arm/cpu/armv7/tegra2/ap20.c index dc5f984..71d9f26 100644
> >> >> >> > --- a/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> >> > +++ b/arch/arm/cpu/armv7/tegra2/ap20.c
> >> >> >> > @@ -31,6 +31,11 @@
> >> >> >> > #include <asm/arch/scu.h>
> >> >> >> > #include <common.h>
> >> >> >> >
> >> >> >> > +/* The default XTal is 12MHz, some boards might use 13MHz one
> >> >> >> > though */ +#ifndef CONFIG_SYS_TEGRA2_XTAL_MHZ
> >> >> >> > +#define CONFIG_SYS_TEGRA2_XTAL_MHZ 12
> >> >> >> > +#endif
> >> >> >> > +
> >> >> >> > u32 s_first_boot = 1;
> >> >> >> >
> >> >> >> > void init_pllx(void)
> >> >> >> > @@ -46,8 +51,8 @@ void init_pllx(void)
> >> >> >> > /* Set PLLX_MISC */
> >> >> >> > writel(1 << PLL_CPCON_SHIFT, &pll->pll_misc);
> >> >> >> >
> >> >> >> > - /* Use 12MHz clock here */
> >> >> >> > - reg = PLL_BYPASS_MASK | (12 << PLL_DIVM_SHIFT);
> >> >> >> > + /* Use 13MHz clock here */
> >> >> >>
> >> >> >> Change comment?
> >> >> >>
> >> >> >> > + reg = PLL_BYPASS_MASK | (CONFIG_SYS_TEGRA2_XTAL_MHZ <<
> >> >> >> > PLL_DIVM_SHIFT); reg |= 1000 << PLL_DIVN_SHIFT;
> >> >> >> > writel(reg, &pll->pll_base);
> >> >> >> >
> >> >> >> > --
> >> >> >> > 1.7.5.4
> >> >> >>
> >> >> >> Regards,
> >> >> >> Simon
> >> >> >
> >> >> > Hi Simon, I'll probably also change the board.c to support this.
> >> >> > But then, can't the board.c be moved to
> >> >> > arch/arm/cpu/armv7/tegra... and it's functions exported via
> >> >> > standard header ? Cheers
> >> >>
> >> >> Which board.c do you mean and which change do you need to make?
> >> >
> >> > Hi Simon,
> >> >
> >> > board/nvidia/common/board.c
> >> >
> >> > same as this patch does .
> >>
> >> Well there are several files called board.c:
> >>
> >> board/nvidia/common/board.c - Common things for Nvidia boards
> >> arch/arm/lib/board.c - Generic ARM things
> >> arch/arm/cpu/armv7/tegra2/board.c - Generic Tegra things
> >>
> >> Since this is to do with Tegra, I think it should stay where it is in
> >> ap20.c and you should just add a config. For now
> >> arch/arm/cpu/armv7/tegra2/board.c just deals with DRAM.
> >
> > Hi Simon,
> >
> > the same register gets set in both boards/.../board.c and ap20.c ...
> > which itself is a bit suspicious.
> >
> > Cheers
>
> Hi Marek,
>
> Yes but I think you are still looking at the old code. The patches are
> applied now to u-boot-arm/master - can you please take a look? The
> clock/pinmux framework is basically complete now so most of the poking
> of registers elsewhere is gone.
>
> Regards,
> Simon
Ah, will do.
Cheers
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2011-10-13 14:04 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-01 0:12 [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Marek Vasut
2011-10-01 0:12 ` [U-Boot] [PATCH 2/2] Tegra2: Add support for Toradex Colibri T20 board Marek Vasut
2011-10-03 3:52 ` Simon Glass
2011-10-03 10:35 ` Marek Vasut
2011-10-03 16:09 ` Stephen Warren
2011-10-03 23:42 ` Simon Glass
2011-10-05 14:10 ` [U-Boot] [PATCH 1/2] Tegra2: Make XTal speed configurable Simon Glass
2011-10-05 17:05 ` Marek Vasut
2011-10-12 0:30 ` Simon Glass
2011-10-12 11:19 ` Marek Vasut
2011-10-12 21:17 ` Simon Glass
2011-10-13 0:27 ` Marek Vasut
2011-10-13 4:31 ` Simon Glass
2011-10-13 14:04 ` Marek Vasut
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).