* [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h
@ 2015-08-11 16:57 Vladimir Zapolskiy
2015-08-11 20:16 ` LEMIEUX, SYLVAIN
2015-08-18 17:53 ` [U-Boot] " Tom Rini
0 siblings, 2 replies; 5+ messages in thread
From: Vladimir Zapolskiy @ 2015-08-11 16:57 UTC (permalink / raw)
To: u-boot
A number of LPC32xx SLC NAND defines is dictated by controller
hardware limits and OOB layout is defined by operating system, the
definitions are common for all users. Since those macro are used
in out of NAND SLC driver code (simple NAND SPL framework), they can
not be placed into the driver, therefore move them from board config
files to arch/config.h
The change also adds OOB layout details specific to small page NAND
devices taken from Linux kernel.
Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
---
The change is based on
* https://patchwork.ozlabs.org/patch/497298/
* https://patchwork.ozlabs.org/patch/497299/
arch/arm/include/asm/arch-lpc32xx/config.h | 29 +++++++++++++++++++++++++++++
include/configs/devkit3250.h | 14 +++-----------
2 files changed, 32 insertions(+), 11 deletions(-)
diff --git a/arch/arm/include/asm/arch-lpc32xx/config.h b/arch/arm/include/asm/arch-lpc32xx/config.h
index d57bc48..d161ad2 100644
--- a/arch/arm/include/asm/arch-lpc32xx/config.h
+++ b/arch/arm/include/asm/arch-lpc32xx/config.h
@@ -55,6 +55,35 @@
/* Ethernet */
#define LPC32XX_ETH_BASE ETHERNET_BASE
+/* NAND */
+#if defined(CONFIG_NAND_LPC32XX_SLC)
+#define NAND_LARGE_BLOCK_PAGE_SIZE 0x800
+#define NAND_SMALL_BLOCK_PAGE_SIZE 0x200
+
+#if !defined(CONFIG_SYS_NAND_PAGE_SIZE)
+#define CONFIG_SYS_NAND_PAGE_SIZE NAND_LARGE_BLOCK_PAGE_SIZE
+#endif
+
+#if (CONFIG_SYS_NAND_PAGE_SIZE == NAND_LARGE_BLOCK_PAGE_SIZE)
+#define CONFIG_SYS_NAND_OOBSIZE 64
+#define CONFIG_SYS_NAND_ECCPOS { 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_BAD_BLOCK_POS NAND_LARGE_BADBLOCK_POS
+#elif (CONFIG_SYS_NAND_PAGE_SIZE == NAND_SMALL_BLOCK_PAGE_SIZE)
+#define CONFIG_SYS_NAND_OOBSIZE 16
+#define CONFIG_SYS_NAND_ECCPOS { 10, 11, 12, 13, 14, 15, }
+#define CONFIG_SYS_NAND_BAD_BLOCK_POS 0
+#else
+#error "CONFIG_SYS_NAND_PAGE_SIZE set to an invalid value"
+#endif
+
+#define CONFIG_SYS_NAND_ECCSIZE 0x100
+#define CONFIG_SYS_NAND_ECCBYTES 3
+#define CONFIG_SYS_NAND_PAGE_COUNT (CONFIG_SYS_NAND_BLOCK_SIZE / \
+ CONFIG_SYS_NAND_PAGE_SIZE)
+#endif /* CONFIG_NAND_LPC32XX_SLC */
+
/* NOR Flash */
#if defined(CONFIG_SYS_FLASH_CFI)
#define CONFIG_FLASH_CFI_DRIVER
diff --git a/include/configs/devkit3250.h b/include/configs/devkit3250.h
index cc6a53e..d3b7620 100644
--- a/include/configs/devkit3250.h
+++ b/include/configs/devkit3250.h
@@ -111,7 +111,10 @@
#define CONFIG_LPC32XX_NAND_SLC_RHOLD 200000000
#define CONFIG_LPC32XX_NAND_SLC_RSETUP 50000000
+#define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
+#define CONFIG_SYS_NAND_PAGE_SIZE NAND_LARGE_BLOCK_PAGE_SIZE
#define CONFIG_SYS_NAND_USE_FLASH_BBT
+
#define CONFIG_CMD_NAND
/*
@@ -199,17 +202,6 @@
#define CONFIG_SPL_NAND_SUPPORT
#define CONFIG_SPL_NAND_DRIVERS
-#define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
-#define CONFIG_SYS_NAND_PAGE_SIZE 0x800
-#define CONFIG_SYS_NAND_ECCSIZE 0x100
-#define CONFIG_SYS_NAND_OOBSIZE 64
-#define CONFIG_SYS_NAND_ECCPOS { 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_ECCBYTES 3
-#define CONFIG_SYS_NAND_PAGE_COUNT 64
-#define CONFIG_SYS_NAND_BAD_BLOCK_POS NAND_LARGE_BADBLOCK_POS
-
#define CONFIG_SPL_NAND_ECC
#define CONFIG_SPL_NAND_SOFTECC
--
2.1.4
^ permalink raw reply related [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h
2015-08-11 16:57 [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h Vladimir Zapolskiy
@ 2015-08-11 20:16 ` LEMIEUX, SYLVAIN
2015-08-11 22:09 ` Vladimir Zapolskiy
2015-08-18 17:53 ` [U-Boot] " Tom Rini
1 sibling, 1 reply; 5+ messages in thread
From: LEMIEUX, SYLVAIN @ 2015-08-11 20:16 UTC (permalink / raw)
To: u-boot
Hi Vladimir,
> -----Original Message-----
> From: Vladimir Zapolskiy [mailto:vz at mleia.com]
> Sent: 11-Aug-15 12:57 PM
>
> A number of LPC32xx SLC NAND defines is dictated by controller
> hardware limits and OOB layout is defined by operating system, the
> definitions are common for all users. Since those macro are used
> in out of NAND SLC driver code (simple NAND SPL framework), they can
> not be placed into the driver, therefore move them from board config
> files to arch/config.h
>
> The change also adds OOB layout details specific to small page NAND
> devices taken from Linux kernel.
>
> Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
> ---
> The change is based on
> * https://patchwork.ozlabs.org/patch/497298/
> * https://patchwork.ozlabs.org/patch/497299/
>
> arch/arm/include/asm/arch-lpc32xx/config.h | 29 +++++++++++++++++++++++++++++
> include/configs/devkit3250.h | 14 +++-----------
> 2 files changed, 32 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm/include/asm/arch-lpc32xx/config.h b/arch/arm/include/asm/arch-lpc32xx/config.h
> index d57bc48..d161ad2 100644
> --- a/arch/arm/include/asm/arch-lpc32xx/config.h
> +++ b/arch/arm/include/asm/arch-lpc32xx/config.h
> @@ -55,6 +55,35 @@
> /* Ethernet */
> #define LPC32XX_ETH_BASE ETHERNET_BASE
>
> +/* NAND */
> +#if defined(CONFIG_NAND_LPC32XX_SLC)
[..]
> +#if (CONFIG_SYS_NAND_PAGE_SIZE == NAND_LARGE_BLOCK_PAGE_SIZE)
> +#define CONFIG_SYS_NAND_OOBSIZE 64
> +#define CONFIG_SYS_NAND_ECCPOS { 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_BAD_BLOCK_POS NAND_LARGE_BADBLOCK_POS
Should we only move to "config.h", the definition that are use
by the NAND SPL framework and the NAND SLC driver?
The following definition are only use by the NAND SPL framework:
* CONFIG_SYS_NAND_ECCPOS
* CONFIG_SYS_NAND_BAD_BLOCK_POS
* CONFIG_SYS_NAND_PAGE_COUNT
[...]
> +#define CONFIG_SYS_NAND_ECCSIZE 0x100
> +#define CONFIG_SYS_NAND_ECCBYTES 3
> +
Do you want to change "lpc32xx_nand_slc.c" to use those 2 define
as part of this patch or do you prefer I change them as part of the
"nand: LPC32xx: add hardware ECC support" patch?
[...]
> --
> 2.1.4
________________________________
This e-mail contains privileged and confidential information intended for the use of the addressees named above. If you are not the intended recipient of this e-mail, you are hereby notified that you must not disseminate, copy or take any action in respect of any information contained in it. If you have received this e-mail in error, please notify the sender immediately by e-mail and immediately destroy this e-mail and its attachments.
^ permalink raw reply [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h
2015-08-11 20:16 ` LEMIEUX, SYLVAIN
@ 2015-08-11 22:09 ` Vladimir Zapolskiy
2015-08-12 19:31 ` LEMIEUX, SYLVAIN
0 siblings, 1 reply; 5+ messages in thread
From: Vladimir Zapolskiy @ 2015-08-11 22:09 UTC (permalink / raw)
To: u-boot
Hi Sylvain,
On 11.08.2015 23:16, LEMIEUX, SYLVAIN wrote:
> Hi Vladimir,
>
>> -----Original Message-----
>> From: Vladimir Zapolskiy [mailto:vz at mleia.com]
>> Sent: 11-Aug-15 12:57 PM
>>
>> A number of LPC32xx SLC NAND defines is dictated by controller
>> hardware limits and OOB layout is defined by operating system, the
>> definitions are common for all users. Since those macro are used
>> in out of NAND SLC driver code (simple NAND SPL framework), they can
>> not be placed into the driver, therefore move them from board config
>> files to arch/config.h
>>
>> The change also adds OOB layout details specific to small page NAND
>> devices taken from Linux kernel.
>>
>> Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
>> ---
>> The change is based on
>> * https://patchwork.ozlabs.org/patch/497298/
>> * https://patchwork.ozlabs.org/patch/497299/
>>
>> arch/arm/include/asm/arch-lpc32xx/config.h | 29 +++++++++++++++++++++++++++++
>> include/configs/devkit3250.h | 14 +++-----------
>> 2 files changed, 32 insertions(+), 11 deletions(-)
>>
>> diff --git a/arch/arm/include/asm/arch-lpc32xx/config.h b/arch/arm/include/asm/arch-lpc32xx/config.h
>> index d57bc48..d161ad2 100644
>> --- a/arch/arm/include/asm/arch-lpc32xx/config.h
>> +++ b/arch/arm/include/asm/arch-lpc32xx/config.h
>> @@ -55,6 +55,35 @@
>> /* Ethernet */
>> #define LPC32XX_ETH_BASE ETHERNET_BASE
>>
>> +/* NAND */
>> +#if defined(CONFIG_NAND_LPC32XX_SLC)
>
> [..]
>
>> +#if (CONFIG_SYS_NAND_PAGE_SIZE == NAND_LARGE_BLOCK_PAGE_SIZE)
>> +#define CONFIG_SYS_NAND_OOBSIZE 64
>> +#define CONFIG_SYS_NAND_ECCPOS { 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_BAD_BLOCK_POS NAND_LARGE_BADBLOCK_POS
>
> Should we only move to "config.h", the definition that are use
> by the NAND SPL framework and the NAND SLC driver?
>
> The following definition are only use by the NAND SPL framework:
> * CONFIG_SYS_NAND_ECCPOS
> * CONFIG_SYS_NAND_BAD_BLOCK_POS
> * CONFIG_SYS_NAND_PAGE_COUNT
The NAND SPL framework uses a longer list of macro definitions (please
check the source code, if you wish):
* CONFIG_SYS_NAND_BLOCK_SIZE
* CONFIG_SYS_NAND_PAGE_SIZE
* CONFIG_SYS_NAND_PAGE_COUNT
* CONFIG_SYS_NAND_ECCSIZE
* CONFIG_SYS_NAND_ECCPOS
* CONFIG_SYS_NAND_ECCBYTES
* CONFIG_SYS_NAND_OOBSIZE
* CONFIG_SYS_NAND_BAD_BLOCK_POS
All of them in the case of large page NAND device are found in
https://patchwork.ozlabs.org/patch/497299/ change and the given values
are common for all LPC32xx boards utilizing NAND SLC.
Generally I would say that not only the macro related to NAND SPL
framework should be moved, but also all the macro dictated by the
hardware constraints, e.g. ECC size and bytes, and all the macro shared
with other platform users like operating systems, this includes for
instance OOB layout and bad block position in OOB.
In other words this list includes everything common among all possible
LPC32xx boards with NAND SLC controller enabled, and hopefully this
change addresses all such macro. If you find any inconsistencies or
missing data in the change, please let me know.
> [...]
>> +#define CONFIG_SYS_NAND_ECCSIZE 0x100
>> +#define CONFIG_SYS_NAND_ECCBYTES 3
>> +
> Do you want to change "lpc32xx_nand_slc.c" to use those 2 define
> as part of this patch or do you prefer I change them as part of the
> "nand: LPC32xx: add hardware ECC support" patch?
This is fine with me, I would appreciate, if you can rebase 2/5 and 3/5
from your last series on top of this change, it should be guaranteed
that the compilation is not broken for devkit3250 board in assumption
that all my previously sent patches are applied.
--
With best wishes,
Vladimir
^ permalink raw reply [flat|nested] 5+ messages in thread* [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h
2015-08-11 22:09 ` Vladimir Zapolskiy
@ 2015-08-12 19:31 ` LEMIEUX, SYLVAIN
0 siblings, 0 replies; 5+ messages in thread
From: LEMIEUX, SYLVAIN @ 2015-08-12 19:31 UTC (permalink / raw)
To: u-boot
Hi Vladimir,
> -----Original Message-----
> From: Vladimir Zapolskiy [mailto:vz at mleia.com]
> Sent: 11-Aug-15 6:10 PM
>
> Hi Sylvain,
>
> On 11.08.2015 23:16, LEMIEUX, SYLVAIN wrote:
> > Hi Vladimir,
> >
> >> -----Original Message-----
> >> From: Vladimir Zapolskiy [mailto:vz at mleia.com]
> >> Sent: 11-Aug-15 12:57 PM
> >>
> >> A number of LPC32xx SLC NAND defines is dictated by controller
> >> hardware limits and OOB layout is defined by operating system, the
> >> definitions are common for all users. Since those macro are used
> >> in out of NAND SLC driver code (simple NAND SPL framework), they can
> >> not be placed into the driver, therefore move them from board config
> >> files to arch/config.h
> >>
> >> The change also adds OOB layout details specific to small page NAND
> >> devices taken from Linux kernel.
> >>
> >> Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
> >> ---
> >> The change is based on
> >> * https://patchwork.ozlabs.org/patch/497298/
> >> * https://patchwork.ozlabs.org/patch/497299/
> >>
> >> arch/arm/include/asm/arch-lpc32xx/config.h | 29 +++++++++++++++++++++++++++++
> >> include/configs/devkit3250.h | 14 +++-----------
> >> 2 files changed, 32 insertions(+), 11 deletions(-)
> >>
> >> diff --git a/arch/arm/include/asm/arch-lpc32xx/config.h b/arch/arm/include/asm/arch-lpc32xx/config.h
> >> index d57bc48..d161ad2 100644
> >> --- a/arch/arm/include/asm/arch-lpc32xx/config.h
> >> +++ b/arch/arm/include/asm/arch-lpc32xx/config.h
> >> @@ -55,6 +55,35 @@
> >> /* Ethernet */
> >> #define LPC32XX_ETH_BASE ETHERNET_BASE
> >>
> >> +/* NAND */
> >> +#if defined(CONFIG_NAND_LPC32XX_SLC)
> >
> > [..]
> >
> >> +#if (CONFIG_SYS_NAND_PAGE_SIZE == NAND_LARGE_BLOCK_PAGE_SIZE)
> >> +#define CONFIG_SYS_NAND_OOBSIZE 64
> >> +#define CONFIG_SYS_NAND_ECCPOS { 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_BAD_BLOCK_POS NAND_LARGE_BADBLOCK_POS
> >
> > Should we only move to "config.h", the definition that are use
> > by the NAND SPL framework and the NAND SLC driver?
> >
>
[...]
>
> In other words this list includes everything common among all possible
> LPC32xx boards with NAND SLC controller enabled, and hopefully this
> change addresses all such macro. If you find any inconsistencies or
> missing data in the change, please let me know.
>
OK
> > [...]
> >> +#define CONFIG_SYS_NAND_ECCSIZE 0x100
> >> +#define CONFIG_SYS_NAND_ECCBYTES 3
> >> +
> > Do you want to change "lpc32xx_nand_slc.c" to use those 2 define
> > as part of this patch or do you prefer I change them as part of the
> > "nand: LPC32xx: add hardware ECC support" patch?
>
> This is fine with me, I would appreciate, if you can rebase 2/5 and 3/5
> from your last series on top of this change, it should be guaranteed
> that the compilation is not broken for devkit3250 board in assumption
> that all my previously sent patches are applied.
I already apply this patch locally and I will rebase the next revision of
patch 2/5 & 3/5 on top of this patch.
I tested the change for the lpc32xx NAND SLC driver;
Tested-by: Sylvain Lemieux <slemieux@tycoint.com>
>
> --
> With best wishes,
> Vladimir
________________________________
This e-mail contains privileged and confidential information intended for the use of the addressees named above. If you are not the intended recipient of this e-mail, you are hereby notified that you must not disseminate, copy or take any action in respect of any information contained in it. If you have received this e-mail in error, please notify the sender immediately by e-mail and immediately destroy this e-mail and its attachments.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [U-Boot] lpc32xx: move common SLC NAND defines to arch/config.h
2015-08-11 16:57 [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h Vladimir Zapolskiy
2015-08-11 20:16 ` LEMIEUX, SYLVAIN
@ 2015-08-18 17:53 ` Tom Rini
1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2015-08-18 17:53 UTC (permalink / raw)
To: u-boot
On Tue, Aug 11, 2015 at 07:57:09PM +0300, Vladimir Zapolskiy wrote:
> A number of LPC32xx SLC NAND defines is dictated by controller
> hardware limits and OOB layout is defined by operating system, the
> definitions are common for all users. Since those macro are used
> in out of NAND SLC driver code (simple NAND SPL framework), they can
> not be placed into the driver, therefore move them from board config
> files to arch/config.h
>
> The change also adds OOB layout details specific to small page NAND
> devices taken from Linux kernel.
>
> Signed-off-by: Vladimir Zapolskiy <vz@mleia.com>
> Tested-by: Sylvain Lemieux <slemieux@tycoint.com>
Applied to u-boot/master, 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/20150818/cd2c235f/attachment.sig>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-08-18 17:53 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-11 16:57 [U-Boot] [PATCH] lpc32xx: move common SLC NAND defines to arch/config.h Vladimir Zapolskiy
2015-08-11 20:16 ` LEMIEUX, SYLVAIN
2015-08-11 22:09 ` Vladimir Zapolskiy
2015-08-12 19:31 ` LEMIEUX, SYLVAIN
2015-08-18 17:53 ` [U-Boot] " Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox