date:20080710

On 23:49 Wed 09 Jul , Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
  
  Le jeudi 22 mai 2008 Ã  00:15 +0200, Stelian Pop a Ã©crit :
  AT91 RSTC registers are battery-backuped, so their values
  are not reset across power cycles. One of those registers,
  the AT91_RSTC_MR register, is being modified by U-Boot, in
  the ethernet initialisation routine, to generate a 500ms
  user reset.
  [...]
  
  It seems that the patch above has not made it in the mainline. Since
  this can cause some nasty boot problems, it would be nice to have this
  upstream for the next stable version.
 
 Jean-Christophe, is this on your queue?
 
I've send a pull requeset for this
and you applied in your tree.

Best Regards,
J.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] pci: Move PCI device configuration check into a separate weak function

On Thursday 10 July 2008, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
  Yes, I think we should put it into the repo now.
 
  Wolfgang, can you please pick my patch up directly?

 I can, if you tell me which one exactly.

The one from this email thread:

[PATCH] pci: Move PCI device configuration check into a separate weak function

From 08.07.2008

Thanks.

Best regards,
Stefan

=
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]
=

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 printf format warnings now visible with the updated format check

This patch fixes ppc4xx related printf format warning. Those warnings are
now visible since patch dc4b0b38d4aadf08826f6c31270f1eecd27964fd
[Fix printf errors.] by Andrew Klossner has been applied. Thanks, this is
really helpful.

Signed-off-by: Stefan Roese [EMAIL PROTECTED]
---
 board/amcc/katmai/cmd_katmai.c |4 ++--
 board/esd/du440/du440.c|4 ++--
 board/lwmon5/sdram.c   |2 +-
 board/zeus/zeus.c  |2 +-
 cpu/ppc4xx/44x_spd_ddr2.c  |   10 +-
 cpu/ppc4xx/4xx_enet.c  |2 +-
 cpu/ppc4xx/denali_spd_ddr2.c   |8 
 cpu/ppc4xx/tlb.c   |4 ++--
 cpu/ppc4xx/traps.c |   26 +-
 9 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/board/amcc/katmai/cmd_katmai.c b/board/amcc/katmai/cmd_katmai.c
index 439be4f..703d225 100644
--- a/board/amcc/katmai/cmd_katmai.c
+++ b/board/amcc/katmai/cmd_katmai.c
@@ -176,7 +176,7 @@ static int do_bootstrap(cmd_tbl_t *cmdtp, int flag, int 
argc, char *argv[])
 #endif
}
 #ifdef DEBUG
-   printf( pin strap0 to write in i2c  = %x\n, data);
+   printf( pin strap0 to write in i2c  = %lx\n, data);
 #endif /* DEBUG */
 
if (i2c_write(chip, 0, 1, (uchar *)data, 4) != 0)
@@ -201,7 +201,7 @@ static int do_bootstrap(cmd_tbl_t *cmdtp, int flag, int 
argc, char *argv[])
data |= 0x05A5;
 
 #ifdef DEBUG
-   printf( pin strap1 to write in i2c  = %x\n, data);
+   printf( pin strap1 to write in i2c  = %lx\n, data);
 #endif /* DEBUG */
 
udelay(1000);
diff --git a/board/esd/du440/du440.c b/board/esd/du440/du440.c
index 3dbb2e1..8765cc1 100644
--- a/board/esd/du440/du440.c
+++ b/board/esd/du440/du440.c
@@ -956,9 +956,9 @@ int do_time(cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
ret = run_command (cmd, 0);
end = get_ticks();
 
-   printf(ticks=%d\n, (ulong)(end - start));
+   printf(ticks=%ld\n, (ulong)(end - start));
us = (ulong)((1000L * (end - start)) / (get_tbclk() / 1000));
-   printf(usec=%d\n, us);
+   printf(usec=%ld\n, us);
 
return ret;
 }
diff --git a/board/lwmon5/sdram.c b/board/lwmon5/sdram.c
index 0a13831..189e824 100644
--- a/board/lwmon5/sdram.c
+++ b/board/lwmon5/sdram.c
@@ -84,7 +84,7 @@ void board_add_ram_info(int use_default)
puts( (ECC not);
 
get_sys_info(board_cfg);
-   printf( enabled, %d MHz, (board_cfg.freqPLB * 2) / 100);
+   printf( enabled, %ld MHz, (board_cfg.freqPLB * 2) / 100);
 
mfsdram(DDR0_03, val);
val = DDR0_03_CASLAT_DECODE(val);
diff --git a/board/zeus/zeus.c b/board/zeus/zeus.c
index 0113d48..33d971a 100644
--- a/board/zeus/zeus.c
+++ b/board/zeus/zeus.c
@@ -280,7 +280,7 @@ static int restore_default(void)
} else {
crc = crc32(0, (u8 *)(buf + 4), FACTORY_RESET_ENV_SIZE - 4);
if (crc != *(u32 *)buf) {
-   printf(ERROR: crc mismatch %08lx %08lx\n, crc, *(u32 
*)buf);
+   printf(ERROR: crc mismatch %08x %08x\n, crc, *(u32 
*)buf);
return -1;
}
 
diff --git a/cpu/ppc4xx/44x_spd_ddr2.c b/cpu/ppc4xx/44x_spd_ddr2.c
index 9a5340c..a27e276 100644
--- a/cpu/ppc4xx/44x_spd_ddr2.c
+++ b/cpu/ppc4xx/44x_spd_ddr2.c
@@ -378,7 +378,7 @@ static phys_size_t sdram_memsize(void)
mem_size+=4096;
break;
default:
-   printf(WARNING: Unsupported bank size 
(SDSZ=0x%x)!\n
+   printf(WARNING: Unsupported bank size 
(SDSZ=0x%lx)!\n
   , sdsz);
mem_size=0;
break;
@@ -860,8 +860,8 @@ static void check_rank_number(unsigned long *dimm_populated,
 
 
if (dimm_rank  MAXRANKS) {
-   printf(ERROR: DRAM DIMM detected with %d ranks 
in 
-  slot %d is not supported.\n, 
dimm_rank, dimm_num);
+   printf(ERROR: DRAM DIMM detected with %lu 
ranks in 
+  slot %lu is not supported.\n, 
dimm_rank, dimm_num);
printf(Only %d ranks are supported for all 
DIMM.\n, MAXRANKS);
printf(Replace the DIMM module with a 
supported DIMM.\n\n);
spd_ddr_init_hang ();
@@ -1062,7 +1062,7 @@ static void program_copt1(unsigned long *dimm_populated,
dimm_32bit = TRUE;
break;
default:
-   printf(WARNING: Detected a DIMM with a data 
width of %d bits.\n,
+   printf(WARNING: Detected a DIMM with a data 
width of %lu

[U-Boot-Users] [PATCH] rtc: Fix printf format warning in m41t60.c

Signed-off-by: Stefan Roese [EMAIL PROTECTED]
---
 drivers/rtc/m41t60.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/rtc/m41t60.c b/drivers/rtc/m41t60.c
index 8a32ea0..ef135ca 100644
--- a/drivers/rtc/m41t60.c
+++ b/drivers/rtc/m41t60.c
@@ -241,7 +241,7 @@ void rtc_reset(void)
 
if (l = 0x3F) {
if ((data[RTC_CTRL]  0x3F) != l) {
-   printf(Setting RTC calibration to 0x%02X\n,
+   printf(Setting RTC calibration to 0x%02lX\n,
   l);
data[RTC_CTRL] = 0xC0;
data[RTC_CTRL] |= (uchar) l;
-- 
1.5.6.2


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 04/10 v2] [ARM] TQMA31: add new boardwith i.MX31 processor

2008-07-10 Thread Martin Krause

[EMAIL PROTECTED] wrote on :
 In message [EMAIL PROTECTED] you wrote:
  
  Question: Why did you accept all the initial patches for
  cpu/arm1136/mx31/interrupts.c and for the boards mx31ads,
  imx31_litekit, imx31_phycore?
 
 Nobody raised concerns, and I did not notice problems  either.  I  do
 not  claim  to catch 100% of all issues (I don't even claim to review
 all postings).

If I could remember correctly, we (TQS) paid DENX for the U-Boot porting
of the mx31ads board. It is a little bit surprising that now the patch
for our own i.mx31 based board is not accepted, although we used the 
mx31ads port as basis. The reason for paying the mx31ads port was to get
an accepted basis for a common i.mx31 board into the official U-Boot
tree, to have a stable basis for further portings of i.mx31 boards (and
to give something back to the great U-Boot community which is hopefully
useful for many other users).

Best Regards,
Martin Krause

--
TQ-Systems GmbH
Muehlstrasse 2, Gut Delling, D-82229 Seefeld
Amtsgericht Muenchen, HRB 105 018, UST-IdNr. DE 811 607 913
Geschaeftsfuehrer: Dipl.-Ing. (FH) Detlef Schneider, Dipl.-Ing. (FH) Ruediger 
Stahl
http://www.tq-group.com

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] MTD/NAND: Fix printf format warning in nand code

This patch fixes NAND related printf format warning. Those warnings are
now visible since patch dc4b0b38d4aadf08826f6c31270f1eecd27964fd
[Fix printf errors.] by Andrew Klossner has been applied. Thanks, this is
really helpful.

Signed-off-by: Stefan Roese [EMAIL PROTECTED]
---
 common/cmd_nand.c|   12 ++--
 drivers/mtd/nand/nand.c  |2 +-
 drivers/mtd/nand/nand_util.c |6 +++---
 3 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/common/cmd_nand.c b/common/cmd_nand.c
index e20c1af..cc7c5ba 100644
--- a/common/cmd_nand.c
+++ b/common/cmd_nand.c
@@ -55,11 +55,11 @@ static int nand_dump(nand_info_t *nand, ulong off)
off = ~(nand-oobblock - 1);
i = nand_read_raw(nand, buf, off, nand-oobblock, nand-oobsize);
if (i  0) {
-   printf(Error (%d) reading page %08x\n, i, off);
+   printf(Error (%d) reading page %08lx\n, i, off);
free(buf);
return 1;
}
-   printf(Page %08x dump:\n, off);
+   printf(Page %08lx dump:\n, off);
i = nand-oobblock  4; p = buf;
while (i--) {
printf( \t%02x %02x %02x %02x %02x %02x %02x %02x
@@ -149,7 +149,7 @@ out:
if (*size == nand-size)
puts(whole chip\n);
else
-   printf(offset 0x%x, size 0x%x\n, *off, *size);
+   printf(offset 0x%lx, size 0x%x\n, *off, *size);
return 0;
 }
 
@@ -181,7 +181,7 @@ int do_nand(cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
putc('\n');
for (i = 0; i  CFG_MAX_NAND_DEVICE; i++) {
if (nand_info[i].name)
-   printf(Device %d: %s, sector size %lu KiB\n,
+   printf(Device %d: %s, sector size %u KiB\n,
i, nand_info[i].name,
nand_info[i].erasesize  10);
}
@@ -238,7 +238,7 @@ int do_nand(cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
printf(\nDevice %d bad blocks:\n, nand_curr_device);
for (off = 0; off  nand-size; off += nand-erasesize)
if (nand_block_isbad(nand, off))
-   printf(  %08x\n, off);
+   printf(  %08lx\n, off);
return 0;
}
 
@@ -417,7 +417,7 @@ int do_nand(cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
if (off == nand-size - nand-oobblock
|| (s != last_status  off != 0))  {
 
-   printf(%08x - %08x: %8d pages 
%s%s%s\n,
+   printf(%08lx - %08lx: %8lu pages 
%s%s%s\n,
   block_start,
   off-1,
   (off-block_start)/nand-oobblock,
diff --git a/drivers/mtd/nand/nand.c b/drivers/mtd/nand/nand.c
index 27b5792..e44470e 100644
--- a/drivers/mtd/nand/nand.c
+++ b/drivers/mtd/nand/nand.c
@@ -70,7 +70,7 @@ void nand_init(void)
if (nand_curr_device == -1)
nand_curr_device = i;
}
-   printf(%lu MiB\n, size / (1024 * 1024));
+   printf(%u MiB\n, size / (1024 * 1024));
 
 #ifdef CFG_NAND_SELECT_DEVICE
/*
diff --git a/drivers/mtd/nand/nand_util.c b/drivers/mtd/nand/nand_util.c
index c82f77b..828cc33 100644
--- a/drivers/mtd/nand/nand_util.c
+++ b/drivers/mtd/nand/nand_util.c
@@ -154,7 +154,7 @@ int nand_erase_opts(nand_info_t *meminfo, const 
nand_erase_options_t *opts)
}
 
if (erase_length  meminfo-erasesize) {
-   printf(Warning: Erase size 0x%08x smaller than one\
+   printf(Warning: Erase size 0x%08lx smaller than one   \
   erase block 0x%08x\n,erase_length, meminfo-erasesize);
printf( Erasing 0x%08x instead\n, meminfo-erasesize);
erase_length = meminfo-erasesize;
@@ -498,7 +498,7 @@ int nand_write_opts(nand_info_t *meminfo, const 
nand_write_options_t *opts)
 * on (slow) serial consoles
 */
if (percent != percent_complete) {
-   printf(\rWriting data at 0x%x 
+   printf(\rWriting data at 0x%lx 
   -- %3d%% complete.,
   mtdoffset, percent);
percent_complete = percent;
@@ -678,7 +678,7 @@ int nand_read_opts(nand_info_t *meminfo, const 
nand_read_options_t *opts)
 */
if (percent != percent_complete) {
if (!opts-quiet)
-   printf(\rReading data from 0x%x 
+   printf(\rReading data from 0x%lx

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host mx01.1and1.com[74.208.5.4] said: 550
[EMAIL PROTECTED]: invalid address (in reply to RCPT TO command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: 13F5F2A93D9
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 11:25:50 +0700 (WIT)

Final-Recipient: rfc822; maria@colomboscuba.com
Original-Recipient: rfc822;maria@colomboscuba.com
Action: failed
Status: 5.0.0
Remote-MTA: dns; mx01.1and1.com
Diagnostic-Code: smtp; 550 maria@colomboscuba.com: invalid address
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 13F5F2A93D9
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 11:25:50 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id DC019362310
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 08:52:18 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: Mail Delivery System
Date: Thu, 10 Jul 2008 08:52:17 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0005_0D7A57AF.4071F440
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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 04/10 v2] [ARM] TQMA31: add new board with i.MX31 processor

2008-07-10 Thread Sascha Hauer

On Thu, Jul 10, 2008 at 12:46:42AM +0200, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
 
  The TQMA31, as well as the mx32ads use the General Purpose Timer 1, 
  which is feeded by a 32768 clock (possibly imx31_litekit and 
  imx31_phycore with 32000 Hz? I don't know.). The prescale divider can 
  only be an integer. 32768/1000 = non-integer.
 
 I think other architectures hav / had similar issues. See for example
 the MIPS fixes that were added some time ago.
 
  Currently, I see only one way to solve this:
  1. Patch the common file cpu/arm1136/mx31/interrupts.c, function 
  interrupt_init(), so that the prescaler divides by 33.
  Alternatively, also make the divider a board specific define.
  2. Patch all mx31 board config header files with CFG_HZ 1000.
  3. All mx31 boards have to be tested.
 
 Sounds like a plan.
 
  And while we are at the timers:
  The udelay function is the next problem:
  udelay cannot go below 1/(32768 Hz) = 30.5 micros
  The current function inserts a theoretical nop if the requested usec 
  value is below 1000. Therefore, IMHO, this timer is relatively 
  imprecise, because the resulting delay is below 1 microsecond.
  IMHO, this all because of that slow input clock.
 
 How is this handled in Linux? I don't think they would let this go
 through?

Linux uses the high frequency timer, or better said, the same timer with
a high frequency input source.

Regards,
  Sascha

-- 
Pengutronix e.K. - Linux Solutions for Science and Industry
---
Kontakt-Informationen finden Sie im Header dieser Mail oder
auf der Webseite - http://www.pengutronix.de/impressum/ -

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [ppc4xx] Please pull git://www.denx.de/git/u-boot-ppc4xx.git

The following changes since commit ac5ba41c2234dd513ed2c879914e964cc05a0135:
  Stefan Roese (1):
Merge branch 'master' of /home/stefan/git/u-boot/u-boot

are available in the git repository at:

  git://www.denx.de/git/u-boot-ppc4xx.git master

Larry Johnson (2):
  ppc4xx: Update and add FDT to Korat board support
  ppc4xx: Remove implementation of testdram() from Korat board support

Stefan Roese (2):
  ppc4xx: Enable support for  2GB SDRAM on AMCC Katmai
  ppc4xx: Fix printf format warnings now visible with the updated format 
check

 board/amcc/katmai/cmd_katmai.c |4 +-
 board/esd/du440/du440.c|4 +-
 board/korat/korat.c|   74 
 board/lwmon5/sdram.c   |2 +-
 board/zeus/zeus.c  |2 +-
 cpu/ppc4xx/44x_spd_ddr2.c  |   64 ++
 cpu/ppc4xx/4xx_enet.c  |2 +-
 cpu/ppc4xx/denali_spd_ddr2.c   |8 ++--
 cpu/ppc4xx/tlb.c   |4 +-
 cpu/ppc4xx/traps.c |   26 +++---
 include/asm-ppc/ppc4xx-sdram.h |4 +-
 include/configs/katmai.h   |7 
 include/configs/korat.h|   11 --
 13 files changed, 128 insertions(+), 84 deletions(-)

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] rtc: Fix printf format warning in m41t60.c

In message [EMAIL PROTECTED] you wrote:
 Signed-off-by: Stefan Roese [EMAIL PROTECTED]
 ---
  drivers/rtc/m41t60.c |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)

Applied, thanks!

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 didn't know it was impossible when I did it.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host mx3.hotmail.com[65.54.245.72] said: 550 Requested
action not taken: mailbox unavailable (in reply to RCPT TO command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: 2AB9B361A6B
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 11:32:39 +0700 (WIT)

Final-Recipient: rfc822; madmax@hotmail.com
Original-Recipient: rfc822;madmax@hotmail.com
Action: failed
Status: 5.0.0
Remote-MTA: dns; mx3.hotmail.com
Diagnostic-Code: smtp; 550 Requested action not taken: mailbox unavailable
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 2AB9B361A6B
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 11:32:39 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id C626736EA84
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 08:55:49 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: Good day
Date: Thu, 10 Jul 2008 08:55:48 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0011_F294AB97.8614A12F
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] usage of git to send patches to u-boot mailinglist

Hi Harald,

Harald Welte [EMAIL PROTECTED] writes:

 can do, even though I believe it is by far not the best tool to do so.
 The problem is that I would have to use one local branch per feature
 (i.e. lots of local branches that need to be kept in sync), and even
 then any incremental changes/fixes to one particular feature are visible
 in the commitlog (and thus result in changelog pollution).

 My best experience so far really is quilt for maintaining patchsets.
 You can keep a large number of patches, easily switch between them and
 keep your modifications organized per-feature, rather than in the
 chronological commit order of a revision control system.

 So what I can probably do is to continue to use quilt up to the point
 where I'd want to send something to a mailinglist, and then put into a
 local git branch, export the patch from there and send it to the list.

 However, any further change to that patch based on feedback from the
 list would again go into the quilt tree, I'd have to start with a clean
 'origin' u-boot git tree and commit the modified change into the git
 tree.   Otherwise we start having all the commit messages (like 'changed
 coding style according to mailinglist feedback') in the code, even
 _before_ that code was ever merged into the respective mainline git
 tree.

 So is this really the preferred workflow? How are others dealing with
 this?  How to avoid commitlog pollution?

I never used quilt, but I believe stacked git (stgit) implements more or
less the same behavior on top of git.

Best regards

Markus Klotzbuecher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] MTD/NAND: Fix printf format warning in nand code

In message [EMAIL PROTECTED] you wrote:
 This patch fixes NAND related printf format warning. Those warnings are
 now visible since patch dc4b0b38d4aadf08826f6c31270f1eecd27964fd
 [Fix printf errors.] by Andrew Klossner has been applied. Thanks, this is
 really helpful.
 
 Signed-off-by: Stefan Roese [EMAIL PROTECTED]
 ---
  common/cmd_nand.c|   12 ++--
  drivers/mtd/nand/nand.c  |2 +-
  drivers/mtd/nand/nand_util.c |6 +++---
  3 files changed, 10 insertions(+), 10 deletions(-)

Applied, thanks.

Scott, I hope this is OK with you. I wanted to have this quickly to
get clean builds again.

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]
Q:  How many DEC repairman does it take to fix a flat ?
A:  Five; four to hold the car up and one to swap tires.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 04/10 v2] [ARM] TQMA31: add new board with i.MX31 processor

In message [EMAIL PROTECTED] you wrote:
 
  CFG_HZ is a constant and has to be 1000.
 
 Why is it a board specific define then?

Historical reasons only. When PPCBoot was created, I played with that
parameter to get a feeling  for  the  involved  overhead,  etc.  This
should  really  be removed, but as you know too many boards depend on
this.

 You could add a software divider in get_timer(). But I think it's better
 to define CFG_HZ to whatever the timer frequency is. At least 49 boards
 do it.

That's what I mean: many boards need fixing.

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]
A man either lives life as it happens to him, meets  it  head-on  and
licks it, or he turns his back on it and starts to wither away.
-- Dr. Boyce, The Menagerie (The Cage), stardate unknown

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] usage of git to send patches to u-boot mailinglist

On Thursday 10 July 2008, Markus Klotzbücher wrote:
  So is this really the preferred workflow? How are others dealing with
  this?  How to avoid commitlog pollution?

 I never used quilt, but I believe stacked git (stgit) implements more or
 less the same behavior on top of git.

Yes, and guilt should be similar too. I'm looking forward to the guilt BOFS at 
OLS in a few weeks.

Best regards,
Stefan

=
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]
=

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] [ppc4xx] Please pull git://www.denx.de/git/u-boot-ppc4xx.git

In message [EMAIL PROTECTED] you wrote:
 The following changes since commit ac5ba41c2234dd513ed2c879914e964cc05a0135:
   Stefan Roese (1):
 Merge branch 'master' of /home/stefan/git/u-boot/u-boot
 
 are available in the git repository at:
 
   git://www.denx.de/git/u-boot-ppc4xx.git master
 
 Larry Johnson (2):
   ppc4xx: Update and add FDT to Korat board support
   ppc4xx: Remove implementation of testdram() from Korat board support
 
 Stefan Roese (2):
   ppc4xx: Enable support for  2GB SDRAM on AMCC Katmai
   ppc4xx: Fix printf format warnings now visible with the updated format 
 check
 
  board/amcc/katmai/cmd_katmai.c |4 +-
  board/esd/du440/du440.c|4 +-
  board/korat/korat.c|   74 
 
  board/lwmon5/sdram.c   |2 +-
  board/zeus/zeus.c  |2 +-
  cpu/ppc4xx/44x_spd_ddr2.c  |   64 ++
  cpu/ppc4xx/4xx_enet.c  |2 +-
  cpu/ppc4xx/denali_spd_ddr2.c   |8 ++--
  cpu/ppc4xx/tlb.c   |4 +-
  cpu/ppc4xx/traps.c |   26 +++---
  include/asm-ppc/ppc4xx-sdram.h |4 +-
  include/configs/katmai.h   |7 
  include/configs/korat.h|   11 --
  13 files changed, 128 insertions(+), 84 deletions(-)

Done. and thanks a lot for all the cleanup.

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]
Immortality consists largely of boredom.
-- Zefrem Cochrane, Metamorphosis, stardate 3219.8

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] usage of git to send patches to u-boot mailinglist

In message [EMAIL PROTECTED] you wrote:
 
 can do, even though I believe it is by far not the best tool to do so.

The definition of the best tool depends on many things, including
previous experience and personal preferences. 

 The problem is that I would have to use one local branch per feature
 (i.e. lots of local branches that need to be kept in sync), and even
 then any incremental changes/fixes to one particular feature are visible
 in the commitlog (and thus result in changelog pollution).

Having many local branches is no problem with git.

Git provides excellent help to rebase such branches, and using
--interactive gives you a lot of options to edit the history.

 So is this really the preferred workflow? How are others dealing with
 this?  How to avoid commitlog pollution?

I started using git-rebase -i, and so far it seems to work fine for
me. But I'm definitely not an expert.

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]
Contrary to popular belief, thinking does not cause brain damage.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 04/10 v2] [ARM] TQMA31: add new board with i.MX31 processor

2008-07-10 Thread Sascha Hauer

On Thu, Jul 10, 2008 at 10:49:09AM +0200, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
  
   CFG_HZ is a constant and has to be 1000.
  
  Why is it a board specific define then?
 
 Historical reasons only. When PPCBoot was created, I played with that
 parameter to get a feeling  for  the  involved  overhead,  etc.  This
 should  really  be removed, but as you know too many boards depend on
 this.
 
  You could add a software divider in get_timer(). But I think it's better
  to define CFG_HZ to whatever the timer frequency is. At least 49 boards
  do it.
 
 That's what I mean: many boards need fixing.

How about using the Linux clocksource framework for U-Boot? I did this
for V2 and it works like a charm. The timer implementation is straight
forward. You only have to setup a free running timer and return its
value, so there's not much to do wrong when implementing new timers.
Currently U-Boot has many timing functions like get_timer,
get_timer_masked, reset_timer, reset_timer_masked, udelay and I suppose
most people do not understand the meaning of those functions, as the
differing implementations show.
Not that I volunteer to do this job though :(

Regards,
  Sascha

-- 
 Pengutronix - Linux Solutions for Science and Industry
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
 Hannoversche Str. 2, 31134 Hildesheim, Germany
   Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 USB devices with multiple configurations

Wolfgang Denk [EMAIL PROTECTED] writes:

 In message [EMAIL PROTECTED] you wrote:
 
 This patch fixes bugs in usbdcore*.c related to the use of devices
 with multiple configurations.
 
 The original code made mistakes about the meaning of configuration value and
 configuration index, and the resulting off-by-one errors resulted in:
 
 * SET_CONFIGURATION always selected the first configuration, no matter what
   wValue is being passed.
 * GET_DESCRIPTOR/CONFIGURATION always returned the descriptor for the first
   configuration (index 0).
 
 Signed-off-by: Harald Welte [EMAIL PROTECTED]

Acked-by: Markus Klotzbuecher [EMAIL PROTECTED]

 Markus, are you going to add this to the USB repo any time soon?

 Or you could just ACK it (if you think so) and tell me to pick up
 directly.

Wolfgang, please apply.

Sorry for the delay.

Best regards

Markus Klotzbuecher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 include sequence in 4xx_pcie.c

This patch now moves common.h to the top of the inlcude list. This
is needed for boards with CONFIG_PHYS_64BIT set (e.g. katmai), so that
the phys_size_t/phys_addr_t are defined to the correct size in this
driver.

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

diff --git a/cpu/ppc4xx/4xx_pcie.c b/cpu/ppc4xx/4xx_pcie.c
index d50a538..9803fcc 100644
--- a/cpu/ppc4xx/4xx_pcie.c
+++ b/cpu/ppc4xx/4xx_pcie.c
@@ -25,11 +25,11 @@
 #define DEBUG
 #endif
 
-#include asm/processor.h
-#include asm-ppc/io.h
-#include ppc4xx.h
 #include common.h
 #include pci.h
+#include ppc4xx.h
+#include asm/processor.h
+#include asm-ppc/io.h
 
 #if (defined(CONFIG_440SPE) || defined(CONFIG_405EX) ||\
 defined(CONFIG_460EX) || defined(CONFIG_460GT))  \
-- 
1.5.6.2


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] ppc4xx: Fix include sequence in 4xx_pcie.c

In message [EMAIL PROTECTED] you wrote:
 This patch now moves common.h to the top of the inlcude list. This
 is needed for boards with CONFIG_PHYS_64BIT set (e.g. katmai), so that
 the phys_size_t/phys_addr_t are defined to the correct size in this
 driver.
 
 Signed-off-by: Stefan Roese [EMAIL PROTECTED]
 ---
  cpu/ppc4xx/4xx_pcie.c |6 +++---
  1 files changed, 3 insertions(+), 3 deletions(-)

Hm. I think we have a serious problem when the correctness of the code
depends on the include order. It seems that include/asm-ppc/types.h
depends on CONFIG_PHYS_64BIT, so include/asm-ppc/types.h must make
sure this variable is set correctly, i. e. we should include common.h
there.

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 don't need a weatherman to know which way the wind blows.
  - Bob Dylan

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 04/10 v2] [ARM] TQMA31: add new board with i.MX31 processor

In message [EMAIL PROTECTED] you wrote:

 How about using the Linux clocksource framework for U-Boot? I did this

I have to admit that I don't know how big that code is?

 Currently U-Boot has many timing functions like get_timer,
 get_timer_masked, reset_timer, reset_timer_masked, udelay and I suppose
 most people do not understand the meaning of those functions, as the
 differing implementations show.

I have to admit that I don't have  a  clue  why  get_timer_masked  or
reset_timer_masked   would   be   needed.  Originally  (see  the  PPC
implementation), there was only get_timer,  reset_timer,  and  udelay
(and IIRC reset_timer was not even necessary).

 Not that I volunteer to do this job though :(

What a pity :-)

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]
Ever try. Ever fail. No matter. Try again. Fail again.  Fail  better.
-- S. Beckett

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 USB devices with multiple configurations

In message [EMAIL PROTECTED] you wrote:
 
 Acked-by: Markus Klotzbuecher [EMAIL PROTECTED]

thanks.

 Wolfgang, please apply.

Done, thanks.

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]
We're all sorry for the other guy when he loses his job to a machine.
But when it comes to your job -- that's different. And it always will
be different.
-- McCoy, The Ultimate Computer, stardate 4729.4

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] ppc4xx: Fix include sequence in 4xx_pcie.c

On Thursday 10 July 2008, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
  This patch now moves common.h to the top of the inlcude list. This
  is needed for boards with CONFIG_PHYS_64BIT set (e.g. katmai), so that
  the phys_size_t/phys_addr_t are defined to the correct size in this
  driver.
 
  Signed-off-by: Stefan Roese [EMAIL PROTECTED]
  ---
   cpu/ppc4xx/4xx_pcie.c |6 +++---
   1 files changed, 3 insertions(+), 3 deletions(-)

 Hm. I think we have a serious problem when the correctness of the code
 depends on the include order. It seems that include/asm-ppc/types.h
 depends on CONFIG_PHYS_64BIT, so include/asm-ppc/types.h must make
 sure this variable is set correctly, i. e. we should include common.h
 there.

Yes, I though too that this problem might affect other files too. But 
unfortunately including common.h from within asm/types.h does not work since 
common.h includes (and needs) types.h itself.

I have no elegant solution to handle this in a more generic way right now. 
Perhaps somebody else has? Ideas welcome. :)

Thanks.

Best regards,
Stefan

=
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]
=

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 04/10 v2] [ARM] TQMA31: add new board with i.MX31 processor

2008-07-10 Thread Sascha Hauer

On Thu, Jul 10, 2008 at 11:46:45AM +0200, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
 
  How about using the Linux clocksource framework for U-Boot? I did this
 
 I have to admit that I don't know how big that code is?

Somewhere next to nothing:

/*
 * clock.c - generic clocksource implementation
 *
 * This file contains the clocksource implementation from the Linux
 * kernel originally by John Stultz
 *
 * Copyright (C) 2004, 2005 IBM, John Stultz ([EMAIL PROTECTED])
 * Copyright (c) 2007 Sascha Hauer [EMAIL PROTECTED], Pengutronix
 *
 * See file CREDITS for list of people who contributed to this
 * project.
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License version 2
 * as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 */

