Re: [PATCH] Added a tftp command

On Tuesday 31 March 2009 18:44:21 kevin.morfitt <at> fearnside-systems.co.uk wrote:
> Adds a "tftp" command that gets a specified file from a TFTP Server and
> stores it in RAM at a specified RAM address. Most of the code already
> exists in board-specific form (eg in board/hymod) but this patch
> extracts it and makes it available as a standard u-boot command.

your patch is horribly word wrapped.  ignoring that, how is this any different 
from the "tftpboot" command ?  i use "t <addr> <file>" to load arbitrary files 
via tftp to arbitrary addresses all the time.
-mike

_______________________________________________
U-Boot mailing list
U-Boot <at> lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [PATCH] Added a tftp command

On Mar 31, 2009, at 5:44 PM, kevin.morfitt <at> fearnside-systems.co.uk  
wrote:

> Adds a "tftp" command that gets a specified file from a TFTP Server  
> and
> stores it in RAM at a specified RAM address. Most of the code already
> exists in board-specific form (eg in board/hymod) but this patch
> extracts it and makes it available as a standard u-boot command.
>
> Signed-off-by: Kevin Morfitt <kevin.morfitt <at> fearnside-systems.co.uk>
> ---
> common/cmd_tftp.c            |   59
> ++++++++++++++++++++++++++++++++++++++++++
> doc/README.tftp              |   29 ++++++++++++++++++++
> include/config_cmd_all.h     |    1 +
> include/config_cmd_default.h |    1 +
> 4 files changed, 90 insertions(+), 0 deletions(-)
> create mode 100755 common/cmd_tftp.c
> create mode 100755 doc/README.tftp

I'm quite confused.  We already have support for this.  I'm pretty  
sure I use it almost daily.

- k

[PATCH] mpc/sbc8641d: Add missing board_lmb_reserves

We're missing the board_lmb_reserve definitions that allow
cpu_mp_lmb_reserve to be called; this means that Linux
is free to reallocate reserved pages.  Linux currently boots
because we're getting lucky - the page we've reserved is
high enough in memory that it isn't allocated by Linux
while we still need it to be in existence.

Signed-off-by: Becky Bruce <beckyb <at> kernel.crashing.org>
---
 board/freescale/mpc8641hpcn/mpc8641hpcn.c |    9 +++++++++
 board/sbc8641d/sbc8641d.c                 |    9 +++++++++
 2 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/board/freescale/mpc8641hpcn/mpc8641hpcn.c b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
index c94fc3f..1e35dfa 100644
--- a/board/freescale/mpc8641hpcn/mpc8641hpcn.c
+++ b/board/freescale/mpc8641hpcn/mpc8641hpcn.c
 <at>  <at>  -374,3 +374,12  <at>  <at>  void board_reset(void)
 	while (1)
 		;
 }
+
+#if (CONFIG_NUM_CPUS > 1)
+extern void cpu_mp_lmb_reserve(struct lmb *lmb);
+
+void board_lmb_reserve(struct lmb *lmb)
+{
+	cpu_mp_lmb_reserve(lmb);
+}
+#endif

Re: [PATCH] Added a tftp command

Mike Frysinger wrote:
> On Tuesday 31 March 2009 18:44:21 kevin.morfitt <at> fearnside-systems.co.uk wrote:
>   
>> Adds a "tftp" command that gets a specified file from a TFTP Server and
>> stores it in RAM at a specified RAM address. Most of the code already
>> exists in board-specific form (eg in board/hymod) but this patch
>> extracts it and makes it available as a standard u-boot command.
>>     
>
> your patch is horribly word wrapped.  ignoring that, how is this any different 
> from the "tftpboot" command ?  i use "t <addr> <file>" to load arbitrary files 
> via tftp to arbitrary addresses all the time.
> -mike
>   
"tftpboot" loads the file then if "autostart" is "yes" it automatically 
boots from the load address. The patch just loads the file then stops. 
When programming a board I use it to copy images across before I program 
them into flash. In effect, it does the same thing as setting 
"autostart" to "off" then using "tftpboot".

I'm new to creating patches which I think explains the horrible word 
wrapping.

Re: [PATCH] Added a tftp command

Kumar Gala wrote:
>
> On Mar 31, 2009, at 5:44 PM, kevin.morfitt <at> fearnside-systems.co.uk wrote:
>
>> Adds a "tftp" command that gets a specified file from a TFTP Server and
>> stores it in RAM at a specified RAM address. Most of the code already
>> exists in board-specific form (eg in board/hymod) but this patch
>> extracts it and makes it available as a standard u-boot command.
>>
>> Signed-off-by: Kevin Morfitt <kevin.morfitt <at> fearnside-systems.co.uk>
>> ---
>> common/cmd_tftp.c            |   59
>> ++++++++++++++++++++++++++++++++++++++++++
>> doc/README.tftp              |   29 ++++++++++++++++++++
>> include/config_cmd_all.h     |    1 +
>> include/config_cmd_default.h |    1 +
>> 4 files changed, 90 insertions(+), 0 deletions(-)
>> create mode 100755 common/cmd_tftp.c
>> create mode 100755 doc/README.tftp
>
> I'm quite confused.  We already have support for this.  I'm pretty 
> sure I use it almost daily.
>
> - k
So do I but I'm working with a u-boot my customer has modified to add it 
as a board-specific command. When I tried to use it on a new board with 
the latest u-boot  I found it wasn't a standard u-boot command. It's 
almost identical to "tftpboot" except that if "autostart" is set to 
"yes" "tftpboot" automatically boots from the file after it's downloaded 
it but "tftp" doesn't. I use it to copy images across then manually 

