* [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support
@ 2016-11-29 13:31 Fabien Parent
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
` (6 more replies)
0 siblings, 7 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
The default AIS image generated contains the SPL, but the latter is not able to
load u-boot because the NAND configuration is missing. This series adds SPL NAND
boot support and results in the AIS image that is generated able to boot from
NAND.
Fabien Parent (6):
NAND: davinci: add support for NAND chips with 16 bits bus
davinci: omapl138_lcdk: use correct define for 16 bit NAND chips
davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE
davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE
davinci: omapl138_lck: remove obsolete define
davinci: omapl138_lcdk: add NAND SPL boot support
board/davinci/da8xxevm/README.omapl138-lcdk | 28 ------------------------
configs/omapl138_lcdk_defconfig | 3 +++
drivers/mtd/nand/davinci_nand.c | 3 +++
include/configs/omapl138_lcdk.h | 34 +++++++++++++++++++++++++----
scripts/config_whitelist.txt | 3 ---
5 files changed, 36 insertions(+), 35 deletions(-)
delete mode 100644 board/davinci/da8xxevm/README.omapl138-lcdk
--
2.10.2
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:08 ` Tom Rini
2016-12-04 0:56 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips Fabien Parent
` (5 subsequent siblings)
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
The OMAPL138-LCD board uses a NAND chip with a 16 bits bus. Add
support into the davinci driver for 16 bit bus NAND chips.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
drivers/mtd/nand/davinci_nand.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c
index 48a8ca7..0624644 100644
--- a/drivers/mtd/nand/davinci_nand.c
+++ b/drivers/mtd/nand/davinci_nand.c
@@ -799,6 +799,9 @@ void davinci_nand_init(struct nand_chip *nand)
#ifdef CONFIG_SYS_NAND_NO_SUBPAGE_WRITE
nand->options |= NAND_NO_SUBPAGE_WRITE;
#endif
+#ifdef CONFIG_SYS_NAND_BUSWIDTH_16BIT
+ nand->options |= NAND_BUSWIDTH_16;
+#endif
#ifdef CONFIG_SYS_NAND_HW_ECC
nand->ecc.mode = NAND_ECC_HW;
nand->ecc.size = 512;
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:08 ` Tom Rini
2016-12-04 0:57 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE Fabien Parent
` (4 subsequent siblings)
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
The omapl138_lcdk header defines CONFIG_SYS_NAND_BUSWIDTH_16_BIT while
the correct name is CONFIG_SYS_NAND_BUSWIDTH_16BIT.
While renaming the only occurrence of CONFIG_SYS_NAND_BUSWIDTH_16_BIT,
let's also remove it from the config_whitelist.txt file.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
include/configs/omapl138_lcdk.h | 2 +-
scripts/config_whitelist.txt | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index 854fc47..b4bcd8f 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -124,7 +124,7 @@
#define CONFIG_SYS_NAND_USE_FLASH_BBT
#define CONFIG_SYS_NAND_4BIT_HW_ECC_OOBFIRST
#define CONFIG_SYS_NAND_PAGE_2K
-#define CONFIG_SYS_NAND_BUSWIDTH_16_BIT
+#define CONFIG_SYS_NAND_BUSWIDTH_16BIT
#define CONFIG_SYS_NAND_CS 3
#define CONFIG_SYS_NAND_BASE DAVINCI_ASYNC_EMIF_DATA_CE3_BASE
#define CONFIG_SYS_CLE_MASK 0x10
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index d0b5e7f..0c7d3cd 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -6654,7 +6654,6 @@ CONFIG_SYS_NAND_BLOCK_SIZE
CONFIG_SYS_NAND_BOOT
CONFIG_SYS_NAND_BR_PRELIM
CONFIG_SYS_NAND_BUSWIDTH_16
-CONFIG_SYS_NAND_BUSWIDTH_16_BIT
CONFIG_SYS_NAND_CE
CONFIG_SYS_NAND_CLE
CONFIG_SYS_NAND_CS
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
2016-11-29 13:31 ` [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE Fabien Parent
` (3 subsequent siblings)
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
CONFIG_SYS_CLE_MASK is not used anywhere. It has probably been
renamed to CONFIG_SYS_NAND_MASK_CLE. Rename it and remove the former
from the config_whitelist.txt file.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
include/configs/omapl138_lcdk.h | 2 +-
scripts/config_whitelist.txt | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index b4bcd8f..f92d6e0 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -127,7 +127,7 @@
#define CONFIG_SYS_NAND_BUSWIDTH_16BIT
#define CONFIG_SYS_NAND_CS 3
#define CONFIG_SYS_NAND_BASE DAVINCI_ASYNC_EMIF_DATA_CE3_BASE
-#define CONFIG_SYS_CLE_MASK 0x10
+#define CONFIG_SYS_NAND_MASK_CLE 0x10
#define CONFIG_SYS_ALE_MASK 0x8
#undef CONFIG_SYS_NAND_HW_ECC
#define CONFIG_SYS_MAX_NAND_DEVICE 1 /* Max number of NAND devices */
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index 0c7d3cd..b2ced15 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -4569,7 +4569,6 @@ CONFIG_SYS_CF_INTC_REG1
CONFIG_SYS_CF_SIZE
CONFIG_SYS_CH7301_I2C
CONFIG_SYS_CKEN
-CONFIG_SYS_CLE_MASK
CONFIG_SYS_CLK
CONFIG_SYS_CLKDIV_CPU
CONFIG_SYS_CLKDIV_HSB
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
` (2 preceding siblings ...)
2016-11-29 13:31 ` [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define Fabien Parent
` (2 subsequent siblings)
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
CONFIG_SYS_ALE_MASK is not used anywhere. It has probably been
renamed to CONFIG_SYS_NAND_MASK_ALE. Rename it and remove the former
from the config_whitelist.txt file.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
include/configs/omapl138_lcdk.h | 2 +-
scripts/config_whitelist.txt | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index f92d6e0..6262147 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -128,7 +128,7 @@
#define CONFIG_SYS_NAND_CS 3
#define CONFIG_SYS_NAND_BASE DAVINCI_ASYNC_EMIF_DATA_CE3_BASE
#define CONFIG_SYS_NAND_MASK_CLE 0x10
-#define CONFIG_SYS_ALE_MASK 0x8
+#define CONFIG_SYS_NAND_MASK_ALE 0x8
#undef CONFIG_SYS_NAND_HW_ECC
#define CONFIG_SYS_MAX_NAND_DEVICE 1 /* Max number of NAND devices */
#define NAND_MAX_CHIPS 1
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index b2ced15..fc8d319 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -4402,7 +4402,6 @@ CONFIG_SYS_ACR_RPTCNT
CONFIG_SYS_ADDRESS_MAP_A
CONFIG_SYS_ADV7611_I2C
CONFIG_SYS_AHB_BASE
-CONFIG_SYS_ALE_MASK
CONFIG_SYS_ALLOC_DPRAM
CONFIG_SYS_ALT_BOOT
CONFIG_SYS_ALT_FLASH
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
` (3 preceding siblings ...)
2016-11-29 13:31 ` [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support Fabien Parent
2016-11-29 18:10 ` [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND " Tom Rini
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
NAND_MAX_CHIPS is not used anymore and has been replaced by
CONFIG_SYS_MAX_NAND_DEVICE. There is no need to keep the former
define.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
include/configs/omapl138_lcdk.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index 6262147..3e27778 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -131,7 +131,6 @@
#define CONFIG_SYS_NAND_MASK_ALE 0x8
#undef CONFIG_SYS_NAND_HW_ECC
#define CONFIG_SYS_MAX_NAND_DEVICE 1 /* Max number of NAND devices */
-#define NAND_MAX_CHIPS 1
#endif
#ifdef CONFIG_SYS_USE_NOR
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
` (4 preceding siblings ...)
2016-11-29 13:31 ` [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define Fabien Parent
@ 2016-11-29 13:31 ` Fabien Parent
2016-11-29 18:10 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 18:10 ` [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND " Tom Rini
6 siblings, 2 replies; 20+ messages in thread
From: Fabien Parent @ 2016-11-29 13:31 UTC (permalink / raw)
To: u-boot
NAND SPL boot was missing. Add it. The README specific to omapl138-lcdk
is also removed because its content does not apply anymore, i.e. the
generated AIS image can be flashed directly to the NAND without
using any external tool to create and bootable AIS image.
Signed-off-by: Fabien Parent <fparent@baylibre.com>
---
board/davinci/da8xxevm/README.omapl138-lcdk | 28 ----------------------------
configs/omapl138_lcdk_defconfig | 3 +++
include/configs/omapl138_lcdk.h | 27 +++++++++++++++++++++++++++
3 files changed, 30 insertions(+), 28 deletions(-)
delete mode 100644 board/davinci/da8xxevm/README.omapl138-lcdk
diff --git a/board/davinci/da8xxevm/README.omapl138-lcdk b/board/davinci/da8xxevm/README.omapl138-lcdk
deleted file mode 100644
index ea0c53d..0000000
--- a/board/davinci/da8xxevm/README.omapl138-lcdk
+++ /dev/null
@@ -1,28 +0,0 @@
-Summary
-=======
-This README assumes you have read README.da850. It contains some additional
-information specific to building the omapl138-lcdk. The AIS file as generated
-by the build is, currently, not useable due to differences in the flash
-available on this board, as compared to the da850evm boards.
-
-Flash Differences
-=================
-Refer to the discussion in [1] for more detail - basically the da850evm uses
-SPI flash whereas the lcdk uses NAND flash to store the bootloader, and
-the support isn't there in the SPL code.
-
-It should be possible to add the support in the SPL code should someone be
-sufficiently motivated.
-
-Using the built image
-=====================
-The output image to use is u-boot.bin. This needs to be converted to an
-AIS file as described in [1] and then flashed using the utitilty linked to
-there and also described in README.da850. You _may_ be able to write using
-u-boot itself, but the commands in README.da850 won't work as they write to
-SPI rather than NAND.
-
-Links
-=====
-[1]
- http://e2e.ti.com/support/dsp/omap_applications_processors/f/42/t/386829
\ No newline at end of file
diff --git a/configs/omapl138_lcdk_defconfig b/configs/omapl138_lcdk_defconfig
index bcd1acb..c5c978f 100644
--- a/configs/omapl138_lcdk_defconfig
+++ b/configs/omapl138_lcdk_defconfig
@@ -3,6 +3,7 @@ CONFIG_ARCH_DAVINCI=y
CONFIG_TARGET_OMAPL138_LCDK=y
CONFIG_SPL_LIBCOMMON_SUPPORT=y
CONFIG_SPL_LIBGENERIC_SUPPORT=y
+CONFIG_SPL_NAND_SUPPORT=y
CONFIG_SPL_SERIAL_SUPPORT=y
CONFIG_BOOTDELAY=3
CONFIG_VERSION_VARIABLE=y
@@ -22,6 +23,8 @@ CONFIG_CMD_PING=y
CONFIG_CMD_EXT2=y
CONFIG_CMD_FAT=y
CONFIG_CMD_UBI=y
+CONFIG_SYS_NAND_U_BOOT_LOCATIONS=y
+CONFIG_SYS_NAND_U_BOOT_OFFS=0x28000
CONFIG_SPI_FLASH=y
CONFIG_SPI_FLASH_STMICRO=y
CONFIG_SPI_FLASH_WINBOND=y
diff --git a/include/configs/omapl138_lcdk.h b/include/configs/omapl138_lcdk.h
index 3e27778..e3f8265 100644
--- a/include/configs/omapl138_lcdk.h
+++ b/include/configs/omapl138_lcdk.h
@@ -131,6 +131,33 @@
#define CONFIG_SYS_NAND_MASK_ALE 0x8
#undef CONFIG_SYS_NAND_HW_ECC
#define CONFIG_SYS_MAX_NAND_DEVICE 1 /* Max number of NAND devices */
+#define CONFIG_SYS_NAND_HW_ECC_OOBFIRST
+#define CONFIG_SYS_NAND_5_ADDR_CYCLE
+#define CONFIG_SYS_NAND_PAGE_SIZE (2 << 10)
+#define CONFIG_SYS_NAND_BLOCK_SIZE (128 << 10)
+#define CONFIG_SYS_NAND_U_BOOT_SIZE 0x200 /*0x60000*/
+#define CONFIG_SYS_NAND_U_BOOT_DST 0xc1080000
+#define CONFIG_SYS_NAND_U_BOOT_START CONFIG_SYS_NAND_U_BOOT_DST
+#define CONFIG_SYS_NAND_U_BOOT_RELOC_SP (CONFIG_SYS_NAND_U_BOOT_DST - \
+ CONFIG_SYS_NAND_U_BOOT_SIZE - \
+ CONFIG_SYS_MALLOC_LEN - \
+ GENERATED_GBL_DATA_SIZE)
+#define CONFIG_SYS_NAND_ECCPOS { \
+ 24, 25, 26, 27, 28, \
+ 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, \
+ 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, \
+ 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, \
+ 59, 60, 61, 62, 63 }
+#define CONFIG_SYS_NAND_PAGE_COUNT 64
+#define CONFIG_SYS_NAND_BAD_BLOCK_POS 0
+#define CONFIG_SYS_NAND_ECCSIZE 512
+#define CONFIG_SYS_NAND_ECCBYTES 10
+#define CONFIG_SYS_NAND_OOBSIZE 64
+#define CONFIG_SPL_NAND_BASE
+#define CONFIG_SPL_NAND_DRIVERS
+#define CONFIG_SPL_NAND_ECC
+#define CONFIG_SPL_NAND_SIMPLE
+#define CONFIG_SPL_NAND_LOAD
#endif
#ifdef CONFIG_SYS_USE_NOR
--
2.10.2
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
@ 2016-11-29 18:08 ` Tom Rini
2016-12-04 0:56 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:08 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:29PM +0100, Fabien Parent wrote:
> The OMAPL138-LCD board uses a NAND chip with a 16 bits bus. Add
> support into the davinci driver for 16 bit bus NAND chips.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/659afbb9/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips
2016-11-29 13:31 ` [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips Fabien Parent
@ 2016-11-29 18:08 ` Tom Rini
2016-12-04 0:57 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:08 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:30PM +0100, Fabien Parent wrote:
> The omapl138_lcdk header defines CONFIG_SYS_NAND_BUSWIDTH_16_BIT while
> the correct name is CONFIG_SYS_NAND_BUSWIDTH_16BIT.
> While renaming the only occurrence of CONFIG_SYS_NAND_BUSWIDTH_16_BIT,
> let's also remove it from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/3c103cdb/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE
2016-11-29 13:31 ` [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE Fabien Parent
@ 2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:09 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:31PM +0100, Fabien Parent wrote:
> CONFIG_SYS_CLE_MASK is not used anywhere. It has probably been
> renamed to CONFIG_SYS_NAND_MASK_CLE. Rename it and remove the former
> from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/bcfef76f/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE
2016-11-29 13:31 ` [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE Fabien Parent
@ 2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:09 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:32PM +0100, Fabien Parent wrote:
> CONFIG_SYS_ALE_MASK is not used anywhere. It has probably been
> renamed to CONFIG_SYS_NAND_MASK_ALE. Rename it and remove the former
> from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/eb77d29d/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define
2016-11-29 13:31 ` [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define Fabien Parent
@ 2016-11-29 18:09 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:09 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:33PM +0100, Fabien Parent wrote:
> NAND_MAX_CHIPS is not used anymore and has been replaced by
> CONFIG_SYS_MAX_NAND_DEVICE. There is no need to keep the former
> define.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/87b4202c/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support
2016-11-29 13:31 ` [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support Fabien Parent
@ 2016-11-29 18:10 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:10 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:34PM +0100, Fabien Parent wrote:
> NAND SPL boot was missing. Add it. The README specific to omapl138-lcdk
> is also removed because its content does not apply anymore, i.e. the
> generated AIS image can be flashed directly to the NAND without
> using any external tool to create and bootable AIS image.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.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: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/d4fb6746/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
` (5 preceding siblings ...)
2016-11-29 13:31 ` [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support Fabien Parent
@ 2016-11-29 18:10 ` Tom Rini
6 siblings, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-11-29 18:10 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:28PM +0100, Fabien Parent wrote:
> The default AIS image generated contains the SPL, but the latter is not able to
> load u-boot because the NAND configuration is missing. This series adds SPL NAND
> boot support and results in the AIS image that is generated able to boot from
> NAND.
>
> Fabien Parent (6):
> NAND: davinci: add support for NAND chips with 16 bits bus
> davinci: omapl138_lcdk: use correct define for 16 bit NAND chips
> davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE
> davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE
> davinci: omapl138_lck: remove obsolete define
> davinci: omapl138_lcdk: add NAND SPL boot support
>
> board/davinci/da8xxevm/README.omapl138-lcdk | 28 ------------------------
> configs/omapl138_lcdk_defconfig | 3 +++
> drivers/mtd/nand/davinci_nand.c | 3 +++
> include/configs/omapl138_lcdk.h | 34 +++++++++++++++++++++++++----
> scripts/config_whitelist.txt | 3 ---
> 5 files changed, 36 insertions(+), 35 deletions(-)
> delete mode 100644 board/davinci/da8xxevm/README.omapl138-lcdk
Everything looks good, so I've reviewed-by'd it. But it also shows a
number of things which should be converted to Kconfig (I stumbled onto
in looking at why you didn't set SYS_NAND_BUSWIDTH_16BIT via defconfig).
Do you have time to move the davinci NAND driver to Kconfig while in
here? Thanks!
>
> --
> 2.10.2
>
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161129/f92fef40/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 1/6] NAND: davinci: add support for NAND chips with 16 bits bus
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
2016-11-29 18:08 ` Tom Rini
@ 2016-12-04 0:56 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 0:56 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:29PM +0100, Fabien Parent wrote:
> The OMAPL138-LCD board uses a NAND chip with a 16 bits bus. Add
> support into the davinci driver for 16 bit bus NAND chips.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/23e0aa92/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips
2016-11-29 13:31 ` [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips Fabien Parent
2016-11-29 18:08 ` Tom Rini
@ 2016-12-04 0:57 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 0:57 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:30PM +0100, Fabien Parent wrote:
> The omapl138_lcdk header defines CONFIG_SYS_NAND_BUSWIDTH_16_BIT while
> the correct name is CONFIG_SYS_NAND_BUSWIDTH_16BIT.
> While renaming the only occurrence of CONFIG_SYS_NAND_BUSWIDTH_16_BIT,
> let's also remove it from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/efd0fd5a/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE
2016-11-29 13:31 ` [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE Fabien Parent
2016-11-29 18:09 ` Tom Rini
@ 2016-12-04 0:59 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 0:59 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:31PM +0100, Fabien Parent wrote:
> CONFIG_SYS_CLE_MASK is not used anywhere. It has probably been
> renamed to CONFIG_SYS_NAND_MASK_CLE. Rename it and remove the former
> from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/cd857460/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE
2016-11-29 13:31 ` [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE Fabien Parent
2016-11-29 18:09 ` Tom Rini
@ 2016-12-04 0:59 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 0:59 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:32PM +0100, Fabien Parent wrote:
> CONFIG_SYS_ALE_MASK is not used anywhere. It has probably been
> renamed to CONFIG_SYS_NAND_MASK_ALE. Rename it and remove the former
> from the config_whitelist.txt file.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/c099cd9c/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 5/6] davinci: omapl138_lck: remove obsolete define
2016-11-29 13:31 ` [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define Fabien Parent
2016-11-29 18:09 ` Tom Rini
@ 2016-12-04 1:00 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 1:00 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:33PM +0100, Fabien Parent wrote:
> NAND_MAX_CHIPS is not used anymore and has been replaced by
> CONFIG_SYS_MAX_NAND_DEVICE. There is no need to keep the former
> define.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/007bd4be/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [U-Boot] [U-Boot, 6/6] davinci: omapl138_lcdk: add NAND SPL boot support
2016-11-29 13:31 ` [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support Fabien Parent
2016-11-29 18:10 ` Tom Rini
@ 2016-12-04 1:00 ` Tom Rini
1 sibling, 0 replies; 20+ messages in thread
From: Tom Rini @ 2016-12-04 1:00 UTC (permalink / raw)
To: u-boot
On Tue, Nov 29, 2016 at 02:31:34PM +0100, Fabien Parent wrote:
> NAND SPL boot was missing. Add it. The README specific to omapl138-lcdk
> is also removed because its content does not apply anymore, i.e. the
> generated AIS image can be flashed directly to the NAND without
> using any external tool to create and bootable AIS image.
>
> Signed-off-by: Fabien Parent <fparent@baylibre.com>
> Reviewed-by: Tom Rini <trini@konsulko.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161203/aa13fc24/attachment.sig>
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2016-12-04 1:00 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-29 13:31 [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND boot support Fabien Parent
2016-11-29 13:31 ` [U-Boot] [PATCH 1/6] NAND: davinci: add support for NAND chips with 16 bits bus Fabien Parent
2016-11-29 18:08 ` Tom Rini
2016-12-04 0:56 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 2/6] davinci: omapl138_lcdk: use correct define for 16 bit NAND chips Fabien Parent
2016-11-29 18:08 ` Tom Rini
2016-12-04 0:57 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 3/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_CLE Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 4/6] davinci: omapl138_lcdk: use correct name for CONFIG_SYS_NAND_MASK_ALE Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 0:59 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 5/6] davinci: omapl138_lck: remove obsolete define Fabien Parent
2016-11-29 18:09 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 13:31 ` [U-Boot] [PATCH 6/6] davinci: omapl138_lcdk: add NAND SPL boot support Fabien Parent
2016-11-29 18:10 ` Tom Rini
2016-12-04 1:00 ` [U-Boot] [U-Boot, " Tom Rini
2016-11-29 18:10 ` [U-Boot] [PATCH 0/6] davinci: omapl138_lcdk: add SPL NAND " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox