[U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7

public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed

* [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header
@ 2015-07-18 16:40 Nishanth Menon
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header Nishanth Menon
                   ` (7 more replies)
  0 siblings, 8 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot


For legacy reasons, keystone2 had kind of stayed completely seperated
from the rest of the TI SoC families. This makes no sense since there
are a lot of common definitions that can be reused between all SoC
configuration allowing for a consistent user experience.

So make keystone2 follow the rest of the TI SoCs based on OMAP
architecture as well.

Test log: http://pastebin.ubuntu.com/11899472/ using ( http://pastebin.ubuntu.com/11899472/)

Changes in V3:
	- Cleanup MMC as part of review comments
	- Started using common addresses and got rid of fdt_addr_high.
	- Additional consolidation
	- picked up acks from previous patches

V2: http://marc.info/?l=u-boot&m=143707376106583&w=2
V1: http://marc.info/?l=u-boot&m=143706008602124&w=2

Nishanth Menon (8):
  configs: split ti_armv7_common into a omap generic header
  board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR
  configs: rename ks2_evm into ti_armv7_keystone2
  configs: ti_armv7_keystone2: start using armv7_common
  configs: ti_armv7_keystone2: switch addresses to generic addresses
  ti_armv7_keystone2: switch to using kernel zImage
  configs: k2*_evm: rename skernel binary names to generated file names
  configs: keystone2: remove distro specific ramdisk name

 board/ti/ks2_evm/board.c                           |  2 +-
 include/configs/am43xx_evm.h                       |  2 +-
 include/configs/k2e_evm.h                          |  9 +--
 include/configs/k2hk_evm.h                         |  9 +--
 include/configs/k2l_evm.h                          |  9 +--
 include/configs/ti_am335x_common.h                 |  2 +-
 include/configs/ti_armv7_common.h                  | 26 ++-----
 .../configs/{ks2_evm.h => ti_armv7_keystone2.h}    | 82 ++++++++--------------
 include/configs/ti_armv7_omap.h                    | 49 +++++++++++++
 include/configs/ti_omap3_common.h                  |  2 +-
 include/configs/ti_omap4_common.h                  |  2 +-
 include/configs/ti_omap5_common.h                  |  2 +-
 12 files changed, 97 insertions(+), 99 deletions(-)
 rename include/configs/{ks2_evm.h => ti_armv7_keystone2.h} (82%)
 create mode 100644 include/configs/ti_armv7_omap.h

-- 
2.1.4

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:35   ` Tom Rini
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR Nishanth Menon
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

TI armv7 based SoCs are based on two architectures - one based on OMAP
generation architecture and others based on Keystone architecture.

Many of the options are architecture specific, however a lot are common
with v7 architecture. So, step 1 will be to move out OMAP specific stuff
from ti_armv7_common into a ti_armv7_omap.h header which is then used
by all the relevant architecture headers.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
V3: No change
V2: https://patchwork.ozlabs.org/patch/496820/
V1: https://patchwork.ozlabs.org/patch/496726/

 include/configs/am43xx_evm.h       |  2 +-
 include/configs/ti_am335x_common.h |  2 +-
 include/configs/ti_armv7_common.h  | 21 ----------------
 include/configs/ti_armv7_omap.h    | 49 ++++++++++++++++++++++++++++++++++++++
 include/configs/ti_omap3_common.h  |  2 +-
 include/configs/ti_omap4_common.h  |  2 +-
 include/configs/ti_omap5_common.h  |  2 +-
 7 files changed, 54 insertions(+), 26 deletions(-)
 create mode 100644 include/configs/ti_armv7_omap.h

diff --git a/include/configs/am43xx_evm.h b/include/configs/am43xx_evm.h
index 33e534a76583..65ac8ef34eb8 100644
--- a/include/configs/am43xx_evm.h
+++ b/include/configs/am43xx_evm.h
@@ -80,7 +80,7 @@
 #endif
 
 /* Now bring in the rest of the common code. */
-#include <configs/ti_armv7_common.h>
+#include <configs/ti_armv7_omap.h>
 
 /* Always 64 KiB env size */
 #define CONFIG_ENV_SIZE			(64 << 10)
diff --git a/include/configs/ti_am335x_common.h b/include/configs/ti_am335x_common.h
index 4d2ae280e7f1..9697431b4496 100644
--- a/include/configs/ti_am335x_common.h
+++ b/include/configs/ti_am335x_common.h
@@ -94,6 +94,6 @@
 #endif
 
 /* Now bring in the rest of the common code. */
-#include <configs/ti_armv7_common.h>
+#include <configs/ti_armv7_omap.h>
 
 #endif	/* __CONFIG_TI_AM335X_COMMON_H__ */
diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
index 0aea7d12ec16..63244dbc83ff 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -18,8 +18,6 @@
 #define __CONFIG_TI_ARMV7_COMMON_H__
 
 /* Common define for many platforms. */
-#define CONFIG_OMAP
-#define CONFIG_OMAP_COMMON
 #define CONFIG_SYS_GENERIC_BOARD
 
 /*
@@ -86,39 +84,20 @@
 #define CONFIG_I2C
 #define CONFIG_CMD_I2C
 #define CONFIG_SYS_I2C
-#define CONFIG_SYS_OMAP24_I2C_SPEED	100000
-#define CONFIG_SYS_OMAP24_I2C_SLAVE	1
-#define CONFIG_SYS_I2C_OMAP24XX
 
 /* MMC/SD IP block */
 #define CONFIG_MMC
 #define CONFIG_GENERIC_MMC
-#define CONFIG_OMAP_HSMMC
 #define CONFIG_CMD_MMC
 
 /* McSPI IP block */
 #define CONFIG_SPI
-#define CONFIG_OMAP3_SPI
 #define CONFIG_CMD_SPI
 
 /* GPIO block */
-#define CONFIG_OMAP_GPIO
 #define CONFIG_CMD_GPIO
 
 /*
- * GPMC NAND block.  We support 1 device and the physical address to
- * access CS0 at is 0x8000000.
- */
-#ifdef CONFIG_NAND
-#define CONFIG_NAND_OMAP_GPMC
-#ifndef CONFIG_SYS_NAND_BASE
-#define CONFIG_SYS_NAND_BASE		0x8000000
-#endif
-#define CONFIG_SYS_MAX_NAND_DEVICE	1
-#define CONFIG_CMD_NAND
-#endif
-
-/*
  * The following are general good-enough settings for U-Boot.  We set a
  * large malloc pool as we generally have a lot of DDR, and we opt for
  * function over binary size in the main portion of U-Boot as this is
diff --git a/include/configs/ti_armv7_omap.h b/include/configs/ti_armv7_omap.h
new file mode 100644
index 000000000000..7548170afcf7
--- /dev/null
+++ b/include/configs/ti_armv7_omap.h
@@ -0,0 +1,49 @@
+/*
+ * ti_armv7_omap.h
+ *
+ * Copyright (C) 2015 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * SPDX-License-Identifier:	GPL-2.0+
+ *
+ * The various ARMv7 SoCs from TI all share a number of IP blocks when
+ * implementing a given feature. This is meant to isolate the features
+ * that are based on OMAP architecture.
+ */
+#ifndef __CONFIG_TI_ARMV7_OMAP_H__
+#define __CONFIG_TI_ARMV7_OMAP_H__
+
+/* Common defines for all OMAP architecture based SoCs */
+#define CONFIG_OMAP
+#define CONFIG_OMAP_COMMON
+
+/* I2C IP block */
+#define CONFIG_SYS_OMAP24_I2C_SPEED	100000
+#define CONFIG_SYS_OMAP24_I2C_SLAVE	1
+#define CONFIG_SYS_I2C_OMAP24XX
+
+/* MMC/SD IP block */
+#define CONFIG_OMAP_HSMMC
+
+/* SPI IP Block */
+#define CONFIG_OMAP3_SPI
+
+/* GPIO block */
+#define CONFIG_OMAP_GPIO
+
+/*
+ * GPMC NAND block.  We support 1 device and the physical address to
+ * access CS0 at is 0x8000000.
+ */
+#ifdef CONFIG_NAND
+#define CONFIG_NAND_OMAP_GPMC
+#ifndef CONFIG_SYS_NAND_BASE
+#define CONFIG_SYS_NAND_BASE		0x8000000
+#endif
+#define CONFIG_SYS_MAX_NAND_DEVICE	1
+#define CONFIG_CMD_NAND
+#endif
+
+/* Now for the remaining common defines */
+#include <configs/ti_armv7_common.h>
+
+#endif /* __CONFIG_TI_ARMV7_OMAP_H__ */
diff --git a/include/configs/ti_omap3_common.h b/include/configs/ti_omap3_common.h
index 429b109afa23..be231a551361 100644
--- a/include/configs/ti_omap3_common.h
+++ b/include/configs/ti_omap3_common.h
@@ -84,6 +84,6 @@
 #endif
 
 /* Now bring in the rest of the common code. */
-#include <configs/ti_armv7_common.h>
+#include <configs/ti_armv7_omap.h>
 
 #endif	/* __CONFIG_TI_OMAP3_COMMON_H__ */
diff --git a/include/configs/ti_omap4_common.h b/include/configs/ti_omap4_common.h
index e96613406b3b..959f312edb5b 100644
--- a/include/configs/ti_omap4_common.h
+++ b/include/configs/ti_omap4_common.h
@@ -52,7 +52,7 @@
 #define CONFIG_SYS_DEFAULT_LPDDR2_TIMINGS
 #endif
 
-#include <configs/ti_armv7_common.h>
+#include <configs/ti_armv7_omap.h>
 
 /*
  * Hardware drivers
diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
index f3e5a7587de9..789542be6710 100644
--- a/include/configs/ti_omap5_common.h
+++ b/include/configs/ti_omap5_common.h
@@ -47,7 +47,7 @@
 #include <asm/arch/cpu.h>
 #include <asm/arch/omap.h>
 
-#include <configs/ti_armv7_common.h>
+#include <configs/ti_armv7_omap.h>
 
 /*
  * Hardware drivers
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header Nishanth Menon
@ 2015-07-22 15:35   ` Tom Rini
  0 siblings, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:29AM -0500, Nishanth Menon wrote:

> TI armv7 based SoCs are based on two architectures - one based on OMAP
> generation architecture and others based on Keystone architecture.
> 
> Many of the options are architecture specific, however a lot are common
> with v7 architecture. So, step 1 will be to move out OMAP specific stuff
> from ti_armv7_common into a ti_armv7_omap.h header which is then used
> by all the relevant architecture headers.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/70262c6f/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:35   ` Tom Rini
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
                   ` (5 subsequent siblings)
  7 siblings, 1 reply; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

CONFIG_LINUX_BOOT_PARAM_ADDR is not a valid configuration option. Do
just like what the rest of the world does.

Acked-by: Vitaly Andrianov <vitalya@ti.com>
Acked-By: Murali Karicheri <m-karicheri2@ti.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
V3: picked up acks - else no change
V2: https://patchwork.ozlabs.org/patch/496821/ (new patch)

 board/ti/ks2_evm/board.c  | 2 +-
 include/configs/ks2_evm.h | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/board/ti/ks2_evm/board.c b/board/ti/ks2_evm/board.c
index 8892a2843df4..0cefb3404403 100644
--- a/board/ti/ks2_evm/board.c
+++ b/board/ti/ks2_evm/board.c
@@ -48,7 +48,7 @@ int dram_init(void)
 
 int board_init(void)
 {
-	gd->bd->bi_boot_params = CONFIG_LINUX_BOOT_PARAM_ADDR;
+	gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100;
 
 	return 0;
 }
diff --git a/include/configs/ks2_evm.h b/include/configs/ks2_evm.h
index e05d56cd8262..d838f270018b 100644
--- a/include/configs/ks2_evm.h
+++ b/include/configs/ks2_evm.h
@@ -307,7 +307,6 @@
 #define CONFIG_OF_BOARD_SETUP
 #define CONFIG_SYS_BARGSIZE		1024
 #define CONFIG_SYS_LOAD_ADDR		(CONFIG_SYS_SDRAM_BASE + 0x08000000)
-#define CONFIG_LINUX_BOOT_PARAM_ADDR	(CONFIG_SYS_SDRAM_BASE + 0x100)
 
 #define CONFIG_SUPPORT_RAW_INITRD
 
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR Nishanth Menon
@ 2015-07-22 15:35   ` Tom Rini
  0 siblings, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:30AM -0500, Nishanth Menon wrote:

> CONFIG_LINUX_BOOT_PARAM_ADDR is not a valid configuration option. Do
> just like what the rest of the world does.
> 
> Acked-by: Vitaly Andrianov <vitalya@ti.com>
> Acked-By: Murali Karicheri <m-karicheri2@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/76d9a50d/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header Nishanth Menon
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:23   ` Murali Karicheri
                     ` (2 more replies)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common Nishanth Menon
                   ` (4 subsequent siblings)
  7 siblings, 3 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

rename the keystone2 common header into an keystone2 architecture
specific header which can then reuse the common ti_armv7 config headers.

Acked-by: Vitaly Andrianov <vitalya@ti.com>
Acked-By: Murali Karicheri <m-karicheri2@ti.com>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
V3: picked up acks, no change.
V2: https://patchwork.ozlabs.org/patch/496822/ (no change)
V1: https://patchwork.ozlabs.org/patch/496728/
 include/configs/k2e_evm.h                           | 2 +-
 include/configs/k2hk_evm.h                          | 2 +-
 include/configs/k2l_evm.h                           | 2 +-
 include/configs/{ks2_evm.h => ti_armv7_keystone2.h} | 0
 4 files changed, 3 insertions(+), 3 deletions(-)
 rename include/configs/{ks2_evm.h => ti_armv7_keystone2.h} (100%)

diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
index a28ceb7064a7..ac50a01b2980 100644
--- a/include/configs/k2e_evm.h
+++ b/include/configs/k2e_evm.h
@@ -27,7 +27,7 @@
 	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
 	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
 
-#include <configs/ks2_evm.h>
+#include <configs/ti_armv7_keystone2.h>
 
 /* SPL SPI Loader Configuration */
 #define CONFIG_SPL_TEXT_BASE           0x0c100000
diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
index eae772178338..29e3403aa082 100644
--- a/include/configs/k2hk_evm.h
+++ b/include/configs/k2hk_evm.h
@@ -27,7 +27,7 @@
 	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
 	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
 
-#include <configs/ks2_evm.h>
+#include <configs/ti_armv7_keystone2.h>
 
 /* SPL SPI Loader Configuration */
 #define CONFIG_SPL_TEXT_BASE		0x0c200000
diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
index 57da0579255a..50d5c991a0bb 100644
--- a/include/configs/k2l_evm.h
+++ b/include/configs/k2l_evm.h
@@ -27,7 +27,7 @@
 	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
 	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
 
-#include <configs/ks2_evm.h>
+#include <configs/ti_armv7_keystone2.h>
 
 /* SPL SPI Loader Configuration */
 #define CONFIG_SPL_TEXT_BASE		0x0c100000
diff --git a/include/configs/ks2_evm.h b/include/configs/ti_armv7_keystone2.h
similarity index 100%
rename from include/configs/ks2_evm.h
rename to include/configs/ti_armv7_keystone2.h
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
@ 2015-07-22 15:23   ` Murali Karicheri
  2015-07-22 15:24   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  2 siblings, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:23 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> rename the keystone2 common header into an keystone2 architecture
> specific header which can then reuse the common ti_armv7 config headers.
>
> Acked-by: Vitaly Andrianov <vitalya@ti.com>
> Acked-By: Murali Karicheri <m-karicheri2@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> V3: picked up acks, no change.
> V2: https://patchwork.ozlabs.org/patch/496822/ (no change)
> V1: https://patchwork.ozlabs.org/patch/496728/
>   include/configs/k2e_evm.h                           | 2 +-
>   include/configs/k2hk_evm.h                          | 2 +-
>   include/configs/k2l_evm.h                           | 2 +-
>   include/configs/{ks2_evm.h => ti_armv7_keystone2.h} | 0
>   4 files changed, 3 insertions(+), 3 deletions(-)
>   rename include/configs/{ks2_evm.h => ti_armv7_keystone2.h} (100%)
>
> diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
> index a28ceb7064a7..ac50a01b2980 100644
> --- a/include/configs/k2e_evm.h
> +++ b/include/configs/k2e_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE           0x0c100000
> diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
> index eae772178338..29e3403aa082 100644
> --- a/include/configs/k2hk_evm.h
> +++ b/include/configs/k2hk_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE		0x0c200000
> diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
> index 57da0579255a..50d5c991a0bb 100644
> --- a/include/configs/k2l_evm.h
> +++ b/include/configs/k2l_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE		0x0c100000
> diff --git a/include/configs/ks2_evm.h b/include/configs/ti_armv7_keystone2.h
> similarity index 100%
> rename from include/configs/ks2_evm.h
> rename to include/configs/ti_armv7_keystone2.h
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
  2015-07-22 15:23   ` Murali Karicheri
@ 2015-07-22 15:24   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  2 siblings, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:24 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> rename the keystone2 common header into an keystone2 architecture
> specific header which can then reuse the common ti_armv7 config headers.
>
> Acked-by: Vitaly Andrianov <vitalya@ti.com>
> Acked-By: Murali Karicheri <m-karicheri2@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> V3: picked up acks, no change.
> V2: https://patchwork.ozlabs.org/patch/496822/ (no change)
> V1: https://patchwork.ozlabs.org/patch/496728/
>   include/configs/k2e_evm.h                           | 2 +-
>   include/configs/k2hk_evm.h                          | 2 +-
>   include/configs/k2l_evm.h                           | 2 +-
>   include/configs/{ks2_evm.h => ti_armv7_keystone2.h} | 0
>   4 files changed, 3 insertions(+), 3 deletions(-)
>   rename include/configs/{ks2_evm.h => ti_armv7_keystone2.h} (100%)
>
> diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
> index a28ceb7064a7..ac50a01b2980 100644
> --- a/include/configs/k2e_evm.h
> +++ b/include/configs/k2e_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE           0x0c100000
> diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
> index eae772178338..29e3403aa082 100644
> --- a/include/configs/k2hk_evm.h
> +++ b/include/configs/k2hk_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE		0x0c200000
> diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
> index 57da0579255a..50d5c991a0bb 100644
> --- a/include/configs/k2l_evm.h
> +++ b/include/configs/k2l_evm.h
> @@ -27,7 +27,7 @@
>   	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
>
> -#include <configs/ks2_evm.h>
> +#include <configs/ti_armv7_keystone2.h>
>
>   /* SPL SPI Loader Configuration */
>   #define CONFIG_SPL_TEXT_BASE		0x0c100000
> diff --git a/include/configs/ks2_evm.h b/include/configs/ti_armv7_keystone2.h
> similarity index 100%
> rename from include/configs/ks2_evm.h
> rename to include/configs/ti_armv7_keystone2.h
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
  2015-07-22 15:23   ` Murali Karicheri
  2015-07-22 15:24   ` Murali Karicheri
@ 2015-07-22 15:35   ` Tom Rini
  2 siblings, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:31AM -0500, Nishanth Menon wrote:

> rename the keystone2 common header into an keystone2 architecture
> specific header which can then reuse the common ti_armv7 config headers.
> 
> Acked-by: Vitaly Andrianov <vitalya@ti.com>
> Acked-By: Murali Karicheri <m-karicheri2@ti.com>
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/6caaeb23/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
                   ` (2 preceding siblings ...)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:19   ` Murali Karicheri
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses Nishanth Menon
                   ` (3 subsequent siblings)
  7 siblings, 1 reply; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

Try to maintain as much commonality by conditionally including stuff
in armv7_common as necessary and removing the common defines from
keystone2 header.

Note: as part of this change, all keystone2 platforms will now start
using the generic u-boot prompt instead of the custom prompt.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
Changes in V3:
	- undefed MMC from ks2
	- removed white space introduced (review comments)
	- address change review comment is split off as patch 5

V2: https://patchwork.ozlabs.org/patch/496823/
V1: https://patchwork.ozlabs.org/patch/496730/

 include/configs/k2e_evm.h            |  2 --
 include/configs/k2hk_evm.h           |  2 --
 include/configs/k2l_evm.h            |  2 --
 include/configs/ti_armv7_common.h    |  5 +++-
 include/configs/ti_armv7_keystone2.h | 55 ++++++++++++------------------------
 5 files changed, 22 insertions(+), 44 deletions(-)

diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
index ac50a01b2980..f1e650141ae1 100644
--- a/include/configs/k2e_evm.h
+++ b/include/configs/k2e_evm.h
@@ -15,8 +15,6 @@
 #define CONFIG_K2E_EVM
 
 /* U-Boot general configuration */
-#define CONFIG_SYS_PROMPT               "K2E EVM # "
-
 #define CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
 	"addr_mon=0x0c140000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
index 29e3403aa082..f8e83de64b63 100644
--- a/include/configs/k2hk_evm.h
+++ b/include/configs/k2hk_evm.h
@@ -15,8 +15,6 @@
 #define CONFIG_K2HK_EVM
 
 /* U-Boot general configuration */
-#define CONFIG_SYS_PROMPT               "K2HK EVM # "
-
 #define CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
 	"addr_mon=0x0c5f0000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
index 50d5c991a0bb..395608a5f6db 100644
--- a/include/configs/k2l_evm.h
+++ b/include/configs/k2l_evm.h
@@ -15,8 +15,6 @@
 #define CONFIG_K2L_EVM
 
 /* U-Boot general configuration */
-#define CONFIG_SYS_PROMPT		"K2L EVM # "
-
 #define CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
 	"addr_mon=0x0c140000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
diff --git a/include/configs/ti_armv7_common.h b/include/configs/ti_armv7_common.h
index 63244dbc83ff..68bfed28820c 100644
--- a/include/configs/ti_armv7_common.h
+++ b/include/configs/ti_armv7_common.h
@@ -74,8 +74,11 @@
 #define CONFIG_NR_DRAM_BANKS		1
 #endif
 #define CONFIG_SYS_SDRAM_BASE		0x80000000
+
+#ifndef CONFIG_SYS_INIT_SP_ADDR
 #define CONFIG_SYS_INIT_SP_ADDR         (NON_SECURE_SRAM_END - \
 						GENERATED_GBL_DATA_SIZE)
+#endif
 
 /* Timer information. */
 #define CONFIG_SYS_PTV			2	/* Divisor: 2^(PTV+1) => 8 */
@@ -140,7 +143,7 @@
  * mtdparts, both for ease of use in U-Boot and for passing information
  * on to the Linux kernel.
  */
-#if defined(CONFIG_SPI_BOOT) || defined(CONFIG_NOR) || defined(CONFIG_NAND)
+#if defined(CONFIG_SPI_BOOT) || defined(CONFIG_NOR) || defined(CONFIG_NAND) || defined(CONFIG_NAND_DAVINCI)
 #define CONFIG_MTD_DEVICE		/* Required for mtdparts */
 #define CONFIG_CMD_MTDPARTS
 #endif
diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index d838f270018b..0e67cf0ebd75 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -14,10 +14,7 @@
 
 /* U-Boot Build Configuration */
 #define CONFIG_SKIP_LOWLEVEL_INIT	/* U-Boot is a 2nd stage loader */
-#define CONFIG_SYS_NO_FLASH		/* that is, no *NOR* flash */
-#define CONFIG_SYS_CONSOLE_INFO_QUIET
 #define CONFIG_BOARD_EARLY_INIT_F
-#define CONFIG_SYS_THUMB_BUILD
 
 /* SoC Configuration */
 #define CONFIG_ARCH_CPU_INIT
@@ -28,11 +25,9 @@
 
 /* Memory Configuration */
 #define CONFIG_NR_DRAM_BANKS		2
-#define CONFIG_SYS_SDRAM_BASE		0x80000000
 #define CONFIG_SYS_LPAE_SDRAM_BASE	0x800000000
 #define CONFIG_MAX_RAM_BANK_SIZE	(2 << 30)       /* 2GB */
 #define CONFIG_STACKSIZE		(512 << 10)     /* 512 KiB */
-#define CONFIG_SYS_MALLOC_LEN		(4 << 20)       /* 4 MiB */
 #define CONFIG_SYS_INIT_SP_ADDR		(CONFIG_SYS_TEXT_BASE - \
 					GENERATED_GBL_DATA_SIZE)
 
