[U-Boot] P2020 L2 cache as SRAM
Hi We're trying to configure the PPC P2020 cpu to use the L2 cache as SRAM so we can load the U-Boot code in there. However we stumble into problems. Sometimes the cpu goes on trap when trying to access this area. Sometimes there's no trap but we seem to access a different area. That's probably a problem with setting up a TLB/LAW. Has anybody already done this and could share some code with us? I've seen that there's a mpc85xx branch with quite some work going on. Where should we base our work on? Should we use the master or is it better to use this branch? I'm used to svn, not git, so there may be other options I don't know about yet. Thanks bye Fabi ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] PCIe and Dell parity interrupt
Hi We have a PCIe board with a PPC P2020 cpu. We're trying to set it up using U-Boot. Without initializing the PCIe part the PC won't even boot. Now we seem to have the initialization part running but we sometimes get an error on the PC: I/O Card Parity Interrupt Some googleing suggests that this is a problem with Dell computers only. In fact our test computer is a Dell as well, in another PC the same card and software came up without a problem. Is there something we did wrong or can do to prevent this error? I'm sure that some of our customers have some Dell's as well, so we'd like to get rid of this error. Thanks bye Fabi ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] SPI: mxc_spi: add SPI clock calculation and setup to the driver
On 01/16/2011 07:17 PM, Anatolij Gustschin wrote: The MXC SPI driver didn't calculate the SPI clock up to now and just used lowest possible divider 512 for DATA RATE in the control register. This results in very low transfer rates. Hi August, I sent last friday a patch for mxc_spi.c, too, adding support for mx35, and the two patches now conflict. Is it ok for you if I rebase your patch on top of mine and I post it again ? Best Regards, Stefano -- = DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] SPI: mxc_spi: add SPI clock calculation and setup to the driver
Hello Stefano, On Mon, 17 Jan 2011 10:00:51 +0100 Stefano Babic sba...@denx.de wrote: On 01/16/2011 07:17 PM, Anatolij Gustschin wrote: The MXC SPI driver didn't calculate the SPI clock up to now and just used lowest possible divider 512 for DATA RATE in the control register. This results in very low transfer rates. Hi August, I sent last friday a patch for mxc_spi.c, too, adding support for mx35, and the two patches now conflict. Is it ok for you if I rebase your patch on top of mine and I post it again ? Yes, it is ok. Thanks, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] SPI: mxc_spi: add SPI clock calculation and setup to the driver
On Mon, 17 Jan 2011 10:30:09 +0100 Anatolij Gustschin ag...@denx.de wrote: Hello Stefano, On Mon, 17 Jan 2011 10:00:51 +0100 Stefano Babic sba...@denx.de wrote: On 01/16/2011 07:17 PM, Anatolij Gustschin wrote: The MXC SPI driver didn't calculate the SPI clock up to now and just used lowest possible divider 512 for DATA RATE in the control register. This results in very low transfer rates. Hi August, I sent last friday a patch for mxc_spi.c, too, adding support for mx35, and the two patches now conflict. Is it ok for you if I rebase your patch on top of mine and I post it again ? Yes, it is ok. BTW: While at it, could you then also please fix a mistake in the commit description: it should be ... just highest possible divider 512. Thanks, Anatolij -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] SPI: mxc_spi: add SPI clock calculation and setup to the driver
On 01/17/2011 10:36 AM, Anatolij Gustschin wrote: BTW: While at it, could you then also please fix a mistake in the commit description: it should be ... just highest possible divider 512. Of course, I will do it. Stefano -- = DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] spi: add support SuperH SPI module
Hi, Shioda-san. 2011/1/17 Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com: SH7757 has SPI module. This patch supports it. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- drivers/spi/Makefile | 1 + drivers/spi/sh_spi.c | 295 ++ 2 files changed, 296 insertions(+), 0 deletions(-) create mode 100644 drivers/spi/sh_spi.c diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile index e34a124..d582fbb 100644 --- a/drivers/spi/Makefile +++ b/drivers/spi/Makefile @@ -37,6 +37,7 @@ COBJS-$(CONFIG_MPC8XXX_SPI) += mpc8xxx_spi.o COBJS-$(CONFIG_MXC_SPI) += mxc_spi.o COBJS-$(CONFIG_OMAP3_SPI) += omap3_spi.o COBJS-$(CONFIG_SOFT_SPI) += soft_spi.o +COBJS-$(CONFIG_SH_SPI) += sh_spi.o COBJS := $(COBJS-y) SRCS := $(COBJS:.o=.c) diff --git a/drivers/spi/sh_spi.c b/drivers/spi/sh_spi.c new file mode 100644 index 000..89ea5e2 --- /dev/null +++ b/drivers/spi/sh_spi.c @@ -0,0 +1,295 @@ +/* + * SH SPI driver + * + * Copyright (C) 2011 Renesas Solutions Corp. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * 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 St, Fifth Floor, Boston, MA 02110-1301 USA + * + */ + +#include common.h +#include malloc.h +#include spi.h +#include asm/io.h + +#define SPI_TBR 0x00 +#define SPI_RBR 0x00 +#define SPI_CR1 0x08 +#define SPI_CR2 0x10 +#define SPI_CR3 0x18 +#define SPI_CR4 0x20 + +/* CR1 */ +#define SPI_TBE 0x80 +#define SPI_TBF 0x40 +#define SPI_RBE 0x20 +#define SPI_RBF 0x10 +#define SPI_PFONRD 0x08 +#define SPI_SSDB 0x04 +#define SPI_SSD 0x02 +#define SPI_SSA 0x01 + +/* CR2 */ +#define SPI_RSTF 0x80 +#define SPI_LOOPBK 0x40 +#define SH_SPI_CPOL 0x20 +#define SH_SPI_CPHA 0x10 +#define SPI_L1M0 0x08 + +/* CR3 */ +#define SPI_MAX_BYTE 0xFF + +/* CR4 */ +#define SPI_TBEI 0x80 +#define SPI_TBFI 0x40 +#define SPI_RBEI 0x20 +#define SPI_RBFI 0x10 +#define SPI_WPABRT 0x04 +#define SPI_SSS 0x01 + +#define SPI_FIFO_SIZE 32 + Please add prefix of SH_. +static void sh_spi_write(unsigned long data, unsigned long offset) +{ + writel(data, CONFIG_SH_SPI_BASE + offset); +} + snip \ + +int spi_cs_is_valid(unsigned int bus, unsigned int cs) +{ + return 1; +} Why does this function return 1 every time? + +void spi_cs_activate(struct spi_slave *slave) +{ + +} + +void spi_cs_deactivate(struct spi_slave *slave) +{ + +} + -- 1.7.1 Best regards, Nobuhiro -- Nobuhiro Iwamatsu iwamatsu at {nigauri.org / debian.org} GPG ID: 40AD1FA6 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] net: sh_eth: add support for SH7757's ETHER
Hi, Shimoda-san. On Mon, Jan 17, 2011 at 01:09:23PM +0900, Yoshihiro Shimoda wrote: SH7757 has ETHER and GETHER. This patch supports EHTER only. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- drivers/net/sh_eth.c | 42 -- drivers/net/sh_eth.h | 53 +++-- 2 files changed, 89 insertions(+), 6 deletions(-) diff --git a/drivers/net/sh_eth.c b/drivers/net/sh_eth.c index 53d918d..155acca 100644 --- a/drivers/net/sh_eth.c +++ b/drivers/net/sh_eth.c @@ -277,6 +277,7 @@ int sh_eth_recv(struct eth_device *dev) static int sh_eth_reset(struct sh_eth_dev *eth) { int port = eth-port; +#if defined(CONFIG_CPU_SH7763) int ret = 0, i; /* Start e-dmac transmitter and receiver */ @@ -296,6 +297,13 @@ static int sh_eth_reset(struct sh_eth_dev *eth) } return ret; +#else + outl(inl(EDMR(port)) | EDMR_SRST, EDMR(port)); + udelay(3000); + outl(inl(EDMR(port)) ~EDMR_SRST, EDMR(port)); + + return 0; +#endif } static int sh_eth_tx_desc_init(struct sh_eth_dev *eth) @@ -339,9 +347,11 @@ static int sh_eth_tx_desc_init(struct sh_eth_dev *eth) /* Point the controller to the tx descriptor list. Must use physical addresses */ outl(ADDR_TO_PHY(port_info-tx_desc_base), TDLAR(port)); +#if defined(CONFIG_CPU_SH7763) outl(ADDR_TO_PHY(port_info-tx_desc_base), TDFAR(port)); outl(ADDR_TO_PHY(cur_tx_desc), TDFXR(port)); outl(0x01, TDFFR(port));/* Last discriptor bit */ +#endif err: return ret; @@ -405,9 +415,11 @@ static int sh_eth_rx_desc_init(struct sh_eth_dev *eth) /* Point the controller to the rx descriptor list */ outl(ADDR_TO_PHY(port_info-rx_desc_base), RDLAR(port)); +#if defined(CONFIG_CPU_SH7763) outl(ADDR_TO_PHY(port_info-rx_desc_base), RDFAR(port)); outl(ADDR_TO_PHY(cur_rx_desc), RDFXR(port)); outl(RDFFR_RDLF, RDFFR(port)); +#endif return ret; @@ -532,11 +544,18 @@ static int sh_eth_config(struct sh_eth_dev *eth, bd_t *bd) outl(0, TFTR(port)); outl((FIFO_SIZE_T | FIFO_SIZE_R), FDR(port)); outl(RMCR_RST, RMCR(port)); +#ifndef CONFIG_CPU_SH7757 outl(0, RPADIR(port)); +#endif outl((FIFO_F_D_RFF | FIFO_F_D_RFD), FCFTR(port)); /* Configure e-mac registers */ +#if defined(CONFIG_CPU_SH7757) + outl(ECSIPR_BRCRXIP | ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | + ECSIPR_MPDIP | ECSIPR_ICDIP, ECSIPR(port)); +#else outl(0, ECSIPR(port)); +#endif /* Set Mac address */ val = dev-enetaddr[0] 24 | dev-enetaddr[1] 16 | @@ -547,11 +566,16 @@ static int sh_eth_config(struct sh_eth_dev *eth, bd_t *bd) outl(val, MALR(port)); outl(RFLR_RFL_MIN, RFLR(port)); +#ifndef CONFIG_CPU_SH7757 outl(0, PIPR(port)); +#endif outl(APR_AP, APR(port)); outl(MPR_MP, MPR(port)); +#ifdef CONFIG_CPU_SH7757 + outl(TPAUSER_UNLIMITED, TPAUSER(port)); +#else outl(TPAUSER_TPAUSE, TPAUSER(port)); - +#endif /* Configure phy */ ret = sh_eth_phy_config(eth); if (ret) { @@ -562,6 +586,8 @@ static int sh_eth_config(struct sh_eth_dev *eth, bd_t *bd) phy_status = sh_eth_mii_read_phy_reg(port, port_info-phy_addr, 1); /* Set the transfer speed */ + val = 0x0; Is this change necessary? +#ifdef CONFIG_CPU_SH7763 if (phy_status (PHY_S_100X_F|PHY_S_100X_H)) { printf(SHETHER_NAME : 100Base/); outl(GECMR_100B, GECMR(port)); @@ -569,14 +595,24 @@ static int sh_eth_config(struct sh_eth_dev *eth, bd_t *bd) printf(SHETHER_NAME : 10Base/); outl(GECMR_10B, GECMR(port)); } +#endif +#if defined(CONFIG_CPU_SH7757) + if (phy_status (PHY_S_100X_F|PHY_S_100X_H)) { + printf(100Base/); + outl(1, RTRATE(port)); + } else { + printf(10Base/); + outl(0, RTRATE(port)); + } +#endif /* Check if full duplex mode is supported by the phy */ if (phy_status (PHY_S_100X_F|PHY_S_10T_F)) { printf(Full\n); - outl((ECMR_CHG_DM|ECMR_RE|ECMR_TE|ECMR_DM), ECMR(port)); + outl(val | (ECMR_CHG_DM|ECMR_RE|ECMR_TE|ECMR_DM), ECMR(port)); } else { printf(Half\n); - outl((ECMR_CHG_DM|ECMR_RE|ECMR_TE), ECMR(port)); + outl(val | (ECMR_CHG_DM|ECMR_RE|ECMR_TE), ECMR(port)); } Is this change necessary? Because val is changed to 0 at the top, I do not think that this change is necessary. Best regards, Nobuhiro ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] sh: add support for sh7757lcr board
On Mon, Jan 17, 2011 at 01:11:08PM +0900, Yoshihiro Shimoda wrote: The R0P7757LC0030RL board has SH7757, 256MB DDR3-SDRAM, SPI ROM, Ethernet, and more. This patch supports the following functions: - 256MB DDR3-SDRAM - SPI ROM - Ethernet Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- arch/sh/include/asm/cpu_sh4.h |2 + arch/sh/include/asm/cpu_sh7757.h| 263 +++ board/renesas/sh7757lcr/Makefile| 42 +++ board/renesas/sh7757lcr/config.mk | 28 ++ board/renesas/sh7757lcr/lowlevel_init.S | 558 +++ board/renesas/sh7757lcr/sh7757lcr.c | 436 board/renesas/sh7757lcr/spi-boot.c | 111 ++ board/renesas/sh7757lcr/u-boot.lds | 101 ++ boards.cfg |1 + doc/README.sh7757lcr| 64 include/configs/sh7757lcr.h | 143 11 files changed, 1749 insertions(+), 0 deletions(-) create mode 100644 arch/sh/include/asm/cpu_sh7757.h create mode 100644 board/renesas/sh7757lcr/Makefile create mode 100644 board/renesas/sh7757lcr/config.mk create mode 100644 board/renesas/sh7757lcr/lowlevel_init.S create mode 100644 board/renesas/sh7757lcr/sh7757lcr.c create mode 100644 board/renesas/sh7757lcr/spi-boot.c create mode 100644 board/renesas/sh7757lcr/u-boot.lds create mode 100644 doc/README.sh7757lcr create mode 100644 include/configs/sh7757lcr.h diff --git a/arch/sh/include/asm/cpu_sh4.h b/arch/sh/include/asm/cpu_sh4.h index fdcebd6..9b29d3a 100644 --- a/arch/sh/include/asm/cpu_sh4.h +++ b/arch/sh/include/asm/cpu_sh4.h @@ -44,6 +44,8 @@ # include asm/cpu_sh7722.h #elif defined (CONFIG_CPU_SH7723) # include asm/cpu_sh7723.h +#elif defined (CONFIG_CPU_SH7757) +# include asm/cpu_sh7757.h #elif defined (CONFIG_CPU_SH7763) # include asm/cpu_sh7763.h #elif defined (CONFIG_CPU_SH7780) diff --git a/arch/sh/include/asm/cpu_sh7757.h b/arch/sh/include/asm/cpu_sh7757.h new file mode 100644 index 000..eb9ff5b --- /dev/null +++ b/arch/sh/include/asm/cpu_sh7757.h @@ -0,0 +1,263 @@ +/* + * Copyright (C) 2011 Renesas Solutions Corp. + * + * 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 _ASM_CPU_SH7757_H_ +#define _ASM_CPU_SH7757_H_ + +#define CCR 0xFF1C +#define WTCNT0xFFCC +#define CCR_CACHE_INIT 0x090b +#define CACHE_OC_NUM_WAYS 1 + +/* Exception */ +#define EXPEVT 0xff24 +#define INTEVT 0xff28 + +/* Watchdog */ +#define WTCSR0 0xffcc0002 +#define WRSTCSR_R0xffcc0003 +#define WRSTCSR_W0xffcc0002 +#define WTCSR_PREFIX 0xa500 +#define WRSTCSR_PREFIX 0x6900 +#define WRSTCSR_WOVF_PREFIX 0x9600 + + +/* SCIF */ +#define SCIF0_BASE 0xfe4b /* The real name is SCIF2 */ +#define SCIF1_BASE 0xfe4c /* The real name is SCIF3 */ +#define SCIF2_BASE 0xfe4d /* The real name is SCIF4 */ + +/* SerMux */ +#define SMR0 0xfe47 + +/* TMU0 */ +#define TSTR 0xFE430004 +#define TOCR 0xFE43 +#define TSTR00xFE430004 +#define TCOR00xFE430008 +#define TCNT00xFE43000C +#define TCR0 0xFE430010 +#define TCOR10xFE430014 +#define TCNT10xFE430018 +#define TCR1 0xFE43001C +#define TCOR20xFE430020 +#define TCNT20xFE430024 +#define TCR2 0xFE430028 +#define TCPR20xFE43002C + +/* BSC */ +#define BCR 0xFE801000 +#define CS0BCR 0xFF802000 +#define CS0WCR 0xFF802008 +#define CS4BCR 0xFF802040 +#define CS4WCR 0xFF802048 +#define CS5BCR 0xFF802050 +#define CS5WCR 0xFF802058 +#define CS6BCR 0xFF802060 +#define CS6WCR 0xFF802068 + +/* DDR3IF */ +#define DDR3IF_BASE 0xfe80 +#define DBSTATE0 (DDR3IF_BASE + 0x0008) +#define DBSTATE1 (DDR3IF_BASE + 0x000c) +#define DBACEN (DDR3IF_BASE + 0x0010) +#define DBRFEN
[U-Boot] [PATCH 03/13] sh: Remove config.mk for r2dplus board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/r2dplus/config.mk | 23 --- include/configs/r2dplus.h |1 + 2 files changed, 1 insertions(+), 23 deletions(-) delete mode 100644 board/renesas/r2dplus/config.mk diff --git a/board/renesas/r2dplus/config.mk b/board/renesas/r2dplus/config.mk deleted file mode 100644 index 55163b9..000 --- a/board/renesas/r2dplus/config.mk +++ /dev/null @@ -1,23 +0,0 @@ -# -# Copyright (C) 2007,2008 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# 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 -# -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# -CONFIG_SYS_TEXT_BASE = 0x0FFC diff --git a/include/configs/r2dplus.h b/include/configs/r2dplus.h index 8689513..5534b49 100644 --- a/include/configs/r2dplus.h +++ b/include/configs/r2dplus.h @@ -40,6 +40,7 @@ #define CONFIG_SYS_SDRAM_BASE (0x8C00) #define CONFIG_SYS_SDRAM_SIZE (0x0400) +#define CONFIG_SYS_TEXT_BASE 0x0FFC #define CONFIG_SYS_LONGHELP #define CONFIG_SYS_PROMPT = #define CONFIG_SYS_CBSIZE 256 -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 02/13] sh: Remove config.mk for ap325rxa board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/ap325rxa/config.mk | 26 -- include/configs/ap325rxa.h |2 ++ 2 files changed, 2 insertions(+), 26 deletions(-) delete mode 100644 board/renesas/ap325rxa/config.mk diff --git a/board/renesas/ap325rxa/config.mk b/board/renesas/ap325rxa/config.mk deleted file mode 100644 index f572afd..000 --- a/board/renesas/ap325rxa/config.mk +++ /dev/null @@ -1,26 +0,0 @@ -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/ap325rxa.h b/include/configs/ap325rxa.h index e7f37f5..711dd3c 100644 --- a/include/configs/ap325rxa.h +++ b/include/configs/ap325rxa.h @@ -63,6 +63,8 @@ #define AP325RXA_FLASH_BASE_1 (0xA000) #define AP325RXA_FLASH_BANK_SIZE (128 * 1024 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x8FFC + /* undef to save memory*/ #define CONFIG_SYS_LONGHELP /* Monitor Command Prompt */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 04/13] sh: Remove config.mk for r7780mp board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/r7780mp/config.mk | 27 --- include/configs/r7780mp.h |1 + 2 files changed, 1 insertions(+), 27 deletions(-) delete mode 100644 board/renesas/r7780mp/config.mk diff --git a/board/renesas/r7780mp/config.mk b/board/renesas/r7780mp/config.mk deleted file mode 100644 index 70ee3fd..000 --- a/board/renesas/r7780mp/config.mk +++ /dev/null @@ -1,27 +0,0 @@ -# -# Copyright (C) 2007,2008 Nobuhiro Iwamatsu -# -# board/r77870mp/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x0FFC diff --git a/include/configs/r7780mp.h b/include/configs/r7780mp.h index 802416f..e1febf3 100644 --- a/include/configs/r7780mp.h +++ b/include/configs/r7780mp.h @@ -60,6 +60,7 @@ /* check for keypress on bootdelay==0 */ /*#define CONFIG_ZERO_BOOTDELAY_CHECK*/ +#define CONFIG_SYS_TEXT_BASE 0x0FFC #define CONFIG_SYS_SDRAM_BASE (0x0800) #define CONFIG_SYS_SDRAM_SIZE (128 * 1024 * 1024) -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 07/13] sh: Remove config.mk for sh7785lcr board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/sh7785lcr/config.mk | 31 --- include/configs/sh7785lcr.h |2 ++ 2 files changed, 2 insertions(+), 31 deletions(-) delete mode 100644 board/renesas/sh7785lcr/config.mk diff --git a/board/renesas/sh7785lcr/config.mk b/board/renesas/sh7785lcr/config.mk deleted file mode 100644 index 6853d2b..000 --- a/board/renesas/sh7785lcr/config.mk +++ /dev/null @@ -1,31 +0,0 @@ -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# -sinclude $(OBJTREE)/board/$(BOARDDIR)/config.tmp - -ifdef CONFIG_SH_32BIT -CONFIG_SYS_TEXT_BASE = 0x8FF8 -else -CONFIG_SYS_TEXT_BASE = 0x0ff8 -endif diff --git a/include/configs/sh7785lcr.h b/include/configs/sh7785lcr.h index a95a759..8eb9a12 100644 --- a/include/configs/sh7785lcr.h +++ b/include/configs/sh7785lcr.h @@ -63,6 +63,7 @@ /* MEMORY */ #if defined(CONFIG_SH_32BIT) +#define CONFIG_SYS_TEXT_BASE 0x8FF8 /* 0x4000 - 0x47FF does not use */ #define CONFIG_SH_SDRAM_OFFSET (0x800) #define SH7785LCR_SDRAM_PHYS_BASE (0x4000 + CONFIG_SH_SDRAM_OFFSET) @@ -72,6 +73,7 @@ #define SH7785LCR_FLASH_BANK_SIZE (64 * 1024 * 1024) #define SH7785LCR_USB_BASE (0xa600) #else +#define CONFIG_SYS_TEXT_BASE 0x0FF8 #define SH7785LCR_SDRAM_BASE (0x0800) #define SH7785LCR_SDRAM_SIZE (128 * 1024 * 1024) #define SH7785LCR_FLASH_BASE_1 (0xa000) -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 01/13] sh: Remove config.mk for MigoR board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/MigoR/config.mk | 31 --- include/configs/MigoR.h |1 + 2 files changed, 1 insertions(+), 31 deletions(-) delete mode 100644 board/renesas/MigoR/config.mk diff --git a/board/renesas/MigoR/config.mk b/board/renesas/MigoR/config.mk deleted file mode 100644 index ffe954c..000 --- a/board/renesas/MigoR/config.mk +++ /dev/null @@ -1,31 +0,0 @@ -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# Copyright (C) 2007 -# Kenati Technologies, Inc. -# -# board/MigoR/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/MigoR.h b/include/configs/MigoR.h index 8a6b8d0..7d9368c 100644 --- a/include/configs/MigoR.h +++ b/include/configs/MigoR.h @@ -59,6 +59,7 @@ #define MIGO_R_FLASH_BASE_1(0xA000) #define MIGO_R_FLASH_BANK_SIZE (64 * 1024 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 09/13] sh: Remove config.mk for ms7722se board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/ms7722se/config.mk | 31 --- include/configs/ms7722se.h |1 + 2 files changed, 1 insertions(+), 31 deletions(-) delete mode 100644 board/ms7722se/config.mk diff --git a/board/ms7722se/config.mk b/board/ms7722se/config.mk deleted file mode 100644 index 3f1606b..000 --- a/board/ms7722se/config.mk +++ /dev/null @@ -1,31 +0,0 @@ -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# Copyright (C) 2007 -# Kenati Technologies, Inc. -# -# board/ms7722se/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/ms7722se.h b/include/configs/ms7722se.h index 1ddadf6..c8d04f7 100644 --- a/include/configs/ms7722se.h +++ b/include/configs/ms7722se.h @@ -57,6 +57,7 @@ #define MS7722SE_FLASH_BASE_1 (0xA000) #define MS7722SE_FLASH_BANK_SIZE (8*1024 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 08/13] sh: Remove config.mk for ms7720se board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/ms7720se/config.mk | 34 -- include/configs/ms7720se.h |1 + 2 files changed, 1 insertions(+), 34 deletions(-) delete mode 100644 board/ms7720se/config.mk diff --git a/board/ms7720se/config.mk b/board/ms7720se/config.mk deleted file mode 100644 index d2944a6..000 --- a/board/ms7720se/config.mk +++ /dev/null @@ -1,34 +0,0 @@ -# -# Copyright (C) 2007 -# Yoshihiro Shimoda shimoda.yoshih...@renesas.com -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# Copyright (C) 2007 -# Kenati Technologies, Inc. -# -# board/ms7720se/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/ms7720se.h b/include/configs/ms7720se.h index 5304237..dd79b2d 100644 --- a/include/configs/ms7720se.h +++ b/include/configs/ms7720se.h @@ -52,6 +52,7 @@ #define MS7720SE_FLASH_BASE_1 0xA000 #define MS7720SE_FLASH_BANK_SIZE (8 * 1024 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 10/13] sh: Remove config.mk for ms7750se board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/ms7750se/config.mk | 23 --- include/configs/ms7750se.h |1 + 2 files changed, 1 insertions(+), 23 deletions(-) delete mode 100644 board/ms7750se/config.mk diff --git a/board/ms7750se/config.mk b/board/ms7750se/config.mk deleted file mode 100644 index ba4d155..000 --- a/board/ms7750se/config.mk +++ /dev/null @@ -1,23 +0,0 @@ -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# 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 -# -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/ms7750se.h b/include/configs/ms7750se.h index 9b43acb..16fb0d4 100644 --- a/include/configs/ms7750se.h +++ b/include/configs/ms7750se.h @@ -64,6 +64,7 @@ /* List of legal baudrate settings for this board */ #define CONFIG_SYS_BAUDRATE_TABLE { 115200, 57600, 38400, 19200, 9600 } +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_MEMTEST_START (CONFIG_SYS_SDRAM_BASE) #define CONFIG_SYS_MEMTEST_END (CONFIG_SYS_TEXT_BASE - 0x10) -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 13/13] sh: Remove config.mk for shmin board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/shmin/config.mk | 27 --- include/configs/shmin.h |1 + 2 files changed, 1 insertions(+), 27 deletions(-) delete mode 100644 board/shmin/config.mk diff --git a/board/shmin/config.mk b/board/shmin/config.mk deleted file mode 100644 index 0c7605e..000 --- a/board/shmin/config.mk +++ /dev/null @@ -1,27 +0,0 @@ -# -# Copyright (C) 2010 Nobuhiro Iwamatsu -# -# u-boot/board/shmin/config.mk -# -# 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 - -# -# TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8DFB diff --git a/include/configs/shmin.h b/include/configs/shmin.h index 54a1587..525f6e2 100644 --- a/include/configs/shmin.h +++ b/include/configs/shmin.h @@ -56,6 +56,7 @@ #define SHMIN_SDRAM_BASE (0x8C00) #define SHMIN_FLASH_BASE_1 (0xA000) +#define CONFIG_SYS_TEXT_BASE 0x8DFB #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 11/13] sh: Remove config.mk for mpr2 board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/mpr2/config.mk | 37 - include/configs/mpr2.h |2 ++ 2 files changed, 2 insertions(+), 37 deletions(-) delete mode 100644 board/mpr2/config.mk diff --git a/board/mpr2/config.mk b/board/mpr2/config.mk deleted file mode 100644 index 4a4bca1..000 --- a/board/mpr2/config.mk +++ /dev/null @@ -1,37 +0,0 @@ -# -# Copyright (C) 2007 -# Yoshihiro Shimoda shimoda.yoshih...@renesas.com -# -# Copyright (C) 2007 -# Nobuhiro Iwamatsu iwama...@nigauri.org -# -# Copyright (C) 2007 -# Kenati Technologies, Inc. -# -# Copyright (C) 2008 -# Mark Jonas mark.jo...@de.bosch.com -# -# board/mpr2/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/mpr2.h b/include/configs/mpr2.h index 311f524..f32899f 100644 --- a/include/configs/mpr2.h +++ b/include/configs/mpr2.h @@ -58,6 +58,8 @@ #define CONFIG_SYS_MONITOR_LEN (128 * 1024) #define CONFIG_SYS_MALLOC_LEN (256 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x8FFC + /* Memory */ #define CONFIG_SYS_SDRAM_BASE 0x8C00 #define CONFIG_SYS_SDRAM_SIZE (64 * 1024 * 1024) -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 12/13] sh: Remove config.mk for espt board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/espt/config.mk |9 - include/configs/espt.h |1 + 2 files changed, 1 insertions(+), 9 deletions(-) delete mode 100644 board/espt/config.mk diff --git a/board/espt/config.mk b/board/espt/config.mk deleted file mode 100644 index 21b51de..000 --- a/board/espt/config.mk +++ /dev/null @@ -1,9 +0,0 @@ -# -# board/espt/config.mk -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC diff --git a/include/configs/espt.h b/include/configs/espt.h index 26389ed..ad3c335 100644 --- a/include/configs/espt.h +++ b/include/configs/espt.h @@ -56,6 +56,7 @@ #define CONFIG_BAUDRATE 115200 #define CONFIG_CONS_SCIF0 1 +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 06/13] sh: Remove config.mk for sh7763rdp board
Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/sh7763rdp/config.mk | 11 --- include/configs/sh7763rdp.h |1 + 2 files changed, 1 insertions(+), 11 deletions(-) delete mode 100644 board/renesas/sh7763rdp/config.mk diff --git a/board/renesas/sh7763rdp/config.mk b/board/renesas/sh7763rdp/config.mk deleted file mode 100644 index 54c1a5b..000 --- a/board/renesas/sh7763rdp/config.mk +++ /dev/null @@ -1,11 +0,0 @@ -# -# board/sh7763rdp/config.mk -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x8FFC - -# PLATFORM_CPPFLAGS += -DCONFIG_MULTIBOOT diff --git a/include/configs/sh7763rdp.h b/include/configs/sh7763rdp.h index 209cb88..6a08413 100644 --- a/include/configs/sh7763rdp.h +++ b/include/configs/sh7763rdp.h @@ -56,6 +56,7 @@ #define CONFIG_BAUDRATE 115200 #define CONFIG_CONS_SCIF2 1 +#define CONFIG_SYS_TEXT_BASE 0x8FFC #define CONFIG_SYS_LONGHELP/* undef to save memory */ #define CONFIG_SYS_PROMPT =/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE 256 /* Buffer size for input from the Console */ -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] sh: Fix MigoR of boards.cfg
There is not break character between board name and CPU. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- boards.cfg |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/boards.cfg b/boards.cfg index 0574bb2..705847c 100644 --- a/boards.cfg +++ b/boards.cfg @@ -755,7 +755,7 @@ r7780mp sh sh4 r7780mp renesas sh7763rdpsh sh4 sh7763rdp renesas- sh7785lcrsh sh4 sh7785lcr renesas- sh7785lcr_32bit sh sh4 sh7785lcr renesas- sh7785lcr:SH_32BIT=1 -MigoRsh sh4 MigoR renesas - +MigoRsh sh4 MigoR renesas- grsim_leon2 sparc leon2 - gaisler gr_cpci_ax2000 sparc leon3 - gaisler gr_ep2s60sparc leon3 - gaisler -- 1.7.2.3 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] net: sh_eth: add cache handling
On Mon, Jan 17, 2011 at 01:08:58PM +0900, Yoshihiro Shimoda wrote: Some CPU needs cache handling. So this patch add the config of CONFIG_SH_ETHER_CACHE_WRITEBACK, and it calls wback function. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com --- drivers/net/sh_eth.c |9 + 1 files changed, 9 insertions(+), 0 deletions(-) Applied, thank. Nobuhiro ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env
On Mon, Jan 17, 2011 at 01:07:20PM +0900, Yoshihiro Shimoda wrote: Fix the problem which cannot build the U-boot, if we only set the CONFIG_ENV_IS_IN_SPI_FLASH. Signed-off-by: Yoshihiro Shimoda yoshihiro.shimoda...@renesas.com Acked-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- include/environment.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/environment.h b/include/environment.h index 082b3e1..8a4f393 100644 --- a/include/environment.h +++ b/include/environment.h @@ -106,6 +106,7 @@ extern unsigned long nand_env_oob_offset; /* Embedded env is only supported for some flash types */ #ifdef CONFIG_ENV_IS_EMBEDDED # if !defined(CONFIG_ENV_IS_IN_FLASH) \ + !defined(CONFIG_ENV_IS_IN_SPI_FLASH) \ !defined(CONFIG_ENV_IS_IN_NAND) \ !defined(CONFIG_ENV_IS_IN_ONENAND) # error CONFIG_ENV_IS_EMBEDDED not supported for your flash type -- 1.7.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 05/13] sh: Remove config.mk for rsk7203 board
I'm working on a rsk7203 derived board. I do not have a eth interface. How can I upgrade the kernel using uboot and the serial line? When I finish can I send the support for my board? Thanks On Monday 17 January 2011 13:50:55 Nobuhiro Iwamatsu wrote: Move CONFIG_SYS_TEXT_BASE to the board's config file, and remove the unnecessary config.mk file. Signed-off-by: Nobuhiro Iwamatsu iwama...@nigauri.org --- board/renesas/rsk7203/config.mk | 28 include/configs/rsk7203.h |1 + 2 files changed, 1 insertions(+), 28 deletions(-) delete mode 100644 board/renesas/rsk7203/config.mk diff --git a/board/renesas/rsk7203/config.mk b/board/renesas/rsk7203/config.mk deleted file mode 100644 index 5b533f6..000 --- a/board/renesas/rsk7203/config.mk +++ /dev/null @@ -1,28 +0,0 @@ -# -# Copyright (C) 2007,2008 Nobuhiro Iwamatsu -# Copyright (C) 2008 Renesas Solutions Corp. -# -# u-boot/board/rsk7203/config.mk -# -# 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 - -# -# CONFIG_SYS_TEXT_BASE refers to image _after_ relocation. -# -# NOTE: Must match value used in u-boot.lds (in this directory). -# - -CONFIG_SYS_TEXT_BASE = 0x0C7C diff --git a/include/configs/rsk7203.h b/include/configs/rsk7203.h index b3feaa8..0802140 100644 --- a/include/configs/rsk7203.h +++ b/include/configs/rsk7203.h @@ -54,6 +54,7 @@ #define RSK7203_FLASH_BASE_1 0x2000 /* Non cache */ #define RSK7203_FLASH_BANK_SIZE (4 * 1024 * 1024) +#define CONFIG_SYS_TEXT_BASE 0x0C7C #define CONFIG_SYS_LONGHELP /* undef to save memory */ #define CONFIG_SYS_PROMPT=/* Monitor Command Prompt */ #define CONFIG_SYS_CBSIZE256 /* Buffer size for input from the Console */ -- Ing. Fabio Giovagnini Aurion s.r.l. P.I e C.F. 00885711200 skype: aurion.giovagnini Tel. +39.051.594.78.24 Fax. +39.051.082.14.49 Cell. +39.335.83.50.919 www.aurion-tech.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] Is theer an IRC channel?
Is there please an IRC channel for real time chat about problem with uboot? Regards -- Ing. Fabio Giovagnini Aurion s.r.l. P.I e C.F. 00885711200 skype: aurion.giovagnini Tel. +39.051.594.78.24 Fax. +39.051.082.14.49 Cell. +39.335.83.50.919 www.aurion-tech.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH][v1] Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR
On Mon, Jan 17, 2011 at 12:40 AM, Prabhakar prabha...@freescale.com wrote: CONFIG_SYS_MPC85xx_SERDES1_ADDR was defined wrong as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET. It should be as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET. Signed-off-by: Prabhakar prabha...@freescale.com Please resubmit this patch using your full name in the signed-off-by line. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] U-Boot mpc8603 support
Is there anyone who has an experience on this CPU with U-Boot? -- M u r at A r t u n, MSc. Design Engineer be conservative in what you do, be liberal in what you accept from others ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] Uboot with at91sam9G20ek
Hello, I have spent now more than 2 weeks with U - boot and want the network interface to work so that tftpboot works but my current state seems not very encouraging. A) I am using the AT91SAM9G20EK board. I downloaded tried the U-Boot.2010.12. I have the following results (issues) when I try to build. 1) make at91sam9g20ek_nandflash_config make CROSS_COMPILE=arm-linux- Error 1: You need to define CONFIG_AT91FAMILY in your board config Action: Defined CONFIG_AT91FAMILY in the file /include/configs/at91sam9260ek.h Also defined CONFIG_AT91SAM9G20EK in the same file 2) Error 2: board.c:233: CONFIG_SYS_SDRAM_BASE undeclared Error 3: board.c:279: CONFIG_SYS_INIT_SP_ADDR undeclared Error 4: board.c:312: CONFIG_SYS_SDRAM_BASE undeclared In the file board.c, #define CONFIG_SYS_SDRAM_BASE 0x #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE - GENERATED_GBL_DATA_SIZE) 3) Error 5: Undefined reference to 'CONFIG_SYS_INIT_SP_ADDR' in file /arc/arm/cpu/arm926ejs/start.S Cannot find a work around Error 6: /drivers/spi/atmel_dataflash_spi.c:161: undefined reference to 'reset_timer_masked' Error 7: /drivers/spi/atmel_dataflash_spi.c:161: undefined reference to 'get_timer_masked' Why are these definitions removed in the 2010.12 build? B) Initially I was trying the u-boot-2010.09. I want to use the MII interface instead of RMII. so #undef ed RMII and #defined CONFIG_MII and CONFIG_CMD_MII The Ping to self as well as host does not function at all despite of correct settings. I tried with putting debug outputs in various functions called. Actually no data is sentout TXD0 - TXD3 and no activity on the TX_EN or EX_ER. At the end, I get ping failed; host xx.xx.xx.xx is not alive. Can anyone guide me regarding the further line of efforts. Thanks a lot in advance. Regards, Abhishek ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] U-Boot mpc8603 support
My apologies... I should have written MPC8306. On Mon, Jan 17, 2011 at 6:03 PM, Murat Artun artunmu...@gmail.com wrote: Is there anyone who has an experience on this CPU with U-Boot? -- M u r at A r t u n, MSc. Design Engineer be conservative in what you do, be liberal in what you accept from others -- M u r at A r t u n, MSc. Design Engineer be conservative in what you do, be liberal in what you accept from others ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/7] Add support for Freescale's mx35pdk board.
Hi, Stefano, 2011/1/14 Stefano Babic sba...@denx.de: The patch adds suupport for the Freescale's mx35pdk board (known as well as mx35_3stack). The board boots from the NOR flash. Following devices are supported: - two ethernet devices (FEC and SMC911x on debug board) - I2C - PMIC (MC13892) via I2C interface - UART - NOR flash (64MB) - NAND flash (2GB) - basic access to mc9sdz60 registers via I2C interface Signed-off-by: Stefano Babic sba...@denx.de --- MAINTAINERS | 1 + MAKEALL | 1 + arch/arm/include/asm/arch-mx35/mmu.h | 172 -- You want to remove this file, is there this file in the mainline? board/freescale/mx35pdk/Makefile | 49 board/freescale/mx35pdk/config.mk | 25 ++ board/freescale/mx35pdk/imximage.cfg | 62 + board/freescale/mx35pdk/lowlevel_init.S | 382 +++ board/freescale/mx35pdk/mx35pdk.c | 313 + board/freescale/mx35pdk/mx35pdk.h | 101 boards.cfg | 1 + doc/README.mx35pdk | 185 +++ include/configs/mx35pdk.h | 269 ++ 12 files changed, 1389 insertions(+), 172 deletions(-) delete mode 100644 arch/arm/include/asm/arch-mx35/mmu.h create mode 100644 board/freescale/mx35pdk/Makefile create mode 100644 board/freescale/mx35pdk/config.mk create mode 100644 board/freescale/mx35pdk/imximage.cfg create mode 100644 board/freescale/mx35pdk/lowlevel_init.S create mode 100644 board/freescale/mx35pdk/mx35pdk.c create mode 100644 board/freescale/mx35pdk/mx35pdk.h create mode 100644 doc/README.mx35pdk create mode 100644 include/configs/mx35pdk.h diff --git a/MAINTAINERS b/MAINTAINERS index d7cd09c..3abb4cb 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -554,6 +554,7 @@ Stefano Babic sba...@denx.de ea20 davinci polaris xscale trizepsiv xscale + mx35pdk i.MX35 mx51evk i.MX51 vision2 i.MX51 diff --git a/MAKEALL b/MAKEALL index a732e6a..31dbfe1 100755 --- a/MAKEALL +++ b/MAKEALL @@ -409,6 +409,7 @@ LIST_ARM11= \ mx31ads \ mx31pdk \ mx31pdk_nand \ + mx35pdk \ qong \ smdk6400 \ tnetv107x_evm \ diff --git a/arch/arm/include/asm/arch-mx35/mmu.h b/arch/arm/include/asm/arch-mx35/mmu.h deleted file mode 100644 index 1b15dba..000 --- a/arch/arm/include/asm/arch-mx35/mmu.h +++ /dev/null @@ -1,172 +0,0 @@ -/* - * Copyright 2004-2010 Freescale Semiconductor, Inc. All Rights Reserved. - */ - Ditto, [snip] diff --git a/board/freescale/mx35pdk/config.mk b/board/freescale/mx35pdk/config.mk new file mode 100644 index 000..3db1c85 --- /dev/null +++ b/board/freescale/mx35pdk/config.mk @@ -0,0 +1,25 @@ +# +# (C) Copyright 2010 +# Stefano Babic DENX Software Engineering sba...@denx.de. +# +# Copyright 2009 Freescale Semiconductor, Inc. All Rights Reserved. +# +# 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 +# +CONFIG_SYS_TEXT_BASE = 0xA000 Need move this config to board config file and remove this file. [snip] ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/7] Add support for Freescale's mx35pdk board.
On 01/17/2011 05:43 PM, Jason Liu wrote: Hi, Stefano, Hi Jason, You want to remove this file, is there this file in the mainline? Of course, it was a mistake. I drop it. diff --git a/board/freescale/mx35pdk/config.mk b/board/freescale/mx35pdk/config.mk new file mode 100644 index 000..3db1c85 --- /dev/null +++ b/board/freescale/mx35pdk/config.mk @@ -0,0 +1,25 @@ +# +# (C) Copyright 2010 +# Stefano Babic DENX Software Engineering sba...@denx.de. +# +# Copyright 2009 Freescale Semiconductor, Inc. All Rights Reserved. +# +# 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 +# +CONFIG_SYS_TEXT_BASE = 0xA000 Need move this config to board config file and remove this file. I have not understood this point. The link address CONFIG_SYS_TEXT_BASE is always set in the config.mk file, because it is needed by the linker process, and it is not set in the configuration file (I think you mean mx35pdk.h). All boards follow this rule. If I move it, the linker does not find it. Do you mean probably something else ? Best regards, Stefano -- = DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/7] Add support for Freescale's mx35pdk board.
On 01/17/2011 05:54 PM, Stefano Babic wrote: Need move this config to board config file and remove this file. I have not understood this point. The link address CONFIG_SYS_TEXT_BASE is always set in the config.mk file, because it is needed by the linker process, and it is not set in the configuration file (I think you mean mx35pdk.h). All boards follow this rule. If I move it, the linker does not find it. Do you mean probably something else ? Sorry, I checked again, I missed the new rule for TEXT_BASE - you are right, I will move it in boards.cfg (my preferred way). Best regards, Stefano Babic -- = DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: off...@denx.de = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] Booting linux 2.6.37+ on rsk7203 eva board
Hi All this is what I do: 1) I run u-Boot getting this output: U-Boot 2010.12-09860-g9cb5761-dirty (Jan 17 2011 - 18:05:12) CPU: SH2A BOARD: MLCD7203 DRAM: 64MB *** Warning - bad CRC, using default environment Destroy Hash Table: 0c7d6df8 table = (null) Create Hash Table: N=72 INSERT: table 0c7d6df8, filled 1/73 rv 0c780144 == name=bootargs value=console=ttySC0,115200 INSERT: table 0c7d6df8, filled 2/73 rv 0c780324 == name=baudrate value=115200 INSERT: table 0c7d6df8, filled 3/73 rv 0c7800cc == name=loadaddr value=0x0C20 INSERT: free(data = 0c77ffb8) INSERT: done fwc addr 2000 cmd f0 00f0 16bit x 16 bit fwc addr 2000 cmd aa 00aa 16bit x 16 bit fwc addr 20005554 cmd 55 0055 16bit x 16 bit fwc addr 2000 cmd 90 0090 16bit x 16 bit fwc addr 2000 cmd f0 00f0 16bit x 16 bit JEDEC PROBE: ID bf 236c 0 Found JEDEC Flash: SST 39VF6402B unlock address index 1 unlock addresses are 0x555/0x2aa erase_region_count = 128 erase_region_size = 65536 flash_protect ON: from 0x2000 to 0x2001 protect on 0 protect on 1 flash_protect ON: from 0x2002 to 0x2002 protect on 2 FLASH: 8MB In:serial Out: serial Err: serial = 2) I load my uImage getting this output from iminfo: = loady ## Ready for binary (ymodem) download to 0x0C20 at 115200 bps... Code, 7627(SOH)/0(STX)/0(CAN) packets, 4 retries ## Total Size = 0x000ee445 = 975941 Bytes = iminfo ## Checking Image at 0c20 ... Legacy image found Image Name: Linux-2.6.37+ Image Type: SuperH Linux Kernel Image (gzip compressed) Data Size:975877 Bytes = 953 KiB Load Address: 0c001000 Entry Point: 0c002000 Verifying Checksum ... OK = Any ideas about why the kernelò do not send out anything? -- Ing. Fabio Giovagnini Aurion s.r.l. P.I e C.F. 00885711200 skype: aurion.giovagnini Tel. +39.051.594.78.24 Fax. +39.051.082.14.49 Cell. +39.335.83.50.919 www.aurion-tech.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Booting linux 2.6.37+ on rsk7203 eva board
Dear Fabio Giovagnini, In message 201101171908.02343.fabio.giovagn...@aurion-tech.com you wrote: Hi All this is what I do: 1) I run u-Boot getting this output: U-Boot 2010.12-09860-g9cb5761-dirty (Jan 17 2011 - 18:05:12) CPU: SH2A BOARD: MLCD7203 This appears to be an out-of-tree port, i. e. code which is completely unknown to us. Any ideas about why the kernelò do not send out anything? We don't know your code, we don't know which exact commands you're running, we don't see your environment, we don't know nothing. How could we then come up with helpful comments? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de There are three ways to get something done: do it yourself, hire someone, or forbid your kids to do it. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Uboot with at91sam9G20ek
Hi Abhishek, I'm also trying to get U-Boot 2010.12 working on an AT91 board. I came across the following mailing list archive that may help. http://old.nabble.com/-U-Boot--various-at91-patches-to-make-some-boards- compilable-again-td30103062.html Here's a tinyurl (http://tinyurl.com/4fkedr3) in case the above line-wrapped oddly. Seems like we may need to patch in AT91 support. I hope someone on the list can indicate when the changes needed for AT91 support are integrated into the master. Please let me know if you get this working in the meantime! Thanks, Chris -Original Message- From: u-boot-boun...@lists.denx.de [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Abhishek Ashtekar Sent: Monday, January 17, 2011 8:19 AM To: u-boot@lists.denx.de Subject: [U-Boot] Uboot with at91sam9G20ek Hello, I have spent now more than 2 weeks with U - boot and want the network interface to work so that tftpboot works but my current state seems not very encouraging. A) I am using the AT91SAM9G20EK board. I downloaded tried the U-Boot.2010.12. I have the following results (issues) when I try to build. 1) make at91sam9g20ek_nandflash_config make CROSS_COMPILE=arm-linux- Error 1: You need to define CONFIG_AT91FAMILY in your board config Action: Defined CONFIG_AT91FAMILY in the file /include/configs/at91sam9260ek.h Also defined CONFIG_AT91SAM9G20EK in the same file 2) Error 2: board.c:233: CONFIG_SYS_SDRAM_BASE undeclared Error 3: board.c:279: CONFIG_SYS_INIT_SP_ADDR undeclared Error 4: board.c:312: CONFIG_SYS_SDRAM_BASE undeclared In the file board.c, #define CONFIG_SYS_SDRAM_BASE 0x #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE - GENERATED_GBL_DATA_SIZE) 3) Error 5: Undefined reference to 'CONFIG_SYS_INIT_SP_ADDR' in file /arc/arm/cpu/arm926ejs/start.S Cannot find a work around Error 6: /drivers/spi/atmel_dataflash_spi.c:161: undefined reference to 'reset_timer_masked' Error 7: /drivers/spi/atmel_dataflash_spi.c:161: undefined reference to 'get_timer_masked' Why are these definitions removed in the 2010.12 build? B) Initially I was trying the u-boot-2010.09. I want to use the MII interface instead of RMII. so #undef ed RMII and #defined CONFIG_MII and CONFIG_CMD_MII The Ping to self as well as host does not function at all despite of correct settings. I tried with putting debug outputs in various functions called. Actually no data is sentout TXD0 - TXD3 and no activity on the TX_EN or EX_ER. At the end, I get ping failed; host xx.xx.xx.xx is not alive. Can anyone guide me regarding the further line of efforts. Thanks a lot in advance. Regards, Abhishek ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Pull request: nand flash
Dear Scott Wood, In message 20110112231524.ga9...@udp111988uds.am.freescale.net you wrote: The following changes since commit c6b734f5aea2ba75caaa1929f7e649ecda8d2f31: Wolfgang Denk (1): Merge branch 'master' of git://git.denx.de/u-boot-sh are available in the git repository at: git://git.denx.de/u-boot-nand-flash.git master Alexander Holler (1): nand: fix bug with multiple NAND devices if CONFIG_MTD_DEVICE is defined. Lei Wen (2): NAND: add the ability to directly write yaffs image mtd: nand: Allow caller to pass alternative ID table to nand_scan_ident() common/cmd_nand.c | 15 +- drivers/mtd/nand/atmel_nand.c |2 +- drivers/mtd/nand/nand.c |2 +- drivers/mtd/nand/nand_base.c | 29 ++- drivers/mtd/nand/nand_util.c | 63 +++- include/linux/mtd/nand.h |4 ++- include/nand.h|2 +- 7 files changed, 90 insertions(+), 27 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de There are no data that cannot be plotted on a straight line if the axis are chosen correctly. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [GIT PULL] Pull request: u-boot-imx
Dear Stefano Babic, In message 4d2e9e54.6040...@denx.de you wrote: Hi Wolfgang, The following changes since commit cdc51c294ad33879c4e57edf4c9d2155381b1d59: Merge branch 'next' of ../next (2010-12-22 21:16:17 +0100) Is this really you working base? Then you should probably rebase against current master first. are available in the git repository at: git://www.denx.de/git/u-boot-imx.git master Please sends your pull request to the Albert, ARM custodian. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de A modem is a baudy house. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [GIT PULL] Pull request: u-boot-i2c
Dear Heiko Schocher, In message 4d2ea97d.2010...@denx.de you wrote: Hello Wolfgang, The following changes since commit 89c95f0cd3f8140f3b8a82a22a6a144c148d09c6: asm-offsets: generate bd_t size (2011-01-09 18:08:20 +0100) are available in the git repository at: git://git.denx.de/u-boot-i2c.git master Chris Packham (1): pca953x: support 16-pin devices README |4 ++ drivers/gpio/pca953x.c | 114 ++-- 2 files changed, 95 insertions(+), 23 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de For every problem there is one solution which is simple, neat, and wrong.- H. L. Mencken ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Please pull u-boot-mpc85xx.git
Dear Kumar Gala, In message Pine.LNX.4.64.1101140150260.8040@localhost.localdomain you wrote: The following changes since commit c6b734f5aea2ba75caaa1929f7e649ecda8d2f31: Merge branch 'master' of git://git.denx.de/u-boot-sh (2011-01-12 23:59:53 +0100) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx master Becky Bruce (9): socrates: rename sdram_setup fixed_sdram() mpc8569mds: Remove unnecessary CONFIG_SYS_LBC_SDRAM_BASE definition MPC8568/MPC8569: Remove CONFIG_DDR_DLL define tqm85xx: create fixed_sdram() to do sdram setup mpc85xx/tlb.c: Allow platforms to specify wimge bits mpc85xx boards: initdram() cleanup/bugfix mpc85xx: rename sdram_init() lbc_sdram_init() 85xx boards: Rename CONFIG_DDR_DLL to CONFIG_SYS_FSL_ERRATUM_DDR_MSYNC_IN MPC8xxx DDR: align informational prints Chenhui Zhao (1): fsl_esdhc: Fix esdhc disabled problem on some platforms Jerry Huang (1): fsl_esdhc: Add the workaround for erratum ESDHC111 (enable on P4080) Kumar Gala (46): powerpc/85xx: Removed support for ATUM8548 board powerpc/85xx: Removed support for MPC8540EVAL board powerpc/85xx: Remove support for PM854/PM856 boards powerpc/85xx: Add support for booting from NAND on MPC8572DS powerpc/85xx: Replace CONFIG_SYS_HAS_SERDES with a weak function powerpc/85xx: Create a SERDES section in Makefile powerpc/86xx: Add SERDES support on MPC8641 MPC8610 powerpc/85xx: Add is_serdes_configured() support for P2020 SERDES powerpc/85xx: Add is_serdes_configured() support for MPC8572 SERDES powerpc/85xx: Add is_serdes_configured() support for MPC8548 SERDES powerpc/85xx: Add is_serdes_configured() support for MPC8568 SERDES powerpc/85xx: Add is_serdes_configured() support for MPC8569 SERDES powerpc/85xx: Add is_serdes_configured() support for MPC8544 SERDES powerpc/85xx: Add is_serdes_configured() support for P1021 SERDES powerpc/8xxx: Replace is_fsl_pci_cfg with is_serdes_configured powerpc/85xx: Cleanup SGMII detection and reporting powerpc/85xx: Fix bug in dcache_disable powerpc/fsl-pci: Determine pci_controller based on cfg addr for dts fixup powerpc/fsl-pci: Add generic code to setup PCIe controllers powerpc/85xx: Rework MPC8572DS pci_init_board to use common FSL PCIe code powerpc/85xx: Rework P2020DS pci_init_board to use common FSL PCIe code powerpc/85xx: Rework MPC8544DS pci_init_board to use common FSL PCIe code powerpc/85xx: Rework MPC8536DS pci_init_board to use common FSL PCIe code powerpc/86xx: Rework MPC8641HPCN pci_init_board to use common FSL PCIe code powerpc/85xx: Rework MPC8548CDS pci_init_board to use common FSL PCIe code powerpc/8xxx: Rework XES boards pci_init_board to use common FSL PCIe code powerpc/85xx: Rework TQM boards pci_init_board to use common FSL PCIe code powerpc/85xx: Rework MPC8568MDS pci_init_board to use common FSL PCIe code powerpc/85xx: Rework MPC8569MDS pci_init_board to use common FSL PCIe code powerpc/85xx: Rework P1_P2_RDB pci_init_board to use common FSL PCIe code powerpc/86xx: Rework MPC8610HPCD pci_init_board to use common FSL PCIe code powerpc/86xx: Rework SBC8641 pci_init_board to use common FSL PCIe code powerpc/85xx: Rework SBC8548 pci_init_board to use common FSL PCIe code powerpc/85xx: Rework corenet_ds pci_init_board to use common FSL PCIe code powerpc/8xxx: Refactor SRIO initialization into common code powerpc/85xx: Convert MPC8548CDS to use common SRIO init code powerpc/85xx: Convert MPC8568MDS to use common SRIO init code powerpc/85xx: Convert MPC8569MDS to use common SRIO init code powerpc/86xx: Enable common SRIO init code powerpc/86xx: Convert MPC8641HPCN to use common SRIO init code powerpc/86xx: Convert SBC8641 to use common SRIO init code powerpc/8xxx: Move fsl_is_spd() into generic 8xxx ddr code powerpc/85xx: Bump up the CONFIG_SYS_BOOTM_LEN to 16M on FSL 85xx boards powerpc/85xx: Add the workaround for erratum CPC-A002 (enable on P4080) powerpc/85xx: Add the workaround for erratum CPC-A003 (enable on P4080) powerpc/85xx: Add the workaround for erratum ELBC-A001 (enable on P4080) Li Yang (1): powerpc/85xx: Add SRIO support to P2020DS Paul Gortmaker (1): MPC8xxx: Update maintainer entry for Wind River sbc8xxx boards Peter Tyser (1): fsl_pci: Update PCIe boot ouput Piergiorgio Beruto (1): powerpc/85xx: Fix wrong SVR value for MPC8567 and MPC8567E processors Roy Zang (2): fsl_esdhc: Add the workaround for erratum ESDHC135 (enable on P4080) fsl_esdhc: Add the workaround for erratum ESDHC136 (enable on P4080) MAINTAINERS
Re: [U-Boot] Is theer an IRC channel?
Hi Fabio, Le 17/01/2011 14:38, Fabio Giovagnini a écrit : Is there please an IRC channel for real time chat about problem with uboot? Not that I know of. Regards Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/1] ARM: */start.S: code cleanup
Hi, Albert, 2010/12/23 Jason Liu r64...@freescale.com: Remove the useless code from start.S Signed-off-by: Jason Liu r64...@freescale.com --- Changes for v2: -Address the comments from Albert to make changes to all ARM cpus with one single patch --- arch/arm/cpu/arm1136/start.S | 2 -- arch/arm/cpu/arm1176/start.S | 2 -- arch/arm/cpu/arm720t/start.S | 2 -- arch/arm/cpu/arm920t/start.S | 2 -- arch/arm/cpu/arm925t/start.S | 2 -- arch/arm/cpu/arm926ejs/start.S | 2 -- arch/arm/cpu/arm946es/start.S | 2 -- arch/arm/cpu/arm_intcm/start.S | 2 -- arch/arm/cpu/armv7/start.S | 2 -- arch/arm/cpu/ixp/start.S | 2 -- arch/arm/cpu/lh7a40x/start.S | 2 -- arch/arm/cpu/pxa/start.S | 2 -- arch/arm/cpu/s3c44b0/start.S | 2 -- arch/arm/cpu/sa1100/start.S | 2 -- 14 files changed, 0 insertions(+), 28 deletions(-) Any feedback about this patch? ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [STATUS] Merge window closed
Hi all, as you may be aware, the merge window ended last weekend. That means we're in review and bug fixing mode now. I'm sorry, but I have to admit tht I have a huge backlog - I did not even find a fraction of the time I'd have needed. I'll try to catch up now, but I need your help. Custodians: Please help me and take some of the load of sorting and processing open patches. An easy way to help me is to go through the open patches list in Patchwork. If you find patches that all into your field of activity, then please grab them, i. e. assign them to you. If you think I (or some other custodian) should handle a patch, assign it to me (or him). If you find a patch incorrectly assigned to you, don't complain - just reassign as appropriate. Oh, and every now and then check your todo list in Patchwork, and handle the patches on your list. Thanks in advance! Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de It is impractical for the standard to attempt to constrain the behavior of code that does not obey the constraints of the standard. - Doug Gwyn ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] .gitignore: ignore generated u-boot.imx
Dear Stefano Babic, In message 1293528207-5246-1-git-send-email-sba...@denx.de you wrote: Signed-off-by: Stefano Babic sba...@denx.de --- .gitignore |1 + 1 files changed, 1 insertions(+), 0 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Perl already has an IDE. It's called Unix. -- Tom Christiansen in 375bd...@cs.colorado.edu ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Is theer an IRC channel?
Kumar suggestied #u-boot on freenode Cheers On Monday 17 January 2011 20:43:47 Albert ARIBAUD wrote: Hi Fabio, Le 17/01/2011 14:38, Fabio Giovagnini a écrit : Is there please an IRC channel for real time chat about problem with uboot? Not that I know of. Regards Amicalement, -- Ing. Fabio Giovagnini Aurion s.r.l. P.I e C.F. 00885711200 skype: aurion.giovagnini Tel. +39.051.594.78.24 Fax. +39.051.082.14.49 Cell. +39.335.83.50.919 www.aurion-tech.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Is theer an IRC channel?
Dear Albert ARIBAUD, In message 4d349bf3.8000...@free.fr you wrote: Le 17/01/2011 14:38, Fabio Giovagnini a =E9crit : Is there please an IRC channel for real time chat about problem with uboo= t? Not that I know of. Oops? You don't know #u-boot @ freenode ? So that's the reason why you didn't show up there yet? Please consider yourself invited! Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Bugs are by far the largest and most successful class of entity, with nearly a million known species. In this res- pect they outnumber all the other known creatures about four to one. -- Professor Snope's Encyclopedia of Animal ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Uboot with at91sam9G20ek
Dear Chris Verges, dear Abhishek, I'm also trying to get U-Boot 2010.12 working on an AT91 board. I came across the following mailing list archive that may help. ... Please have a look at git://git.denx.de/u-boot-atmel.git, rework101229 branch, there is a working port for the at91sam9260ek which should also work for the at91sam9g20ek (if I am not mistaken). Best Regards, Reinhard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] nios2: reset cfi flash before reading env
Dear Thomas Chou, In message 4d252aab.6040...@wytron.com.tw you wrote: Instead of making fixed assumptions about flash type and it's properties here you should use generic routines from the CFI driver to do the reset. ... env_init() goes before flash_init() in board.c. So we don't know the flash type and cannot use those generic cfi routines based on flash_info. I followed this in cfi_flash.c void __flash_cmd_reset(flash_info_t *info) I recommend you have a second, closer look. Notice that flash_cmd_reset() comes with __attribute__ weak, because it does not work as a general solution - it may work here, but other code may be needed on other flash chips. Your code does not provide such an option. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de God is real, unless declared integer. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH V2] mpc83xx:fix pcie configuration space read/write
Dear Baidu Boy, In message 000401cbb527$cc725cb0$6401a8c0@LENOVOE5CA6843 you wrote: This patch fix a problem for the pcie enumeration when the mpc83xx pcie controller is connected with switch or we use both of the two pcie controller. Signed-off-by: Baidu Boy liucai@gmail.com --- Changes for V2: - code refine You submit this as V2, when it is atleast V4 of this patch? Please see http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions Do you think that code refine is a sufficient description of what you have changed compared to previous versions of this patch? It is not so for me, which is why I hereby NAK this patch. Sorry. Also, Scott already pointed out that SubmittingPatches lists the requirement of using your real name (sorry, no pseudonyms or anonymous contributions.) for the Signed-off-by line. Please confirm that Baidu Boy is your real name. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de The light at the end of the tunnel is usually a No Exit sign. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] spi: add new driver for OpenCores tiny_spi
Dear Thomas Chou, In message 1294626279-8601-1-git-send-email-tho...@wytron.com.tw you wrote: This patch adds support for OpenCores tiny_spi. http://opencores.org/project,tiny_spi Signed-off-by: Thomas Chou tho...@wytron.com.tw --- for u-boot v2, use const and clean up as Mike suggested. ... +void spi_cs_activate(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + unsigned int cs = slave-cs; + gpio_set_value(cs, tiny_spi-flg); + debug(%s: SPI_CS_GPIO:%x\n, __func__, gpio_get_value(cs)); +} Please separate declartations and code with a blank line. Please fix globally. ... +int spi_claim_bus(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + const struct tiny_spi_host *host = tiny_spi-host; + debug(%s: bus:%i cs:%i\n, __func__, slave-bus, slave-cs); + gpio_direction_output(slave-cs, !tiny_spi-flg); + writel(tiny_spi-mode, host-base + TINY_SPI_CONTROL); + writel(tiny_spi-baud, host-base + TINY_SPI_BAUD); Please do not use base + offset notation. Use a proper C struct instead. Please fix globally. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de A conservative is a man who believes that nothing should be done for the first time. - Alfred E. Wiggam ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v5 1/8] those files are jz4740 base files
Dear Xiangfu Liu, In message 1294589925-32639-2-git-send-email-xian...@openmobilefree.net you wrote: Signed-off-by: Xiangfu Liu xian...@openmobilefree.net Acked-by: Daniel z...@ingenic.cn Acked-by: ChangWei Jia cw...@ingenic.cn --- arch/mips/cpu/xburst/Makefile| 50 +++ arch/mips/cpu/xburst/config.mk | 33 ++ ... There is no description of the changes in this version of the patch. The notes in the introductory message are _not_ sufficient, please see http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions (bullet 2) ... + while (1) { + cache_op(Hit_Writeback_Inv_D, addr); + cache_op(Hit_Invalidate_I, addr); We do not allow CamelCase identifiers in U-Boot. Please fix globally. + while ( !__rtc_write_ready()) + ; No space after the paren; please fix globally. ... +int serial_init (void) +{ +#if !defined(CONFIG_NAND_U_BOOT) || defined(CONFIG_NAND_SPL) + volatile u8 *uart_fcr = (volatile u8 *)(CONFIG_SYS_UART_BASE + OFF_FCR); + volatile u8 *uart_lcr = (volatile u8 *)(CONFIG_SYS_UART_BASE + OFF_LCR); + volatile u8 *uart_ier = (volatile u8 *)(CONFIG_SYS_UART_BASE + OFF_IER); + volatile u8 *uart_sircr = (volatile u8 *)(CONFIG_SYS_UART_BASE + OFF_SIRCR); Please do not use base + offset notation. Declare a proper C struct instead. Please fix globally. + /* Disable port interrupts while changing hardware */ + *uart_ier = 0; And use proper I/O accessors to access device registers. Please fix globally. ... +static ulong timestamp; +static ulong lastdec; As already has been pointed out several times before: this will not work because BSS cannot be accessed before relocation. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de We don't have to protect the environment -- the Second Coming is at hand. - James Watt ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v5 2/8] this is jz4740 head file
Dear Xiangfu Liu, In message 1294589925-32639-3-git-send-email-xian...@openmobilefree.net you wrote: Signed-off-by: Xiangfu Liu xian...@openmobilefree.net Acked-by: Daniel z...@ingenic.cn Acked-by: ChangWei Jia cw...@ingenic.cn --- arch/mips/include/asm/jz4740.h | 1380 1 files changed, 1380 insertions(+), 0 deletions(-) create mode 100644 arch/mips/include/asm/jz4740.h Description of changes against previous version missing. ... +/* + * INTC (Interrupt Controller) + */ +#define INTC_ISR (INTC_BASE + 0x00) +#define INTC_IMR (INTC_BASE + 0x04) +#define INTC_IMSR(INTC_BASE + 0x08) +#define INTC_IMCR(INTC_BASE + 0x0c) +#define INTC_IPR (INTC_BASE + 0x10) NAK. Please use a C struct instead. ... +/* + * RTC + */ +#define RTC_RCR (RTC_BASE + 0x00) /* RTC Control Register */ +#define RTC_RSR (RTC_BASE + 0x04) /* RTC Second Register */ +#define RTC_RSAR (RTC_BASE + 0x08) /* RTC Second Alarm Register */ +#define RTC_RGR (RTC_BASE + 0x0c) /* RTC Regulator Register */ + +#define RTC_HCR (RTC_BASE + 0x20) /* Hibernate Control Register */ +#define RTC_HWFCR(RTC_BASE + 0x24) /* Hibernate Wakeup Filter Counter Reg */ +#define RTC_HRCR (RTC_BASE + 0x28) /* Hibernate Reset Counter Register */ +#define RTC_HWCR (RTC_BASE + 0x2c) /* Hibernate Wakeup Control Register */ +#define RTC_HWRSR(RTC_BASE + 0x30) /* Hibernate Wakeup Status Register */ +#define RTC_HSPR (RTC_BASE + 0x34) /* Hibernate Scratch Pattern Register */ Ditto. Use C structs. I stop my review here. There are too many global issues that need to be fixed first. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de It is dangerous to be sincere unless you are also stupid. - George Bernard Shaw ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] S5P: timer: replace bss variable by gd
Dear Minkyu Kang, In message aanlktikozfrd9sylg4v3aoqrfed9bm0jpqqoi+qzy...@mail.gmail.com you wrote: I object against using variables wich have somewhat self-explanatory names for different purposes. Yes, I know. If so, do I have to add new variables at global_data? Maybe that is not good way too. How you think? I asked this several times before: ther eis currently ongoing efforts for aunified clock handling for ARM in the Linux kernel. Maybe we can benefit from this in any way? One more question, what is the purpose of tbl and tbu? tbl (time base, lower part) and tbu (time base, upper part) are the software representation of two 32 bit hardware registers available in Power Architecture systems, where they form a free-runnign 64 bit counter that starts automatically as soon as the CPU comes out of reeset. There have been attempts to re-write the (sometimes somewhat obscure) timer implementations on ARM based on such a model. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Intel told us the Pentium would have RISK features... ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/8] armv7: cache maintenance operations for armv7
Dear Aneesh V, In message 4d2a9164.5020...@ti.com you wrote: Unless we see a specific example which uses this feature, we should not add provisions that make the code more complicated than needed. Agree. But do you think the pointer based approach makes it overly complex? Not overly complex, but more complex than needed. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de A consultant is a person who borrows your watch, tells you what time it is, pockets the watch, and sends you a bill for it. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Booting linux 2.6.37+ on rsk7203 eva board
These are my news Hi all, I have this condition now when I do make zImage and make bootstrap (i link zImage at 0x0004 after a simple bootloader) the system work and zImage decompress itself at 0x0C10 when I make make uiImage of the same kernel and I execute it usinf u-bbot it doesn't work. There could be 2 reasons 1) address problem bootargs problem, but my kernel is built using BOOTCMD_OVERWRITE option. Any ideas? Thanks On Monday 17 January 2011 19:24:14 Fabio Giovagnini wrote: Ok, rigth. I'mm newbe of u-bbot and I'd loke to contribute. What could be the usefull info for you? Thanks in advance On Monday 17 January 2011 19:21:27 Wolfgang Denk wrote: Dear Fabio Giovagnini, In message 201101171908.02343.fabio.giovagn...@aurion-tech.com you wrote: Hi All this is what I do: 1) I run u-Boot getting this output: U-Boot 2010.12-09860-g9cb5761-dirty (Jan 17 2011 - 18:05:12) CPU: SH2A BOARD: MLCD7203 This appears to be an out-of-tree port, i. e. code which is completely unknown to us. Any ideas about why the kernelò do not send out anything? We don't know your code, we don't know which exact commands you're running, we don't see your environment, we don't know nothing. How could we then come up with helpful comments? Best regards, Wolfgang Denk -- Ing. Fabio Giovagnini Aurion s.r.l. P.I e C.F. 00885711200 skype: aurion.giovagnini Tel. +39.051.594.78.24 Fax. +39.051.082.14.49 Cell. +39.335.83.50.919 www.aurion-tech.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 6/8] armv7: adapt omap4 to the new cache maintenance framework
Dear Aneesh V, In message 4d2b18c6.2010...@ti.com you wrote: A handful of board config files are using it, but in actual code it's only referenced by armv7 code. Shall I change them all to CONFIG_SYS_L2_OFF Yes, please do (in a separate patch). Thanks. I shall add documentation. Thanks. Sure. I will take care in future. BTW, that was not caught by checkpatch.pl. Is there anyway to catch such errors using some tool? I use just visual inspection, and vi when in doubt. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Sorry, but my karma just ran over your dogma. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/8] armv7: adapt omap3 to the new cache maintenance framework
Dear Aneesh V, In message 4d2b1a90.9030...@ti.com you wrote: +struct __attribute__ ((__packed__)) emu_hal_params { + u32 num_params; + u32 param1; +}; Why exactly do we need the __attribute__ ((__packed__)) here? Because a pointer to it has to be passed to ROM code and ROM code wouldn't expect any padding. But padding would only be needed if there were any alignment requirements requiring it. There are none here. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de The combination of a number of things to make existence worthwhile. Yes, the philosophy of 'none,' meaning 'all.' -- Spock and Lincoln, The Savage Curtain, stardate 5906.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] Small coding style fix in lib/asm-offsets.c
Dear Stefan Roese, In message 1294660080-30928-1-git-send-email...@denx.de you wrote: Signed-off-by: Stefan Roese s...@denx.de --- lib/asm-offsets.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Of all possible committee reactions to any given agenda item, the reaction that will occur is the one which will liberate the greatest amount of hot air.-- Thomas L. Martin ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v3] ARM: Avoid compiler optimization for usages of readb, writeb and friends.
Dear Dirk Behme, In message 4d2b3036.4010...@googlemail.com you wrote: The question I was thinking about with my patch was what's Right Thing? ;) The Right Thing i not to make specific assumptions how the compiler might handle volatile pointers. It's my understanding that we don't fix read*() and write*() because they are broken. We touch them to work around a broken tool chain. No. Please re-read volatile-considered-harmful.txt in the linux/Documentation directory: accessing I/O memory directly through pointers is frowned upon and does not work on all architectures. Those accessors are written to prevent unwanted optimization. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de It is dangerous to be sincere unless you are also stupid. - George Bernard Shaw ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] ppc4xx: Support dual link OSD
Dear Dirk Eibach, In message 1294662805-11704-3-git-send-email-eib...@gdsys.de you wrote: #define BASE_WIDTH 32 @@ -38,12 +44,18 @@ enum { REG_CONTROL = 0x0010, REG_MPC3W_CONTROL = 0x001a, + REG_EXT_INTERRUPT = 0x001c, + REG_EXT_INTERRUPT_ENABLE = 0x001e, + REG_IIC_WRITE_MAILBOX = 0x0030, + REG_IIC_WRITE_MAILBOX_EXT = 0x0032, + REG_IIC_READ_MAILBOX = 0x0034, + REG_IIC_READ_MAILBOX_EXT = 0x0036, REG_VIDEOCONTROL = 0x0042, - REG_OSDVERSION = 0x0100, - REG_OSDFEATURES = 0x0102, - REG_OSDCONTROL = 0x0104, - REG_XY_SIZE = 0x0106, - REG_VIDEOMEM = 0x0800, + REG_OSDVERSION = CONFIG_SYS_FPGA_OSD_BASE + 0x, + REG_OSDFEATURES = CONFIG_SYS_FPGA_OSD_BASE + 0x0002, + REG_OSDCONTROL = CONFIG_SYS_FPGA_OSD_BASE + 0x0004, + REG_XY_SIZE = CONFIG_SYS_FPGA_OSD_BASE + 0x0006, + REG_VIDEOMEM = CONFIG_SYS_FPGA_OSD_MEM, This has a certain smell to me - it smells like device accesses based on a base + offset notation, which is not waht we want to see. Why don't you use proper C structs instead to describe your devices? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Those who will be able to conquer software will be able to conquer the world. - Tadahiro Sekimoto, president, NEC Corp. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 4/4] powerpc: Add LINK_OFF calls in early C-code.
Dear Joakim Tjernlund, In message of6a573b77.9119deb0-onc1257814.0069a574-c1257814.00753...@transmode.se you wrote: It would break only if link address != load address. That is, if you want to use my new CONFIG_SYS_TRUE_PIC feature and be able to load u-boot at any address regardless of link address you would have to add LINK_OFF calls into print_83xx_arb_event() too if you want to use it. Doesn't this add a requirement for future generic pre-relocation code to comply with, to avoid breaking your board? Yes, but I don't mind if my board breaks from time to time. After all it isn't in u-boot so I have had to deal with quite a few breakages already. It is my hope this new feature will spread to other boards as time pass. You mean you ask to add code that is not only highly fragile but even known to be broken for other boards than yours, and the only board that uses the feature and has been tested with it isn't in u-boot ? Sorry, but it makes no sense to me to add all this. NAK. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de I am pleased to see that we have differences. May we together become greater than the sum of both of us. -- Surak of Vulcan, The Savage Curtain, stardate 5906.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [RFC PATCH] ARM: print gcc version
Dear Alexander Holler, In message 4d2bb9ac.1040...@ahsoftware.de you wrote: On the other hand, that would require changes to many files, and the gcc __VERSION__ string is pretty long on some systems, which may cause additional problems. Hmm, here it includes only 4.5.2, the plain version and not the vendor version. Therfore I was happy with that, because the vendor version would be too long to display at startup. Well, what I see is for example 4.5.1 20100924 (Red Hat 4.5.1-4) with the native GCC on Fedora 14; some other tool chains provide similar long results. I also doubt that we need this information with each boot message (where it just slows down booting). Hmm, I haven't counted the time needed to display additional 9 characters, but displaying the version at startup is handy for screen It's about 1 millisecond at 115 kbps, or 10 milliseconds at 9600 ;-) dumps (e.g. stuff pasted by people with problems). I think we're already printing way too muc information- and still people don't include even the most vital information like which version of U-Boot they are running. Would it not be sufficient to add this to the code of do_version() so we can get this information when we want it, i. e. when running the version command? I would like that. But as you said, my intention was really only because of the current problems with gcc for arm. So will you prepare a patch? [TIA!] Besides that, I would found it handy to see the used compiler for u-boot's where I don't have the source by hand, e.g. those already found on boards. But that might be because I'm curious by default. ;) Well, eventually we should print more than __VERSION__, then. Becasue if you see 4.4.1 - what does it tell you? You don't see that this is the Sourcery G++ Lite 2010q1-202 version of GCC, or that 4.5.0 20100729 (prerelease) is a specific Poky release? Anyway, I would found it a nice feature, at startup or when running the version command, but both aren't a must. I think it would be a really useful extension to the version command. Looking forwad to seeing your patch. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de A conservative is a man who believes that nothing should be done for the first time. - Alfred E. Wiggam ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] mpq101: initial support for Mercury Computer Systems MPQ101 board
Dear Alex Dubov, In message 369392.26813...@web37607.mail.mud.yahoo.com you wrote: Could you please be a bit more specific, i. e. like quoting file names and line numbers? These were produced by grep -nr ddr\-\ board/ | grep -v out_be I can attach a listing, but it should be quicker this way. Up until now it seemed a rather common practice to initialize ddr controllers directly, bypassing io accessors. Indeed. Even after the recent removal of some boards this list remains: board/freescale/mpc8349itx/mpc8349itx.c board/freescale/mpc8536ds/mpc8536ds.c board/freescale/mpc8540ads/mpc8540ads.c board/freescale/mpc8560ads/mpc8560ads.c board/freescale/mpc8569mds/mpc8569mds.c board/freescale/mpc8572ds/mpc8572ds.c board/freescale/mpc8610hpcd/mpc8610hpcd.c board/freescale/mpc8641hpcn/mpc8641hpcn.c board/freescale/p2020ds/p2020ds.c board/sbc8560/sbc8560.c board/sbc8641d/sbc8641d.c board/socrates/sdram.c board/tqc/tqm85xx/sdram.c Kumar, Andy - do you have any plans to fix this for the freescale boards? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de I've got a bad feeling about this. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] Top config.mk: add include/config.mk
Seems to that the top level config.mk should include the auto generated include/config.mk so that all Makefile's pickup those definitions. Signed-off-by: Joakim Tjernlund joakim.tjernl...@transmode.se --- config.mk |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/config.mk b/config.mk index bcda776..c7534fb 100644 --- a/config.mk +++ b/config.mk @@ -117,6 +117,7 @@ RANLIB = $(CROSS_COMPILE)RANLIB # Load generated board configuration sinclude $(OBJTREE)/include/autoconf.mk +sinclude $(OBJTREE)/include/config.mk # Some architecture config.mk files need to know what CPUDIR is set to, # so calculate CPUDIR before including ARCH/SOC/CPU config.mk files. -- 1.7.3.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] powerpc/85xx: Move RESET_VECTOR_ADDRESS into config.h
Dear Kumar Gala, In message c1777a39-1166-4b3f-8484-8b193d0fc...@kernel.crashing.org you wrote: On Jan 12, 2011, at 3:27 AM, Wolfgang Denk wrote: Dear Kumar Gala, In message 1294824140-22519-1-git-send-email-ga...@kernel.crashing.org you wrote: Rather than defining it config.mk we can set it in config.h and remove config.mk from several boards that don't need it. We mimic what 4xx does and introduce CONFIG_RESET_VECTOR_ADDRESS for config.h to set. Signed-off-by: Kumar Gala ga...@kernel.crashing.org What's this? A resubmit of the previous posting? Or a new version? With any changes? Which changes? SOrry, for got the 'v2' when I posted. Its a new version that removes the extra ifdef checking around RESET_VECTOR_ADDRESS in u-boot.lds But it still has the #ifndef RESET_VECTOR_ADDRESS part where you said it was not needed ? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Prepare for tomorrow -- get ready. -- Edith Keeler, The City On the Edge of Forever, stardate unknown ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] Hash problem...
I've stumbled across a problem in U-boot 2010.12's hashing algorithm. In my particular case the size of the has table is 347 (due to my default environment being 2258 bytes long as reported on entry to himport_r), and I have in my envrironment the variable ramdiskimage as well as preboot; part of my startup is to do some housecleaning then delete preboot so its only done once. But when I load u-boot with an empty environemnt environment, I see the following problem: ... = printenv ramdiskimage=rootfs.ext2.gz.uboot = echo $ramdiskimage = setenv ramdiskimage=rootfs.ext2.gz.uboot = printenv ... ramdiskimage=rootfs.ext2.gz.uboot ramdiskimage=rootfs.ext2.gz.uboot ... After spending an entire day digging into the hash using GDB/BDI on my ARM board, I've findally figured out that the hash key of ramdiskimage and preboot are the same modulus 347, and this is problematic because on the initial hash import, preboot is placed into the hash first (at idx 190 since the table is sorted), and then ramdiskimage collides with preboot causing the 2nd probe (at idx 191) to occur which works fine. Unfortunately as part of the housecleaning, preboot is deleted and the environemnt saved. The delete of preboot removes entry at idx 190 and the next lookup of ramdiskimage sees that idx 190 is empty and believes that the ramdiskimage is not in the table ionstead of rehashing to find it at idx 191. The hash delete code is in error; instead of just removing the deleted key, it should instead allocate a new hashtable, hash all the keys into the new table except for the deleted key and then reclaim the old table (and deleted key). -- Peter Barada peter.bar...@logicpd.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v4 7/8] imximage: Add MX53 boot image support
Dear Jason Liu, In message 1294129662-680-7-git-send-email-r64...@freescale.com you wrote: This patch add the MX53 boot image support. This patch has been tested on Freescale MX53EVK board and MX51EVK board. Signed-off-by: Jason Liu r64...@freescale.com ... static table_entry_t imximage_cmds[] = { + {CMD_IMAGE_VERSION, IMAGE_VERSION,image version, }, {CMD_BOOT_FROM, BOOT_FROM,boot command, }, {CMD_DATA, DATA, Reg Write Data, }, {-1,, , }, Can we please keep the table sorted? {-1,, Invalid, }, }; +/* + * IMXIMAGE version definition for i.MX chips + */ +static table_entry_t imximage_versions[] = { + {IMXIMAGE_V1, , (i.MX25/35/51 compatible), }, + {IMXIMAGE_V2, , (i.MX53 compatible), }, + {-1,, (Invalid), }, +}; static struct imx_header imximage_header; +static uint32_t imximage_version; + +static set_dcd_val_t set_dcd_val; +static set_dcd_rst_t set_dcd_rst; +static set_imx_hdr_t set_imx_hdr; static uint32_t get_cfg_value(char *token, char *name, int linenr) { @@ -71,58 +85,264 @@ static uint32_t get_cfg_value(char *token, char *name, int linenr) return value; + /* Try to detect V1 */ + if (fhdr_v1-app_code_barker == APP_CODE_BARKER + imx_hdr-header.hdr_v1.dcd_table.preamble.barker == DCD_BARKER) + + return IMXIMAGE_V1; This needs braces. Please fix globally. +static void set_dcd_rst_v1(struct imx_header *imxhdr, uint32_t dcd_len, + char *name, int lineno) Could you please add a comment what the somewhat cryptic name set_dcd_rst_v1 is supposed to mean? Similar for the other functions like set_dcd_rst_v2() etc.? + if (dcd_len MAX_HW_CFG_SIZE_V1) { + fprintf(stderr, Error: %s[%d] - + DCD table exceeds maximum size(%d)\n, + name, lineno, MAX_HW_CFG_SIZE_V1); + } + + dcd_v1-preamble.barker = DCD_BARKER; + dcd_v1-preamble.length = dcd_len * sizeof(dcd_type_addr_data_t); +} It seems these functions can run into error situations - yet they are of type void, i. e. they don't return any information about such errors to their callers. In the end, you cannot test the result oif running the command to find out if it had worked or not. This must be fixed (globally). + case CMD_IMAGE_VERSION: + imximage_version = + get_cfg_value(token, + name, lineno); + if (cmd_ver_first == 0) { + fprintf(stderr, + Error: %s[%d] - + IMAGE_VERSION command + need appear the first + before other valid + command in the file\n, + name, lineno); + exit(EXIT_FAILURE); + } Your nesting is too deep. Consider reorganizing the code. Stefano, Albert - I apologize for the late review. Please don't pull this into mainline as is. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v4 5/8] mxc_i2c: add support for MX53 processor
Dear Stefano Babic, In message 4d2d78b1.9090...@denx.de you wrote: On 01/04/2011 09:27 AM, Jason Liu wrote: This patch add I2C support for Freescale MX53 processor Signed-off-by: Jason Liu r64...@freescale.com --- Changes for v2: -address the comments of Heiko, add #if defined(CONFIG_MX31) to avoid break MX31 build. Move CONFIG_HARD_I2C to the top of the file and fix the error message from: #error define CONFIG_SYS_I2C_PORT to use the I2C driver to #error define CONFIG_SYS_I2C_MXxx_PORTx to use the I2C driver Changes for v3: -address the comments from Albert, change #ifdef to #if defined() for the sake of homogeneity --- drivers/i2c/mxc_i2c.c | 21 ++--- 1 files changed, 18 insertions(+), 3 deletions(-) Applied to u-boot-imx, thanks. Please wait for the ACK from the I2C custodian first. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de There are three ways to get something done: (1) Do it yourself. (2) Hire someone to do it for you. (3) Forbid your kids to do it. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] p1022ds: allow for board-specific ngPIXIS functions
The ngPIXIS is an FPGA used on the reference boards of most Freescale PowerPC SOCs. Although programming the ngPIXIS is mostly standard on all boards that have it, the P1022DS is unique in that the ngPIXIS needs to be programmed in indirect mode whenever the video display (DIU) is active. To support indirect mode, and to make it easier to support other quirks on future reference boards, the low-level ngPIXIS functions are all marked as weak, so that board-specific code can override any of them. We take advantage of this feature on the P1022DS, so that we can properly reset the board when the DIU is active. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/ngpixis.c | 54 board/freescale/common/ngpixis.h |6 +++ board/freescale/p1022ds/diu.c| 64 - 3 files changed, 109 insertions(+), 15 deletions(-) diff --git a/board/freescale/common/ngpixis.c b/board/freescale/common/ngpixis.c index a135fbe..a432e33 100644 --- a/board/freescale/common/ngpixis.c +++ b/board/freescale/common/ngpixis.c @@ -35,61 +35,89 @@ #include common.h #include command.h -#include watchdog.h -#include asm/cache.h #include asm/io.h #include ngpixis.h +static u8 __pixis_read(unsigned int reg) +{ + void *p = (void *)PIXIS_BASE; + + return in_8(p + reg); +} +u8 pixis_read(unsigned int reg) __attribute__((weak, alias(__pixis_read))); + +static void __pixis_write(unsigned int reg, u8 value) +{ + void *p = (void *)PIXIS_BASE; + + out_8(p + reg, value); +} +void pixis_write(unsigned int reg, u8 value) + __attribute__((weak, alias(__pixis_write))); + /* * Reset the board. This ignores the ENx registers. */ -void pixis_reset(void) +void __pixis_reset(void) { - out_8(pixis-rst, 0); + PIXIS_WRITE(rst, 0); while (1); } +void pixis_reset(void) __attribute__((weak, alias(__pixis_reset))); /* * Reset the board. Like pixis_reset(), but it honors the ENx registers. */ -void pixis_bank_reset(void) +void __pixis_bank_reset(void) { - out_8(pixis-vctl, 0); - out_8(pixis-vctl, 1); + PIXIS_WRITE(vctl, 0); + PIXIS_WRITE(vctl, 1); while (1); } +void pixis_bank_reset(void) __attribute__((weak, alias(__pixis_bank_reset))); /** * Set the boot bank to the power-on default bank */ -void clear_altbank(void) +void __clear_altbank(void) { + u8 reg; + /* Tell the ngPIXIS to use this the bits in the physical switch for the * boot bank value, instead of the SWx register. We need to be careful * only to set the bits in SWx that correspond to the boot bank. */ - clrbits_8(PIXIS_EN(PIXIS_LBMAP_SWITCH), PIXIS_LBMAP_MASK); + reg = PIXIS_READ(s[PIXIS_LBMAP_SWITCH - 1].en); + reg = ~PIXIS_LBMAP_MASK; + PIXIS_WRITE(s[PIXIS_LBMAP_SWITCH - 1].en, reg); } +void clear_altbank(void) __attribute__((weak, alias(__clear_altbank))); /** * Set the boot bank to the alternate bank */ -void set_altbank(void) +void __set_altbank(void) { + u8 reg; + /* Program the alternate bank number into the SWx register. */ - clrsetbits_8(PIXIS_SW(PIXIS_LBMAP_SWITCH), PIXIS_LBMAP_MASK, -PIXIS_LBMAP_ALTBANK); + reg = PIXIS_READ(s[PIXIS_LBMAP_SWITCH - 1].sw); + reg = (reg ~PIXIS_LBMAP_MASK) | PIXIS_LBMAP_ALTBANK; + PIXIS_WRITE(s[PIXIS_LBMAP_SWITCH - 1].sw, reg); /* Tell the ngPIXIS to use this the bits in the SWx register for the * boot bank value, instead of the physical switch. We need to be * careful only to set the bits in SWx that correspond to the boot bank. */ - setbits_8(PIXIS_EN(PIXIS_LBMAP_SWITCH), PIXIS_LBMAP_MASK); + reg = PIXIS_READ(s[PIXIS_LBMAP_SWITCH - 1].en); + reg |= PIXIS_LBMAP_MASK; + PIXIS_WRITE(s[PIXIS_LBMAP_SWITCH - 1].en, reg); } +void set_altbank(void) __attribute__((weak, alias(__set_altbank))); int pixis_reset_cmd(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) diff --git a/board/freescale/common/ngpixis.h b/board/freescale/common/ngpixis.h index 089408b..8a7ddfa 100644 --- a/board/freescale/common/ngpixis.h +++ b/board/freescale/common/ngpixis.h @@ -55,3 +55,9 @@ typedef struct ngpixis { /* The PIXIS EN register that corresponds to board switch X, where x = 1 */ #define PIXIS_EN(x)(pixis-s[(x) - 1].en) + +u8 pixis_read(unsigned int reg); +void pixis_write(unsigned int reg, u8 value); + +#define PIXIS_READ(reg) pixis_read(offsetof(ngpixis_t, reg)) +#define PIXIS_WRITE(reg, value) pixis_write(offsetof(ngpixis_t, reg), value) diff --git a/board/freescale/p1022ds/diu.c b/board/freescale/p1022ds/diu.c index 8f5305c..61679ac 100644 --- a/board/freescale/p1022ds/diu.c +++ b/board/freescale/p1022ds/diu.c @@ -139,8 +139,6 @@ int platform_diu_init(unsigned int *xres, unsigned int *yres) return fsl_diu_init(*xres,
Re: [U-Boot] [PATCH 4/4] powerpc: Add LINK_OFF calls in early C-code.
Wolfgang Denk w...@denx.de wrote on 2011/01/17 23:11:59: Dear Joakim Tjernlund, In message of6a573b77.9119deb0-onc1257814.0069a574-c1257814.00753...@transmode.se you wrote: It would break only if link address != load address. That is, if you want to use my new CONFIG_SYS_TRUE_PIC feature and be able to load u-boot at any address regardless of link address you would have to add LINK_OFF calls into print_83xx_arb_event() too if you want to use it. Doesn't this add a requirement for future generic pre-relocation code to comply with, to avoid breaking your board? Yes, but I don't mind if my board breaks from time to time. After all it isn't in u-boot so I have had to deal with quite a few breakages already. It is my hope this new feature will spread to other boards as time pass. You mean you ask to add code that is not only highly fragile but even known to be broken for other boards than yours, and the only board that uses the feature and has been tested with it isn't in u-boot ? No other board is broken. This new function is neutral to other boards. I am merely saying as my board is the first user of this new feature I expect minor breakage of my board from time to time when someone adds a new function that needs LINK_OFF to work on my board but forgets to actually add the LINK_OFF call. Once more boards uses my new feature this problem goes away. Wolfgang, once you indicated you were interested in such feature as I have added but my first impl. had LINK_OFF calls all over the place, still you were tempted to add the feature. Now that I have reduced the LINK_OFF calls to a minimum you suddenly want to reject it even though 95% of the LINK_OFF calls are gone. Why this change of heart? Jocke ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/2] armv7: add support for s5pc210 universal board
Dear Minkyu Kang, In message 4d2ed0e1.7090...@samsung.com you wrote: This patch adds support for Samsung s5pc210 universal board Signed-off-by: Minkyu Kang mk7.k...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com ... diff --git a/MAKEALL b/MAKEALL index a732e6a..a5b265f 100755 --- a/MAKEALL +++ b/MAKEALL @@ -434,6 +434,7 @@ LIST_ARMV7= \ omap4_panda \ omap4_sdp4430 \ s5p_goni\ + s5pc210_universal \ smdkc100\ NO - boards don't get added to MAKEALL any more. MAKEALL automatically picks them up from boards.cfg. Please remove. ... --- /dev/null +++ b/board/samsung/universal_c210/config.mk ... +CONFIG_SYS_TEXT_BASE = 0x4480 No. Please remove config.mk and define CONFIG_SYS_TEXT_BASE in your board config file instead. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de If I don't document something, it's usually either for a good reason, or a bad reason. In this case it's a good reason. :-) - Larry Wall in 1992jan17.005405.16...@netlabs.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] ppc, 8xx: remove obsolete km8xx boards from keymile
Dear Holger Brunck, In message 4d2f0071.5070...@keymile.com you wrote: The MPC852 based mgsuvd and kmsupx4 boards from keymile were initially ported but later on not developed further. So the respective files were removed to avoid unneeded merging and maintenance. Signed-off-by: Holger Brunck holger.bru...@keymile.com --- Changes for v2: - fix typos in commit message - remove boards from MAINTAINERS file Sorry, does not apply: Applying: ppc, 8xx: remove obsolete km8xx boards from keymile fatal: corrupt patch at line 814 Repository lacks necessary blobs to fall back on 3-way merge. Cannot fall back to three-way merge. Patch failed at 0001 ppc, 8xx: remove obsolete km8xx boards from keymile Please fix and resubmit. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de I'm what passes for a Unix guru in my office. This is a frightening concept. - Lee Ann Goldstein, in 3k55ba$c...@butch.lmsc.lockheed.com ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2 v2] BLOCK: Add freescale IMX51 PATA driver
Dear stefano babic, In message 4d30e2e4.1070...@denx.de you wrote: Am 13.01.2011 18:27, schrieb Marek Vasut: Signed-off-by: Marek Vasut marek.va...@gmail.com --- v2: Fixed line-too-long bugs as proposed by Stefano drivers/block/Makefile |1 + drivers/block/mxc_ata.c | 149 +++ 2 files changed, 150 insertions(+), 0 deletions(-) create mode 100644 drivers/block/mxc_ata.c Acked-by: Stefano Babic sba...@denx.de Acked-by: Wolfgang Denk w...@denx.de Stefano, please pull through IMX tree. Thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Star Trek Lives! ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 0/4 V2] Add basic NVIDIA Tegra2 SoC support
Dear Tom Warren, In message AANLkTi=f-14jnam_dy5c1sgtt0nmqm-pnok6gdg4d...@mail.gmail.com you wrote: 1) IO access functions - I pre-reviewed my patch series with Wolfgang (to hopefully catch any blatant errors and smooth the process) and he indicated that C structs and I/O accessor funcs or macros were preferred to my base+offset original code. I don't think I said preferred. I apologize if I really should have been unclear. Fact is, the use of I/O accessors is mandatory for any new code going in. Since the ARM is 32-bit, and all of our registers are I/O mapped, it made sense just to cast the necessary HW mem-mapped regs as volatile structs and access the members directly. Works well, is easy to read understand, etc. Let me know (with examples, if possible) how I can make it better. It does NOT make sense. Please read the linux/Documentation/volatile-considered-harmful.txt document, and see recent dicussion in the ARM: Avoid compiler optimization for usages of readb, writeb and friends. thread about what happens when you use just volatile pointers. 2) Compiling out support for UARTA or UARTD - didn't seem necessary - size isn't an issue at this point with Tegra2 U-Boot, Maybe you don't care, but we do. Please see http://www.denx.de/wiki/U-Boot/DesignPrinciples, item 5 I was going to respond to your review w/a direct, inline reply, but I thought it better to get the V2 patch out there before the weekend (we're off for MLK, as well). I'm under some pressure to get a baseline Tegra2 patchset in before the merge window closes. I'll be sure to respond to each issue directly on the list in the future, though. Please also make sure to maintain a changelog of in each of your patches (see second bullet in http://www.denx.de/wiki/view/U-Boot/Patches#Sending_updated_patch_versions). Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de IMPORTANT NOTICE TO PURCHASERS: The Entire Physical Universe, Inclu- ding This Product, May One Day Collapse Back into an Infinitesimally Small Space. Should Another Universe Subsequently Re-emerge, the Existence of This Product in That Universe Cannot Be Guaranteed. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 0/4 V2] Add basic NVIDIA Tegra2 SoC support
Dear Tom Warren, In message aanlktikcq_0cnfrj_5s2zshcj-2qyvi24yeh0dcqe...@mail.gmail.com you wrote: So instead of, say uart-lcr = 0, you'd prefer writel(0, uart-lcr), where writel = __arch_putl(v, a) = (*(volatile unsigned int *)(a) = (v))? Please see the thread ARM: Avoid compiler optimization for usages of readb, writeb and friends. Is that different enough from 'uart-lcr = 0' to warrant the change? Yes, it is - in recent (i. e. fixed) code. Does it add some HW barriers or forced read-before-write that the 'volatile' struct doesn't? Yes, it does. See above. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Conquest is easy. Control is not. -- Kirk, Mirror, Mirror, stardate unknown ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] p1022ds: allow for board-specific ngPIXIS functions
Dear Timur Tabi, In message 1295304993-19369-1-git-send-email-ti...@freescale.com you wrote: The ngPIXIS is an FPGA used on the reference boards of most Freescale PowerPC SOCs. Although programming the ngPIXIS is mostly standard on all boards that have it, the P1022DS is unique in that the ngPIXIS needs to be programmed in indirect mode whenever the video display (DIU) is active. To support indirect mode, and to make it easier to support other quirks on future reference boards, the low-level ngPIXIS functions are all marked as weak, so that board-specific code can override any of them. We take advantage of this feature on the P1022DS, so that we can properly reset the board when the DIU is active. Signed-off-by: Timur Tabi ti...@freescale.com --- board/freescale/common/ngpixis.c | 54 board/freescale/common/ngpixis.h |6 +++ board/freescale/p1022ds/diu.c| 64 - 3 files changed, 109 insertions(+), 15 deletions(-) Just for the record: the merge window is closed, so this is next stuff. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de How much net work could a network work, if a network could net work? ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] powerpc/85xx: Move RESET_VECTOR_ADDRESS into config.h
On Jan 17, 2011, at 4:46 PM, Wolfgang Denk wrote: Dear Kumar Gala, In message c1777a39-1166-4b3f-8484-8b193d0fc...@kernel.crashing.org you wrote: On Jan 12, 2011, at 3:27 AM, Wolfgang Denk wrote: Dear Kumar Gala, In message 1294824140-22519-1-git-send-email-ga...@kernel.crashing.org you wrote: Rather than defining it config.mk we can set it in config.h and remove config.mk from several boards that don't need it. We mimic what 4xx does and introduce CONFIG_RESET_VECTOR_ADDRESS for config.h to set. Signed-off-by: Kumar Gala ga...@kernel.crashing.org What's this? A resubmit of the previous posting? Or a new version? With any changes? Which changes? SOrry, for got the 'v2' when I posted. Its a new version that removes the extra ifdef checking around RESET_VECTOR_ADDRESS in u-boot.lds But it still has the #ifndef RESET_VECTOR_ADDRESS part where you said it was not needed ? Best regards, Wolfgang Denk -- Most have posted the wrong version. - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/2] powerpc/8xxx: Add new hwconfig APIs to address early parsing used by DDR init
On Jan 9, 2011, at 2:29 PM, Kumar Gala wrote: There are several users of the hwconfig APIs (8xxx DDR) before we have the environment properly setup. This causes issues because of the numerous ways the environment might be accessed because of the non-volatile memory it might be stored in. Additionally the access might be so early that memory isn't even properly setup for us. Towards resolving these issues we provide versions of all the hwconfig APIs that can be passed in a buffer to parse and leave it to the caller to determine how to allocate and populate the buffer. We use the _f naming convention for these new APIs even though they are perfectly useable after relocation and the environment being ready. We also now warn if the non-f APIs are called before the environment is ready to allow users to address the issues. Finally, we convert the 8xxx DDR code to utilize the new APIs to hopefully address the issue once and for all. We have the 8xxx DDR code create a buffer on the stack and populate it via getenv_f(). Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- * Add a comment about the buffer size arch/powerpc/cpu/mpc8xxx/ddr/options.c | 78 ++--- common/hwconfig.c | 86 +--- include/hwconfig.h | 68 +++-- 3 files changed, 155 insertions(+), 77 deletions(-) Wolfgang, Reminder to take a look at this patch. Its a precursor to York's DDR cleanup patches. - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 4/4] powerpc: Add LINK_OFF calls in early C-code.
Dear Joakim Tjernlund, In message of71b40f2e.5c935ba8-onc125781b.007cb9e6-c125781b.007e5...@transmode.se you wrote: No other board is broken. This new function is neutral to other boards. Well, I see this differntly. Wolfgang, once you indicated you were interested in such feature as I have added but my first impl. had LINK_OFF calls all over the place, still you were tempted to add the feature. Now that I have reduced the LINK_OFF calls to a minimum you suddenly want to reject it even though 95% of the LINK_OFF calls are gone. Why this change of heart? Yes, I am definitely interested in this feature. But I still consider the impact to make this really working to heavy (and by working I mean working for all boards, out of the box, without having to go through iterations of breakage to find out where else a LINK_OFF needs to be added). In the current form, your simplification results just from the fact that you just added the bare minimum needed for your board. If - as you hope - more and more boards would use this, more and more of these now omitted LINK_OFFs would have to be added again. The result would be exactly the same mess as your original patch - it doesn't small any better when you try to feed it to me in small bites. And frankly, adding this stuff for an out-of-tree port is, um..., sorry, but I don't find polite words atm. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de An Elephant is a mouse with an Operating System. - Knuth ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v3] powerpc/85xx: Move RESET_VECTOR_ADDRESS into config.h
Rather than defining it config.mk we can set it in config.h and remove config.mk from several boards that don't need it. We mimic what 4xx does and introduce CONFIG_RESET_VECTOR_ADDRESS for config.h to set. Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/u-boot.lds |8 ++-- board/freescale/corenet_ds/config.mk | 27 --- board/freescale/mpc8536ds/config.mk | 14 +- board/freescale/mpc8572ds/config.mk |2 -- board/freescale/p1022ds/config.mk| 10 -- board/freescale/p1_p2_rdb/config.mk | 12 board/freescale/p2020ds/config.mk| 26 -- include/configs/MPC8536DS.h |6 ++ include/configs/MPC8572DS.h |4 include/configs/P1022DS.h|4 include/configs/P1_P2_RDB.h |6 ++ include/configs/P2020DS.h|4 include/configs/corenet_ds.h |4 13 files changed, 35 insertions(+), 92 deletions(-) delete mode 100644 board/freescale/corenet_ds/config.mk delete mode 100644 board/freescale/p1022ds/config.mk delete mode 100644 board/freescale/p2020ds/config.mk diff --git a/arch/powerpc/cpu/mpc85xx/u-boot.lds b/arch/powerpc/cpu/mpc85xx/u-boot.lds index 67d7763..506c7f2 100644 --- a/arch/powerpc/cpu/mpc85xx/u-boot.lds +++ b/arch/powerpc/cpu/mpc85xx/u-boot.lds @@ -1,5 +1,5 @@ /* - * Copyright 2007-2009 Freescale Semiconductor, Inc. + * Copyright 2007-2009, 2011 Freescale Semiconductor, Inc. * * See file CREDITS for list of people who contributed to this * project. @@ -20,7 +20,11 @@ * MA 02111-1307 USA */ -#ifndef RESET_VECTOR_ADDRESS +#include config.h/* CONFIG_BOARDDIR */ + +#ifdef CONFIG_RESET_VECTOR_ADDRESS +#define RESET_VECTOR_ADDRESS CONFIG_RESET_VECTOR_ADDRESS +#else #define RESET_VECTOR_ADDRESS 0xfffc #endif diff --git a/board/freescale/corenet_ds/config.mk b/board/freescale/corenet_ds/config.mk deleted file mode 100644 index 15bbf20..000 --- a/board/freescale/corenet_ds/config.mk +++ /dev/null @@ -1,27 +0,0 @@ -# -# Copyright 2007-2009 Freescale Semiconductor, Inc. -# -# 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 -# - -# -# P4080DS board -# - -RESET_VECTOR_ADDRESS = 0xeffc diff --git a/board/freescale/mpc8536ds/config.mk b/board/freescale/mpc8536ds/config.mk index b7deb4a..228d8c0 100644 --- a/board/freescale/mpc8536ds/config.mk +++ b/board/freescale/mpc8536ds/config.mk @@ -1,5 +1,5 @@ # -# Copyright 2008 Freescale Semiconductor. +# Copyright 2008, 2011 Freescale Semiconductor. # # See file CREDITS for list of people who contributed to this # project. @@ -28,15 +28,3 @@ ifeq ($(CONFIG_NAND), y) LDSCRIPT := $(TOPDIR)/$(CPUDIR)/u-boot-nand.lds endif endif - -ifeq ($(CONFIG_SDCARD), y) -RESET_VECTOR_ADDRESS = 0xf8fc -endif - -ifeq ($(CONFIG_SPIFLASH), y) -RESET_VECTOR_ADDRESS = 0xf8fc -endif - -ifndef RESET_VECTOR_ADDRESS -RESET_VECTOR_ADDRESS = 0xeffc -endif diff --git a/board/freescale/mpc8572ds/config.mk b/board/freescale/mpc8572ds/config.mk index 7fd6412..53fda6f 100644 --- a/board/freescale/mpc8572ds/config.mk +++ b/board/freescale/mpc8572ds/config.mk @@ -28,5 +28,3 @@ ifeq ($(CONFIG_NAND), y) LDSCRIPT := $(TOPDIR)/$(CPUDIR)/u-boot-nand.lds endif endif - -RESET_VECTOR_ADDRESS = 0xeffc diff --git a/board/freescale/p1022ds/config.mk b/board/freescale/p1022ds/config.mk deleted file mode 100644 index a953fdd..000 --- a/board/freescale/p1022ds/config.mk +++ /dev/null @@ -1,10 +0,0 @@ -# -# Copyright 2010 Freescale Semiconductor, Inc. -# -# This program is free software; you can redistribute it and/or modify it -# under the terms of the GNU General Public License as published by the Free -# Software Foundation; either version 2 of the License, or (at your option) -# any later version. -# - -RESET_VECTOR_ADDRESS = 0xeffc diff --git a/board/freescale/p1_p2_rdb/config.mk b/board/freescale/p1_p2_rdb/config.mk index eececaa..d57b50e 100644 --- a/board/freescale/p1_p2_rdb/config.mk +++ b/board/freescale/p1_p2_rdb/config.mk @@ -29,15 +29,3 @@ ifeq ($(CONFIG_NAND), y) LDSCRIPT := $(TOPDIR)/$(CPUDIR)/u-boot-nand.lds endif endif - -ifeq ($(CONFIG_SDCARD), y)
Re: [U-Boot] [PATCH 4/4] powerpc: Add LINK_OFF calls in early C-code.
Wolfgang Denk w...@denx.de wrote on 2011/01/18 00:42:10: Dear Joakim Tjernlund, In message of71b40f2e.5c935ba8-onc125781b.007cb9e6-c125781b.007e5...@transmode.se you wrote: No other board is broken. This new function is neutral to other boards. Well, I see this differntly. Wolfgang, once you indicated you were interested in such feature as I have added but my first impl. had LINK_OFF calls all over the place, still you were tempted to add the feature. Now that I have reduced the LINK_OFF calls to a minimum you suddenly want to reject it even though 95% of the LINK_OFF calls are gone. Why this change of heart? Yes, I am definitely interested in this feature. Good. But I still consider the impact to make this really working to heavy (and by working I mean working for all boards, out of the box, without having to go through iterations of breakage to find out where else a LINK_OFF needs to be added). Yes, but the target area is fairly small. Only code the runs before relocation which isn't that much. How do you think a solution for all boards would look like? The only other method is can think of is some MMU trickery and I don't even see how you can make that work on all boards and you would probably be locked to specific address ranges if you use BATs as defined on PowerPC In the current form, your simplification results just from the fact that you just added the bare minimum needed for your board. If - as you hope - more and more boards would use this, more and more of these now omitted LINK_OFFs would have to be added again. The result would be exactly the same mess as your original patch - it doesn't small any better when you try to feed it to me in small bites. My first patch was pretty much only my board too and the result for all boards would be MUCH smaller than the first patch approach. I did only did one board because that was what I could test and I wanted feedback from the list. Did you expect me to blindly port this to every board? The ground work for 83xx is done, a few more LINK_OFF calls is probably needed for all 83xx boards, I can take a stab at the remaining 83xx boards once you embrace this method. The rest will have to be up to the other board maintainers, if they want this feature. And frankly, adding this stuff for an out-of-tree port is, um..., sorry, but I don't find polite words atm. Yet you are interested in such a feature. Sorry but I too have a hard time finding polite words. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] nios2: reset cfi flash before reading env
Dear Wolfgang, On 01/18/2011 05:02 AM, Wolfgang Denk wrote: void __flash_cmd_reset(flash_info_t *info) I recommend you have a second, closer look. Notice that flash_cmd_reset() comes with __attribute__ weak, because it does not work as a general solution - it may work here, but other code may be needed on other flash chips. Your code does not provide such an option. Thanks a lot. I will add a weak attribute. Best regards, Thomas ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2] spi: add new driver for OpenCores tiny_spi
Dear Wolfgang, On 01/18/2011 05:23 AM, Wolfgang Denk wrote: Please separate declartations and code with a blank line. Please fix globally. Please do not use base + offset notation. Use a proper C struct instead. Please fix globally. Thank you very much for the coding style correction. I will follow them. Best regards, Thomas ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v2] nios2: reset cfi flash before reading env
Flash might be in unknown state when u-boot is started with jtag. And got wrong env data. So reset it in board early init. We cannot use generic cfi flash routines, because flash_init() is not run yet. Signed-off-by: Thomas Chou tho...@wytron.com.tw --- v2 use weak func to reset flash, as Wolfgang suggested. reset the chip where env is located. board/altera/nios2-generic/nios2-generic.c | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/board/altera/nios2-generic/nios2-generic.c b/board/altera/nios2-generic/nios2-generic.c index 89848cf..220a4c4 100644 --- a/board/altera/nios2-generic/nios2-generic.c +++ b/board/altera/nios2-generic/nios2-generic.c @@ -24,12 +24,28 @@ #include common.h #include netdev.h +#include mtd/cfi_flash.h +#include asm/io.h void text_base_hook(void); /* nop hook for text_base.S */ +#if defined(CONFIG_ENV_IS_IN_FLASH) defined(CONFIG_ENV_ADDR) +static void __early_flash_cmd_reset(void) +{ + /* reset flash before we read env */ + writeb(AMD_CMD_RESET, CONFIG_ENV_ADDR); + writeb(FLASH_CMD_RESET, CONFIG_ENV_ADDR); +} +void early_flash_cmd_reset(void) + __attribute__((weak,alias(__early_flash_cmd_reset))); +#endif + int board_early_init_f(void) { text_base_hook(); +#if defined(CONFIG_ENV_IS_IN_FLASH) defined(CONFIG_ENV_ADDR) + early_flash_cmd_reset(); +#endif return 0; } -- 1.7.3.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH][v2] Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR
From: Prabhakar prabha...@freescale.com CONFIG_SYS_MPC85xx_SERDES1_ADDR was defined wrong as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET. It should be as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET. Signed-off-by: Prabhakar Kushwaha prabha...@freescale.com --- Based of http://git.denx.de/u-boot.git (branch master) Top commit: commit c6b734f5aea2ba75caaa1929f7e649ecda8d2f31 Merge: 072f412 a972089 Author: Wolfgang Denk w...@denx.de Date: Wed Jan 12 23:59:53 2011 +0100 Merge branch 'master' of git://git.denx.de/u-boot-sh Changes for v1: Incorporated Wolfgang Denk's comments - Fixed header length to 74 - Rebased to http://git.denx.de/u-boot.git (branch master) Changes for v2: Incorporated Timur Tabi's comments - Added full name in Signed-off-by arch/powerpc/include/asm/immap_85xx.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/include/asm/immap_85xx.h b/arch/powerpc/include/asm/immap_85xx.h index 30c64eb..6a2daf2 100644 --- a/arch/powerpc/include/asm/immap_85xx.h +++ b/arch/powerpc/include/asm/immap_85xx.h @@ -2307,7 +2307,7 @@ typedef struct ccsr_pme { #define CONFIG_SYS_MPC85xx_CPM_ADDR \ (CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_CPM_OFFSET) #define CONFIG_SYS_MPC85xx_SERDES1_ADDR \ - (CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET) + (CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET) #define CONFIG_SYS_MPC85xx_SERDES2_ADDR \ (CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET) #define CONFIG_SYS_FSL_CORENET_SERDES_ADDR \ -- 1.6.3.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH v3] spi: add new driver for OpenCores tiny_spi
This patch adds support for OpenCores tiny_spi. http://opencores.org/project,tiny_spi Signed-off-by: Thomas Chou tho...@wytron.com.tw --- for u-boot v2, use const and clean up as Mike suggested. v3, use struct instead of base+offset as Wolfgang suggested. drivers/spi/Makefile |1 + drivers/spi/oc_tiny_spi.c | 248 + 2 files changed, 249 insertions(+), 0 deletions(-) create mode 100644 drivers/spi/oc_tiny_spi.c diff --git a/drivers/spi/Makefile b/drivers/spi/Makefile index e34a124..8ad1d7f 100644 --- a/drivers/spi/Makefile +++ b/drivers/spi/Makefile @@ -35,6 +35,7 @@ COBJS-$(CONFIG_KIRKWOOD_SPI) += kirkwood_spi.o COBJS-$(CONFIG_MPC52XX_SPI) += mpc52xx_spi.o COBJS-$(CONFIG_MPC8XXX_SPI) += mpc8xxx_spi.o COBJS-$(CONFIG_MXC_SPI) += mxc_spi.o +COBJS-$(CONFIG_OC_TINY_SPI) += oc_tiny_spi.o COBJS-$(CONFIG_OMAP3_SPI) += omap3_spi.o COBJS-$(CONFIG_SOFT_SPI) += soft_spi.o diff --git a/drivers/spi/oc_tiny_spi.c b/drivers/spi/oc_tiny_spi.c new file mode 100644 index 000..f379da1 --- /dev/null +++ b/drivers/spi/oc_tiny_spi.c @@ -0,0 +1,248 @@ +/* + * Opencore tiny_spi driver + * + * http://opencores.org/project,tiny_spi + * + * based on bfin_spi.c + * Copyright (c) 2005-2008 Analog Devices Inc. + * Copyright (C) 2010 Thomas Chou tho...@wytron.com.tw + * + * Licensed under the GPL-2 or later. + */ + +#include common.h +#include asm/io.h +#include malloc.h +#include spi.h +#include asm/gpio.h + +#define TINY_SPI_STATUS_TXE 0x1 +#define TINY_SPI_STATUS_TXR 0x2 + +struct tiny_spi_regs { + volatile unsigned rxdata; /* Rx data reg */ + volatile unsigned txdata; /* Tx data reg */ + volatile unsigned status; /* Status reg */ + volatile unsigned control; /* Control reg */ + volatile unsigned baud; /* Baud reg */ +}; + +struct tiny_spi_host { + uint base; + uint freq; + uint baudwidth; +}; +static const struct tiny_spi_host tiny_spi_host_list[] = + CONFIG_SYS_TINY_SPI_LIST; + +struct tiny_spi_slave { + struct spi_slave slave; + const struct tiny_spi_host *host; + uint mode; + uint baud; + uint flg; +}; +#define to_tiny_spi_slave(s) container_of(s, struct tiny_spi_slave, slave) + +int spi_cs_is_valid(unsigned int bus, unsigned int cs) +{ + return bus ARRAY_SIZE(tiny_spi_host_list) gpio_is_valid(cs); +} + +void spi_cs_activate(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + unsigned int cs = slave-cs; + + gpio_set_value(cs, tiny_spi-flg); + debug(%s: SPI_CS_GPIO:%x\n, __func__, gpio_get_value(cs)); +} + +void spi_cs_deactivate(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + unsigned int cs = slave-cs; + + gpio_set_value(cs, !tiny_spi-flg); + debug(%s: SPI_CS_GPIO:%x\n, __func__, gpio_get_value(cs)); +} + +void spi_set_speed(struct spi_slave *slave, uint hz) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + const struct tiny_spi_host *host = tiny_spi-host; + + tiny_spi-baud = min(DIV_ROUND_UP(host-freq, hz * 2), +(1 host-baudwidth)) - 1; + debug(%s: speed %u actual %u\n, __func__, hz, + host-freq / ((tiny_spi-baud + 1) * 2)); +} + +void spi_init(void) +{ +} + +struct spi_slave *spi_setup_slave(unsigned int bus, unsigned int cs, + unsigned int hz, unsigned int mode) +{ + struct tiny_spi_slave *tiny_spi; + + if (!spi_cs_is_valid(bus, cs) || gpio_request(cs, tiny_spi)) + return NULL; + + tiny_spi = malloc(sizeof(*tiny_spi)); + if (!tiny_spi) + return NULL; + memset(tiny_spi, 0, sizeof(*tiny_spi)); + + tiny_spi-slave.bus = bus; + tiny_spi-slave.cs = cs; + tiny_spi-host = tiny_spi_host_list[bus]; + tiny_spi-mode = mode (SPI_CPOL | SPI_CPHA); + tiny_spi-flg = mode SPI_CS_HIGH ? 1 : 0; + spi_set_speed(tiny_spi-slave, hz); + + debug(%s: bus:%i cs:%i base:%lx\n, __func__, + bus, cs, tiny_spi-host-base); + return tiny_spi-slave; +} + +void spi_free_slave(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + + gpio_free(slave-cs); + free(tiny_spi); +} + +int spi_claim_bus(struct spi_slave *slave) +{ + struct tiny_spi_slave *tiny_spi = to_tiny_spi_slave(slave); + struct tiny_spi_regs *regs = (void *)tiny_spi-host-base; + + debug(%s: bus:%i cs:%i\n, __func__, slave-bus, slave-cs); + gpio_direction_output(slave-cs, !tiny_spi-flg); + writel(tiny_spi-mode, regs-control); + writel(tiny_spi-baud, regs-baud); + return 0; +} + +void spi_release_bus(struct spi_slave *slave) +{ + debug(%s: bus:%i cs:%i\n, __func__, slave-bus, slave-cs); +} + +#ifndef CONFIG_TINY_SPI_IDLE_VAL +# define
Re: [U-Boot] [PATCH][v2] Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR
On Jan 17, 2011, at 9:33 PM, Prabhakar Kushwaha wrote: From: Prabhakar prabha...@freescale.com CONFIG_SYS_MPC85xx_SERDES1_ADDR was defined wrong as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET. It should be as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET. Signed-off-by: Prabhakar Kushwaha prabha...@freescale.com --- Based of http://git.denx.de/u-boot.git (branch master) Top commit: commit c6b734f5aea2ba75caaa1929f7e649ecda8d2f31 Merge: 072f412 a972089 Author: Wolfgang Denk w...@denx.de Date: Wed Jan 12 23:59:53 2011 +0100 Merge branch 'master' of git://git.denx.de/u-boot-sh Changes for v1: Incorporated Wolfgang Denk's comments - Fixed header length to 74 - Rebased to http://git.denx.de/u-boot.git (branch master) Changes for v2: Incorporated Timur Tabi's comments - Added full name in Signed-off-by arch/powerpc/include/asm/immap_85xx.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] powerpc/85xx: Add Support for Freescale P1010 Processor
On Jan 14, 2011, at 10:41 AM, Kumar Gala wrote: From: Poonam Aggrwal poonam.aggr...@freescale.com Key Features include of the P1010: * e500v2 core frequency operation of 500 to 800 MHz * Power consumption less than 5.0 W at 800 MHz core speed * Dual SATA 3 Gbps controllers with integrated PHY * Dual PCI Express controllers * Three 10/100/1000 Mbps enhanced triple-speed Ethernet controllers (eTSECs) * TCP/IP acceleration and classification capabilities * IEEE 1588 support * Lossless flow control * RGMII, SGMII * DDR3 with support for a 32-bit data interface (40 bits including ECC), up to 800 MHz data rate 32/16-bit DDR3 memory controller * Dedicated security engine featuring trusted boot * TDM interface * Dual controller area networks (FlexCAN) controller * SD/MMC card controller supporting booting from Flash cards * USB 2.0 host and device controller with an on-chip, high-speed PHY * Integrated Flash controller (IFC) * Power Management Controller (PMC) * Four-channel, general-purpose DMA controller * I2C controller * Serial peripheral interface (SPI) controller with master and slave support * System timers including a periodic interrupt timer, real-time clock, software watchdog timer, and four general-purpose timers * Dual DUARTs Signed-off-by: Poonam Aggrwal poonam.aggr...@freescale.com Signed-off-by: Dipen Dudhat dipen.dud...@freescale.com Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/Makefile|1 + arch/powerpc/cpu/mpc8xxx/cpu.c |2 ++ arch/powerpc/include/asm/config.h|6 -- arch/powerpc/include/asm/processor.h |2 ++ drivers/misc/fsl_law.c |1 + 5 files changed, 10 insertions(+), 2 deletions(-) applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/2] powerpc/85xx: Add Support for Freescale P1014 Processor
On Jan 14, 2011, at 10:41 AM, Kumar Gala wrote: From: Poonam Aggrwal poonam.aggr...@freescale.com The P1014 is similar to the P1010 processor with the following differences: - 16bit DDR with ECC. (P1010 has 32bit DDR w/o ECC) - no eCAN interface. (P1010 has 2 eCAN interfaces) - Two SGMII interface (P1010 has 3 SGMII) - No secure boot Signed-off-by: Poonam Aggrwal poonam.aggr...@freescale.com Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/Makefile|1 + arch/powerpc/cpu/mpc8xxx/cpu.c |2 ++ arch/powerpc/include/asm/config.h|4 ++-- arch/powerpc/include/asm/processor.h |2 ++ drivers/misc/fsl_law.c |2 +- 5 files changed, 8 insertions(+), 3 deletions(-) applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/3] powerpc/p3041: Add various p3041 specific information
On Jan 13, 2011, at 4:46 AM, Kumar Gala wrote: Add P3041 SoC specific information: * SERDES Table * LIODN setup * Portal configuration Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/Makefile |2 + arch/powerpc/cpu/mpc85xx/p3041_ids.c| 105 + arch/powerpc/cpu/mpc85xx/p3041_serdes.c | 151 +++ 3 files changed, 258 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/cpu/mpc85xx/p3041_ids.c create mode 100644 arch/powerpc/cpu/mpc85xx/p3041_serdes.c applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] powerpc/p5020: Add various p5020 specific information
On Jan 13, 2011, at 4:46 AM, Kumar Gala wrote: Add P5020 SoC specific information: * SERDES Table * LIODN setup * Portal configuration Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/Makefile |2 + arch/powerpc/cpu/mpc85xx/p5020_ids.c| 105 + arch/powerpc/cpu/mpc85xx/p5020_serdes.c | 151 +++ 3 files changed, 258 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/cpu/mpc85xx/p5020_ids.c create mode 100644 arch/powerpc/cpu/mpc85xx/p5020_serdes.c applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 3/3 v2] powerpc/p2040: Add various p2040 specific information
On Jan 13, 2011, at 4:53 AM, Kumar Gala wrote: Add P2040 SoC specific information: * SERDES Table * Added p2040 to cpu_type_list and SVR list * Added number of LAWs for p2040 * Set CONFIG_MAX_CPUS to 4 for p2040 Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- * Fixed commit message * Added some additional P2040 specific defines/cfg applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v3] powerpc/85xx: Move RESET_VECTOR_ADDRESS into config.h
On Jan 17, 2011, at 5:55 PM, Kumar Gala wrote: Rather than defining it config.mk we can set it in config.h and remove config.mk from several boards that don't need it. We mimic what 4xx does and introduce CONFIG_RESET_VECTOR_ADDRESS for config.h to set. Signed-off-by: Kumar Gala ga...@kernel.crashing.org --- arch/powerpc/cpu/mpc85xx/u-boot.lds |8 ++-- board/freescale/corenet_ds/config.mk | 27 --- board/freescale/mpc8536ds/config.mk | 14 +- board/freescale/mpc8572ds/config.mk |2 -- board/freescale/p1022ds/config.mk| 10 -- board/freescale/p1_p2_rdb/config.mk | 12 board/freescale/p2020ds/config.mk| 26 -- include/configs/MPC8536DS.h |6 ++ include/configs/MPC8572DS.h |4 include/configs/P1022DS.h|4 include/configs/P1_P2_RDB.h |6 ++ include/configs/P2020DS.h|4 include/configs/corenet_ds.h |4 13 files changed, 35 insertions(+), 92 deletions(-) delete mode 100644 board/freescale/corenet_ds/config.mk delete mode 100644 board/freescale/p1022ds/config.mk delete mode 100644 board/freescale/p2020ds/config.mk applied to 85xx - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] mpq101: initial support for Mercury Computer Systems MPQ101 board
On Jan 17, 2011, at 4:38 PM, Wolfgang Denk wrote: Dear Alex Dubov, In message 369392.26813...@web37607.mail.mud.yahoo.com you wrote: Could you please be a bit more specific, i. e. like quoting file names and line numbers? These were produced by grep -nr ddr\-\ board/ | grep -v out_be I can attach a listing, but it should be quicker this way. Up until now it seemed a rather common practice to initialize ddr controllers directly, bypassing io accessors. Indeed. Even after the recent removal of some boards this list remains: board/freescale/mpc8349itx/mpc8349itx.c board/freescale/mpc8536ds/mpc8536ds.c board/freescale/mpc8540ads/mpc8540ads.c board/freescale/mpc8560ads/mpc8560ads.c board/freescale/mpc8569mds/mpc8569mds.c board/freescale/mpc8572ds/mpc8572ds.c board/freescale/mpc8610hpcd/mpc8610hpcd.c board/freescale/mpc8641hpcn/mpc8641hpcn.c board/freescale/p2020ds/p2020ds.c board/sbc8560/sbc8560.c board/sbc8641d/sbc8641d.c board/socrates/sdram.c board/tqc/tqm85xx/sdram.c Kumar, Andy - do you have any plans to fix this for the freescale boards? Best regards, Wolfgang Denk I don't have an plans to fix this, however we can add it to the TODO list. Just isn't high priority at this point. - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] (no subject)
The following changes since commit e1ccf97c5d7651664d37c0c5aa243874b8851b2d: Merge branch 'master' of git://git.denx.de/u-boot-mpc85xx (2011-01-17 20:31:46 +0100) are available in the git repository at: git://git.denx.de/u-boot-mpc85xx master Haiying Wang (1): 8xxx/ddr: add support to only compute the ddr sdram size Kumar Gala (4): powerpc/85xx: Move RESET_VECTOR_ADDRESS into config.h powerpc/p3041: Add various p3041 specific information powerpc/p5020: Add various p5020 specific information powerpc/p2040: Add various p2040 specific information Poonam Aggrwal (2): powerpc/85xx: Add Support for Freescale P1010 Processor powerpc/85xx: Add Support for Freescale P1014 Processor Prabhakar (1): Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR arch/powerpc/cpu/mpc85xx/Makefile|6 + arch/powerpc/cpu/mpc85xx/p2040_serdes.c | 66 + arch/powerpc/cpu/mpc85xx/p3041_ids.c | 105 + arch/powerpc/cpu/mpc85xx/p3041_serdes.c | 151 ++ arch/powerpc/cpu/mpc85xx/p5020_ids.c | 105 + arch/powerpc/cpu/mpc85xx/p5020_serdes.c | 151 ++ arch/powerpc/cpu/mpc85xx/u-boot.lds |8 +- arch/powerpc/cpu/mpc8xxx/cpu.c |6 + arch/powerpc/cpu/mpc8xxx/ddr/ctrl_regs.c | 10 ++- arch/powerpc/cpu/mpc8xxx/ddr/ddr.h |8 +- arch/powerpc/cpu/mpc8xxx/ddr/main.c | 31 +- arch/powerpc/include/asm/config.h| 10 ++- arch/powerpc/include/asm/immap_85xx.h|2 +- arch/powerpc/include/asm/processor.h |6 + board/freescale/corenet_ds/config.mk | 27 -- board/freescale/mpc8536ds/config.mk | 14 +--- board/freescale/mpc8572ds/config.mk |2 - board/freescale/p1022ds/config.mk| 10 -- board/freescale/p1_p2_rdb/config.mk | 12 --- board/freescale/p2020ds/config.mk| 26 - drivers/misc/fsl_law.c |7 +- include/configs/MPC8536DS.h |6 + include/configs/MPC8572DS.h |4 + include/configs/P1022DS.h|4 + include/configs/P1_P2_RDB.h |6 + include/configs/P2020DS.h|4 + include/configs/corenet_ds.h |4 + 27 files changed, 683 insertions(+), 108 deletions(-) create mode 100644 arch/powerpc/cpu/mpc85xx/p2040_serdes.c create mode 100644 arch/powerpc/cpu/mpc85xx/p3041_ids.c create mode 100644 arch/powerpc/cpu/mpc85xx/p3041_serdes.c create mode 100644 arch/powerpc/cpu/mpc85xx/p5020_ids.c create mode 100644 arch/powerpc/cpu/mpc85xx/p5020_serdes.c delete mode 100644 board/freescale/corenet_ds/config.mk delete mode 100644 board/freescale/p1022ds/config.mk delete mode 100644 board/freescale/p2020ds/config.mk ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v5] mpq101: initial support for Mercury Computer Systems MPQ101 board
On Jan 17, 2011, at 1:26 AM, Alex Dubov wrote: +/* + * Merury Computers MPQ101 board configuration file + * + */ +#ifndef __CONFIG_H +#define __CONFIG_H + +#ifdef CONFIG_36BIT +# define CONFIG_PHYS_64BIT 1 +#endif + +/* High Level Configuration Options */ +#define CONFIG_BOOKE 1 /* BOOKE */ +#define CONFIG_E500 1 /* BOOKE e500 family */ +#define CONFIG_MPC85xx1 /* MPC8540/60/55/41/48 */ +#define CONFIG_MPC85481 /* MPC8548 specific */ +#define CONFIG_MPQ101 1 /* MPQ101 board specific */ + +#define CONFIG_SYS_SRIO 1 /* enable serial RapidIO */ +#define CONFIG_TSEC_ENET 1 /* tsec ethernet support */ +#define CONFIG_INTERRUPTS 1 /* enable pci, srio, ddr interrupts */ +#define CONFIG_FSL_LAW1 /* Use common FSL init code */ Remove all the '1'. Yes a ton of config.h do this and its not needed. We need to clean them up but haven't gotten around to it. + +/* + * These can be toggled for performance analysis, otherwise use default. + */ +#define CONFIG_L2_CACHE 1 /* toggle L2 cache */ +#define CONFIG_BTB1 /* toggle branch predition */ + +#define CONFIG_PANIC_HANG 1 + +/* + * Only possible on E500 Version 2 or newer cores. + */ +#define CONFIG_ENABLE_36BIT_PHYS 1 + +#ifdef CONFIG_PHYS_64BIT +# define CONFIG_ADDR_MAP 1 +# define CONFIG_SYS_NUM_ADDR_MAP 16 /* number of TLB1 entries */ +#endif + + +#define CONFIG_SYS_CLK_FREQ3300 /* sysclk for MPC85xx */ + +/* + * Base addresses -- Note these are effective addresses where the + * actual resources get mapped (not physical addresses) + */ +#define CONFIG_SYS_CCSRBAR_DEFAULT 0xff70 +#define CONFIG_SYS_CCSRBAR 0xe000 + +#ifdef CONFIG_PHYS_64BIT +# define CONFIG_SYS_CCSRBAR_PHYS 0xfe000ull +#else +# define CONFIG_SYS_CCSRBAR_PHYS CONFIG_SYS_CCSRBAR +#endif + +#define CONFIG_SYS_IMMRCONFIG_SYS_CCSRBAR + +/* DDR Setup */ +#define CONFIG_FSL_DDR2 + +#define CONFIG_ECC_INIT_VIA_DDRCONTROLLER 1 /* DDR controller or DMA? */ + +#define CONFIG_MEM_INIT_VALUE0xDeadBeef +#define CONFIG_SYS_DDR_SDRAM_BASE0x /* DDR is system memory*/ +#define CONFIG_SYS_SDRAM_BASECONFIG_SYS_DDR_SDRAM_BASE + +#define CONFIG_NUM_DDR_CONTROLLERS 1 +#define CONFIG_DIMM_SLOTS_PER_CTLR 1 +#define CONFIG_CHIP_SELECTS_PER_CTRL (2 * CONFIG_DIMM_SLOTS_PER_CTLR) + +/* Fixed 512MB DDR2 parameters */ +#define CONFIG_SYS_SDRAM_SIZE_LOG29 /* DDR is 512MB */ +#define CONFIG_SYS_DDR_CS0_BNDS 0x001f +#define CONFIG_SYS_DDR_CS0_CONFIG0x80014102 +#define CONFIG_SYS_DDR_TIMING_3 0x0001 +#define CONFIG_SYS_DDR_TIMING_0 0x00260802 +#define CONFIG_SYS_DDR_TIMING_1 0x5c47a432 +#define CONFIG_SYS_DDR_TIMING_1_PERF 0x49352322 +#define CONFIG_SYS_DDR_TIMING_2 0x03984cce +#define CONFIG_SYS_DDR_TIMING_2_PERF 0x14904cca +#define CONFIG_SYS_DDR_MODE_10x00400442 +#define CONFIG_SYS_DDR_MODE_1_PERF 0x00480432 +#define CONFIG_SYS_DDR_MODE_20x +#define CONFIG_SYS_DDR_MODE_2_PERF 0x +#define CONFIG_SYS_DDR_INTERVAL 0x08200100 +#define CONFIG_SYS_DDR_INTERVAL_PERF 0x06180100 +#define CONFIG_SYS_DDR_CLK_CTRL 0x0380 +#define CONFIG_SYS_DDR_CONTROL 0xc3008000 /* Type = DDR2 */ +#define CONFIG_SYS_DDR_CONTROL2 0x0440 + +#define CONFIG_SYS_ALT_MEMTEST 1 +#define CONFIG_SYS_MEMTEST_START 0x0ff0 /* memtest works on */ +#define CONFIG_SYS_MEMTEST_END 0x0ffc + +/* + * RAM definitions + */ +#define CONFIG_SYS_INIT_RAM_LOCK 1 +#define CONFIG_SYS_INIT_RAM_ADDR 0xe401 /* Initial RAM address */ +#define CONFIG_SYS_INIT_RAM_SIZE 0x4000 /* Size of used area in RAM */ + +#define CONFIG_SYS_GBL_DATA_OFFSET (CONFIG_SYS_INIT_RAM_SIZE \ + - GENERATED_GBL_DATA_SIZE) +#define CONFIG_SYS_INIT_SP_OFFSET CONFIG_SYS_GBL_DATA_OFFSET + +#define CONFIG_SYS_MONITOR_LEN (256 * 1024) /* Reserve 256 kB for Mon */ +#define CONFIG_SYS_MALLOC_LEN (1024 * 1024) /* Reserved for malloc */ + +/* + * Local Bus Definitions + */ +#define CONFIG_SYS_LBC_LCRR 0x0004 /* LB clock ratio reg */ +#define CONFIG_SYS_LBC_LBCR 0x /* LB config reg */ + + +/* + * FLASH on the Local Bus + * One bank, 128M, using the CFI driver. + */ +#define CONFIG_SYS_BOOT_BLOCK 0xf800/* boot TLB block */ +#define CONFIG_SYS_FLASH_BASE CONFIG_SYS_BOOT_BLOCK /* start of FLASH 128M */ + +#ifdef CONFIG_PHYS_64BIT +# define CONFIG_SYS_FLASH_BASE_PHYS 0xff800ull +#else +# define CONFIG_SYS_FLASH_BASE_PHYS CONFIG_SYS_FLASH_BASE +#endif + +/* 0xf8001801 */ +#define CONFIG_SYS_BR0_PRELIM (BR_PHYS_ADDR(CONFIG_SYS_FLASH_BASE_PHYS) \ +| BR_PS_32 | BR_V) + +/* 0xf8006ff7 */ +#define CONFIG_SYS_OR0_PRELIM (MEG_TO_AM(128) | OR_GPCM_XAM | OR_GPCM_CSNT \ +| OR_GPCM_ACS_DIV2 |
Re: [U-Boot] Memset 0 in heap initialization?
On 09.12.2010 17:14, Dirk Behme wrote: In common/dlmalloc.c in the heap initialization mem_malloc_init() [1] it looks like the whole heap is initialized with zero: memset((void *)mem_malloc_start, 0, size); What's the reason for doing this? I know that the .bss segment has to be zeroed at system start up, but why doing this for the heap, too? It was my understanding that one could make no assumption about the initial content of memory returned by malloc() (?). Sorry if I miss the obvious ;) Thanks Dirk [1] http://git.denx.de/?p=u-boot.git;a=blob;f=common/dlmalloc.c;h=e9bab09b8eac7c0ec4900a3685f15a807b448c31;hb=refs/heads/master#l1529 Any idea regarding this? Thanks Dirk ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/8] armv7: adapt omap3 to the new cache maintenance framework
Dear Wolfgang, On Tuesday 18 January 2011 03:25 AM, Wolfgang Denk wrote: Dear Aneesh V, In message4d2b1a90.9030...@ti.com you wrote: +struct __attribute__ ((__packed__)) emu_hal_params { + u32 num_params; + u32 param1; +}; Why exactly do we need the __attribute__ ((__packed__)) here? Because a pointer to it has to be passed to ROM code and ROM code wouldn't expect any padding. But padding would only be needed if there were any alignment requirements requiring it. There are none here. Yes. But, is that guaranteed? I didn't want to make it compiler dependent. Isn't it safer to use '__attribute__ ((__packed__))' Best regards, Aneesh ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Memset 0 in heap initialization?
On Jan 17, 2011, at 11:17 PM, Dirk Behme wrote: On 09.12.2010 17:14, Dirk Behme wrote: In common/dlmalloc.c in the heap initialization mem_malloc_init() [1] it looks like the whole heap is initialized with zero: memset((void *)mem_malloc_start, 0, size); What's the reason for doing this? I know that the .bss segment has to be zeroed at system start up, but why doing this for the heap, too? It was my understanding that one could make no assumption about the initial content of memory returned by malloc() (?). Sorry if I miss the obvious ;) Thanks Dirk [1] http://git.denx.de/?p=u-boot.git;a=blob;f=common/dlmalloc.c;h=e9bab09b8eac7c0ec4900a3685f15a807b448c31;hb=refs/heads/master#l1529 Any idea regarding this? Part of this is because we define MORECORE_CLEARS 1 which does assume memory is zero'd out. - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 1/1] ARM: */start.S: code cleanup
Dear Jason Liu, Dear Albert Aribaud, Am 17.01.2011 um 20:54 schrieb Jason Liu: Hi, Albert, 2010/12/23 Jason Liu r64...@freescale.com: Remove the useless code from start.S Signed-off-by: Jason Liu r64...@freescale.com --- Changes for v2: -Address the comments from Albert to make changes to all ARM cpus with one single patch --- arch/arm/cpu/arm920t/start.S |2 -- Any feedback about this patch? Tested-by: Andreas Bießmann andreas.de...@googlemail.com This patch is correct for arm920t. This patch is most probable correct for other architectures. regards Andreas Bießmann ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot