Re: [U-Boot-Users] help

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> I am working with u =96 boot integration of ARM based board for various tes=
> ts
> 
> I am able to add the different commands but unable to run them
> simultaneously

It is considerd highly impolite to post the same question several
times in short sequence, especially when you additionally change the
sander ID's. Please don't do this.

> Content-Type: text/html; charset=WINDOWS-1252
> Content-Transfer-Encoding: quoted-printable
> Content-Disposition: inline

And NEVER post HTML to this list!

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
You are in a maze of UUCP connections, all alike.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] "huge" (6 second) delay when booting linux kernel (coldfire)

[w . wegner]

Hi,

On 4 Apr 2008 at 8:13, [EMAIL PROTECTED] wrote:

> Hi,
> 
> I have a problem on startup on my Coldfire MCF5373L board.
> 
> When the flash is empty, something before booting the linux kernel
> takes too long such that the watchdog resets my board before there
> is a chance for the kernel to take over control.

with some more puts() I found there is around 4 second delay in this
line of do_bootm_linux() in lib_m68k/m68k_linux.c:

cmd_end = cmd_start + strlen(cmdline);

My environment looks like this:

URMEL > printenv
bootargs= console=ttyS2,115200 rootfstype=romfs loaderversion=$loaderversion
bootcmd=protect off 0x8 0x1ff;run env_check;run xilinxload&&run alteralo
ad&&bootm 0x8;update;reset
bootdelay=1
baudrate=115200
ethaddr=00:00:00:00:00:09
ipaddr=192.168.100.2
serverip=192.168.100.1
loaderversion=10
alterafile=0
xilinxfile=0
xilinxload=imxtract 0x54 $xilinxfile 0x4100&&fpga load 0 0x4100 
$filesize
alteraload=imxtract 0x6c $alterafile 0x4100&&fpga load 1 0x4100 
$filesize
env_check=if test $env_default -eq 1; then setenv env_default 0;saveenv;fi
ethact=FEC0
env_default=0
stdin=serial
stdout=serial
stderr=serial

Environment size: 643/32764 bytes

At first glance I can not see an obvioius reason what goes wrong with the
cmdline...

Regards,
Wolfgang


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] "huge" (6 second) delay when booting linux kernel (coldfire)

[w . wegner]

Hi,

I have a problem on startup on my Coldfire MCF5373L board.

When the flash is empty, something before booting the linux kernel
takes too long such that the watchdog resets my board before there
is a chance for the kernel to take over control.

This happens only if the (32MB) flash is empty, if there is a
filesystem (JFFS2) in it, the delay is much shorter.

Everything is fine until and including uncompressing the kernel.
I added an entry in lib_m68k/m68k_linux.c to see if the delay
happens in U-Boot or in the kernel:

 puts("Boot\n");
(*kernel) (kbd, initrd_start, initrd_end, cmd_start, cmd_end);

When booting without the watchdog, I get this (ttyS2 is the
console port):

   Uncompressing Kernel Image ... OK
BootðttyS2 at 0xfc068000 (irq = 92) is a builtin ColdFire UART

Between printing the "OK" from uncompressing and "Boot" is
where the delay happens.

Can anybody give a hint where this delay comes from?

Thank you,
Wolfgang


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] help

[bankupalli nagendra]

 Hi all



I am working with u – boot integration of ARM based board for various tests

I am able to add the different commands but unable to run them
simultaneously

The last command which  I added is only able to run. Remaining commands are
just getting hanged

Can you please suggest the way to debug this ?



Regards,

Nagendra

TES Electronics Solutions

Bangalore
-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] unable to run the multiple commands

[Nagendra]

Hi all
 
I am working with u - boot integration of ARM based board for various
tests 
I am able to add the different commands but unable to run them
simultaneously 
The last command which  I added is only able to run. Remaining commands
are just getting hanged
Can you please suggest the way to debug this ?
 
Regards,
Nagendra
TES Electronics Solutions
Bangalore
 
-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Josh Boyer]

On Fri, 2008-04-04 at 00:08 +0200, Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
> >
> > > Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes 
> > > accepted now.
> > > Topic for #u-boot set by wdenk at 2008-04-01 10:27:53
> > >
> > >   
> > OK, where do you announce these things, so I can be sure to notice next 
> > time?  I'm not trying to be a smart-ass, but am really curious.
> 
> Well, the quote above refers to the subject on the "u-boot" IRC
> channel.

Which would be #u-boot on Freenode (irc.freenode.net) for those that
don't know the channel even exists.

josh


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Build u-boot for at91sam9260ek

[Jean-Christophe PLAGNIOL-VILLARD]

On 20:40 Thu 03 Apr , Norman wrote:
> I have downloaded the 1.3.2 source and wonder where can I locate the latest
> stable patches for this platform? I saw an AT91 fork in the source tree, would
> that be a better starting point.
> 
> Thanks,
> Norman

Take a look to the AT91 repository

I'll will add the last Stelian's patch in to the testing branch

Best Regards,
J.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] FDT: missing stdout-path

[Jerry Van Baren]

Scott Wood wrote:
> On Thu, Apr 03, 2008 at 11:56:08AM -0400, Bill Cook wrote:
>> Hello experts,
>>
>> With U-Boot-1.3.2 and a Freescale MPC8313ERDB,
>> I'm getting this message from bootm
>>
>> ...
>>Booting using the fdt at 0xfc70
>>Loading Ramdisk to 07ca1000, end 07f419bf ... OK
>>Loading Device Tree to 007fc000, end 007fefff ... OK
>>WARNING: could not set linux,stdout-path FDT_ERR_NOTFOUND.
>>ERROR: /chosen node create failed - must RESET the board to recover.
>>Resetting the board.
>>
>>U-Boot 1.3.2 (Apr  2 2008 - 10:02:55) MPC83XX
>>
>>Reset Status: Software Hard, External/Internal Soft, External/Internal
>> Hard
>>
>>CPU:   e300c3, MPC8313E, Rev: 10 at 333.333 MHz, CSB:  166 MHz
>> ...
>>
>> What do I need to add to DTS file or u-boot environment to make this go?
> 
> You probably need serial0, serial1, etc. aliases.  See the device trees
> in head-of-tree Linux.
> 
> -Scott

...and that error message could stand improvement to give the poor user 
a better clue that he is missing the /aliases node or /aliases/serial0 
property.

The current error message reminds me of the "Fog in Seattle" joke
   

gvb

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] newcomer

[Jerry Van Baren]

 wrote:
> hi experters:
>  
>I'm a newcomer in U-boot, I want to understand the U-boot,but i don't 
> know how to start!Anyone can help me ?
> Thanks,
>Peter wang

Hi Peter,

Start by looking in the u-boot code/documentation to see if your 
processor is supported.  Look through boards using that processor to get 
a feel for what needs to be customized for a port.

See the DULG manual:
   

Browse the source:
   

Read the README:


(especially starting at line 3732 ;-)


Then come back and ask specific questions and we will give you more 
specific answers.

Best regards,
gvb

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Build u-boot for at91sam9260ek

[Norman]

I have downloaded the 1.3.2 source and wonder where can I locate the latest
stable patches for this platform? I saw an AT91 fork in the source tree, would
that be a better starting point.

Thanks,
Norman


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> > Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes accepted 
> > now.
> > Topic for #u-boot set by wdenk at 2008-04-01 10:27:53
> >
> >   
> OK, where do you announce these things, so I can be sure to notice next 
> time?  I'm not trying to be a smart-ass, but am really curious.

Well, the quote above refers to the subject on the "u-boot" IRC
channel.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
The IQ of the group is the lowest IQ of a member of the group divided
by the number of people in the group.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Fix host tool build breakage, take two

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> > Even simple tungs like a printf() require that you include the correct
> > host header files as you will link your host application against the
> > host libraries, too.
> 
> That's because printf() is from the host's libraries.  lib_generic/md5.c
> is not a host-provided library, so you don't want to use a host-provided
> header.

Correct, and that's whY I agree that renaming it to make name
collisions at least unlikely is a good thing.

> It is much more important to be robust against random crap in
> /usr/include (something u-boot can't control) than changes in the u-boot
> tree itself.

Agreed, too.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
Give a man a fish, and you feed him for a day. Teach a man  to  fish,
and he'll invite himself over for dinner. - Calvin Keegan

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] cfi_flash: Support buffered writes on non-standard Spansion NOR flash

[Jean-Christophe PLAGNIOL-VILLARD]

On 13:36 Thu 03 Apr , Guennadi Liakhovetski wrote:
> Some NOR flash chip from Spansion, for example, the s29ws-n MirrorBit 
> series require different addresses for buffered write commands. Define a 
> configuration option to support buffered writes on those chips. A more 
> elegant solution would be to automatically detect those chips by parsing 
> their CFI records, but that would require introduction of a fixup table 
> into the cfi_flash driver.
> 

It will be good to detect it. On some hardware you may have some pip out
compatible flash with this future or an other buffer interface.

We are in fix window, if possible you maybe can provide this support for
the next merge window

but thanks


Best Regards,
J.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [RFC] Rename include/md5.h to u-boot-md5.h

[Timur Tabi]

Andy Fleming wrote:
> Some systems have md5.h installed in /usr/include/.  This isn't the desired
> file (we want the one in include/md5.h).  This will avoid the conflict.
> This fixes the host tools building problem
> 
> Signed-off-by: Andy Fleming <[EMAIL PROTECTED]>

Acked-by: Timur Tabi <[EMAIL PROTECTED]>

This fixes a build break for me.  Please apply.

-- 
Timur Tabi
Linux kernel developer at Freescale

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] IDE: fix bug in reset sequence

[Jean-Christophe PLAGNIOL-VILLARD]

On 13:37 Thu 03 Apr , Martin Krause wrote:
> According to the ata (ata5) specification the RESET- signal
> shall be asserted for at least 25 us. Without this patch,
> the RESET- signal is asserted on some boards for only < 1 us
> (e. g. on the TQM5200). This patch adds a general delay of
> 25 us to the RESET- signal.
> 
> Without this patch a Platinum 4 GiB CF card is not recognised
> properly on boards with a TQM5200 (STK52xx, TB5200).
> 
> Signed-off-by: Martin Krause <[EMAIL PROTECTED]>
> ---
> 
>  common/cmd_ide.c |3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> 
> diff --git a/common/cmd_ide.c b/common/cmd_ide.c
> index 8ace970..f9cd422 100644
> --- a/common/cmd_ide.c
> +++ b/common/cmd_ide.c
> @@ -1529,6 +1529,9 @@ static void ide_reset (void)
>  
>   ide_set_reset (1); /* assert reset */
>  
> + /* the reset signal shall be asserted for et least 25 us */
  ^
  for at least ;-)

btw this need to be tested with other CF ex <>8GB and some hard drive

Best Regards,
J.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Pointer to similar board?

[PatM]

That was it! Went back to 1.2.0, flashed my NOR and upon reboot got the
uboot menu.

Thanks very much! Being new to this stuff I always assume its me doing
something wrong and spend a few days wracking my brain and overheating
the google servers before actually asking a question. I learn a lot but
its hard on the hairline 8)

Pat Meloy
Canuckleland

On Thu, 2008-04-03 at 20:20 +0200, Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
> >
> > > I changed the boot delay in configs/smdk2410.h to -1 to make sure I
> > > could get the right port (swapping the cable around in case it wasn't
> > > uart 0 being used. I also tried 60 delay. Still nothing.
> 
> I am not really sure if the smdk2410 port is working at all at the
> moment.
> 
> > It was some time ago, I last used the SMDK2410. But I'm pretty sure,
> > that it prints some boot messages. You should see this messages regardles
> > if you boot from flash, ethernet, or whatever.
> 
> I can only speak for the s3c2400 (trab, and most probably also
> smdk2400), which were broken by this commit:
> 
> commit 80767a6cead9990d9e77e62be947843c2c72f469
> Author: Peter Pearse <[EMAIL PROTECTED]>
> Date:   Wed Sep 5 16:04:41 2007 +0100
> 
> Changed API name to coloured_led.h
> Removed code using deprecated ifdef CONFIG_BOOTBINFUNC
> Tidied other cpu/arm920t/start.S code
> 
> You may want to try the commit previous to this one...
> 
> Best regards,
> 
> Wolfgang Denk
> 


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Fix host tool build breakage, take two

[Andy Fleming]

On Thu, Apr 3, 2008 at 1:36 PM, Scott Wood <[EMAIL PROTECTED]> wrote:
> On Thu, Apr 03, 2008 at 08:26:23PM +0200, Wolfgang Denk wrote:
>  > In message <[EMAIL PROTECTED]> you wrote:
>  > >
>  > > I think this makes sense for code that we for example link from host's
>  > > standard libraries. But for code compiled from files from the U-Boot
>  > > tree (like lib_generic/md5.c), we shouldn't include host's header files.
>  >
>  > But you, you should. Note that we are talking about host tools  here,
>  > i.  e.  stuff that is supposed to run in the host environment. Assume
>  > your're building for a big-endian, 32 bit Power system.  Assume  your
>  > host is a little-endian x86_64 system.
>  >
>  > Even simple tungs like a printf() require that you include the correct
>  > host header files as you will link your host application against the
>  > host libraries, too.
>
>  That's because printf() is from the host's libraries.  lib_generic/md5.c
>  is not a host-provided library, so you don't want to use a host-provided
>  header.

I agree with this.  Ideally, we should search system headers when we
want them, and u-boot headers when we want *them*.  However, my
Make-fu is not strong enough to figure that one out.


>
>  It is much more important to be robust against random crap in
>  /usr/include (something u-boot can't control) than changes in the u-boot
>  tree itself.

Well, IMNSHO, renaming the internal header to u-boot-md5.h solves the
problem with a high degree of robustness (random non-u-boot packages
are unlikely to install a header with u-boot in the name), and avoids
the dangers of relative path lookups.  It so happens I already sent a
patch to do that...  :)

That said, if we could figure out how to rejigger the Makefile so that
it found u-boot's md5.h first, that would be better.  I suspect that
involves a similar patch, though.  These are the header categories:

1) U-Boot headers only meant for target compilation
2) System headers
3) U-Boot headers meant for host compilation (probably for both host and target)

When we're building for the target, we just don't search the system.
But for host compilation, we want to exclude #1.  To do that, we'd
have to identify every header which should never be compiled on the
host, and move it into another directory which won't get searched when
we're using HOST_CC.

Personally, I suspect category 3 is much smaller than category 1, so
it's probably easier to uniquify the smaller category somehow.  We
could, for instance, also just identify those files and move them into
include/generic/ or somesuch.

Andy

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Guennadi Liakhovetski wrote:
> On Thu, 3 Apr 2008, Ben Warren wrote:
>
>   
>> Wolfgang Denk wrote:
>> 
>>> Oh, there has been an announcement:
>>>
>>> Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes accepted 
>>> now.
>>> Topic for #u-boot set by wdenk at 2008-04-01 10:27:53
>>>   
>
> ^^^
>
>   
>> OK, where do you announce these things, so I can be sure to notice next 
>> time?  I'm not trying to be a smart-ass, but am really curious.
>> 
>
> i.e. "#u-boot" IRC channel on irc.freenode.net.
>   
Oh, I didn't know there was a U-boot IRC channel.  That will surely make 
me more productive!

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Guennadi Liakhovetski]

On Thu, 3 Apr 2008, Ben Warren wrote:

> Wolfgang Denk wrote:
> >
> > Oh, there has been an announcement:
> >
> > Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes accepted 
> > now.
> > Topic for #u-boot set by wdenk at 2008-04-01 10:27:53

  ^^^

> OK, where do you announce these things, so I can be sure to notice next 
> time?  I'm not trying to be a smart-ass, but am really curious.

i.e. "#u-boot" IRC channel on irc.freenode.net.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [EMAIL PROTECTED]

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
>   
>> I know the timetable says the merge window is closed, but there hasn't 
>> been any type of announcement and history tells me that there's probably 
>> 
>
> Oh, there has been an announcement:
>
> Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes accepted 
> now.
> Topic for #u-boot set by wdenk at 2008-04-01 10:27:53
>
>   
OK, where do you announce these things, so I can be sure to notice next 
time?  I'm not trying to be a smart-ass, but am really curious.
> And we don't need formal announcements, or do we? We all know that fe
> follow the deadlines  of  our  timetable  with  the  precision  of  a
> Rolex(*).
>
>
> (*) one of those you get on ebay really cheap, that is :-)
>   
Who needs eBay when I get all these wonderful e-mails?  And it's not 
only watches!

cheers,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
>   
>>> * Topic for #u-boot is: Merge Window for 1.3.3 is over - only bug fixes 
>>> accepted now.
>>> * Topic for #u-boot set by wdenk at Tue Apr  1 10:27:53 2008
>>>   
>>>   
>> OK then.  Next revision it is.
>> 
>
> Nope. We're not bureaucrats. We do what technically makes sense. And
> to me it makes much sense to see this go in now. Thanks in advance.
>
>   
Works for me.  I'll make it happen.

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Fix host tool build breakage, take two

[Scott Wood]

On Thu, Apr 03, 2008 at 08:26:23PM +0200, Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
> >
> > I think this makes sense for code that we for example link from host's
> > standard libraries. But for code compiled from files from the U-Boot
> > tree (like lib_generic/md5.c), we shouldn't include host's header files.
> 
> But you, you should. Note that we are talking about host tools  here,
> i.  e.  stuff that is supposed to run in the host environment. Assume
> your're building for a big-endian, 32 bit Power system.  Assume  your
> host is a little-endian x86_64 system.
> 
> Even simple tungs like a printf() require that you include the correct
> host header files as you will link your host application against the
> host libraries, too.

That's because printf() is from the host's libraries.  lib_generic/md5.c
is not a host-provided library, so you don't want to use a host-provided
header.

> > > 2) Use relative file names (as suggested by Kumar) or similar methods
> > >to make sure we pick up the md5.h header file from a local
> > >directory instead from /usr/include.
> > > 3) Rename md5.h to make sure we use a pick up our own file instead of
> > >some unrelated file which happens to have the same name.
> > 
> > 2) seems to be more robust, though -- 3) works until someone installs a
> 
> 2 is not robust. It fails easily as soon as you move or rename files,
> as son as you miss to create one  of  trhe  needed  directories  when
> building out of tree, etc.

It is much more important to be robust against random crap in
/usr/include (something u-boot can't control) than changes in the u-boot
tree itself.

-Scott

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> > * Topic for #u-boot is: Merge Window for 1.3.3 is over - only bug fixes 
> > accepted now.
> > * Topic for #u-boot set by wdenk at Tue Apr  1 10:27:53 2008
> >   
> OK then.  Next revision it is.

Nope. We're not bureaucrats. We do what technically makes sense. And
to me it makes much sense to see this go in now. Thanks in advance.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
"The X11 source code style is ATROCIOUS and should not be used  as  a
model."   - Doug Gwyn

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> I know the timetable says the merge window is closed, but there hasn't 
> been any type of announcement and history tells me that there's probably 

Oh, there has been an announcement:

Topic for #u-boot: Merge Window for 1.3.3 is over - only bug fixes accepted now.
Topic for #u-boot set by wdenk at 2008-04-01 10:27:53

And we don't need formal announcements, or do we? We all know that fe
follow the deadlines  of  our  timetable  with  the  precision  of  a
Rolex(*).


(*) one of those you get on ebay really cheap, that is :-)

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
"May your future be limited only by your dreams."
- Christa McAuliffe

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Fix host tool build breakage, take two

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> I think this makes sense for code that we for example link from host's
> standard libraries. But for code compiled from files from the U-Boot
> tree (like lib_generic/md5.c), we shouldn't include host's header files.

But you, you should. Note that we are talking about host tools  here,
i.  e.  stuff that is supposed to run in the host environment. Assume
your're building for a big-endian, 32 bit Power system.  Assume  your
host is a little-endian x86_64 system.

Even simple tungs like a printf() require that you include the correct
host header files as you will link your host application against the
host libraries, too.

> > 2) Use relative file names (as suggested by Kumar) or similar methods
> >to make sure we pick up the md5.h header file from a local
> >directory instead from /usr/include.
> > 3) Rename md5.h to make sure we use a pick up our own file instead of
> >some unrelated file which happens to have the same name.
> 
> 2) seems to be more robust, though -- 3) works until someone installs a

2 is not robust. It fails easily as soon as you move or rename files,
as son as you miss to create one  of  trhe  needed  directories  when
building out of tree, etc.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
I had the rare misfortune of being one of the first people to try and
implement a PL/1 compiler. -- T. Cheatham

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Pointer to similar board?

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
>
> > I changed the boot delay in configs/smdk2410.h to -1 to make sure I
> > could get the right port (swapping the cable around in case it wasn't
> > uart 0 being used. I also tried 60 delay. Still nothing.

I am not really sure if the smdk2410 port is working at all at the
moment.

> It was some time ago, I last used the SMDK2410. But I'm pretty sure,
> that it prints some boot messages. You should see this messages regardles
> if you boot from flash, ethernet, or whatever.

I can only speak for the s3c2400 (trab, and most probably also
smdk2400), which were broken by this commit:

commit 80767a6cead9990d9e77e62be947843c2c72f469
Author: Peter Pearse <[EMAIL PROTECTED]>
Date:   Wed Sep 5 16:04:41 2007 +0100

Changed API name to coloured_led.h
Removed code using deprecated ifdef CONFIG_BOOTBINFUNC
Tidied other cpu/arm920t/start.S code

You may want to try the commit previous to this one...

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
Don't panic.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
> 
> > Much better!  Thanks for doing all this work.  The only problem is that 
> > you're
> > providing it relative to another patch that hasn't been accepted.
> 
> Well, the merge window is closed, so, I cannot send in any more new 
> features. Whereas if we consider this as a fix - maybe it would be 
> possible?

Providing a different version of a patch is a fix of some kind.

Yes, "we" consider this as a fix.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
What is research but a blind date with knowledge?  -- Will Harvey

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] MPC8343 PCI setup

[Tor Krill]

On 4/3/2008, "Andre Schwarz" <[EMAIL PROTECTED]> wrote:

>All,

Hi Andre and others,

>currently I'm trying to bring up a mpc8343 based system.
>Latest u-boot v1.3.2 is running fine.
>
>The MPC8343 has a single 32-Bit PCI Bus running at 66MHz.
>Connected are a FPGA (IDSEL 11 + IRQ0) and a miniPCI Slot (IDSEL12 + 
>IRQ1).
>
>Unfortunately the kernel panics during/after PCI setup.

Not sure if this is related. But i got strange problems with PCI after
the patch "Add CONFIG_PCI_SKIP_HOST_BRIDGE config option"
(55774b512fdf63c..) was added.

My symptoms where missed interrupts on sata dma in Linux. U-boot where
however fine. Adding the above config solved my problems. Not really
sure why i experienced this. We have no PCI bridge on our board.

/Tor

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Guennadi Liakhovetski wrote:
> On Thu, 3 Apr 2008, Ben Warren wrote:
>
>   
>> I know the timetable says the merge window is closed, but there hasn't been
>> any type of announcement and history tells me that there's probably still
>> time.  I have some things I'd still like to get in if possible, but only WD
>> can say for sure, of course.
>> 
>
> He has:
>
> * Now talking on #u-boot
> * Topic for #u-boot is: Merge Window for 1.3.3 is over - only bug fixes 
> accepted now.
> * Topic for #u-boot set by wdenk at Tue Apr  1 10:27:53 2008
>   
OK then.  Next revision it is.

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Guennadi Liakhovetski]

On Thu, 3 Apr 2008, Ben Warren wrote:

> I know the timetable says the merge window is closed, but there hasn't been
> any type of announcement and history tells me that there's probably still
> time.  I have some things I'd still like to get in if possible, but only WD
> can say for sure, of course.

He has:

* Now talking on #u-boot
* Topic for #u-boot is: Merge Window for 1.3.3 is over - only bug fixes 
accepted now.
* Topic for #u-boot set by wdenk at Tue Apr  1 10:27:53 2008

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [EMAIL PROTECTED]

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Guennadi Liakhovetski wrote:
> On Thu, 3 Apr 2008, Ben Warren wrote:
>
>   
>> Guennadi Liakhovetski wrote:
>> 
>>> Replace direct register address derefencing with accessor functions.
>>> Restrict explicitly 32-bit bus-width, extend affected configurations
>>> respectively.
>>>
>>>   
>>>   
>> Much better!  Thanks for doing all this work.  The only problem is that 
>> you're
>> providing it relative to another patch that hasn't been accepted.
>> 
>
> Well, the merge window is closed, so, I cannot send in any more new 
> features. Whereas if we consider this as a fix - maybe it would be 
> possible?
>
>   
I know the timetable says the merge window is closed, but there hasn't 
been any type of announcement and history tells me that there's probably 
still time.  I have some things I'd still like to get in if possible, 
but only WD can say for sure, of course.

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] MPC8343 PCI setup

[Andre Schwarz]

All,

currently I'm trying to bring up a mpc8343 based system.
Latest u-boot v1.3.2 is running fine.

The MPC8343 has a single 32-Bit PCI Bus running at 66MHz.
Connected are a FPGA (IDSEL 11 + IRQ0) and a miniPCI Slot (IDSEL12 + 
IRQ1).

Unfortunately the kernel panics during/after PCI setup.

As always I expect my device tree to be wrong or at least is missing 
something.
I startet with "mpc834x_mds.dts", removed the 2nd PCI and switched to 
"PowerPC,[EMAIL PROTECTED]".



u-boot shows this pci node :


[EMAIL PROTECTED] {
cell-index = <0x0001>;
interrupt-map-mask = [00 00 f8 00 00 00 00 00 00 00 00 
00 00 00 00 07];
interrupt-map = [00 00 58 00 00 00 00 00 00 00 00 00 00 
00 00 01 00 00 00 01 00 00 00 30 00 00 00 08 00 00
60 00 00 00 00 00 00 00 00 00 00 00 00 01 00 00 00 01 00 00 00 11 00 00 
00 08];
interrupt-parent = <0x0001>;
interrupts = <0x0042 0x0008>;
bus-range = <0x 0x>;
ranges = [02 00 00 00 00 00 00 00 90 00 00 00 90 00 00 
00 00 00 00 00 10 00 00 00 42 00 00 00 00 00 00 00 8
0 00 00 00 80 00 00 00 00 00 00 00 10 00 00 00 01 00 00 00 00 00 00 00 
00 00 00 00 e2 00 00 00 00 00 00 00 00 10 00 00];
clock-frequency = <0x03f940aa>;
#interrupt-cells = <0x0001>;
#size-cells = <0x0002>;
#address-cells = <0x0003>;
reg = <0xe0008500 0x0100>;
compatible = "fsl,mpc8349-pci";
device_type = "pci";
};




Booting the kernel :


Using mvBlueLYNX-M7 machine description
Linux version 2.6.25-rc7-01000-ga1ba6f0-dirty ([EMAIL PROTECTED]) (gcc version 
4.0.0 (DENX ELDK 4.1 4.0.0)) #23 PREEMPT Thu Apr 3 1
6:46:17 CEST 2008
Found initrd at 0xc3978000:0xc3c0
console [udbg0] enabled
setup_arch: bootmem
mvblm7_setup_arch()
Adding PCI host bridge /[EMAIL PROTECTED]
Found MPC83xx PCI host bridge at 0xe0008500. Firmware bus 
number: 0->0
 ->Hose at 0xc02bf000, cfg_addr=0xfdffd300,cfg_data=0xfdffd304
PCI host bridge /[EMAIL PROTECTED] (primary) ranges:
 MEM 0x9000..0x9fff -> 0x9000
 MEM 0x8000..0x8fff -> 0x8000 Prefetch
  IO 0xe200..0xe20f -> 0x
Unable to handle kernel paging request for data at address 0x



Call trace gives address last called function "of_device_is_compatible".

Any help is welcome !


regards,
Andre Schwarz
Matrix Vision



MATRIX VISION GmbH, Talstraße 16, DE-71570 Oppenweiler  - Registergericht: 
Amtsgericht Stuttgart, HRB 271090
Geschäftsführer: Gerhard Thullner, Werner Armingeon, Uwe Furtner

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Fix host tool build breakage, take two

[Bartlomiej Sieka]

Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
>> I think that generally it is a better idea to use U-Boot's includes when
>> building for the host system, as this gives us better control over what
>> exactly gets included. But then on the other hand, tools/Makefils has this:
> 
> But U-boot doesn't have the knowledge about the target system that the
> target distribution has.
> 
>> CPPFLAGS   = -idirafter $(SRCTREE)/include \
>>  -idirafter $(OBJTREE)/include2 \
>>  -idirafter $(OBJTREE)/include \
>>
>> Could anyone comment on the reasons why we try U-Boot's includes after
>> system includes? Perhaps it would be a good idea to reverse the order --
>> see below for a quick RFC patch (compile-tested on two arm and two ppc
>> boards, each arch with and without CONFIG_FIT enabled).
> 
> Don't! I guess you tried just one configuration, and probably not even
> building on a 32 versus a 64 bit host system.
> 
> When building tools with the host compiler that are supposed  to  run
> on  the host system we should always use the host's header files.

I think this makes sense for code that we for example link from host's
standard libraries. But for code compiled from files from the U-Boot
tree (like lib_generic/md5.c), we shouldn't include host's header files.

> 
> The current problem comes from the fact  that  old  versions  of  the
> cyrus-sasl-devel  package  install  a  /usr/include/md5.h  file which
> probably NOT intended for direct use, but only within the context  of
> the  SASL  package; recent versions of the same package install it in
> /usr/include/sasl/md5.h instead.
> 
> To solve this problem I see three solutions:
> 
> 1) Fix the broken host systems for example by installing more recent
>versions of the cyrus-sasl-devel package; this would be best, but
>is unfortunately not possible everywhere.
> 2) Use relative file names (as suggested by Kumar) or similar methods
>to make sure we pick up the md5.h header file from a local
>directory instead from /usr/include.
> 3) Rename md5.h to make sure we use a pick up our own file instead of
>some unrelated file which happens to have the same name.
> 
> My vote goes for 3).

2) seems to be more robust, though -- 3) works until someone installs a
system header file which happens to have the name that we came up with
for the U-Boot's own header file. Also, we used 2) for libfdt compiled
for mkimage.

Regards,
Bartlomiej

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] FDT: missing stdout-path

[Scott Wood]