@@ -49,15 +44,10 @@
 #define CONFIG_SPL_STACK		(CONFIG_SYS_SPL_MALLOC_START + \
 					CONFIG_SYS_SPL_MALLOC_SIZE + \
 					CONFIG_SPL_STACK_SIZE - 4)
-#define CONFIG_SPL_LIBCOMMON_SUPPORT
-#define CONFIG_SPL_LIBGENERIC_SUPPORT
-#define CONFIG_SPL_SERIAL_SUPPORT
 #define CONFIG_SPL_SPI_FLASH_SUPPORT
 #define CONFIG_SPL_SPI_SUPPORT
-#define CONFIG_SPL_BOARD_INIT
 #define CONFIG_SPL_SPI_LOAD
 #define CONFIG_SYS_SPI_U_BOOT_OFFS	CONFIG_SPL_PAD_TO
-#define CONFIG_SPL_FRAMEWORK
 
 /* UART Configuration */
 #define CONFIG_SYS_NS16550
@@ -68,13 +58,10 @@
 #define CONFIG_SYS_NS16550_COM2		KS2_UART1_BASE
 #define CONFIG_SYS_NS16550_CLK		clk_get_rate(KS2_CLK1_6)
 #define CONFIG_CONS_INDEX		1
-#define CONFIG_BAUDRATE			115200
 
 /* SPI Configuration */
