[U-Boot] [PATCH 3/3] TI: DaVinci: Create configuration file for da850 EVM

[U-Boot] [PATCH 3/3] TI: DaVinci: Create configuration file for da850 EVM

[Sudhakar Rajashekhara]

Provides initial support for TI OMAP-L138/DA850 SoC devices
on a Logic PD EVM board.

Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
---
 MAINTAINERS                |    4 +
 MAKEALL                    |    1 +
 Makefile                   |    3 +
 include/configs/da850evm.h |  135 ++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 143 insertions(+), 0 deletions(-)
 create mode 100644 include/configs/da850evm.h

diff --git a/MAINTAINERS b/MAINTAINERS
index a5d5835..c52803d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -758,6 +758,10 @@ Nick Thompson <nick.thompson@gefanuc.com>
 
 	da830evm	ARM926EJS (DA830/OMAP-L137)
 
+Sudhakar Rajashekhara <sudhakar.raj@ti.com>
+
+	da850evm	ARM926EJS (DA850/OMAP-L138)
+
 Albin Tonnerre <albin.tonnerre@free-electrons.com>
 
 	sbc35_a9g20	ARM926EJS (AT91SAM9G20 SoC)
diff --git a/MAKEALL b/MAKEALL
index 34bc4a1..cdb6676 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -559,6 +559,7 @@ LIST_ARM9="			\
 	cp946es			\
 	cp966			\
 	da830evm		\
+	da850evm		\
 	edb9301			\
 	edb9302			\
 	edb9302a		\
diff --git a/Makefile b/Makefile
index 393b180..0f21db6 100644
--- a/Makefile
+++ b/Makefile
@@ -2913,6 +2913,9 @@ cp922_XA10_config	\
 cp1026_config: unconfig
 	@board/armltd/integrator/split_by_variant.sh cp $@
 
+da850evm_config:	unconfig
+	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
+
 da830evm_config:	unconfig
 	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
 
diff --git a/include/configs/da850evm.h b/include/configs/da850evm.h
new file mode 100644
index 0000000..2fcc76d
--- /dev/null
+++ b/include/configs/da850evm.h
@@ -0,0 +1,135 @@
+/*
+ * Copyright (C) 2010 Texas Instruments Incorporated
+ *
+ * Based on davinci_dvevm.h. Original Copyrights follow:
+ *
+ * Copyright (C) 2007 Sergey Kubushyn <ksi@koi8.net>
+ *
+ * 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., 675 Mass Ave, Cambridge, MA 02139, USA.
+ */
+
+#ifndef __CONFIG_H
+#define __CONFIG_H
+
+/*
+ * Board
+ */
+
+/*
+ * SoC Configuration
+ */
+#define CONFIG_MACH_DAVINCI_DA850_EVM
+#define CONFIG_ARM926EJS		/* arm926ejs CPU core */
+#define CONFIG_SOC_DA8XX		/* TI DA8xx SoC */
+#define CONFIG_SYS_CLK_FREQ		clk_get(DAVINCI_ARM_CLKID)
+#define CONFIG_SYS_OSCIN_FREQ		24000000
+#define CONFIG_SYS_TIMERBASE		DAVINCI_TIMER0_BASE
+#define CONFIG_SYS_HZ_CLOCK		clk_get(DAVINCI_AUXCLK_CLKID)
+#define CONFIG_SYS_HZ			1000
+#define CONFIG_SKIP_LOWLEVEL_INIT
+#define CONFIG_SKIP_RELOCATE_UBOOT	/* to a proper address, init done */
+
+/*
+ * Memory Info
+ */
+#define CONFIG_SYS_MALLOC_LEN	(0x10000 + 1*1024*1024) /* malloc() len */
+#define CONFIG_SYS_GBL_DATA_SIZE	128 /* reserved for initial data */
+#define PHYS_SDRAM_1		DAVINCI_DDR_EMIF_DATA_BASE /* DDR Start */
+#define PHYS_SDRAM_1_SIZE	(64 << 20) /* SDRAM size 64MB */
+#define CONFIG_SYS_MEMTEST_START	PHYS_SDRAM_1 + 0x2000000 /* memtest start addr */
+#define CONFIG_SYS_MEMTEST_END 	(PHYS_SDRAM_1 + 0x2000000 + 16*1024*1024) /* 16MB test */
+#define CONFIG_NR_DRAM_BANKS	1 /* we have 1 bank of DRAM */
+#define CONFIG_STACKSIZE	(256*1024) /* regular stack */
+
+/*
+ * Serial Driver info
+ */
+#define CONFIG_SYS_NS16550
+#define CONFIG_SYS_NS16550_SERIAL
+#define CONFIG_SYS_NS16550_REG_SIZE	-4	/* NS16550 register size */
+#define CONFIG_SYS_NS16550_COM1	DAVINCI_UART2_BASE /* Base address of UART2 */
+#define CONFIG_SYS_NS16550_CLK	clk_get(DAVINCI_UART2_CLKID)
+#define CONFIG_CONS_INDEX	1		/* use UART0 for console */
+#define CONFIG_BAUDRATE		115200		/* Default baud rate */
+#define CONFIG_SYS_BAUDRATE_TABLE	{ 9600, 19200, 38400, 57600, 115200 }
+
+/*
+ * I2C Configuration
+ */
+#define CONFIG_HARD_I2C
+#define CONFIG_DRIVER_DAVINCI_I2C
+#define CONFIG_SYS_I2C_SPEED		25000
+#define CONFIG_SYS_I2C_SLAVE		10 /* Bogus, master-only in U-Boot */
+
+/*
+ * U-Boot general configuration
+ */
+#define CONFIG_BOOTFILE		"uImage" /* Boot file name */
+#define CONFIG_SYS_PROMPT	"DA850-evm > " /* Command Prompt */
+#define CONFIG_SYS_CBSIZE	1024 /* Console I/O Buffer Size	*/
+#define CONFIG_SYS_PBSIZE	(CONFIG_SYS_CBSIZE+sizeof(CONFIG_SYS_PROMPT)+16)
+#define CONFIG_SYS_MAXARGS	16 /* max number of command args */
+#define CONFIG_SYS_BARGSIZE	CONFIG_SYS_CBSIZE /* Boot Args Buffer Size */
+#define CONFIG_SYS_LOAD_ADDR	(PHYS_SDRAM_1 + 0x700000)
+#define CONFIG_VERSION_VARIABLE
+#define CONFIG_AUTO_COMPLETE
+#define CONFIG_SYS_HUSH_PARSER
+#define CONFIG_SYS_PROMPT_HUSH_PS2	"> "
+#define CONFIG_CMDLINE_EDITING
+#define CONFIG_SYS_LONGHELP
+#define CONFIG_CRC32_VERIFY
+#define CONFIG_MX_CYCLIC
+
+/*
+ * Linux Information
+ */
+#define LINUX_BOOT_PARAM_ADDR	(CONFIG_SYS_MEMTEST_START + 0x100)
+#define CONFIG_CMDLINE_TAG
+#define CONFIG_SETUP_MEMORY_TAGS
+#define CONFIG_BOOTARGS		\
+	"mem=32M console=ttyS2,115200n8 root=/dev/mtdblock2 rw noinitrd ip=dhcp"
+#define CONFIG_BOOTDELAY	3
+
+/*
+ * U-Boot commands
+ */
+#include <config_cmd_default.h>
+#define CONFIG_CMD_ENV
+#define CONFIG_CMD_ASKENV
+#define CONFIG_CMD_DHCP
+#define CONFIG_CMD_DIAG
+#define CONFIG_CMD_MII
+#define CONFIG_CMD_PING
+#define CONFIG_CMD_SAVES
+#define CONFIG_CMD_MEMORY
+
+#ifndef CONFIG_DRIVER_TI_EMAC
+#undef CONFIG_CMD_NET
+#undef CONFIG_CMD_DHCP
+#undef CONFIG_CMD_MII
+#undef CONFIG_CMD_PING
+#endif
+
+#if !defined(CONFIG_USE_NAND) && \
+	!defined(CONFIG_USE_NOR) && \
+	!defined(CONFIG_USE_SPIFLASH)
+#define CONFIG_ENV_IS_NOWHERE
+#define CONFIG_SYS_NO_FLASH
+#define CONFIG_ENV_SIZE		(16 << 10)
+#undef CONFIG_CMD_IMLS
+#undef CONFIG_CMD_ENV
+#endif
+
+#endif /* __CONFIG_H */
-- 
1.5.6

[U-Boot] [PATCH 3/3] TI: DaVinci: Create configuration file for da850 EVM

[Wolfgang Denk]

Dear Sudhakar Rajashekhara,

In message <1272445230-427-1-git-send-email-sudhakar.raj@ti.com> you wrote:
> Provides initial support for TI OMAP-L138/DA850 SoC devices
> on a Logic PD EVM board.
> 
> Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
> ---
>  MAINTAINERS                |    4 +
>  MAKEALL                    |    1 +
>  Makefile                   |    3 +
>  include/configs/da850evm.h |  135 ++++++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 143 insertions(+), 0 deletions(-)
>  create mode 100644 include/configs/da850evm.h

Patches 2 and 3 should be merged into one, please.

> diff --git a/MAINTAINERS b/MAINTAINERS
> index a5d5835..c52803d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -758,6 +758,10 @@ Nick Thompson <nick.thompson@gefanuc.com>
>  
>  	da830evm	ARM926EJS (DA830/OMAP-L137)
>  
> +Sudhakar Rajashekhara <sudhakar.raj@ti.com>
> +
> +	da850evm	ARM926EJS (DA850/OMAP-L138)
> +
>  Albin Tonnerre <albin.tonnerre@free-electrons.com>

Please keep list sorted.