On Thu, Apr 03, 2008 at 11:56:08AM -0400, Bill Cook wrote:
> Hello experts,
> 
> With U-Boot-1.3.2 and a Freescale MPC8313ERDB,
> I'm getting this message from bootm
> 
> ...
>Booting using the fdt at 0xfc70
>Loading Ramdisk to 07ca1000, end 07f419bf ... OK
>Loading Device Tree to 007fc000, end 007fefff ... OK
>WARNING: could not set linux,stdout-path FDT_ERR_NOTFOUND.
>ERROR: /chosen node create failed - must RESET the board to recover.
>Resetting the board.
> 
>U-Boot 1.3.2 (Apr  2 2008 - 10:02:55) MPC83XX
> 
>Reset Status: Software Hard, External/Internal Soft, External/Internal
> Hard
> 
>CPU:   e300c3, MPC8313E, Rev: 10 at 333.333 MHz, CSB:  166 MHz
> ...
> 
> What do I need to add to DTS file or u-boot environment to make this go?

You probably need serial0, serial1, etc. aliases.  See the device trees
in head-of-tree Linux.

-Scott

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] FDT: missing stdout-path

[Bill Cook]

Hello experts,

With U-Boot-1.3.2 and a Freescale MPC8313ERDB,
I'm getting this message from bootm

...
   Booting using the fdt at 0xfc70
   Loading Ramdisk to 07ca1000, end 07f419bf ... OK
   Loading Device Tree to 007fc000, end 007fefff ... OK
   WARNING: could not set linux,stdout-path FDT_ERR_NOTFOUND.
   ERROR: /chosen node create failed - must RESET the board to recover.
   Resetting the board.

   U-Boot 1.3.2 (Apr  2 2008 - 10:02:55) MPC83XX

   Reset Status: Software Hard, External/Internal Soft, External/Internal
Hard

   CPU:   e300c3, MPC8313E, Rev: 10 at 333.333 MHz, CSB:  166 MHz
...

What do I need to add to DTS file or u-boot environment to make this go?


Thanks

Bill Cook

_
[EMAIL PROTECTED]

Imaging Solutions Group of NY
1387 Fairport Rd., Suite 890
Fairport, NY 14450
http://ISGcameras.com




-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Pointer to similar board?

[Martin Krause]

Hi,

PatM wrote on Thursday, April 03, 2008 5:01 PM:
> Oops, you caught me 8) Its the CS8900, not built in.
> 
> When I compile the kernel as a zImage (changing mach_types entry for
> SMDK2410 to my unit's ID) it has no problem talking to the serial port
> at ttySAC0. Using the SMDK2410 for u-boot I get no serial port
> activity at all, not even the LED that indicates the target's serial
> ports are in use.
> 
> I changed the boot delay in configs/smdk2410.h to -1 to make sure I
> could get the right port (swapping the cable around in case it wasn't
> uart 0 being used. I also tried 60 delay. Still nothing.

You should see some U-Boot boot messegas reagardles what boot delay
you configured. Maybe your clock configuration is wrong (wrong baudrate).
Or the SMDK2410 uses another serial interface as console as your board?
Or the SDRAM configuration is not suitable for your board. Or, ... there
are many things that could go wrong. Unfortunately thers is no quick
way to port U-Boot to a new hardware (see the "U-Boot Porting Guide"
section in the README in the U-Boot source tree:
http://www.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=blob;f=README;h=5cbe7c1e80daabeee8c43247db7041b19025;hb=HEAD
 :-)

> Perhaps I'm expecting the wrong result from u-boot? Is the SMDK2410
> default config supposed to give a prompt at all or go straight to DHCP
> et. al?

It was some time ago, I last used the SMDK2410. But I'm pretty sure,
that it prints some boot messages. You should see this messages regardles
if you boot from flash, ethernet, or whatever.

Please keep the list on cc.

Regards,
Martin

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH v2] net: make ARP timeout configurable

[Guennadi Liakhovetski]

Currently the timeout waiting for an ARP reply is hard set to 5 seconds. 
On i.MX31ADS due to a hardware "strangeness" up to four first IP packets 
to the boards get lost, which typically are ARP replies. By configuring 
the timeout to a lower value we significantly improve the first network 
transfer time on this board. The timeout is specified in milliseconds, 
later internally it is converted to deciseconds, because it has to be 
converted to hardware ticks, and CFG_HZ ranges from 900 to 2700 on 
different boards.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

diff --git a/README b/README
index a9663a3..9249064 100644
--- a/README
+++ b/README
@@ -1547,6 +1547,10 @@ The following options need to be configured:
before giving up the operation. If not defined, a
default value of 5 is used.
 
+   CONFIG_ARP_TIMEOUT
+
+   Timeout waiting for an ARP reply in milliseconds.
+
 - Command Interpreter:
CONFIG_AUTO_COMPLETE
 
diff --git a/net/net.c b/net/net.c
index 44feee2..2128bd4 100644
--- a/net/net.c
+++ b/net/net.c
@@ -94,11 +94,22 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#define ARP_TIMEOUT5UL /* Seconds before trying ARP 
again */
+#ifndefCONFIG_ARP_TIMEOUT
+# define ARP_TIMEOUT   50UL/* Deciseconds before trying ARP again 
*/
+#elif (CONFIG_ARP_TIMEOUT < 100)
+# error "Due to possible overflow CONFIG_ARP_TIMEOUT must be greater than 
100ms"
+#else
+# if (CONFIG_ARP_TIMEOUT % 100)
+#  warning "Supported ARP_TIMEOUT precision is 100ms"
+# endif
+# define ARP_TIMEOUT   (CONFIG_ARP_TIMEOUT / 100)
+#endif
+
+
 #ifndefCONFIG_NET_RETRY_COUNT
-# define ARP_TIMEOUT_COUNT 5   /* # of timeouts before giving 
up  */
+# define ARP_TIMEOUT_COUNT 5   /* # of timeouts before giving up  */
 #else
-# define ARP_TIMEOUT_COUNT  (CONFIG_NET_RETRY_COUNT)
+# define ARP_TIMEOUT_COUNT CONFIG_NET_RETRY_COUNT
 #endif
 
 #if 0
@@ -129,7 +140,7 @@ uchar   NetOurEther[6]; /* Our ethernet 
address */
 uchar  NetServerEther[6] = /* Boot server enet address 
*/
{ 0, 0, 0, 0, 0, 0 };
 IPaddr_t   NetOurIP;   /* Our IP addr (0 = unknown)
*/
-IPaddr_t   NetServerIP;/* Our IP addr (0 = unknown)
*/
+IPaddr_t   NetServerIP;/* Server IP addr (0 = unknown) 
*/
 volatile uchar *NetRxPkt;  /* Current receive packet   
*/
 intNetRxPktLen;/* Current rx packet length 
*/
 unsigned   NetIPID;/* IP packet ID 
*/
@@ -253,7 +264,7 @@ void ArpTimeoutCheck(void)
t = get_timer(0);
 
/* check for arp timeout */
-   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ) {
+   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ / 10) {
NetArpWaitTry++;
 
if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
@@ -494,7 +505,7 @@ restart:
 *  Check the ethernet for a new packet.  The ethernet
 *  receive routine will process it.
 */
-   eth_rx();
+   eth_rx();
 
/*
 *  Abort if ctrl-c was pressed.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH v2] MX31ADS network and flash updates

[Guennadi Liakhovetski]

This patch depends on the previous two patches, introducing the 
CONFIG_FLASH_SPANSION_S29WS_N and CONFIG_ARP_TIMEOUT configuration 
options. It allows U-Boot to use buffered writes to the Spansion NOR flash 
installed on this board, and eliminates long delays in network transfers 
after the board startup.

Also modify flash layout to embed main and redundant environment blocks in 
the U-Boot image.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

Changes since v1: now specifying CONFIG_ARP_TIMEOUT in milliseconds

diff --git a/board/mx31ads/mx31ads.c b/board/mx31ads/mx31ads.c
index 7c50c02..5a7d8c9 100644
--- a/board/mx31ads/mx31ads.c
+++ b/board/mx31ads/mx31ads.c
@@ -38,18 +38,18 @@ int dram_init (void)
 int board_init (void)
 {
int i;
-#if 0
+
/* CS0: Nor Flash */
/*
-* These are values from the RedBoot sources by Freescale. However,
-* under U-Boot with this configuration 32-bit accesses don't work,
-* lower 16 bits of data are read twice for each 32-bit read.
+* CS0L and CS0A values are from the RedBoot sources by Freescale
+* and are also equal to those used by Sascha Hauer for the Phytec
+* i.MX31 board. CS0U is just a slightly optimized hardware default:
+* the only non-zero field "Wait State Control" is set to half the
+* default value.
 */
-   __REG(CSCR_U(0)) = 0x23524E80;
-   __REG(CSCR_L(0)) = 0x1D03; /* WRAP bit (1) is suspicious here, but
-   * disabling it doesn't help either */
+   __REG(CSCR_U(0)) = 0x0f00;
+   __REG(CSCR_L(0)) = 0x1D03;
__REG(CSCR_A(0)) = 0x00720900;
-#endif
 
/* setup pins for UART1 */
mx31_gpio_mux(MUX_RXD1__UART1_RXD_MUX);
diff --git a/board/mx31ads/u-boot.lds b/board/mx31ads/u-boot.lds
index 1460adc..49713d4 100644
--- a/board/mx31ads/u-boot.lds
+++ b/board/mx31ads/u-boot.lds
@@ -34,7 +34,18 @@ SECTIONS
. = ALIGN(4);
.text  :
{
- cpu/arm1136/start.o   (.text)
+ /* WARNING - the following is hand-optimized to fit within*/
+ /* the sector layout of our flash chips!  XXX FIXME XXX   */
+
+ cpu/arm1136/start.o   (.text)
+ board/mx31ads/libmx31ads.a(.text)
+ lib_arm/libarm.a  (.text)
+ net/libnet.a  (.text)
+ drivers/mtd/libmtd.a  (.text)
+
+ . = DEFINED(env_offset) ? env_offset : .;
+ common/environment.o(.text)
+
  *(.text)
}
 
diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
index 77a9a83..9b72c89 100644
--- a/include/configs/mx31ads.h
+++ b/include/configs/mx31ads.h
@@ -72,7 +72,6 @@
 
 #include 
 
-#define CONFIG_CMD_MII
 #define CONFIG_CMD_PING
 
 #define CONFIG_BOOTDELAY   3
@@ -91,7 +90,19 @@
 
 #define CONFIG_DRIVER_CS8900   1
 #define CS8900_BASE0xb4020300
-#define CS8900_BUS16   1   /* the Linux driver does accesses as 
shorts */
+#define CS8900_BUS16   1   /* follow the Linux driver */
+
+/*
+ * The MX31ADS board seems to have a hardware "peculiarity" confirmed under
+ * U-Boot, RedBoot and Linux: the ethernet Rx signal is reaching the CS8900A
+ * controller inverted. The controller is capable of detecting and correcting
+ * this, but it needs 4 network packets for that. Which means, at startup, you
+ * will not receive answers to the first 4 packest, unless there have been some
+ * broadcasts on the network, or your board is on a hub. Reducing the ARP
+ * timeout from default 5 seconds to 200ms we speed up the initial TFTP
+ * transfer, should the user wish one, significantly.
+ */
+#define CONFIG_ARP_TIMEOUT 200UL
 
 /*
  * Miscellaneous configurable options
@@ -100,7 +111,7 @@
 #define CFG_PROMPT "=> "
 #define CFG_CBSIZE 256 /* Console I/O Buffer Size */
 /* Print Buffer Size */
-#define CFG_PBSIZE (CFG_CBSIZE+sizeof(CFG_PROMPT)+16)
+#define CFG_PBSIZE (CFG_CBSIZE + sizeof(CFG_PROMPT) + 16)
 #define CFG_MAXARGS16  /* max number of command args */
 #define CFG_BARGSIZE   CFG_CBSIZE  /* Boot Argument Buffer Size */
 
@@ -136,25 +147,29 @@
 #define CFG_MAX_FLASH_BANKS1   /* max number of memory banks */
 #define CFG_MAX_FLASH_SECT 262 /* max number of sectors on one 
chip */
 #define CFG_MONITOR_BASE   CFG_FLASH_BASE  /* Monitor at beginning of 
flash */
-#define CFG_MONITOR_LEN(128 * 1024)/* Reserve 128KiB */
+#define CFG_MONITOR_LEN(256 * 1024)/* Reserve 256KiB */
 
 #defineCFG_ENV_IS_IN_FLASH 1
 #define CFG_ENV_SECT_SIZE  (32 * 1024)
 #define CFG_ENV_SIZE   CFG_ENV_SECT_SIZE
+
+/* Address and size of Redundant Environment Sector*/
+#define CFG_ENV_OFFSET_REDUND  (CFG_ENV_OFFSET + CFG_ENV_SIZE)
+#define CFG_ENV_SIZE_REDUNDCF

Re: [U-Boot-Users] [PATCH] net: make ARP timeout configurable

[Ben Warren]

Guennadi Liakhovetski wrote:
> On Thu, 3 Apr 2008, Ben Warren wrote:
>
>   
>> Hi Guennadi,
>>
>> Guennadi Liakhovetski wrote:
>> 
>>> Currently the timeout waiting for an ARP reply is hard set to 5 seconds. On
>>> i.MX31ADS due to a hardware "strangeness" up to four first IP packets to the
>>> boards get lost, which typically are ARP replies. By configuring the timeout
>>> to a lower value we significantly improve the first network transfer time on
>>> this board. The timeout is specified in deciseconds, because it has to be
>>> converted to hardware ticks, and CFG_HZ ranges from 900 to 2700 on
>>> different boards.
>>>
>>>   
>> I like this, but let's stick to SI units please, probably ms.  This isn't an
>> important enough calculation to worry about precision, only overflow, so just
>> make sure you handle order of operations properly and it should be fine.  If
>> I'm not thinking this through properly, which is entirely possible, make the
>> code do a /100 to get into ds rather than asking the user to provide it.  I'm
>> concerned that people will say ***WTF is a decisecond!***
>> 
>
> Ok, expect a new version shortly.
>
>   
>>> @@ -253,7 +258,7 @@ void ArpTimeoutCheck(void)
>>> t = get_timer(0);
>>> /* check for arp timeout */
>>> -   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ) {
>>> +   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ / 10) {
>>> NetArpWaitTry++;
>>> if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
>>> @@ -494,7 +499,7 @@ restart:
>>>  *  Check the ethernet for a new packet.  The ethernet
>>>  *  receive routine will process it.
>>>  */
>>> -   eth_rx();
>>> +   eth_rx();
>>>   
>>>   
>> Not sure why you did this.  Is it indented from the if() once applied?
>> 
>
> It is not under an if - it is another hunk.
>   
Oh yeah.  Sorry.

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] net: make ARP timeout configurable

[Guennadi Liakhovetski]

On Thu, 3 Apr 2008, Ben Warren wrote:

> Hi Guennadi,
> 
> Guennadi Liakhovetski wrote:
> > Currently the timeout waiting for an ARP reply is hard set to 5 seconds. On
> > i.MX31ADS due to a hardware "strangeness" up to four first IP packets to the
> > boards get lost, which typically are ARP replies. By configuring the timeout
> > to a lower value we significantly improve the first network transfer time on
> > this board. The timeout is specified in deciseconds, because it has to be
> > converted to hardware ticks, and CFG_HZ ranges from 900 to 2700 on
> > different boards.
> > 
> I like this, but let's stick to SI units please, probably ms.  This isn't an
> important enough calculation to worry about precision, only overflow, so just
> make sure you handle order of operations properly and it should be fine.  If
> I'm not thinking this through properly, which is entirely possible, make the
> code do a /100 to get into ds rather than asking the user to provide it.  I'm
> concerned that people will say ***WTF is a decisecond!***

