Re: [U-Boot] [PATCH 3/3] cpuat91: fix board support

Le 15/04/2011 15:49, Eric Bénard a écrit : > To: alb...@aribaud.net Please do not use this address of mine for U-Boot related stuff; use albert.u.b...@aribaud.net (Cc:ed). Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://li

Re: [U-Boot] [PATCH v2 3/4] ftsdmc020: move ftsdmc020.h to include/faraday

Hi Macpaul Lin, Le 16/04/2011 08:43, Macpaul Lin a écrit : > Hi Albert, > > 2011/4/16 Albert ARIBAUD: >> Hi MacPaul Lin, >> >>>3 files changed, 104 insertions(+), 104 deletions(-) >>>delete mode 100644 arch/arm/include/asm/arch-a320/ftsdmc020.h >>>create mode 100644 include/faraday/fts

Re: [U-Boot] [PATCH v2 3/4] ftsdmc020: move ftsdmc020.h to include/faraday

Hi Albert, 2011/4/16 Albert ARIBAUD : > Hi MacPaul Lin, > >>   3 files changed, 104 insertions(+), 104 deletions(-) >>   delete mode 100644 arch/arm/include/asm/arch-a320/ftsdmc020.h >>   create mode 100644 include/faraday/ftsdmc020.h > > Please use git format-patch -M or -C to get the move actual

Re: [U-Boot] [PATCH v4 0/4] Add DIG297 board and OMAP3 fixes

Hi all, Le 15/04/2011 14:48, Luca Ceresoli a écrit : > Hi all, > > this patch series cleans up some OMAP3 stuff and adds DIG297 board support. > > New in v4: > - patch 2 gets rid of some ugly extern variables in mem.c; > - patch 3 creates a new file for register definitions (inspired by the px

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Le 16/04/2011 08:18, Macpaul Lin a écrit : > HI all, > > 2011/4/15 Mike Frysinger: up to Wolfgang how he feels about ifdef indentation >>> >>> In this specific case of #ifdef indentation I feel that the original >>> form (which causes checkpatch warnings) is actually easier to read, so >>> I t

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Hi all, Le 15/04/2011 12:09, Wolfgang Denk a écrit : > Dear Mike Frysinger, > > In message<201104150453.21441.vap...@gentoo.org> you wrote: >> On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: > #if !defined(CONFIG_ENV_IS_IN_EEPROM)&& \ > -!defined(CONFIG_ENV_IS_IN_FLASH)&

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

HI all, 2011/4/15 Mike Frysinger : >> > up to Wolfgang how he feels about ifdef indentation >> >> In this specific case of #ifdef indentation I feel that the original >> form (which causes checkpatch warnings) is actually easier to read, so >> I tend to keep it.  But I am aware that this is incons

Re: [U-Boot] [PATCH] mpc83xx fdt: do not adjust clock frequency of external UARTs

Hello Timur Tabi wrote: > Just out of curiosity -- do you have an example of a device tree with > an external UART? Yes. Dave ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 4/4] ftsmc020: move ftsmc020 static mem controller to driver/mtd

Hi MacMaul Lin, Le 22/03/2011 06:01, Macpaul Lin a écrit : > Move the header file and definitions of ftsmc020 > static memory control unit from a320 SoC folder to > "drivers/mtd" folder. > > This change will let other SoC which also use ftsmc020 > could share the same header file. > > Signed-off-b

Re: [U-Boot] [PATCH v2 3/4] ftsdmc020: move ftsdmc020.h to include/faraday

Hi MacPaul Lin, Le 22/03/2011 06:01, Macpaul Lin a écrit : > Move the header file "ftsdmc020.h" (SDRAM Controller) > to "include/faraday" folder. > > This change will let other SoC which also use ftsdmc020 > could share the same header file. > > Signed-off-by: Macpaul Lin > --- > Changes for v2: >

Re: [U-Boot] [PATCH] Replace obsolete e-mail address

Le 13/04/2011 21:42, Albert Aribaud a écrit : > From: Albert (U-Boot) > > > Signed-off-by: Albert (U-Boot) > --- Shamelessly applied to u-boot-arm. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listin

Re: [U-Boot] [PATCH v4 3/3] MX31: mx31pdk: Print the cause of reset

