Re: [U-Boot] Pull request u-boot-marvell.git

2010-05-21 Thread Prafulla Wadaskar
> -Original Message- > From: Wolfgang Denk [mailto:w...@denx.de] > Sent: Saturday, May 22, 2010 2:02 AM > To: Prafulla Wadaskar > Cc: Tom Rix; u-boot@lists.denx.de > Subject: Re: [U-Boot] Pull request u-boot-marvell.git > > Dear Prafulla Wadaskar, > > In message > com> you wrote: >

Re: [U-Boot] [PATCH] config.mk: use different host compiler for OS X 10.6

2010-05-21 Thread Mike Frysinger
On Friday 21 May 2010 17:33:14 Andreas Bießmann wrote: > ifeq ($(HOSTOS),darwin) > +# get major and minor product version (e.g. 10.6 for Snow Leopard) > +DARWIN_VERSION = $(shell sw_vers -productVersion | cut -f 1,2 -d '.') > + > +# Snow Leopards build environmenthas no longer restrictions a

Re: [U-Boot] x86 Custodianship

2010-05-21 Thread Graeme Russ
Hi Wolfgang, On 22/05/10 01:11, Wolfgang Denk wrote: > Dear Graeme Russ, [snip] > No need to. The x86 repo is ready for you. > Thanks Wolfgang One quick question - Since I am currently the only x86 developer, how do you want to handle the SOB + Ack'd for x86 patches? I feel that to date be a

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Wolfgang Denk
Dear Kumar Gala, In message <7e86bc8f-7e3f-49b1-b1c5-b21991303...@kernel.crashing.org> you wrote: > > wrote: > > > >> +#ifdef CONFIG_FSL_NGPIXIS > >> +#include "ngpixis.h" > >> +#define PIXIS_VSYSCLK0 offsetof(struct ngpixis, sclk[0]) > >> +#define PIXIS_VSYSCLK1 offsetof(struct

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > > >> +{ > >> + const unsigned long input_freq = CONFIG_ICS307_REFCLK_HZ; > >> + unsigned long VDW = ((cw1 << 1) & 0x1FE) + ((cw2 >> 7) & 1); > >> + unsigned long RDW = cw2 & 0x7F; > >> + unsigned long OD = ics307_s_to_od[cw0 & 0x7]

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Kumar Gala
On May 21, 2010, at 10:57 AM, Timur Tabi wrote: > On Fri, May 21, 2010 at 4:18 AM, Kumar Gala wrote: > >> +#ifdef CONFIG_FSL_NGPIXIS >> +#include "ngpixis.h" >> +#define PIXIS_VSYSCLK0 offsetof(struct ngpixis, sclk[0]) >> +#define PIXIS_VSYSCLK1 offsetof(struct ngpixis, sclk[1])

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Kumar Gala
On May 21, 2010, at 10:57 AM, Timur Tabi wrote: > On Fri, May 21, 2010 at 4:18 AM, Kumar Gala wrote: > >> +#ifdef CONFIG_FSL_NGPIXIS >> +#include "ngpixis.h" >> +#define PIXIS_VSYSCLK0 offsetof(struct ngpixis, sclk[0]) >> +#define PIXIS_VSYSCLK1 offsetof(struct ngpixis, sclk[1])

[U-Boot] [PATCH] config.mk: use different host compiler for OS X 10.6

2010-05-21 Thread Andreas Bießmann
Compiling tools subdirectory on Mac OS X 10.6 (Snow Leopard) complains about wrong syntax in system includes. In file included from /usr/include/stdio.h:444, from /Users/andreas/Documents/source/u-boot/include/compiler.h:26, from /Users/andreas/Documents/source/u

[U-Boot] [PATCH] a320evb: fix udelay / __udelay confusion

2010-05-21 Thread Wolfgang Denk
Fix the following compiler problems: arch/arm/cpu/arm920t/a320/liba320.a(timer.o): In function `udelay': /home/wd/git/u-boot/work/arch/arm/cpu/arm920t/a320/timer.c:160: multiple definition of `udelay' lib/libgeneric.a(time.o):/home/wd/git/u-boot/work/lib/time.c:34: first defined here lib/libgene

[U-Boot] [PATCH] ARM: */timer.c: fix spelling and vertical alignment

2010-05-21 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- arch/arm/cpu/arm1136/mx31/timer.c|2 +- arch/arm/cpu/arm1136/omap24xx/timer.c| 22 +++--- arch/arm/cpu/arm920t/a320/timer.c|2 +- arch/arm/cpu/arm926ejs/omap/timer.c |4 ++-- arch/arm/cpu/arm926ejs/versatile/t

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: > Well, the function is "strMHZ", and it prints a frequency in MHz with > exactly 3 digits precision - so rounding to the nearest kHz is > exactly the right thing to do. Actually, it's not really three significant digits. "799.992" is six significant digits. If you you want

Re: [U-Boot] [PATCH] arm:io.h: define __io() macro if not defined erlier

2010-05-21 Thread Wolfgang Denk
Dear =?UTF-8?q?Andreas=20Bie=C3=9Fmann?=, In message <1274259427-29386-1-git-send-email-biessm...@corscience.de> you wrote: > This patch adds __io() macro to enable {in|out}[bwl] macros in > arch/arm/asm/io.h > > Commit a45dde2293c816138e53c26eca6fd0322583f9a6 uses {in|out}[bwl] > macros in > DM

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message <4bf6ecd0.4060...@freescale.com> you wrote: > > However, the current code adds a 1 to the result, which is definitely wrong. > On my board, for example, it returns 9001, which is not correct. So > the existing "rounding" algorithm is broken. Agreed. > However,

Re: [U-Boot] [PATCH v2] fsl/85xx: add clkdvdr and pmuxcr2 to global utilities structure definition

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message <1274377539-14922-1-git-send-email-ti...@freescale.com> you wrote: > Add the 'clkdvdr' and 'pmuxcr2' registers to the 85xx definition of > struct ccsr_gur. Please keep the subject line short enought to fit as title of the commit message. Also, it makes little sense to

Re: [U-Boot] Pull request u-boot-marvell.git

2010-05-21 Thread Wolfgang Denk
Dear Prafulla Wadaskar, In message you wrote: > Hi Tom > > Pls pull > > The following changes since commit 0cf7d2440e1aa91ca65f1fc08ba309184d376da6: > Stefano Babic (1): > Add SPI support to mx51evk board > > are available in the git repository at: > > u-boot-marvell.git master b

[U-Boot] [GIT PULL] Please pull u-boot-mpc83xx.git

2010-05-21 Thread Kim Phillips
Wolfgang, please pull one cpu init fix, and a typo fix*: The following changes since commit 2f05e394fccf62a4693c6b8323de725f90d1f003: fsl_diu_fb.c: fix build warnings (2010-05-17 23:34:18 +0200) are available in the git repository at: git://git.denx.de/u-boot-mpc83xx.git master Horst Krons

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: > 799.992 MHz seems to be 24 * 33,333,000 Hz. Are you sure this is the > exact quartz frequency on your board? If yes, then the number printed > should probably remain 799.992 MHz. Or is it 33,333,333 Hz? Or > 33,000,000 ? Ok, I see your point. Technically, I am returning a

Re: [U-Boot] [PATCH] mpc83xx: don't shift pre-shifted ACR, SPCR, SCCR bitfield masks in cpu_init.c

2010-05-21 Thread Kim Phillips
On Wed, 19 May 2010 17:06:46 -0500 Kim Phillips wrote: > commit c7190f028fa950d4d36b6d0b4bb3fc72602ec54c "mpc83xx: > retain POR values of non-configured ACR, SPCR, SCCR, and LCRR > bitfields" incorrectly shifted _ (e.g. > ACR_PIPE_DEP) values that were preshifted by their > definition in mpc83xx.

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

2010-05-21 Thread Wolfgang Denk
Dear Stefan Roese, In message <201005191058.49502...@denx.de> you wrote: > The following changes since commit a2a0a7171303de5d8ce099344efde2e29ee36eb0: > > Makefile: fix out-of-tree building of "u-boot.img" target (2010-05-16 > 02:18:38 +0200) > > are available in the git repository at: > g

Re: [U-Boot] [PATCH] Fixed two typos in arch/powerpc/cpu/mpc83xx/start.S.

2010-05-21 Thread Kim Phillips
On Tue, 18 May 2010 10:37:05 +0200 Horst Kronstorfer wrote: > > Signed-off-by: Horst Kronstorfer > --- applied. Thanks, Kim ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Capability to select from more than one kernel to boot ?

2010-05-21 Thread Kim Phillips
On Fri, 21 May 2010 17:35:58 +0100 Ayewin Oung wrote: > Hi list > > I like to make Linux kernel "updates" robust to user/power failurer, during > the porcess. > > Is there a well accepted way to do this for u-boot? using some form of file > system (ubifs, jffs2, etc ?) with u-boot where kerne

Re: [U-Boot] Pull Request for imx

2010-05-21 Thread Wolfgang Denk
Dear Stefano Babic, In message <4bf394ae.10...@denx.de> you wrote: > Wolfgang, > > could you pull u-boot-imx in the time Tom is not available ? Tom has > already merged most of the patches in u-boot-arm, I resend you the > pull-request for completeness: > > The following changes since commit ab9

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message <4bf6e5df.5020...@freescale.com> you wrote: > > >> So here's a better version of that function that rounds to the nearest > >> MHz and is of a proper coding style: > > > > Why do we need that? > > Um, because you complained about it? I mean, why do we need that funct

[U-Boot] [PATCH] Fix "par[t]ition" typo.

2010-05-21 Thread Wolfgang Denk
Signed-off-by: Wolfgang Denk --- common/cmd_flash.c|6 +++--- common/cmd_nand.c |2 +- doc/README.JFFS2_NAND |2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/common/cmd_flash.c b/common/cmd_flash.c index 3773412..6361c4e 100644 --- a/common/cmd_flash.c +++

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: >> So here's a better version of that function that rounds to the nearest >> MHz and is of a proper coding style: > > Why do we need that? Um, because you complained about it? >> > +static unsigned long ics307_clk_freq(unsigned char cw0, unsigned char cw1, >> > +

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message you wrote: > > So here's a better version of that function that rounds to the nearest > MHz and is of a proper coding style: Why do we need that? > And the result: > > Clock Configuration: >CPU0:800 MHz, CPU1:800 MHz, >CCB:400 MHz, >DDR:3

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
On Fri, May 21, 2010 at 1:59 PM, Timur Tabi wrote: > Is this rounding algorithm the reason why I get output like this: > > Clock Configuration: >       CPU0:799.992 MHz, CPU1:799.992 MHz, >       CCB:399.996 MHz, >       DDR:299.997 MHz (599.994 MT/s data rate) (Asynchronous), LBC:25 >  MHz > > I

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano, In message <4bf6b724.6090...@denx.de> you wrote: > > > Adding spaces to the variable value? This makes little sense to me. > > It's just a waste of storage space and boot time. > > Well, I can agree with you, however I have already seen this case... I believe you. I have seen stra

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
On Fri, May 21, 2010 at 4:18 AM, Kumar Gala wrote: > +        * Adding 1 to get a "nicely" rounded number, but this needs > +        * more tweaking to get a "properly" rounded number. > +        */ > + > +       freq = 1 + (input_freq * 2 * (VDW + 8) / ((RDW + 2) * OD)); Is this rounding algori

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Peter Tyser
On Fri, 2010-05-21 at 13:34 -0500, Scott Wood wrote: > On 05/21/2010 01:18 PM, Timur Tabi wrote: > > Scott Wood wrote: > >> On 05/21/2010 01:07 PM, Timur Tabi wrote: > >>> On most Freescale 85xx boards, the CONFIG_SYS_CLK_FREQ and > >>> CONFIG_DDR_CLK_FREQ > >>> macros are defined like this: > >>

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Scott Wood
On 05/21/2010 01:36 PM, Timur Tabi wrote: > Scott Wood wrote: > >>> Doh. Git commit delete those lines because they began with a "#". >> >> Hmm... is there any way to override that and insert such a line into a >> git commit? > > I was going to just insert a blank space or some other character bef

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
On Fri, May 21, 2010 at 4:18 AM, Kumar Gala wrote: > +COBJS-$(CONFIG_P2020DS)                += ics307_clk.o How about changing this to: COBJS-$(CONFIG_ICS307_REFCLK_HZ) += ics307_clk.o -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot ma

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Timur Tabi
Timur Tabi wrote: > Well, you'll have to convince Wolfgang of that, not me. He won't accept my > P1022DS board patch until I fix this "problem". Actually, if you look at Kumar's ICS307 patch, you'll see he fixes this problem for any board that uses the ICS307: > -#ifndef __ASSEMBLY__ > -extern

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Timur Tabi
Scott Wood wrote: >> Doh. Git commit delete those lines because they began with a "#". > > Hmm... is there any way to override that and insert such a line into a > git commit? I was going to just insert a blank space or some other character before the #. > I think the board header file (or so

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Scott Wood
On 05/21/2010 01:18 PM, Timur Tabi wrote: > Scott Wood wrote: >> On 05/21/2010 01:07 PM, Timur Tabi wrote: >>> On most Freescale 85xx boards, the CONFIG_SYS_CLK_FREQ and >>> CONFIG_DDR_CLK_FREQ >>> macros are defined like this: >> >> Like what? > > Doh. Git commit delete those lines because they

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Timur Tabi
Scott Wood wrote: > On 05/21/2010 01:07 PM, Timur Tabi wrote: >> On most Freescale 85xx boards, the CONFIG_SYS_CLK_FREQ and >> CONFIG_DDR_CLK_FREQ >> macros are defined like this: > > Like what? Doh. Git commit delete those lines because they began with a "#". This was supposed to be there: #

Re: [U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Scott Wood
On 05/21/2010 01:07 PM, Timur Tabi wrote: > On most Freescale 85xx boards, the CONFIG_SYS_CLK_FREQ and CONFIG_DDR_CLK_FREQ > macros are defined like this: Like what? > This means that in order to use these macros, the callers must have prototypes > for the corresponding functions. On 85xx, only

[U-Boot] [PATCH] mpc85xx: add function prototypes for sys and ddr clocks to speed.c

2010-05-21 Thread Timur Tabi
On most Freescale 85xx boards, the CONFIG_SYS_CLK_FREQ and CONFIG_DDR_CLK_FREQ macros are defined like this: This means that in order to use these macros, the callers must have prototypes for the corresponding functions. On 85xx, only speed.c uses these macros, so let's define the prototypes ther

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Wolfgang Denk wrote: > Adding spaces to the variable value? This makes little sense to me. > It's just a waste of storage space and boot time. Well, I can agree with you, however I have already seen this case... > If you feel your environment is so complicated to read that you want > such "forma

[U-Boot] Capability to select from more than one kernel to boot ?

2010-05-21 Thread Ayewin Oung
Hi list I like to make Linux kernel "updates" robust to user/power failurer, during the porcess. Is there a well accepted way to do this for u-boot? using some form of file system (ubifs, jffs2, etc ?) with u-boot where kernel is stored etc etc.. Any pointers much appreciated. ayewin

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: >> > +#ifndef __ASSEMBLY__ >> > +extern unsigned long calculate_board_sys_clk(void); >> > +extern unsigned long calculate_board_ddr_clk(void); >> > +#endif > > >>> > > >>> Please move to appropriate header file. > ... >> > Well, this

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano Babic, In message <4bf6a381.7020...@denx.de> you wrote: > Wolfgang Denk wrote: > > >> a config file with a list of pairs to be set, > >> separated by a TAB character. > > > > I think we should be less restrictive here. Please split at the first > > white space, and allow for multip

Re: [U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Timur Tabi
On Fri, May 21, 2010 at 4:18 AM, Kumar Gala wrote: > +#ifdef CONFIG_FSL_NGPIXIS > +#include "ngpixis.h" > +#define PIXIS_VSYSCLK0         offsetof(struct ngpixis, sclk[0]) > +#define PIXIS_VSYSCLK1         offsetof(struct ngpixis, sclk[1]) > +#define PIXIS_VSYSCLK2         offsetof(struct ngpixis

Re: [U-Boot] [PATCH] powerpc: add support for the FreescaleP1022DS reference board

2010-05-21 Thread Timur Tabi
Liu Dave-R63238 wrote: > Did you grab the patch from our latest BSP tree? No, I've been using the ltib patches. I guess I should use the repository on git.ap instead. > It seems like your patch doesn't base on latest upstream tree, IIRC, the > PCI stuff > in mpc8xxx/pci_cfg.c is ready in upstre

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Wolfgang Denk wrote: >> a config file with a list of pairs to be set, >> separated by a TAB character. > > I think we should be less restrictive here. Please split at the first > white space, and allow for multiple white spaces. There is a reason why I set to this format. There should be a case

Re: [U-Boot] x86 Custodianship

2010-05-21 Thread Wolfgang Denk
Dear Graeme Russ, In message <4bf52993.80...@gmail.com> you wrote: > > I've been a bit busy at home with bub #2 to chase this up, but have you > decided whether or not to create an x86 repository? If so, I would be happy > to take up custodianship of it and 'fly the flag' for U-Boot on the x86 >

[U-Boot] S spoštovanjem,

2010-05-21 Thread WHITON MARK
Garancije za posojila ponudbe z obrestno mero v višini 2,5% mesečno. Ali imate slabo kreditno ali ste potrebujejo denar za plačilo računov? hočem raba to medij, da vas obvestim, da sem postal zanesljiv upravičenca pomoči, kot bom vesel, da vam ponudimo posojilo z obrestno mero 2,5% stopnja. Stor

Re: [U-Boot] [Nios2-dev] [PATCH 4/6 v10] spi: add altera spi controller support

2010-05-21 Thread Ian Abbott
On 30/04/10 04:34, Thomas Chou wrote: > This patch adds the driver of altera spi controller, which is > used as epcs/spi flash controller. It also works with mmc_spi > driver. > > This driver support more than one spi bus, with base list declared > #define CONFIG_SYS_ALTERA_SPI_LIST { BASE_0,BASE_

Re: [U-Boot] [Nios2-dev] [PATCH 6/6 v3] nios2: add spi flash support to nios2-generic board

2010-05-21 Thread Ian Abbott
On 30/04/10 04:34, Thomas Chou wrote: > This patch enables the altera_spi and spi_flash drivers for the > nios2-generic board. It allows access to the EPCS/SPI flash on > the Altera EP1C20 board. > > Signed-off-by: Thomas Chou Tested with EPCS64N flash chip on Altera 2C35 development board. Tes

Re: [U-Boot] [Nios2-dev] [PATCH 3/6 v3] nios2: add gpio support to nios2-generic board

2010-05-21 Thread Ian Abbott
On 30/04/10 04:34, Thomas Chou wrote: > This patch adds gpio support of Altera PIO component to the > nios2-generic board. Though it drives only gpio_led at the > moment, it supports bidirectional port to control bit-banging > I2C, NAND flash busy status or button switches, etc. > > Signed-off-by:

Re: [U-Boot] [Nios2-dev] [PATCH 2/6 v2] misc: add gpio based status led driver

2010-05-21 Thread Ian Abbott
On 30/04/10 04:34, Thomas Chou wrote: > This patch adds a status led driver followed the GPIO access > conventions of Linux. The led mask is used to specify the gpio pin. > > Signed-off-by: Thomas Chou Tested on Altera 2C35 development board. Tested-by: Ian Abbott -- -=( Ian Abbott @ MEV Ltd

Re: [U-Boot] [Nios2-dev] [PATCH 1/6 v4] nios2: add gpio support

2010-05-21 Thread Ian Abbott
On 30/04/10 04:34, Thomas Chou wrote: > This patch adds driver for a trivial gpio core, which is described > in http://nioswiki.com/GPIO. It is used for gpio led and nand flash > interface in u-boot. > > When CONFIG_SYS_GPIO_BASE is not defined, board may provide > its own driver. > > Signed-off-

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: > Unless you pay me for this (or FSL) you should not even rely on this. Heh. > Well, there are things and things. > > When we discover that crap has piled up here and there, it's a good > idea to perform a clean up. I agree 100%. > When the first one adds some feature or

Re: [U-Boot] use of global static variables?

2010-05-21 Thread Wolfgang Denk
Dear Peter Tyser, In message <1274450989.18152.273.ca...@petert> you wrote: > > > With the relocation changes are global statics variables ok if we know code > > is called after relocation? > > Yes, they should work as expected after relocation. > > Prior to relocation you should be able to re

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message <4bf68e6a.4070...@freescale.com> you wrote: > > > Do you really expect me to review all patches submitted on this list > > in scrupulous detail? > > No, apparently just mine. Unless you pay me for this (or FSL) you should not even rely on this. > > It would be nice

Re: [U-Boot] use of global static variables?

2010-05-21 Thread Peter Tyser
Hi Kumar, On Fri, 2010-05-21 at 07:52 -0500, Kumar Gala wrote: > With the relocation changes are global statics variables ok if we know code > is called after relocation? Yes, they should work as expected after relocation. Prior to relocation you should be able to read the values of the global

Re: [U-Boot] use of global static variables?

2010-05-21 Thread Wolfgang Denk
Dear Kumar Gala, In message you wrote: > With the relocation changes are global statics variables ok if we > know code is called after relocation? They have always been ok (as far as global variables can be ok) if you were aware of their restrictions before relocation. Best regards, Wolfgang

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Timur Tabi
Wolfgang Denk wrote: > fsl_ddr_sdram() does NOT the same as get_ram_size(). It does not > perform any actual testing, and will not detect errors in the SPD > information or in the code processing it. Ok, I'll add it. >> This code is identical to the code in the p2020ds.c, so I'm just >> mirrorin

Re: [U-Boot] [PATCH] DaVinci: Improve DaVinci SPI speed.

2010-05-21 Thread Ben Gardiner
On Fri, May 21, 2010 at 9:15 AM, Ben Gardiner wrote: > Applies cleanly and compiles with da850evm_config on u-boot-omap-l1 > master branch -- 5f16b85 of 2f05e39 . Also running the resulting Sorry, that should be 5f16b85 of git://arago-project.org/git/people/sekhar/u-boot-omapl1.git . -- Ben Gar

Re: [U-Boot] [PATCH] DaVinci: Improve DaVinci SPI speed.

2010-05-21 Thread Ben Gardiner
On Thu, May 13, 2010 at 8:57 AM, Delio Brignoli wrote: > Reduce the number of reads per byte transferred on the BUF register from 2 to > 1 and > take advantage of the TX buffer in the SPI module. On LogicPD OMAP-L138 EVM, > SPI read throughput goes up from ~0.8Mbyte/s to ~1.3Mbyte/s. Tested with

Re: [U-Boot] [PATCH] Improve DaVinci SPI speed

2010-05-21 Thread Wolfgang Denk
Dear Delio Brignoli, In message <25096244-c685-4d68-bbf9-6df2f154a...@audioscience.com> you wrote: > Reduce the number of reads per byte transferred on the BUF register from 2 to > 1 and > take advantage of the TX buffer in the SPI module. ... > + /* if data is available */ > +

[U-Boot] use of global static variables?

2010-05-21 Thread Kumar Gala
With the relocation changes are global statics variables ok if we know code is called after relocation? - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for P2020SERDES

2010-05-21 Thread Kumar Gala
On May 21, 2010, at 6:26 AM, Li Yang-R58472 wrote: >> Subject: [U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for >> P2020SERDES >> >> Add the ability to determine if a given IP block connected on SERDES is >> configured. This is useful for things like PCIe and SRIO since they are >>

Re: [U-Boot] [PATCH] Improve DaVinci SPI speed

2010-05-21 Thread Delio Brignoli
Hello Sekhar, I resubmitted the patch a few days ago (13th of May) with the changes you requested. Please ack it, so it can be applied. Thanks -- Delio On 18/05/2010, at 14:25, Nori, Sekhar wrote: > On Thu, May 13, 2010 at 18:27:51, Delio Brignoli wrote: >> Reduce the number of reads per byte t

Re: [U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for P2020SERDES

2010-05-21 Thread Li Yang
On 5/21/2010 5:17 PM, Kumar Gala wrote: > >Add the ability to determine if a given IP block connected on SERDES is >configured. This is useful for things like PCIe and SRIO since they are >only ever connected on SERDES. > >Signed-off-by: Kumar Gala mailto:ga...@kernel.crashing.org>> I'

[U-Boot] NAND write issue is DM355

2010-05-21 Thread Mohamed Thalib H
Hi all, I am porting u-boot to TI DM355 based board. When I write to nand using nand write command upto offset of 0x6 it is working fine. but If I write above 0x7 it is writing to 0x0 address. Can some one give me suggestion how to correct this issue. NAND size 64M -- Best Regards, M

Re: [U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Wolfgang Denk
Dear Stefano Babic, In message <1274439131-22807-1-git-send-email-sba...@denx.de> you wrote: > Add a sort of batch mode to fw_setenv, allowing to set > multiple variables in one shot, without updating the flash after > each set as now. It is added the possibility to pass Thanks; that's a welcome

[U-Boot] [PATCH] Tools: set multiple variable with fw_setenv utility

2010-05-21 Thread Stefano Babic
Add a sort of batch mode to fw_setenv, allowing to set multiple variables in one shot, without updating the flash after each set as now. It is added the possibility to pass a config file with a list of pairs to be set, separated by a TAB character. Signed-off-by: Stefano Babic --- tools/env/fw_

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Kumar Gala
On May 20, 2010, at 5:01 PM, Timur Tabi wrote: > diff --git a/arch/powerpc/cpu/mpc8xxx/pci_cfg.c > b/arch/powerpc/cpu/mpc8xxx/pci_cfg.c > index 85995ca..fc79fe1 100644 > --- a/arch/powerpc/cpu/mpc8xxx/pci_cfg.c > +++ b/arch/powerpc/cpu/mpc8xxx/pci_cfg.c > @@ -186,6 +186,32 @@ static struct pci_i

[U-Boot] Rusya Kargolarınızın Kapasites i ne olursa olsun.Biz Adresinizden Alıp Müşter inizin Adresine Kadar Teslim Garantisi ile Taşıyoruz .

2010-05-21 Thread MPS & ICS EXPRESS CARGO
MPS Express - RUSYA gönderilerini kabul etmeye devam edeceğini açıkladı Rusya Merkezli MPS Express & Logistics firması müşterilerine Rusya Gümrüğündeki uygulamalar sebebiyle sorun yaşayan özel\şahış veya kurumlar için gümrüklü yada gümrüksüz tüm Rusya Kargolarını paketlerini almaya devam ed

Re: [U-Boot] [PATCH 2/2] fsl: rename 'dma' to 'brdcfg1' in the ngPIXIS structure

2010-05-21 Thread Kumar Gala
On May 20, 2010, at 11:16 AM, Timur Tabi wrote: > The ngPIXIS is a board-specific FPGA, but the definition of the registers > is mostly consistent. On boards where it matter, register 9 is called > 'brdcfg1' instead of 'dma', so rename the variable in the ngpixis_t > definition. > > Signed-off-

Re: [U-Boot] [PATCH v2] fsl/85xx: add clkdvdr and pmuxcr2 to global utilities structure definition

2010-05-21 Thread Kumar Gala
On May 20, 2010, at 12:45 PM, Timur Tabi wrote: > Add the 'clkdvdr' and 'pmuxcr2' registers to the 85xx definition of > struct ccsr_gur. > > Signed-off-by: Timur Tabi > --- > > This patch replaces "fsl/85xx: add clkdvdr to global utilities structure > definition". > Thanks to Sergei Shtylyov.

[U-Boot] [PATCH] ppc/85xx: Convert MPC8536DS to using board common ICS307 code

2010-05-21 Thread Kumar Gala
Signed-off-by: Kumar Gala --- board/freescale/common/Makefile |1 + board/freescale/mpc8536ds/mpc8536ds.c | 148 - include/configs/MPC8536DS.h | 14 +-- 3 files changed, 6 insertions(+), 157 deletions(-) diff --git a/board/freescale/common/M

[U-Boot] [PATCH] Move ICS CLK chip frequenty calculation code into a common board library

2010-05-21 Thread Kumar Gala
We have several boards that use the same ICS307 CLK chip to drive the System clock and DDR clock. Move the code into a common location so we share it. Convert the P2020DS board as the first to use the new common ICS307 code. Signed-off-by: Kumar Gala --- board/freescale/common/Makefile |

[U-Boot] [PATCH] ppc/85xx: Convert MPC8572DS to using board common ICS307 code

2010-05-21 Thread Kumar Gala
Signed-off-by: Kumar Gala --- board/freescale/common/Makefile |1 + board/freescale/mpc8572ds/mpc8572ds.c | 145 + include/configs/MPC8572DS.h | 14 +-- 3 files changed, 7 insertions(+), 153 deletions(-) diff --git a/board/freescale/common/M

[U-Boot] [PATCH] 85xx/p2020ds: Use is_serdes_configured() to determine of PCIe enabled

2010-05-21 Thread Kumar Gala
The new is_serdes_configured covers a broader range of devices than the PCI specific code. Use it instead as we convert away from the is_fsl_pci_cfg() code. Additionally move to setting LAWs for PCI based on if its configured. Also updated PCI FDT fixup code to remove PCI controllers from dtb if

[U-Boot] [PATCH] 85xx: Add is_serdes_configured() support for P2020 SERDES

2010-05-21 Thread Kumar Gala
Add the ability to determine if a given IP block connected on SERDES is configured. This is useful for things like PCIe and SRIO since they are only ever connected on SERDES. Signed-off-by: Kumar Gala --- arch/powerpc/cpu/mpc85xx/Makefile |6 ++- arch/powerpc/cpu/mpc85xx/p2020_serdes.

[U-Boot] [PATCH] Convert Makefiles from COBJS-${} to COBJS-$()

2010-05-21 Thread Kumar Gala
Match style we use almost everywhere else Signed-off-by: Kumar Gala --- arch/powerpc/cpu/mpc512x/Makefile| 10 +- board/freescale/common/Makefile | 22 +++--- board/freescale/mpc8610hpcd/Makefile |2 +- 3 files changed, 17 insertions(+), 17 deletions(-)

Re: [U-Boot] Watchdog support for ppc4xx

2010-05-21 Thread Detlev Zundel
Hi Wolfgang, > Dear Stefan Roese, > > In message <201005201355.01964...@denx.de> you wrote: >> >> > Also I would like to determine in u-boot if a >> > reset was caused by the watchdog timer using the TSR WRS field. If it >> > was reset by the watchdog we will boot into a failsafe partition to >>

Re: [U-Boot] Support for MB86R01 cpu

2010-05-21 Thread Detlev Zundel
Hi Thirumalai, > Hi Matthias, > >> Hi Thirumalai >> >> Am 18.05.2010 08:29, schrieb Thirumalai: >>> Hi Matthias, >>> I am going to use u-boot alone for my development. Which debugger >>> is >>> suitable for debugging u-boot and initial board bringup activity? >>> My choice is BDI2000. Is

[U-Boot] Video Driver Support

2010-05-21 Thread Thirumalai
Hi Stefan, I just want know whether the following GPU is supported by u-boot driver. ATI RADEON E4690(M96). Or Is there any other way to add support for this GPU. I want linux need to be booted on this GPU based Graphics Card. -Thirumalai ___

Re: [U-Boot] Flash Erase and Write timeout for S29GL512P

2010-05-21 Thread Wolfgang Denk
Dear prakash bedge, In message you wrote: > > Flash chip I am using is S29GL512P and as per datasheet flash erase time is > 1024 Seconds and flash program time is 492 Seconds. Be careful. 1024 s is chip erase time, but we use sector erase only, so it's only 3.5 seconds. Same for the 492 s - th

Re: [U-Boot] [PATCH] powerpc: add support for the Freescale P1022DS reference board

2010-05-21 Thread Wolfgang Denk
Dear Timur Tabi, In message you wrote: ... > >> +/* ranges for parameters: > >> + * =A0wr_data_delay =3D 0-6 > >> + * =A0clk adjust =3D 0-8 > >> + * =A0cpo 2-0x1E (30) > >> + */ > > > > Incorrect multiline comment style. > > Sorry, what's wrong with this, specifically? Should it look like this