[U-Boot] [PATCH] MTD:NAND Enable nand lock, unlock feature

2008-12-13 Thread Nishanth Menon
Enable nand lock, unlock and status of lock feature. Not every device and platform requires this, hence, it is under define for CONFIG_CMD_NAND_LOCK_UNLOCK Nand unlock and status operate on block boundary instead of page boundary. Details in: http://www.micron.com/products/partdetail?part=MT29C2G2

[U-Boot] Warning in ffs2_1pass.c

2008-12-13 Thread Dirk Behme
Just fyi: Using git head (89d56f5503eed351efe5ab0b4dd0f1e888fd2336: Merge branch 'master' of git://git.denx.de/u-boot-ppc4xx) I get warning: -- cut -- ffs2_1pass.c:1411:1: warning: "min_t" redefined In file include/nand.h:30, from jffs2_1pass.c:151: include/linux/mtd/compat.h:3

Re: [U-Boot] [PATCH] cmd_sf: rename "speed" to "hz"

2008-12-13 Thread Jerry Van Baren
Mike Frysinger wrote: > On Thursday 11 December 2008 18:59:46 Wolfgang Denk wrote: >> In message Mike Frysinger you wrote: >>> The term "hz" is used everywhere else when talking about the frequency of >>> the SPI bus, so have the sf command use it as well to stay consistent. >>> It even presents i

Re: [U-Boot] reducing u-boot.bin size

2008-12-13 Thread Wolfgang Denk
Dear Fahd Abidi, In message <71dea556544d7a4f9c6848402d818434036...@usi01.ultsol.local> you wrote: > > I am trying to reduce the size of the u-boot.bin file so that it fits > into a 256KB promjet. I disabled all the drivers that I didn't need and > have just kept the NOR driver but the binary si

[U-Boot] [PATCH] usb: fix wait_ms declaration

2008-12-13 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD diff --git a/common/usb.c b/common/usb.c index ee18152..3ed2e34 100644 --- a/common/usb.c +++ b/common/usb.c @@ -83,16 +83,6 @@ void usb_hub_reset(void); static int hub_port_reset(struct usb_device *dev, int port, unsigned

[U-Boot] [PATCH] integratorap: fix PCI support

2008-12-13 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD diff --git a/include/configs/integratorap.h b/include/configs/integratorap.h index 6ce3b4d..d7f617f 100644 --- a/include/configs/integratorap.h +++ b/include/configs/integratorap.h @@ -143,7 +143,7 @@ * PCI definitions */ -/*#define CONFIG_PCI

[U-Boot] [PATCH] move ARM Ltd. to verdor dir

2008-12-13 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD --- Makefile |6 +++--- board/armltd/.gitignore|2 ++ board/{ => armltd}/integratorap/Makefile |0 board/{ => armltd}/integratorap/config.mk |0

Re: [U-Boot] [PATCH] usb: fix wait_ms declaration

2008-12-13 Thread Wolfgang Denk
Dear Jean-Christophe PLAGNIOL-VILLARD, In message <1229198787-24017-1-git-send-email-plagn...@jcrosoft.com> you wrote: ... > +static void __inline__ wait_ms(unsigned long ms) > +{ > + while(ms-->0) White space before and after the '>', please. And should we not check for a '0' argument first

Re: [U-Boot] [PATCH] move ARM Ltd. to verdor dir

2008-12-13 Thread Wolfgang Denk
Dear Jean-Christophe PLAGNIOL-VILLARD, In message <1229199118-24844-1-git-send-email-plagn...@jcrosoft.com> you wrote: > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD Please fix the subject (s/verdor/vendor/) before checking in this patch. Best regards, Wolfgang Denk -- DENX Software Eng

Re: [U-Boot] [PATCH] usb: fix wait_ms declaration

2008-12-13 Thread Magnus Lilja
2008/12/13 Wolfgang Denk : > Dear Jean-Christophe PLAGNIOL-VILLARD, > > In message <1229198787-24017-1-git-send-email-plagn...@jcrosoft.com> you > wrote: > ... >> +static void __inline__ wait_ms(unsigned long ms) >> +{ >> + while(ms-->0) > > White space before and after the '>', please. And w

Re: [U-Boot] FPGA programming driver architecture

2008-12-13 Thread Wolfgang Denk
Dear Hugo Villeneuve, In message <20081212205916.5404fe43.h...@hugovil.com> you wrote: > > You probably have missed the email I sent 30 seconds after having sent > the first one stating it was the wrong list. Please read your emails > carefully next time. I can read them only as fast as they get

Re: [U-Boot] [RFC USB PATCH V2] USB ehci fix and test on ixp4xx hardware

2008-12-13 Thread Remy Bohmer
Hello Michael, 2008/12/12 Michael Trimarchi : > EHCI fix code and ixp4xx test. > USB ehci configuration parameter: > > #define CONFIG_CMD_USB 1 > #define CONFIG_USB_STORAGE 1 > #define CONFIG_USB_EHCI > #define CONFIG_USB_EHCI_IXP4XX 1 > #define CONFIG_EHCI_IS_TDI 1 > #define C

[U-Boot] fat.c:707: warning: array subscript is above array bounds

2008-12-13 Thread Wolfgang Denk
Hello, building U-Boot with recent toolchains (like GCC-4.3.2) results in this warning: fat.c: In function 'read_bootsectandvi': fat.c:707: warning: array subscript is above array bounds The respective code looks like this: fs/fat/fat.c: 705 /* Terminate fs_type string. Writing past t

Re: [U-Boot] fat.c:707: warning: array subscript is above array bounds

2008-12-13 Thread Jean-Christophe PLAGNIOL-VILLARD
On 23:13 Sat 13 Dec , Wolfgang Denk wrote: > Hello, > > building U-Boot with recent toolchains (like GCC-4.3.2) results in > this warning: > > fat.c: In function 'read_bootsectandvi': > fat.c:707: warning: array subscript is above array bounds > > The respective code looks like this: > > fs

Re: [U-Boot] [PATCH] Remove compiler warning: target CPU does not support interworking

2008-12-13 Thread Wolfgang Denk
Dear Sergei, In message <200811011312.ma1dc7bt008...@wooster.emcraft.com> you wrote: > From 7fd5e5aaf0c61f47c89bf59f2b37cc987e97849a Mon Sep 17 00:00:00 2001 > From: Sergei Poselenov > Date: Fri, 19 Sep 2008 12:07:34 +0200 > Subject: [PATCH] Remove compiler warning: target CPU does not support >

Re: [U-Boot] [PATCH V3] Fix FIT and fdt blob support to have CONFIG_OF_LIBFDT and CONFIG_FIT independant

2008-12-13 Thread Wolfgang Denk
Dear Jean-Christophe PLAGNIOL-VILLARD, In message <1227530031-15268-1-git-send-email-plagn...@jcrosoft.com> you wrote: > FDT support is used for both FIT style images and architectures (ppc, m68k, > sparc) > that can pass a fdt blob to an OS.. > > For other arch and board which do not pass a fdt

Re: [U-Boot] fat.c:707: warning: array subscript is above array bounds

2008-12-13 Thread Wolfgang Denk
Dear Jean-Christophe PLAGNIOL-VILLARD, In message <2008121325.gn15...@game.jcrosoft.org> you wrote: > > > 705 /* Terminate fs_type string. Writing past the end of vistart > > 706is ok - it's just the buffer. */ > > 707 vistart->fs_type[8] = '\0'; > > why not do

Re: [U-Boot] [RFC][PATCH] Code Clean-up (weak functions)

2008-12-13 Thread Remy Bohmer
Hello Graeme, 2008/12/13 Graeme Russ : > This patch makes all definitions, declarations and usages of weak functions > consistent. > > Signed-off-by: Graeme Russ Just curious: What is the relation of this patch to the problem discussed earlier: http://www.mail-archive.com/u-boot@lists.denx.de/ms

Re: [U-Boot] fat.c:707: warning: array subscript is above array bounds

2008-12-13 Thread David Hawkins
Wolfgang Denk wrote: > Dear Jean-Christophe PLAGNIOL-VILLARD, > > In message <2008121325.gn15...@game.jcrosoft.org> you wrote: >>> 705 /* Terminate fs_type string. Writing past the end of vistart >>> 706is ok - it's just the buffer. */ >>> 707 vistart->fs_type[8]

[U-Boot] [PATCH v4 2/4] Introduce addr_map library

2008-12-13 Thread Kumar Gala
Add a library that helps in translating between virtual and physical addresses. This library can be useful as a simple means to implement map_physmem() and virt_to_phys() for platforms that need functionality beyond the simple 1:1 mapping. Signed-off-by: Kumar Gala --- * Folded in Becky's phys_s

[U-Boot] [PATCH v4 1/4] Introduce virt_to_phys()

2008-12-13 Thread Kumar Gala
virt_to_phys() returns the physical address given a virtual. In most cases this will just the input value as the vast majority of systems run in a 1:1 mode. However in systems that are not running this way it should report the physical address or ~0 if no mapping exists for the given virtual addr

[U-Boot] [PATCH v4 4/4] ppc: Use addrmap in virt_to_phys and map_physmem.

2008-12-13 Thread Kumar Gala
If we have addr map support enabled use the mapping functions to implement virt_to_phys() and map_physmem(). Signed-off-by: Kumar Gala --- * No change, just reposting as part of the sequence include/asm-ppc/io.h | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git

[U-Boot] [PATCH v4 3/4] 85xx: Add support to populate addr map based on TLB settings

2008-12-13 Thread Kumar Gala
Signed-off-by: Kumar Gala --- * Folded in Becky's phys_size_t changes cpu/mpc85xx/tlb.c | 34 ++ include/asm-ppc/mmu.h |3 +++ lib_ppc/board.c |8 3 files changed, 45 insertions(+), 0 deletions(-) diff --git a/cpu/mpc85xx/tlb.c b/cpu/

Re: [U-Boot] [PATCH 05/11] lib_generic/addr_map: Change size to phys_size_t from ulong

2008-12-13 Thread Kumar Gala
On Dec 3, 2008, at 11:04 PM, Becky Bruce wrote: > Some parts can support large physical mappings; so make the > size argument to addrmap_set_entry a phys_size_t. Also update > the 85xx tlb code to be sure the size quantity is unsigned. > > Signed-off-by: Becky Bruce > --- > cpu/mpc85xx/tlb.c