Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Reinhard Meyer
Wolfgang Denk wrote: > Just to give a datapoint for speed: > > measured on TQM5200 (MPC5200 at 400 MHz, environment size 16 kB): > > OLD implementation: > > => sete ttt 'date;for i in 0 1 2 3 4 5 6 7 8 9 ; do for j in 0 1 2 3 4 > 5 6 7 8 9 ; do for k in 0 1 2 3 4 5 6 7 8 9 ; do run tt ; done

[U-Boot] [PATCH v3] PXA: New MMC driver

2010-07-17 Thread Marek Vasut
The new driver is a complete rewrite. It uses the MMC framework and should support both pxa2xx and pxa3xx. Tested on: - Palm Tungsten|C PXA255 - Aeronix ZipitZ2 PXA270 - Marvell Zylonite 300 PXA300 This driver needs testing though. Signed-off-by: Marek Vasut --- drivers/mmc/Makefi

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-07-17 Thread Tabi Timur-B04825
On Jul 17, 2010, at 10:17 PM, "Jerry Van Baren" wrote: >> Signed-off-by: Timur Tabi > ^ > Need a space in your SOB line? That's your mailer, not me. Check the archive -- my message is correct. > I assume this is a "live" patch, and replaces the patch with the subject

Re: [U-Boot] [PATCH] common: add a grepenv command

2010-07-17 Thread Jerry Van Baren
On 07/16/2010 04:41 PM, Scott Wood wrote: > On Thu, 15 Jul 2010 22:15:22 -0500 > Kim Phillips wrote: > >> u-boot environments, esp. when boards are shared across multiple users, >> can get pretty large and time consuming to visually parse. >> The grepenv command this patch adds can be used in lieu

Re: [U-Boot] [PATCH v3] fdt: Add function to alloc phandle values

2010-07-17 Thread Jerry Van Baren
On 07/10/2010 09:25 AM, Kumar Gala wrote: > If we are creating reference (handles) to nodes in a device tree we need > to first create a new phandle in node and this needs a new phandle > value. So we search through the whole dtb to find the max phandle value > and return the next greater value fo

Re: [U-Boot] [PATCH] always relocate fdt into an lmb-allocated memory block

2010-07-17 Thread Jerry Van Baren
Hi Timur, On 05/24/2010 04:10 PM, Timur Tabi wrote: > The device tree (fdt) must always exist in within the bootmap (usually the > first 16MB of RAM). If it doesn't, then boot_relocate_fdt() will allocate an > LMB region in the bootmap and copy the fdt into that region. It will also > increase t

Re: [U-Boot] [PATCH 1/2] fdt: Add fdt_translate_address to convert reg node to cpu phys addr

2010-07-17 Thread Jerry Van Baren
Hi Kumar, On 07/17/2010 11:49 AM, Kumar Gala wrote: > > On Jul 4, 2010, at 1:18 PM, Kumar Gala wrote: > >> This code is extracted out of the Linux Kernel code from >> arch/powerpc/kernel/prom_parse.c. >> >> We maintain some of the same structure to support multiple bus types even >> though we only

Re: [U-Boot] [PATCH 2/2] fdt: Add fdt_node_offset_by_compat_reg helper

2010-07-17 Thread Jerry Van Baren
Hi Kumar, On 07/17/2010 11:49 AM, Kumar Gala wrote: > > On Jul 4, 2010, at 1:18 PM, Kumar Gala wrote: > >> Given a compatiable string and physical address try and find a node that compatible >> matches. This is useful when we want to find a specific device node to >> update (for exa

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Mike Frysinger
On Saturday, July 17, 2010 17:41:57 Wolfgang Denk wrote: > Mike Frysinger wrote: > > Reinhard that it is very common for boards to be deployed with a small > > env, so i wonder if the normal runtime experience is actually faster, or > > if the difference is about the same as system noise: boot u-bo

[U-Boot] [PATCH] new env: fix off-by-one error in setenv command

2010-07-17 Thread Wolfgang Denk
Fix an off-by-one error in the string concatenation for the setevn command which would write over the end of the allocated area, resulting eventually in a crash due to corruption of the malloc metadata. Signed-off-by: Wolfgang Denk --- Posted as incremental patch here to provide quick fix to test

Re: [U-Boot] [PATCH 4/5] Remove support for CONFIG_HAS_UID and "forceenv" command

2010-07-17 Thread Sergey Kubushyn
On Sat, 17 Jul 2010, Wolfgang Denk wrote: Ack by: Sergey Kubushyn --- > This (undocumented) concept was only in use for the MVSMR and > davinci_schmoogie Sergey Kubushyn boards. > Drop it for now. If really needed, it should be reimplemented > later in the context of the new environment comman

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Wolfgang Denk
In message <1279395948-25864-1-git-send-email...@denx.de> I wrote: > > The following patch series adds some utilities (qsort and hash table > based database functions) and uses these to reimplement the code used > for the internal handling of environment variables. > > > Motivation: > > * Old en

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201007171728.27716.vap...@gentoo.org> you wrote: > > i wonder if we could #ifdef the sorting/hashing. i tend to agree with The sorting does not take much: -> size lib/qsort.o textdata bss dec hex filename 264

Re: [U-Boot] [PATCH 1/5] Add basic errno support.

2010-07-17 Thread Wolfgang Denk
Dear Mike Frysinger, In message <201007171717.27154.vap...@gentoo.org> you wrote: > > > +++ b/lib/errno.c > > @@ -0,0 +1 @@ > > +int errno = 0; > > drop the "= 0" so that errno ends up in the bss ? This does not make any difference; the tools are clever enough to figure that out: -> nm

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Wolfgang Denk
Dear Reinhard Meyer, In message <4c42190b.8060...@emk-elektronik.de> you wrote: > > how many users are out there that have such huge environments that speed > is of any concern? This is not so much a question of huge environments. Even with small environments (say, 2 kb) speed may quickly become

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Mike Frysinger
On Saturday, July 17, 2010 16:56:43 Reinhard Meyer wrote: > Wolfgang Denk wrote: > > Disadvantages: > > > > - Image size grows by typically 5...7 KiB (might shrink a bit again on > > > > systems with redundant environment with a following patch series) > > Looking at the patches I can't really

Re: [U-Boot] [PATCH 1/5] Add basic errno support.

2010-07-17 Thread Mike Frysinger
On Saturday, July 17, 2010 15:45:44 Wolfgang Denk wrote: > --- /dev/null > +++ b/lib/errno.c > @@ -0,0 +1 @@ > +int errno = 0; drop the "= 0" so that errno ends up in the bss ? -mike signature.asc Description: This is a digitally signed message part. _

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Reinhard Meyer
Wolfgang Denk wrote: > The following patch series adds some utilities (qsort and hash table > based database functions) and uses these to reimplement the code used > for the internal handling of environment variables. > > > Motivation: > > * Old environment code used a pessimizing implementation: >

Re: [U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Wolfgang Denk
In message <1279395948-25864-1-git-send-email...@denx.de> you wrote: > The following patch series adds some utilities (qsort and hash table > based database functions) and uses these to reimplement the code used > for the internal handling of environment variables. Note: the patches are also avail

[U-Boot] [PATCH 3/5] Add hash table support as base for new environment code

2010-07-17 Thread Wolfgang Denk
This implementation is based on code from uClibc-0.9.30.3 but was modified and extended for use within U-Boot. Major modifications and extensions: * hsearch() [modified / extended]: - While the standard version does not make any assumptions about the type of the stored data objects at all,

[U-Boot] [PATCH 0/5] New environment code

2010-07-17 Thread Wolfgang Denk
The following patch series adds some utilities (qsort and hash table based database functions) and uses these to reimplement the code used for the internal handling of environment variables. Motivation: * Old environment code used a pessimizing implementation: - variable lookup used linear sea

[U-Boot] [PATCH 4/5] Remove support for CONFIG_HAS_UID and "forceenv" command

2010-07-17 Thread Wolfgang Denk
This (undocumented) concept was only in use for the MVSMR and davinci_schmoogie Sergey Kubushyn boards. Drop it for now. If really needed, it should be reimplemented later in the context of the new environment command set. Signed-off-by: Wolfgang Denk Cc: Andre Schwarz Cc: Sergey Kubushyn ---

[U-Boot] [PATCH 2/5] Add qsort - add support for sorting data arrays

2010-07-17 Thread Wolfgang Denk
Code adapted from uClibc-0.9.30.3 Signed-off-by: Wolfgang Denk --- include/common.h |4 +++ lib/Makefile |1 + lib/qsort.c | 69 ++ 3 files changed, 74 insertions(+), 0 deletions(-) create mode 100644 lib/qsort.c diff --git

[U-Boot] [PATCH 1/5] Add basic errno support.

2010-07-17 Thread Wolfgang Denk
Needed for hash table support; probably useful in a lot of other places as well. Signed-off-by: Wolfgang Denk --- include/errno.h |9 + lib/Makefile|1 + lib/errno.c |1 + 3 files changed, 11 insertions(+), 0 deletions(-) create mode 100644 include/errno.h create mo

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

2010-07-17 Thread Wolfgang Denk
Dear Anatolij Gustschin, In message <20100717001217.46e99...@wker> you wrote: > Dear Wolfgang, > > The following changes since commit b6c208ab1ebc2ac01e0029b8456210617ff67156: > Wolfgang Denk (1): > Merge branch 'master' of git://git.denx.de/u-boot-mpc85xx > > are available in the git

Re: [U-Boot] [PATCH v3] Drop support for GTH board

2010-07-17 Thread Wolfgang Denk
In message <1278362793-21116-1-git-send-email...@denx.de> you wrote: > The board maintainer states: > > The GTH board is obsolete and has not been manufactured for > several years. > To my knowledge, no recent U-Boot build has been tested on that > card. > > So drop support for th

Re: [U-Boot] [PATCH 2/2] fdt: Add fdt_node_offset_by_compat_reg helper

2010-07-17 Thread Kumar Gala
On Jul 4, 2010, at 1:18 PM, Kumar Gala wrote: > Given a compatiable string and physical address try and find a node that > matches. This is useful when we want to find a specific device node to > update (for example if we have multiple PCI nodes we can use the > physical address to distinquish t

Re: [U-Boot] [PATCH 1/2] fdt: Add fdt_translate_address to convert reg node to cpu phys addr

2010-07-17 Thread Kumar Gala
On Jul 4, 2010, at 1:18 PM, Kumar Gala wrote: > This code is extracted out of the Linux Kernel code from > arch/powerpc/kernel/prom_parse.c. > > We maintain some of the same structure to support multiple bus types even > though we only have one in the current code. In the future we might want >

Re: [U-Boot] [PATCH 3/4] powerpc/86xx: Rename PCI1/2 to PCIE1/2 on MPC8641HPCN & SBC8641

2010-07-17 Thread Kumar Gala
On Jul 9, 2010, at 12:18 AM, Kumar Gala wrote: > The MPC8641 boards actually only have PCIE not PCI. Rename so we are > uniform with regards to names so we can replace this code with templated > code. > > Signed-off-by: Kumar Gala > --- > board/freescale/mpc8641hpcn/law.c | 18 +++---

Re: [U-Boot] [PATCH 2/4] powerpc/86xx: Move PCI/PCIe address defines into common immap_86xx.h

2010-07-17 Thread Kumar Gala
On Jul 9, 2010, at 12:18 AM, Kumar Gala wrote: > Remove dupliacted setting of PCI/PCIe address and offsets in board > config.h. Renamed CONFIG_SYS_PCI1/2_ADDR to CONFIG_SYS_PCI1/2ADDR on > MPC8641 boards since its really PCIE controllers and not PCI. > > Signed-off-by: Kumar Gala > --- > arch/

Re: [U-Boot] [PATCH 1/4] powerpc/85xx: Move PCI/PCIe address defines into common immap_85xx.h

2010-07-17 Thread Kumar Gala
On Jul 9, 2010, at 12:18 AM, Kumar Gala wrote: > Remove dupliacted setting of PCI/PCIe address and offsets in board > config.h. > > Signed-off-by: Kumar Gala > --- > arch/powerpc/include/asm/immap_85xx.h | 20 > include/configs/ATUM8548.h|3 --- > include/co