-#define CONFIG_SPI
 #define CONFIG_SPI_FLASH_STMICRO
 #define CONFIG_DAVINCI_SPI
-#define CONFIG_CMD_SPI
 #define CONFIG_SYS_SPI_CLK		clk_get_rate(KS2_CLK1_6)
 #define CONFIG_SF_DEFAULT_SPEED		30000000
 #define CONFIG_ENV_SPI_MAX_HZ		CONFIG_SF_DEFAULT_SPEED
@@ -148,7 +135,6 @@
 #define CONFIG_AEMIF_CNTRL_BASE		KS2_AEMIF_CNTRL_BASE
 
 /* I2C Configuration */
-#define CONFIG_SYS_I2C
 #define CONFIG_SYS_I2C_DAVINCI
 #define CONFIG_SYS_DAVINCI_I2C_SPEED	100000
 #define CONFIG_SYS_DAVINCI_I2C_SLAVE	0x10 /* SMBus host address */
@@ -185,7 +171,6 @@
 #define CONFIG_ENV_IS_IN_NAND
 #define CONFIG_ENV_OFFSET			0x100000
 #define CONFIG_MTD_PARTITIONS
-#define CONFIG_MTD_DEVICE
 #define CONFIG_RBTREE
 #define CONFIG_LZO
 #define MTDIDS_DEFAULT			"nand0=davinci_nand.0"