#include common.h
#include asm-generic/div64.h
#include clock.h

static struct clocksource *current_clock;
static uint64_t time_ns;

/**
 * get_time_ns - get current timestamp in nanoseconds
 */
uint64_t get_time_ns(void)
{
struct clocksource *cs = current_clock;
uint64_t cycle_now, cycle_delta;
uint64_t ns_offset;

/* read clocksource: */
cycle_now = cs-read();

/* calculate the delta since the last call: */
cycle_delta = (cycle_now - cs-cycle_last)  cs-mask;

/* convert to nanoseconds: */
ns_offset = cyc2ns(cs, cycle_delta);

cs-cycle_last = cycle_now;

time_ns += ns_offset;
return time_ns;
}
EXPORT_SYMBOL(get_time_ns);

/**
 * clocksource_hz2mult - calculates mult from hz and shift
 * @hz: Clocksource frequency in Hz
 * @shift_constant: Clocksource shift factor
 *
 * Helper functions that converts a hz counter
 * frequency to a timsource multiplier, given the
 * clocksource shift value
 */
uint32_t clocksource_hz2mult(uint32_t hz, uint32_t shift_constant)
{
/*  hz = cyc/(Billion ns)
 *  mult/2^shift  = ns/cyc
 *  mult = ns/cyc * 2^shift
 *  mult = 1Billion/hz * 2^shift
 *  mult = 10 * 2^shift / hz
 *  mult = (10shift) / hz
 */
uint64_t tmp = ((uint64_t)10)  shift_constant;

tmp += hz/2; /* round for do_div */
do_div(tmp, hz);

return (uint32_t)tmp;
}

int is_timeout(uint64_t start_ns, uint64_t time_offset_ns)
{

if (start_ns + time_offset_ns  get_time_ns())
return 1;
else
return 0;
}
EXPORT_SYMBOL(is_timeout);

void udelay(unsigned long usecs)
{
uint64_t start = get_time_ns();

while(!is_timeout(start, usecs * 1000));
}
EXPORT_SYMBOL(udelay);

void mdelay(unsigned long msecs)
{
uint64_t start = get_time_ns();

while(!is_timeout(start, msecs * 100));
}
EXPORT_SYMBOL(mdelay);

int init_clock(struct clocksource *cs)
{
current_clock = cs;
return 0;
}

The clocksource driver itself adds another 100 lines of code (for mx31).
The implementation of udelay shows how to implement timeouts, the main
usage of timing functions.

Regards,
  Sascha



-- 
-- 
 Pengutronix - Linux Solutions for Science and Industry
   Handelsregister:  Amtsgericht Hildesheim, HRA 2686
 Hannoversche Str. 2, 31134 Hildesheim, Germany
   Phone: +49-5121-206917-0 |  Fax: +49-5121-206917-9

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] [PATCH] Fix usb part command

Wolfgang Denk [EMAIL PROTECTED] writes:

 In message [EMAIL PROTECTED] you wrote:
 From: Christian Eggers [EMAIL PROTECTED]
 
 Only print partition for selected device if user supplied the dev arg with 
 the usb part [dev] command.

 What is the rationale of this modification, i. e. which bug are you
 fixing?

I think this fix is correct. If I do

usb part

U-Boot (correcly) prints the partitions of all devices. If I type

usb part 0 

i only want the partitions on device 0. Without this patch I still get
all partitions on all devices. This is wrong.

 I think current behaviour is consistent  between  ide  part,  scsi
 part,  usb part etc. so if there is a problem here this might need
 to be changed elsewhere, too ?

I just took a look at cmd_ide, and AFAICS it does exactly what USB will
do with this patch.

Best regards

Markus Klotzbuecher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] [PATCH] Fix usb part command

Christian Eggers [EMAIL PROTECTED] writes:

 From: Christian Eggers [EMAIL PROTECTED]

 Only print partition for selected device if user supplied the dev arg with 
 the usb part [dev] command.

 Signed-off-by: Christian Eggers [EMAIL PROTECTED]
Acked-by: Markus Klotzbuecher [EMAIL PROTECTED]

Wolfgang, please apply! 

Best regards

Markus Klotzbuecher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 2/3] avr32: Use CONFIG_ATMEL_MCI to select the atmel_mci driver

On Thu, 10 Jul 2008 00:05:27 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 Applied, with a merge conflict include/configs/atngw100.h; please
 check the result. Thanks.

Looks good to me. Thanks.

Ho-hum...a default ATNGW100 build seems to have gone from 0 warnings to
around 20 quite recently. I'll have a look at that when I get back from
vacation.

Haavard

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host mx1.us.apache.org[140.211.11.136] said:
552 Message rejected as it is spam (body) (in reply to end of DATA command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: 2BCC335816B
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 12:52:40 +0700 (WIT)

Final-Recipient: rfc822; tomcat-dev@jakarta.apache.org
Original-Recipient: rfc822;tomcat-dev@jakarta.apache.org
Action: failed
Status: 5.0.0
Remote-MTA: dns; mx1.us.apache.org
Diagnostic-Code: smtp; 552 Message rejected as it is spam (body)
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 2BCC335816B
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 12:52:40 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 723ED2A942F
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 08:21:55 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: cctfljr
Date: Thu, 10 Jul 2008 08:21:54 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0003_B4CE7537.A8DD5FBD
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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] sh: add support for SH7785

Renesas SH7785 has DDR2-SDRAM controller, PCI, and other.
This patch supports CPU register's header file.

Signed-off-by: Yoshihiro Shimoda [EMAIL PROTECTED]
---

PATCH v2 :
 - add Iwamatsu-san e-mail address in include/asm-sh/cpu_sh7785.h.
 - fix my e-mail address in same file.

 include/asm-sh/cpu_sh4.h|2 +
 include/asm-sh/cpu_sh7785.h |  156 +++
 2 files changed, 158 insertions(+), 0 deletions(-)
 create mode 100644 include/asm-sh/cpu_sh7785.h

diff --git a/include/asm-sh/cpu_sh4.h b/include/asm-sh/cpu_sh4.h
index 5a8a5a1..de6eb5a 100644
--- a/include/asm-sh/cpu_sh4.h
+++ b/include/asm-sh/cpu_sh4.h
@@ -39,6 +39,8 @@
 # include asm/cpu_sh7763.h
 #elif defined (CONFIG_CPU_SH7780)
 # include asm/cpu_sh7780.h
+#elif defined (CONFIG_CPU_SH7785)
+# include asm/cpu_sh7785.h
 #else
 # error Unknown SH4 variant
 #endif
diff --git a/include/asm-sh/cpu_sh7785.h b/include/asm-sh/cpu_sh7785.h
new file mode 100644
index 000..4a4dfc9
--- /dev/null
+++ b/include/asm-sh/cpu_sh7785.h
@@ -0,0 +1,156 @@
+#ifndef_ASM_CPU_SH7785_H_
+#define_ASM_CPU_SH7785_H_
+
+/*
+ * Copyright (c) 2007,2008 Nobuhiro Iwamatsu [EMAIL PROTECTED]
+ * Copyright (c) 2008 Yusuke Goda [EMAIL PROTECTED]
+ * Copyright (c) 2008 Yoshihiro Shimoda [EMAIL PROTECTED]
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
+ * MA 02111-1307 USA
+ *
+ */
+
+#defineCACHE_OC_NUM_WAYS   1
+#defineCCR_CACHE_INIT  0x090b
+
+/* Exceptions  */
+#defineTRA 0xFF20
+#defineEXPEVT  0xFF24
+#defineINTEVT  0xFF28
+
+/* Cache Controller */
+#defineCCR 0xFF1C
+#defineQACR0   0xFF38
+#defineQACR1   0xFF3C
+#defineRAMCR   0xFF74
+
+/* Watchdog Timer and Reset */
+#defineWTCNT   WDTCNT
+#defineWDTST   0xFFCC
+#defineWDTCSR  0xFFCC0004
+#defineWDTBST  0xFFCC0008
+#defineWDTCNT  0xFFCC0010
+#defineWDTBCNT 0xFFCC0018
+
+/* Timer Unit */
+#defineTSTRTSTR0
+#defineTOCR0xFFD8
+#defineTSTR0   0xFFD80004
+#defineTCOR0   0xFFD80008
+#defineTCNT0   0xFFD8000C
+#defineTCR00xFFD80010
+#defineTCOR1   0xFFD80014
+#defineTCNT1   0xFFD80018
+#defineTCR10xFFD8001C
+#defineTCOR2   0xFFD80020
+#defineTCNT2   0xFFD80024
+#defineTCR20xFFD80028
+#defineTCPR2   0xFFD8002C
+#defineTSTR1   0xFFDC0004
+#defineTCOR3   0xFFDC0008
+#defineTCNT3   0xFFDC000C
+#defineTCR30xFFDC0010
+#defineTCOR4   0xFFDC0014
+#defineTCNT4   0xFFDC0018
+#defineTCR40xFFDC001C
+#defineTCOR5   0xFFDC0020
+#defineTCNT5   0xFFDC0024
+#defineTCR50xFFDC0028
+
+/* Serial CommunicationInterface with FIFO */
+#defineSCIF1_BASE  0xffeb
+
+/* LBSC */
+#define MMSELR 0xfc400020
+#define LBSC_BASE  0xff80
+#define BCR(LBSC_BASE + 0x1000)
+#define CS0BCR (LBSC_BASE + 0x2000)
+#define CS1BCR (LBSC_BASE + 0x2010)
+#define CS2BCR (LBSC_BASE + 0x2020)
+#define CS3BCR (LBSC_BASE + 0x2030)
+#define CS4BCR (LBSC_BASE + 0x2040)
+#define CS5BCR (LBSC_BASE + 0x2050)
+#define CS6BCR (LBSC_BASE + 0x2060)
+#define CS0WCR (LBSC_BASE + 0x2008)
+#define CS1WCR (LBSC_BASE + 0x2018)
+#define CS2WCR (LBSC_BASE + 0x2028)
+#define CS3WCR (LBSC_BASE + 0x2038)
+#define CS4WCR (LBSC_BASE + 0x2048)
+#define CS5WCR (LBSC_BASE + 0x2058)
+#define CS6WCR (LBSC_BASE + 0x2068)
+#define CS5PCR (LBSC_BASE + 0x2070)
+#define CS6PCR (LBSC_BASE + 0x2080)
+
+/* PCI Controller */
+#defineSH7780_PCIECR   0xFE08
+#defineSH7780_PCIVID   0xFE04
+#defineSH7780_PCIDID   0xFE040002
+#defineSH7780_PCICMD   0xFE040004
+#defineSH7780_PCISTATUS0xFE040006
+#defineSH7780_PCIRID   0xFE040008
+#defineSH7780_PCIPIF   0xFE040009
+#defineSH7780_PCISUB   0xFE04000A
+#defineSH7780_PCIBCC   0xFE04000B
+#defineSH7780_PCICLS   0xFE04000C

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host in-mta01.plasa.com[125.160.17.23] said: 550 No such
account [EMAIL PROTECTED] (in reply to RCPT TO command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: 425C435FE45
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 13:44:53 +0700 (WIT)

Final-Recipient: rfc822; michael@telkom.net
Original-Recipient: rfc822;michael@telkom.net
Action: failed
Status: 5.0.0
Remote-MTA: dns; in-mta01.plasa.com
Diagnostic-Code: smtp; 550 No such account michael@telkom.net
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 425C435FE45
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 13:44:53 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 996A935FFBD
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 10:54:49 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: hello
Date: Thu, 10 Jul 2008 10:54:48 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0014_623A31B2.13539FFA
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host in-mta01.plasa.com[125.160.17.23] said: 550 No such
account [EMAIL PROTECTED] (in reply to RCPT TO command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: ECF1936EB3E
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 13:49:15 +0700 (WIT)

Final-Recipient: rfc822; fred@telkom.net
Original-Recipient: rfc822;fred@telkom.net
Action: failed
Status: 5.0.0
Remote-MTA: dns; in-mta01.plasa.com
Diagnostic-Code: smtp; 550 No such account fred@telkom.net
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id ECF1936EB3E
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 13:49:15 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id AF21E2AA2A5
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 11:40:16 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: Good day
Date: Thu, 10 Jul 2008 11:40:15 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0014_9624A367.EF6C6181
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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] [PATCH] Fix usb part command

In message [EMAIL PROTECTED] you wrote:
 Christian Eggers [EMAIL PROTECTED] writes:
 
  From: Christian Eggers [EMAIL PROTECTED]
 
  Only print partition for selected device if user supplied the dev arg 
  with the usb part [dev] command.
 
  Signed-off-by: Christian Eggers [EMAIL PROTECTED]
 Acked-by: Markus Klotzbuecher [EMAIL PROTECTED]
 
 Wolfgang, please apply! 

Done. Thanks.

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's the sound a name makes when it's dropped?

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] [PATCH] Fix usb part command

In message [EMAIL PROTECTED] you wrote:

  Only print partition for selected device if user supplied the dev arg 
  with the usb part [dev] command.
 
  What is the rationale of this modification, i. e. which bug are you
  fixing?
 
 I think this fix is correct. If I do

I see. Thanks.

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]
Many companies that have made themselves dependent on [the  equipment
of  a  certain  major  manufacturer] (and in doing so have sold their
soul to the devil) will collapse under the sheer weight  of  the  un-
mastered complexity of their data processing systems.
  -- Edsger W. Dijkstra, SIGPLAN Notices, Volume 17, Number 5

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] usb: add support for R8A66597 usb controller

2008/07/10 4:15, Jean-Christophe PLAGNIOL-VILLARD wrote:
  +
 +#include common.h
 +#include usb.h
 +#include asm/io.h
 +
 +#include r8a66597.h
 +
 +#if defined(CONFIG_USB_R8A66597_HCD)
 please move to the Makefile

Thank you for your comment.
I wlll move to the Makefile.

 +
 +#ifdef R8A66597_DEBUG
 +#define R8A66597_DPRINT printf
 +#else
 +#define R8A66597_DPRINT(...)
 +#endif
 why not using debug()

I'm sorry. I didn't know debug(). I will use it.

 +
 +static const char hcd_name[] = r8a66597_hcd;
 +static unsigned short clock = CONFIG_R8A66597_XTAL;
 +static unsigned short vif = CONFIG_R8A66597_LDRV;
 +static unsigned short endian = CONFIG_R8A66597_ENDIAN;
 +static struct r8a66597 gr8a66597;
 is it not possible to have a better name for the struct and the var
 and btw for the function naming?

Because XTAL, LDRV, DEVADD are a register name or register bit name,
I think that it is a good name...
But I will change a name of the struct r8a66957 gr8a66597 to
struct r8a66597 r8a66597_dev.

 +
 +static void set_devadd_reg(struct r8a66597 *r8a66597, u8 r8a66597_address,
 +   u16 usbspd, u8 upphub, u8 hubport, int port)
 +{
 +u16 val;
 +unsigned long devadd_reg = get_devadd_addr(r8a66597_address);
 +
 +val = (upphub  11) | (hubport  8) | (usbspd  6) | (port  0x0001);
 +r8a66597_write(r8a66597, val, devadd_reg);
 +}
 +

Best Regards,
Yoshihiro Shimoda

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] [PATCH] Fix usb part command

In message [EMAIL PROTECTED] you wrote:
 From: Christian Eggers [EMAIL PROTECTED]
 
 Only print partition for selected device if user supplied the dev arg with 
 the usb part [dev] command.
 
 Signed-off-by: Christian Eggers [EMAIL PROTECTED]
 ---
  common/cmd_usb.c |   19 +++
  1 files changed, 15 insertions(+), 4 deletions(-)

Applied, thanks.

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 find this a nice feature but it is not according to  the  documen-
tation. Or is it a BUG?   Let's call it an accidental feature. :-)
   - Larry Wall in [EMAIL PROTECTED]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] sh: Renesas R0P7785LC0011RL board support

2008/07/10 4:28, Jean-Christophe PLAGNIOL-VILLARD wrote:
 +include $(TOPDIR)/config.mk
 +
 +LIB = lib$(BOARD).a
 +
 +COBJS   := r0p7785lc0011rl.o selfcheck.o rtl8169_mac.o
 +SOBJS   := lowlevel_init.o
 +
 +$(LIB): $(COBJS) $(SOBJS)
 +$(AR) crv $@ $(COBJS) $(SOBJS)
 please use $(ARFLAGS) instead of 'crv'

I will fix it.

 +
 +clean:
 +rm -f $(SOBJS) $(OBJS)
 +
 +distclean:  clean
 +rm -f $(LIB) core *.bak .depend
 pelase add the $(obj)

I will fix it, too.

 +
 +#
 +
 +.depend:Makefile $(SOBJS:.o=.S) $(OBJS:.o=.c)
 +$(CC) -M $(CPPFLAGS) $(SOBJS:.o=.S) $(OBJS:.o=.c)  $@
 +
 +-include .depend
 +
 + * MA 02111-1307 USA
 + */
 +
 +#define PCIREG_8(_adr)  (*(volatile unsigned char *)(_adr))
 +#define PCIREG_32(_adr) (*(volatile unsigned long *)(_adr))
 +#define PCI_PAR PCIREG_32(0xfe0401c0)
 +#define PCI_PDR PCIREG_32(0xfe040220)
 +#define PCI_CR  PCIREG_32(0xfe040100)
 +#define PCI_CONF1   PCIREG_32(0xfe040004)
 +
 +#define EEPROM_ADD  0x0050
 +#define HIGH1
 +#define LOW 0
 +
 +#define PCI_PROG0x80
 +
 +#define PCI_EEP_ADDRESS (unsigned short)0x0007
 +#define PCI_MAC_ADDRESS_SIZE3
 +
 +#define TIME1   100
 +#define TIME2   2
 
 +#define BIT_DUMMY   0
 +#define MAC_EEP_READ1
 +#define MAC_EEP_WRITE   2
 +#define MAC_EEP_ERACE   3
 +#define MAC_EEP_EWEN4
 +#define MAC_EEP_EWDS5
 +
 +#define DEBUG 0
 Please remove

I will remove DEBUG.

 Could you add few comment about the data?

I'm sorry. I cannot add comment, because I don't know
detail of the data.

 +
 +/* RTL8169 */
 +const unsigned short EEPROM_W_Data_8169_A[] = {
 +0x8129, 0x10ec, 0x8169, 0x1154, 0x032b,
 +0x4020, 0xa101
 +};
 +const unsigned short EEPROM_W_Data_8169_B[] = {
 +0x4d15, 0xf7c2, 0x8000, 0x, 0x, 0x1300,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x2000,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x,
 +0x, 0x, 0x, 0x, 0x, 0x, 0x, 0x
 +};
 +
 
 +
 +int do_set_mac(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 +{
 +int i;
 +unsigned char mac[6];
 +char *s, *e;
 +
 +if (argc != 2) {
 +printf(Usage:\n%s\n, cmdtp-usage);
 +return 1;
 +}
 +
 +s = argv[1];
 +
 +for (i = 0; i  6; i++) {
 +mac[i] = s ? simple_strtoul(s, e, 16) : 0;
 +if (s)
 +s = (*e) ? e + 1 : e;
 +}
 +mac_write(mac);
 +
 +return 0;
 +}
 +
 +U_BOOT_CMD(
 +setmac, 2,  1,  do_set_mac,
 +setmac - write MAC address for RTL8110SCL\n,
 +\n
 +setmac mac address - write MAC address for RTL8110SCL\n
 +);
 +
 +int do_print_mac(cmd_tbl_t *cmdtp, int flag, int argc, char *argv[])
 +{
 +int i;
 +uchar mac[6];
 +
 +if (argc != 1) {
 +printf(Usage:\n%s\n, cmdtp-usage);
 +return 1;
 +}
 +
 +mac_read();
 +
 +return 0;
 +}
 +
 +U_BOOT_CMD(
 +printmac,   1,  1,  do_print_mac,
 +printmac - print MAC address for RTL8110\n,
 +\n
 +- print MAC address for RTL8110\n
 +);
 Could you add a README for this board and describe its specific command?

Yes, I will add a README.

 +
 +
 +U_BOOT_CMD(
 +hwtest, 2,  1,  do_hw_test,
 +hwtest - hardware test for R0P7785LC0011RL board\n,
 +\n
 +hwtest all   - test all hardware\n
 +hwtest pld   - output pld version\n
 +hwtest led   - test LED\n
 +hwtest dipsw - test DIPSW\n
 +hwtest sm107 - output SM107 version\n
 +hwtest usb   - test R8A66597\n
 +hwtest i2c   - output PCA9564 version\n
 +hwtest net   - compare RTL8110 ID\n
 +hwtest sata  - compare SiI3512 ID\n
 +hwtest pci   - output PCI slot device ID\n
 +);
 idem
 +
  +
 +#define CONFIG_NETMASK  255.255.255.0
 +#define CONFIG_IPADDR   192.168.10.100
 +#define CONFIG_SERVERIP 192.168.10.77
 +#define CONFIG_GATEWAYIP192.168.10.77
 please remove this 4 defines

OK. I will remove it.

 +
 +#define __io
 +#define __mem_pci
 is it really the good place to define it??

I will remove it, Because It is unnecessary.

 +
 +#define CONFIG_PCI_MEM_BUS  0xFD00  /* Memory space base addr */
 +#define CONFIG_PCI_MEM_PHYS CONFIG_PCI_MEM_BUS
 +#define CONFIG_PCI_MEM_SIZE 0x0100  /* Size of Memory window */
 +

Best Regards,
Yoshihiro Shimoda


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along

Re: [U-Boot-Users] [PATCH] sh: Renesas R0P7785LC0011RL board support

2008/07/10 7:28, Kim Phillips wrote:
 On Wed, 09 Jul 2008 21:07:47 +0900
 Yoshihiro Shimoda [EMAIL PROTECTED] wrote:
 
 This board has SH7785, 512MB DDR2-SDRAM, NOR Flash,
 Graphic controller, Ethernet, USB, SD, and RTC.

 This patch supports the following functions:
  - 128MB DDR2-SDRAM (29-bit address mode only)
  - NOR Flash
  - USB host
  - Ethernet

 Signed-off-by: Yoshihiro Shimoda [EMAIL PROTECTED]
 ---
  MAINTAINERS |1 +
  MAKEALL |1 +
  Makefile|5 +
  board/r0p7785lc0011rl/Makefile  |   43 
  board/r0p7785lc0011rl/config.mk |   26 +++
  board/r0p7785lc0011rl/lowlevel_init.S   |  318 +++
  board/r0p7785lc0011rl/r0p7785lc0011rl.c |   51 +
  board/r0p7785lc0011rl/rtl8169.h |   62 ++
  board/r0p7785lc0011rl/rtl8169_mac.c |  354 
 +++
  board/r0p7785lc0011rl/selfcheck.c   |  196 +
  board/r0p7785lc0011rl/u-boot.lds|   97 +
  include/configs/r0p7785lc0011rl.h   |  173 +++
 
 Can this be renamed something simpler, like sh7785rdp or something?

Thank you for your comment.
Yes, I will rename the file sh7785lcr.

Best Regards,
Yoshihiro Shimoda

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host liszt.debian.org[82.195.75.100] said: 550 5.1.1
[EMAIL PROTECTED]: Recipient address rejected: User unknown in
virtual alias table (in reply to RCPT TO command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: BF91036F5E0
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 13:16:48 +0700 (WIT)

Final-Recipient: rfc822; bush@lists.debian.org
Original-Recipient: rfc822;bush@lists.debian.org
Action: failed
Status: 5.1.1
Remote-MTA: dns; liszt.debian.org
Diagnostic-Code: smtp; 550 5.1.1 bush@lists.debian.org: Recipient address
	rejected: User unknown in virtual alias table
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id BF91036F5E0
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 13:16:48 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 1E3C32A869B
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 09:01:00 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: Mail Transaction Failed
Date: Thu, 10 Jul 2008 09:00:59 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0001_1A43E3E1.9BF30690
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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] add 'license' command to u-boot?commandline

2008-07-10 Thread Detlev Zundel

Hi,

Wolfgang Denk wrote:
In message [EMAIL PROTECTED] you wrote:
Bleah on the small C program (nothing personal, just hate to add
Yet Another Program to the build process). How about using echo
for prelim text (if necessary) and xxd to create the struct
instead? I like making maintenance Somebody Else's Problem
(SEP[1])

$ man xxd

How standard is xxd across distributions and operating systems?

Best regards,

Wolfgang Denk

In debian, it is (also) provided by the vim-common package. It looks
like vim (the project) is the origin of the command. That was
Harald's objection to using xxd. Unfortunately, it appears to be a
reasonable objection.

$ dpkg -S /usr/bin/xxd
vim-common: /usr/bin/xxd

IMHO, vim and xxd are essential programs so I'm OK with using
xxd. OTOH, I understand some people actually prefer emacs and may have
a problem with requiring vim to be installed. ;-/

Even though I'm an enthusiastic Emacs user, I have different reasons for
my vote. As far as build dependencies go, I'd like to restrict this to
the LSB standard[1]. Xdd is not in there, so thats why I vote for the
small C program (unless we can adapt the script and use 'od').

Cheers
Detlev

[1]
http://refspecs.freestandards.org/LSB_3.2.0/LSB-Core-generic/LSB-Core-generic/toccommand.html

--
Perfection (in design) is achieved not when there is nothing more to add,
but rather when there is nothing more to take away.
-- Antoine de Saint-Exupery
--
DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: [EMAIL PROTECTED]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Tftp command !!

2008-07-10 Thread Amit Kumar


Dear Wolfgang Denk,

 If you had to make changes, this means that you are not running on the
very same hardware as trhe board configuration was made for.

[Amit]: If you look at my so called changes they don't make any difference. 
For example #defining one FLAG does not mean that there is change. README 
clearly mentions that if one wants particular piece of code to be enabled then 
#define it. Secondly, there is increase in number of sectors.
I think you have not looked at what I meant by changes.

 Obviously you failed to port U-Boot correctly to your hardware. This
can be anything.

[Amit]: In my case there is no need of porting because as per my above 
explanation there is no change in the code and I am able to run commands from 
hyper terminal like saveenv, setenv, erase, fli, bdi ... etc

 We cannot ignore it. It is legal advice from your company, isn't it?

[Amit]: As per the disclaimer If you are not the intended recipient and you 
copy, disclose, or distribute this e-mail or its contents to any other person 
then such actions are unlawful. In this case, intended recipient is 
u-boot-users@lists.sourceforge.net ... so I think there should not be any 
problem.

 This means that your posting of this message to a public maling list was 
 unlawful. Please do not do this again. Such incidents may be
reported to your management.

[Amit]: By my above explanation, I don't think I have done anything wrong. If 
you still think I have done something wrong or I have missed something, kindly 
explain me.

Regards,
Amit Kumar
Project Manager
Infosys, India

-Original Message-
From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
Sent: Wednesday, July 09, 2008 5:43 PM
To: Amit Kumar
Cc: u-boot-users@lists.sourceforge.net
Subject: Re: [U-Boot-Users] Tftp command !!

Dear Amit,

in message [EMAIL PROTECTED] you wrote:

 In this code, I have made only two changes:

If you had to make changes, this means that you are not running on the
very same hardware as trhe board configuration was made for.

This WILL NOT work.

If you have custom hardware, you will have to port U-Boot to that
hardware.

 Any idea why file transfer is not happening. I have already followed step 4.6 
 of DULG-tqm8xxl.pdf

Obviously you failed to port U-Boot correctly to your hardware. This
can be anything.

 Amit
 P.S: Please ignore the disclaimer as I cannot remove it.

We cannot ignore it. It is legal advice from your company, isn't it?

  CAUTION - Disclaimer *
 This e-mail contains PRIVILEGED AND CONFIDENTIAL INFORMATION intended solely
 for the use of the addressee(s). If you are not the intended recipient, please
 notify the sender by e-mail and delete the original message. Further, you are 
 not
 to copy, disclose, or distribute this e-mail or its contents to any other 
 person and
 any such actions are unlawful. This e-mail may contain viruses. Infosys has 
 taken

This means that your posting of this message to a public maling list
was unlawful. Please do not do this again. Such incidents may be
reported to your management.

 --_000_3F8E087F6663404B945D1FCF703777D115C5C7DD93BLRKECMBX06ad_
 Content-Type: text/html;
   charset=utf-8
 Content-Transfer-Encoding: quoted-printable

And do not post HTML here.

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 express preference for a chronological  sequence  of  events  which
precludes a violence.   - Terry Pratchett, _The Dark Side of the Sun_

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] usb: add support for R8A66597 usb controller

In message [EMAIL PROTECTED] you wrote:

  is it not possible to have a better name for the struct and the var
  and btw for the function naming?
 
 Because XTAL, LDRV, DEVADD are a register name or register bit name,
 I think that it is a good name...
 But I will change a name of the struct r8a66957 gr8a66597 to
 struct r8a66597 r8a66597_dev.

Can you not find some simpler name that r8a66597? That's a mess to
type and to read.

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]
Crash programs fail because they are based on the theory  that,  with
nine women pregnant, you can get a baby a month.  - Wernher von Braun

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] add 'license' command to u-boot?commandline

On Thursday 10 July 2008, Detlev Zundel wrote:
  IMHO, vim and xxd are essential programs so I'm OK with using
  xxd. OTOH, I understand some people actually prefer emacs and may have
  a problem with requiring vim to be installed.  ;-/

 Even though I'm an enthusiastic Emacs user, I have different reasons for
 my vote.  As far as build dependencies go, I'd like to restrict this to
 the LSB standard[1].  Xdd is not in there, so thats why I vote for the
 small C program (unless we can adapt the script and use 'od').

I would prefer this C program solution too.

Best regards,
Stefan

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] Remove [EMAIL PROTECTED] from MAINTAINERS

Mail to [EMAIL PROTECTED] bounces because the user doesn't exist
anymore. You can't be a maintainer without a valid e-mail address, so
move all boards that used to be maintained by Kyle Harris to the
orphaned list.

Currently, only PowerPC has a list of orphaned boards, so this patch
creates one for ARM as well.

Signed-off-by: Haavard Skinnemoen [EMAIL PROTECTED]
---
On Sun, 06 Jul 2008 00:32:01 +0200 Wolfgang Denk [EMAIL PROTECTED] wrote:
 ... don't jusst delete the orphaned boards, but add them to the
 Unknown / orphaned list.

I created a new Unknown/orphaned list for ARM since the existing one
was located in the PowerPC section and only had PowerPC boards on it.
Is that ok?

 MAINTAINERS |   14 --
 1 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a3d70b1..6065e42 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -492,12 +492,6 @@ Kshitij Gupta [EMAIL PROTECTED]
omap1510inn ARM925T
omap1610inn ARM926EJS
 
-Kyle Harris [EMAIL PROTECTED]
-
-   lubbock xscale
-   cradle  xscale
-   ixdp425 xscale
-
 Gary Jennejohn [EMAIL PROTECTED]
 
smdk2400ARM920T
@@ -591,6 +585,14 @@ Michael Schwingen [EMAIL PROTECTED]
actux3  xscale
actux4  xscale
 
+-
+
+Unknown / orphaned boards:
+
+   cradle  xscale
+   ixdp425 xscale
+   lubbock xscale
+
 #
 # x86 Systems: #
 #  #
