[U-Boot] [PATCH][v3] powerpc/mpc85xx:Disable Debug TLB entry for non-minimal SPL

2013-07-04 Thread Prabhakar Kushwaha
CONFIG_SPL_BUILD creates debug TLB entry, so disable it before init_tlbs. CONFIG_SPL_INIT_MINIMAL never creates any debug TLB entry, so no need of disable_tlb(). Signed-off-by: Prabhakar Kushwaha --- changes for v2: Removed checkpatch warning changes for v3: Incorporated Scott's comments

[U-Boot] Ethernet support on mx6slevk board

2013-07-04 Thread Shawn Guo
Hi, I'm running U-Boot 2013.07-rc2 and seeing Ethernet is failed to initialize. Is it a known issue or something not supported yet? It's unlikely my setup problem, since I have a vendor U-Boot running the board just fine. Shawn ___ U-Boot mailing list

Re: [U-Boot] [RESEND PATCH 1/4] arm:goni: Update configuration for goni target

2013-07-04 Thread Jaehoon Chung
On 07/04/2013 07:52 PM, Lukasz Majewski wrote: > From: Arkadiusz Wlodarczyk > > Configuration file for GONI has been updated to support FAT file system, > new mmc partitioning scheme and read linux kernel from eMMC instead of > OneNAND. > > Signed-off-by: Arkadiusz Wlodarczyk > Signed-off-by: K

Re: [U-Boot] : MACB TX timeout

2013-07-04 Thread Jagan Teki
Hi Bo Shen, Thanks for your info. On Fri, Jul 5, 2013 at 7:04 AM, Bo Shen wrote: > Hi Jagan Teki, > > > On 7/5/2013 03:36, Jagan Teki wrote: >> >> Hi, >> >> Did anyone find this TX timeout issue on macb(gmac)? >> I observed this issue when i enable d-cache. >> >> Log: >> >> zynq-uboot> ping

Re: [U-Boot] : MACB TX timeout

2013-07-04 Thread Bo Shen
Hi Jagan Teki, On 7/5/2013 03:36, Jagan Teki wrote: Hi, Did anyone find this TX timeout issue on macb(gmac)? I observed this issue when i enable d-cache. Log: zynq-uboot> ping 10.10.70.101 gmac0: PHY present at 7 gmac0: link up, 1000Mbps full-duplex (lpa: 0x3800) Using gmac0 device gmac0:

Re: [U-Boot] [U-boot] DFU tools copyright

2013-07-04 Thread Stefan Schmidt
Hello. On Thu, 2013-07-04 at 11:37, Lukasz Majewski wrote: > On Thu, 04 Jul 2013 17:20:34 +0800, tiger...@viatech.com.cn wrote: > > > Hi, experts: > > I found DFU tool code had been put in drivers/dfu directory. > > Previously, it was owned by Samsung Electronics. > > > > Could you be more spec

[U-Boot] [PATCH] zmx25: Select CONFIG_OF_LIBFDT

2013-07-04 Thread Fabio Estevam
From: Fabio Estevam Allow the boot of a device tree kernel. Signed-off-by: Fabio Estevam --- include/configs/zmx25.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/zmx25.h b/include/configs/zmx25.h index e9216d9..871009d 100644 --- a/include/configs/zmx25.h +++ b/include/c

[U-Boot] [PATCH v2 2/5] bootm: Disable interrupts only when loading

2013-07-04 Thread Simon Glass
With the move of the interrupt code to earlier in the sequence, we exposed a problem where the interrupts are disabled at each bootm stage. This is not correct - it should be done only once. Let's disable interrupts in the LOAD stage. Put the code in a function for clarity. Also, bootz lost its in

[U-Boot] [PATCH v2 3/5] bootm: Require boot function only if it is about to be used

2013-07-04 Thread Simon Glass
The original bootm code (before commit 35fc84f) did not check for a valid boot function in the subcommand case, which was incorrect. This check was introduced in all cases, but in fact we should only check for the function when we need it. Otherwise in some cases the check fires before the OS type

[U-Boot] [PATCH v2 5/5] bootm: Add the missing PREP stage to bootz and correct image handling

2013-07-04 Thread Simon Glass
In the recent bootm refactor, the PREP stage was missing in the bootz command. This causes unpredictable behaviour. The use of a local variable means that the reset of cmd_bootm.c does not in fact use the same image structure, so remove this. Also manually set the OS type to Linux, since this is