@@ -197,8 +182,6 @@
 #define CONFIG_USB_XHCI
 #define CONFIG_USB_XHCI_KEYSTONE
 #define CONFIG_SYS_USB_XHCI_MAX_ROOT_PORTS	2
-#define CONFIG_USB_STORAGE
-#define CONFIG_DOS_PARTITION
 #define CONFIG_EFI_PARTITION
 #define CONFIG_FS_FAT
 #define CONFIG_SYS_CACHELINE_SIZE		64
@@ -208,39 +191,25 @@
 #define CONFIG_USB_PHY_CFG_BASE			KS2_USB_PHY_CFG_BASE
 
 /* U-Boot command configuration */
-#define CONFIG_CMD_ASKENV
 #define CONFIG_CMD_DHCP
-#define CONFIG_CMD_I2C
 #define CONFIG_CMD_PING
 #define CONFIG_CMD_SAVES
-#define CONFIG_CMD_MTDPARTS
 #define CONFIG_CMD_NAND
 #define CONFIG_CMD_UBI
 #define CONFIG_CMD_UBIFS
 #define CONFIG_CMD_SF
 #define CONFIG_CMD_EEPROM
 #define CONFIG_CMD_USB
-#define CONFIG_CMD_FAT
-#define CONFIG_CMD_FS_GENERIC
 
 /* U-Boot general configuration */
-#define CONFIG_SYS_GENERIC_BOARD
 #define CONFIG_MISC_INIT_R
-#define CONFIG_SYS_CBSIZE		1024
-#define CONFIG_SYS_PBSIZE		2048
-#define CONFIG_SYS_MAXARGS		16
-#define CONFIG_SYS_HUSH_PARSER
-#define CONFIG_SYS_LONGHELP
 #define CONFIG_CRC32_VERIFY
 #define CONFIG_MX_CYCLIC
-#define CONFIG_CMDLINE_EDITING
-#define CONFIG_VERSION_VARIABLE
 #define CONFIG_TIMESTAMP
 
 /* EDMA3 */
 #define CONFIG_TI_EDMA3
 
-#define CONFIG_BOOTDELAY		3
 #define CONFIG_BOOTFILE			"uImage"
 #define CONFIG_EXTRA_ENV_SETTINGS					\
 	CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
@@ -301,14 +270,26 @@
 #define CONFIG_BOOTARGS							\
 
 /* Linux interfacing */
-#define CONFIG_CMDLINE_TAG
-#define CONFIG_SETUP_MEMORY_TAGS
-#define CONFIG_OF_LIBFDT		1
 #define CONFIG_OF_BOARD_SETUP
-#define CONFIG_SYS_BARGSIZE		1024
-#define CONFIG_SYS_LOAD_ADDR		(CONFIG_SYS_SDRAM_BASE + 0x08000000)
 
-#define CONFIG_SUPPORT_RAW_INITRD
+/* Now for the remaining common defines */
+#include <configs/ti_armv7_common.h>
+
+/* We wont be loading up OS from SPL for now.. */
+#undef CONFIG_SPL_OS_BOOT
+
+/* We do not have MMC support.. yet.. */
+#undef CONFIG_SPL_LIBDISK_SUPPORT
+#undef CONFIG_SPL_MMC_SUPPORT
+#undef CONFIG_SPL_FAT_SUPPORT
+#undef CONFIG_SPL_EXT_SUPPORT
+#undef CONFIG_MMC
+#undef CONFIG_GENERIC_MMC
+#undef CONFIG_CMD_MMC
+
+/* And no support for GPIO, yet.. */
+#undef CONFIG_SPL_GPIO_SUPPORT
+#undef CONFIG_CMD_GPIO
 
 /* we may include files below only after all above definitions */
 #include <asm/arch/hardware.h>
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common Nishanth Menon
@ 2015-07-22 15:19   ` Murali Karicheri
  2015-07-22 15:28     ` Tom Rini
  0 siblings, 1 reply; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:19 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> Try to maintain as much commonality by conditionally including stuff