-- 
1.5.4.3


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] add 'license' command to u-boot?commandline

2008-07-10 Thread Harald Welte

On Thu, Jul 10, 2008 at 12:51:03PM +0200, Detlev Zundel wrote:

 Even though I'm an enthusiastic Emacs user, I have different reasons for
 my vote.  As far as build dependencies go, I'd like to restrict this to
 the LSB standard[1].  Xdd is not in there, so thats why I vote for the
 small C program (unless we can adapt the script and use 'od').

I think sticking to LSB for build dependencies is a very reasonable choice.

I still don't think the small C program is that much of a problem:

1) dependency tracking should ensure it is only built when the license
   feature is enabled by the board-level config.  I think this will be
   a minority case.

2) compilation of native programs has already to be possible since there
   are other native tools such as mkimage built during the compilation

Cheers,
-- 
- Harald Welte [EMAIL PROTECTED]  http://openmoko.org/

Software for the world's first truly open Free Software mobile phone

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] add 'license' command to u-boot?commandline

Harald Welte wrote:
 On Thu, Jul 10, 2008 at 12:51:03PM +0200, Detlev Zundel wrote:
 
 Even though I'm an enthusiastic Emacs user, I have different reasons for
 my vote.  As far as build dependencies go, I'd like to restrict this to
 the LSB standard[1].  Xdd is not in there, so thats why I vote for the
 small C program (unless we can adapt the script and use 'od').
 
 I think sticking to LSB for build dependencies is a very reasonable choice.
 
 I still don't think the small C program is that much of a problem:
 
 1) dependency tracking should ensure it is only built when the license
feature is enabled by the board-level config.  I think this will be
a minority case.
 
 2) compilation of native programs has already to be possible since there
are other native tools such as mkimage built during the compilation
 
 Cheers,

FWIIW, I agree with Detlev  Harald and withdraw my bleah objection.

Best regards,
gvb


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Release status - things to be done

[removing my private e-mail address from Cc]

On Sun, 06 Jul 2008 01:05:40 +0200
Wolfgang Denk [EMAIL PROTECTED] wrote:

 2582  05/21 Haavard Skinnemoe  [PATCH] MMC: Consolidate MMC/SD command 
 definitions
   - Rejected, needs to be rebased  

I see you applied v2 instead -- good. Please ignore v1 as it was posted
before the big whitespace cleanup.

  2584  05/21 Haavard Skinnemoe  [PATCH] Remove [EMAIL PROTECTED] from 
 MAINTAINERS
   - asked submitter for fixes / cleanup  

I'll send you a new patch later today...if I can find the original
commit.

 3648  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 1/6] AT572D940HF-EB 
 Support
  3649  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 2/6] AT572D940HF-EB 
 Support
  3650  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 3/6] AT572D940HF-EB 
 Support
  3651  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 4/6] AT572D940HF-EB 
 Support
  3652  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 5/6] AT572D940HF-EB 
 Support
  3653  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 6/6] AT572D940HF-EB 
 Support
  3654  06/11 Antonio R. Costa  [U-Boot-Users] [PATCH 1/3] SDHC Support for 
 AT572d940HF-EB
  3733  06/12 Ulf Samuelsson   Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3735  06/12 COSTA, Antonio   Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3737  06/12 Ulf Samuelsson   Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3739  06/12 Jerry Van BarenRe: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3859  06/15 Haavard Skinnemoe  Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3897  06/16 [EMAIL PROTECTED]  Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3899  06/17 Haavard Skinnemoe  Re: [U-Boot-Users] [PATCH 1/3] SDHC Support 
 for AT572d940HF-EB
  3740  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 1/6] AT572D940HF-EB 
 Support v2 (board folder)
  3741  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 2/2] AT572D940HF-EB 
 Support v2 (SDHC support part 2)
  3742  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 1/2] AT572D940HF-EB 
 Support v2 (SDHC support part 1)
  3743  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 3/6] AT572D940HF-EB 
 Support v2 (include files part 1)
  3744  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 5/6] AT572D940HF-EB 
 Support v2 (ethernet files)
  3745  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 4/6] AT572D940HF-EB 
 Support v2 (include files part 2)
  3747  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 2/6] AT572D940HF-EB 
 Support v2 (cpu folder)
  3748  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 6/6] AT572D940HF-EB 
 Support v2 (configuration files)
  3746  06/12 Antonio R. Costa  [U-Boot-Users] [PATCH 1/1] mmc_verbosity 
 variable
  3764  06/12 Haavard Skinnemoe  Re: [U-Boot-Users] [PATCH 1/2] AT572D940HF-EB 
 Support v2 (SDHC support part 1)
   - still under discussion;  who is reponsible?  

I think Jean-Christophe should be responsible for most of this, but I
obviously want to be kept in the loop with the MMC/SD/SDHC stuff and
possibly other stuff that is or ought to be shared with avr32.

Now that the MMC driver has been moved into drivers/mmc, I'd really
like to see a patch with the minimum amount of changes needed to make
the driver work on AT91 and AT57 as well. With that in place, we'll
have a great baseline to add new features like SDHC and MMC+ support,
and the rest of the AT57 board support can be merged independently.

Haavard

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 0/1] Moved initialization of AVR32 Ethernet controllers to board_eth_init()

On Sat,  5 Jul 2008 00:08:47 -0700
Ben Warren [EMAIL PROTECTED] wrote:

 More cleanup of net/eth.c, this time taking care of AVR32 boards.  I got rid
 of the board/eth.c files because board_eth_init() needs to be in a file where
 there are other strong symbols in order to override the weak board_eth_init() 
 in
 net/eth.c.  I've compiled this code  and verified that the strong
 board_eth_init() symbols are linked in, but don't have hardware to test on.
 If somebody can, please try out and let me know.

Looks good to me, but I don't have the chance to test it.

 These patches are being staged in the 'testing' branch of the net repo.

Maybe someone else could give it a try?

Haavard

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 v2] Remove [EMAIL PROTECTED] from MAINTAINERS

Haavard Skinnemoen wrote:
 Mail to [EMAIL PROTECTED] bounces because the user doesn't exist
 anymore. You can't be a maintainer without a valid e-mail address, so
 move all boards that used to be maintained by Kyle Harris to the
 orphaned list.
 
 Currently, only PowerPC has a list of orphaned boards, so this patch
 creates one for ARM as well.
 
 Signed-off-by: Haavard Skinnemoen [EMAIL PROTECTED]
 ---
 On Sun, 06 Jul 2008 00:32:01 +0200 Wolfgang Denk [EMAIL PROTECTED] wrote:
 ... don't jusst delete the orphaned boards, but add them to the
 Unknown / orphaned list.
 
 I created a new Unknown/orphaned list for ARM since the existing one
 was located in the PowerPC section and only had PowerPC boards on it.
 Is that ok?
 
  MAINTAINERS |   14 --
  1 files changed, 8 insertions(+), 6 deletions(-)
 
 diff --git a/MAINTAINERS b/MAINTAINERS
 index a3d70b1..6065e42 100644
 --- a/MAINTAINERS
 +++ b/MAINTAINERS
 @@ -492,12 +492,6 @@ Kshitij Gupta [EMAIL PROTECTED]
   omap1510inn ARM925T
   omap1610inn ARM926EJS
  
 -Kyle Harris [EMAIL PROTECTED]
 -
 - lubbock xscale
 - cradle  xscale
 - ixdp425 xscale
 -
  Gary Jennejohn [EMAIL PROTECTED]
  
   smdk2400ARM920T
 @@ -591,6 +585,14 @@ Michael Schwingen [EMAIL PROTECTED]
   actux3  xscale
   actux4  xscale
  
 +-
 +
 +Unknown / orphaned boards:
 +
 + cradle  xscale
 + ixdp425 xscale
 + lubbock xscale
 +
  #
  # x86 Systems:   
 #
  ##

Would it be reasonable to add an optional tag last known maintainer 
for orphaned boards so someone can chase it without pawing through git 
history?  E.g.:

+-
+
+Unknown / orphaned boards:
+
+Last known maintainer: Kyle Harris [EMAIL PROTECTED]
+   cradle  xscale  
+   ixdp425 xscale  
+   lubbock xscale  
+

Best regards,
gvb

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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: Remove redundant ft_board_setup() functions from some 4xx boards

This patch removes some ft_board_setup() functions from some 4xx boards.
This can be done since we now have a default weak implementation for this
in cpu/ppc4xx/fdt.c. Only board in need for a different/custom
implementation like canyonlands need their own version.

Signed-off-by: Stefan Roese [EMAIL PROTECTED]
---
 board/amcc/katmai/katmai.c |   21 -
 board/amcc/kilauea/kilauea.c   |   21 -
 board/amcc/makalu/makalu.c |   21 -
 board/amcc/sequoia/sequoia.c   |   21 -
 board/amcc/yosemite/yosemite.c |   21 -
 board/esd/pmc440/pmc440.c  |   21 -
 board/prodrive/alpr/alpr.c |   21 -
 cpu/ppc4xx/fdt.c   |   12 ++--
 8 files changed, 10 insertions(+), 149 deletions(-)

diff --git a/board/amcc/katmai/katmai.c b/board/amcc/katmai/katmai.c
index 3a0b18f..f2bed5c 100644
--- a/board/amcc/katmai/katmai.c
+++ b/board/amcc/katmai/katmai.c
@@ -517,24 +517,3 @@ int post_hotkeys_pressed(void)
return (ctrlc());
 }
 #endif
-
-#if defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP)
-void ft_board_setup(void *blob, bd_t *bd)
-{
-   u32 val[4];
-   int rc;
-
-   ft_cpu_setup(blob, bd);
-
-   /* Fixup NOR mapping */
-   val[0] = 0; /* chip select number */
-   val[1] = 0; /* always 0 */
-   val[2] = gd-bd-bi_flashstart;
-   val[3] = gd-bd-bi_flashsize;
-   rc = fdt_find_and_setprop(blob, /plb/opb/ebc, ranges,
- val, sizeof(val), 1);
-   if (rc)
-   printf(Unable to update property NOR mapping, err=%s\n,
-  fdt_strerror(rc));
-}
-#endif /* defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP) */
diff --git a/board/amcc/kilauea/kilauea.c b/board/amcc/kilauea/kilauea.c
index f30dc8f..7b10255 100644
--- a/board/amcc/kilauea/kilauea.c
+++ b/board/amcc/kilauea/kilauea.c
@@ -374,24 +374,3 @@ int post_hotkeys_pressed(void)
return 0;   /* No hotkeys supported */
 }
 #endif /* CONFIG_POST */
-
-#if defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP)
-void ft_board_setup(void *blob, bd_t *bd)
-{
-   u32 val[4];
-   int rc;
-
-   ft_cpu_setup(blob, bd);
-
-   /* Fixup NOR mapping */
-   val[0] = 0; /* chip select number */
-   val[1] = 0; /* always 0 */
-   val[2] = gd-bd-bi_flashstart;
-   val[3] = gd-bd-bi_flashsize;
-   rc = fdt_find_and_setprop(blob, /plb/opb/ebc, ranges,
- val, sizeof(val), 1);
-   if (rc)
-   printf(Unable to update property NOR mapping, err=%s\n,
-  fdt_strerror(rc));
-}
-#endif /* defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP) */
diff --git a/board/amcc/makalu/makalu.c b/board/amcc/makalu/makalu.c
index 9baec9a..2b4d3d4 100644
--- a/board/amcc/makalu/makalu.c
+++ b/board/amcc/makalu/makalu.c
@@ -330,24 +330,3 @@ int post_hotkeys_pressed(void)
return 0;   /* No hotkeys supported */
 }
 #endif /* CONFIG_POST */
-
-#if defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP)
-void ft_board_setup(void *blob, bd_t *bd)
-{
-   u32 val[4];
-   int rc;
-
-   ft_cpu_setup(blob, bd);
-
-   /* Fixup NOR mapping */
-   val[0] = 0; /* chip select number */
-   val[1] = 0; /* always 0 */
-   val[2] = gd-bd-bi_flashstart;
-   val[3] = gd-bd-bi_flashsize;
-   rc = fdt_find_and_setprop(blob, /plb/opb/ebc, ranges,
- val, sizeof(val), 1);
-   if (rc)
-   printf(Unable to update property NOR mapping, err=%s\n,
-  fdt_strerror(rc));
-}
-#endif /* defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP) */
diff --git a/board/amcc/sequoia/sequoia.c b/board/amcc/sequoia/sequoia.c
index 5ff9787..b833092 100644
--- a/board/amcc/sequoia/sequoia.c
+++ b/board/amcc/sequoia/sequoia.c
@@ -509,24 +509,3 @@ int post_hotkeys_pressed(void)
return 0;   /* No hotkeys supported */
 }
 #endif /* CONFIG_POST */
-
-#if defined(CONFIG_OF_LIBFDT)  defined(CONFIG_OF_BOARD_SETUP)
-void ft_board_setup(void *blob, bd_t *bd)
-{
-   u32 val[4];
-   int rc;
-
-   ft_cpu_setup(blob, bd);
-
-   /* Fixup NOR mapping */
-   val[0] = 0; /* chip select number */
-   val[1] = 0; /* always 0 */
-   val[2] = gd-bd-bi_flashstart;
-   val[3] = gd-bd-bi_flashsize;
-   rc = fdt_find_and_setprop(blob, /plb/opb/ebc, ranges,
- val, sizeof(val), 1);
-   if (rc)
-   printf(Unable to update property NOR mapping, err=%s\n,
-  fdt_strerror(rc));
-}
-#endif /* defined(CONFIG_OF_LIBFDT)

Re: [U-Boot-Users] [PATCH v2] Remove [EMAIL PROTECTED] from MAINTAINERS

On 07:35 Thu 10 Jul , Jerry Van Baren wrote:
  +
  +Unknown / orphaned boards:
  +
  +   cradle  xscale
  +   ixdp425 xscale
  +   lubbock xscale
  +
   #
   # x86 Systems: 
  #
   #  #
 
 Would it be reasonable to add an optional tag last known maintainer 
 for orphaned boards so someone can chase it without pawing through git 
 history?  E.g.:
 
 +-
 +
 +Unknown / orphaned boards:
 +
 +Last known maintainer: Kyle Harris [EMAIL PROTECTED]
 + cradle  xscale  
 + ixdp425 xscale  
 + lubbock xscale  
 +
 
Good idea

Best Regards,
J.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] usb: add support for R8A66597 usb controller

2008/07/10 20:01, Wolfgang Denk wrote:
 In message [EMAIL PROTECTED] you wrote:
 is it not possible to have a better name for the struct and the var
 and btw for the function naming?
 Because XTAL, LDRV, DEVADD are a register name or register bit name,
 I think that it is a good name...
 But I will change a name of the struct r8a66957 gr8a66597 to
 struct r8a66597 r8a66597_dev.
 
 Can you not find some simpler name that r8a66597? That's a mess to
 type and to read.

Thank you for your comment.

I can change that struct r8a66597 for a simple name. For example,
its name is struct r597. However, I do not want to change a source
file name and CONFIG_ name, because R8A66597 is product name.

Best Regards,
Yoshihiro Shimoda


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] USB: shutdown USB before booting


This patch fixes a potentially serious issue related to USB which was
discouvered by Martin Krause [EMAIL PROTECTED] and fixed for
ARM920T. Martin wrote:

  Turn off USB to prevent the host controller from writing to the
  SDRAM while Linux is booting. This could happen, because the HCCA
  (Host Controller Communication Area) lies within the SDRAM and the
  host controller writes continously to this area (as busmaster!), for
  example to increase the HccaFrameNumber variable, which happens
  every 1 ms.

This is a slightly modified version of the patch in order to shutdown
USB when booting on all architectures.

Signed-off-by: Markus Klotzbuecher [EMAIL PROTECTED]
---
 common/cmd_bootm.c |   18 ++
 1 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
