Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-15 Thread Joakim Tjernlund
Wolfgang Grandegger wrote on 15/09/2009 13:53:13: > > Joakim Tjernlund wrote: > >> Wolfgang Grandegger wrote: > >> > >>> I did not follow the thread yet, sorry. I implemented AN2819 for Linux > >>> (see http://lxr.linux.no/#linux+v2.6.31/drivers

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-14 Thread Joakim Tjernlund
> > Wolfgang Grandegger wrote: > > > I did not follow the thread yet, sorry. I implemented AN2819 for Linux > > (see http://lxr.linux.no/#linux+v2.6.31/drivers/i2c/busses/i2c-mpc.c) > > some time ago using Timur's table approach. But there is no difference > > between the table and the algorithm to

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-14 Thread Joakim Tjernlund
> > Timur Tabi wrote: > > Wolfgang Grandegger wrote: > > > >> I did not follow the thread yet, sorry. I implemented AN2819 for Linux > >> (see http://lxr.linux.no/#linux+v2.6.31/drivers/i2c/busses/i2c-mpc.c) > >> some time ago using Timur's table approach. But there is no difference > >> between th

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-11 Thread Joakim Tjernlund
> > Timur Tabi wrote on 10/09/2009 18:13:03: > > > > Joakim Tjernlund wrote: > > > > > This calculation does not seem to match AN2919. > > > > When I wrote the code, AN2919 was much smaller than what you have today. > > > > > Suppose o

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 18:13:03: > > Joakim Tjernlund wrote: > > > This calculation does not seem to match AN2919. > > When I wrote the code, AN2919 was much smaller than what you have today. > > > Suppose one used only Table 7(almost what we have if you ex

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 18:13:03: > > Joakim Tjernlund wrote: > > > This calculation does not seem to match AN2919. > > When I wrote the code, AN2919 was much smaller than what you have today. > > > Suppose one used only Table 7(almost what we have if you ex

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 17:26:29: > > Joakim Tjernlund wrote: > > > Looking a bit harder at the table I don't understand some entries, where > > does > > the entries with dfsr != 1 come from? They don't look like any table in > > AN2919

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 17:22:38: > > Joakim Tjernlund wrote: > > > Come on, just because my board is somewhat broken, it doesn't mean the > > driver is correct. If I define my speed to 100KHz I get > > a DFSR of 22, way over what is allowed for my board. &g

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 15:29:35: > > Joakim Tjernlund wrote: > > >> A while back, someone posted a version of this code that computed the > >> values > >> of fdr/dfsr. I nack'd that patch because I thought the algorithm was too > > > >

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 15:29:35: > > Joakim Tjernlund wrote: > > >> A while back, someone posted a version of this code that computed the > >> values > >> of fdr/dfsr. I nack'd that patch because I thought the algorithm was too > > > >

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
Timur Tabi wrote on 10/09/2009 15:07:36: > > Joakim Tjernlund wrote: > > > BTW, the fdr and dfsr calculations appears totally bogus. It seems > > like the table is taken from some examples in AN2919 and it is pure luck > > that it works most of the time. For me it do

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-10 Thread Joakim Tjernlund
> timur.t...@gmail.com wrote on 09/09/2009 16:24:15: > > > > On Wed, Sep 9, 2009 at 4:19 AM, Joakim > > Tjernlund wrote: > > > > > > I wonder if this hides another problem too. > > > if the timeout hits, -1 is returned. > > > >

Re: [U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-09 Thread Joakim Tjernlund
timur.t...@gmail.com wrote on 09/09/2009 16:24:15: > > On Wed, Sep 9, 2009 at 4:19 AM, Joakim > Tjernlund wrote: > > > > I wonder if this hides another problem too. > > if the timeout hits, -1 is returned. > > > > Then in i2c_read()/i2c_write() you h

[U-Boot] fsl_i2c: increase I2C timeout values and make them configurable

2009-09-09 Thread Joakim Tjernlund
I wonder if this hides another problem too. if the timeout hits, -1 is returned. Then in i2c_read()/i2c_write() you have: if (i2c_wait4bus() >= 0 && i2c_write_addr(dev, I2C_WRITE_BIT, 0) != 0 && __i2c_write(&a[4 - alen], alen) == alen) i = 0; /* No

Re: [U-Boot] PowerPC -mrelocatable

2009-09-09 Thread Joakim Tjernlund
> > Dear Peter, > > In message <1252426573.6005.253.ca...@localhost.localdomain> you wrote: > > > > Going over the emails and my own testing, it looks the following > > versions worked: > ... > > Thanks for the detailed analysis. > > I remember that gcc-3.4.x has always been marked as "suspicious"

Re: [U-Boot] PowerPC -mrelocatable

2009-08-27 Thread Joakim Tjernlund
> > Peter Tyser wrote: > > On Thu, 2009-08-27 at 10:38 -0500, Scott Wood wrote: > >> Someone tried to get proper relocation working a while ago, but ran into > >> toolchain bugs. Maybe current toolchains are better... > > > > X-ES's board's in U-Boot fully relocate to SDRAM with the > > CONFIG_REL

Re: [U-Boot] [PATCH] 85xx: Fix the wrong SYS_CLK_IN for 8569MDS

2009-05-18 Thread Joakim Tjernlund
> > The SYS_CLK_IN of MPC8569MDS is 66.66MHz, > The DDR_CLK_IN is same with SYS_CLK_IN in 8569 processor. > so, change the SYS_CLK_IN from 66MHz to 66.66MHz. > > Signed-off-by: Dave Liu > --- > include/configs/MPC8569MDS.h |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff -

[U-Boot] Hard SPI broken on mpc832x?

2009-05-04 Thread Joakim Tjernlund
I get this when compiling on a fairly recent u-boot: mpc8xxx_spi.c: In function `spi_init': mpc8xxx_spi.c:70: error: syntax error before '*' token mpc8xxx_spi.c:76: error: `spi' undeclared (first use in this function) mpc8xxx_spi.c:76: error: (Each undeclared identifier is reported only once

Re: [U-Boot] PCI on mpc832x?

2009-04-29 Thread Joakim Tjernlund
Kumar Gala wrote on 29/04/2009 14:35:19: > > > I think I got the mapping of memory space now, thanks. > > > > A few questions though. I have noted that some boards map the PCI IO > > space > > to 0x0 and some use the same address space as the CPU. If I use > > the CPU address space as PCI IO spa

Re: [U-Boot] PCI on mpc832x?

2009-04-29 Thread Joakim Tjernlund
Kumar Gala wrote on 27/04/2009 16:12:35: > From: > > Kumar Gala > > To: > > Joakim Tjernlund > > Cc: > > Scott Wood , u-boot@lists.denx.de > > Date: > > 27/04/2009 16:15 > > Subject: > > Re: [U-Boot] PCI on mpc832x? &g

Re: [U-Boot] PCI on mpc832x?

2009-04-24 Thread Joakim Tjernlund
Scott Wood wrote on 23/04/2009 18:40:01: > > On Thu, Apr 23, 2009 at 03:32:11PM +0200, Joakim Tjernlund wrote: > > Still trying to wrap my head around PCI and I wonder if I need to do some > > HW init in u-boot in order to use the PCI controller in Linux? > > Yes.

[U-Boot] PCI on mpc832x?

2009-04-23 Thread Joakim Tjernlund
Still trying to wrap my head around PCI and I wonder if I need to do some HW init in u-boot in order to use the PCI controller in Linux? Obviously I need to pass PCI settings in the OF tree, but other than that? Jocke ___ U-Boot mailing list U-Boot@list

Re: [U-Boot] [PATCH] 83xx: Fix some bugs in spd sdram code

2009-03-09 Thread Joakim Tjernlund
> > On Wed, 25 Feb 2009 10:30:26 +0100 > Joakim Tjernlund wrote: > > > > > 1, 2 and 5 > > > > Acked-by: Joakim Tjernlund > > > > > > > > Don't understand cpo and haven't looked at 3 so > > > > I can't s

Re: [U-Boot] [PATCH] 83xx: Fix some bugs in spd sdram code

2009-02-25 Thread Joakim Tjernlund
. cpo > > > 5. make the AL to min to gain better performance. > > > > > > The Micron MT9HTF6472CHY-667D1 DIMMs test passed on > > > MPC837xEMDS platform at 266MHz/333MHz/400MHz data rate. > > > > > > Reported-by: Joakim Tjernlund > > >

Re: [U-Boot] [PATCH] 83xx: Fix some bugs in spd sdram code

2009-02-25 Thread Joakim Tjernlund
ance. > > The Micron MT9HTF6472CHY-667D1 DIMMs test passed on > MPC837xEMDS platform at 266MHz/333MHz/400MHz data rate. > > Reported-by: Joakim Tjernlund > Signed-off-by: Dave Liu 1, 2 and 5 Acked-by: Joakim Tjernlund Don't understand cpo and haven't lo

Re: [U-Boot] [PATCH] fsl-ddr: Fix some bugs in the ddr infrastructure

2009-02-25 Thread Joakim Tjernlund
>add the check the min requirement for tWTR. > > Reported-by: Joakim Tjernlund > Signed-off-by: Dave Liu Acked-by: Joakim Tjernlund ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] spd_dram.c for mpc83xx broken

2009-02-20 Thread Joakim Tjernlund
> > On Tue, 17 Feb 2009 20:25:03 +0100 > Joakim Tjernlund wrote: > > > The spd code for mpc83xx is so broken it isn't funny. This is what I had > > to do to get my > > Micron MT47H64M16-3. This isn't against current u-boot, but it is a start. > >

Re: [U-Boot] spd_dram.c for mpc83xx broken

2009-02-18 Thread Joakim Tjernlund
Kim Phillips wrote on 18/02/2009 02:35:50: > > On Tue, 17 Feb 2009 20:25:03 +0100 > Joakim Tjernlund wrote: > > > The spd code for mpc83xx is so broken it isn't funny. This is what I had > > to do to get my > > Micron MT47H64M16-3. This isn't ag

[U-Boot] spd_dram.c for mpc83xx broken

2009-02-17 Thread Joakim Tjernlund
The spd code for mpc83xx is so broken it isn't funny. This is what I had to do to get my Micron MT47H64M16-3. This isn't against current u-boot, but it is a start. Jocke PS. Sorry for the attachment, trying to use Lotus Notes as my new E-mail client and I can't find how to insert a f

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
> Hello Joakim, > > Joakim Tjernlund wrote: > > Heiko Schocher wrote on 28/01/2009 11:54:22: > >> Joakim Tjernlund wrote: > >>>> This patch adds the possibility to call a board specific > >>>> i2c bus reset routine for the fsl_i2c bus drive

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
Heiko Schocher wrote on 28/01/2009 11:54:22: > Hello Joakim, > > Joakim Tjernlund wrote: > >> This patch adds the possibility to call a board specific > >> i2c bus reset routine for the fsl_i2c bus driver, and adds > >> this option for the keymile kmeter

Re: [U-Boot] [PATCH 19/31] i2c, mpc83xx: add CONFIG_SYS_I2C_INIT_BOARD for fsl_i2c

2009-01-28 Thread Joakim Tjernlund
> > This patch adds the possibility to call a board specific > i2c bus reset routine for the fsl_i2c bus driver, and adds > this option for the keymile kmeter1 board. > > Signed-off-by: Heiko Schocher > --- > board/keymile/common/common.c | 14 ++ > drivers/i2c/fsl_i2c.c |

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

2008-12-25 Thread Joakim Tjernlund
> -Original Message- > From: Shinya Kuribayashi [mailto:skuri...@ruby.dti.ne.jp] > Sent: den 24 december 2008 17:55 > To: Joakim Tjernlund > Cc: 'Shinya Kuribayashi'; 'Graeme Russ'; u-boot@lists.denx.de > Subject: Re: [U-Boot] [RFC][PATCH] Code

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

2008-12-24 Thread Joakim Tjernlund
> > > - There is no purely weak functions and therfore no longer code like: > > if (do_something) > > do_somthing(); > > All instances have been replaced by empty functions with an alias. e.g. > > void __do_something (args) {} > > do_something(args) __atttribute__((weak, alia

Re: [U-Boot] [PATCH V4] cmd_bdinfo: move implementation to arch instead of common

2008-12-04 Thread Joakim Tjernlund
On Thu, 2008-12-04 at 13:35 +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > introduce 3 new weak functions board_bdinfo, cpu_bdinfo and soc_bdinfo to > allow > board, cpu and soc to print more information > > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]> > --- > diff with V

Re: [U-Boot] weak functions versus conditional compile

2008-11-19 Thread Joakim Tjernlund
On Wed, 2008-11-19 at 14:54 +0100, Haavard Skinnemoen wrote: > Mike Frysinger <[EMAIL PROTECTED]> wrote: > > On Monday 17 November 2008 16:17:54 Graeme Russ wrote: > > > Should I declare these functions as weak in the core i386 code and use a > > > config #define to override or should I seperate th

Re: [U-Boot] [PATCH/RFC] Some speed improvements to U-Boot JFFS2 code

2008-11-11 Thread Joakim Tjernlund
On Mon, 2008-11-10 at 22:49 -0800, Alexey Neyman wrote: > Michael, > > I tried to reproduce the bug you mentioned (multiple appearance of > directories in ls output) using the script you provided, to no avail: > ls correctly showed the top-level directory, both with and without > CFG_JFFS2_SORT

Re: [U-Boot] [PATCH/RFC] Some speed improvements to U-Boot JFFS2 code

2008-11-04 Thread Joakim Tjernlund
On Tue, 2008-11-04 at 13:31 +0100, Michael Lawnick wrote: > Michael Lawnick said the following: > ... > > - output of access rights is still corrupted, e.g.: > > d-ÿÿ-ÿ0 Tue Oct 21 11:08:17 2008 10 > > --ÿÿ-ÿ 262144 Thu Jan 01 00:00:33 1970 u-boot.bin > > d-ÿÿ-ÿ0 Tu

Re: [U-Boot] [PATCH] Handle environment variables like commands

2008-10-15 Thread Joakim Tjernlund
On Wed, 2008-10-15 at 12:57 +0200, Wolfgang Denk wrote: > Dear Stefano, > > In message <[EMAIL PROTECTED]> you wrote: > > The command interpreter checks always if an environment variable > > for that name exists and in this case the content of the variable > > is executed. It becomes possible to r

Re: [U-Boot] [PATCH RFC] mpc8572ds relocatable

2008-10-14 Thread Joakim Tjernlund
On Mon, 2008-10-13 at 20:59 -0700, Swarthout Edward L wrote: > From: Joakim Tjernlund [mailto:[EMAIL PROTECTED] > > > > The __eabi_uconvert() function skips NULL ptrs. > > > > Perhaps this is the missing piece needed in start.S for PPC? > > I'm not s

Re: [U-Boot] [PATCH RFC] mpc8572ds relocatable

2008-10-13 Thread Joakim Tjernlund
On Mon, 2008-10-13 at 00:53 +0200, Wolfgang Denk wrote: > Dear Ed Swarthout, > > In message <[EMAIL PROTECTED]> you wrote: > > Fixes boot crash from bad string pointers in get_table_entry_name > > when flash is erased or differs from current u-boot image. > > > > Signed-off-by: Ed Swarthout <[EMA

Re: [U-Boot] JFFS2-Summary support for u-boot

2008-09-29 Thread Joakim Tjernlund
On Mon, 2008-09-29 at 12:49 +, Michael Lawnick wrote: > Joakim Tjernlund transmode.se> writes: > > > > Has anybody tried to (and succeeded in) speed up u-boot? > > > > Yes, there was an attempt some time ago(months or a year, dunno) where > > someon

Re: [U-Boot] JFFS2-Summary support for u-boot

2008-09-29 Thread Joakim Tjernlund
On Mon, 2008-09-29 at 11:34 +, Michael Lawnick wrote: > Joakim Tjernlund transmode.se> writes: > > > > > On Mon, 2008-09-29 at 08:45 +, Michael Lawnick wrote: > > > Hi *, > > > > > > please feel free to correct me if I tell sh**: > &

Re: [U-Boot] JFFS2-Summary support for u-boot

2008-09-29 Thread Joakim Tjernlund
On Mon, 2008-09-29 at 08:45 +, Michael Lawnick wrote: > Hi *, > > please feel free to correct me if I tell sh**: > As far as I was told, bad JFFS2 boot performance of u-boot is result of not > supporting summary. (Because there is no JFFS2 but only JFFS support?) It is not missing SUMMARY, it

Re: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_char from a..." breaks imx31_phycore

2008-09-05 Thread Joakim Tjernlund
> -Original Message- > From: Magnus Lilja [mailto:[EMAIL PROTECTED] > Sent: den 5 september 2008 21:16 > To: Joakim Tjernlund > Cc: Guennadi Liakhovetski; U-Boot@lists.denx.de > Subject: Re: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_ch

Re: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_char from a..." breaks imx31_phycore

2008-09-05 Thread Joakim Tjernlund
> -Original Message- > From: Guennadi Liakhovetski [mailto:[EMAIL PROTECTED] > Sent: den 5 september 2008 21:26 > To: Joakim Tjernlund > Cc: U-Boot@lists.denx.de; Jean-Christophe PLAGNIOL-VILLARD; Remy Bohmer > Subject: RE: [REGRESSION] commit b502611b51... "Change

Re: [U-Boot] [REGRESSION] commit b502611b51... "Change env_get_char from a..." breaks imx31_phycore

2008-09-05 Thread Joakim Tjernlund
> -Original Message- > From: Guennadi Liakhovetski [mailto:[EMAIL PROTECTED] > Sent: den 5 september 2008 20:01 > To: U-Boot@lists.denx.de > Cc: Joakim Tjernlund > Subject: [REGRESSION] commit b502611b51... "Change env_get_char from a..." > breaks

<    3   4   5   6   7   8