Re: [PATCH] Added a tftp command

On Tuesday 31 March 2009 19:40:27 kevin.morfitt <at> fearnside-systems.co.uk wrote:
> Mike Frysinger wrote:
> > On Tuesday 31 March 2009 18:44:21 kevin.morfitt wrote:
> >> Adds a "tftp" command that gets a specified file from a TFTP Server and
> >> stores it in RAM at a specified RAM address. Most of the code already
> >> exists in board-specific form (eg in board/hymod) but this patch
> >> extracts it and makes it available as a standard u-boot command.
> >
> > your patch is horribly word wrapped.  ignoring that, how is this any
> > different from the "tftpboot" command ?  i use "t <addr> <file>" to load
> > arbitrary files via tftp to arbitrary addresses all the time.
>
> "tftpboot" loads the file then if "autostart" is "yes" it automatically
> boots from the load address. The patch just loads the file then stops.
> When programming a board I use it to copy images across before I program
> them into flash. In effect, it does the same thing as setting
> "autostart" to "off" then using "tftpboot".

so dont set autostart ?
-mike

_______________________________________________
U-Boot mailing list
U-Boot <at> lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[PATCH] Add support for Faraday Ethernet IP FTMAC100

Dear Jean-Christophe PLAGNIOL-VILLARD,

On Tue Mar 31 10:51:31 CEST 2009, you wrote:
> > This patch adds support for Faraday Technology Ethernet IP - FTMAC100
> >
> Do you have any plan to add a SOC and a board?
Yes, I need some time to cleanup the source though.

I think to put some drivers first and the board might be added 
later in next merge window

Best Regards,

Po-Yu Chuang (莊博宇)
=== Faraday Technology Corporation, Headquarters TW ===
= 智原科技股份有限公司 
= Dept.: CTD/SD/SSD
= Tel  : 886-3-578-7888 Ext:8818
= Fax  : 886-3-666-3958
= Email: ratbert <at> faraday-tech.com
= WWW  : www.faraday-tech.com 
=======================================================

********************* Confidentiality Notice ************************
This electronic message and any attachments may contain
confidential and legally privileged information or
information which is otherwise protected from disclosure.
If you are not the intended recipient,please do not disclose
the contents, either in whole or in part, to anyone,and
immediately delete the message and any attachments from

(no subject)

hello:
      recently,i'm using the new edition of u-boot,the edition u-boot-2009-rc1.and i met some
troubles.so i need your help.
      i want to use the usbkeyboard as a console,how can i add this function to the u-boot?
because the interrupt is masked in the u-boot.and i've read your document,you said i need a script.but i
don't know how to write that,can you offer me somehelp.by the way,the mcu of my board is
s3c2410.
                                                                                                
yours:Iverson
David
                                                                                                       2009.4.1

      ___________________________________________________________ 
  好玩贺卡等你发，邮箱贺卡全新上线！ 
http://card.mail.cn.yahoo.com/

_______________________________________________
U-Boot mailing list
U-Boot <at> lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[PATCH] Add support for Faraday Ethernet IP FTMAC100

Dear Mike Frysinger,

On Tue Mar 31 11:18:44 CEST 2009, you wrote:
> > This patch adds support for Faraday Technology Ethernet IP - FTMAC100
> kind of redundant statement considering the summary ...

Sorry I have not realized the rules here.
It looks like things here will be put into CHANGELOG.
I will remove this (or put some more meaningful descriptions) 
next time.

> > Signed-off-by: Po-Yu Chuang <ratbert at faraday-tech.com>
> you should really be using git to generate patches and send e-mails.  also, 
> signed-off-by (and similar tags) use real addresses, not munged ones.

OK, I will fix it in the new patches. Thank you.

Best Regards,

Po-Yu Chuang (莊博宇)
=== Faraday Technology Corporation, Headquarters TW ===
= 智原科技股份有限公司 
= Dept.: CTD/SD/SSD
= Tel  : 886-3-578-7888 Ext:8818
= Fax  : 886-3-666-3958
= Email: ratbert <at> faraday-tech.com
= WWW  : www.faraday-tech.com 
=======================================================

********************* Confidentiality Notice ************************

[PATCH] Add support for Faraday RTC IP FTRTC

Dear Wolfgang Denk,

I am a newbie and I have not realized all the rules, so I 
have some questions.

On Tue Mar 31 12:24:03 CEST 2009, you wrote:
> > +static unsigned int	rtc	= CONFIG_SYS_RTC_BASE;
> 
> Please use a proper C structure to describe the RTC; do not use base
> address plus offsets.

Do you mean U-Boot prefers to use structure to describe 
registers? What if there were large gaps between registers?

For example, in our ethernet IP FTMAC100,  there is nothing
in 0x34 - 0x84, 0xa0 - 0xc0. Should I use structure to
describe it?

I thought that placing a lot of paddings in a structure is ulgy. 
How do you think? Is there a guideline about this?

Also in drivers/rtc/pl031.c, the RTC_WRITE_REG and 
RTC_READ_REG use base + offset.

> outl() is not portable across architectures?
> inl() is not portable across architectures?

Sorry I don't understand.

There are outl() and inl() in most architectures except blackfin,