Le 16/04/2011 04:54, Fabio Estevam a écrit : > Print the cause of reset and also change the board name to only 'MX31PDK'. > > Signed-off-by: Fabio Estevam > --- > board/freescale/mx31pdk/mx31pdk.c |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/board/freescale/mx3

Re: [U-Boot] [PATCH v4 1/3] ARM: MX31: Fix file name label

Le 16/04/2011 04:54, Fabio Estevam a écrit : > Commit 5d2c154 (IMX: MX31: Cleanup include files and drop nasty #ifdef in > drivers) > renamed mx31-imx-regs.h to imx-regs.h. > > Change the file label accordingly. > > Signed-off-by: Fabio Estevam > --- > This patch series should were applied in the

Re: [U-Boot] [PATCH v4 1/3] ARM: MX31: Fix file name label

Hi Fabio, Le 16/04/2011 04:54, Fabio Estevam a écrit : > Commit 5d2c154 (IMX: MX31: Cleanup include files and drop nasty #ifdef in > drivers) > renamed mx31-imx-regs.h to imx-regs.h. > > Change the file label accordingly. > > Signed-off-by: Fabio Estevam > --- > This patch series should were appl

[U-Boot] [PATCH v4 3/3] MX31: mx31pdk: Print the cause of reset

Print the cause of reset and also change the board name to only 'MX31PDK'. Signed-off-by: Fabio Estevam --- board/freescale/mx31pdk/mx31pdk.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/board/freescale/mx31pdk/mx31pdk.c b/board/freescale/mx31pdk/mx31pdk.c index 3f

[U-Boot] [PATCH v4 2/3] MX31: Introduce get_reset_cause()

Introduce get_reset_cause() function to indicate the source of the reset. Signed-off-by: Fabio Estevam --- arch/arm/cpu/arm1136/mx31/generic.c| 26 + arch/arm/include/asm/arch-mx31/sys_proto.h | 29 2 files changed, 55 insertions(+

[U-Boot] [PATCH v4 1/3] ARM: MX31: Fix file name label

Commit 5d2c154 (IMX: MX31: Cleanup include files and drop nasty #ifdef in drivers) renamed mx31-imx-regs.h to imx-regs.h. Change the file label accordingly. Signed-off-by: Fabio Estevam --- This patch series should were applied in the u-boot-arm tree. It also needs the following patch to be app

[U-Boot] [PATCH V2 2/2] mmc: enable partition switch fucntion for emmc

For emmc, it may has upto 7 partitions: two boot partitions, one user partition, one RPMB partition and four general purpose partitions. (Refer to JESD84-A44.pdf/page 154) As bootloader may need to read out or reflashing images on those different partitions, it is better to enable the partition sw

[U-Boot] [PATCH V2 1/2] cmd_mmc: eliminate device num in the mmc command

>From now on, follow the general rule "mmc dev [dev]" to change the mmc command applied device, like ide and usb... Signed-off-by: Lei Wen --- Changelog: V2: use the "mmc_cur_dev" to replace explicitly specify the dev num in mmc command. common/cmd_mmc.c | 173 +++-

[U-Boot] [PATCH V2 0/2] mmc: add partition support

V2: use the "mmc_cur_dev" to replace explicitly specify the dev num in mmc command. change the switch parittion command per Wolfgang's suggestion Lei Wen (2): cmd_mmc: eliminate device num in the mmc command mmc: enable partition switch fucntion for emmc common/cmd_mmc.c | 198

[U-Boot] [PATCH] sf: kick watchdog when polling

From: Patrick Sestier The status polling can take a while, so make sure we kick the watchdog after each successful poll. Signed-off-by: Patrick Sestier Signed-off-by: Mike Frysinger --- drivers/mtd/spi/spi_flash.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/driv

[U-Boot] Pull request: nand flash

The following changes since commit 73e5476e1edf1b860dbd9b5fc21ef32ac1b551ba: MAINTAINERS: fix email address case (2011-04-13 22:40:51 +0200) are available in the git repository at: git://git.denx.de/u-boot-nand-flash.git master Alex Waterman (1): nand_spl: Fix large page nand_command()

Re: [U-Boot] [PATCH] powerpc/85xx: Integrated Flash Controller NAND support

On Sat, Apr 09, 2011 at 12:59:04PM -0500, Kumar Gala wrote: > From: Dipen Dudhat > > Add nand_spl and 8-bit NAND support on IFC controller. > > Signed-off-by: Dipen Dudhat > CC: Scott Wood > --- > arch/powerpc/cpu/mpc85xx/cpu_init_nand.c | 10 + > arch/powerpc/cpu/mpc85xx/u-boot-nand_sp

Re: [U-Boot] [PATCH] powerpc/85xx: Modify NAND loader makefiles to compile NAND_SPL linker script

On Sat, Apr 09, 2011 at 12:52:32PM -0500, Kumar Gala wrote: > From: Dipen Dudhat > > Modify eLBC based platform's NAND loader Makefile to preprocess nand loader > linker script and then use it. > > Signed-off-by: Dipen Dudhat > CC: Scott Wood > Signed-off-by: Kumar Gala > --- > nand_spl/boar

Re: [U-Boot] [PATCH 6/6] EA20: do not use subpage write for NAND

On Fri, 15 Apr 2011 19:34:48 +0200 Stefano Babic wrote: > On 04/13/2011 06:24 PM, Scott Wood wrote: > > Hi Scott, > > > Davinci-specific #defines do not belong in nand_base.c[1]. The controller > > driver should be able to set "this isn't supported" options just as well as > > the chip data --

Re: [U-Boot] [PATCH V3 1/2] MX5: factor out boot cause funciton to common code

On 04/15/2011 02:47 PM, Fabio Estevam wrote: >> +char *get_reset_cause(void) >> +{ >> +u32 cause; >> +struct src *src_regs = (struct src >> *)SRC_BASE_ADDR; >> + >> +cause = readl(&src_regs->srsr); > > You need to mask the 7 LSB of SRSR register. > > If you don´t bit 16 can still affe

Re: [U-Boot] [PATCH v3 2/3] MX31: Introduce get_reset_cause()

On 04/15/2011 04:21 AM, Fabio Estevam wrote: > Introduce get_reset_cause() function to indicate the source of the reset. > > Signed-off-by: Fabio Estevam > --- > arch/arm/cpu/arm1136/mx31/generic.c | 26 ++ > arch/arm/include/asm/arch-mx31/imx-regs.h |2 ++ >

Re: [U-Boot] [PATCH v5] ARM: mx31: Print the silicon version

On 04/12/2011 04:18 AM, Fabio Estevam wrote: > Use the same method of the Linux kernel to print the MX31 silicon version on > boot. > > Tested on a MX31PDK with a 2.0 silicon, where it shows: > > CPU: Freescale i.MX31 rev 2.0 at 531 MHz > > Signed-off-by: Fabio Estevam > Applied to u-boot-

Re: [U-Boot] [PATCH] mpc83xx fdt: do not adjust clock frequency of external UARTs

On Fri, Apr 15, 2011 at 7:30 AM, David Müller wrote: > The current 83xx FDT implementation adjusts the clock frequency of all > 16550 UARTs found in the device tree. This behaviour is ok for the UARTs > which are part of the SoC, but wrong for any additional external UART. Just out of curiosity -

[U-Boot] [PATCH V4 2/2] MX53: support for freescale MX53LOCO board

From: Liu Hui-R64343 This patch add initial support for freescale MX53LOCO board. Network(FEC),SD/MMC, UART have been supported by this patch. Signed-off-by: Jason Liu --- changes since v3: - include other two small patch into this commit, mx53loco: set mmc env to MMC slot1 MX5: Enable flat-dev

[U-Boot] [PATCH V4 1/2] MX5: factor out boot cause funciton to common code

From: Liu Hui-R64343 factor out boot cause funciton to common code to avoid the duplicate code in each board support package Signed-off-by: Jason Liu --- changes since v3: - add full boot reset cause --- arch/arm/cpu/armv7/mx5/soc.c | 27 +++ arch/arm/inc

Re: [U-Boot] [PATCH 6/6] EA20: do not use subpage write for NAND

On 04/13/2011 06:24 PM, Scott Wood wrote: Hi Scott, > Davinci-specific #defines do not belong in nand_base.c[1]. The controller > driver should be able to set "this isn't supported" options just as well as > the chip data -- I just don't think it should be limited to this specific > one. surely

Re: [U-Boot] [PATCH 1/1] MX5:MX53: support for freescale MX53LOCO board

Hi, Stefano, 2011/4/16 Stefano Babic : > On 04/14/2011 10:12 AM, Jason Liu wrote: >> Hi, Stefano, > > Hi Jason, > >> Please drop the mx53loco board support patch from u-boot-imx master branch >> and I will resubmit the following patches soon since the merge window will be >> closed soon. > > Ok -

Re: [U-Boot] [PATCH 1/1] MX5:MX53: support for freescale MX53LOCO board

On 04/14/2011 10:12 AM, Jason Liu wrote: > Hi, Stefano, Hi Jason, > Please drop the mx53loco board support patch from u-boot-imx master branch > and I will resubmit the following patches soon since the merge window will be > closed soon. Ok - I'll do it. > > (1) clean-up patch:factor out the b

Re: [U-Boot] [PATCH V1 1/1] MX5: Keep L2 cache enabled before jump to kernel

Hi, Stefano, 2011/4/15 Stefano Babic : > On 04/13/2011 03:25 PM, Jason Liu wrote: > > Hi Jason, > >> Currently, Linux kernel does not do any L2 cache enable >> Operation.So,Keep L2 cache enabled(L2EN=1) in the u-boot >> before Jump to the Linux Kernel and thus L2 cache can be >> effectively used i

Re: [U-Boot] [PATCH] MIPS: Introduce --gc-sections for MIPS

On 04/16/2011 12:16 AM, daniel.schwierz...@googlemail.com wrote: > All architectures but MIPS are using --gc-sections on final linking. > This patch introduces that feature for MIPS to reduce the memory and > flash footprint. > > Signed-off-by: Daniel Schwierzeck > Cc: Shinya Kuribayashi > Cc: W

[U-Boot] [PATCH] MIPS: Introduce --gc-sections for MIPS

All architectures but MIPS are using --gc-sections on final linking. This patch introduces that feature for MIPS to reduce the memory and flash footprint. Signed-off-by: Daniel Schwierzeck Cc: Shinya Kuribayashi Cc: Wolfgang Denk Cc: Stefan Roese Cc: Thomas Lange Cc: Vlad Lungu --- Tested wi

Re: [U-Boot] [PATCH V1 1/1] MX5: Keep L2 cache enabled before jump to kernel

On 04/13/2011 03:25 PM, Jason Liu wrote: Hi Jason, > Currently, Linux kernel does not do any L2 cache enable > Operation.So,Keep L2 cache enabled(L2EN=1) in the u-boot > before Jump to the Linux Kernel and thus L2 cache can be > effectively used in Linux Kernel. If the cache is not active in the

Re: [U-Boot] is any bug in cp?

This makes sense as one should *not* copy from one sector in a flash to the same sector in a flash. Flash is erased in sectors, usually 128K and should be treated as a complete sector. The cp command is usually used to copy from RAM to FLASH or from FLASH to RAM. What you are doing seems illogical

Re: [U-Boot] [PATCH 3/3] cpuat91: fix board support

Hi, On 03/04/2011 18:35, Eric Bénard wrote: > - fix board support following relocation changes > - switch to boards.cfg > - disable i2c to keep size under 128kiB (1 sector) > > Signed-off-by: Eric Bénard > --- any news concerning this patch ? http://patchwork.ozlabs.org/patch/89536/ Thanks, Eric

Re: [U-Boot] [PATCH 2/3] cpu9260: update board support

Hi, On 03/04/2011 18:35, Eric Bénard wrote: > - update to new relocation code > - switch to boards.cfg > - get rid of LEGACY (still a little hack in .h to compile) > - add nand boot configuration > - boot tested for the following configurations : > 9260 (64MB RAM& nor boot) > 9260_nan

[U-Boot] [PATCH 1/1] at91: reworked support for otc570 board

After relocation rework, the meesc and otc570 board support was broken. This patch will fix the otc570 board. Signed-off-by: Daniel Gorsulowski --- This patch is based on u-boot-atmel/rework101229 branch (minus the last 5 patches) plus the 'at91: fixed at91sam9263 system file' patch in u-boot-at

[U-Boot] [PATCH v4 2/4] ARMV7: OMAP3: Cleanup extern variables in mem.c

Removed boot_flash_* extern variables. boot_flash_type was totally unused. The other ones were actually constants, so they have been replaced with #defines in the board config files. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc: Sandeep Paulraj --- Changes in v4: - thi

[U-Boot] [PATCH v4 3/4] ARMV7: OMAP3: Add GPMC_CONFIGx register value definitions

Signed-off-by: Luca Ceresoli --- Changes in v4: - this patch is new in v4. arch/arm/include/asm/arch-omap3/omap3-regs.h | 95 ++ 1 files changed, 95 insertions(+), 0 deletions(-) create mode 100644 arch/arm/include/asm/arch-omap3/omap3-regs.h diff --git a/arch/arm/in

[U-Boot] [PATCH v4 4/4] ARMV7: OMAP3: Add support for Comelit DIG297 board

Board support for the DIG297 board manufactured by Comelit Group SpA. It is a custom board based on the BeagleBoard by Texas Instruments. The board support is based on the BeagleBoard implementation. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc

[U-Boot] [PATCH 4/4] ARMV7: OMAP3: Add support for Comelit DIG297 board

Board support for the DIG297 board manufactured by Comelit Group SpA. It is a custom board based on the BeagleBoard by Texas Instruments. The board support is based on the BeagleBoard implementation. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc

[U-Boot] [PATCH v4 0/4] Add DIG297 board and OMAP3 fixes

Hi all, this patch series cleans up some OMAP3 stuff and adds DIG297 board support. New in v4: - patch 2 gets rid of some ugly extern variables in mem.c; - patch 3 creates a new file for register definitions (inspired by the pxa and imx code base), and defines GPMC_CONFIGx register values.

[U-Boot] [PATCH v4 1/4] ARMV7: OMAP3: Fix preprocessor check for CONFIG_OMAP34XX

CONFIG_OMAP34XX must be checked for existence, not value. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc: Sandeep Paulraj --- Changes in v2: - this patch is new in v2. Changes in v3: none. Changes in v4: none. arch/arm/cpu/armv7/start.S |2 +- 1 files changed, 1

[U-Boot] [PATCH 3/4] ARMV7: OMAP3: Add GPMC_CONFIGx register value definitions

Signed-off-by: Luca Ceresoli --- Changes in v4: - this patch is new in v4. arch/arm/include/asm/arch-omap3/omap3-regs.h | 95 ++ 1 files changed, 95 insertions(+), 0 deletions(-) create mode 100644 arch/arm/include/asm/arch-omap3/omap3-regs.h diff --git a/arch/arm/in

[U-Boot] [PATCH 2/4] ARMV7: OMAP3: Cleanup extern variables in mem.c

Removed boot_flash_* extern variables. boot_flash_type was totally unused. The other ones were actually constants, so they have been replaced with #defines in the board config files. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc: Sandeep Paulraj --- Changes in v4: - thi

[U-Boot] [PATCH 1/4] ARMV7: OMAP3: Fix preprocessor check for CONFIG_OMAP34XX

CONFIG_OMAP34XX must be checked for existence, not value. Signed-off-by: Luca Ceresoli Cc: Wolfgang Denk Cc: Albert Aribaud Cc: Sandeep Paulraj --- Changes in v2: - this patch is new in v2. Changes in v3: none. Changes in v4: none. arch/arm/cpu/armv7/start.S |2 +- 1 files changed, 1

[U-Boot] [PATCH 0/4] Add DIG297 board and OMAP3 fixes

Hi all, this patch series cleans up some OMAP3 stuff and adds DIG297 board support. New in v4: - patch 2 gets rid of some ugly extern variables in mem.c; - patch 3 creates a new file for register definitions (inspired by the pxa and imx code base), and defines GPMC_CONFIGx register values.

Re: [U-Boot] [PATCH] xilinx_emaclite.c ping-pong macro names

Please find attached the checked patch. Sorry for the inconvenience. Kind regards, Alain Quoting Wolfgang Denk : Dear alain.pet...@space.unibe.ch, In message <20110415124815.1253591nkjx5a...@mail.unibe.ch> you wrote: The macro name configuring Ping/Pong didn't match. It has been checked on

Re: [U-Boot] [PATCH V3 1/2] MX5: factor out boot cause funciton to common code

Hi Jason, --- On Fri, 4/15/11, Jason Liu wrote: ... > +char *get_reset_cause(void) > +{ > +    u32 cause; > +    struct src *src_regs = (struct src > *)SRC_BASE_ADDR; > + > +    cause = readl(&src_regs->srsr); You need to mask the 7 LSB of SRSR register. If you don´t bit 16 can still affect its

[U-Boot] [PATCH] mpc83xx fdt: do not adjust clock frequency of external UARTs

The current 83xx FDT implementation adjusts the clock frequency of all 16550 UARTs found in the device tree. This behaviour is ok for the UARTs which are part of the SoC, but wrong for any additional external UART. Signed-off-by: David Müller --- arch/powerpc/cpu/mpc83xx/fdt.c | 27 +

Re: [U-Boot] [PATCH] xilinx_emaclite.c ping-pong macro names

Dear alain.pet...@space.unibe.ch, In message <20110415124815.1253591nkjx5a...@mail.unibe.ch> you wrote: > > The macro name configuring Ping/Pong didn't match. It has been > checked on a Spartan3e Starterkit. This should probably be part of the commit message. Your patch has a number of codi

[U-Boot] [PATCH] xilinx_emaclite.c ping-pong macro names

Dear all, The macro name configuring Ping/Pong didn't match. It has been checked on a Spartan3e Starterkit. Kind regards, Alain >From 35b00080dd397dfa4d8dae2d84423c36a34fe061 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alain=20P=C3=A9teut?= Date: Fri, 15 Apr 2011 12:44:37 +0200 Subject: [PATC

Re: [U-Boot] is any bug in cp?

Dear "=?utf-8?B?aGFja2x1LnVib290?=", In message <201104151715238470...@gmail.com> you wrote: > em.. > how linux does that? What exactly are you doing in Linux? Are we talking about user-space access through a file sytem (and thus block devices) based on the MTD layer? Or about raw access to the

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

On Friday, April 15, 2011 06:09:12 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > > On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: > > > >> #if !defined(CONFIG_ENV_IS_IN_EEPROM)&& \ > > > >> > > > >> -!defined(CONFIG_ENV_IS_IN_FLASH) && \ > > > >> + !defined(CONFIG_ENV_

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Dear Mike Frysinger, In message <201104150453.21441.vap...@gentoo.org> you wrote: > > > > On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: > > >> #if !defined(CONFIG_ENV_IS_IN_EEPROM)&& \ > > >> -!defined(CONFIG_ENV_IS_IN_FLASH) && \ > > >> + !defined(CONFIG_ENV_IS_IN_FLASH)

Re: [U-Boot] [PATCH v2 2/5] Add Ethernet hardware MAC address framework to usbnet

Le 15/04/2011 10:25, Mike Frysinger a écrit : > On Friday, April 15, 2011 04:13:11 Albert ARIBAUD wrote: >> What I disagree upon is that just because they're tools we should disregard >> their warnings > > i never said any such thing > -mike My bad. What I disagree upon is that just because they'r

Re: [U-Boot] is any bug in cp?

em.. how linux does that? does linux read the data to mem first before it write to the same bank of a nor flash? thanks much~ -- hacklu.uboot 2011-04-15 - 发件人：Wolfgang Denk 发送

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Hi Mike, 2011/4/15 Mike Frysinger : >> Do you mean the replacement with space to ident? >> >> According to the 2.6.38.1's checkpatch, it reported: >> WARNING: please, no spaces at the start of a line > > this particular check has already been posted upstream as generally causing > more harm than g

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

On Friday, April 15, 2011 04:25:58 Macpaul Lin wrote: > 2011/4/15 Mike Frysinger: > > On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: > >> #if !defined(CONFIG_ENV_IS_IN_EEPROM)&& \ > >> -!defined(CONFIG_ENV_IS_IN_FLASH) && \ > >> + !defined(CONFIG_ENV_IS_IN_FLASH)&& \

Re: [U-Boot] is any bug in cp?

Dear "hacklu.uboot", In message <201104151401034442...@gmail.com> you wrote: > > there is some bugs when I cp to flash like this: > cp.b addr1 addr2 0x100 > > when the two addr in the same bank of a flash. it go wrongs > > finally the date of addr2 is 0x8080808080... > > cp to another bank of

Re: [U-Boot] [PATCH] ARM: fix stack pointer adjustment in board_init_f()

Dear Eric Cooper, In message <1302820357-30318-1-git-send-email-...@cmu.edu> you wrote: > Since addr_sp is a byte address, it should be adjusted by 12 here. > > Signed-off-by: Eric Cooper > Cc: Albert ARIBAUD > --- > arch/arm/lib/board.c |2 +- > 1 files changed, 1 insertions(+), 1 deletio

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Hi Mike, 2011/4/15 Mike Frysinger : > On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: >>  #if !defined(CONFIG_ENV_IS_IN_EEPROM)        && \ >> -    !defined(CONFIG_ENV_IS_IN_FLASH) && \ >> +     !defined(CONFIG_ENV_IS_IN_FLASH)        && \ > > not sure this one hunk is desirable, but the res

Re: [U-Boot] [PATCH v2 2/5] Add Ethernet hardware MAC address framework to usbnet

On Friday, April 15, 2011 04:13:11 Albert ARIBAUD wrote: > What I disagree upon is that just because they're tools we should disregard > their warnings i never said any such thing -mike signature.asc Description: This is a digitally signed message part. __

Re: [U-Boot] [PATCH v2 2/5] Add Ethernet hardware MAC address framework to usbnet

Le 15/04/2011 09:56, Mike Frysinger a écrit : > On Thursday, April 14, 2011 02:51:42 Albert ARIBAUD wrote: >> Le 14/04/2011 08:12, Mike Frysinger a écrit : >>> so i stand by my statement that checkpatch is a tool and does *not* get >>> the final say. blindly following a tool is good -- if you're b

Re: [U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

On Friday, April 15, 2011 03:02:58 Macpaul Lin wrote: > #if !defined(CONFIG_ENV_IS_IN_EEPROM)&& \ > -!defined(CONFIG_ENV_IS_IN_FLASH) && \ > + !defined(CONFIG_ENV_IS_IN_FLASH)&& \ not sure this one hunk is desirable, but the rest are fine -mike signature.asc Description:

Re: [U-Boot] [PATCH v2] lib/hashtable: fix env var autocompletion

On Tuesday, April 05, 2011 13:14:50 Kim Phillips wrote: > commit c81c1222427f268d29ba999c82e2477c428e7bab "Fix hash > table deletion to prevent lost entries" broke environment > variable autocompletion by accidentally inverting the > condition for 'used' in hmatch_r(). > > 'used' is 0 if the hash

Re: [U-Boot] [PATCH v2 2/5] Add Ethernet hardware MAC address framework to usbnet

On Thursday, April 14, 2011 02:51:42 Albert ARIBAUD wrote: > Le 14/04/2011 08:12, Mike Frysinger a écrit : > > so i stand by my statement that checkpatch is a tool and does *not* get > > the final say. blindly following a tool is good -- if you're blind. > > When it emits warnings, the C toolchai

Re: [U-Boot] [PATCH 6/9] cmd_sf: add handler for +len arg for erase command

On Thursday, April 14, 2011 16:52:36 Richard Retanubun wrote: > On 04/12/11 02:35, Mike Frysinger wrote: > > This patch adds [+]len handler for the erase command that will > > automatically round up the requested erase length to the flash's > > sector_size. > > --- > > Richard: I noticed some issue

[U-Boot] [PATCH V3 2/2] MX53: support for freescale MX53LOCO board

From: Liu Hui-R64343 This patch add initial support for freescale MX53LOCO board. Network(FEC),SD/MMC, UART have been supported by this patch. Signed-off-by: Jason Liu --- changes since V2: -factor out the boot cause function to common code, -fix gd->ram_size with full memory size -remove the '

[U-Boot] [PATCH V3 1/2] MX5: factor out boot cause funciton to common code

From: Liu Hui-R64343 factor out boot cause funciton to common code to avoid the duplicate code in each board support package Signed-off-by: Jason Liu --- arch/arm/cpu/armv7/mx5/soc.c | 18 ++ arch/arm/include/asm/arch-mx5/sys_proto.h |1 + 2 files changed, 19

[U-Boot] [PATCH] cmd_nvedit.c: clean up with checkpatch

Code clean up of cmd_nvedit.c by using checkpatch.pl. Signed-off-by: Macpaul Lin --- Note: This patch has fixed most of the ERRORs and WARNINGs in cmd_nvedit.c. However, the remains were not clean up for avoiding broken the consistence. common/cmd_nvedit.c | 146 +--