> diff --git a/Makefile b/Makefile
> index 393b180..0f21db6 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2913,6 +2913,9 @@ cp922_XA10_config	\
>  cp1026_config: unconfig
>  	@board/armltd/integrator/split_by_variant.sh cp $@
>  
> +da850evm_config:	unconfig
> +	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
> +
>  da830evm_config:	unconfig
>  	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci

Please keep list sorted, and use just a single entry, i. e.:

	da830evm_config	\
	da850evm_config:	unconfig
		@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci

> diff --git a/include/configs/da850evm.h b/include/configs/da850evm.h
> new file mode 100644
> index 0000000..2fcc76d
> --- /dev/null
> +++ b/include/configs/da850evm.h
...
> +/*
> + * Memory Info
> + */
> +#define CONFIG_SYS_MALLOC_LEN	(0x10000 + 1*1024*1024) /* malloc() len */
> +#define CONFIG_SYS_GBL_DATA_SIZE	128 /* reserved for initial data */
> +#define PHYS_SDRAM_1		DAVINCI_DDR_EMIF_DATA_BASE /* DDR Start */
> +#define PHYS_SDRAM_1_SIZE	(64 << 20) /* SDRAM size 64MB */
> +#define CONFIG_SYS_MEMTEST_START	PHYS_SDRAM_1 + 0x2000000 /* memtest start addr */
> +#define CONFIG_SYS_MEMTEST_END 	(PHYS_SDRAM_1 + 0x2000000 + 16*1024*1024) /* 16MB test */

Line too long.


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: wd at denx.de
The reasonable man adapts himself to the world; the unreasonable  one
persists  in  trying  to  adapt  the  world to himself. Therefore all
progress depends on the unreasonable man."      - George Bernard Shaw

[U-Boot] [PATCH 3/3] TI: DaVinci: Create configuration file for da850 EVM

[Sudhakar Rajashekhara]

Hi,

On Fri, May 07, 2010 at 03:36:49, Wolfgang Denk wrote:
> Dear Sudhakar Rajashekhara,
> 
> In message <1272445230-427-1-git-send-email-sudhakar.raj@ti.com> you wrote:
> > Provides initial support for TI OMAP-L138/DA850 SoC devices
> > on a Logic PD EVM board.
> > 
> > Signed-off-by: Sudhakar Rajashekhara <sudhakar.raj@ti.com>
> > ---
> >  MAINTAINERS                |    4 +
> >  MAKEALL                    |    1 +
> >  Makefile                   |    3 +
> >  include/configs/da850evm.h |  135 ++++++++++++++++++++++++++++++++++++++++++++
> >  4 files changed, 143 insertions(+), 0 deletions(-)
> >  create mode 100644 include/configs/da850evm.h
> 
> Patches 2 and 3 should be merged into one, please.
> 

Ok, I'll merge them.

> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index a5d5835..c52803d 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -758,6 +758,10 @@ Nick Thompson <nick.thompson@gefanuc.com>
> >  
> >  	da830evm	ARM926EJS (DA830/OMAP-L137)
> >  
> > +Sudhakar Rajashekhara <sudhakar.raj@ti.com>
> > +
> > +	da850evm	ARM926EJS (DA850/OMAP-L138)
> > +
> >  Albin Tonnerre <albin.tonnerre@free-electrons.com>
> 
> Please keep list sorted.
> 

Ok.

> > diff --git a/Makefile b/Makefile
> > index 393b180..0f21db6 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -2913,6 +2913,9 @@ cp922_XA10_config	\
> >  cp1026_config: unconfig
> >  	@board/armltd/integrator/split_by_variant.sh cp $@
> >  
> > +da850evm_config:	unconfig
> > +	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
> > +
> >  da830evm_config:	unconfig
> >  	@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
> 
> Please keep list sorted, and use just a single entry, i. e.:
> 

Ok.

> 	da830evm_config	\
> 	da850evm_config:	unconfig
> 		@$(MKCONFIG) $(@:_config=) arm arm926ejs da8xxevm davinci davinci
> 
> > diff --git a/include/configs/da850evm.h b/include/configs/da850evm.h
> > new file mode 100644
> > index 0000000..2fcc76d
> > --- /dev/null
> > +++ b/include/configs/da850evm.h
> ...
> > +/*
> > + * Memory Info
> > + */
> > +#define CONFIG_SYS_MALLOC_LEN	(0x10000 + 1*1024*1024) /* malloc() len */
> > +#define CONFIG_SYS_GBL_DATA_SIZE	128 /* reserved for initial data */
> > +#define PHYS_SDRAM_1		DAVINCI_DDR_EMIF_DATA_BASE /* DDR Start */
> > +#define PHYS_SDRAM_1_SIZE	(64 << 20) /* SDRAM size 64MB */
> > +#define CONFIG_SYS_MEMTEST_START	PHYS_SDRAM_1 + 0x2000000 /* memtest start addr */
> > +#define CONFIG_SYS_MEMTEST_END 	(PHYS_SDRAM_1 + 0x2000000 + 16*1024*1024) /* 16MB test */
> 
> Line too long.
> 

I'll address the review comments and will submit the updated version.

Thanks,
Sudhakar