Re: [U-Boot] [PATCH 1/3 v2] net: emaclite: Setup RX/TX ping pong for every instance

2011-10-10 Thread Michal Simek
Michal Simek wrote: Setup RX/TX ping-pong buffer for every emaclite IP separately. The next patch move initialization directly to board code. Signed-off-by: Michal Simek mon...@monstr.eu --- v2: Fix coding style violations --- drivers/net/xilinx_emaclite.c | 123

[U-Boot] Pull request - microblaze

2011-10-10 Thread Michal Simek
Dear Wolfgang Denk, please pull the following changes to your tree. Thanks, Michal The following changes since commit 0841ca90f22d73b0ea4642ef1ce33d879bb2f3ff: Wolfgang Denk (1): arm920t/s3c24x0/usb_ohci.c: fix warning: variable ... set but not used are available in the git

[U-Boot] microblaze - serial multi

2011-10-10 Thread Michal Simek
Hi, my uartlite serial multi patch is upstream that's why I can enable serial multi for microblaze. There is one issue with serial.h where are spaces at the start of a line. I just follow coding style there. If you require to fix it, I will do it. Thanks, Michal

[U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Michal Simek
Enable support for SERIAL_MULTI. Microblaze can use uart16550 and uartlite drivers. Signed-off-by: Michal Simek mon...@monstr.eu --- Note: Follow serial.h conding style. --- arch/microblaze/lib/board.c |4 include/configs/microblaze-generic.h |2 ++ include/serial.h

Re: [U-Boot] [PATCH 4/5 v2] microblaze: Fix unaligned.h for endians

2011-10-10 Thread Michal Simek
Mike Frysinger wrote: Acked-by: Mike Frysinger vap...@gentoo.org -mike Thanks. Michal -- Michal Simek, Ing. (M.Eng) w: www.monstr.eu p: +42-0-721842854 Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/ Microblaze U-BOOT custodian

Re: [U-Boot] [PATCH 0/2]: OMAP3 SPL updates

2011-10-10 Thread Igor Grinberg
Hi Tom, On 10/10/11 05:01, Tom Rini wrote: On Sun, Oct 9, 2011 at 2:21 AM, Igor Grinberg grinb...@compulab.co.il wrote: On 10/07/11 02:28, Tom Rini wrote: On Tue, Oct 4, 2011 at 7:59 AM, Tom Rini tr...@ti.com wrote: Hey all, The following short series does two things. First, it re-works

Re: [U-Boot] [PATCH 05/13] I2c: add missing i2c_set_bus_num to mxc_i2c

2011-10-10 Thread Heiko Schocher
Hello Stefano, Stefano Babic wrote: The function i2c_set_bus_num() is used on most SOCs. The driver does not support multiple I2C busses, and an empty function is added to be compatible with general code (example: PMIC driver). Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko

Re: [U-Boot] [PATCH 05/13] I2c: add missing i2c_set_bus_num to mxc_i2c

2011-10-10 Thread Stefano Babic
On 10/10/2011 10:50 AM, Heiko Schocher wrote: Hello Stefano, Stefano Babic wrote: The function i2c_set_bus_num() is used on most SOCs. The driver does not support multiple I2C busses, and an empty function is added to be compatible with general code (example: PMIC driver). Signed-off-by:

[U-Boot] [PATCH V2 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Stefano Babic
New default, weak i2c_set_bus_num() function. Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko Schocher h...@denx.de --- Changes since V1: - add a weak function i2c_set_bus_num() to cmd_i2c.c instead of adding a dummy function to mxc_i2c.c. (Heiko Schocher) common/cmd_i2c.c |8

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Graeme Russ
On 10/10/11, Michal Simek mon...@monstr.eu wrote: Enable support for SERIAL_MULTI. Microblaze can use uart16550 and uartlite drivers. Signed-off-by: Michal Simek mon...@monstr.eu --- Note: Follow serial.h conding style. --- arch/microblaze/lib/board.c |4

Re: [U-Boot] [PATCH V2 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Heiko Schocher
Hello Stefano, Stefano Babic wrote: New default, weak i2c_set_bus_num() function. Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko Schocher h...@denx.de --- Changes since V1: - add a weak function i2c_set_bus_num() to cmd_i2c.c instead of adding a dummy function to mxc_i2c.c.

Re: [U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-10 Thread Wolfgang Denk
Dear Vadim Bendebury, In message 20111010025327.119eb40...@eskimo.mtv.corp.google.com you wrote: TPM (Trusted Platform Module) is an integrated circuit and software platform that provides computer manufacturers with the core components of a subsystem used to assure authenticity, integrity and

[U-Boot] [PATCH V3 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Stefano Babic
New default, weak i2c_set_bus_num() function. Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko Schocher h...@denx.de --- Changes since V2: - codestyling: drop newline (Heiko Schocher) Changes since V1: - add a weak function i2c_set_bus_num() to cmd_i2c.c instead of adding a dummy function

Re: [U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-10 Thread Wolfgang Denk
Dear Mike Frysinger, In message 20111018.30473.vap...@gentoo.org you wrote: simplify this with one line: #define tpm_debug(fmt, args...) debug(lpc_tpm: fmt, ## args) This would break as soon as anybody passes a variable for fmt. Best regards, Wolfgang Denk -- DENX Software

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Michal Simek
Graeme Russ wrote: On 10/10/11, Michal Simek mon...@monstr.eu wrote: Enable support for SERIAL_MULTI. Microblaze can use uart16550 and uartlite drivers. Signed-off-by: Michal Simek mon...@monstr.eu --- Note: Follow serial.h conding style. --- arch/microblaze/lib/board.c |

[U-Boot] [PATCH] UBI: init eba tables before wl when attaching a device

2011-10-10 Thread Holger Brunck
This fixes that u-boot gets stuck when a bitflip was detected during ubi part ubi_device. If a bitflip was detected UBI tries to copy the PEB to a different place. This needs that the eba table are initialized, but this was done after the wear levelling worker detects the bitflip. So changes the

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Stefano Babic
On 10/06/2011 01:36 AM, Ilya Yanok wrote: DaVinci EMAC is present on TI AM35xx SoCs (ARMv7) which run with D-Cache enabled by default. So we have to take care and flush/invalidate the cache before/after the DMA operations. Signed-off-by: Ilya Yanok ya...@emcraft.com Hi Ilya, ---

[U-Boot] [PATCH] mpc86xx: implement clocks command

2011-10-10 Thread spundhan . patches
From: Prabhakar Lad prabhakar.cse...@gmail.com print the clock information while booting causes additional delay. Implemented the clocks command for printing the CPU, MPX, DDR and LBC frequency info. Signed-off-by: Prabhakar Lad prabhakar.cse...@gmail.com --- arch/powerpc/cpu/mpc86xx/cpu.c |

[U-Boot] [RESEND PATCH] mpc86xx: implement clocks command

2011-10-10 Thread prabhakar . csengg
From: Prabhakar Lad prabhakar.cse...@gmail.com print the clock information while booting causes additional delay. Implemented the clocks command for printing the CPU, MPX, DDR and LBC frequency info. Signed-off-by: Prabhakar Lad prabhakar.cse...@gmail.com --- arch/powerpc/cpu/mpc86xx/cpu.c |

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Wolfgang Denk
Dear Michal Simek, In message 4e92ce64.5000...@monstr.eu you wrote: defined(CONFIG_MB86R0x) || defined(CONFIG_MPC5xxx) || \ defined(CONFIG_MPC83xx) || defined(CONFIG_MPC85xx) || \ defined(CONFIG_MPC86xx) || defined(CONFIG_SYS_SC520) || \ -defined(CONFIG_TEGRA2) +

[U-Boot] [PATCH v2 1/4] macb: initial support for Cadence GEM

2011-10-10 Thread Dave Aldridge
The Cadence GEM is based on the MACB Ethernet controller but has a few small changes with regards to register and bitfield placement. This patch detects the presence of a GEM by reading the module ID register and setting a flag appropriately. This handles the new HW address, USRIO and hash

[U-Boot] [PATCH v2 4/4] macb: allow GEM to have configurable receive buffer size

2011-10-10 Thread Dave Aldridge
GEM has configurable receive buffer sizes so requires this to be programmed up. Signed-off-by: Dave Aldridge fovs...@gmail.com --- drivers/net/macb.c | 19 +++ drivers/net/macb.h |5 + 2 files changed, 24 insertions(+), 0 deletions(-) diff --git a/drivers/net/macb.c

[U-Boot] [PATCH v2 3/4] macb: support DMA bus widths 32 bits

2011-10-10 Thread Dave Aldridge
Some GEM implementations may support DMA bus widths up to 128 bits. We can get the maximum supported DMA bus width from the design configuration register so use that to program the device up. Signed-off-by: Dave Aldridge fovs...@gmail.com --- drivers/net/macb.c | 25 +

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Ilya Yanok
Hi Stefano, On 10.10.2011 16:35, Stefano Babic wrote: +#ifdef DAVINCI_EMAC_DCACHE +static inline void davinci_flush(void *addr, int size) +{ +flush_dcache_range((unsigned long)addr, +(unsigned long)addr + size); +} There is no check with the cache linesize. I get

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Stefano Babic
On 10/10/2011 03:03 PM, Ilya Yanok wrote: Hi Stefano, Hi Ilya, On 10.10.2011 16:35, Stefano Babic wrote: +#ifdef DAVINCI_EMAC_DCACHE +static inline void davinci_flush(void *addr, int size) +{ + flush_dcache_range((unsigned long)addr, + (unsigned long)addr + size);

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Wolfgang Denk
Dear Stefano Babic, In message 4e92f05f.4030...@denx.de you wrote: There is a CONFIG_SYS_CACHELINE_SIZE. However, I see recent patches that can help in our case ( cache: add ALLOC_CACHE_ALIGN_BUFFER macro): http://patchwork.ozlabs.org/patch/117698/ Wolfgang replied he has already

Re: [U-Boot] [PATCH v2] NAND: davinci: choose correct 1-bit h/w ECC reg

2011-10-10 Thread Laurence Withers
On Mon, Sep 26, 2011 at 04:02:30PM +, Laurence Withers wrote: In nand_davinci_readecc(), select the correct NANDFnECC register based on CONFIG_SYS_NAND_CS rather than hardcoding the choice of NANDF1ECC. This allows 1-bit hardware ECC to work with chip select other than CS2. Note this now

Re: [U-Boot] (no subject)

2011-10-10 Thread Detlev Zundel
Hi victor, how to reset or re program this part item s29gl064n90tf103.. can you please help me Do you really expect us that we understand what your problem us? I for one have got no clue what you want to do and so I seen no way in helping you. If you want help from others, a well posed

Re: [U-Boot] [PATCH V2 1/2] mmc: change magic number to macro define

2011-10-10 Thread Lei Wen
Hi Wolfgang, On Fri, Oct 7, 2011 at 1:39 AM, Wolfgang Denk w...@denx.de wrote: Dear Lei Wen, In message calzhosrhbf2vmu5olp3hwh4yq4xfip19ajd24gn4sy-rm6b...@mail.gmail.com you wrote: The ext_csd current usage in mmc.c is not too much, here I mean only few of the fields of the ext_csd is

Re: [U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 06:50:31 Wolfgang Denk wrote: Mike Frysinger wrote: simplify this with one line: #define tpm_debug(fmt, args...) debug(lpc_tpm: fmt, ## args) This would break as soon as anybody passes a variable for fmt. true, but tpm_debug() is local to this file, and if no

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 08:57:28 Wolfgang Denk wrote: Michal Simek wrote: defined(CONFIG_MB86R0x) || defined(CONFIG_MPC5xxx) || \ defined(CONFIG_MPC83xx) || defined(CONFIG_MPC85xx) || \ defined(CONFIG_MPC86xx) || defined(CONFIG_SYS_SC520) || \ -

Re: [U-Boot] [PATCH V3 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 06:50:12 Stefano Babic wrote: +int __def_i2c_set_bus_num(unsigned int bus) +{ + return 0; +} +int i2c_set_bus_num(unsigned int) + __attribute__((weak, alias(__def_i2c_set_bus_num))); any reason this can't just be: __weak int i2c_set_bus_num(unsigned int

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 09:39:16 Wolfgang Denk wrote: Stefano Babic wrote: There is a CONFIG_SYS_CACHELINE_SIZE. However, I see recent patches that can help in our case ( cache: add ALLOC_CACHE_ALIGN_BUFFER macro): http://patchwork.ozlabs.org/patch/117698/ Wolfgang replied he has

Re: [U-Boot] [PATCH] mx31pdk: Remove unneeded config

2011-10-10 Thread Fabio Estevam
Hi Wolfgang, On Tue, Sep 13, 2011 at 8:10 AM, Stefano Babic sba...@denx.de wrote: On 09/12/2011 06:32 PM, Fabio Estevam wrote: Currently there are two config options for building a U-boot binary for MX31PDK: make mx31pdk_config or, make mx31pdk_nand_config mx31pdk_config was developed

Re: [U-Boot] [PATCH V3 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Stefano Babic
On 10/10/2011 05:19 PM, Mike Frysinger wrote: On Monday 10 October 2011 06:50:12 Stefano Babic wrote: Hi Mike, +int __def_i2c_set_bus_num(unsigned int bus) +{ +return 0; +} +int i2c_set_bus_num(unsigned int) +__attribute__((weak, alias(__def_i2c_set_bus_num))); any reason this

[U-Boot] [PATCH 0/3] USB keyboard driver rework (round 2)

2011-10-10 Thread Marek Vasut
This series reworks the common/usb_kbd.c driver. The code in the driver was messy so this is mostly cleanup and reorganisation. NOTE: I don't consider it a V2 of the previous series because the patch 1/2 from the old series is no longer present and there were also some changes in the dead code

[U-Boot] [PATCH 3/3] USB: Drop dead code from usb_kbd.c

2011-10-10 Thread Marek Vasut
Signed-off-by: Marek Vasut marek.va...@gmail.com Cc: Mike Frysinger vap...@gentoo.org Cc: Remy Bohmer li...@bohmer.net Cc: Wolfgang Denk w...@denx.de --- common/usb_kbd.c | 379 -- 1 files changed, 0 insertions(+), 379 deletions(-) diff --git

[U-Boot] [PATCH 1/3] USB: Squash checkpatch warnings in usb_kbd.c

2011-10-10 Thread Marek Vasut
Signed-off-by: Marek Vasut marek.va...@gmail.com Cc: Mike Frysinger vap...@gentoo.org Cc: Remy Bohmer li...@bohmer.net Cc: Wolfgang Denk w...@denx.de --- common/usb_kbd.c | 608 -- 1 files changed, 319 insertions(+), 289 deletions(-) diff

[U-Boot] [PATCH 2/3] USB: Rework usb_kbd.c

2011-10-10 Thread Marek Vasut
* Support dynamic allocation of devices * Passing data via usb device privptr * Reorder functions to avoid forward declarations * Introduce generic polling mechanism to fix musb and ehci-hcd breakage due to using extern new; to access keyboard driver data! Signed-off-by: Marek Vasut

Re: [U-Boot] [PATCH] mx31pdk: Remove unneeded config

2011-10-10 Thread Stefano Babic
On 10/10/2011 05:28 PM, Fabio Estevam wrote: Hi Wolfgang, On Tue, Sep 13, 2011 at 8:10 AM, Stefano Babic sba...@denx.de wrote: On 09/12/2011 06:32 PM, Fabio Estevam wrote: Currently there are two config options for building a U-boot binary for MX31PDK: make mx31pdk_config or, make

Re: [U-Boot] [PATCH] mx31pdk: Remove unneeded config

2011-10-10 Thread Stefano Babic
On 09/12/2011 06:32 PM, Fabio Estevam wrote: Currently there are two config options for building a U-boot binary for MX31PDK: make mx31pdk_config or, make mx31pdk_nand_config mx31pdk_config was developed first when no NAND SPL support was available for MX31 and it requires that

Re: [U-Boot] [PATCH v3 4/4] tegra2: Add support for Ventana

2011-10-10 Thread Stephen Warren
Simon Glass wrote at Friday, October 07, 2011 6:03 PM: On Thu, Oct 6, 2011 at 10:53 AM, Stephen Warren swar...@nvidia.com wrote: Ventana is a board which is very similar to Seaboard. Support it by re-using board/nvidia/seaboard/seaboard.c with minor run-time conditionals. ... diff --git

Re: [U-Boot] [PATCH V2 1/2] mmc: change magic number to macro define

2011-10-10 Thread Wolfgang Denk
Dear Lei Wen, In message CALZhoSQbvKj0MtqryeHX-4LkvQJR2=b9u_m4yjjfm1mjv2m...@mail.gmail.com you wrote: So macro may looks more concise and could parse from its meaning easily eno= ugh. We do not accept (typeless) register offset definitions. Please use a struct, so the compiler

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Wolfgang Denk
Dear Mike Frysinger, In message 201110101118.05350.vap...@gentoo.org you wrote: defined(CONFIG_MB86R0x) || defined(CONFIG_MPC5xxx) || \ defined(CONFIG_MPC83xx) || defined(CONFIG_MPC85xx) || \ defined(CONFIG_MPC86xx) || defined(CONFIG_SYS_SC520) || \ -

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Wolfgang Denk
Dear Mike Frysinger, In message 201110101124.22548.vap...@gentoo.org you wrote: See the rest of the thread. I had applied this patch set to a loal tree, but it was breaking hundreds of systems, so had to back out the patches again. I'm eager to get this code in myself, but it needs

Re: [U-Boot] [PATCH] mx31pdk: Remove unneeded config

2011-10-10 Thread Wolfgang Denk
Dear Fabio Estevam, In message caomzo5c0pe3f+4h8t7svpsfucjufp_-z27zzqrobynnvy5s...@mail.gmail.com you wrote: Ping? Pong. What's up? This is i.MX, and I haven't seen any ARM (nor even i.MX) pull requests yet. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD:

Re: [U-Boot] [PATCH V3 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 11:31:39 Stefano Babic wrote: On 10/10/2011 05:19 PM, Mike Frysinger wrote: On Monday 10 October 2011 06:50:12 Stefano Babic wrote: +int __def_i2c_set_bus_num(unsigned int bus) +{ + return 0; +} +int i2c_set_bus_num(unsigned int) + __attribute__((weak,

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 13:35:13 Wolfgang Denk wrote: Mike Frysinger wrote: defined(CONFIG_MB86R0x) || defined(CONFIG_MPC5xxx) || \ defined(CONFIG_MPC83xx) || defined(CONFIG_MPC85xx) || \ defined(CONFIG_MPC86xx) || defined(CONFIG_SYS_SC520) || \ -

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 13:44:21 Wolfgang Denk wrote: Mike Frysinger wrote: See the rest of the thread. I had applied this patch set to a loal tree, but it was breaking hundreds of systems, so had to back out the patches again. I'm eager to get this code in myself, but it needs

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Anton Staaf
On Mon, Oct 10, 2011 at 10:44 AM, Wolfgang Denk w...@denx.de wrote: Dear Mike Frysinger, In message 201110101124.22548.vap...@gentoo.org you wrote: See the rest of the thread.  I had applied this patch set to a loal tree, but it was breaking hundreds of systems, so had to back out the

Re: [U-Boot] [PATCH V2 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Tabi Timur-B04825
On Mon, Oct 10, 2011 at 4:35 AM, Stefano Babic sba...@denx.de wrote: New default, weak i2c_set_bus_num() function. Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko Schocher h...@denx.de I would prefer to see a new set of I2C functions that take a bus number as a parameter, so that we can

[U-Boot] [PATCH v6 0/20] New 'sandbox' test architecture for U-Boot

2011-10-10 Thread Simon Glass
This patch set points towards a possible way to improve the test infrastructure in U-Boot. The goal is to have a test suite that can run in a minute or two on a Linux PC and test all non-platform code. This set aims to be just enough boot to U-Boot to a command prompt. You can type help;

Re: [U-Boot] [PATCH 1/3] USB: Squash checkpatch warnings in usb_kbd.c

2011-10-10 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH v6 15/20] sandbox: Add serial uart

2011-10-10 Thread Simon Glass
This uart simply writes to stdout and reads from stdin. We might imagine instead buffering the data so that a test interface can check output and inject input. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v4: - Remove serial_exit() drivers/serial/Makefile |1 +

[U-Boot] [PATCH v6 04/20] sandbox: Add compiler defines to support a 64-bit x86_64 platform

2011-10-10 Thread Simon Glass
This sets __WORDSIZE to 8 correctly on 64-bit machines. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Update commit message to remove 'temporary' - Allow __WORDSIZE to be defined in Makefile / elsewhere include/compiler.h | 12 +++- 1 files changed, 11

[U-Boot] [PATCH v6 14/20] sandbox: Add main program

2011-10-10 Thread Simon Glass
Add a main program so that we can run U-Boot. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v4: - Move main() into arch/sandbox/cpu/start.c arch/sandbox/cpu/Makefile |2 +- arch/sandbox/cpu/start.c | 33 + 2 files changed, 34 insertions(+), 1

[U-Boot] [PATCH v6 09/20] sandbox: Add bootm support

2011-10-10 Thread Simon Glass
This adds sandbox architecture support to bootm, although it is probably not useful to load sandbox code into the address space and execute it. This change at least make the file build correctly on 64-bit machines. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Fix cast of int

[U-Boot] [PATCH v6 19/20] Add generic gpio.h in asm-generic

2011-10-10 Thread Simon Glass
Since we want want to have a standard GPIO interface, this adds a definition for this into include/asm-generic/gpio.h. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v5: - Move the introduction of include/asm-generic/gpio.h into a separate commit - Add comments to GPIO functions

[U-Boot] [PATCH v6 02/20] Fix use of int as pointer in image.c

2011-10-10 Thread Simon Glass
It is better to use %p in this case. Signed-off-by: Simon Glass s...@chromium.org Acked-by: Mike Frysinger vap...@gentoo.org --- Changes in v2: - Split this change out from 'Add architecture image support' common/image.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[U-Boot] [PATCH v6 01/20] sandbox: Add architecture header files

2011-10-10 Thread Simon Glass
This adds required header files for the sandbox architecture, and a basic description of what sandbox is (README.sandbox). Signed-off-by: Simon Glass s...@chromium.org --- Changes in v6: - Remove volatile keyword from arch/sandbox/include/asm/bitops.h arch/sandbox/include/asm/bitops.h |

[U-Boot] [PATCH v6 07/20] sandbox: Add sandbox board

2011-10-10 Thread Simon Glass
This adds basic files for the sandbox board. The lds file is very simple since we can rely mostly on the linker defaults. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Remove clean and dist-clean targets from Makefile - Move lds script out of the board directory Changes in

[U-Boot] [PATCH v6 03/20] sandbox: Add architecture image support

2011-10-10 Thread Simon Glass
We won't actually load an image with this architecture, but we still need to define it. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v3: - Add architecture image support for sandbox Changes in v5: - Define IH_ARCH_DEFAULT which is now required arch/sandbox/include/asm/u-boot.h |

[U-Boot] [PATCH v6 16/20] sandbox: Add basic config file

2011-10-10 Thread Simon Glass
This basic provides required features along with a basic command set. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Remove CONFIG_SYS_SDRAM_BASE which is always 0 for sandbox boards - Fix #definetab - Remove CONFIG_LMB Changes in v4: - Add CONFIG_SYS_SDRAM_SIZE for size of

[U-Boot] [PATCH v6 06/20] sandbox: Add architecture lib files

2011-10-10 Thread Simon Glass
These files are taken from the ARM board implementation and then reduced to remove unneeded cruft. Ideally we would work towards unifying arch/xxx/lib files, particularly board.c. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Fix commit message typo, sadly - Remove ARM cruft

[U-Boot] [PATCH v6 13/20] sandbox: Add OS dependent layer

2011-10-10 Thread Simon Glass
We want to keep all OS-dependent code in once place, with a simple interface to U-Boot. For now, this is that place. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Move os layer into arch/sandbox - Remove clean and dist-clean targets from Makefile - Try and fail to remove the

[U-Boot] [PATCH v6 20/20] sandbox: Makefile changes to build sandbox architecture

2011-10-10 Thread Simon Glass
At this point U-Boot will build and run on x86 under Linux. The idea is to define a new architecture called 'sandbox', alongside ARM and x86. This runs natively on Linux to suit the host machine. All hardware access is either omitted or emulated. The purpose of this system is to test the bulk of

[U-Boot] [PATCH v6 10/20] sandbox: Disable built-in malloc

2011-10-10 Thread Simon Glass
We prefer to U-Boot's malloc but for now it is easier to use the C library's version. Signed-off-by: Simon Glass s...@chromium.org --- common/Makefile |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/common/Makefile b/common/Makefile index 371a0d9..fee6de1 100644 ---

[U-Boot] [PATCH v6 12/20] sandbox: Force command sections to be 4-byte aligned

2011-10-10 Thread Simon Glass
By default sections are 16-byte aligned on some architectures, but the command name structure (struct cmd_tbl_s) does not have padding to 16 bytes. We use the linker to collect elements of an array of these structures and then at run-time we want this array to be accessible. If the linker puts

[U-Boot] [PATCH v6 11/20] sandbox: Disable standalone/API support

2011-10-10 Thread Simon Glass
This is less useful on the sandbox architecture since we can simply link all our code with U-Boot. However some interest has been expressed in implementing this support. For now, it is disabled. Signed-off-by: Simon Glass s...@chromium.org --- Makefile |9 ++--- 1 files changed, 6

[U-Boot] [PATCH v6 05/20] sandbox: Add cpu files

2011-10-10 Thread Simon Glass
This is an initial implementation with all functions defined but not working. The lds file is very simple since we can mostly rely on the linker defaults. Signed-off-by: Simon Glass s...@chromium.org --- Changes in v2: - Remove setting of LDSCRIPT (top level Makefile does this anyway) - Add

[U-Boot] [PATCH v6 18/20] Adjust dependency rules to permit per-file flags

2011-10-10 Thread Simon Glass
The dependency rules are currently done in a shell 'for' loop. This does not permit Makefile variables to adjust preprocessor flags as is done with normal compile flags, using the CFLAGS_path/file.o syntax. This change moves the dependency generation into the Makefile itself, and permits a

[U-Boot] [PATCH v6 17/20] Use uintptr_t for 32/64-bit compatibility

2011-10-10 Thread Simon Glass
This fixes a few problems when building on 64-bit machines. Signed-off-by: Simon Glass s...@chromium.org --- common/cmd_mem.c |2 +- common/fdt_support.c |8 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/common/cmd_mem.c b/common/cmd_mem.c index

[U-Boot] [PATCH v6 08/20] sandbox: Add board info for architecture

2011-10-10 Thread Simon Glass
This is required for the bdinfo command to work. This also cleans up the #ifdef mess for ethernet and lnum a little. Signed-off-by: Simon Glass s...@chromium.org --- common/cmd_bdinfo.c | 34 ++ 1 files changed, 30 insertions(+), 4 deletions(-) diff --git

Re: [U-Boot] [PATCH 2/3] USB: Rework usb_kbd.c

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 11:34:26 Marek Vasut wrote: --- a/common/usb_kbd.c +++ b/common/usb_kbd.c -#undef USB_KBD_DEBUG +#ifdef USB_KBD_DEBUG +#define USB_KBD_PRINTF(fmt, args...) printf(fmt, ##args) +#else +#define USB_KBD_PRINTF(fmt, args...) +#endif could we possibly just get

Re: [U-Boot] [PATCH 3/3] USB: Drop dead code from usb_kbd.c

2011-10-10 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH V2 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 13:53:48 Tabi Timur-B04825 wrote: On Mon, Oct 10, 2011 at 4:35 AM, Stefano Babic sba...@denx.de wrote: New default, weak i2c_set_bus_num() function. Signed-off-by: Stefano Babic sba...@denx.de Cc: Heiko Schocher h...@denx.de I would prefer to see a new set of

Re: [U-Boot] [PATCH] microblaze: Enable SERIAL_MULTI

2011-10-10 Thread Wolfgang Denk
Dear Mike Frysinger, In message 201110101352.15660.vap...@gentoo.org you wrote: it's driver specific code in this file, so it doesn't make much sense to have the board config define something just to get these prototypes It could be a CPU specific header file, then. some of these

[U-Boot] [PATCH v2] Move timestamp and version files into 'generated' subdir

2011-10-10 Thread Simon Glass
There is a rather subtle build problem where the build time stamp is not updated for out-of-tree builds if there exists an in-tree build which has a valid timestamp file. So if you do an in-tree build, then an out-of-tree build your timestamp will not change. The correct timestamp_autogenerated.h

Re: [U-Boot] [PATCH V2 05/13] i2c: Create common default i2c_set_bus_num() function

2011-10-10 Thread Timur Tabi
Mike Frysinger wrote: i think that'd require a much larger rework of the framework and thus would be better to do in addition to Stefano's work rather than in place of ? Hmmm... I guess it would be easier to do the rework eventually if i2c_set_bus_num() is universal, instead of just for

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Wolfgang Denk
Dear Anton Staaf, In message CAF6FioWbLz9JKBnj5sJGALtj=bjylnbfvunna6fe62y42t6...@mail.gmail.com you wrote: When PPC was building again, I tested it on ARM (which I assumed was OK, given that this was Anton's primary architecture). =A0That was when I finally gave up, see Yes, the

Re: [U-Boot] [PATCH 2/3] USB: Rework usb_kbd.c

2011-10-10 Thread Marek Vasut
On Monday, October 10, 2011 08:24:48 PM Mike Frysinger wrote: On Monday 10 October 2011 11:34:26 Marek Vasut wrote: --- a/common/usb_kbd.c +++ b/common/usb_kbd.c -#undef USB_KBD_DEBUG +#ifdef USB_KBD_DEBUG +#define USB_KBD_PRINTF(fmt, args...) printf(fmt, ##args) +#else

Re: [U-Boot] [PATCH 2/3] USB: Rework usb_kbd.c

2011-10-10 Thread Mike Frysinger
On Monday 10 October 2011 14:33:47 Marek Vasut wrote: On Monday, October 10, 2011 08:24:48 PM Mike Frysinger wrote: On Monday 10 October 2011 11:34:26 Marek Vasut wrote: --- a/common/usb_kbd.c +++ b/common/usb_kbd.c -#undef USB_KBD_DEBUG +#ifdef USB_KBD_DEBUG +#define

Re: [U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

2011-10-10 Thread Anton Staaf
On Mon, Oct 10, 2011 at 11:31 AM, Wolfgang Denk w...@denx.de wrote: Dear Anton Staaf, In message CAF6FioWbLz9JKBnj5sJGALtj=bjylnbfvunna6fe62y42t6...@mail.gmail.com you wrote: When PPC was building again, I tested it on ARM (which I assumed was OK, given that this was Anton's primary

[U-Boot] [PATCH v3] Move timestamp and version files into 'generated' subdir

2011-10-10 Thread Simon Glass
There is a rather subtle build problem where the build time stamp is not updated for out-of-tree builds if there exists an in-tree build which has a valid timestamp file. So if you do an in-tree build, then an out-of-tree build your timestamp will not change. The correct timestamp_autogenerated.h

[U-Boot] WARNING: space prohibited between function name and open parenthesis '('

2011-10-10 Thread Stephen Warren
I'm getting a lot of the following when running checkpatch: WARNING: space prohibited between function name and open parenthesis '(' Should I ignore these? So far, I've written completely new code such that it doesn't trigger this warning, but where editing existing code (e.g. just changing the

Re: [U-Boot] [PATCH 0/4] Buffer overruns in printf

2011-10-10 Thread Simon Glass
Hi, On Mon, Sep 26, 2011 at 11:52 PM, Albert ARIBAUD albert.u.b...@aribaud.net wrote: On 27/09/2011 00:28, Scott Wood wrote: - users who actually wisht to limit outpout ca use either You say actually wish to limit output as if let it corrupt memory if it's too large is the normal thing to

Re: [U-Boot] [PATCH] Move timestamp and version files into 'generated' subdir

2011-10-10 Thread Simon Glass
Hi Mike, On Fri, Oct 7, 2011 at 6:05 PM, Mike Frysinger vap...@gentoo.org wrote: i like generated headers all being in the same place however, you forgot to update include/.gitignore :) -mike OK I did this and sent a new patch set to remove those lines, and a third (oops) since someone has

[U-Boot] [PATCH v2 1/3] Add limits.h to hold basic limits

2011-10-10 Thread Simon Glass
This brings a basic limits.h implementation into U-Boot. Signed-off-by: Simon Glass s...@chromium.org --- fs/ubifs/ubifs.h |4 +--- include/limits.h | 40 2 files changed, 41 insertions(+), 3 deletions(-) create mode 100644 include/limits.h diff

[U-Boot] [PATCH v2 3/3] Make printf and vprintf safe from buffer overruns

2011-10-10 Thread Simon Glass
From: Sonny Rao sonny...@chromium.org From: Sonny Rao sonny...@chromium.org utilize the added vscnprintf functions to avoid buffer overruns The implementation is fairly dumb in that it doesn't detect that the buffer is too small, but at least will not cause crashes. Signed-off-by: Sonny Rao

[U-Boot] [PATCH v2 2/3] Add safe vsnprintf and snprintf library functions

2011-10-10 Thread Simon Glass
From: Sonny Rao sonny...@chromium.org From: Sonny Rao sonny...@chromium.org These functions are useful in U-Boot because they allow a graceful failure rather than an unpredictable stack overflow when printf() buffers are exceeded. Mostly copied from the Linux kernel. I copied vscnprintf and

[U-Boot] [PATCH v2 0/3] Buffer overruns in printf

2011-10-10 Thread Simon Glass
The printf family of functions in U-Boot cannot deal with a situation where the caller provides a buffer which turns out to be too small for the format string. This can result in buffer overflows, stack overflows and other bad behavior. This patch series tidies this up in the common vsprintf.c

Re: [U-Boot] [PATCH] mcx: support for HTKW mcx board

2011-10-10 Thread Ilya Yanok
Hi Igor, thanks for your comments, I'll post the updated patches soon. Regards, Ilya. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH] imx: fix coding style

2011-10-10 Thread Fabio Estevam
Fix checkpatch warning and errors in several i.MX related files. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- arch/arm/cpu/arm1136/mx31/timer.c | 22 +++-- arch/arm/cpu/arm1136/mx35/generic.c |2 +- arch/arm/cpu/arm926ejs/mx25/generic.c | 110

Re: [U-Boot] [PATCH v3] Move timestamp and version files into 'generated' subdir

2011-10-10 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 1/5] usb:gadget:s5p USB Device Controller (UDC) implementation

2011-10-10 Thread Scott Wood
On 10/09/2011 02:43 PM, Wolfgang Denk wrote: Dear Lukasz Majewski, In message 20111007093950.17d94287@lmajewski.digital.local you wrote: Most warnings are related to lines over 80 characters. This code is a debug code. As fair as I remember debug print code shall not be break in Linux

Re: [U-Boot] [PATCH] Introduce generic TPM support in u-boot

2011-10-10 Thread Vadim Bendebury
Wolfgang, thank you for your comments, I'll address them in a follow-up submission, but I have a question regarding the register access (and the issue was indeed brought up by vapier@ at an earlier review on a different submission). On Mon, Oct 10, 2011 at 3:45 AM, Wolfgang Denk w...@denx.de

Re: [U-Boot] [PATCH v2 0/6] tegra2: Add more clock/pinmux functionality

2011-10-10 Thread Simon Glass
Hi Albert, On Tue, Sep 27, 2011 at 9:04 AM, Simon Glass s...@chromium.org wrote: Hi Albert, On Tue, Sep 27, 2011 at 12:21 AM, Albert ARIBAUD albert.u.b...@aribaud.net wrote: Hi Simon, On 26/09/2011 19:06, Simon Glass wrote: Hi Albert, On Wed, Sep 21, 2011 at 3:40 PM, Simon

[U-Boot] [PATCH] arm: Tidy up of PRAM init to use getenv_f()

2011-10-10 Thread Simon Glass
It seems that few people use CONFIG_PRAM on ARM. The getenv_r() function has been renamed to getenv_f(). Signed-off-by: Simon Glass s...@chromium.org --- arch/arm/lib/board.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c

Re: [U-Boot] [PATCH v2] NAND: davinci: choose correct 1-bit h/w ECC reg

2011-10-10 Thread Scott Wood
On 09/26/2011 11:02 AM, Laurence Withers wrote: In nand_davinci_readecc(), select the correct NANDFnECC register based on CONFIG_SYS_NAND_CS rather than hardcoding the choice of NANDF1ECC. This allows 1-bit hardware ECC to work with chip select other than CS2. Note this now matches the usage

Re: [U-Boot] WARNING: space prohibited between function name and open parenthesis '('

2011-10-10 Thread Wolfgang Denk
Dear Stephen Warren, In message 74cdbe0f657a3d45afbb94109fb122ff173b23b...@hqmail01.nvidia.com you wrote: I'm getting a lot of the following when running checkpatch: WARNING: space prohibited between function name and open parenthesis '(' Should I ignore these? No, these should be fixed.

  1   2   >