index 959689e..f914815 100644
--- a/common/cmd_bootm.c
+++ b/common/cmd_bootm.c
@@ -36,6 +36,10 @@
 #include lmb.h
 #include asm/byteorder.h
 
+#if (CONFIG_COMMANDS  CFG_CMD_USB)
+#include usb.h
+#endif
+
 #ifdef CFG_HUSH_PARSER
 #include hush.h
 #endif
@@ -213,6 +217,20 @@ int do_bootm (cmd_tbl_t *cmdtp, int flag, int argc, char 
*argv[])
 */
iflag = disable_interrupts();
 
+#if (CONFIG_COMMANDS  CFG_CMD_USB)
+   /*
+* turn off USB to prevent the host controller from writing to the
+* SDRAM while Linux is booting. This could happen (at least for OHCI
+* controller), because the HCCA (Host Controller Communication Area)
+* lies within the SDRAM and the host controller writes continously to
+* this area (as busmaster!). The HccaFrameNumber is for example
+* updated every 1 ms within the HCCA structure in SDRAM! For more
+* details see the OpenHCI specification.
+*/
+   usb_stop();
+#endif
+
+
 #ifdef CONFIG_AMIGAONEG3SE
/*
 * We've possible left the caches enabled during
-- 
1.5.5.1


Best regards

Markus Klotzbuecher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 0/1] Moved initialization of AVR32 Ethernet controllers to board_eth_init()

2008-07-10 Thread Hans-Christian Egtvedt

On Thu, 2008-07-10 at 13:24 +0200, Haavard Skinnemoen wrote:
 On Sat,  5 Jul 2008 00:08:47 -0700
 Ben Warren [EMAIL PROTECTED] wrote:
 
  More cleanup of net/eth.c, this time taking care of AVR32 boards.  I got rid
  of the board/eth.c files because board_eth_init() needs to be in a file 
  where
  there are other strong symbols in order to override the weak 
  board_eth_init() in
  net/eth.c.  I've compiled this code  and verified that the strong
  board_eth_init() symbols are linked in, but don't have hardware to test on.
  If somebody can, please try out and let me know.
 
 Looks good to me, but I don't have the chance to test it.
 

I tried looking into it, but I do not see who is going to call
board_eth_init() ? Is this supposed to be done in net/eth.c or
lib_avr32/board.c ?

  These patches are being staged in the 'testing' branch of the net repo.
 
 Maybe someone else could give it a try?
 

I tried it (just reworked a bit to my board from EarthLCD) and it does
not detect the PHY. I am working on master from the AVR32 branch in GIT.

PS! I am not on u-boot-users, and is there a archive which is faster to
scroll than the one on sf.net?

-- 
With kind regards,
Hans-Christian Egtvedt, Applications Engineer


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 0/1] Moved initialization of AVR32 Ethernet controllers to board_eth_init()

2008-07-10 Thread Hans-Christian Egtvedt

On Thu, 2008-07-10 at 14:45 +0200, Hans-Christian Egtvedt wrote:
 On Thu, 2008-07-10 at 13:24 +0200, Haavard Skinnemoen wrote:
  On Sat,  5 Jul 2008 00:08:47 -0700
  Ben Warren [EMAIL PROTECTED] wrote:
  
   More cleanup of net/eth.c, this time taking care of AVR32 boards.  I got 
   rid
   of the board/eth.c files because board_eth_init() needs to be in a file 
   where
   there are other strong symbols in order to override the weak 
   board_eth_init() in
   net/eth.c.  I've compiled this code  and verified that the strong
   board_eth_init() symbols are linked in, but don't have hardware to test 
   on.
   If somebody can, please try out and let me know.
  
  Looks good to me, but I don't have the chance to test it.
  
 
 I tried looking into it, but I do not see who is going to call
 board_eth_init() ? Is this supposed to be done in net/eth.c or
 lib_avr32/board.c ?
 

Cheese, I am blind, just found it almost on top in the eth_initialize()
function.

I will pull the master and see if it works.

snipp

-- 
With kind regards,
Hans-Christian Egtvedt, Applications Engineer


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 0/1] Moved initialization of AVR32 Ethernet controllers to board_eth_init()

2008-07-10 Thread Hans-Christian Egtvedt

On Thu, 2008-07-10 at 14:47 +0200, Hans-Christian Egtvedt wrote:
 On Thu, 2008-07-10 at 14:45 +0200, Hans-Christian Egtvedt wrote:
  On Thu, 2008-07-10 at 13:24 +0200, Haavard Skinnemoen wrote:

snipp

   Looks good to me, but I don't have the chance to test it.
   
  
  I tried looking into it, but I do not see who is going to call
  board_eth_init() ? Is this supposed to be done in net/eth.c or
  lib_avr32/board.c ?
  
 
 Cheese, I am blind, just found it almost on top in the eth_initialize()
 function.
 
 I will pull the master and see if it works.
 

And it works, proof below (-: (Net: did not work before pulling master)

U-Boot 1.3.3-00248-gae9bc0c-dirty (Jul 10 2008 - 14:50:32) 

U-Boot code:  - f6b8  data: 000155c0 - 0004bd18
malloc: Using memory from 0x10f74000 to 0x10fb4000
DMA: Using memory from 0x10f7 to 0x10f74000
Flash:  8 MB at address 0x
DRAM Configuration:
Bank #0: 1000 16 MB
In:serial
Out:   serial
Err:   serial
Net:   macb0
Press SPACE to abort autoboot in 1 seconds
macb0: link up, 100Mbps full-duplex (lpa: 0x45e1)
BOOTP broadcast 1
DHCP client bound to address 10.191.252.36
Using macb0 device
TFTP from server 10.191.252.83; our IP address is 10.191.252.36
Filename 'uimage_br_stk'.
Load address: 0x1040
Loading: T #
 ###
done
Bytes transferred = 788 (10f6ec hex)
## Booting kernel from Legacy Image at 1040 ...
   Image Name:   Linux-2.6.25.10.atmel.2
   Image Type:   AVR32 Linux Kernel Image (gzip compressed)
   Data Size:724 Bytes =  1.1 MB
   Load Address: 1000
   Entry Point:  9000
   Verifying Checksum ... OK
   Uncompressing Kernel Image ... OK

Starting kernel at 9000 (params at 10f74008)...

Linux version 2.6.25.10.atmel.2 ([EMAIL PROTECTED]) (gcc version 
4.2.2-atmel.1.0.8) #68 Thu Jul 10 13:16:42 CEST 2008

-- 
With kind regards,
Hans-Christian Egtvedt, Applications Engineer


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] USB: shutdown USB before booting

Hi Markus,

On Thursday 10 July 2008, Markus Klotzbücher wrote:
 This patch fixes a potentially serious issue related to USB which was
 discouvered by Martin Krause [EMAIL PROTECTED] and fixed for
 ARM920T. Martin wrote:

   Turn off USB to prevent the host controller from writing to the
   SDRAM while Linux is booting. This could happen, because the HCCA
   (Host Controller Communication Area) lies within the SDRAM and the
   host controller writes continously to this area (as busmaster!), for
   example to increase the HccaFrameNumber variable, which happens
   every 1 ms.

 This is a slightly modified version of the patch in order to shutdown
 USB when booting on all architectures.

Yes, I remember hearing of this problem. I thought it was already fixed. Seems 
not to be the case. Please find a comment below.

 Signed-off-by: Markus Klotzbuecher [EMAIL PROTECTED]
 ---
  common/cmd_bootm.c |   18 ++
  1 files changed, 18 insertions(+), 0 deletions(-)

 diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c

By implementing this usb_stop() in do_bootm() you prevent this problem in many 
cases (e.g. Linux booting). But unfortunately not in all cases. For 
example bootelf or bootvx are not fixed.

Can't this be solved in a different way. By calling usb_stop each time after 
an USB command completes. IIRC, this is how it is done in the U-Boot network 
implementation too.

What do you think?

Best regards,
Stefan

=
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]
=

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Re: [U-Boot-Users] Tftp command !!

Dear Amit,

Preliminary comment: Wolfgang is lashing out at stupid disclaimers, 
don't take it personally.  We all realize you didn't write the stupid 
disclaimer and cannot remove it.

Amit Kumar wrote:
 Dear Wolfgang,
 
 Kindly read disclaimer properly or show to your lawyer.
 
 This e-mail contains PRIVILEGED AND CONFIDENTIAL INFORMATION
 intended solely for the use of the addressee(s).

No it doesn't.  Your email was neither PRIVILEGED nor CONFIDENTIAL.

In the USofA, the term privileged as used in this context is a 
specific type of communications with a lawyer who is in a formal, legal 
way your (or your company's) representative.  This email clearly falls 
outside of that category.

Email is *never* CONFIDENTIAL if it is sent unencrypted on an 
unprotected network.  If it is encrypted and/or on a network internal to 
your company or over a secured VPN, you may have a reasonable 
expectation that the email is confidential within the bounds of your 
company and/or the limitations of the encryption.  This email clearly 
falls outside of that category.

Further, this email list is known to be archived by several servers 
(e.g. sourceforge, gmane, nabble) which are then indexed and cached by 
several search engines (e.g. google, yahoo, microsoft), not to mention 
snapshotted by archivers (e.g. the wayback engine).  Anyone now and in 
the future that has internet access can read it.  Obviously the list of 
addressees does not include everyone in the present and definitely not 
everyone in the future.

 You are not understanding the disclaimer at all. I am highly
 surprised. If your lawyer says that you are correct then as mentioned
 by you earlier you can report to management of my company.

If your company's legal disclaimer was actually written by a lawyer, and 
I strongly suspect it was not, that lawyer is either totally ignorant of 
the inherent characteristics of email or totally incompetent.  If the 
former, he should be educated on the inherent characteristics of email 
(see the analogy below).  If the latter, your company should replace him.

Rather than putting stupid useless disclaimers on non-PRIVILEGED, 
non-CONFIDENTIAL emails, a competent lawyer would/should give competent 
legal advice like DON'T EVER SEND PRIVILEGED OR CONFIDENTIAL INFORMATION 
over a service that has no expectations of privacy or confidentiality 
and go medieval on anybody that disregarded that advice.

As an analogy, sending PRIVILEGED AND CONFIDENTIAL information over 
email is *worse* than sending PRIVILEGED AND CONFIDENTIAL information 
written on a postcard and mailed out.  For a postcard, anybody between 
your outbox and the recipient's inbox can read the postcard, *most* of 
them *legally.*  This includes all of the mail handlers in your company, 
all of the mail handlers in the route from your company to the 
destination company (including any government security services, 
especially if it crosses a border) and potentially industrial spies (OK, 
*they* might be reading the postcard illegally, but legalities never 
stop a good spy).

 You are preventing people like us from sharing information on this
 esteemed mailing list for which I think you are the moderator.

Relax.  The information is still being shared.

 Regards,
 Amit

HTH,
gvb

 -Original Message-
 From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
 Sent: Thursday, July 10, 2008 4:37 PM
 To: Amit Kumar
 Cc: u-boot-users@lists.sourceforge.net
 Subject: Re: [U-Boot-Users] Tftp command !!
 
 In message [EMAIL PROTECTED] you wrote:
 If you had to make changes, this means that you are not running on the
 very same hardware as trhe board configuration was made for.
 
 I don't really believe that.
 
 [Amit]: If you look at my so called changes they don't make any 
 difference. For example #defining one FLAG does not mean that there is 
 change. README clearly mentions that if one wants particular piece of code 
 to be enabled then #define it. Secondly,
 there is increase in number of sectors.
 
 But on the  hardware as trhe board configuration was made for, the
 number ofd sectors is OK. Which means you must have somewhat different
 hardware.
 
 But if you don't believe me I can't help it. Feel free to find your
 own way.
 
 [Amit]: As per the disclaimer If you are not the intended recipient and you 
 copy, disclose, or distribute this e-mail or its contents to any other 
 person then such actions are unlawful. In this case, intended recipient is 
 [EMAIL PROTECTED]
 
 e.net ... so I think there should not be any problem.
 
 A mailing-list is not a person in the legal sense. Ask you lawyer. You
 are in trouble.
 
 
 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]
 When a man sits with a pretty girl for  an  hour,  it  seems  like  a
 minute.  But

[U-Boot-Users] Undelivered Mail Returned to Sender

This is the mail system at host efw-budpar.budpar.go.id.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

   The mail system

[EMAIL PROTECTED]: host mx1.mail.sg1.yahoo.com[124.108.116.72] said: 554
delivery error: dd This user doesn't have a yahoo.co.id account
([EMAIL PROTECTED]) [-5] - mta131.mail.sg1.yahoo.com (in reply to end of 
DATA
command)
Reporting-MTA: dns; efw-budpar.budpar.go.id
X-Postfix-Queue-ID: E8FA4359B8C
X-Postfix-Sender: rfc822; u-boot-users@lists.sourceforge.net
Arrival-Date: Thu, 10 Jul 2008 15:56:31 +0700 (WIT)

Final-Recipient: rfc822; leo@yahoo.co.id
Original-Recipient: rfc822;leo@yahoo.co.id
Action: failed
Status: 5.0.0
Remote-MTA: dns; mx1.mail.sg1.yahoo.com
Diagnostic-Code: smtp; 554 delivery error: dd This user doesn't have a
	yahoo.co.id account (leo@yahoo.co.id) [-5] - mta131.mail.sg1.yahoo.com
Received: from localhost (localhost [127.0.0.1])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id E8FA4359B8C
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 15:56:31 +0700 (WIT)
X-Virus-Scanned: by Endian Firewall
Received: from lists.sourceforge.net (unknown [172.17.100.133])
	by efw-budpar.budpar.go.id (Postfix) with ESMTP id 3D77144EA49
	for [EMAIL PROTECTED]; Thu, 10 Jul 2008 13:11:40 +0700 (WIT)
From: u-boot-users@lists.sourceforge.net
To: [EMAIL PROTECTED]
Subject: Error
Date: Thu, 10 Jul 2008 13:11:39 +0700
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary==_NextPart_000_0003_AC4EC6B6.B4DEDD6E
X-Priority: 3
X-MSMail-Priority: Normal
Message-Id: [EMAIL PROTECTED]
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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 1/4][RFC] Add initial high speed support to USB code.

Hi Tor,

Tor Krill [EMAIL PROTECTED] writes:

 Add high speed support to USB code. Extracted from Juniper Networks patch.

 I know that the mergewindow is closed but wanted to get feedback on these
 patches if possible.

Thanks again for fixing this up. AFAIK patches look fine, only two
 things:

- please address the issues pointed out by Michael, especially removal
  of the C macro.

- Secondly it would be nice if we could have a short README.ehci or
  similar which explains what works and what not, and what needs to be
  done to add further board/HC support.

Thank you in advance.

Best regards

Markus Klotzbücher

--
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]

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Support fot the ML507 Board 1/4

2008-07-10 Thread Ricardo

Hi List:

 This is my first contribution to u-boot. I have ported u-boot to the
ML507 Board by Xilinx.
http://www.xilinx.com/products/devkits/HW-V5-ML507-UNI-G.htm

This boards includes an FPGA Virtex 5 FX with an embedded PowerPC 440.

The port supports:
-Virtex 5 ppc440x5
-XIlinx Interrupt Controller
-Xilinx I2C Controller (Interrupted mode)
-Xilinx Uart Lite (simple port)
-Xilinx LL_TEMA (Interrupted and SGDMA)
-Save environment on board eeprom
-DTT support for the ADT sensor on board (new hwmon driver)
-Dummy I2C driver (for testing purposes)

This patch works against the last commit to the p4xx branch.

I am a researcher of the Universidad Autonoma de Madrid, and this work
has been supported by Q-Technology ( http://qtec.com ) under a
Research Agreement.

Any comment will be very welcomed.

PS: I have divided the patch in four parts for the mailing list
-- 
Ricardo Ribalda
http://www.eps.uam.es/~rribalda/


ml507_1_of_4.diff.bz2
Description: BZip2 compressed data
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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] Support fot the ML507 Board 1/4

2008-07-10 Thread Ricardo

Part 3

On Thu, Jul 10, 2008 at 6:07 PM, Ricardo [EMAIL PROTECTED] wrote:
 Part 2

 On Thu, Jul 10, 2008 at 6:06 PM, Ricardo [EMAIL PROTECTED] wrote:
 Hi List:

  This is my first contribution to u-boot. I have ported u-boot to the
 ML507 Board by Xilinx.
 http://www.xilinx.com/products/devkits/HW-V5-ML507-UNI-G.htm

 This boards includes an FPGA Virtex 5 FX with an embedded PowerPC 440.

 The port supports:
-Virtex 5 ppc440x5
-XIlinx Interrupt Controller
-Xilinx I2C Controller (Interrupted mode)
-Xilinx Uart Lite (simple port)
-Xilinx LL_TEMA (Interrupted and SGDMA)
-Save environment on board eeprom
-DTT support for the ADT sensor on board (new hwmon driver)
-Dummy I2C driver (for testing purposes)

 This patch works against the last commit to the p4xx branch.

 I am a researcher of the Universidad Autonoma de Madrid, and this work
 has been supported by Q-Technology ( http://qtec.com ) under a
 Research Agreement.

 Any comment will be very welcomed.

 PS: I have divided the patch in four parts for the mailing list
 --
 Ricardo Ribalda
 http://www.eps.uam.es/~rribalda/




 --
 Ricardo Ribalda
 http://www.eps.uam.es/~rribalda/




-- 
Ricardo Ribalda
http://www.eps.uam.es/~rribalda/


ml507_3_of_4.diff.bz2
Description: BZip2 compressed data
-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
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] add 'license' command to u-boot?commandline

On 13:08 Thu 10 Jul , Stefan Roese wrote:
 On Thursday 10 July 2008, Detlev Zundel wrote:
   IMHO, vim and xxd are essential programs so I'm OK with using
   xxd. OTOH, I understand some people actually prefer emacs and may have
   a problem with requiring vim to be installed.  ;-/
 
  Even though I'm an enthusiastic Emacs user, I have different reasons for
  my vote.  As far as build dependencies go, I'd like to restrict this to
  the LSB standard[1].  Xdd is not in there, so thats why I vote for the
  small C program (unless we can adapt the script and use 'od').
 
 I would prefer this C program solution too.

I would prefer this C program solution too.

Best Regards,
J.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Pull Request: Please pull u-boot-mpc86xx.git

2008-07-10 Thread Jon Loeliger

Wolfgang,

The following changes are available in the git repository at:

  git://www.denx.de/git/u-boot-mpc86xx.git master


Jason Jin (1):
  Feed the watchdog in u-boot for 8610 board.

Kumar Gala (2):
  MPC8641HPCN: Report board id, board version and fpga version.
  MPC8610HPCD: Report board id, board version and fpga version.

Thanks,
jdl

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Support for the ML507 Xilinx Board

2008-07-10 Thread Ricardo Ribalda Delgado

Hi List:

Sorry for the multiple submitions, but I think that this will safe
a lot of time for our main developers. BTW: This is the first time
I use git

This is my first contribution to u-boot. I have ported u-boot to the
ML507 Board by Xilinx.
http://www.xilinx.com/products/devkits/HW-V5-ML507-UNI-G.htm

This boards includes an FPGA Virtex 5 FX with an embedded PowerPC 440.

The port supports:
   -Virtex 5 ppc440x5
   -XIlinx Interrupt Controller
   -Xilinx I2C Controller (Interrupted mode)
   -Xilinx Uart Lite (simple port)
   -Xilinx LL_TEMA (Interrupted and SGDMA)
   -Save environment on board eeprom
   -DTT support for the ADT sensor on board (new hwmon driver)
   -Dummy I2C driver (for testing purposes)

This patch works against the last commit to the p4xx branch.

I am a researcher of the Universidad Autonoma de Madrid, and this work
has been supported by Q-Technology ( http://qtec.com ) under a
Research Agreement.

Any comment will be very welcomed.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] Support fot the ML507 Board 1/4

In message [EMAIL PROTECTED] you wrote:
 
 And Part  4

Could you please explain why the subject says part 1, and the
message body says part 4 ?

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]
Computers make excellent and efficient servants, but I have  no  wish
to  serve under them. Captain, a starship also runs on loyalty to one
man. And nothing can replace it or him.
-- Spock, The Ultimate Computer, stardate 4729.4

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] Support fot the ML507 Board 1/4

2008-07-10 Thread Ricardo

Hello Wolfgang

  I wanted the four parts to have the same subject, to easy the
handling of the files. The first submition was 4 .bz2 dependent
files... But later on the evening I have prepared 8 different
INdependent patches following the guidelines on the u-boot site. Sorry
for the mess. I have sent some minutes ago an apologize and a
explanation. Please check it if you have enough time. It is my first
time with git, and I am a bit confused (even after a day of reading)


 Best regards and thanks

On Thu, Jul 10, 2008 at 8:18 PM, Wolfgang Denk [EMAIL PROTECTED] wrote:
 In message [EMAIL PROTECTED] you wrote:

 And Part  4

 Could you please explain why the subject says part 1, and the
 message body says part 4 ?

 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]
 Computers make excellent and efficient servants, but I have  no  wish
 to  serve under them. Captain, a starship also runs on loyalty to one
 man. And nothing can replace it or him.
-- Spock, The Ultimate Computer, stardate 4729.4

 -
 Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
 Studies have shown that voting for your favorite open source project,
 along with a healthy diet, reduces your potential for chronic lameness
 and boredom. Vote Now at http://www.sourceforge.net/community/cca08
 ___
 U-Boot-Users mailing list
 U-Boot-Users@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/u-boot-users




-- 
Ricardo Ribalda
http://www.eps.uam.es/~rribalda/

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] -Support fot the ADT7640 Monitor chip

2008-07-10 Thread Ricardo Ribalda Delgado

Hello again

  I have modified the license of all the patches, and they are now located at:
 http://www.ii.uam.es/~rribalda/ml507.tgz
  Waiting for more comments

  Thanks for your time and comments




On Thu, Jul 10, 2008 at 8:16 PM, Ricardo Ribalda Delgado
[EMAIL PROTECTED] wrote:
 Hello Jerry

 I didn't know that I will change the license and resend the
 patches as soon as I get more feedback to keep the mailing list as
 silent as possible.

  Again: Thanks

 On Thu, Jul 10, 2008 at 8:10 PM, Jerry Van Baren [EMAIL PROTECTED] wrote:
 Ricardo Ribalda Delgado wrote:

 Signed-off-by: Ricardo Ribalda Delgado [EMAIL PROTECTED]
 ---
  drivers/hwmon/Makefile  |1 +
  drivers/hwmon/adt7460.c |   89
 +++
  drivers/hwmon/adt7460.h |   49 ++
  include/dtt.h   |2 +-
  4 files changed, 140 insertions(+), 1 deletions(-)
  create mode 100644 drivers/hwmon/adt7460.c
  create mode 100644 drivers/hwmon/adt7460.h

 [snip]

 +++ b/drivers/hwmon/adt7460.c
 @@ -0,0 +1,89 @@
 +/*   +(C) Copyright 2008
 +Ricado Ribalda, Universidad Autonoma de Madrid,
 ricardo.ribaldaatuam.es , ricardo.ribaldaatgmail.com
 +This work has been supported by: Q-Technology  http://qtec.com/
 +
 +This program is free software: you can redistribute it and/or modify
 +it under the terms of the GNU General Public License as published by
 +the Free Software Foundation, either version 3 of the License, or

 Hi Ricardo,

 GPLv3 is a problem because most of u-boot is licensed at GPLv2 (possibly
 with an upgrade clause).  The problem is, GPLv3 is not backward compatible
 with GPLv2 because it adds restrictions.
  http://www.fsf.org/licensing/licenses/

 If you wish to have this added to u-boot, you will need to change the
 licensing to GPLv2 (note that you can still keep the any later upgrade
 clause).

 [snip]

 diff --git a/drivers/hwmon/adt7460.h b/drivers/hwmon/adt7460.h
 new file mode 100644
 index 000..48666f7
 --- /dev/null
 +++ b/drivers/hwmon/adt7460.h
 @@ -0,0 +1,49 @@
 +/*   +(C) Copyright 2008
 +Ricado Ribalda, Universidad Autonoma de Madrid,
 ricardo.ribaldaatuam.es , ricardo.ribaldaatgmail.com
 +This work has been supported by: Q-Technology  http://qtec.com/+
 +This program is free software: you can redistribute it and/or modify
 +it under the terms of the GNU General Public License as published by
 +the Free Software Foundation, either version 3 of the License, or

 Ditto.

 Best regards,
 gvb




 --
 Ricardo Ribalda
 http://www.eps.uam.es/~rribalda/




-- 
Ricardo Ribalda
http://www.eps.uam.es/~rribalda/

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] Adds support for the PPC440x5 Processor on Virtex5 FX

In message [EMAIL PROTECTED] you wrote:
 
 diff --git a/cpu/ppc4xx/4xx_enet.c b/cpu/ppc4xx/4xx_enet.c
 index 4e863dc..d55ce56 100644
 --- a/cpu/ppc4xx/4xx_enet.c
 +++ b/cpu/ppc4xx/4xx_enet.c
 @@ -97,7 +97,7 @@
   * network support enabled.
   * Remark: CONFIG_405 describes Xilinx PPC405 FPGA without EMAC controller!
   */
 -#if defined(CONFIG_CMD_NET)  !defined(CONFIG_405)  
 !defined(CONFIG_IOP480)
 +#if defined(CONFIG_CMD_NET)  !defined(CONFIG_405)  
 !defined(CONFIG_IOP480)  !defined(CONFIG_440_VIRTEX5)