Ok, expect a new version shortly.

> > @@ -253,7 +258,7 @@ void ArpTimeoutCheck(void)
> > t = get_timer(0);
> > /* check for arp timeout */
> > -   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ) {
> > +   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ / 10) {
> > NetArpWaitTry++;
> > if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
> > @@ -494,7 +499,7 @@ restart:
> >  *  Check the ethernet for a new packet.  The ethernet
> >  *  receive routine will process it.
> >  */
> > -   eth_rx();
> > +   eth_rx();
> >   
> Not sure why you did this.  Is it indented from the if() once applied?

It is not under an if - it is another hunk.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [EMAIL PROTECTED]

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Guennadi Liakhovetski]

On Thu, 3 Apr 2008, Ben Warren wrote:

> Guennadi Liakhovetski wrote:
> > Replace direct register address derefencing with accessor functions.
> > Restrict explicitly 32-bit bus-width, extend affected configurations
> > respectively.
> > 
> >   
> Much better!  Thanks for doing all this work.  The only problem is that you're
> providing it relative to another patch that hasn't been accepted.

Well, the merge window is closed, so, I cannot send in any more new 
features. Whereas if we consider this as a fix - maybe it would be 
possible?

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.

DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: [EMAIL PROTECTED]

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] net: make ARP timeout configurable

[Ben Warren]

Hi Guennadi,

Guennadi Liakhovetski wrote:
> Currently the timeout waiting for an ARP reply is hard set to 5 seconds. 
> On i.MX31ADS due to a hardware "strangeness" up to four first IP packets 
> to the boards get lost, which typically are ARP replies. By configuring 
> the timeout to a lower value we significantly improve the first network 
> transfer time on this board. The timeout is specified in deciseconds, 
> because it has to be converted to hardware ticks, and CFG_HZ ranges from 
> 900 to 2700 on different boards.
>
>   
I like this, but let's stick to SI units please, probably ms.  This 
isn't an important enough calculation to worry about precision, only 
overflow, so just make sure you handle order of operations properly and 
it should be fine.  If I'm not thinking this through properly, which is 
entirely possible, make the code do a /100 to get into ds rather than 
asking the user to provide it.  I'm concerned that people will say 
***WTF is a decisecond!***
> Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>
>
> ---
>
> diff --git a/README b/README
> index a9663a3..9249064 100644
> --- a/README
> +++ b/README
> @@ -1547,6 +1547,10 @@ The following options need to be configured:
>   before giving up the operation. If not defined, a
>   default value of 5 is used.
>  
> + CONFIG_ARP_TIMEOUT
> +
> + Timeout waiting for an ARP reply in deciseconds.
> +
>  - Command Interpreter:
>   CONFIG_AUTO_COMPLETE
>  
> diff --git a/net/net.c b/net/net.c
> index 44feee2..2128bd4 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -94,11 +94,16 @@
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> -#define ARP_TIMEOUT  5UL /* Seconds before trying ARP 
> again */
> +#ifndef  CONFIG_ARP_TIMEOUT
> +# define ARP_TIMEOUT 50UL/* Deciseconds before trying ARP again 
> */
> +#else
> +# define ARP_TIMEOUT CONFIG_ARP_TIMEOUT
> +#endif
> +
>  #ifndef  CONFIG_NET_RETRY_COUNT
> -# define ARP_TIMEOUT_COUNT   5   /* # of timeouts before giving 
> up  */
> +# define ARP_TIMEOUT_COUNT   5   /* # of timeouts before giving up  */
>  #else
> -# define ARP_TIMEOUT_COUNT  (CONFIG_NET_RETRY_COUNT)
> +# define ARP_TIMEOUT_COUNT   CONFIG_NET_RETRY_COUNT
>  #endif
>  
>  #if 0
> @@ -129,7 +134,7 @@ uchar NetOurEther[6]; /* Our ethernet 
> address */
>  ucharNetServerEther[6] = /* Boot server enet address 
> */
>   { 0, 0, 0, 0, 0, 0 };
>  IPaddr_t NetOurIP;   /* Our IP addr (0 = unknown)
> */
> -IPaddr_t NetServerIP;/* Our IP addr (0 = unknown)
> */
> +IPaddr_t NetServerIP;/* Server IP addr (0 = unknown) 
> */
>  volatile uchar *NetRxPkt;/* Current receive packet   
> */
>  int  NetRxPktLen;/* Current rx packet length 
> */
>  unsigned NetIPID;/* IP packet ID 
> */
> @@ -253,7 +258,7 @@ void ArpTimeoutCheck(void)
>   t = get_timer(0);
>  
>   /* check for arp timeout */
> - if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ) {
> + if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ / 10) {
>   NetArpWaitTry++;
>  
>   if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
> @@ -494,7 +499,7 @@ restart:
>*  Check the ethernet for a new packet.  The ethernet
>*  receive routine will process it.
>*/
> - eth_rx();
> + eth_rx();
>   
Not sure why you did this.  Is it indented from the if() once applied?

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] Clean up smsc911x driver

[Ben Warren]

Guennadi Liakhovetski wrote:
> Replace direct register address derefencing with accessor functions. 
> Restrict explicitly 32-bit bus-width, extend affected configurations 
> respectively.
>
>   
Much better!  Thanks for doing all this work.  The only problem is that 
you're providing it relative to another patch that hasn't been accepted.

regards,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH 4/7] add SMSC LAN9x1x Network driver

[Ben Warren]

Hi Mark,

Mark Jonas wrote:
> Hi,
>
>   
>>  This driver seems to be a superset of yours, so this would be the one
>>  to go in once it's compliant.  Sorry for the lack of communication on
>>  my part.
>> 
>
> I am just sorry for the wasted hours it took to clean it up.
>
> So long,
> Mark
>   
We really appreciate the hard work you put into this.  Would you be able 
to try out the patch that Guennadi just posted and add your 'Acked-by'?

thanks,
Ben

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 6/6] AT91SAM9260EK: Normalize BOOTARGS

[Stelian Pop]

Set bootargs to sane values so that Linux boots out of the box in the
three supported configurations:
- boot from dataflash (CS0 or CS1): the bootstrap, U-Boot, its environment
and the Linux kernel are located in the dataflash, the JFFS2 root
filesystem occupies the whole NAND memory.
- boot from NAND: the bootstrap, U-Boot, its environment,
the Linux kernel and the JFFS2 root filesystem are all located in
different partitions of the NAND memory.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 include/configs/at91sam9260ek.h |   14 --
 1 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
index 0aac0f5..e25985e 100644
--- a/include/configs/at91sam9260ek.h
+++ b/include/configs/at91sam9260ek.h
@@ -56,8 +56,6 @@
 #define CONFIG_USART3  1   /* USART 3 is DBGU */
 
 #define CONFIG_BOOTDELAY   3
-#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
-   "root=/dev/mtdblock0 rw rootfstype=jffs2"
 
 /* #define CONFIG_ENV_OVERWRITE1 */
 
@@ -144,6 +142,10 @@
 #define CFG_ENV_ADDR   (CFG_DATAFLASH_LOGIC_ADDR_CS0 + CFG_ENV_OFFSET)
 #define CFG_ENV_SIZE   0x4200
 #define CONFIG_BOOTCOMMAND "cp.b 0xC003DE00 0x2200 0x200040; bootm"
+#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
+   "root=/dev/mtdblock0 " \
+   "mtdparts=at91_nand:-(root) "\
+   "rw rootfstype=jffs2"
 
 #elif CFG_USE_DATAFLASH_CS1
 
@@ -154,6 +156,10 @@
 #define CFG_ENV_ADDR   (CFG_DATAFLASH_LOGIC_ADDR_CS1 + CFG_ENV_OFFSET)
 #define CFG_ENV_SIZE   0x4200
 #define CONFIG_BOOTCOMMAND "cp.b 0xD003DE00 0x2200 0x200040; bootm"
+#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
+   "root=/dev/mtdblock0 " \
+   "mtdparts=at91_nand:-(root) "\
+   "rw rootfstype=jffs2"
 
 #else /* CFG_USE_NANDFLASH */
 
@@ -163,6 +169,10 @@
 #define CFG_ENV_OFFSET_REDUND  0x8
 #define CFG_ENV_SIZE   0x2 /* 1 sector = 128 kB */
 #define CONFIG_BOOTCOMMAND "nand read 0x2200 0xA 0x20; bootm"
+#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
+   "root=/dev/mtdblock5 " \
+   
"mtdparts=at91_nand:128k(bootstrap)ro,256k(uboot)ro,128k(env1)ro,128k(env2)ro,2M(linux),-(root)
 " \
+   "rw rootfstype=jffs2"
 
 #endif
 
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 5/6] AT91SAM9260EK: Normalize SPI timings

[Stelian Pop]

Set the SPI clock to 15 MHz, like Linux does.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 include/configs/at91sam9260ek.h |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
index 8d02416..0aac0f5 100644
--- a/include/configs/at91sam9260ek.h
+++ b/include/configs/at91sam9260ek.h
@@ -96,7 +96,7 @@
 #define CFG_MAX_DATAFLASH_BANKS2
 #define CFG_DATAFLASH_LOGIC_ADDR_CS0   0xC000  /* CS0 */
 #define CFG_DATAFLASH_LOGIC_ADDR_CS1   0xD000  /* CS1 */
-#define AT91_SPI_CLK   3300
+#define AT91_SPI_CLK   1500
 #define DATAFLASH_TCSS (0x1a << 16)
 #define DATAFLASH_TCHS (0x1 << 24)
 
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 4/6] AT91SAM9260EK: Handle 8 or 16 bit NAND

[Stelian Pop]

Enable the user specify in the board configuration header if the
board is fitted with 8 bit (CFG_NAND_DBW_8) or 16 bit (CFG_NAND_DBW_16)
NAND memory.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 board/atmel/at91sam9260ek/at91sam9260ek.c |7 ++-
 board/atmel/at91sam9260ek/nand.c  |3 +++
 include/configs/at91sam9260ek.h   |1 +
 3 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/board/atmel/at91sam9260ek/at91sam9260ek.c 
b/board/atmel/at91sam9260ek/at91sam9260ek.c
index a55468e..7d325e8 100644
--- a/board/atmel/at91sam9260ek/at91sam9260ek.c
+++ b/board/atmel/at91sam9260ek/at91sam9260ek.c
@@ -90,7 +90,12 @@ static void at91sam9260ek_nand_hw_init(void)
at91_sys_write(AT91_SMC_MODE(3),
   AT91_SMC_READMODE | AT91_SMC_WRITEMODE |
   AT91_SMC_EXNWMODE_DISABLE |
-  AT91_SMC_DBW_8 | AT91_SMC_TDF_(2));
+#ifdef CFG_NAND_DBW_16
+  AT91_SMC_DBW_16 |
+#else /* CFG_NAND_DBW_8 */
+  AT91_SMC_DBW_8 |
+#endif
+  AT91_SMC_TDF_(2));
 
at91_sys_write(AT91_PMC_PCER, 1 << AT91SAM9260_ID_PIOC);
 
diff --git a/board/atmel/at91sam9260ek/nand.c b/board/atmel/at91sam9260ek/nand.c
index abb788a..a92b105 100644
--- a/board/atmel/at91sam9260ek/nand.c
+++ b/board/atmel/at91sam9260ek/nand.c
@@ -68,6 +68,9 @@ static int at91sam9260ek_nand_ready(struct mtd_info *mtd)
 int board_nand_init(struct nand_chip *nand)
 {
nand->eccmode = NAND_ECC_SOFT;
+#ifdef CFG_NAND_DBW_16
+   nand->options = NAND_BUSWIDTH_16;
+#endif
nand->hwcontrol = at91sam9260ek_nand_hwcontrol;
nand->dev_ready = at91sam9260ek_nand_ready;
nand->chip_delay = 20;
diff --git a/include/configs/at91sam9260ek.h b/include/configs/at91sam9260ek.h
index 96d1b8d..8d02416 100644
--- a/include/configs/at91sam9260ek.h
+++ b/include/configs/at91sam9260ek.h
@@ -104,6 +104,7 @@
 #define NAND_MAX_CHIPS 1
 #define CFG_MAX_NAND_DEVICE1
 #define CFG_NAND_BASE  0x4000
+#define CFG_NAND_DBW_8 1
 
 /* NOR flash - no real flash on this board */
 #define CFG_NO_FLASH   1
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH ARM/AT91 0/6] AT91CAP9/AT91SAM9260 cleanups

[Stelian Pop]

Hi,

The following patches do a few cleanups for the AT91CAP9ADK and
AT91SAM9260EK boards:
- enable handling of 8 or 16 bit NAND memories
- set the SPI clock to 15MHz (like Linux does)
- set bootargs to sane values so that Linux boots out of
  the box in the supported configurations.

Please apply,

Thanks,

Stelian.


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 1/6] AT91CAP9ADK: Handle 8 or 16 bit NAND

[Stelian Pop]

Enable the user specify in the board configuration header if the
board is fitted with 8 bit (CFG_NAND_DBW_8) or 16 bit (CFG_NAND_DBW_16)
NAND memory.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 board/atmel/at91cap9adk/at91cap9adk.c |8 ++--
 board/atmel/at91cap9adk/nand.c|3 +++
 include/configs/at91cap9adk.h |1 +
 3 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/board/atmel/at91cap9adk/at91cap9adk.c 
b/board/atmel/at91cap9adk/at91cap9adk.c
index 7444c5a..8f96228 100644
--- a/board/atmel/at91cap9adk/at91cap9adk.c
+++ b/board/atmel/at91cap9adk/at91cap9adk.c
@@ -116,7 +116,12 @@ static void at91cap9_nand_hw_init(void)
at91_sys_write(AT91_SMC_MODE(3),
   AT91_SMC_READMODE | AT91_SMC_WRITEMODE |
   AT91_SMC_EXNWMODE_DISABLE |
-  AT91_SMC_DBW_8 | AT91_SMC_TDF_(1));
+#ifdef CFG_NAND_DBW_16
+  AT91_SMC_DBW_16 |
+#else /* CFG_NAND_DBW_8 */
+  AT91_SMC_DBW_8 |
+#endif
+  AT91_SMC_TDF_(1));
 
at91_sys_write(AT91_PMC_PCER, 1 << AT91CAP9_ID_PIOABCD);
 
@@ -252,7 +257,6 @@ int board_init(void)
 #ifdef CONFIG_USB_OHCI_NEW
at91cap9_uhp_hw_init();
 #endif
-
return 0;
 }
 
diff --git a/board/atmel/at91cap9adk/nand.c b/board/atmel/at91cap9adk/nand.c
index c72b024..5d49aee 100644
--- a/board/atmel/at91cap9adk/nand.c
+++ b/board/atmel/at91cap9adk/nand.c
@@ -63,6 +63,9 @@ static void at91cap9adk_nand_hwcontrol(struct mtd_info *mtd, 
int cmd)
 int board_nand_init(struct nand_chip *nand)
 {
nand->eccmode = NAND_ECC_SOFT;
+#ifdef CFG_NAND_DBW_16
+   nand->options = NAND_BUSWIDTH_16;
+#endif
nand->hwcontrol = at91cap9adk_nand_hwcontrol;
nand->chip_delay = 20;
 
diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
index dab21d0..88527a4 100644
--- a/include/configs/at91cap9adk.h
+++ b/include/configs/at91cap9adk.h
@@ -110,6 +110,7 @@
 #define NAND_MAX_CHIPS 1
 #define CFG_MAX_NAND_DEVICE1
 #define CFG_NAND_BASE  0x4000
+#define CFG_NAND_DBW_8 1
 
 /* Ethernet */
 #define CONFIG_MACB1
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 3/6] AT91CAP9ADK: Normalize BOOTARGS

[Stelian Pop]

Set bootargs to sane values so that Linux boots out of the box in the two
supported configurations:
- boot from dataflash: the bootstrap, U-Boot, its environment
and the Linux kernel are located in the dataflash, the JFFS2 root
filesystem occupies the whole NAND memory.
- boot from NOR: the bootstrap, U-Boot, its environment
and the Linux kernel are located in the NOR memory, the JFFS2 root
filesystem occupies the whole NAND memory.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 include/configs/at91cap9adk.h |   10 --
 1 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
index a648ce0..78b2283 100644
--- a/include/configs/at91cap9adk.h
+++ b/include/configs/at91cap9adk.h
@@ -56,8 +56,6 @@
 #define CONFIG_USART3  1   /* USART 3 is DBGU */
 
 #define CONFIG_BOOTDELAY   3
-#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
-   "root=/dev/mtdblock1 rw rootfstype=jffs2"
 
 /* #define CONFIG_ENV_OVERWRITE1 */
 
@@ -145,6 +143,10 @@
 #define CFG_ENV_ADDR   (CFG_DATAFLASH_LOGIC_ADDR_CS0 + CFG_ENV_OFFSET)
 #define CFG_ENV_SIZE   0x4200
 #define CONFIG_BOOTCOMMAND "cp.b 0xC003DE00 0x7200 0x200040; bootm"
+#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
+   "root=/dev/mtdblock1 " \
+   
"mtdparts=physmap-flash.0:-(nor);at91_nand:-(root) "\
+   "rw rootfstype=jffs2"
 
 #else
 
@@ -155,6 +157,10 @@
 #define CFG_ENV_ADDR   (PHYS_FLASH_1 + CFG_ENV_OFFSET)
 #define CFG_ENV_SIZE   0x4000
 #define CONFIG_BOOTCOMMAND "cp.b 0x1004 0x7200 0x20; bootm"
+#define CONFIG_BOOTARGS"console=ttyS0,115200 " \
+   "root=/dev/mtdblock4 " \
+   
"mtdparts=physmap-flash.0:16k(bootstrap)ro,16k(env),224k(uboot)ro,-(linux);at91_nand:-(root)
 "\
+   "rw rootfstype=jffs2"
 
 #endif
 
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH 2/6] AT91CAP9ADK: Normalize SPI timings

[Stelian Pop]

Set the SPI clock to 15 MHz, like Linux does.

Signed-off-by: Stelian Pop <[EMAIL PROTECTED]>
---
 include/configs/at91cap9adk.h |6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/configs/at91cap9adk.h b/include/configs/at91cap9adk.h
index 88527a4..a648ce0 100644
--- a/include/configs/at91cap9adk.h
+++ b/include/configs/at91cap9adk.h
@@ -94,9 +94,9 @@
 #define CFG_SPI_WRITE_TOUT (5*CFG_HZ)
 #define CFG_MAX_DATAFLASH_BANKS1
 #define CFG_DATAFLASH_LOGIC_ADDR_CS0   0xC000  /* CS0 */
-#define AT91_SPI_CLK   2000
-#define DATAFLASH_TCSS (0xFA << 16)
-#define DATAFLASH_TCHS (0x8 << 24)
+#define AT91_SPI_CLK   1500
+#define DATAFLASH_TCSS (0x1a << 16)
+#define DATAFLASH_TCHS (0x1 << 24)
 
 /* NOR flash */
 #define CFG_FLASH_CFI  1
-- 
1.5.3.3


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] New Image format: headers hashes.

[Luigi 'Comio' Mantellini]

Hi Bartlomej,


On gio, 2008-04-03 at 13:48 +0200, Bartlomiej Sieka wrote:
...
> >>
> > 
> > Before to make this test, you should strip the elf-file in order to omit
> > the non-used code. Alternatively you should check the size of the
> > u-boot.bin file (that is striped by definition).
> 
> "objcopy -O binary" which is used to create u-boot.bin from u-boot
> doesn't remove unused functions -- code for fit_image_set_hashes (and
> other functions removed from under #ifdef USE_HOSTCC by the patch) is
> present in u-boot.bin.

I'm sorry, I believed (wrongly) that linker was sufficiently intelligent
to discard the unused symbols and reallocate the rest.
Do you know tools (or ld options) that allow to discard unused
symbols/functions?

Thanks a lot. today I learned something new.

luigi

> If you add enough of such dead code, you'll see the increase in size of 
> u-boot.bin, or run into linking problems -- depending on the particular 
> linker script used for the target.
> 
> Regards,
> Bartlomiej
-- 
 __   Luigi Mantellini
   .'__'. R&D - Software
  (.'  '.)Industrie Dial Face S.p.A.
  ( :==: )Via Canzo, 4
  ('.__.')20068 Peschiera Borromeo (MI), Italy
   '.__.' Tel.: +39 02 5167 2813
  Fax:  +39 02 5167 2459
Ind.  Dial Face   Email: [EMAIL PROTECTED]
www.idf-hit.com   GPG fingerprint: 3DD1 7B71 FBDF 6376 1B4A
   B003 175F E979 907E 1650





-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] ppc4xx: Fix 4xx enet driver to support 460GT EMAC2+3

[Stefan Roese]

This patch fixes a problem with the RGMII setup of the 460GT. The 460GT
has 2 RGMII instances and we need to configure the 2nd RGMII instance
for the EMAC2+3 channels.

Signed-off-by: Stefan Roese <[EMAIL PROTECTED]>
---
 cpu/ppc4xx/4xx_enet.c |   16 +---
 1 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/cpu/ppc4xx/4xx_enet.c b/cpu/ppc4xx/4xx_enet.c
index 21ba922..007cb4f 100644
--- a/cpu/ppc4xx/4xx_enet.c
+++ b/cpu/ppc4xx/4xx_enet.c
@@ -717,6 +717,11 @@ static int ppc_4xx_eth_init (struct eth_device *dev, bd_t 
* bis)
 #ifdef CONFIG_4xx_DCACHE
static u32 last_used_ea = 0;
 #endif
+#if defined(CONFIG_440EPX) || defined(CONFIG_440GRX) || \
+defined(CONFIG_460EX) || defined(CONFIG_460GT) || \
+defined(CONFIG_405EX)
+   int rgmii_channel;
+#endif
 
EMAC_4XX_HW_PST hw_p = dev->priv;
 
@@ -1022,12 +1027,17 @@ static int ppc_4xx_eth_init (struct eth_device *dev, 
bd_t * bis)
 #if defined(CONFIG_440EPX) || defined(CONFIG_440GRX) || \
 defined(CONFIG_460EX) || defined(CONFIG_460GT) || \
 defined(CONFIG_405EX)
+   if (devnum >= 2)
+   rgmii_channel = devnum - 2;
+   else
+   rgmii_channel = devnum;
+
if (speed == 1000)
-   reg = (RGMII_SSR_SP_1000MBPS << RGMII_SSR_V (devnum));
+   reg = (RGMII_SSR_SP_1000MBPS << RGMII_SSR_V(rgmii_channel));
else if (speed == 100)
-   reg = (RGMII_SSR_SP_100MBPS << RGMII_SSR_V (devnum));
+   reg = (RGMII_SSR_SP_100MBPS << RGMII_SSR_V(rgmii_channel));
else if (speed == 10)
-   reg = (RGMII_SSR_SP_10MBPS << RGMII_SSR_V (devnum));
+   reg = (RGMII_SSR_SP_10MBPS << RGMII_SSR_V(rgmii_channel));
else {
printf("Error in RGMII Speed\n");
return -1;
-- 
1.5.4.5


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5

[Sascha Laue]

Subject: [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5
Change configuartion for lwmon5 board.

Signed-off-by: Sascha Laue <[EMAIL PROTECTED]>
---
 include/configs/lwmon5.h |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/include/configs/lwmon5.h b/include/configs/lwmon5.h
index ced7ba6..cb2c4b5 100644
--- a/include/configs/lwmon5.h
+++ b/include/configs/lwmon5.h
@@ -434,7 +434,6 @@
 #define CONFIG_CMDLINE_EDITING 1   /* add command line history */
 #define CONFIG_LOOPW1   /* enable loopw command */
 #define CONFIG_MX_CYCLIC1   /* enable mdc/mwc commands  */
-#define CONFIG_ZERO_BOOTDELAY_CHECK/* check for keypress on bootdelay==0 */
 #define CONFIG_VERSION_VARIABLE 1  /* include version env variable */
 
 /*---
-- 
1.5.2.4

 Original-Nachricht 
> Datum: Thu, 03 Apr 2008 11:38:20 +0200
> Von: Wolfgang Denk <[EMAIL PROTECTED]>
> An: "Sascha Laue" <[EMAIL PROTECTED]>
> CC: u-boot-users@lists.sourceforge.net
> Betreff: Re: [U-Boot-Users] [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5

> In message <[EMAIL PROTECTED]> you wrote:
> > Hi, 
> > 
> > Subject: [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5
> > 
> > Change configuartion for lwmon5 board.
> > 
> > Signed-off-by: Sascha Laue <[EMAIL PROTECTED]>
> > ---
> >  include/configs/lwmon5.h |2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/include/configs/lwmon5.h b/include/configs/lwmon5.h
> > index ced7ba6..ff4bc80 100644
> > --- a/include/configs/lwmon5.h
> > +++ b/include/configs/lwmon5.h
> > @@ -434,7 +434,9 @@
> >  #define CONFIG_CMDLINE_EDITING 1   /* add command line history 
> > */
> >  #define CONFIG_LOOPW1   /* enable loopw command
> */
> >  #define CONFIG_MX_CYCLIC1   /* enable mdc/mwc commands 
> */
> > +#if 0   /* enable only for debug */
> >  #define CONFIG_ZERO_BOOTDELAY_CHECK/* check for keypress on
> bootdelay==0 */
> > +#endif
> >  #define CONFIG_VERSION_VARIABLE 1  /* include version env variable */
> 
> Please don't add dead code. If you want to remove that line, then
> please remove it. No "#if 0", please.
> 
> Viele Grüße,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
> There are three ways to get something done:
> (1) Do it yourself.
> (2) Hire someone to do it for you.
> (3) Forbid your kids to do it.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Clean up smsc911x driver

[Guennadi Liakhovetski]

Replace direct register address derefencing with accessor functions. 
Restrict explicitly 32-bit bus-width, extend affected configurations 
respectively.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

Sascha, please test.

diff --git a/drivers/net/smc911x.c b/drivers/net/smc911x.c
index c332189..c17dcf4 100644
--- a/drivers/net/smc911x.c
+++ b/drivers/net/smc911x.c
@@ -30,321 +30,332 @@
 #include 
 #include 
 
-#define mdelay(n)   udelay((n)*1000)
+#ifdef CONFIG_DRIVER_SMC911X_32_BIT
+static inline u32 reg_read(u32 addr)
+{
+   return *(volatile u32*)addr;
+}
+static inline void reg_write(u32 addr, u32 val)
+{
+   *(volatile u32*)addr = val;
+}
+#else
+#error "SMC911X: Only 32-bit bus is supported"
+#endif
 
-#define __REG(x) (*((volatile u32 *)(x)))
+#define mdelay(n)   udelay((n)*1000)
 
 /* Below are the register offsets and bit definitions
  * of the Lan911x memory space
  */
-#define RX_DATA_FIFO__REG(CONFIG_DRIVER_SMC911X_BASE + 0x00)
-
-#define TX_DATA_FIFO__REG(CONFIG_DRIVER_SMC911X_BASE + 0x20)
-#defineTX_CMD_A_INT_ON_COMP(0x8000)
-#defineTX_CMD_A_INT_BUF_END_ALGN   (0x0300)
-#defineTX_CMD_A_INT_4_BYTE_ALGN(0x)
-#defineTX_CMD_A_INT_16_BYTE_ALGN   (0x0100)
-#defineTX_CMD_A_INT_32_BYTE_ALGN   (0x0200)
-#defineTX_CMD_A_INT_DATA_OFFSET(0x001F)
-#defineTX_CMD_A_INT_FIRST_SEG  (0x2000)
-#defineTX_CMD_A_INT_LAST_SEG   (0x1000)
-#defineTX_CMD_A_BUF_SIZE   (0x07FF)
-#defineTX_CMD_B_PKT_TAG(0x)
-#defineTX_CMD_B_ADD_CRC_DISABLE(0x2000)
-#defineTX_CMD_B_DISABLE_PADDING(0x1000)
-#defineTX_CMD_B_PKT_BYTE_LENGTH(0x07FF)
-
-#define RX_STATUS_FIFO __REG(CONFIG_DRIVER_SMC911X_BASE + 0x40)
-#defineRX_STS_PKT_LEN  (0x3FFF)
-#defineRX_STS_ES   (0x8000)
-#defineRX_STS_BCST (0x2000)
-#defineRX_STS_LEN_ERR  (0x1000)
-#defineRX_STS_RUNT_ERR (0x0800)
-#defineRX_STS_MCAST(0x0400)
-#defineRX_STS_TOO_LONG (0x0080)
-#defineRX_STS_COLL (0x0040)
-#defineRX_STS_ETH_TYPE (0x0020)
-#defineRX_STS_WDOG_TMT (0x0010)
-#defineRX_STS_MII_ERR  (0x0008)
-#defineRX_STS_DRIBBLING(0x0004)
-#defineRX_STS_CRC_ERR  (0x0002)
-#define RX_STATUS_FIFO_PEEK__REG(CONFIG_DRIVER_SMC911X_BASE + 0x44)
-#define TX_STATUS_FIFO __REG(CONFIG_DRIVER_SMC911X_BASE + 0x48)
-#defineTX_STS_TAG  (0x)
-#defineTX_STS_ES   (0x8000)
-#defineTX_STS_LOC  (0x0800)
-#defineTX_STS_NO_CARR  (0x0400)
-#defineTX_STS_LATE_COLL(0x0200)
-#defineTX_STS_MANY_COLL(0x0100)
-#defineTX_STS_COLL_CNT (0x0078)
-#defineTX_STS_MANY_DEFER   (0x0004)
-#defineTX_STS_UNDERRUN (0x0002)
-#defineTX_STS_DEFERRED (0x0001)
-#define TX_STATUS_FIFO_PEEK__REG(CONFIG_DRIVER_SMC911X_BASE + 0x4C)
-#define ID_REV __REG(CONFIG_DRIVER_SMC911X_BASE + 0x50)
-#defineID_REV_CHIP_ID  (0x)  /* RO */
-#defineID_REV_REV_ID   (0x)  /* RO */
-
-#define INT_CFG__REG(CONFIG_DRIVER_SMC911X_BASE + 0x54)
-#defineINT_CFG_INT_DEAS(0xFF00)  /* R/W */
-#defineINT_CFG_INT_DEAS_CLR(0x4000)
-#defineINT_CFG_INT_DEAS_STS(0x2000)
-#defineINT_CFG_IRQ_INT (0x1000)  /* RO */
-#defineINT_CFG_IRQ_EN  (0x0100)  /* R/W */
-#defineINT_CFG_IRQ_POL (0x0010)  /* R/W 
Not Affected by SW Reset */
-#defineINT_CFG_IRQ_TYPE(0x0001)  /* R/W 
Not Affected by SW Reset */
-
-#define INT_STS__REG(CONFIG_DRIVER_SMC911X_BASE + 0x58)
-#defineINT_STS_SW_INT  (0x8000)  /* R/WC */
-#defineINT_STS_TXSTOP_INT  (0x0200)  /* R

[U-Boot-Users] [PATCH] TQM5200: fix default IDE reset level

[Martin Krause]

Bevore the first call of ide_reset() the default level of
the IDE reset signal on the TQM5200 is low (reset asserted).
This patch sets the default value to high (reset not asserted).

Currently this patch fixes no real problem, but it is cleaner
to assert the reset only on demand, and not permanently.

Signed-off-by: Martin Krause <[EMAIL PROTECTED]>
---

 board/tqm5200/tqm5200.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)


diff --git a/board/tqm5200/tqm5200.c b/board/tqm5200/tqm5200.c
index e67145e..3715267 100644
--- a/board/tqm5200/tqm5200.c
+++ b/board/tqm5200/tqm5200.c
@@ -316,6 +316,9 @@ void init_ide_reset (void)
/* Configure PSC1_4 as GPIO output for ATA reset */
*(vu_long *) MPC5XXX_WU_GPIO_ENABLE |= GPIO_PSC1_4;
*(vu_long *) MPC5XXX_WU_GPIO_DIR|= GPIO_PSC1_4;
+
+   /* per default the ATA reset is de-asserted */
+   *(vu_long *) MPC5XXX_WU_GPIO_DATA_O |=  GPIO_PSC1_4;
 #endif
 }
 


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] [Cosmetic] Realign CONFIG_EXTRA_ENV_SETTING for Keymile boards.

[Detlev Zundel]

Signed-off-by: Detlev Zundel <[EMAIL PROTECTED]>
---
 include/configs/mgcoge.h |   56 +++---
 include/configs/mgsuvd.h |   50 
 2 files changed, 53 insertions(+), 53 deletions(-)

diff --git a/include/configs/mgcoge.h b/include/configs/mgcoge.h
index f0fed06..2924acc 100644
--- a/include/configs/mgcoge.h
+++ b/include/configs/mgcoge.h
@@ -89,35 +89,35 @@
 /*
  * Default environment settings
  */
-#define CONFIG_EXTRA_ENV_SETTINGS  \
-   "netdev=eth0\0" \
-   "u-boot_addr=10\0"  \
-   "kernel_addr=20\0"  \
-   "fdt_addr=40\0" \
-   "rootpath=/opt/eldk-4.2/ppc_82xx\0" \
-   "u-boot=/tftpboot/mgcoge/u-boot.bin\0"  \
-   "bootfile=/tftpboot/mgcoge/uImage\0"\
-   "fdt_file=/tftpboot/mgcoge/mgcoge.dtb\0"\
-   "load=tftp ${u-boot_addr} ${u-boot}\0"  \
-   "update=prot off fe00 fe03; era fe00 fe03; "\
-   "cp.b ${u-boot_addr} fe00 ${filesize};" \
-   "prot on fe00 fe03\0"   \
-   "ramargs=setenv bootargs root=/dev/ram rw\0"\
-   "nfsargs=setenv bootargs root=/dev/nfs rw " \
-   "nfsroot=${serverip}:${rootpath}\0" \
+#define CONFIG_EXTRA_ENV_SETTINGS  
\
+   "netdev=eth0\0" 
\
+   "u-boot_addr=10\0"  
\
+   "kernel_addr=20\0"  
\
+   "fdt_addr=40\0" 
\
+   "rootpath=/opt/eldk-4.2/ppc_82xx\0" 
\
+   "u-boot=/tftpboot/mgcoge/u-boot.bin\0"  
\
+   "bootfile=/tftpboot/mgcoge/uImage\0"
\
+   "fdt_file=/tftpboot/mgcoge/mgcoge.dtb\0"
\
+   "load=tftp ${u-boot_addr} ${u-boot}\0"  
\
+   "update=prot off fe00 fe03; era fe00 fe03; "
\
+   "cp.b ${u-boot_addr} fe00 ${filesize};" 
\
+   "prot on fe00 fe03\0"   
\
+   "ramargs=setenv bootargs root=/dev/ram rw\0"
\
+   "nfsargs=setenv bootargs root=/dev/nfs rw " 
\
+   "nfsroot=${serverip}:${rootpath}\0" 
\
"addcons=setenv bootargs ${bootargs} console=ttyCPM0,${baudrate}\0" 
\
-   "addmtd=setenv bootargs ${bootargs} ${mtdparts}\0"  \
-   "addip=setenv bootargs ${bootargs} "\
-   "ip=${ipaddr}:${serverip}:${gatewayip}:"\
-   "${netmask}:${hostname}:${netdev}:off panic=1\0"\
-   "net_nfs=tftp ${kernel_addr} ${bootfile}; " \
-   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcons;"\
-   "bootm ${kernel_addr} - ${fdt_addr}\0"  \
-   "net_self=tftp ${kernel_addr} ${bootfile}; "\
-   "tftp ${fdt_addr} ${fdt_file}; "\
-   "tftp ${ramdisk_addr} ${ramdisk_file}; "\
-   "run ramargs addip; "   \
-   "bootm ${kernel_addr} ${ramdisk_addr} ${fdt_addr}\0"\
+   "addmtd=setenv bootargs ${bootargs} ${mtdparts}\0"  
\
+   "addip=setenv bootargs ${bootargs} "
\
+   "ip=${ipaddr}:${serverip}:${gatewayip}:"
\
+   "${netmask}:${hostname}:${netdev}:off panic=1\0"
\
+   "net_nfs=tftp ${kernel_addr} ${bootfile}; " 
\
+   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcons;"  
\
+   "bootm ${kernel_addr} - ${fdt_addr}\0"  
\
+   "net_self=tftp ${kernel_addr} ${bootfile}; "
\
+   "tftp ${fdt_addr} ${fdt_file}; "
\
+   "tftp ${ramdisk_addr} ${ramdisk_file}; "
\
+   "run ramargs addip; "   
\
+   "bootm ${kernel_addr} ${ramdisk_addr} ${fdt_addr}\0"
\
""
 #define CONFIG_BOOTCOMMAND 

[U-Boot-Users] [PATCH] Rename 'addcon' to 'addcons' environment command for Keymile boards.

[Detlev Zundel]

The latter name with 13 users is already established, so we will use
that.

Signed-off-by: Detlev Zundel <[EMAIL PROTECTED]>
---
 include/configs/mgcoge.h |4 ++--
 include/configs/mgsuvd.h |4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/configs/mgcoge.h b/include/configs/mgcoge.h
index b1e3d53..f0fed06 100644
--- a/include/configs/mgcoge.h
+++ b/include/configs/mgcoge.h
@@ -105,13 +105,13 @@
"ramargs=setenv bootargs root=/dev/ram rw\0"\
"nfsargs=setenv bootargs root=/dev/nfs rw " \
"nfsroot=${serverip}:${rootpath}\0" \
-   "addcon=setenv bootargs ${bootargs} console=ttyCPM0,${baudrate}\0"  
\
+   "addcons=setenv bootargs ${bootargs} console=ttyCPM0,${baudrate}\0" 
\
"addmtd=setenv bootargs ${bootargs} ${mtdparts}\0"  \
"addip=setenv bootargs ${bootargs} "\
"ip=${ipaddr}:${serverip}:${gatewayip}:"\
"${netmask}:${hostname}:${netdev}:off panic=1\0"\
"net_nfs=tftp ${kernel_addr} ${bootfile}; " \
-   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcon;"\
+   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcons;"\
"bootm ${kernel_addr} - ${fdt_addr}\0"  \
"net_self=tftp ${kernel_addr} ${bootfile}; "\
"tftp ${fdt_addr} ${fdt_file}; "\
diff --git a/include/configs/mgsuvd.h b/include/configs/mgsuvd.h
index 5482089..aa026d9 100644
--- a/include/configs/mgsuvd.h
+++ b/include/configs/mgsuvd.h
@@ -61,7 +61,7 @@
 
 #define CONFIG_EXTRA_ENV_SETTINGS  \
"netdev=eth0\0" \
-   "addcon=setenv bootargs ${bootargs} console=ttyCPM0,${baudrate}\0"  
\
+   "addcons=setenv bootargs ${bootargs} console=ttyCPM0,${baudrate}\0" 
\
"nfsargs=setenv bootargs root=/dev/nfs rw " \
"nfsroot=${serverip}:${rootpath}\0" \
"ramargs=setenv bootargs root=/dev/ram rw\0"\
@@ -73,7 +73,7 @@
"flash_self=run ramargs addip;" \
"bootm ${kernel_addr} ${ramdisk_addr}\0"\
"net_nfs=tftp ${kernel_addr} ${bootfile}; " \
-   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcon;"\
+   "tftp ${fdt_addr} ${fdt_file}; run nfsargs addip addcons;"\
"bootm ${kernel_addr} - ${fdt_addr}\0"  \
"rootpath=/opt/eldk/ppc_8xx\0"  \
"bootfile=/tftpboot/mgsuvd/uImage\0"\
-- 
1.5.4.2


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] reg to configure DSPI in mcf54455

[Karthikeyan Kadirvel, Chennai]

Hi,

 

I have the u-boot code of mcf54455evb.now I am designing a new board
with processor of mcf54452.

 

When I make the changes in the u-boot code for configuring SPI, i am
getting errors like this 

undefined reference to `spi_init_f' 
undefined reference to `spi_init_r' 

(in the file ~/board.c) 

undefined reference to `spi_read' 
undefined reference to `spi_write' 
undefined reference to `spi_init_f' 

(in the file ~/cmd_eeprom.c) 

 

I am following the README file. i have done the modification for SPI
configuration what they mentioned in the read me file.

Even though the two C files have inludes to an .h file (common.h) that
define these functions...Any ideas what is happening here? The thing is,
the only place I can see that these functions are given are in a file
called spi.c, which exists in *some* board config folders. Does this
mean that i need an spi.c file added into the /board/mcf54455evb/
folder? I have to write the definitions for those function?

 

Thanks & Regards,

Karthik

 

 



DISCLAIMER:
---

The contents of this e-mail and any attachment(s) are confidential and intended 
for the named recipient(s) only.
It shall not attach any liability on the originator or HCL or its affiliates. 
Any views or opinions presented in 
this email are solely those of the author and may not necessarily reflect the 
opinions of HCL or its affiliates.
Any form of reproduction, dissemination, copying, disclosure, modification, 
distribution and / or publication of 
this message without the prior written consent of the author of this e-mail is 
strictly prohibited. If you have
received this email in error please delete it and notify the sender 
immediately. Before opening any mail and 
attachments please check them for viruses and defect.


Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] New Image format: headers hashes.

[Bartlomiej Sieka]

Luigi 'Comio' Mantellini wrote:
> Hi Bartlomiej,
> 
> thank you for the answers. See my comments below.
> 
> On gio, 2008-04-03 at 11:31 +0200, Bartlomiej Sieka wrote:
>> Hi Luigi,
>>
>> You wrote:
>>> On gio, 2008-03-27 at 12:13 +0100, L
>>> - I'm interested to calculate and store from U-boot code but the 
>>> fit_image_set_hashes is compiled only when the USE_HOSTCC symbol is 
>>> defined. Why? Is there a particular reason?
>> There's no need for the function that you mention in current U-Boot
>> code, that's the reason why it's under USE_HOSTCC. If you're planning on
>> adding code to U-Boot that will use fit_image_set_hashes(), then just
>> remove the USE_HOSTCC #define's from around this function.
>>
>>
>>  > I think that a lot of
>>  > useful functions (ifdef-ed by USE_HOSTCC) should be available also in
>>  > u-boot code, anyway the linker will drop the unused functions.
>>
>> I don't think the linker will do this in case of U-Boot:
>>
>> With the following patch:
>>
>> diff --git a/common/image.c b/common/image.c
>> index f04826a..850a9e4 100644
>> --- a/common/image.c
>> +++ b/common/image.c
>> @@ -1950,7 +1950,6 @@ static int calculate_hash (const void *data, int 
>> data_len,
>>  return 0;
>>   }
>>
>> -#ifdef USE_HOSTCC
>>   /**
>>* fit_set_hashes - process FIT component image nodes and calculate hashes
>>* @fit: pointer to the FIT format image header
>> @@ -2119,7 +2118,6 @@ int fit_image_hash_set_value (void *fit, int 
>> noffset, uint
>>
>>  return 0;
>>   }
>> -#endif /* USE_HOSTCC */
>>
>>   /**
>>* fit_image_check_hashes - verify data intergity
>>
>>
>> I see:
>>
>> $ ${CROSS_COMPILE}nm -S common/image.o | grep fit_image_set_hashes
>> 155c 01f4 T fit_image_set_hashes
>> $ ${CROSS_COMPILE}nm -S common/libcommon.a | grep fit_image_set_hashes
>> 155c 01f4 T fit_image_set_hashes
>> $ ${CROSS_COMPILE}nm -S u-boot | grep fit_image_set_hashes
>> fffb1924 01f4 T fit_image_set_hashes
>>
>>
>> There's also size increase with the patch applied:
>> textdata bss dec hex filename
>>   284615   15556  357120  657291   a078b ./u-boot
>>
>> textdata bss dec hex filename
>>   285875   15572  357120  658567   a0c87 u-boot
>>
>>
> 
> Before to make this test, you should strip the elf-file in order to omit
> the non-used code. Alternatively you should check the size of the
> u-boot.bin file (that is striped by definition).

"objcopy -O binary" which is used to create u-boot.bin from u-boot
doesn't remove unused functions -- code for fit_image_set_hashes (and
other functions removed from under #ifdef USE_HOSTCC by the patch) is
present in u-boot.bin.

If you add enough of such dead code, you'll see the increase in size of 
u-boot.bin, or run into linking problems -- depending on the particular 
linker script used for the target.

Regards,
Bartlomiej

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] IDE: fix bug in reset sequence

[Martin Krause]

According to the ata (ata5) specification the RESET- signal
shall be asserted for at least 25 us. Without this patch,
the RESET- signal is asserted on some boards for only < 1 us
(e. g. on the TQM5200). This patch adds a general delay of
25 us to the RESET- signal.

Without this patch a Platinum 4 GiB CF card is not recognised
properly on boards with a TQM5200 (STK52xx, TB5200).

Signed-off-by: Martin Krause <[EMAIL PROTECTED]>
---

 common/cmd_ide.c |3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)


diff --git a/common/cmd_ide.c b/common/cmd_ide.c
index 8ace970..f9cd422 100644
--- a/common/cmd_ide.c
+++ b/common/cmd_ide.c
@@ -1529,6 +1529,9 @@ static void ide_reset (void)
 
ide_set_reset (1); /* assert reset */
 
+   /* the reset signal shall be asserted for et least 25 us */
+   udelay(25);
+
WATCHDOG_RESET();
 
 #ifdef CFG_PB_12V_ENABLE


-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] minor cs8900 driver clean up

[Guennadi Liakhovetski]

Remove a redundant register definition, clean up some coding style 
violations.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c
index 55ef346..458b517 100644
--- a/drivers/net/cs8900.c
+++ b/drivers/net/cs8900.c
@@ -65,14 +65,14 @@ static unsigned short get_reg_init_bus (int regno)
c = CS8900_BUS16_0;
 
CS8900_PPTR = regno;
-   return (unsigned short) CS8900_PDATA;
+   return CS8900_PDATA;
 }
 #endif
 
 static unsigned short get_reg (int regno)
 {
CS8900_PPTR = regno;
-   return (unsigned short) CS8900_PDATA;
+   return CS8900_PDATA;
 }
 
 
@@ -131,7 +131,7 @@ void cs8900_get_enetaddr (uchar * addr)
if (get_reg_init_bus (PP_ChipID) != 0x630e)
return;
eth_reset ();
-   if ((get_reg (PP_SelfST) & (PP_SelfSTAT_EEPROM | 
PP_SelfSTAT_EEPROM_OK)) ==
+   if ((get_reg (PP_SelfSTAT) & (PP_SelfSTAT_EEPROM | 
PP_SelfSTAT_EEPROM_OK)) ==
(PP_SelfSTAT_EEPROM | PP_SelfSTAT_EEPROM_OK)) {
 
/* Load the MAC from EEPROM */
@@ -168,7 +168,6 @@ void cs8900_get_enetaddr (uchar * addr)
debug ("### Set environment from HW MAC addr = 
\"%s\"\n", ethaddr);
setenv ("ethaddr", ethaddr);
}
-
}
 }
 
@@ -183,7 +182,6 @@ void eth_halt (void)
 
 int eth_init (bd_t * bd)
 {
-
/* verify chip id */
if (get_reg_init_bus (PP_ChipID) != 0x630e) {
printf ("CS8900 Ethernet chip not found?!\n");
@@ -201,7 +199,7 @@ int eth_init (bd_t * bd)
 }
 
 /* Get a data block via Ethernet */
-extern int eth_rx (void)
+int eth_rx (void)
 {
int i;
unsigned short rxlen;
@@ -233,7 +231,7 @@ extern int eth_rx (void)
 }
 
 /* Send a data block via Ethernet. */
-extern int eth_send (volatile void *packet, int length)
+int eth_send (volatile void *packet, int length)
 {
volatile unsigned short *addr;
int tmo;
@@ -281,7 +279,8 @@ retry:
 
 static void cs8900_e2prom_ready(void)
 {
-   while(get_reg(PP_SelfST) & SI_BUSY);
+   while (get_reg(PP_SelfSTAT) & SI_BUSY)
+   ;
 }
 
 /***/
diff --git a/drivers/net/cs8900.h b/drivers/net/cs8900.h
index f886d10..f9c32dd 100644
--- a/drivers/net/cs8900.h
+++ b/drivers/net/cs8900.h
@@ -243,7 +243,6 @@
 
 /* EEPROM Kram */
 #define SI_BUSY 0x0100
-#define PP_SelfST 0x0136   /*  Self State register */
 #define PP_EECMD 0x0040/*  NVR Interface Command register */
 #define PP_EEData 0x0042   /*  NVR Interface Data Register */
 #define EEPROM_WRITE_EN0x00F0

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] MX31ADS network and flash updates

[Guennadi Liakhovetski]

This patch depends on the previous two patches, introducing the 
CONFIG_FLASH_SPANSION_S29WS_N and CONFIG_ARP_TIMEOUT configuration 
options. It allows U-Boot to use buffered writes to the Spansion NOR flash 
installed on this board, and eliminates long delays in network transfers 
after the board startup.

Also modify flash layout to embed main and redundant environment blocks in 
the U-Boot image.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

diff --git a/board/mx31ads/mx31ads.c b/board/mx31ads/mx31ads.c
index 7c50c02..5a7d8c9 100644
--- a/board/mx31ads/mx31ads.c
+++ b/board/mx31ads/mx31ads.c
@@ -38,18 +38,18 @@ int dram_init (void)
 int board_init (void)
 {
int i;
-#if 0
+
/* CS0: Nor Flash */
/*
-* These are values from the RedBoot sources by Freescale. However,
-* under U-Boot with this configuration 32-bit accesses don't work,
-* lower 16 bits of data are read twice for each 32-bit read.
+* CS0L and CS0A values are from the RedBoot sources by Freescale
+* and are also equal to those used by Sascha Hauer for the Phytec
+* i.MX31 board. CS0U is just a slightly optimized hardware default:
+* the only non-zero field "Wait State Control" is set to half the
+* default value.
 */
-   __REG(CSCR_U(0)) = 0x23524E80;
-   __REG(CSCR_L(0)) = 0x1D03; /* WRAP bit (1) is suspicious here, but
-   * disabling it doesn't help either */
+   __REG(CSCR_U(0)) = 0x0f00;
+   __REG(CSCR_L(0)) = 0x1D03;
__REG(CSCR_A(0)) = 0x00720900;
-#endif
 
/* setup pins for UART1 */
mx31_gpio_mux(MUX_RXD1__UART1_RXD_MUX);
diff --git a/board/mx31ads/u-boot.lds b/board/mx31ads/u-boot.lds
index 1460adc..49713d4 100644
--- a/board/mx31ads/u-boot.lds
+++ b/board/mx31ads/u-boot.lds
@@ -34,7 +34,18 @@ SECTIONS
. = ALIGN(4);
.text  :
{
- cpu/arm1136/start.o   (.text)
+ /* WARNING - the following is hand-optimized to fit within*/
+ /* the sector layout of our flash chips!  XXX FIXME XXX   */
+
+ cpu/arm1136/start.o   (.text)
+ board/mx31ads/libmx31ads.a(.text)
+ lib_arm/libarm.a  (.text)
+ net/libnet.a  (.text)
+ drivers/mtd/libmtd.a  (.text)
+
+ . = DEFINED(env_offset) ? env_offset : .;
+ common/environment.o(.text)
+
  *(.text)
}
 
diff --git a/include/configs/mx31ads.h b/include/configs/mx31ads.h
index 77a9a83..9b72c89 100644
--- a/include/configs/mx31ads.h
+++ b/include/configs/mx31ads.h
@@ -72,7 +72,6 @@
 
 #include 
 
-#define CONFIG_CMD_MII
 #define CONFIG_CMD_PING
 
 #define CONFIG_BOOTDELAY   3
@@ -91,7 +90,19 @@
 
 #define CONFIG_DRIVER_CS8900   1
 #define CS8900_BASE0xb4020300
-#define CS8900_BUS16   1   /* the Linux driver does accesses as 
shorts */
+#define CS8900_BUS16   1   /* follow the Linux driver */
+
+/*
+ * The MX31ADS board seems to have a hardware "peculiarity" confirmed under
+ * U-Boot, RedBoot and Linux: the ethernet Rx signal is reaching the CS8900A
+ * controller inverted. The controller is capable of detecting and correcting
+ * this, but it needs 4 network packets for that. Which means, at startup, you
+ * will not receive answers to the first 4 packest, unless there have been some
+ * broadcasts on the network, or your board is on a hub. Reducing the ARP
+ * timeout from default 5 seconds to 200ms we speed up the initial TFTP
+ * transfer, should the user wish one, significantly.
+ */
+#define CONFIG_ARP_TIMEOUT 2UL
 
 /*
  * Miscellaneous configurable options
@@ -100,7 +111,7 @@
 #define CFG_PROMPT "=> "
 #define CFG_CBSIZE 256 /* Console I/O Buffer Size */
 /* Print Buffer Size */
-#define CFG_PBSIZE (CFG_CBSIZE+sizeof(CFG_PROMPT)+16)
+#define CFG_PBSIZE (CFG_CBSIZE + sizeof(CFG_PROMPT) + 16)
 #define CFG_MAXARGS16  /* max number of command args */
 #define CFG_BARGSIZE   CFG_CBSIZE  /* Boot Argument Buffer Size */
 
@@ -136,25 +147,29 @@
 #define CFG_MAX_FLASH_BANKS1   /* max number of memory banks */
 #define CFG_MAX_FLASH_SECT 262 /* max number of sectors on one 
chip */
 #define CFG_MONITOR_BASE   CFG_FLASH_BASE  /* Monitor at beginning of 
flash */
-#define CFG_MONITOR_LEN(128 * 1024)/* Reserve 128KiB */
+#define CFG_MONITOR_LEN(256 * 1024)/* Reserve 256KiB */
 
 #defineCFG_ENV_IS_IN_FLASH 1
 #define CFG_ENV_SECT_SIZE  (32 * 1024)
 #define CFG_ENV_SIZE   CFG_ENV_SECT_SIZE
+
+/* Address and size of Redundant Environment Sector*/
+#define CFG_ENV_OFFSET_REDUND  (CFG_ENV_OFFSET + CFG_ENV_SIZE)
+#define CFG_ENV_SIZE_REDUNDCFG_ENV_SIZE
+
 /* S29WS256N NOR flash has 4 32KiB small sectors at the b

[U-Boot-Users] [PATCH] cfi_flash: Support buffered writes on non-standard Spansion NOR flash

[Guennadi Liakhovetski]

Some NOR flash chip from Spansion, for example, the s29ws-n MirrorBit 
series require different addresses for buffered write commands. Define a 
configuration option to support buffered writes on those chips. A more 
elegant solution would be to automatically detect those chips by parsing 
their CFI records, but that would require introduction of a fixup table 
into the cfi_flash driver.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

diff --git a/README b/README
index a9663a3..9249064 100644
--- a/README
+++ b/README
@@ -2014,6 +2018,13 @@ Configuration Settings:
This option also enables the building of the cfi_flash driver
in the drivers directory
 
+- CFG_FLASH_USE_BUFFER_WRITE
+   Use buffered writes to flash.
+
+- CONFIG_FLASH_SPANSION_S29WS_N
+   s29ws-n MirrorBit flash has non-standard addresses for buffered
+   write commands.
+
 - CFG_FLASH_QUIET_TEST
If this option is defined, the common CFI flash doesn't
print it's warning upon not recognized FLASH banks. This
diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
index f04c72d..d7f73cb 100644
--- a/drivers/mtd/cfi_flash.c
+++ b/drivers/mtd/cfi_flash.c
@@ -828,25 +828,29 @@ static int flash_write_cfibuffer (flash_info_t * info, 
ulong dest, uchar * cp,
void *dst = map_physmem(dest, len, MAP_NOCACHE);
void *dst2 = dst;
int flag = 0;
+   uint offset = 0;
+   unsigned int shift;
 
switch (info->portwidth) {
case FLASH_CFI_8BIT:
-   cnt = len;
+   shift = 0;
break;
case FLASH_CFI_16BIT:
-   cnt = len >> 1;
+   shift = 1;
break;
case FLASH_CFI_32BIT:
-   cnt = len >> 2;
+   shift = 2;
break;
case FLASH_CFI_64BIT:
-   cnt = len >> 3;
+   shift = 3;
break;
default:
retcode = ERR_INVAL;
goto out_unmap;
}
 
+   cnt = len >> shift;
+
while ((cnt-- > 0) && (flag == 0)) {
switch (info->portwidth) {
case FLASH_CFI_8BIT:
@@ -890,23 +894,7 @@ static int flash_write_cfibuffer (flash_info_t * info, 
ulong dest, uchar * cp,
if (retcode == ERR_OK) {
/* reduce the number of loops by the width of
 * the port */
-   switch (info->portwidth) {
-   case FLASH_CFI_8BIT:
-   cnt = len;
-   break;
-   case FLASH_CFI_16BIT:
-   cnt = len >> 1;
-   break;
-   case FLASH_CFI_32BIT:
-   cnt = len >> 2;
-   break;
-   case FLASH_CFI_64BIT:
-   cnt = len >> 3;
-   break;
-   default:
-   retcode = ERR_INVAL;
-   goto out_unmap;
-   }
+   cnt = len >> shift;
flash_write_cmd (info, sector, 0, (uchar) cnt - 1);
while (cnt-- > 0) {
switch (info->portwidth) {
@@ -943,36 +931,34 @@ static int flash_write_cfibuffer (flash_info_t * info, 
ulong dest, uchar * cp,
case CFI_CMDSET_AMD_STANDARD:
case CFI_CMDSET_AMD_EXTENDED:
flash_unlock_seq(info,0);
-   flash_write_cmd (info, sector, 0, AMD_CMD_WRITE_TO_BUFFER);
+
+#ifdef CONFIG_FLASH_SPANSION_S29WS_N
+   offset = ((unsigned long)dst - info->start[sector]) >> shift;
+#endif
+   flash_write_cmd(info, sector, offset, AMD_CMD_WRITE_TO_BUFFER);
+   cnt = len >> shift;
+   flash_write_cmd(info, sector, offset, (uchar)cnt - 1);
 
switch (info->portwidth) {
case FLASH_CFI_8BIT:
-   cnt = len;
-   flash_write_cmd (info, sector, 0,  (uchar) cnt - 1);
while (cnt-- > 0) {
flash_write8(flash_read8(src), dst);
src += 1, dst += 1;
}
break;
case FLASH_CFI_16BIT:
-   cnt = len >> 1;
-   flash_write_cmd (info, sector, 0,  (uchar) cnt - 1);
while (cnt-- > 0) {
flash_write16(flash_read16(src), dst);
src += 2, dst += 2;
}
break;
case FLASH_CFI_32BIT:
-   cnt = len >> 2;
-   flash_write_cmd (info, sector, 0,  

[U-Boot-Users] [PATCH] net: make ARP timeout configurable

[Guennadi Liakhovetski]

Currently the timeout waiting for an ARP reply is hard set to 5 seconds. 
On i.MX31ADS due to a hardware "strangeness" up to four first IP packets 
to the boards get lost, which typically are ARP replies. By configuring 
the timeout to a lower value we significantly improve the first network 
transfer time on this board. The timeout is specified in deciseconds, 
because it has to be converted to hardware ticks, and CFG_HZ ranges from 
900 to 2700 on different boards.

Signed-off-by: Guennadi Liakhovetski <[EMAIL PROTECTED]>

---

diff --git a/README b/README
index a9663a3..9249064 100644
--- a/README
+++ b/README
@@ -1547,6 +1547,10 @@ The following options need to be configured:
before giving up the operation. If not defined, a
default value of 5 is used.
 
+   CONFIG_ARP_TIMEOUT
+
+   Timeout waiting for an ARP reply in deciseconds.
+
 - Command Interpreter:
CONFIG_AUTO_COMPLETE
 
diff --git a/net/net.c b/net/net.c
index 44feee2..2128bd4 100644
--- a/net/net.c
+++ b/net/net.c
@@ -94,11 +94,16 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#define ARP_TIMEOUT5UL /* Seconds before trying ARP 
again */
+#ifndefCONFIG_ARP_TIMEOUT
+# define ARP_TIMEOUT   50UL/* Deciseconds before trying ARP again 
*/
+#else
+# define ARP_TIMEOUT   CONFIG_ARP_TIMEOUT
+#endif
+
 #ifndefCONFIG_NET_RETRY_COUNT
-# define ARP_TIMEOUT_COUNT 5   /* # of timeouts before giving 
up  */
+# define ARP_TIMEOUT_COUNT 5   /* # of timeouts before giving up  */
 #else
-# define ARP_TIMEOUT_COUNT  (CONFIG_NET_RETRY_COUNT)
+# define ARP_TIMEOUT_COUNT CONFIG_NET_RETRY_COUNT
 #endif
 
 #if 0
@@ -129,7 +134,7 @@ uchar   NetOurEther[6]; /* Our ethernet 
address */
 uchar  NetServerEther[6] = /* Boot server enet address 
*/
{ 0, 0, 0, 0, 0, 0 };
 IPaddr_t   NetOurIP;   /* Our IP addr (0 = unknown)
*/
-IPaddr_t   NetServerIP;/* Our IP addr (0 = unknown)
*/
+IPaddr_t   NetServerIP;/* Server IP addr (0 = unknown) 
*/
 volatile uchar *NetRxPkt;  /* Current receive packet   
*/
 intNetRxPktLen;/* Current rx packet length 
*/
 unsigned   NetIPID;/* IP packet ID 
*/
@@ -253,7 +258,7 @@ void ArpTimeoutCheck(void)
t = get_timer(0);
 
/* check for arp timeout */
-   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ) {
+   if ((t - NetArpWaitTimerStart) > ARP_TIMEOUT * CFG_HZ / 10) {
NetArpWaitTry++;
 
if (NetArpWaitTry >= ARP_TIMEOUT_COUNT) {
@@ -494,7 +499,7 @@ restart:
 *  Check the ethernet for a new packet.  The ethernet
 *  receive routine will process it.
 */
-   eth_rx();
+   eth_rx();
 
/*
 *  Abort if ctrl-c was pressed.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] New Image format: headers hashes.

[Luigi 'Comio' Mantellini]

Hi Bartlomiej,

thank you for the answers. See my comments below.

On gio, 2008-04-03 at 11:31 +0200, Bartlomiej Sieka wrote:
> Hi Luigi,
> 
> You wrote:
> > On gio, 2008-03-27 at 12:13 +0100, Luigi 'Comio' Mantellini wrote:
> > 
> >>
> >> - The mkimage calculate hashes before that timestamp is added to
> >> image.
> >> is it better exchange the order of these operations (before add
> >> timestamp then calculate hashes)?
> 
> Please see below.
> 
> 
> >> - Does the new format assure that the complete image (_all_ sections,
> >> images, ramdisks, configurations, blobs) are covered by at least an
> >> hash
> >> value?
> 
> Currently, the hash calculation is performed only over binary data of
> component images (kernel, ramdisk, fdt, etc). In particular, timestamp
> information is not protected by the hash, so the ordering of hash
> calculation and timestamp addition doesn't matter.
> 

Ok. I agree

> 
>  >> should mkimage add at least a hash value for the root node?
> 
> Adding a hash value for the root node only will not be easy. An easier
> approach I think would be to add a hash value for the entire FIT image.
> 
Ok.

> 
> > In addition to the previous question:
> > 
> > - I'm interested to calculate and store from U-boot code but the 
> > fit_image_set_hashes is compiled only when the USE_HOSTCC symbol is 
> > defined. Why? Is there a particular reason?
> 
> There's no need for the function that you mention in current U-Boot
> code, that's the reason why it's under USE_HOSTCC. If you're planning on
> adding code to U-Boot that will use fit_image_set_hashes(), then just
> remove the USE_HOSTCC #define's from around this function.
> 
> 
>  > I think that a lot of
>  > useful functions (ifdef-ed by USE_HOSTCC) should be available also in
>  > u-boot code, anyway the linker will drop the unused functions.
> 
> I don't think the linker will do this in case of U-Boot:
> 
> With the following patch:
> 
> diff --git a/common/image.c b/common/image.c
> index f04826a..850a9e4 100644
> --- a/common/image.c
> +++ b/common/image.c
> @@ -1950,7 +1950,6 @@ static int calculate_hash (const void *data, int 
> data_len,
>   return 0;
>   }
> 
> -#ifdef USE_HOSTCC
>   /**
>* fit_set_hashes - process FIT component image nodes and calculate hashes
>* @fit: pointer to the FIT format image header
> @@ -2119,7 +2118,6 @@ int fit_image_hash_set_value (void *fit, int 
> noffset, uint
> 
>   return 0;
>   }
> -#endif /* USE_HOSTCC */
> 
>   /**
>* fit_image_check_hashes - verify data intergity
> 
> 
> I see:
> 
> $ ${CROSS_COMPILE}nm -S common/image.o | grep fit_image_set_hashes
> 155c 01f4 T fit_image_set_hashes
> $ ${CROSS_COMPILE}nm -S common/libcommon.a | grep fit_image_set_hashes
> 155c 01f4 T fit_image_set_hashes
> $ ${CROSS_COMPILE}nm -S u-boot | grep fit_image_set_hashes
> fffb1924 01f4 T fit_image_set_hashes
> 
> 
> There's also size increase with the patch applied:
> textdata bss dec hex filename
>   284615   15556  357120  657291   a078b ./u-boot
> 
> textdata bss dec hex filename
>   285875   15572  357120  658567   a0c87 u-boot
> 
> 

Before to make this test, you should strip the elf-file in order to omit
the non-used code. Alternatively you should check the size of the
u-boot.bin file (that is striped by definition).

Best regards.

luigi

> HTH,
> Bartlomiej
-- 
 __   Luigi Mantellini
   .'__'. R&D - Software
  (.'  '.)Industrie Dial Face S.p.A.
  ( :==: )Via Canzo, 4
  ('.__.')20068 Peschiera Borromeo (MI), Italy
   '.__.' Tel.: +39 02 5167 2813
  Fax:  +39 02 5167 2459
Ind.  Dial Face   Email: [EMAIL PROTECTED]
www.idf-hit.com   GPG fingerprint: 3DD1 7B71 FBDF 6376 1B4A
   B003 175F E979 907E 1650





-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5

[Wolfgang Denk]

In message <[EMAIL PROTECTED]> you wrote:
> Hi, 
> 
> Subject: [PATCH] disable CONFIG_ZERO_BOOTDELAY for lwmon5
> 
> Change configuartion for lwmon5 board.
> 
> Signed-off-by: Sascha Laue <[EMAIL PROTECTED]>
> ---
>  include/configs/lwmon5.h |2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/lwmon5.h b/include/configs/lwmon5.h
> index ced7ba6..ff4bc80 100644
> --- a/include/configs/lwmon5.h
> +++ b/include/configs/lwmon5.h
> @@ -434,7 +434,9 @@
>  #define CONFIG_CMDLINE_EDITING   1   /* add command line history 
> */
>  #define CONFIG_LOOPW1   /* enable loopw command */
>  #define CONFIG_MX_CYCLIC1   /* enable mdc/mwc commands  */
> +#if 0   /* enable only for debug */
>  #define CONFIG_ZERO_BOOTDELAY_CHECK  /* check for keypress on bootdelay==0 */
> +#endif
>  #define CONFIG_VERSION_VARIABLE 1/* include version env variable */

Please don't add dead code. If you want to remove that line, then
please remove it. No "#if 0", please.

Viele Grüße,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]
There are three ways to get something done:
(1) Do it yourself.
(2) Hire someone to do it for you.
(3) Forbid your kids to do it.

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] New Image format: headers hashes.

[Bartlomiej Sieka]

Hi Luigi,

You wrote:
> On gio, 2008-03-27 at 12:13 +0100, Luigi 'Comio' Mantellini wrote:
> 
>>
>> - The mkimage calculate hashes before that timestamp is added to
>> image.
>> is it better exchange the order of these operations (before add
>> timestamp then calculate hashes)?

Please see below.


>> - Does the new format assure that the complete image (_all_ sections,
>> images, ramdisks, configurations, blobs) are covered by at least an
>> hash
>> value?

Currently, the hash calculation is performed only over binary data of
component images (kernel, ramdisk, fdt, etc). In particular, timestamp
information is not protected by the hash, so the ordering of hash
calculation and timestamp addition doesn't matter.


 >> should mkimage add at least a hash value for the root node?

Adding a hash value for the root node only will not be easy. An easier
approach I think would be to add a hash value for the entire FIT image.


> In addition to the previous question:
> 
> - I'm interested to calculate and store from U-boot code but the 
> fit_image_set_hashes is compiled only when the USE_HOSTCC symbol is 
> defined. Why? Is there a particular reason?

There's no need for the function that you mention in current U-Boot
code, that's the reason why it's under USE_HOSTCC. If you're planning on
adding code to U-Boot that will use fit_image_set_hashes(), then just
remove the USE_HOSTCC #define's from around this function.


 > I think that a lot of
 > useful functions (ifdef-ed by USE_HOSTCC) should be available also in
 > u-boot code, anyway the linker will drop the unused functions.

I don't think the linker will do this in case of U-Boot:

With the following patch:

diff --git a/common/image.c b/common/image.c
index f04826a..850a9e4 100644
--- a/common/image.c
+++ b/common/image.c
@@ -1950,7 +1950,6 @@ static int calculate_hash (const void *data, int 
data_len,
return 0;
  }

-#ifdef USE_HOSTCC
  /**
   * fit_set_hashes - process FIT component image nodes and calculate hashes
   * @fit: pointer to the FIT format image header
@@ -2119,7 +2118,6 @@ int fit_image_hash_set_value (void *fit, int 
noffset, uint

return 0;
  }
-#endif /* USE_HOSTCC */

  /**
   * fit_image_check_hashes - verify data intergity


I see:

$ ${CROSS_COMPILE}nm -S common/image.o | grep fit_image_set_hashes
155c 01f4 T fit_image_set_hashes
$ ${CROSS_COMPILE}nm -S common/libcommon.a | grep fit_image_set_hashes
155c 01f4 T fit_image_set_hashes
$ ${CROSS_COMPILE}nm -S u-boot | grep fit_image_set_hashes
fffb1924 01f4 T fit_image_set_hashes


There's also size increase with the patch applied:
textdata bss dec hex filename
  284615   15556  357120  657291   a078b ./u-boot

textdata bss dec hex filename
  285875   15572  357120  658567   a0c87 u-boot


HTH,
Bartlomiej

-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] newcomer

[王飞王飞]

hi experters:
 
   I'm a newcomer in U-boot, I want to understand the U-boot,but i don't know 
how to start!Anyone can help me ?
Thanks,
   Peter wang
 
_
MSN 中文网，最新时尚生活资讯，白领聚集门户。
http://cn.msn.com-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] help - u-boot header & compression

[Luigi 'Comio' Mantellini]

Hi Avinash,

You should check the nand_spl and openand_ipl directories to see
examples how to start u-boot from a not bootable memory (like nand).

Best regards,

luigi


On mer, 2008-04-02 at 14:26 -0700, Avinash Vijayvergia wrote:
> Hi Luig
> 
> Thank you for your mail. I actually have two questions:
>  1. What is the format of u-boot header? Can you refer me to some
> documentation?
>  2. I want to compress u-boot. I have picked up nrv2b.c from
> Koders.com which is in use by many people :). I couldn't
> understand how it does encoding and decoding. I wanted to use
> this algo and do compression and decompression of my binaries.
> You can search google for nrv2b.c and you will get it from
> Koder.com.
> Thanks for your help
> Avinash
> 
> - Original Message 
> From: Luigi 'Comio' Mantellini <[EMAIL PROTECTED]>
> To: Avinash Vijayvergia <[EMAIL PROTECTED]>
> Cc: u-boot-users@lists.sourceforge.net
> Sent: Wednesday, April 2, 2008 2:19:36 PM
> Subject: Re: [U-Boot-Users] help - u-boot header & compression
> 
> Hi Avinash,
> 
> I'm not undestanding if you want compress the U-Boot code or if you
> want compress the uImage image.
> Anyway, I don't know the nrv2b algorithm, can you give me any links?
> 
> thanks,
> 
> luigi
> 
> Il giorno mer, 02/04/2008 alle 13.54 -0700, Avinash Vijayvergia ha
> scritto: 
> > Hi All
> > 
> > I am working on AT91SAM9260 ARM9 core. I have very small flash
> > memory and hence need to compress the u-boot. I have picked nrv2b
> > algo for that. Can someone suggest me how to put the compression
> > info in the u-boot header. I want to know the details of u-boot
> > header too.
> > 
> > Thanks
> > Avinash
> > 
> > 
> > 
> > 
> > You rock. That's why Blockbuster's offering you one month of
> > Blockbuster Total Access, No Cost. 
> > -
> > Check out the new SourceForge.net Marketplace.
> > It's the best place to buy or sell services for
> > just about anything Open Source.
> > http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> > ___ U-Boot-Users mailing list 
> > U-Boot-Users@lists.sourceforge.net 
> > https://lists.sourceforge.net/lists/listinfo/u-boot-users
> 
> Ing. Luigi Mantellini
> Industrie Dial Face S.p.A.
> Via Canzo, 4 
> 20068 Peschiera Borromeo (MI)
> Tel.: +39 02 5167 2813
> Fax: +39 02 5167 2459
> E-mail:
> [EMAIL PROTECTED]
> 
> 
> 
> 
> 
> 
> 
> __
> You rock. That's why Blockbuster's offering you one month of
> Blockbuster Total Access, No Cost.
-- 
 __   Luigi Mantellini
   .'__'. R&D - Software
  (.'  '.)Industrie Dial Face S.p.A.
  ( :==: )Via Canzo, 4
  ('.__.')20068 Peschiera Borromeo (MI), Italy
   '.__.' Tel.: +39 02 5167 2813
  Fax:  +39 02 5167 2459
Ind.  Dial Face   Email: [EMAIL PROTECTED]
www.idf-hit.com   GPG fingerprint: 3DD1 7B71 FBDF 6376 1B4A
   B003 175F E979 907E 1650





-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] mpc83xx: Fix the SATA clock setting of 837x targets

[Dave Liu]

Currently the SATA controller clock is configured as CSB clock,
usually the CSB clock is 400/333/266MHz.

However, The SATA IP block is only guaranteed to operate up to
200 MHz as stated in the HW spec.

The bug is reported by Joe D'Abbraccio <[EMAIL PROTECTED]>

This patch makes the SATA clock as half of CSB clock.

Signed-off-by: Dave Liu <[EMAIL PROTECTED]>
---
 include/configs/MPC837XEMDS.h |2 +-
 include/configs/MPC837XERDB.h |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/configs/MPC837XEMDS.h b/include/configs/MPC837XEMDS.h
index 7c4e76e..7fc0f7e 100644
--- a/include/configs/MPC837XEMDS.h
+++ b/include/configs/MPC837XEMDS.h
@@ -96,7 +96,7 @@
  */
 #define CFG_SCCR_TSEC1CM   1   /* CSB:eTSEC1 = 1:1 */
 #define CFG_SCCR_TSEC2CM   1   /* CSB:eTSEC2 = 1:1 */
-#define CFG_SCCR_SATACMSCCR_SATACM_1   /* CSB:SATA[0:3] = 1:1 
*/
+#define CFG_SCCR_SATACMSCCR_SATACM_2   /* CSB:SATA[0:3] = 2:1 
*/
 
 /*
  * System IO Config
diff --git a/include/configs/MPC837XERDB.h b/include/configs/MPC837XERDB.h
index eaac525..c698ff8 100644
--- a/include/configs/MPC837XERDB.h
+++ b/include/configs/MPC837XERDB.h
@@ -108,7 +108,7 @@
 /* System Clock Configuration Register */
 #define CFG_SCCR_TSEC1CM   1   /* eTSEC1 clock mode (0-3) */
 #define CFG_SCCR_TSEC2CM   1   /* eTSEC2 clock mode (0-3) */
-#define CFG_SCCR_SATACMSCCR_SATACM_1   /* SATA1-4 clock mode 
(0-3) */
+#define CFG_SCCR_SATACMSCCR_SATACM_2   /* SATA1-4 clock mode 
(0-3) */
 
 /*
  * System IO Config
-- 
1.5.4.rc4




-
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users