From: Derald Woods <woods.technical@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM: at91: Convert SPL_GENERATE_ATMEL_PMECC_HEADER to Kconfig
Date: Fri, 11 Jan 2019 10:51:14 -0600 [thread overview]
Message-ID: <CA+CtpRjRQ_OzHFpxqejgjzsczY73FF57ML6hhYj9rxenTzjmXg@mail.gmail.com> (raw)
In-Reply-To: <ca1a7c04-8b24-ab31-095d-324eff581471@microchip.com>
On Fri, Jan 11, 2019, 5:27 AM <Eugen.Hristev@microchip.com wrote:
>
>
> On 10.01.2019 03:00, Derald Woods wrote:
> > On Mon, Jan 7, 2019 at 3:40 AM <Eugen.Hristev@microchip.com
> > <mailto:Eugen.Hristev@microchip.com>> wrote:
> >
> > Hi Derald,
> >
> > Thank you for the patch,
> >
> > Some minor questions inline.
> >
> >
> >
> > Sorry for the late response. Family and day job take priority.
> >
> > The patch is the simplest technical conversion that I could produce with
> > my board (SAMA5D3-XPLD) as the test platform. I was very careful not to
> > touch anything unrelated. It is not intended to address permutations
> > outside of the PMECC Kconfig scope. I have submitted these type of
> > conversions to U-Boot in the past. Adding speculative things is risky
> > when you only have one development board. The patch works well, for me,
> > and addresses the original need. All items, in the patch, are currently
> > in scope. If there are additions and/or modifications, feel free to
> > modify the patch. It is complete from my perspective. I do not have any
> > additional time to chase this any further. I have been developing for
> > some time now and can handle an 'out of tree' patch, in my environment,
> > until something better comes along. This patch is just my contribution
> > to the process.
>
> Hello Derald,
>
> From my point of view the PMECC header generation for SPL should be
> done if and only if the SPL is being placed in NAND flash namely
> defconfigs having *nandflash* in the name.
>
> Thus I disagree with enabling it on spiflash configurations (like you
> did for the *mmc* configurations looks OK)
>
The patch is about 'one' thing. Kconfig conversion. Things that you are
mentioning do not invalidate my patch. I tried not to add things that were
not there originally. That is generally discouraged scope creep. The patch
does its job in a provable manner with changing existing usage.
Derald
If you have any reasons or arguments , or anyone else does, please
> correct me.
>
> If you do not have time to change the patch, that is OK, I can take over
> at some point when I have the time or someone else can pick and rework a
> little bit on that specific point and make a v2.
>
> Thanks again,
>
> Eugen
>
> >
> > Cheers,
> >
> > Derald
> >
> >
> > On 15.12.2018 09:36, Derald D. Woods wrote:
> > > This commit converts the following items to Kconfig:
> > >
> > > CONFIG_ATMEL_NAND_HWECC
> > > CONFIG_ATMEL_NAND_HW_PMECC
> > > CONFIG_PMECC_CAP
> > > CONFIG_PMECC_SECTOR_SIZE
> > > CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > [PMECC References]
> > > https://www.at91.com/linux4sam/bin/view/Linux4SAM/PmeccConfigure
> > > https://www.at91.com/linux4sam/bin/view/Linux4SAM/AT91Bootstrap
> > >
> > > [Mailing List Thread]
> > > https://lists.denx.de/pipermail/u-boot/2018-December/350666.html
> > >
> > > Fixes: 5541543f ("configs: at91: Remove CONFIG_SYS_EXTRA_OPTIONS
> > assignment")
> > > Reported-by: Daniel Evans <photonthunder@gmail.com
> > <mailto:photonthunder@gmail.com>>
> > > Cc: Eugen Hristev <eugen.hristev@microchip.com
> > <mailto:eugen.hristev@microchip.com>>
> > > Signed-off-by: Derald D. Woods <woods.technical@gmail.com
> > <mailto:woods.technical@gmail.com>>
> > > ---
> > > configs/at91sam9n12ek_mmc_defconfig | 1 +
> > > configs/at91sam9n12ek_nandflash_defconfig | 3 ++
> > > configs/at91sam9n12ek_spiflash_defconfig | 3 ++
> > > configs/at91sam9x5ek_dataflash_defconfig | 3 ++
> > > configs/at91sam9x5ek_mmc_defconfig | 1 +
> > > configs/at91sam9x5ek_nandflash_defconfig | 3 ++
> > > configs/at91sam9x5ek_spiflash_defconfig | 3 ++
> > > configs/sama5d3_xplained_mmc_defconfig | 1 +
> > > configs/sama5d3_xplained_nandflash_defconfig | 3 ++
> > > configs/sama5d3xek_mmc_defconfig | 1 +
> > > configs/sama5d3xek_nandflash_defconfig | 3 ++
> > > configs/sama5d3xek_spiflash_defconfig | 3 ++
> > > configs/sama5d4_xplained_mmc_defconfig | 1 +
> > > configs/sama5d4_xplained_nandflash_defconfig | 3 ++
> > > configs/sama5d4_xplained_spiflash_defconfig | 3 ++
> > > configs/sama5d4ek_mmc_defconfig | 1 +
> > > configs/sama5d4ek_nandflash_defconfig | 3 ++
> > > configs/sama5d4ek_spiflash_defconfig | 3 ++
> > > configs/wb45n_defconfig | 3 ++
> > > configs/wb50n_defconfig | 3 ++
> > > doc/README.atmel_pmecc | 13 ++++---
> > > drivers/mtd/nand/raw/Kconfig | 38
> > ++++++++++++++++++++
> > > include/configs/at91sam9n12ek.h | 7 ----
> > > include/configs/at91sam9x5ek.h | 7 ----
> > > include/configs/sama5d3_xplained.h | 7 ----
> > > include/configs/sama5d3xek.h | 7 ----
> > > include/configs/sama5d4_xplained.h | 6 ----
> > > include/configs/sama5d4ek.h | 6 ----
> > > include/configs/wb45n.h | 7 ----
> > > include/configs/wb50n.h | 6 ----
> > > scripts/config_whitelist.txt | 5 ---
> > > 31 files changed, 92 insertions(+), 65 deletions(-)
> > >
> > > diff --git a/configs/at91sam9n12ek_mmc_defconfig
> > b/configs/at91sam9n12ek_mmc_defconfig
> > > index 6b2cfe9c42..7b46eb498d 100644
> > > --- a/configs/at91sam9n12ek_mmc_defconfig
> > > +++ b/configs/at91sam9n12ek_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9N12EK=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9n12ek_nandflash_defconfig
> > b/configs/at91sam9n12ek_nandflash_defconfig
> > > index 354c24ff16..4e34a517fd 100644
> > > --- a/configs/at91sam9n12ek_nandflash_defconfig
> > > +++ b/configs/at91sam9n12ek_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9N12EK=y
> > > +CONFIG_PMECC_CAP=2
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9n12ek_spiflash_defconfig
> > b/configs/at91sam9n12ek_spiflash_defconfig
> > > index 63889355bf..500c3ab088 100644
> > > --- a/configs/at91sam9n12ek_spiflash_defconfig
> > > +++ b/configs/at91sam9n12ek_spiflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9N12EK=y
> > > +CONFIG_PMECC_CAP=2
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> >
> > Do we need to generate the PMECC header for SPI_BOOT configurations ?
> > I would say this would be required only for NAND_BOOT
> >
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9x5ek_dataflash_defconfig
> > b/configs/at91sam9x5ek_dataflash_defconfig
> > > index dc13509715..60d865cc23 100644
> > > --- a/configs/at91sam9x5ek_dataflash_defconfig
> > > +++ b/configs/at91sam9x5ek_dataflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9X5EK=y
> > > +CONFIG_PMECC_CAP=2
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9x5ek_mmc_defconfig
> > b/configs/at91sam9x5ek_mmc_defconfig
> > > index ff86f93e61..75f7fbd199 100644
> > > --- a/configs/at91sam9x5ek_mmc_defconfig
> > > +++ b/configs/at91sam9x5ek_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9X5EK=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9x5ek_nandflash_defconfig
> > b/configs/at91sam9x5ek_nandflash_defconfig
> > > index b2b3ddb6d9..dd6bdee7f9 100644
> > > --- a/configs/at91sam9x5ek_nandflash_defconfig
> > > +++ b/configs/at91sam9x5ek_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9X5EK=y
> > > +CONFIG_PMECC_CAP=2
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/at91sam9x5ek_spiflash_defconfig
> > b/configs/at91sam9x5ek_spiflash_defconfig
> > > index d0eebcdc75..8d8108888c 100644
> > > --- a/configs/at91sam9x5ek_spiflash_defconfig
> > > +++ b/configs/at91sam9x5ek_spiflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_AT91SAM9X5EK=y
> > > +CONFIG_PMECC_CAP=2
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> >
> > Ditto.
> >
> > > CONFIG_SYS_MALLOC_F_LEN=0x2000
> > > CONFIG_DEBUG_UART_BOARD_INIT=y
> > > CONFIG_DEBUG_UART_BASE=0xfffff200
> > > diff --git a/configs/sama5d3_xplained_mmc_defconfig
> > b/configs/sama5d3_xplained_mmc_defconfig
> > > index eab38ec366..51d5aeaf0e 100644
> > > --- a/configs/sama5d3_xplained_mmc_defconfig
> > > +++ b/configs/sama5d3_xplained_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D3_XPLAINED=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d3_xplained_nandflash_defconfig
> > b/configs/sama5d3_xplained_nandflash_defconfig
> > > index ff7d2bffdd..ce1f28607e 100644
> > > --- a/configs/sama5d3_xplained_nandflash_defconfig
> > > +++ b/configs/sama5d3_xplained_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D3_XPLAINED=y
> > > +CONFIG_PMECC_CAP=4
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d3xek_mmc_defconfig
> > b/configs/sama5d3xek_mmc_defconfig
> > > index 6faea0ec3f..49d4334213 100644
> > > --- a/configs/sama5d3xek_mmc_defconfig
> > > +++ b/configs/sama5d3xek_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D3XEK=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d3xek_nandflash_defconfig
> > b/configs/sama5d3xek_nandflash_defconfig
> > > index e641279ee1..b0d56e0e3e 100644
> > > --- a/configs/sama5d3xek_nandflash_defconfig
> > > +++ b/configs/sama5d3xek_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D3XEK=y
> > > +CONFIG_PMECC_CAP=4
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d3xek_spiflash_defconfig
> > b/configs/sama5d3xek_spiflash_defconfig
> > > index 37f603d528..cf2fb9e65b 100644
> > > --- a/configs/sama5d3xek_spiflash_defconfig
> > > +++ b/configs/sama5d3xek_spiflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D3XEK=y
> > > +CONFIG_PMECC_CAP=4
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> >
> > Ditto
> >
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4_xplained_mmc_defconfig
> > b/configs/sama5d4_xplained_mmc_defconfig
> > > index bfcea3f519..35b8302b60 100644
> > > --- a/configs/sama5d4_xplained_mmc_defconfig
> > > +++ b/configs/sama5d4_xplained_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4_XPLAINED=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4_xplained_nandflash_defconfig
> > b/configs/sama5d4_xplained_nandflash_defconfig
> > > index 2c1b7f17c0..93714ed0c7 100644
> > > --- a/configs/sama5d4_xplained_nandflash_defconfig
> > > +++ b/configs/sama5d4_xplained_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4_XPLAINED=y
> > > +CONFIG_PMECC_CAP=8
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4_xplained_spiflash_defconfig
> > b/configs/sama5d4_xplained_spiflash_defconfig
> > > index a89dd11a8c..e1eb0c72cd 100644
> > > --- a/configs/sama5d4_xplained_spiflash_defconfig
> > > +++ b/configs/sama5d4_xplained_spiflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4_XPLAINED=y
> > > +CONFIG_PMECC_CAP=8
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> >
> > Ditto
> >
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4ek_mmc_defconfig
> > b/configs/sama5d4ek_mmc_defconfig
> > > index 45e6539913..bd238f7f14 100644
> > > --- a/configs/sama5d4ek_mmc_defconfig
> > > +++ b/configs/sama5d4ek_mmc_defconfig
> > > @@ -2,6 +2,7 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4EK=y
> > > +# CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER is not set
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4ek_nandflash_defconfig
> > b/configs/sama5d4ek_nandflash_defconfig
> > > index 1497110452..3816bfb273 100644
> > > --- a/configs/sama5d4ek_nandflash_defconfig
> > > +++ b/configs/sama5d4ek_nandflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4EK=y
> > > +CONFIG_PMECC_CAP=8
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/sama5d4ek_spiflash_defconfig
> > b/configs/sama5d4ek_spiflash_defconfig
> > > index dad32b5711..242081e6a1 100644
> > > --- a/configs/sama5d4ek_spiflash_defconfig
> > > +++ b/configs/sama5d4ek_spiflash_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x26f00000
> > > CONFIG_TARGET_SAMA5D4EK=y
> > > +CONFIG_PMECC_CAP=8
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> >
> > Ditto
> >
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/wb45n_defconfig b/configs/wb45n_defconfig
> > > index 9d881e5d41..15fc17ac00 100644
> > > --- a/configs/wb45n_defconfig
> > > +++ b/configs/wb45n_defconfig
> > > @@ -3,6 +3,9 @@ CONFIG_SYS_THUMB_BUILD=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x23f00000
> > > CONFIG_TARGET_WB45N=y
> > > +CONFIG_PMECC_CAP=4
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/configs/wb50n_defconfig b/configs/wb50n_defconfig
> > > index c74eba1012..a7e9a53448 100644
> > > --- a/configs/wb50n_defconfig
> > > +++ b/configs/wb50n_defconfig
> > > @@ -2,6 +2,9 @@ CONFIG_ARM=y
> > > CONFIG_ARCH_AT91=y
> > > CONFIG_SYS_TEXT_BASE=0x23f00000
> > > CONFIG_TARGET_WB50N=y
> > > +CONFIG_PMECC_CAP=8
> > > +CONFIG_PMECC_SECTOR_SIZE=512
> > > +CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > > CONFIG_SPL_GPIO_SUPPORT=y
> > > CONFIG_SPL_LIBCOMMON_SUPPORT=y
> > > CONFIG_SPL_LIBGENERIC_SUPPORT=y
> > > diff --git a/doc/README.atmel_pmecc b/doc/README.atmel_pmecc
> > > index 274d860f06..c86d085779 100644
> > > --- a/doc/README.atmel_pmecc
> > > +++ b/doc/README.atmel_pmecc
> > > @@ -20,13 +20,12 @@ To use PMECC in this driver, the user needs
> > to set:
> > > 2. The PMECC sector size: CONFIG_PMECC_SECTOR_SIZE.
> > > It only can be 512 or 1024.
> > >
> > > -Take AT91SAM9X5EK as an example, the board definition file likes:
> > > +Take 'configs/at91sam9x5ek_nandflash_defconfig' as an example,
> > the board
> > > +configuration file has the following entries:
> > >
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC 1
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC 1
> > > -#define CONFIG_PMECC_CAP 2
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > + CONFIG_PMECC_CAP=2
> > > + CONFIG_PMECC_SECTOR_SIZE=512
> > > + CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER=y
> > >
> > > How to enable PMECC header for direct programmable boot.bin
> > > -----------------------------------------------------------
> > > @@ -40,7 +39,7 @@ sama5d3 SoC spec (as of 03. April 2014) defines
> > how this PMECC header has to
> > > look like. In order to do so we have a new image type added to
> > mkimage to
> > > generate this PMECC header and integrated this into the build
> > process of SPL.
> > >
> > > -To enable the generation of atmel PMECC header for SPL one need
> > to define
> > > +To enable the generation of atmel PMECC header for SPL one needs
> > to define
> > > CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER. The required parameters
> > are taken from
> > > board configuration and compiled into the host tools
> > atmel_pmecc_params. This
> > > tool will be called in build process to parametrize mkimage for
> > atmelimage
> > > diff --git a/drivers/mtd/nand/raw/Kconfig
> > b/drivers/mtd/nand/raw/Kconfig
> > > index 008f7b4b4b..b99732736a 100644
> > > --- a/drivers/mtd/nand/raw/Kconfig
> > > +++ b/drivers/mtd/nand/raw/Kconfig
> > > @@ -16,6 +16,44 @@ config NAND_ATMEL
> > > Enable this driver for NAND flash platforms using an
> > Atmel NAND
> > > controller.
> > >
> > > +if NAND_ATMEL
> > > +
> > > +config ATMEL_NAND_HWECC
> > > + bool "Atmel Hardware ECC"
> > > + default n
> > > +
> > > +config ATMEL_NAND_HW_PMECC
> > > + bool "Atmel Programmable Multibit ECC (PMECC)"
> > > + select ATMEL_NAND_HWECC
> > > + default n
> > > + help
> > > + The Programmable Multibit ECC (PMECC) controller is a
> > programmable
> > > + binary BCH(Bose, Chaudhuri and Hocquenghem) encoder and
> > decoder.
> > > +
> > > +config PMECC_CAP
> > > + int "PMECC Correctable ECC Bits"
> > > + depends on ATMEL_NAND_HW_PMECC
> > > + default 2
> > > + help
> > > + Correctable ECC bits, can be 2, 4, 8, 12, and 24.
> > > +
> > > +config PMECC_SECTOR_SIZE
> > > + int "PMECC Sector Size"
> > > + depends on ATMEL_NAND_HW_PMECC
> > > + default 512
> > > + help
> > > + Sector size, in bytes, can be 512 or 1024.
> > > +
> > > +config SPL_GENERATE_ATMEL_PMECC_HEADER
> > > + bool "Atmel PMECC Header Generation"
> > > + select ATMEL_NAND_HWECC
> > > + select ATMEL_NAND_HW_PMECC
> > > + default n
> > > + help
> > > + Generate Programmable Multibit ECC (PMECC) header for SPL
> > image.
> > > +
> > > +endif
> > > +
> > > config NAND_DAVINCI
> > > bool "Support TI Davinci NAND controller"
> > > help
> > > diff --git a/include/configs/at91sam9n12ek.h
> > b/include/configs/at91sam9n12ek.h
> > > index 6cd267eee6..777a99b730 100644
> > > --- a/include/configs/at91sam9n12ek.h
> > > +++ b/include/configs/at91sam9n12ek.h
> > > @@ -59,12 +59,6 @@
> > > #define CONFIG_SYS_NAND_READY_PIN GPIO_PIN_PD(5)
> > > #endif
> > >
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > -#define CONFIG_PMECC_CAP 2
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > -
> > > #define CONFIG_EXTRA_ENV_SETTINGS
> > \
> > > "console=console=ttyS0,115200\0"
> > \
> > > "mtdparts="CONFIG_MTDPARTS_DEFAULT"\0"
> > \
> > > @@ -177,6 +171,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/at91sam9x5ek.h
> > b/include/configs/at91sam9x5ek.h
> > > index 63305a7cdd..6adb965c3c 100644
> > > --- a/include/configs/at91sam9x5ek.h
> > > +++ b/include/configs/at91sam9x5ek.h
> > > @@ -55,12 +55,6 @@
> > > #define CONFIG_SYS_NAND_READY_PIN AT91_PIN_PD5
> > > #endif
> > >
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC 1
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC 1
> > > -#define CONFIG_PMECC_CAP 2
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > -
> > > /* USB */
> > > #ifdef CONFIG_CMD_USB
> > > #ifndef CONFIG_USB_EHCI_HCD
> > > @@ -151,6 +145,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/sama5d3_xplained.h
> > b/include/configs/sama5d3_xplained.h
> > > index d0d8087ca3..8a9a19d38e 100644
> > > --- a/include/configs/sama5d3_xplained.h
> > > +++ b/include/configs/sama5d3_xplained.h
> > > @@ -43,14 +43,8 @@
> > > #define CONFIG_SYS_NAND_MASK_CLE (1 << 22)
> > > #define CONFIG_SYS_NAND_ONFI_DETECTION
> > > #endif
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > -#define CONFIG_PMECC_CAP 4
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > >
> > > /* USB */
> > > -
> > > #ifdef CONFIG_CMD_USB
> > > #define CONFIG_USB_ATMEL
> > > #define CONFIG_USB_ATMEL_CLK_SEL_UPLL
> > > @@ -88,6 +82,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/sama5d3xek.h
> > b/include/configs/sama5d3xek.h
> > > index 4d3c3b8314..ca1c2b0861 100644
> > > --- a/include/configs/sama5d3xek.h
> > > +++ b/include/configs/sama5d3xek.h
> > > @@ -62,14 +62,8 @@
> > > #define CONFIG_SYS_NAND_MASK_CLE (1 << 22)
> > > #define CONFIG_SYS_NAND_ONFI_DETECTION
> > > #endif
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > -#define CONFIG_PMECC_CAP 4
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > >
> > > /* USB */
> > > -
> > Unrelated change ?
> >
> > > #ifdef CONFIG_CMD_USB
> > > #define CONFIG_USB_ATMEL_CLK_SEL_UPLL
> > > #define CONFIG_USB_OHCI_NEW
> > > @@ -109,6 +103,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/sama5d4_xplained.h
> > b/include/configs/sama5d4_xplained.h
> > > index 7f8ac178f3..bbb16993ec 100644
> > > --- a/include/configs/sama5d4_xplained.h
> > > +++ b/include/configs/sama5d4_xplained.h
> > > @@ -37,9 +37,6 @@
> > > /* our CLE is AD22 */
> > > #define CONFIG_SYS_NAND_MASK_CLE (1 << 22)
> > > #define CONFIG_SYS_NAND_ONFI_DETECTION
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > #endif
> > >
> > > /* SPL */
> > > @@ -64,8 +61,6 @@
> > > #define CONFIG_SPL_NAND_DRIVERS
> > > #define CONFIG_SPL_NAND_BASE
> > > #endif
> > > -#define CONFIG_PMECC_CAP 8
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > #define CONFIG_SYS_NAND_U_BOOT_OFFS 0x40000
> > > #define CONFIG_SYS_NAND_5_ADDR_CYCLE
> > > #define CONFIG_SYS_NAND_PAGE_SIZE 0x1000
> > > @@ -73,6 +68,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 224
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x40000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/sama5d4ek.h
> > b/include/configs/sama5d4ek.h
> > > index aa8573d8ba..d58041650c 100644
> > > --- a/include/configs/sama5d4ek.h
> > > +++ b/include/configs/sama5d4ek.h
> > > @@ -37,9 +37,6 @@
> > > /* our CLE is AD22 */
> > > #define CONFIG_SYS_NAND_MASK_CLE (1 << 22)
> > > #define CONFIG_SYS_NAND_ONFI_DETECTION
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > #endif
> > >
> > > /* SPL */
> > > @@ -63,8 +60,6 @@
> > > #define CONFIG_SPL_NAND_DRIVERS
> > > #define CONFIG_SPL_NAND_BASE
> > > #endif
> > > -#define CONFIG_PMECC_CAP 8
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > #define CONFIG_SYS_NAND_U_BOOT_OFFS 0x40000
> > > #define CONFIG_SYS_NAND_5_ADDR_CYCLE
> > > #define CONFIG_SYS_NAND_PAGE_SIZE 0x1000
> > > @@ -72,6 +67,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 224
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x40000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/include/configs/wb45n.h b/include/configs/wb45n.h
> > > index b516b66c35..add4019b31 100644
> > > --- a/include/configs/wb45n.h
> > > +++ b/include/configs/wb45n.h
> > > @@ -48,12 +48,6 @@
> > > #define CONFIG_SYS_NAND_ENABLE_PIN AT91_PIN_PD4
> > > #define CONFIG_SYS_NAND_READY_PIN AT91_PIN_PD5
> > >
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC 1
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC 1
> > > -#define CONFIG_PMECC_CAP 4
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > > -
> > > #define CONFIG_RBTREE
> > > #define CONFIG_LZO
> > >
> > > @@ -141,6 +135,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif /* __CONFIG_H__ */
> > > diff --git a/include/configs/wb50n.h b/include/configs/wb50n.h
> > > index 40ca9d602b..2684b6c16c 100644
> > > --- a/include/configs/wb50n.h
> > > +++ b/include/configs/wb50n.h
> > > @@ -57,11 +57,6 @@
> > > /* our CLE is AD22 */
> > > #define CONFIG_SYS_NAND_MASK_CLE (1 << 22)
> > > #define CONFIG_SYS_NAND_ONFI_DETECTION
> > > -/* PMECC & PMERRLOC */
> > > -#define CONFIG_ATMEL_NAND_HWECC
> > > -#define CONFIG_ATMEL_NAND_HW_PMECC
> > > -#define CONFIG_PMECC_CAP 8
> > > -#define CONFIG_PMECC_SECTOR_SIZE 512
> > >
> > > /* Ethernet Hardware */
> > > #define CONFIG_MACB
> > > @@ -118,6 +113,5 @@
> > > #define CONFIG_SYS_NAND_OOBSIZE 64
> > > #define CONFIG_SYS_NAND_BLOCK_SIZE 0x20000
> > > #define CONFIG_SYS_NAND_BAD_BLOCK_POS 0x0
> > > -#define CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > >
> > > #endif
> > > diff --git a/scripts/config_whitelist.txt
> > b/scripts/config_whitelist.txt
> > > index b8addeaf69..f798c6542a 100644
> > > --- a/scripts/config_whitelist.txt
> > > +++ b/scripts/config_whitelist.txt
> > > @@ -97,8 +97,6 @@ CONFIG_ATMEL_LCD_BGR555
> > > CONFIG_ATMEL_LCD_RGB565
> > > CONFIG_ATMEL_LEGACY
> > > CONFIG_ATMEL_MCI_8BIT
> > > -CONFIG_ATMEL_NAND_HWECC
> > > -CONFIG_ATMEL_NAND_HW_PMECC
> > > CONFIG_ATMEL_SPI0
> > > CONFIG_AT_TRANS
> > > CONFIG_AUTO_ZRELADDR
> > > @@ -1509,8 +1507,6 @@ CONFIG_PLATINUM_PROJECT
> > > CONFIG_PM
> > > CONFIG_PMC_BR_PRELIM
> > > CONFIG_PMC_OR_PRELIM
> > > -CONFIG_PMECC_CAP
> > > -CONFIG_PMECC_SECTOR_SIZE
> > > CONFIG_PME_PLAT_CLK_DIV
> > > CONFIG_PMU
> > > CONFIG_PMW_BASE
> > > @@ -1875,7 +1871,6 @@ CONFIG_SPL_FS_LOAD_ARGS_NAME
> > > CONFIG_SPL_FS_LOAD_KERNEL_NAME
> > > CONFIG_SPL_FS_LOAD_PAYLOAD_NAME
> > > CONFIG_SPL_GD_ADDR
> > > -CONFIG_SPL_GENERATE_ATMEL_PMECC_HEADER
> > > CONFIG_SPL_INIT_MINIMAL
> > > CONFIG_SPL_JR0_LIODN_NS
> > > CONFIG_SPL_JR0_LIODN_S
> > >
> >
>
next prev parent reply other threads:[~2019-01-11 16:51 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-05 0:15 [U-Boot] boot.bin on SD Card for SAMA5D3 Xplained Daniel Evans
2018-12-05 14:36 ` Eugen.Hristev at microchip.com
2018-12-05 18:08 ` Daniel Evans
2018-12-05 20:38 ` Robert Nelson
2018-12-06 8:02 ` Eugen.Hristev at microchip.com
2018-12-08 2:37 ` Derald D. Woods
2018-12-08 19:49 ` [U-Boot] [PATCH] ARM: at91: Fix 'boot.bin' generation when CONFIG_SD_BOOT is enabled Derald D. Woods
2018-12-10 8:32 ` Eugen.Hristev at microchip.com
2018-12-10 13:01 ` Derald D. Woods
2018-12-10 14:03 ` Eugen.Hristev at microchip.com
2018-12-10 14:54 ` Derald Woods
2018-12-10 15:14 ` Eugen.Hristev at microchip.com
2018-12-11 5:48 ` Derald D. Woods
2018-12-15 7:36 ` [U-Boot] [PATCH] ARM: at91: Convert SPL_GENERATE_ATMEL_PMECC_HEADER to Kconfig Derald D. Woods
2018-12-28 0:04 ` Derald Woods
2019-01-07 9:40 ` Eugen.Hristev at microchip.com
2019-01-10 1:00 ` Derald Woods
2019-01-11 11:27 ` Eugen.Hristev at microchip.com
2019-01-11 16:51 ` Derald Woods [this message]
2019-01-11 16:53 ` Derald Woods
2019-01-18 8:37 ` Eugen.Hristev at microchip.com
2019-01-19 3:26 ` Derald D. Woods
2019-01-19 20:02 ` [U-Boot] [PATCH 0/2] ARM: at91: NAND PMECC Kconfig conversion Derald D. Woods
2019-01-19 20:02 ` [U-Boot] [PATCH 1/2] nand: atmel: Replace SYS_NAND_ECC_BASE with ATMEL_BASE_ECC Derald D. Woods
2019-01-19 20:02 ` [U-Boot] [PATCH 2/2] ARM: at91: Convert SPL_GENERATE_ATMEL_PMECC_HEADER to Kconfig Derald D. Woods
2019-01-19 23:29 ` [U-Boot] [PATCH 0/2] ARM: at91: NAND PMECC Kconfig conversion Tom Rini
2019-01-20 0:50 ` Derald Woods
2019-01-20 3:37 ` [U-Boot] [PATCH v2 0/1] " Derald D. Woods
2019-01-20 3:37 ` [U-Boot] [PATCH v3 1/1] ARM: at91: Convert SPL_GENERATE_ATMEL_PMECC_HEADER to Kconfig Derald D. Woods
2019-01-20 12:57 ` Tom Rini
2019-01-20 14:25 ` Derald Woods
2019-01-21 16:58 ` [U-Boot] " Tom Rini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CA+CtpRjRQ_OzHFpxqejgjzsczY73FF57ML6hhYj9rxenTzjmXg@mail.gmail.com \
--to=woods.technical@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.