> in armv7_common as necessary and removing the common defines from
> keystone2 header.

>
>   /* U-Boot Build Configuration */
>   #define CONFIG_SKIP_LOWLEVEL_INIT	/* U-Boot is a 2nd stage loader */
> -#define CONFIG_SYS_NO_FLASH		/* that is, no *NOR* flash */
> -#define CONFIG_SYS_CONSOLE_INFO_QUIET
>   #define CONFIG_BOARD_EARLY_INIT_F
> -#define CONFIG_SYS_THUMB_BUILD

One thing I see is THUMB build is only enabled for SPL build in 
ti_armv7_common.h. With this transition, regular build will not be using 
THUMB build anymore.  This is not right. Please fix it. Many platforms 
configs such as tbs2910.h, colibri_vf.h, highbank.h etc has this enabled 
for both builds as this will reduce the image size.

>
>   /* SoC Configuration */

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common
  2015-07-22 15:19   ` Murali Karicheri
@ 2015-07-22 15:28     ` Tom Rini
  0 siblings, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:28 UTC (permalink / raw)
  To: u-boot

On Wed, Jul 22, 2015 at 11:19:29AM -0400, Murali Karicheri wrote:
> On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> >Try to maintain as much commonality by conditionally including stuff
> >in armv7_common as necessary and removing the common defines from
> >keystone2 header.
> 
> >
> >  /* U-Boot Build Configuration */
> >  #define CONFIG_SKIP_LOWLEVEL_INIT	/* U-Boot is a 2nd stage loader */
> >-#define CONFIG_SYS_NO_FLASH		/* that is, no *NOR* flash */
> >-#define CONFIG_SYS_CONSOLE_INFO_QUIET
> >  #define CONFIG_BOARD_EARLY_INIT_F
> >-#define CONFIG_SYS_THUMB_BUILD
> 
> One thing I see is THUMB build is only enabled for SPL build in
> ti_armv7_common.h. With this transition, regular build will not be
> using THUMB build anymore.  This is not right. Please fix it. Many
> platforms configs such as tbs2910.h, colibri_vf.h, highbank.h etc
> has this enabled for both builds as this will reduce the image size.

Good point.  Lets just whack thumb on for all platforms (so after the
patch to make things include ti_armv7_omap.h, add that in).  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/53f7eafa/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
                   ` (3 preceding siblings ...)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:22   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage Nishanth Menon
                   ` (2 subsequent siblings)
  7 siblings, 2 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

Use the defaults defined in DEFAULT_LINUX_BOOT_ENV

Signed-off-by: Nishanth Menon <nm@ti.com>
---
new patch based on review comments on https://patchwork.ozlabs.org/patch/496823/

 include/configs/ti_armv7_keystone2.h | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index 0e67cf0ebd75..ddc2e23f1370 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -212,30 +212,26 @@
 
 #define CONFIG_BOOTFILE			"uImage"
 #define CONFIG_EXTRA_ENV_SETTINGS					\
+	DEFAULT_LINUX_BOOT_ENV						\
 	CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
 	"boot=ubi\0"							\
 	"tftp_root=/\0"							\
 	"nfs_root=/export\0"						\
 	"mem_lpae=1\0"							\
 	"mem_reserve=512M\0"						\
-	"addr_fdt=0x87000000\0"						\
-	"addr_kern=0x88000000\0"					\
-	"addr_uboot=0x87000000\0"					\
-	"addr_fs=0x82000000\0"						\
 	"addr_ubi=0x82000000\0"						\
 	"addr_secdb_key=0xc000000\0"					\
-	"fdt_high=0xffffffff\0"						\
 	"name_kern=uImage-keystone-evm.bin\0"				\
 	"run_mon=mon_install ${addr_mon}\0"				\
-	"run_kern=bootm ${addr_kern} - ${addr_fdt}\0"			\
+	"run_kern=bootm ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
 		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
-	"get_fdt_net=dhcp ${addr_fdt} ${tftp_root}/${name_fdt}\0"	\
-	"get_fdt_ubi=ubifsload ${addr_fdt} ${name_fdt}\0"		\
-	"get_kern_net=dhcp ${addr_kern} ${tftp_root}/${name_kern}\0"	\
-	"get_kern_ubi=ubifsload ${addr_kern} ${name_kern}\0"		\
+	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_fdt_ubi=ubifsload ${fdtaddr} ${name_fdt}\0"		\
+	"get_kern_net=dhcp ${loadaddr} ${tftp_root}/${name_kern}\0"	\
+	"get_kern_ubi=ubifsload ${loadaddr} ${name_kern}\0"		\
 	"get_mon_net=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
 	"get_mon_ubi=ubifsload ${addr_mon} ${name_mon}\0"		\
 	"get_uboot_net=dhcp ${addr_uboot} ${tftp_root}/${name_uboot}\0"	\
@@ -248,10 +244,10 @@
 		"root=/dev/nfs rw nfsroot=${serverip}:${nfs_root},"	\
 		"${nfs_options} ip=dhcp\0"				\
 	"nfs_options=v3,tcp,rsize=4096,wsize=4096\0"			\
-	"get_fdt_ramfs=dhcp ${addr_fdt} ${tftp_root}/${name_fdt}\0"	\
-	"get_kern_ramfs=dhcp ${addr_kern} ${tftp_root}/${name_kern}\0"	\
+	"get_fdt_ramfs=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
+	"get_kern_ramfs=dhcp ${loadaddr} ${tftp_root}/${name_kern}\0"	\
 	"get_mon_ramfs=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
-	"get_fs_ramfs=dhcp ${addr_fs} ${tftp_root}/${name_fs}\0"	\
+	"get_fs_ramfs=dhcp ${rdaddr} ${tftp_root}/${name_fs}\0"	\
 	"get_ubi_net=dhcp ${addr_ubi} ${tftp_root}/${name_ubi}\0"	\
 	"burn_ubi=nand erase.part ubifs; "				\
 		"nand write ${addr_ubi} ubifs ${filesize}\0"		\
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses Nishanth Menon
@ 2015-07-22 15:22   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:22 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> Use the defaults defined in DEFAULT_LINUX_BOOT_ENV
>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> new patch based on review comments on https://patchwork.ozlabs.org/patch/496823/
>
>   include/configs/ti_armv7_keystone2.h | 22 +++++++++-------------
>   1 file changed, 9 insertions(+), 13 deletions(-)
>
> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
> index 0e67cf0ebd75..ddc2e23f1370 100644
> --- a/include/configs/ti_armv7_keystone2.h
> +++ b/include/configs/ti_armv7_keystone2.h
> @@ -212,30 +212,26 @@
>
>   #define CONFIG_BOOTFILE			"uImage"
>   #define CONFIG_EXTRA_ENV_SETTINGS					\
> +	DEFAULT_LINUX_BOOT_ENV						\
>   	CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
>   	"boot=ubi\0"							\
>   	"tftp_root=/\0"							\
>   	"nfs_root=/export\0"						\
>   	"mem_lpae=1\0"							\
>   	"mem_reserve=512M\0"						\
> -	"addr_fdt=0x87000000\0"						\
> -	"addr_kern=0x88000000\0"					\
> -	"addr_uboot=0x87000000\0"					\
> -	"addr_fs=0x82000000\0"						\
>   	"addr_ubi=0x82000000\0"						\
>   	"addr_secdb_key=0xc000000\0"					\
> -	"fdt_high=0xffffffff\0"						\
>   	"name_kern=uImage-keystone-evm.bin\0"				\
>   	"run_mon=mon_install ${addr_mon}\0"				\
> -	"run_kern=bootm ${addr_kern} - ${addr_fdt}\0"			\
> +	"run_kern=bootm ${loadaddr} - ${fdtaddr}\0"			\
>   	"init_net=run args_all args_net\0"				\
>   	"init_ubi=run args_all args_ubi; "				\
>   		"ubi part ubifs; ubifsmount ubi:boot;"			\
>   		"ubifsload ${addr_secdb_key} securedb.key.bin;\0"       \
> -	"get_fdt_net=dhcp ${addr_fdt} ${tftp_root}/${name_fdt}\0"	\
> -	"get_fdt_ubi=ubifsload ${addr_fdt} ${name_fdt}\0"		\
> -	"get_kern_net=dhcp ${addr_kern} ${tftp_root}/${name_kern}\0"	\
> -	"get_kern_ubi=ubifsload ${addr_kern} ${name_kern}\0"		\
> +	"get_fdt_net=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
> +	"get_fdt_ubi=ubifsload ${fdtaddr} ${name_fdt}\0"		\
> +	"get_kern_net=dhcp ${loadaddr} ${tftp_root}/${name_kern}\0"	\
> +	"get_kern_ubi=ubifsload ${loadaddr} ${name_kern}\0"		\
>   	"get_mon_net=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
>   	"get_mon_ubi=ubifsload ${addr_mon} ${name_mon}\0"		\
>   	"get_uboot_net=dhcp ${addr_uboot} ${tftp_root}/${name_uboot}\0"	\
> @@ -248,10 +244,10 @@
>   		"root=/dev/nfs rw nfsroot=${serverip}:${nfs_root},"	\
>   		"${nfs_options} ip=dhcp\0"				\
>   	"nfs_options=v3,tcp,rsize=4096,wsize=4096\0"			\
> -	"get_fdt_ramfs=dhcp ${addr_fdt} ${tftp_root}/${name_fdt}\0"	\
> -	"get_kern_ramfs=dhcp ${addr_kern} ${tftp_root}/${name_kern}\0"	\
> +	"get_fdt_ramfs=dhcp ${fdtaddr} ${tftp_root}/${name_fdt}\0"	\
> +	"get_kern_ramfs=dhcp ${loadaddr} ${tftp_root}/${name_kern}\0"	\
>   	"get_mon_ramfs=dhcp ${addr_mon} ${tftp_root}/${name_mon}\0"	\
> -	"get_fs_ramfs=dhcp ${addr_fs} ${tftp_root}/${name_fs}\0"	\
> +	"get_fs_ramfs=dhcp ${rdaddr} ${tftp_root}/${name_fs}\0"	\
>   	"get_ubi_net=dhcp ${addr_ubi} ${tftp_root}/${name_ubi}\0"	\
>   	"burn_ubi=nand erase.part ubifs; "				\
>   		"nand write ${addr_ubi} ubifs ${filesize}\0"		\
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses Nishanth Menon
  2015-07-22 15:22   ` Murali Karicheri
@ 2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:33AM -0500, Nishanth Menon wrote:

> Use the defaults defined in DEFAULT_LINUX_BOOT_ENV
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/1775f62a/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
                   ` (4 preceding siblings ...)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:21   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names Nishanth Menon
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name Nishanth Menon
  7 siblings, 2 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

Switch to using zImage instead of uImage. and while at it, start using
bootz as default. While at it, get rid of BOOTIMAGE define and start
using Linux upstream dtb file names.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
New patch.

 include/configs/k2e_evm.h            | 2 +-
 include/configs/k2hk_evm.h           | 2 +-
 include/configs/k2l_evm.h            | 2 +-
 include/configs/ti_armv7_keystone2.h | 5 ++---
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
index f1e650141ae1..afb289ca4bad 100644
--- a/include/configs/k2e_evm.h
+++ b/include/configs/k2e_evm.h
@@ -19,7 +19,7 @@
 	"addr_mon=0x0c140000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
-	"name_fdt=uImage-k2e-evm.dtb\0"					\
+	"name_fdt=k2e-evm.dtb\0"					\
 	"name_mon=skern-k2e-evm.bin\0"					\
 	"name_ubi=k2e-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
index f8e83de64b63..c2d3ec7c70cb 100644
--- a/include/configs/k2hk_evm.h
+++ b/include/configs/k2hk_evm.h
@@ -19,7 +19,7 @@
 	"addr_mon=0x0c5f0000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
-	"name_fdt=uImage-k2hk-evm.dtb\0"				\
+	"name_fdt=k2hk-evm.dtb\0"				\
 	"name_mon=skern-k2hk-evm.bin\0"					\
 	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
index 395608a5f6db..1957287a57c7 100644
--- a/include/configs/k2l_evm.h
+++ b/include/configs/k2l_evm.h
@@ -19,7 +19,7 @@
 	"addr_mon=0x0c140000\0"						\
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,4096\0"	\
-	"name_fdt=uImage-k2l-evm.dtb\0"					\
+	"name_fdt=k2l-evm.dtb\0"					\
 	"name_mon=skern-k2l-evm.bin\0"					\
 	"name_ubi=k2l-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index ddc2e23f1370..e327f3e03073 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -210,7 +210,6 @@
 /* EDMA3 */
 #define CONFIG_TI_EDMA3
 
-#define CONFIG_BOOTFILE			"uImage"
 #define CONFIG_EXTRA_ENV_SETTINGS					\
 	DEFAULT_LINUX_BOOT_ENV						\
 	CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
@@ -221,9 +220,9 @@
 	"mem_reserve=512M\0"						\
 	"addr_ubi=0x82000000\0"						\
 	"addr_secdb_key=0xc000000\0"					\
-	"name_kern=uImage-keystone-evm.bin\0"				\
+	"name_kern=zImage\0"						\
 	"run_mon=mon_install ${addr_mon}\0"				\