Line too long.

  #if !(defined(CONFIG_MII) || defined(CONFIG_CMD_MII))
  #error CONFIG_MII has to be defined!
 diff --git a/cpu/ppc4xx/4xx_uart.c b/cpu/ppc4xx/4xx_uart.c
 index a7587d4..e79c48c 100644
 --- a/cpu/ppc4xx/4xx_uart.c
 +++ b/cpu/ppc4xx/4xx_uart.c
 @@ -48,6 +48,7 @@
  #include watchdog.h
  #include asm/ppc4xx-intvec.h
  
 +#if !defined(CONFIG_440_VIRTEX5)
  #ifdef CONFIG_SERIAL_MULTI
  #include serial.h
  #endif
 @@ -58,6 +59,7 @@
  
  DECLARE_GLOBAL_DATA_PTR;
  
 +

Too many empty lines.

  #if defined(CONFIG_405GP) || defined(CONFIG_405CR) || \
  defined(CONFIG_405EP) || defined(CONFIG_405EZ) || \
  defined(CONFIG_405EX) || defined(CONFIG_440)
 @@ -873,3 +875,5 @@ int serial_tstc(void)
  #endif /* CONFIG_SERIAL_MULTI */
  
  #endif   /* CONFIG_405GP || CONFIG_405CR */
 +

Unnecessary empty line.

 +#endif
 diff --git a/cpu/ppc4xx/cpu.c b/cpu/ppc4xx/cpu.c
 index 39f439d..defbbba 100644
 --- a/cpu/ppc4xx/cpu.c
 +++ b/cpu/ppc4xx/cpu.c
 @@ -508,6 +508,11 @@ int checkcpu (void)
   puts(GT Rev. A);
   strcpy(addstr, Security/Kasumi support);
   break;
 + 
 + case PVR_VIRTEX5:
 + printf( VIRTEX5);
 + break;
 +
  

Too many empty lines.

 --- a/cpu/ppc4xx/speed.c
 +++ b/cpu/ppc4xx/speed.c
 @@ -415,7 +415,7 @@ ulong get_PCI_freq (void)
   return sys_info.freqPCI;
  }
  
 -#elif !defined(CONFIG_440GX)  !defined(CONFIG_440SP)  
 !defined(CONFIG_440SPE)
 +#elif !defined(CONFIG_440GX)  !defined(CONFIG_440SP)  
 !defined(CONFIG_440SPE) !defined(CONFIG_440_VIRTEX5)

Line too long.

 --- a/cpu/ppc4xx/start.S
 +++ b/cpu/ppc4xx/start.S
 @@ -367,6 +367,7 @@ skip_debug_init:
   /**/
   /* Setup interrupt vectors */
   /**/
 + li  r0,0

Why do you think this was necessary?

 --- a/net/eth.c
 +++ b/net/eth.c
 @@ -291,6 +291,7 @@ int eth_initialize(bd_t *bis)
   at91sam9_eth_initialize(bis);
  #endif
  
 +

Too many empoty lines.

   if (!eth_devices) {
   puts (No ethernet found.\n);
   show_boot_progress (-64);
 @@ -621,7 +622,7 @@ int eth_initialize(bd_t *bis)
   at91rm9200_miiphy_initialize(bis);
  #endif
  #if defined(CONFIG_4xx)  !defined(CONFIG_IOP480) \
 -  !defined(CONFIG_AP1000)  !defined(CONFIG_405)
 +  !defined(CONFIG_AP1000)  !defined(CONFIG_405)  
 !defined(CONFIG_440_VIRTEX5)

Line too long.


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]
Some people march to the beat of a different drummer. And some people
tango!

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] -Support fot the ADT7640 Monitor chip

2008-07-10 Thread Ricardo Ribalda Delgado

Hello


  All the comments are right, I will resend it tomorrow.


   Best regards and thanks


 --
 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]
 Earth -- mother of the most beautiful women in the universe.
-- Apollo, Who Mourns for Adonais? stardate 3468.1




-- 
Ricardo Ribalda
http://www.eps.uam.es/~rribalda/

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] AT91 pull request

Hi,
please pull
The following changes since commit 4109df6f75fc00ab7da56d286ba50149a0d16a69:
  Kim Phillips (1):
silence misc printf formatting compiler warnings

are available in the git repository at:

  git://git.denx.de/u-boot-at91.git master

Jean-Christophe PLAGNIOL-VILLARD (1):
  MAKEALL: remove duplicated at91 from ARM9 list and add LIST_at91 to arm

Sergey Lapin (1):
  DataFlash AT45DB021 support

 MAKEALL |   12 +---
 drivers/mtd/dataflash.c |   14 ++
 include/dataflash.h |1 +
 3 files changed, 16 insertions(+), 11 deletions(-)

Best Regards,
J.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 V2] Round the serial port clock divisor value returned by calc_divisor()

In message [EMAIL PROTECTED] you wrote:
 Round the serial port clock divisor value returned by
 calc_divisor().
 
 Signed-off-by: Hugo Villeneuve [EMAIL PROTECTED]
 Signed-off-by: John Roberts [EMAIL PROTECTED]
 
 ---
 
 Rounding is important, especially when using high baud rates
 values like 115200bps. When using the non-rounded value, some
 boards will work and some won't.
 
  drivers/serial/serial.c |6 +-
  1 files changed, 5 insertions(+), 1 deletions(-)
 
 diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
 index 76425d8..1192f07 100644
 --- a/drivers/serial/serial.c
 +++ b/drivers/serial/serial.c
 @@ -144,8 +144,12 @@ static int calc_divisor (NS16550_t port)
  #else
  #define MODE_X_DIV 16
  #endif
 - return (CFG_NS16550_CLK / MODE_X_DIV / gd-baudrate);
  
 + /* Compute divisor value. Normally, we should simply return:
 +  *   CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate
 +  * but we need to round that value by adding 0.5 or 8/16.
 +  * Rounding is especially important at high baud rates. */
 + return (((16 * CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate) + 8) / 16;
  }

Your patch causes problems with some boards:

Configuring for SBC8540 board...
serial.c: In function 'calc_divisor':
serial.c:153: warning: integer overflow in expression
serial.c:153: warning: integer overflow in expression

Configuring for sbc8548 board...
serial.c: In function 'calc_divisor':
serial.c:153: warning: integer overflow in expression
serial.c:153: warning: integer overflow in expression


Do you have a quick fix, or shall I back out the patch?

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]
To understand a program you must become  both  the  machine  and  the
program.

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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, resend] FDT memory and pci node fixes for MPC8260ADS

In message [EMAIL PROTECTED] you wrote:
 Signed-off-by: Matvejchikov Ilya [EMAIL PROTECTED]
 ---
  board/freescale/mpc8260ads/mpc8260ads.c |   27 +++
  cpu/mpc8260/pci.c   |   12 
  include/configs/MPC8260ADS.h|7 +++
  3 files changed, 46 insertions(+), 0 deletions(-)
...
 --- a/cpu/mpc8260/pci.c
 +++ b/cpu/mpc8260/pci.c
 @@ -33,6 +33,10 @@
  #include mpc8260.h
  #include asm/m8260_pci.h
  #include asm/io.h
 +#ifdef CONFIG_OF_LIBFDT
 +#include libfdt.h
 +#include fdt_support.h
 +#endif
 
  #if defined CONFIG_MPC8266ADS || defined CONFIG_MPC8272 || defined 
 CONFIG_PM826
  DECLARE_GLOBAL_DATA_PTR;
 @@ -449,4 +453,12 @@ void pci_mpc8250_init (struct pci_controller *hose)
   immap-im_pci.pci_emr |= cpu_to_le32 (PCI_ERROR_PCI_NO_RSP);
  }
 
 +#if defined(CONFIG_OF_LIBFDT)
 +void ft_pci_setup(void *blob, bd_t *bd)
 +{
 + do_fixup_by_prop_u32(blob, device_type, pci, 4,
 + clock-frequency, bd-pci_clk, 1);
 +}
 +#endif

Your patch causes problems:

Configuring for MPC8260ADS board...
pci.c: In function 'ft_pci_setup':
pci.c:460: error: 'bd_t' has no member named 'pci_clk'
make[1]: *** [pci.o] Error 1


Do you have a quick fix or shall I back out the patch?

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]
Drun'? 'm not drun'! You woudn' dare call m' drun' if I was sober!
 - Terry Pratchett, _Men at Arms_

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] =?utf-8?q?Fix=20build=20time=20warnings=20in=20function=20=E2=80=98mmc=5Fdecode=5Fcsd=E2=80=99:

Signed-off-by: Marcel Ziswiler [EMAIL PROTECTED]
---
 cpu/pxa/mmc.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cpu/pxa/mmc.c b/cpu/pxa/mmc.c
index 4495a80..8c529a3 100644
--- a/cpu/pxa/mmc.c
+++ b/cpu/pxa/mmc.c
@@ -535,7 +535,7 @@ static void mmc_decode_csd(uint32_t * resp)
mmc_dev.removable = 0;
mmc_dev.block_read = mmc_bread;
 
-   printf(Detected: %u blocks of %u bytes (%uMB) , mmc_dev.lba,
+   printf(Detected: %lu blocks of %lu bytes (%luMB) , (unsigned 
long)mmc_dev.lba,
   mmc_dev.blksz, mmc_dev.lba * mmc_dev.blksz / (1024 * 1024));
 }
 
-- 
1.5.4.1


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Fix some more print() format errors.

Signed-off-by: Wolfgang Denk [EMAIL PROTECTED]
---
 board/ads5121/ads5121.c   |6 --
 board/atum8548/atum8548.c |2 +-
 board/bc3450/cmd_bc3450.c |2 +-
 board/cm5200/cmd_cm5200.c |8 
 board/cm5200/fwupdate.c   |2 +-
 board/fads/flash.c|2 +-
 board/freescale/mpc8544ds/mpc8544ds.c |2 +-
 board/freescale/mpc8610hpcd/mpc8610hpcd.c |4 ++--
 board/freescale/mpc8641hpcn/mpc8641hpcn.c |4 ++--
 board/mcc200/auto_update.c|2 +-
 board/mpc8540eval/flash.c |8 +---
 board/netstal/common/nm_bsp.c |   10 ++
 board/o2dnt/flash.c   |2 +-
 board/sandburst/metrobox/metrobox.c   |2 +-
 board/spc1920/hpi.c   |3 ++-
 board/total5200/total5200.c   |   14 +++---
 board/tqc/tqm5200/cmd_stk52xx.c   |2 +-
 board/tqc/tqm8272/tqm8272.c   |4 ++--
 common/cmd_nand.c |4 ++--
 common/cmd_reginfo.c  |   22 +++---
 cpu/mpc512x/speed.c   |2 +-
 cpu/mpc86xx/traps.c   |2 +-
 drivers/net/uli526x.c |2 +-
 examples/test_burst.c |2 +-
 lib_generic/lmb.c |   20 +++-
 net/net.c |3 ++-
 26 files changed, 73 insertions(+), 63 deletions(-)

diff --git a/board/ads5121/ads5121.c b/board/ads5121/ads5121.c
index de59991..d5cee64 100644
--- a/board/ads5121/ads5121.c
+++ b/board/ads5121/ads5121.c
@@ -29,6 +29,8 @@
 #ifdef CONFIG_MISC_INIT_R
 #include i2c.h
 #endif
+#include iopin.h /* for iopin_initialize() prototype */
+
 /* Clocks in use */
 #define SCCR1_CLOCKS_EN(CLOCK_SCCR1_CFG_EN |   
\
 CLOCK_SCCR1_LPC_EN |   \
@@ -52,8 +54,7 @@ long int fixed_sdram(void);
 int board_early_init_f (void)
 {
volatile immap_t *im = (immap_t *) CFG_IMMR;
-   u32 lpcaw, tmp32;
-   int i;
+   u32 lpcaw;
 
/*
 * Initialize Local Window for the CPLD registers access (CS2 selects
@@ -206,6 +207,7 @@ long int fixed_sdram (void)
 int misc_init_r(void)
 {
u8 tmp_val;
+   extern int ads5121_diu_init(void);
 
/* Using this for DIU init before the driver in linux takes over
 *  Enable the TFP410 Encoder (I2C address 0x38)
diff --git a/board/atum8548/atum8548.c b/board/atum8548/atum8548.c
index 4d8c6fb..34f4599 100644
--- a/board/atum8548/atum8548.c
+++ b/board/atum8548/atum8548.c
@@ -53,7 +53,7 @@ int checkboard (void)
volatile ccsr_local_ecm_t *ecm = (void *)(CFG_MPC85xx_ECM_ADDR);
 
if ((uint)gur-porpllsr != 0xe00e) {
-   printf(immap size error %x\n,gur-porpllsr);
+   printf(immap size error %lx\n,(ulong)gur-porpllsr);
}
printf (Board: ATUM8548\n);
 
diff --git a/board/bc3450/cmd_bc3450.c b/board/bc3450/cmd_bc3450.c
index f7f0013..48bc65d 100644
--- a/board/bc3450/cmd_bc3450.c
+++ b/board/bc3450/cmd_bc3450.c
@@ -189,7 +189,7 @@ int cmd_dip (cmd_tbl_t * cmdtp, int flag, int argc, char 
*argv[])
if (rc  0x0F)
return -1;
 
-   printf (0x%x\n, rc);
+   printf (0x%lx\n, rc);
return 0;
 }
 
diff --git a/board/cm5200/cmd_cm5200.c b/board/cm5200/cmd_cm5200.c
index 354f2bf..2201bdd 100644
--- a/board/cm5200/cmd_cm5200.c
+++ b/board/cm5200/cmd_cm5200.c
@@ -335,7 +335,7 @@ int do_rs232(char *argv[])
if (((*(vu_long *)MPC5XXX_WU_GPIO_DATA_I)  0x1000) !=
0x1000) {
error_status = 2;
-   printf(%s: failure at rs232_4, rxd status is %d 
+   printf(%s: failure at rs232_4, rxd status is %lu 
(should be 1)\n, __FUNCTION__,
((*(vu_long *)MPC5XXX_WU_GPIO_DATA_I) 
0x1000)  28);
@@ -350,7 +350,7 @@ int do_rs232(char *argv[])
if (((*(vu_long *)MPC5XXX_WU_GPIO_DATA_I)  0x1000) !=
0x) {
error_status = 2;
-   printf(%s: failure at rs232_4, rxd status is %d 
+   printf(%s: failure at rs232_4, rxd status is %lu 
(should be 0)\n, __FUNCTION__,
((*(vu_long *)MPC5XXX_WU_GPIO_DATA_I) 
0x1000)  28);
@@ -366,7 +366,7 @@ int do_rs232(char *argv[])
if (((*(vu_long *)MPC5XXX_WU_GPIO_DATA_I)  0x2000) !=
0x2000) {
error_status = 3;
-   printf(%s: failure at rs232_4, cts status is

Re: [U-Boot-Users] [PATCH] =?utf-8?q?Fix=20build= 20time=20warnings=20in=20function=20=E2=80 =98mmc=5Fdecode=5Fcsd=E2=80=99:

Sorry, don't know exactly why it did that utf-8 crap. Will repost ASAP.


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Fix build time warnings in function mmc_decode_csd.

---
 cpu/pxa/mmc.c |5 +++--
 1 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/cpu/pxa/mmc.c b/cpu/pxa/mmc.c
index 4495a80..97ac4a5 100644
--- a/cpu/pxa/mmc.c
+++ b/cpu/pxa/mmc.c
@@ -535,8 +535,9 @@ static void mmc_decode_csd(uint32_t * resp)
mmc_dev.removable = 0;
mmc_dev.block_read = mmc_bread;
 
-   printf(Detected: %u blocks of %u bytes (%uMB) , mmc_dev.lba,
-  mmc_dev.blksz, mmc_dev.lba * mmc_dev.blksz / (1024 * 1024));
+   printf(Detected: %lu blocks of %lu bytes (%luMB) ,
+  (unsigned long)mmc_dev.lba, mmc_dev.blksz,
+  mmc_dev.lba * mmc_dev.blksz / (1024 * 1024));
 }
 
 int
-- 
1.5.4.1


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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 V2] Round the serial port clock divisor value returned by calc_divisor()


Wolfgang Denk wrote:

In message [EMAIL PROTECTED] you wrote:

Round the serial port clock divisor value returned by
calc_divisor().

Signed-off-by: Hugo Villeneuve [EMAIL PROTECTED]
Signed-off-by: John Roberts [EMAIL PROTECTED]

---

Rounding is important, especially when using high baud rates
values like 115200bps. When using the non-rounded value, some
boards will work and some won't.

 drivers/serial/serial.c |6 +-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index 76425d8..1192f07 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial/serial.c
@@ -144,8 +144,12 @@ static int calc_divisor (NS16550_t port)
 #else
 #define MODE_X_DIV 16
 #endif
-   return (CFG_NS16550_CLK / MODE_X_DIV / gd-baudrate);
 
+	/* Compute divisor value. Normally, we should simply return:

+*   CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate
+* but we need to round that value by adding 0.5 or 8/16.
+* Rounding is especially important at high baud rates. */
+   return (((16 * CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate) + 8) / 16;
 }


Your patch causes problems with some boards:

Configuring for SBC8540 board...
serial.c: In function 'calc_divisor':
serial.c:153: warning: integer overflow in expression
serial.c:153: warning: integer overflow in expression

Configuring for sbc8548 board...
serial.c: In function 'calc_divisor':
serial.c:153: warning: integer overflow in expression
serial.c:153: warning: integer overflow in expression


Do you have a quick fix, or shall I back out the patch?

Best regards,

Wolfgang Denk


The formula I proposed previously in this thread compiles OK.  I cannot 
verify it is correct on real hardware, however.

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/43499/focus=43506

My apologies for providing the patch as an attachment.  I need to add 
the external editor plugin so I can use tbird for proper inline patches.


Best regards,
gvb

From 97b1ea8273eec4e45df71e074e0790d39fd6536d Mon Sep 17 00:00:00 2001
From: Gerald Van Baren [EMAIL PROTECTED]
Date: Thu, 10 Jul 2008 19:44:28 -0400
Subject: [PATCH] Round the serial port clock divisor value returned by calc_divisor()

This algorithm is better at avoiding integer overflow.

Signed-off-by: Gerald Van Baren [EMAIL PROTECTED]
---
 drivers/serial/serial.c |4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index 182ca2d..48579ae 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial/serial.c
@@ -150,7 +150,9 @@ static int calc_divisor (NS16550_t port)
 	 * but we need to round that value by adding 0.5 or 8/16.
 	 * Rounding is especially important at high baud rates.
 	 */
-	return (((16 * CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate) + 8) / 16;
+	return (CFG_NS16550_CLK + (gd-baudrate / 2)) /
+		(MODE_X_DIV * gd-baudrate);
+
 }
 
 #if !defined(CONFIG_SERIAL_MULTI)
-- 
1.5.6

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Enable passing of ATAGs required by latest Linux kernel.

---
 include/configs/pxa255_idp.h |4 
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/include/configs/pxa255_idp.h b/include/configs/pxa255_idp.h
index 4a9cadb..b7ea1a9 100644
--- a/include/configs/pxa255_idp.h
+++ b/include/configs/pxa255_idp.h
@@ -208,6 +208,10 @@
 /* protect off   */
 
 
+#define CONFIG_CMDLINE_TAG 1   /* enable passing of ATAGs */
+#define CONFIG_SETUP_MEMORY_TAGS   1
+/* #define CONFIG_INITRD_TAG   1 */
+
 #if defined(CONFIG_CMD_KGDB)
 #define CONFIG_KGDB_BAUDRATE   115200  /* speed to run kgdb serial 
port */
 #define CONFIG_KGDB_SER_INDEX  2   /* which serial port to use */
-- 
1.5.4.1


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] [PATCH] Round the serial port clock divisor value returned by calc_divisor()

This formula is better at avoiding integer overflow.

Signed-off-by: Gerald Van Baren [EMAIL PROTECTED]
---

(try 2)

The formula I proposed previously in this thread compiles OK.  I cannot
verify it is correct on real hardware, however.
http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/43499/focus=43506

Best regards,
gvb 

 drivers/serial/serial.c |4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/serial.c b/drivers/serial/serial.c
index 182ca2d..48579ae 100644
--- a/drivers/serial/serial.c
+++ b/drivers/serial/serial.c
@@ -150,7 +150,9 @@ static int calc_divisor (NS16550_t port)
 * but we need to round that value by adding 0.5 or 8/16.
 * Rounding is especially important at high baud rates.
 */
-   return (((16 * CFG_NS16550_CLK) / MODE_X_DIV / gd-baudrate) + 8) / 16;
+   return (CFG_NS16550_CLK + (gd-baudrate / 2)) /
+   (MODE_X_DIV * gd-baudrate);
+
 }
 
 #if !defined(CONFIG_SERIAL_MULTI)
-- 
1.5.6


-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
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] usb: add support for R8A66597 usb controller

2008-07-10 Thread Nobuhiro Iwamatsu

Hi, Wolfgang.

2008/7/11 Wolfgang Denk [EMAIL PROTECTED]:
 In message [EMAIL PROTECTED] you wrote:

  Can you not find some simpler name that r8a66597? That's a mess to
  type and to read.

 Thank you for your comment.

 I can change that struct r8a66597 for a simple name. For example,
 its name is struct r597. However, I do not want to change a source
 file name and CONFIG_ name, because R8A66597 is product name.

 Doesn't the board have an easy to spell and easy to remember nickname
 or so?

Unfortunately this tip does not have nicname and other.
We want to acquire a name same as Linux kernel USB driver..

Best regards,
 Nobuhiro

-
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

[U-Boot-Users] Undelivered Mail Returned to Sender