[U-Boot] [PATCH v2 4/5] bootm: Clean up bootz_setup() function

2013-07-04 Thread Simon Glass
This function has no prototype in the headers and passes void * around, thus requiring several casts. Tidy this up. - Add new patch to clean up bootz_setup() function Signed-off-by: Simon Glass --- Changes in v2: None arch/arm/lib/bootm.c | 14 -- common/cmd_bootm.c | 8 +++-

[U-Boot] [PATCH v2 1/5] bootm: Handle errors consistently

2013-07-04 Thread Simon Glass
A recent bootm fix left the error path incomplete. Reinstate this so that failures in bootm stages are handled properly. Signed-off-by: Simon Glass --- Changes in v2: - Correct checking in the no-error case common/cmd_bootm.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --gi

Re: [U-Boot] [PATCH 3/3] bootm: Add the missing PREP stage to bootz

2013-07-04 Thread Simon Glass
Hi Robert, On Thu, Jul 4, 2013 at 12:06 AM, Robert Nelson wrote: > > diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c > > index 7b3e459..5749057 100644 > > --- a/common/cmd_bootm.c > > +++ b/common/cmd_bootm.c > > @@ -1758,6 +1758,10 @@ static int bootz_start(cmd_tbl_t *cmdtp, int > > flag, i

Re: [U-Boot] Regression: bootz command fails in v2013.07-rc2

2013-07-04 Thread Simon Glass
Hi, On Thu, Jul 4, 2013 at 4:32 PM, Tetsuyuki Kobayashi wrote: > Hello, > > I found bootz command causes 'undefined instruction' exception > on KZM-A9-GT board (armv7) in v2013.07-rc2. > > KZM-A9-GT# tftp zImage.kzm9g.v310; bootz > smc911x: detected LAN9221 controller > smc911x: phy initialized >

[U-Boot] : MACB TX timeout

2013-07-04 Thread Jagan Teki
Hi, Did anyone find this TX timeout issue on macb(gmac)? I observed this issue when i enable d-cache. Log: zynq-uboot> ping 10.10.70.101 gmac0: PHY present at 7 gmac0: link up, 1000Mbps full-duplex (lpa: 0x3800) Using gmac0 device gmac0: TX timeout gmac0: TX timeout ping failed; host 10.10.7

[U-Boot] TLB mapping for pcie mem space for fsl corenet processors

2013-07-04 Thread Sughosh Ganu
hi, The tlb entries for the pcie mem space for the corenet SoC's is done for 1.5GiB but certain boards use all the 4 pcie controller instantiations, and each controller is assigned 512MiB size in the config files. Should the tlb entries not map 2GiB space as against 1.5GiB. Am i missing something.

Re: [U-Boot] [PATCH v3 2/2] cmd_sf: let "sf update" preserve the final part of the last sector

2013-07-04 Thread Jagan Teki
On Thu, Jul 4, 2013 at 12:03 AM, Gerlando Falauto wrote: > Since "sf update" erases the last block as a whole, but only rewrites > the meaningful initial part of it, the rest would be left erased, > potentially erasing meaningful information. > So, as a safety measure, have it rewrite the original

Re: [U-Boot] [PATCH] cmd_bootm.c: Correct check/return for unsupported sub-command

2013-07-04 Thread Tom Rini
On Thu, Jul 04, 2013 at 03:24:10PM +0200, Andreas Bie?mann wrote: > On 07/03/2013 02:37 PM, Tom Rini wrote: > > On Tue, Jul 02, 2013 at 03:05:22PM +0200, Andreas Bie?mann wrote: > >> On 07/01/2013 03:09 PM, Tom Rini wrote: > >>> With the do_bootm_states re-organization, we have the call to any > >>

[U-Boot] dm3730: dpll5 init broken with 19.2MHz?

2013-07-04 Thread Andreas Naumann
Hi, I have a custom OMAP37xx board running with a fairly uncommon 19.2MHz oscillator. The general clock setup seems fine so far, but the dpll5_init_34xx() used in prcm_init() for 36xx cpu-types gives me trouble in the form of numerous stack dumps in the kernel boot. Reason is: [3.396636]

Re: [U-Boot] [PATCH 3/3] Makefile: fix readelf usage

2013-07-04 Thread Albert ARIBAUD
Hi Andreas, On Thu, 04 Jul 2013 15:25:53 +0200, "Andreas Bießmann" wrote: > Hi Albert, > > On 07/04/2013 02:09 PM, Albert ARIBAUD wrote: > > Hi Andreas, > > > > On Sun, 30 Jun 2013 13:15:06 +0200, Andreas Bießmann > > wrote: > > > >> Some OS (like OS X) do not provide a generic readelf. We s

[U-Boot] [PATCH 1/4] powerpc/83xx: remove staticness for qe_iop_conf_tab

2013-07-04 Thread Holger Brunck
commit a5510058 powerpc/83xx/km: make local functions and structs static removed the staticness also from this struct. But this struct is needed in arch/powerpc/cpu/mpc83xx/cpu_init.c and declared as extern. Signed-off-by: Holger Brunck --- board/keymile/km83xx/km83xx.c | 2 +- 1 file changed,

[U-Boot] [PATCH 4/4] km/scripts: fix ramfs

2013-07-04 Thread Holger Brunck
From: Andreas Huber 'actual_bank' is not used anymore, instead boot_bank is used. Signed-off-by: Andreas Huber --- board/keymile/scripts/ramfs-common.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/keymile/scripts/ramfs-common.txt b/board/keymile/scripts/ramfs-co

[U-Boot] [PATCH 3/4] powerpc/km: drop unused CONFIG_SYS_DTT_LOW_TEMP

2013-07-04 Thread Holger Brunck
This define is not used in u-boot code, we can drop this define safely. Signed-off-by: Holger Brunck --- include/configs/km/km83xx-common.h | 1 - include/configs/km82xx.h | 1 - 2 files changed, 2 deletions(-) diff --git a/include/configs/km/km83xx-common.h b/include/configs/km/km83

[U-Boot] [PATCH 2/4] km/common: switch on CMD_GREPENV

2013-07-04 Thread Holger Brunck
Signed-off-by: Holger Brunck --- include/configs/km/keymile-common.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/km/keymile-common.h b/include/configs/km/keymile-common.h index 3b15c4e..cef4860 100644 --- a/include/configs/km/keymile-common.h +++ b/include/configs/km/keym

Re: [U-Boot] [PATCH 3/3] Makefile: fix readelf usage

2013-07-04 Thread Andreas Bießmann
Hi Albert, On 07/04/2013 02:09 PM, Albert ARIBAUD wrote: > Hi Andreas, > > On Sun, 30 Jun 2013 13:15:06 +0200, Andreas Bießmann > wrote: > >> Some OS (like OS X) do not provide a generic readelf. We should enforce to >> use >> the toochain provided readelf instead, to do so use $(CROSS_COMPILE

[U-Boot] [PATCH 2/3] arm/km: drop unneeded define

2013-07-04 Thread Holger Brunck
CONFIG_BOOTCOUNT_LIMIT is used on all boards from this board series. So remove this unneeded define. Signed-off-by: Holger Brunck --- board/keymile/km_arm/km_arm.c | 4 1 file changed, 4 deletions(-) diff --git a/board/keymile/km_arm/km_arm.c b/board/keymile/km_arm/km_arm.c index b944887..

[U-Boot] [PATCH 1/3] arm/km: add support for km_kirkwood_128m16 board

2013-07-04 Thread Holger Brunck
From: Karlheinz Jerg The board is similar to the standard km_kirkwood board. From a u-boot point of view, the only difference is an increased 256 MiB DRAM (128M16). A board based on this design is for example the SUP12. Signed-off-by: Karlheinz Jerg Signed-off-by: Holger Brunck --- boards.cfg

[U-Boot] [PATCH 3/3] arm/km: make local function startup_allowed static

2013-07-04 Thread Holger Brunck
Signed-off-by: Holger Brunck --- board/keymile/km_arm/km_arm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/keymile/km_arm/km_arm.c b/board/keymile/km_arm/km_arm.c index 24fcf70..29f85ff 100644 --- a/board/keymile/km_arm/km_arm.c +++ b/board/keymile/km_arm/km_arm.c @@

Re: [U-Boot] [PATCH] cmd_bootm.c: Correct check/return for unsupported sub-command

2013-07-04 Thread Andreas Bießmann
On 07/03/2013 02:37 PM, Tom Rini wrote: > On Tue, Jul 02, 2013 at 03:05:22PM +0200, Andreas Bie?mann wrote: >> On 07/01/2013 03:09 PM, Tom Rini wrote: >>> With the do_bootm_states re-organization, we have the call to any >>> potential sub-commands in a single spot. If one fails, we can then stop >

Re: [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning

2013-07-04 Thread Timur Tabi
On Fri, Jun 14, 2013 at 3:21 AM, Wolfgang Denk wrote: > diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c > b/board/freescale/mpc8323erdb/mpc8323erdb.c > index f29b2f4..533cb08 100644 > --- a/board/freescale/mpc8323erdb/mpc8323erdb.c > +++ b/board/freescale/mpc8323erdb/mpc8323erdb.c > @@ -1

Re: [U-Boot] Please pull u-boot-ti/master

2013-07-04 Thread Albert ARIBAUD
Hi Tom, On Tue, 2 Jul 2013 16:05:52 -0400, Tom Rini wrote: > Hello, > > The following changes since commit e6c7f86f03b0ad25e9ef70df3ee1989b6b789d7c: > > Merge branch 'u-boot-imx/master' into 'u-boot-arm/master' (2013-06-28 > 17:51:13 +0200) > > are available in the git repository at: > >

Re: [U-Boot] [PATCH 3/3] Makefile: fix readelf usage

2013-07-04 Thread Albert ARIBAUD
Hi Andreas, On Sun, 30 Jun 2013 13:15:06 +0200, Andreas Bießmann wrote: > Some OS (like OS X) do not provide a generic readelf. We should enforce to use > the toochain provided readelf instead, to do so use $(CROSS_COMPILE)readelf. > > Signed-off-by: Andreas Bießmann > --- > Makefile | 2 +- >

Re: [U-Boot] [PATCH v5 1/3] arm: spl: Fix SPL booting for OMAP3

2013-07-04 Thread Albert ARIBAUD
Hi Tom, On Wed, 3 Jul 2013 15:47:27 -0400, Tom Rini wrote: > On Thu, Jun 27, 2013 at 10:27:26AM +0200, Albert ARIBAUD wrote: > > Hi Stefan, > > > > On Tue, 25 Jun 2013 09:14:12 +0200, Stefan Roese wrote: > > > > > Fix a problem with a re-assignment of r8 in the SPL version. > > > > > > This

Re: [U-Boot] [PATCH] usb: fix unaligned access in device_qual()

2013-07-04 Thread Albert ARIBAUD
Hi Marek, On Wed, 3 Jul 2013 15:30:20 +0200, Marek Vasut wrote: > Dear Albert ARIBAUD, > > > Hi Marek, > > > > On Thu, 27 Jun 2013 15:23:33 +0200, Marek Vasut wrote: > > > Hello Albert, > > > > > > > Hi Marek, > > > > > > > > On Thu, 27 Jun 2013 13:26:39 +0200, Marek Vasut wrote: > > > > >

Re: [U-Boot] [PATCH 0/6] Upstream environment changes being used in Yocto/OE

2013-07-04 Thread Albert ARIBAUD
Hi Fabio, On Fri, 28 Jun 2013 18:56:46 -0300, Fabio Estevam wrote: > On Fri, Jun 28, 2013 at 6:52 PM, Otavio Salvador > wrote: > > This changes were being done in every version of U-Boot and it > > makes sense to try to merge them upstream. > > > > Please review them and ack/nack them. > > I t

Re: [U-Boot] [PATCH] mx6: Fix calculation of emi_slow clock rate

2013-07-04 Thread Dirk Behme
On 04.07.2013 13:27, Andrew Gabbasov wrote: This is porting of Freescale's patch from version imx_v2009.08_3.0.35_4.0.0, that fixes the obvious mistype of bits offset macro name (ACLK_EMI_PODF_OFFSET was used instead of ACLK_EMI_SLOW_PODF_OFFSET). Using the occasion, change the variable name 'em

Re: [U-Boot] [PATCH v2 6/7] ARM: extend non-secure switch to also go into HYP mode

2013-07-04 Thread Andre Przywara
On 06/28/2013 05:51 AM, Masahiro Yamada wrote: Hi Andre [RFC] I'd like to suggest to separate HYP-switching code from Non-secure switching. Thanks for stepping up and providing a use-case! The first version of the patches had those two separate cases, but I later merged them in favor of read

[U-Boot] [PATCH] mx6: Fix calculation of emi_slow clock rate

2013-07-04 Thread Andrew Gabbasov
This is porting of Freescale's patch from version imx_v2009.08_3.0.35_4.0.0, that fixes the obvious mistype of bits offset macro name (ACLK_EMI_PODF_OFFSET was used instead of ACLK_EMI_SLOW_PODF_OFFSET). Using the occasion, change the variable name 'emi_slow_pof' to more consistent 'emi_slow_podf'

[U-Boot] Reminder of "floating" Samsung patches

2013-07-04 Thread Lukasz Majewski
Hi Minkyu, Can you evaluate following patches: CONFIG (TRATS): http://patchwork.ozlabs.org/patch/255474/ PMIC (trats): http://patchwork.ozlabs.org/patch/254152/ and following resend patches (for GONI): http://patchwork.ozlabs.org/patch/256872/ http://patchwork.ozlabs.org/patch/256873/ http://p

[U-Boot] [PATCH v5 1/2] powerpc/85xx: Add C29x SoC support

2013-07-04 Thread Po Liu
From: Mingkai Hu The Freescale C29x family is a high performance crypto co-processor. It combines a single e500v2 core with necessary SEC engine. There're three SoC types(C291, C292, C293) with the following features: - 512K L2 Cache/SRAM and 512 KB platform SRAM - DDR3/DDR3L 32bit DDR control

[U-Boot] [PATCH v5 2/2] powerpc/c29xpcie: add support for C29XPCIE board

2013-07-04 Thread Po Liu
From: Mingkai Hu C29XPCIE board is a series of Freescale PCIe add-in cards to perform as public key crypto accelerator or secure key management module. It includes C293PCIE board, C293PCIE board and C291PCIE board. - 512KB platform SRAM in addition to 512K L2 Cache/SRAM - 512MB soldered DDR3 3

[U-Boot] [RESEND PATCH 4/4] arm:goni: Add support for USB mass storage

2013-07-04 Thread Lukasz Majewski
From: Arkadiusz Wlodarczyk This commit enables support for USB mass storage composite function. It defines platform code and enables it at config file. Signed-off-by: Arkadiusz Wlodarczyk Signed-off-by: Kyungmin Park Tested-by: Arkadiusz Wlodarczyk Cc: Minkyu Kang --- board/samsung/goni/gon

[U-Boot] [RESEND PATCH 0/4] arm:goni: Update GONI configuration

2013-07-04 Thread Lukasz Majewski
Samsung's GONI target has been updated to support: - New GPT partition layout - USB Mass Storage Gadget - DFU support - Loading uImage from eMMC not OneNAND Arkadiusz Wlodarczyk (4): arm:goni: Update configuration for goni target arm:goni:dfu Add support for DFU at GONI target arm:goni: Upda

[U-Boot] [RESEND PATCH 3/4] arm:goni: Update of GONI partitioning scheme at eMMC

2013-07-04 Thread Lukasz Majewski
From: Arkadiusz Wlodarczyk New partitioning scheme has been defined at GONI target. It complies with new GPT partitioning introduced previously. Default partition layout has been defined at s5p_goni.h Signed-off-by: Arkadiusz Wlodarczyk Signed-off-by: Kyungmin Park Tested-by: Arkadiusz Wlodar

[U-Boot] [RESEND PATCH 2/4] arm:goni:dfu Add support for DFU at GONI target

2013-07-04 Thread Lukasz Majewski
From: Arkadiusz Wlodarczyk Proper adjustment for supporting DFU at GONI target has been made. The s5p_goni.h file has been updated. Moreover the code for low level USB initialization has been added to GONI board code. Signed-off-by: Arkadiusz Wlodarczyk Signed-off-by: Kyungmin Park Tested-by:

[U-Boot] [RESEND PATCH 1/4] arm:goni: Update configuration for goni target

2013-07-04 Thread Lukasz Majewski
From: Arkadiusz Wlodarczyk Configuration file for GONI has been updated to support FAT file system, new mmc partitioning scheme and read linux kernel from eMMC instead of OneNAND. Signed-off-by: Arkadiusz Wlodarczyk Signed-off-by: Kyungmin Park Tested-by: Arkadiusz Wlodarczyk Cc: Minkyu Kang

Re: [U-Boot] [U-boot] DFU tools copyright

2013-07-04 Thread Lukasz Majewski
On Thu, 04 Jul 2013 17:20:34 +0800, tiger...@viatech.com.cn wrote: > Hi, experts: > I found DFU tool code had been put in drivers/dfu directory. > Previously, it was owned by Samsung Electronics. > Could you be more specific here? All DFU related files have following headers: /* * g_dnl.c --

[U-Boot] [U-boot] DFU tools copyright

2013-07-04 Thread TigerLiu
Hi, experts: I found DFU tool code had been put in drivers/dfu directory. Previously, it was owned by Samsung Electronics. Because it has been merged into uboot library. So, it could be used or released by every ARM SOC vendor? Best wishes, ___ U-Boot m

[U-Boot] [PATCH] cramfs: fix bug for wrong filename comparison

2013-07-04 Thread Holger Brunck
If we have the following entry in cramfs: => cramfsls -rw-r--r-- 1922689 uImage cramfsload would also succeed if we try to do: => cramfsload uImage_1 CRAMFS load complete: 1922689 bytes loaded to 0x10 The old code succeeds if the begin of the filename we search matches with a filename store

Re: [U-Boot] [PATCH v3] drivers:power:max77686: add function to set voltage and mode

2013-07-04 Thread Piotr Wilczek
Dear Minkyu Kang, Please let me know if you have any more comments on this patch. Also, there is another patchset to review: http://patchwork.ozlabs.org/patch/245307/ Best regards, Piotr Wilczek > -Original Message- > From: Piotr Wilczek [mailto:p.wilc...@samsung.com] > Sent: Tuesday, J

Re: [U-Boot] [PATCH v2 4/7] ARM: switch to non-secure state during bootm execution

2013-07-04 Thread Andre Przywara
On 06/28/2013 05:18 AM, Masahiro Yamada wrote: Hi Andre. --- a/arch/arm/lib/Makefile +++ b/arch/arm/lib/Makefile @@ -60,6 +60,8 @@ COBJS-y += reset.o COBJS-y += cache.o COBJS-y += cache-cp15.o +COBJS-$(CONFIG_ARMV7_VIRT) += virt-v7.o + Judging from the file name virt-v

[U-Boot] Regression: bootz command fails in v2013.07-rc2

2013-07-04 Thread Tetsuyuki Kobayashi
Hello, I found bootz command causes 'undefined instruction' exception on KZM-A9-GT board (armv7) in v2013.07-rc2. KZM-A9-GT# tftp zImage.kzm9g.v310; bootz smc911x: detected LAN9221 controller smc911x: phy initialized smc911x: MAC 00:01:9b:04:04:05 Using smc911x-0 device TFTP from server 192.168.1

Re: [U-Boot] [PATCH 2/2] am335x_evm: Update, document Falcon Mode support

2013-07-04 Thread Romain Izard
Tom> +A further word of warning about using eMMC and partition tables. When Tom> +working with SD cards we can get away with erasing small areas at a time, Tom> +however on eMMC we must keep erases aligned to eraseblocks and thus the Tom> +first erase we issue will erase the partition table. Pete

Re: [U-Boot] [PATCH v2 1/7] ARM: prepare armv7.h to be included from assembly source

2013-07-04 Thread Andre Przywara
On 06/28/2013 03:00 AM, Masahiro Yamada wrote: > Hello Andre, Hi, thanks a lot for the review! I included most of the fixes you proposed in the next version I will send out soon. Very useful comments, thanks again! >> diff --git a/arch/arm/include/asm/armv7.h b/arch/arm/include/asm/armv7.h >> in

[U-Boot] [resend][PATCH v3 5/5] ARM: at91: atmel_nand: add code to check the ONFI parameter ECC requirement

2013-07-04 Thread Josh Wu
1. if CONFIG_SYS_NAND_ONFI_DETECTION is defined, driver will check NAND flash's ecc minimum requirement in ONFI parameter. a) if CONFIG_PMECC_CAP, CONFIG_PMECC_SECTOR_SIZE are defined. then use it. Driver will display a WARNING if the values are different from ONFI parameters. b)

Re: [U-Boot] [PATCH v3 5/5] ARM: at91: atmel_nand: add code to check the ONFI parameter ECC requirement

2013-07-04 Thread Josh Wu
On 7/4/2013 6:12 AM, Scott Wood wrote: On 07/02/2013 10:11:49 PM, Josh Wu wrote: +if (!get_onfi_ecc_param(chip, cap, sector_size)) +MTDDEBUG(MTD_DEBUG_LEVEL1, "ONFI params, minimum required ECC: %d bits in %d bytes\n", +*cap, *sector_size); Use braces arou