-	"run_kern=bootm ${loadaddr} - ${fdtaddr}\0"			\
+	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
 	"init_ubi=run args_all args_ubi; "				\
 		"ubi part ubifs; ubifsmount ubi:boot;"			\
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage Nishanth Menon
@ 2015-07-22 15:21   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:21 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> Switch to using zImage instead of uImage. and while at it, start using
> bootz as default. While at it, get rid of BOOTIMAGE define and start
> using Linux upstream dtb file names.
>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> New patch.
>
>   include/configs/k2e_evm.h            | 2 +-
>   include/configs/k2hk_evm.h           | 2 +-
>   include/configs/k2l_evm.h            | 2 +-
>   include/configs/ti_armv7_keystone2.h | 5 ++---
>   4 files changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
> index f1e650141ae1..afb289ca4bad 100644
> --- a/include/configs/k2e_evm.h
> +++ b/include/configs/k2e_evm.h
> @@ -19,7 +19,7 @@
>   	"addr_mon=0x0c140000\0"						\
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
> -	"name_fdt=uImage-k2e-evm.dtb\0"					\
> +	"name_fdt=k2e-evm.dtb\0"					\
>   	"name_mon=skern-k2e-evm.bin\0"					\
>   	"name_ubi=k2e-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
> diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
> index f8e83de64b63..c2d3ec7c70cb 100644
> --- a/include/configs/k2hk_evm.h
> +++ b/include/configs/k2hk_evm.h
> @@ -19,7 +19,7 @@
>   	"addr_mon=0x0c5f0000\0"						\
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
> -	"name_fdt=uImage-k2hk-evm.dtb\0"				\
> +	"name_fdt=k2hk-evm.dtb\0"				\
>   	"name_mon=skern-k2hk-evm.bin\0"					\
>   	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
> diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
> index 395608a5f6db..1957287a57c7 100644
> --- a/include/configs/k2l_evm.h
> +++ b/include/configs/k2l_evm.h
> @@ -19,7 +19,7 @@
>   	"addr_mon=0x0c140000\0"						\
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,4096\0"	\
> -	"name_fdt=uImage-k2l-evm.dtb\0"					\
> +	"name_fdt=k2l-evm.dtb\0"					\
>   	"name_mon=skern-k2l-evm.bin\0"					\
>   	"name_ubi=k2l-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
> index ddc2e23f1370..e327f3e03073 100644
> --- a/include/configs/ti_armv7_keystone2.h
> +++ b/include/configs/ti_armv7_keystone2.h
> @@ -210,7 +210,6 @@
>   /* EDMA3 */
>   #define CONFIG_TI_EDMA3
>
> -#define CONFIG_BOOTFILE			"uImage"
>   #define CONFIG_EXTRA_ENV_SETTINGS					\
>   	DEFAULT_LINUX_BOOT_ENV						\
>   	CONFIG_EXTRA_ENV_KS2_BOARD_SETTINGS				\
> @@ -221,9 +220,9 @@
>   	"mem_reserve=512M\0"						\
>   	"addr_ubi=0x82000000\0"						\
>   	"addr_secdb_key=0xc000000\0"					\
> -	"name_kern=uImage-keystone-evm.bin\0"				\
> +	"name_kern=zImage\0"						\
>   	"run_mon=mon_install ${addr_mon}\0"				\
> -	"run_kern=bootm ${loadaddr} - ${fdtaddr}\0"			\
> +	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
>   	"init_net=run args_all args_net\0"				\
>   	"init_ubi=run args_all args_ubi; "				\
>   		"ubi part ubifs; ubifsmount ubi:boot;"			\
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage Nishanth Menon
  2015-07-22 15:21   ` Murali Karicheri
@ 2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:34AM -0500, Nishanth Menon wrote:

> Switch to using zImage instead of uImage. and while at it, start using
> bootz as default. While at it, get rid of BOOTIMAGE define and start
> using Linux upstream dtb file names.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/a5633db3/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
                   ` (5 preceding siblings ...)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:21   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name Nishanth Menon
  7 siblings, 2 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

using http://git.ti.com/keystone-linux/boot-monitor/trees/master as
reference (tag K2_BM_15.07) the generated files do not have evm
extensions by default. So dont use -evm extension.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
New patch

 include/configs/k2e_evm.h  | 2 +-
 include/configs/k2hk_evm.h | 2 +-
 include/configs/k2l_evm.h  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
index afb289ca4bad..4f4ebf53ec59 100644
--- a/include/configs/k2e_evm.h
+++ b/include/configs/k2e_evm.h
@@ -20,7 +20,7 @@
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
 	"name_fdt=k2e-evm.dtb\0"					\
-	"name_mon=skern-k2e-evm.bin\0"					\
+	"name_mon=skern-k2e.bin\0"					\
 	"name_ubi=k2e-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
 	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
index c2d3ec7c70cb..6c6dcb1e5ed3 100644
--- a/include/configs/k2hk_evm.h
+++ b/include/configs/k2hk_evm.h
@@ -20,7 +20,7 @@
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
 	"name_fdt=k2hk-evm.dtb\0"				\
-	"name_mon=skern-k2hk-evm.bin\0"					\
+	"name_mon=skern-k2hk.bin\0"					\
 	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
 	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
index 1957287a57c7..9bacfa49c430 100644
--- a/include/configs/k2l_evm.h
+++ b/include/configs/k2l_evm.h
@@ -20,7 +20,7 @@
 	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
 	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,4096\0"	\
 	"name_fdt=k2l-evm.dtb\0"					\
-	"name_mon=skern-k2l-evm.bin\0"					\
+	"name_mon=skern-k2l.bin\0"					\
 	"name_ubi=k2l-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
 	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names Nishanth Menon
@ 2015-07-22 15:21   ` Murali Karicheri
  2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:21 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> using http://git.ti.com/keystone-linux/boot-monitor/trees/master as
> reference (tag K2_BM_15.07) the generated files do not have evm
> extensions by default. So dont use -evm extension.
>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> New patch
>
>   include/configs/k2e_evm.h  | 2 +-
>   include/configs/k2hk_evm.h | 2 +-
>   include/configs/k2l_evm.h  | 2 +-
>   3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
> index afb289ca4bad..4f4ebf53ec59 100644
> --- a/include/configs/k2e_evm.h
> +++ b/include/configs/k2e_evm.h
> @@ -20,7 +20,7 @@
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
>   	"name_fdt=k2e-evm.dtb\0"					\
> -	"name_mon=skern-k2e-evm.bin\0"					\
> +	"name_mon=skern-k2e.bin\0"					\
>   	"name_ubi=k2e-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
> diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
> index c2d3ec7c70cb..6c6dcb1e5ed3 100644
> --- a/include/configs/k2hk_evm.h
> +++ b/include/configs/k2hk_evm.h
> @@ -20,7 +20,7 @@
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,2048\0"	\
>   	"name_fdt=k2hk-evm.dtb\0"				\
> -	"name_mon=skern-k2hk-evm.bin\0"					\
> +	"name_mon=skern-k2hk.bin\0"					\
>   	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
> diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
> index 1957287a57c7..9bacfa49c430 100644
> --- a/include/configs/k2l_evm.h
> +++ b/include/configs/k2l_evm.h
> @@ -20,7 +20,7 @@
>   	"args_ubi=setenv bootargs ${bootargs} rootfstype=ubifs "	\
>   	"root=ubi0:rootfs rootflags=sync rw ubi.mtd=ubifs,4096\0"	\
>   	"name_fdt=k2l-evm.dtb\0"					\
> -	"name_mon=skern-k2l-evm.bin\0"					\
> +	"name_mon=skern-k2l.bin\0"					\
>   	"name_ubi=k2l-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
>   	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names Nishanth Menon
  2015-07-22 15:21   ` Murali Karicheri
@ 2015-07-22 15:35   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:35 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:35AM -0500, Nishanth Menon wrote:

> using http://git.ti.com/keystone-linux/boot-monitor/trees/master as
> reference (tag K2_BM_15.07) the generated files do not have evm
> extensions by default. So dont use -evm extension.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/018041de/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name
  2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
                   ` (6 preceding siblings ...)
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names Nishanth Menon
@ 2015-07-18 16:40 ` Nishanth Menon
  2015-07-22 15:20   ` Murali Karicheri
  2015-07-22 15:32   ` Tom Rini
  7 siblings, 2 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-18 16:40 UTC (permalink / raw)
  To: u-boot

u-boot should support multiple distros as necessary. So, DONOT hardcode
distro specific image name in package. Instead we use a generic
ramdisk.gz as image name which can be modified by distro specific logic.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
New patch

 include/configs/k2e_evm.h            | 1 -
 include/configs/k2hk_evm.h           | 1 -
 include/configs/k2l_evm.h            | 1 -
 include/configs/ti_armv7_keystone2.h | 1 +
 4 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
index 4f4ebf53ec59..59ce6fa0bdb3 100644
--- a/include/configs/k2e_evm.h
+++ b/include/configs/k2e_evm.h
@@ -23,7 +23,6 @@
 	"name_mon=skern-k2e.bin\0"					\
 	"name_ubi=k2e-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
-	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
 
 #include <configs/ti_armv7_keystone2.h>
 
diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
index 6c6dcb1e5ed3..835dd0e13b4b 100644
--- a/include/configs/k2hk_evm.h
+++ b/include/configs/k2hk_evm.h
@@ -23,7 +23,6 @@
 	"name_mon=skern-k2hk.bin\0"					\
 	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
-	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
 
 #include <configs/ti_armv7_keystone2.h>
 
diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
index 9bacfa49c430..a7c835953b73 100644
--- a/include/configs/k2l_evm.h
+++ b/include/configs/k2l_evm.h
@@ -23,7 +23,6 @@
 	"name_mon=skern-k2l.bin\0"					\
 	"name_ubi=k2l-evm-ubifs.ubi\0"					\
 	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
-	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
 
 #include <configs/ti_armv7_keystone2.h>
 
diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index e327f3e03073..198e3650fd55 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -221,6 +221,7 @@
 	"addr_ubi=0x82000000\0"						\
 	"addr_secdb_key=0xc000000\0"					\
 	"name_kern=zImage\0"						\
+	"name_fs=ramdisk.gz\0"						\
 	"run_mon=mon_install ${addr_mon}\0"				\
 	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
 	"init_net=run args_all args_net\0"				\
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name Nishanth Menon
@ 2015-07-22 15:20   ` Murali Karicheri
  2015-07-22 15:32   ` Tom Rini
  1 sibling, 0 replies; 25+ messages in thread
From: Murali Karicheri @ 2015-07-22 15:20 UTC (permalink / raw)
  To: u-boot

On 07/18/2015 12:40 PM, Nishanth Menon wrote:
> u-boot should support multiple distros as necessary. So, DONOT hardcode
> distro specific image name in package. Instead we use a generic
> ramdisk.gz as image name which can be modified by distro specific logic.
>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> New patch
>
>   include/configs/k2e_evm.h            | 1 -
>   include/configs/k2hk_evm.h           | 1 -
>   include/configs/k2l_evm.h            | 1 -
>   include/configs/ti_armv7_keystone2.h | 1 +
>   4 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/include/configs/k2e_evm.h b/include/configs/k2e_evm.h
> index 4f4ebf53ec59..59ce6fa0bdb3 100644
> --- a/include/configs/k2e_evm.h
> +++ b/include/configs/k2e_evm.h
> @@ -23,7 +23,6 @@
>   	"name_mon=skern-k2e.bin\0"					\
>   	"name_ubi=k2e-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2e-evm.gph\0"				\
> -	"name_fs=arago-console-image-k2e-evm.cpio.gz\0"
>
>   #include <configs/ti_armv7_keystone2.h>
>
> diff --git a/include/configs/k2hk_evm.h b/include/configs/k2hk_evm.h
> index 6c6dcb1e5ed3..835dd0e13b4b 100644
> --- a/include/configs/k2hk_evm.h
> +++ b/include/configs/k2hk_evm.h
> @@ -23,7 +23,6 @@
>   	"name_mon=skern-k2hk.bin\0"					\
>   	"name_ubi=k2hk-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2hk-evm.gph\0"				\
> -	"name_fs=arago-console-image-k2hk-evm.cpio.gz\0"
>
>   #include <configs/ti_armv7_keystone2.h>
>
> diff --git a/include/configs/k2l_evm.h b/include/configs/k2l_evm.h
> index 9bacfa49c430..a7c835953b73 100644
> --- a/include/configs/k2l_evm.h
> +++ b/include/configs/k2l_evm.h
> @@ -23,7 +23,6 @@
>   	"name_mon=skern-k2l.bin\0"					\
>   	"name_ubi=k2l-evm-ubifs.ubi\0"					\
>   	"name_uboot=u-boot-spi-k2l-evm.gph\0"				\
> -	"name_fs=arago-console-image-k2l-evm.cpio.gz\0"
>
>   #include <configs/ti_armv7_keystone2.h>
>
> diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
> index e327f3e03073..198e3650fd55 100644
> --- a/include/configs/ti_armv7_keystone2.h
> +++ b/include/configs/ti_armv7_keystone2.h
> @@ -221,6 +221,7 @@
>   	"addr_ubi=0x82000000\0"						\
>   	"addr_secdb_key=0xc000000\0"					\
>   	"name_kern=zImage\0"						\
> +	"name_fs=ramdisk.gz\0"						\
>   	"run_mon=mon_install ${addr_mon}\0"				\
>   	"run_kern=bootz ${loadaddr} - ${fdtaddr}\0"			\
>   	"init_net=run args_all args_net\0"				\
>
Reviewed-by: Murali Karicheri <m-karicheri2@ti.com>

-- 
Murali Karicheri
Linux Kernel, Keystone

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name
  2015-07-18 16:40 ` [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name Nishanth Menon
  2015-07-22 15:20   ` Murali Karicheri
@ 2015-07-22 15:32   ` Tom Rini
  2015-07-22 15:33     ` Nishanth Menon
  1 sibling, 1 reply; 25+ messages in thread
From: Tom Rini @ 2015-07-22 15:32 UTC (permalink / raw)
  To: u-boot

On Sat, Jul 18, 2015 at 11:40:36AM -0500, Nishanth Menon wrote:

> u-boot should support multiple distros as necessary. So, DONOT hardcode
> distro specific image name in package. Instead we use a generic
> ramdisk.gz as image name which can be modified by distro specific logic.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>

Instead of trying to whack this stuff to be more generic lets just punt
on it and drop when moving to config_distro_bootcmd.h

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150722/e0fe25b5/attachment.sig>

^ permalink raw reply	[flat|nested] 25+ messages in thread

* [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name
  2015-07-22 15:32   ` Tom Rini
@ 2015-07-22 15:33     ` Nishanth Menon
  0 siblings, 0 replies; 25+ messages in thread
From: Nishanth Menon @ 2015-07-22 15:33 UTC (permalink / raw)
  To: u-boot

On Wed, Jul 22, 2015 at 10:32 AM, Tom Rini <trini@konsulko.com> wrote:
>
> Instead of trying to whack this stuff to be more generic lets just punt
> on it and drop when moving to config_distro_bootcmd.h
OK. will do so.

-- 
---
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 25+ messages in thread

end of thread, other threads:[~2015-07-22 15:35 UTC | newest]

Thread overview: 25+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-18 16:40 [U-Boot] [PATCH V3 0/8] configs: make keystone2 config to start using arm_v7_common header Nishanth Menon
2015-07-18 16:40 ` [U-Boot] [PATCH V3 1/8] configs: split ti_armv7_common into a omap generic header Nishanth Menon
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 2/8] board: ks2_evm: get rid of bogus CONFIG_LINUX_BOOT_PARAM_ADDR Nishanth Menon
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 3/8] configs: rename ks2_evm into ti_armv7_keystone2 Nishanth Menon
2015-07-22 15:23   ` Murali Karicheri
2015-07-22 15:24   ` Murali Karicheri
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 4/8] configs: ti_armv7_keystone2: start using armv7_common Nishanth Menon
2015-07-22 15:19   ` Murali Karicheri
2015-07-22 15:28     ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 5/8] configs: ti_armv7_keystone2: switch addresses to generic addresses Nishanth Menon
2015-07-22 15:22   ` Murali Karicheri
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 6/8] ti_armv7_keystone2: switch to using kernel zImage Nishanth Menon
2015-07-22 15:21   ` Murali Karicheri
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 7/8] configs: k2*_evm: rename skernel binary names to generated file names Nishanth Menon
2015-07-22 15:21   ` Murali Karicheri
2015-07-22 15:35   ` Tom Rini
2015-07-18 16:40 ` [U-Boot] [PATCH V3 8/8] configs: keystone2: remove distro specific ramdisk name Nishanth Menon
2015-07-22 15:20   ` Murali Karicheri
2015-07-22 15:32   ` Tom Rini
2015-07-22 15:33     ` Nishanth Menon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox