Re: [U-Boot] Hi

2010-09-03 Thread Tobias Waldekranz (Knutsson)
Did you download the version from hawkboard.org?

In that case, you should be able to follow the instructions on:

http://elinux.org/Hawkboard#Compiling_u-boot_.28bootloader.29

If you are using the git version of u-boot replace
omapl_hawkboard_config with da850evm_config and try to boot that
first. Then as a second step, you can add custom board support for the
Hawkboard.

On Thu, Sep 2, 2010 at 15:45, Vaishali Dhakate
vaishali.dhak...@sukrutsystems.com wrote:
 Hi
 Tobias thank you for the reply ...
 I am using Fedora so I downloaded this ARM Toolchain.

 http://fedoraproject.org/wiki/Architectures/ARM/CrossToolchain

 Installed this toolchain successfully and able to cross compile code for 
 ARM...

 I downloaded u-boot source code ,extracted it... Now want to compile
 this source code using this ARM Toolchain that I installed 

 http://www.linuxfordevices.com/c/a/Linux-For-Devices-Articles/Introduction-to-Das-UBoot-the-universal-open-source-bootloader/

 Now trying to figure out this using the above document... Not very
 clear... as to how to do this ... will read the README of u-boot ..
 Let me know !
 Thanks and Regards
 Vaishali

 On 9/1/10, Tobias Waldekranz (Knutsson) tobias.knuts...@gmail.com wrote:
 You can find the official port of u-boot for the Hawkboard over at
 hawkboard.org, but if you want to build the latest version of u-boot
 that can be done as well.

 The Hawkboard is very similar to the DA850 EVM so as a first step, you
 could simply build u-boot for that board and it should boot just fine.


 On Tue, Aug 31, 2010 at 13:19, Vaishali Dhakate
 vaishali.dhak...@sukrutsystems.com wrote:
 Hi ,
 I am trying to cross compile some libraries for ARM board
 I will port a Linux OS to the hawkboard.
 Want to study u-boot specific to this board so I ll be able to do
 this. The first chore is to port u-boot on this board.
 PLz help me , I was going through the uboot readme which directed me
 to straight away mail and share my concern.
 Thanks for the help
 Regards
 Vaishali
 ___
 U-Boot mailing list
 U-Boot@lists.denx.de
 http://lists.denx.de/mailman/listinfo/u-boot




 --

 Hälsningar/Regards
 Tobias Waldekranz





-- 

Hälsningar/Regards
Tobias Waldekranz
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: Add further watchdog reset calls

2010-09-03 Thread Stefan Roese
Hi Detlev,

On Thursday 02 September 2010 18:40:38 Detlev Zundel wrote:
   case TYPE:
  +   if (strm-outcb != Z_NULL)
  +   (*strm-outcb)(Z_NULL, 0); /* call WATCHDOG_RESET */
  
   if (flush == Z_BLOCK) goto inf_leave;
   
   case TYPEDO:
   if (state-last) {
 
 Can you please tell me (and include in the comment) why this calls a
 watchdog reset and why you did not use a regular plain WATCHDOG_RESET?

I did it this way, because that's the way these watchdog reset calls have been 
implemented in the U-Boot zlib version till now. Frankly I'm not sure why it 
was done this way instead of using regular plain WATCHDOG_RESET calls. 
Perhaps Wolfgang remembers the reasoning behind it.

I could change this function pointer approach to regular plain 
WATCHDOG_RESET call though if preferred. Just let me know and I'll try to 
come up with a new patch version.

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] APM82xxx: Add CPU and other peripheral support

2010-09-03 Thread Stefan Roese
Hi Marri,

On Friday 03 September 2010 03:48:44 Tirumala Marri wrote:
 mtdcr   ISRAM0_SB0CR,r1
   
   @@ -752,7 +754,11 @@ _start:
 mtdcr   ISRAM1_PMEG,r1
 
 lis r1,0x0004   /* BAS = 4_0004_ */
   
   +#if defined(CONFIG_APM82XXX) /* APM82XXX only has 32KB of OCM */
   + ori r1,r1,0x0784/* 32k */
   +#else
   
 ori r1,r1,0x0984/* 64k */
  
  Please get rid of these magic numbers and add a #define for a symbolic
  constant so the code becomes readable, and the selection can be done
  on a per-CPU base, i. e. without needing #ifdef's here.
 
 [Marri] I will define CONFIG_SYS_OCM_SIZE . But for now this is only added
 Bluestone.h. I need to remove  #if defined(CONFIG_APM821XX) I will have
 to
 Add CONFIG_SYS_OCM_SIZE to all the board config files which are based on
 460Ex and 460GT.
 I think that would be completely different patch.

Since the OCM size is SoC specific, I would prefer not to put such a define in 
each and every board config header, but in a SoC header. There you could 
automatically select the OCM size depending on the SoC type. No change in the 
board config header needed.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] APM82xxx: Add CPU and other peripheral support

2010-09-03 Thread Stefan Roese
Hi Wolfgang,

On Thursday 02 September 2010 21:01:37 Wolfgang Denk wrote:
 Hm.., the whole CPU ID code in arch/powerpc/cpu/ppc4xx/cpu.c has
 become a mess.
 
 Stefan:
 
 is there any deeper logic for the repeated #ifdef's there, when we
 finally do a switch on the PVR anyway?
 
 Instead of painstakingly concatenating  AMCC PowerPC 4 with 05
 and later GP Rev. B we could shrink this into a plain
 
   case PVR_405GP_RB:
   puts(AMCC PowerPC 405GP Rev. B);
   break;
 
 Or am I missing something?

IIRC, then the main reason for this is the code size reduction. Using your 
version would increase the code size quite a bit. But I concur, it would be 
much better readable. So just let me know, and I'll try to come up with a 
patch cleaning this up a bit.

 Let's get rid of all these nested #ifdef's...

As described in some comments, the problem is that some SoC's have the same 
PVR (for example 440EP rev C and 440GR rev B have same PVR). I have no idea 
how this identification could be solved without such ifdef's.

But you are right. Some of the ifdef's can be removed. Again, this would also 
result in a bigger image size though.

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Hi

2010-09-03 Thread Wolfgang Denk
Dear Tobias Waldekranz (Knutsson),

please STOP full quoting / top posting.

Read http://www.netmeister.org/news/learn2quote.html

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: w...@denx.de
Monday is an awful way to spend one seventh of your life.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: Add further watchdog reset calls

2010-09-03 Thread Wolfgang Denk
Dear Stefan Roese,

In message 201009030910.08643...@denx.de you wrote:
 
 I did it this way, because that's the way these watchdog reset calls have 
 been 
 implemented in the U-Boot zlib version till now. Frankly I'm not sure why it 
 was done this way instead of using regular plain WATCHDOG_RESET calls. 
 Perhaps Wolfgang remembers the reasoning behind it.

It allows to easily adjust the granularity of trigger points depending
on data block size.

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: w...@denx.de
You know that feeling when you're leaning back  on  a  stool  and  it
starts to tip over? Well, that's how I feel all the time.
- Steven Wright
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] APM82xxx: Add CPU and other peripheral support

2010-09-03 Thread Wolfgang Denk
Dear Stefan Roese,

In message 201009030946.22440...@denx.de you wrote:

 IIRC, then the main reason for this is the code size reduction. Using your 
 version would increase the code size quite a bit. But I concur, it would be 
 much better readable. So just let me know, and I'll try to come up with a 
 patch cleaning this up a bit.

Maybe some common parts (like AMCC PowerPC ) could be split off.

For the rest, the compelex code needs also space, so I'm not sure
about the end result.

 As described in some comments, the problem is that some SoC's have the same 
 PVR (for example 440EP rev C and 440GR rev B have same PVR). I have no idea 
 how this identification could be solved without such ifdef's.

I see.

 But you are right. Some of the ifdef's can be removed. Again, this would also 
 result in a bigger image size though.

Really? We should measure the effect.

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: w...@denx.de
The most exciting phrase to hear in science, the one that heralds new
discoveries, is not Eureka! (I found it!) but That's funny ...
  -- Isaac Asimov
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] atmel_usart: change register access to C structure

2010-09-03 Thread Andreas Bießmann
 This patch introduces C structure definition for register footprint of atmel's
 usart.

Signed-off-by: Andreas Bießmann biessm...@corscience.de
---
 drivers/serial/atmel_usart.c |   36 --
 drivers/serial/atmel_usart.h |   56 ++
 2 files changed, 46 insertions(+), 46 deletions(-)

diff --git a/drivers/serial/atmel_usart.c b/drivers/serial/atmel_usart.c
index cad3412..bfa1f3a 100644
--- a/drivers/serial/atmel_usart.c
+++ b/drivers/serial/atmel_usart.c
@@ -1,6 +1,9 @@
 /*
  * Copyright (C) 2004-2006 Atmel Corporation
  *
+ * Modified to support C structur SoC access by
+ * Andreas Bießmann biessm...@corscience.de
+ *
  * 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
@@ -16,10 +19,6 @@
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 #include common.h
-#ifndef CONFIG_AT91_LEGACY
-#define CONFIG_AT91_LEGACY
-#warning Please update to use C structur SoC access !
-#endif
 #include watchdog.h
 
 #include asm/io.h
@@ -46,6 +45,7 @@ DECLARE_GLOBAL_DATA_PTR;
 
 void serial_setbrg(void)
 {
+   atmel_usart3_t *usart = (atmel_usart3_t*)USART_BASE;
unsigned long divisor;
unsigned long usart_hz;
 
@@ -56,32 +56,37 @@ void serial_setbrg(void)
 */
usart_hz = get_usart_clk_rate(USART_ID);
divisor = (usart_hz / 16 + gd-baudrate / 2) / gd-baudrate;
-   usart3_writel(BRGR, USART3_BF(CD, divisor));
+   writel(USART3_BF(CD, divisor), usart-brgr);
 }
 
 int serial_init(void)
 {
-   usart3_writel(CR, USART3_BIT(RSTRX) | USART3_BIT(RSTTX));
+   atmel_usart3_t *usart = (atmel_usart3_t*)USART_BASE;
+
+   writel(USART3_BIT(RSTRX) | USART3_BIT(RSTTX), usart-cr);
 
serial_setbrg();
 
-   usart3_writel(CR, USART3_BIT(RXEN) | USART3_BIT(TXEN));
-   usart3_writel(MR, (USART3_BF(USART_MODE, USART3_USART_MODE_NORMAL)
+   writel(USART3_BIT(RXEN) | USART3_BIT(TXEN), usart-cr);
+   writel((USART3_BF(USART_MODE, USART3_USART_MODE_NORMAL)
   | USART3_BF(USCLKS, USART3_USCLKS_MCK)
   | USART3_BF(CHRL, USART3_CHRL_8)
   | USART3_BF(PAR, USART3_PAR_NONE)
-  | USART3_BF(NBSTOP, USART3_NBSTOP_1)));
+  | USART3_BF(NBSTOP, USART3_NBSTOP_1)),
+  usart-mr);
 
return 0;
 }
 
 void serial_putc(char c)
 {
+   atmel_usart3_t *usart = (atmel_usart3_t*)USART_BASE;
+
if (c == '\n')
serial_putc('\r');
 
-   while (!(usart3_readl(CSR)  USART3_BIT(TXRDY))) ;
-   usart3_writel(THR, c);
+   while (!(readl(usart-csr)  USART3_BIT(TXRDY)));
+   writel(c, usart-thr);
 }
 
 void serial_puts(const char *s)
@@ -92,12 +97,15 @@ void serial_puts(const char *s)
 
 int serial_getc(void)
 {
-   while (!(usart3_readl(CSR)  USART3_BIT(RXRDY)))
+   atmel_usart3_t *usart = (atmel_usart3_t*)USART_BASE;
+
+   while (!(readl(usart-csr)  USART3_BIT(RXRDY)))
 WATCHDOG_RESET();
-   return usart3_readl(RHR);
+   return readl(usart-rhr);
 }
 
 int serial_tstc(void)
 {
-   return (usart3_readl(CSR)  USART3_BIT(RXRDY)) != 0;
+   atmel_usart3_t *usart = (atmel_usart3_t*)USART_BASE;
+   return (readl(usart-csr)  USART3_BIT(RXRDY)) != 0;
 }
diff --git a/drivers/serial/atmel_usart.h b/drivers/serial/atmel_usart.h
index af3773a..7cfc2d5 100644
--- a/drivers/serial/atmel_usart.h
+++ b/drivers/serial/atmel_usart.h
@@ -3,6 +3,9 @@
  *
  * Copyright (C) 2005-2006 Atmel Corporation
  *
+ * Modified to support C structure SoC access by
+ * Andreas Bießmann biessm...@corscience.de
+ *
  * 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
@@ -20,32 +23,27 @@
 #ifndef __DRIVERS_ATMEL_USART_H__
 #define __DRIVERS_ATMEL_USART_H__
 
-/* USART3 register offsets */
-#define USART3_CR  0x
-#define USART3_MR  0x0004
-#define USART3_IER 0x0008
-#define USART3_IDR 0x000c
-#define USART3_IMR 0x0010
-#define USART3_CSR 0x0014
-#define USART3_RHR 0x0018
-#define USART3_THR 0x001c
-#define USART3_BRGR0x0020
-#define USART3_RTOR0x0024
-#define USART3_TTGR0x0028
-#define USART3_FIDI0x0040
-#define USART3_NER 0x0044
-#define USART3_XXR 0x0048
-#define USART3_IFR

Re: [U-Boot] [PATCH v3] AT91: MCI: add SD/MMC driver using mmc framework

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 This patch adds SD/MMC support to AT91SAM9260/9XE/9G20
 using the generic MMC framework.
 
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
 The driver itself is not dependent on any particular AT91 or AVR32
 device, but further work in device specific files is
 required to enable its use for other AT91 (and AVR32) devices:
 at91*_devices.c, hardware.h, at91*.h.
 Patches welcome!
 
  arch/arm/cpu/arm926ejs/at91/at91sam9260_devices.c |   21 ++
  arch/arm/include/asm/arch-at91/at91_common.h  |1 +
  arch/arm/include/asm/arch-at91/clk.h  |5 +
  arch/arm/include/asm/arch-at91/hardware.h |1 +
  doc/README.atmel_mci  |   86 +
  drivers/mmc/Makefile  |5 +-
  drivers/mmc/atmel_mci.h   |   48 +++-
  drivers/mmc/gen_atmel_mci.c   |  353 
 +
  include/mmc.h |   15 +-
  9 files changed, 530 insertions(+), 5 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4] AT91: add RTT and GPBR based RTC

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 From: Reinhard Meyer i...@emk-elektronik.de
 
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/include/asm/arch-at91/at91_gpbr.h |   45 +
  arch/arm/include/asm/arch-at91/at91_rtt.h  |   36 ++
  drivers/rtc/Makefile   |1 +
  drivers/rtc/at91sam9_rtt.c |  100 
 
  4 files changed, 182 insertions(+), 0 deletions(-)
  create mode 100644 arch/arm/include/asm/arch-at91/at91_gpbr.h
  create mode 100644 arch/arm/include/asm/arch-at91/at91_rtt.h
  create mode 100644 drivers/rtc/at91sam9_rtt.c
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v3] AT91: fix at91sam9260.h for AT91SAM9XE

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 Define the different location of the GPBRs for the 9XE
 Define the proper CPU Name
 
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/include/asm/arch-at91/at91sam9260.h |   18 ++
  1 files changed, 14 insertions(+), 4 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ppc4xx: Fix APC405 board support

2010-09-03 Thread Stefan Roese
On Wednesday 25 August 2010 17:02:28 Matthias Fuchs wrote:
 Opps, after a long time I tested recent u-boot on our
 APC405 board. This simple fix makes networking work again.

Applied to u-boot-ppc4xx/master. Thanks.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91: reset.c: fix comments, add option

2010-09-03 Thread Reinhard Meyer
Detlev Zundel schrieb:
 Hi Reinhard,
 
 - The comment was wrong/misleading
 - One would assume that a u-boot reset command should also
 reset the external hardware outside the SoC. Since its unknown
 whether some boards rely on that NOT being so, asserting the
 external reset signal is optional

 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/cpu/arm926ejs/at91/reset.c |   17 -
  1 files changed, 8 insertions(+), 9 deletions(-)
Applied (with requested changes) to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ppc4xx: Fix 440EPx bug in reconfigure_pll()

2010-09-03 Thread Stefan Roese
On Thursday 26 August 2010 17:14:51 Stefan Roese wrote:
 This patch fixes a bug in reconfigure_pll(), where the detection of
 the current bootstrap option is wrong. The ICS bits where incorrectly
 shifted. This bug was found on the lwmon5 board, which uses bootstrap
 option H (I2C bootstrap EEPROM).
 
 Additionally a bit of code was moved into the if statement, since its
 only used after later on. No need to run this code all the time.
 
 Also, a few empty lines are added to make the code better readable.

Applied to u-boot-ppc4xx/master. Thanks.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] ppc4xx: Invalidate d-cache when used as init-ram

2010-09-03 Thread Stefan Roese
On Tuesday 31 August 2010 11:27:14 Stefan Roese wrote:
 We need to invalidate the data cache after it has been used as init-ram.
 
 This problem was detected on the lwmon5 update.

Applied to u-boot-ppc4xx/master. Thanks.
 
Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91/AVR32: atmel_spi.c: flush RDR before next SPI transaction

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 From: Reinhard Meyer i...@emk-elektronik.de
 
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  drivers/spi/atmel_spi.c |   13 +++--
  1 files changed, 11 insertions(+), 2 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91: add option to enable pullups in at91sam9260_devices.c

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/cpu/arm926ejs/at91/at91sam9260_devices.c |   35 
 +++--
  1 files changed, 25 insertions(+), 10 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91: add 2nd SPI to 9260/9XE/9G20

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/include/asm/arch-at91/hardware.h   |1 +
  arch/arm/include/asm/arch-at91/memory-map.h |1 +
  2 files changed, 2 insertions(+), 0 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Please pull u-boot-ppc4xx/master

2010-09-03 Thread Stefan Roese
The following changes since commit bd2313078114c4b44c4a5ce149af43bcb7fc8854:

  Merge branch 'master' of ssh://gemini/home/wd/git/u-boot/master (2010-08-18 
21:16:35 
+0200)

are available in the git repository at:

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

Matthias Fuchs (1):
  ppc4xx: Fix APC405 board support

Stefan Roese (2):
  ppc4xx: Fix 440EPx bug in reconfigure_pll()
  ppc4xx: Invalidate d-cache when used as init-ram

 arch/powerpc/cpu/ppc4xx/cpu_init.c |   27 ++-
 arch/powerpc/cpu/ppc4xx/start.S|5 +
 include/configs/APC405.h   |1 +
 include/ppc440.h   |1 +
 4 files changed, 25 insertions(+), 9 deletions(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91: Add arch_preboot_os which disables PIT in a faster way

2010-09-03 Thread Reinhard Meyer
Dear Alexander Stein,
 ...
 
 Signed-off-by: Alexander Stein alexander.st...@systec-electronic.com
 ...
 + /*  Disble PITC
 +  * Add 0x1000 to current counter to stop it faster
 +  * without waiting for wrapping back to 0
 +  */
 + writel(cpiv + 0x1000, pit-mr);
 ...

Please fix the multi-line comment.

Thanks, Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91_emac.h: fix typo in register definition

2010-09-03 Thread Reinhard Meyer
Dear Andreas Bießmann,
 Signed-off-by: Andreas Bießmann andreas.de...@googlemail.com
 ---
  arch/arm/include/asm/arch-at91/at91_emac.h |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
 
 diff --git a/arch/arm/include/asm/arch-at91/at91_emac.h 
 b/arch/arm/include/asm/arch-at91/at91_emac.h
 index 45ae333..0e2ff78 100644
 --- a/arch/arm/include/asm/arch-at91/at91_emac.h
 +++ b/arch/arm/include/asm/arch-at91/at91_emac.h
 @@ -61,7 +61,7 @@ typedef struct at91_emac {
   u32  reserved2[3];
   u32  hsh;
   u32  hsl;
 - u32  sh1l;
 + u32  sa1l;
   u32  sa1h;
   u32  sa2l;
   u32  sa2h;

Please resubmit as pure text, not base64 encoded.

It arrived base64 encoded here...

Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91_pit: Fix AT91_PIT_MR_PIV_MASK macro

2010-09-03 Thread Reinhard Meyer
Alexander Stein schrieb:
 Signed-off-by: Alexander Stein alexander.st...@systec-electronic.com
 ---
  arch/arm/include/asm/arch-at91/at91_pit.h |2 +-
  1 files changed, 1 insertions(+), 1 deletions(-)
Applied to u-boot-atmel/next
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 3/3] avr32: Add simple paging support

2010-09-03 Thread Andreas Bießmann
Dear Haavard Skinnemnoen,

Am 12.08.2010 08:52, schrieb Haavard Skinnemoen:
 Use the MMU hardware to set up 1:1 mappings between physical and virtual
 addresses. This allows us to bypass the cache when accessing the flash
 without having to do any physical-to-virtual address mapping in the CFI
 driver.
 
 The virtual memory mappings are defined at compile time through a sorted
 array of virtual memory range objects. When a TLB miss exception
 happens, the exception handler does a binary search through the array
 until it finds a matching entry and loads it into the TLB. The u-boot
 image itself is covered by a fixed TLB entry which is never replaced.
 
 This makes the 'saveenv' command work again on ATNGW100 and other boards
 using the CFI driver, hopefully without breaking any rules.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com

Tested-by: Andreas Bießmann biessm...@corscience.de

there were no changes to this patch since last submission. Applying this
should be done soon.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/3] avr32: Use uncached() macro to get an address for SDRAM init

2010-09-03 Thread Andreas Bießmann
Dear Haavard Skinnemoen,

Am 12.08.2010 08:52, schrieb Haavard Skinnemoen:
 The paging system which is required to set up caching properties has not
 yet been initialized when the SDRAM is initialized. So when the
 map_physmem() function is converted to return the physical address
 unchanged, the SDRAM initialization will break on some boards.
 
 The avr32-specific uncached() macro will return an address which will
 always cause uncached accessed to be made. Since this happens in the
 board code, using avr32-specific features should be ok, and will allow
 the SDRAM initialization to keep working.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com

Tested-by: Andreas Bießmann biessm...@corscience.de

one colleague reported similar problems initialising SDRAM on our own
board. I could not reproduce this issue here, however this patch fixed
the problem. Thanks for that, it came the right time.

This patch should be applied too.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/2] APM82xxx: Add CPU and other peripheral support

2010-09-03 Thread Stefan Roese
Hi Wolfgang,

On Friday 03 September 2010 10:32:53 Wolfgang Denk wrote:
 Dear Stefan Roese,
 
 In message 201009030946.22440...@denx.de you wrote:
  IIRC, then the main reason for this is the code size reduction. Using
  your version would increase the code size quite a bit. But I concur, it
  would be much better readable. So just let me know, and I'll try to come
  up with a patch cleaning this up a bit.
 
 Maybe some common parts (like AMCC PowerPC ) could be split off.

Yes, this would help a bit. I'll add this on list.
 
 For the rest, the compelex code needs also space, so I'm not sure
 about the end result.
 
  As described in some comments, the problem is that some SoC's have the
  same PVR (for example 440EP rev C and 440GR rev B have same PVR). I have
  no idea how this identification could be solved without such ifdef's.
 
 I see.
 
  But you are right. Some of the ifdef's can be removed. Again, this would
  also result in a bigger image size though.
 
 Really? We should measure the effect.

I'll work on this a bit in the near future and will post my findings.

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91_emac.h: fix typo in register definition

2010-09-03 Thread Andreas Bießmann
Dear Reinhard Meyer,

Am 03.09.2010 12:25, schrieb Reinhard Meyer:
 Dear Andreas Bießmann,

 Please resubmit as pure text, not base64 encoded.

my MUA tells me:

---8---
Message-Id: 1283082856-82859-1-git-send-email-andreas.de...@googlemail.com
...
Content-Transfer-Encoding: 8bit
---8---

 It arrived base64 encoded here...

however I can re-submit it this evening.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] zlib: Add further watchdog reset calls

2010-09-03 Thread Stefan Roese
Hi Wolfgang,

On Friday 03 September 2010 10:30:24 Wolfgang Denk wrote:
  I did it this way, because that's the way these watchdog reset calls have
  been implemented in the U-Boot zlib version till now. Frankly I'm not
  sure why it was done this way instead of using regular plain
  WATCHDOG_RESET calls. Perhaps Wolfgang remembers the reasoning behind
  it.
 
 It allows to easily adjust the granularity of trigger points depending
 on data block size.

Hmmm, I fail to see how the current implementation would differ from the one 
Detlev suggested:

outcb is initialised with either WATCHDOG_RESET or NULL in gunzip.c. Later 
on in zlib.c, the function referenced by outcb is called if not NULL. So those 
statements:

if (strm-outcb != Z_NULL)
(*strm-outcb)(Z_NULL, 0);

could be replaced by:

WATCHDOG_RESET;

Perhaps I'm missing something?

Cheers,
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: off...@denx.de
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91_emac.h: fix typo in register definition

2010-09-03 Thread Reinhard Meyer
Andreas Bießmann schrieb:
 Dear Reinhard Meyer,
 
 Am 03.09.2010 12:25, schrieb Reinhard Meyer:
 Dear Andreas Bießmann,
 
 Please resubmit as pure text, not base64 encoded.
 
 my MUA tells me:
 
 ---8---
 Message-Id: 1283082856-82859-1-git-send-email-andreas.de...@googlemail.com
 ...
 Content-Transfer-Encoding: 8bit
 ---8---

I saw that, but.. see below...

Your other patches are in plain text, only this one sticks out:)

Thats what I see here:

From - Tue Aug 31 03:47:17 2010
^^^this is already odd
X-Mozilla-Status: 0003
X-Mozilla-Status2: 
Return-Path: u-boot-boun...@lists.denx.de
Delivery-Date: Sun, 29 Aug 2010 13:54:56 +0200
Received: from theia.denx.de (theia.denx.de [85.214.87.163])
...
Received: from andreas-mbp.erlangen.biessmann.tld ([188.110.235.134])
by mx.google.com with ESMTPS id a48sm9995891eei.12.2010.08.29.04.54.33
(version=SSLv3 cipher=RC4-MD5); Sun, 29 Aug 2010 04:54:36 -0700 (PDT)
From: =?UTF-8?q?Andreas=20Bie=C3=9Fmann?= andreas.de...@googlemail.com
To: js_at...@scharsoft.de
Date: Sun, 29 Aug 2010 13:54:16 +0200
Message-Id: 1283082856-82859-1-git-send-email-andreas.de...@googlemail.com
X-Mailer: git-send-email 1.7.2.2
MIME-Version: 1.0
Cc: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] at91_emac.h: fix typo in register definition
X-BeenThere: u-boot@lists.denx.de
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: U-Boot discussion u-boot.lists.denx.de
List-Unsubscribe: http://lists.denx.de/mailman/listinfo/u-boot,
mailto:u-boot-requ...@lists.denx.de?subject=unsubscribe
List-Archive: http://lists.denx.de/pipermail/u-boot
List-Post: mailto:u-boot@lists.denx.de
List-Help: mailto:u-boot-requ...@lists.denx.de?subject=help
List-Subscribe: http://lists.denx.de/mailman/listinfo/u-boot,
mailto:u-boot-requ...@lists.denx.de?subject=subscribe
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: base64
^^ SOMEWHERE this got added..??!!??
Sender: u-boot-boun...@lists.denx.de
Errors-To: u-boot-boun...@lists.denx.de
X-UI-Junk: AutoNotJunk -999 (UWL);
  V01:kVTWu2o7:n/dRKR8q2mLw2neqNYB2VOEAdY83mL/SkCvDJaRkwqv7AuJgS9D
  iO6n89pJE08XJIzZqsh8dZK2kR6Wh8Jax0XeN5TcTMPzHZTn3ZIVPDRBGu39Mxkx
  P4ycvYG1SNPQi7ipoUxr+k8c1oHU3c43T71YwisN4qJddFx1xspcjGea82oy2oth
  ip3NFogITh6Q8qOnogrv7trspflWXycC0CLad3ubw8O6wkwuyK6Je6EbyYlYIdwx
  M2jg4v+NBrzG8qiUXS32idBDWs4eBggIWlRSFytJEv+/5gomLi+JIEyB8s8Y+jAs
  NwoIBbsdXhE9C
X-Nemesis-Spam: whitelist
Envelope-To: u-b...@emk-elektronik.de

U2lnbmVkLW9mZi1ieTogQW5kcmVhcyBCaWXDn21hbm4gPGFuZHJlYXMuZGV2ZWxAZ29vZ2xlbWFp
bC5jb20+Ci0tLQogYXJjaC9hcm0vaW5jbHVkZS9hc20vYXJjaC1hdDkxL2F0OTFfZW1hYy5oIHwg
ICAgMiArLQogMSBmaWxlcyBjaGFuZ2VkLCAxIGluc2VydGlvbnMoKyksIDEgZGVsZXRpb25zKC0p
CgpkaWZmIC0tZ2l0IGEvYXJjaC9hcm0vaW5jbHVkZS9hc20vYXJjaC1hdDkxL2F0OTFfZW1hYy5o
IGIvYXJjaC9hcm0vaW5jbHVkZS9hc20vYXJjaC1hdDkxL2F0OTFfZW1hYy5oCmluZGV4IDQ1YWUz
MzMuLjBlMmZmNzggMTAwNjQ0Ci0tLSBhL2FyY2gvYXJtL2luY2x1ZGUvYXNtL2FyY2gtYXQ5MS9h
dDkxX2VtYWMuaAorKysgYi9hcmNoL2FybS9pbmNsdWRlL2FzbS9hcmNoLWF0OTEvYXQ5MV9lbWFj
LmgKQEAgLTYxLDcgKzYxLDcgQEAgdHlwZWRlZiBzdHJ1Y3QgYXQ5MV9lbWFjIHsKIAl1MzIJIHJl
c2VydmVkMlszXTsKIAl1MzIJIGhzaDsKIAl1MzIJIGhzbDsKLQl1MzIJIHNoMWw7CisJdTMyCSBz
YTFsOwogCXUzMgkgc2ExaDsKIAl1MzIJIHNhMmw7CiAJdTMyCSBzYTJoOwotLSAKMS43LjIuMgoK
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KVS1Cb290IG1h
aWxpbmcgbGlzdApVLUJvb3RAbGlzdHMuZGVueC5kZQpodHRwOi8vbGlzdHMuZGVueC5kZS9tYWls
bWFuL2xpc3RpbmZvL3UtYm9vdAo=



___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/3] avr32: Print unrelocated PC on exception

2010-09-03 Thread Reinhard Meyer
Hello Andreas,

did/can you test that one as well?
Then I can take all 3 of the series in.

Thanks,
Reinhard

 In addition to the real PC value, also print the value of PC after
 subtracting the relocation offset. This value will match the address in
 the ELF file so it's much easier to figure out where things went wrong.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
 ---
  arch/avr32/cpu/exception.c |3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)
 
 diff --git a/arch/avr32/cpu/exception.c b/arch/avr32/cpu/exception.c
 index dc9c300..b21ef1f 100644
 --- a/arch/avr32/cpu/exception.c
 +++ b/arch/avr32/cpu/exception.c
 @@ -59,7 +59,8 @@ void do_unknown_exception(unsigned int ecr, struct pt_regs 
 *regs)
  {
   unsigned int mode;
  
 - printf(\n *** Unhandled exception %u at PC=0x%08lx\n, ecr, regs-pc);
 + printf(\n *** Unhandled exception %u at PC=0x%08lx [%08lx]\n,
 + ecr, regs-pc, regs-pc - gd-reloc_off);
  
   switch (ecr) {
   case ECR_BUS_ERROR_WRITE:


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/3] avr32: Print unrelocated PC on exception

2010-09-03 Thread Andreas Bießmann
Dear Reinhard Meyer,

Am 03.09.2010 13:51, schrieb Reinhard Meyer:
 Hello Andreas,
 
 did/can you test that one as well?
 Then I can take all 3 of the series in.

Well I have applied this patch but have never tested this till now. Yes
it works and shows correct information.

 In addition to the real PC value, also print the value of PC after
 subtracting the relocation offset. This value will match the address in
 the ELF file so it's much easier to figure out where things went wrong.

 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com

Tested-by: Andreas Bießmann biessm...@corscience.de

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] at91_emac.h: fix typo in register definition

2010-09-03 Thread Andreas Bießmann
Dear Reinhard Meyer,

Am 03.09.2010 13:42, schrieb Reinhard Meyer:
 Andreas Bießmann schrieb:
 Dear Reinhard Meyer,
 Am 03.09.2010 12:25, schrieb Reinhard Meyer:
 Dear Andreas Bießmann,

 Please resubmit as pure text, not base64 encoded.

 my MUA tells me:

 ---8---
 Message-Id: 1283082856-82859-1-git-send-email-andreas.de...@googlemail.com
 ...
 Content-Transfer-Encoding: 8bit
 ---8---
 
 I saw that, but.. see below...
 
 Your other patches are in plain text, only this one sticks out:)
 
 Thats what I see here:

[snip]

 Content-Transfer-Encoding: base64
 ^^ SOMEWHERE this got added..??!!??

[snap]

ouch ... this was my first patch sent from my macbook at home (through
gmail). Will investigate that this evening.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] AT91: add 2nd SPI to 9260/9XE/9G20

2010-09-03 Thread Reinhard Meyer
Reinhard Meyer schrieb:
 Reinhard Meyer schrieb:
 Signed-off-by: Reinhard Meyer u-b...@emk-elektronik.de
 ---
  arch/arm/include/asm/arch-at91/hardware.h   |1 +
  arch/arm/include/asm/arch-at91/memory-map.h |1 +
  2 files changed, 2 insertions(+), 0 deletions(-)
 Applied to u-boot-atmel/next

De-applied, it breaks one other board. Needs rework in the
#define cascade for SPI1_BASE through memory-map.h,
hardware.h, SoC.h :(
Reinhard


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] P1011RDB P1020RDB: NAND bootstrap too big

2010-09-03 Thread Kumar Gala
 
 cd /home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb  
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld -Bstatic -T 
 /home/bryan/Source/u-boot/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds -Ttext 
 0xfff0  -n start.o resetvec.o cache.o cpu_init_early.o cpu_init_nand.o
 fsl_law.o law.o nand_boot.o nand_boot_fsl_elbc.o ns16550.o tlb.o tlb_table.o 
 -L 
 /opt/WG/ppc-linux26-glibc211/toolchain/bin/../lib/gcc/powerpc-timesys-linux-gnuspe/4.4.4
  -lgcc \
 
-Map /home/bryan/Source/u-boot/nand_spl/u-boot-spl.map \
 
-o /home/bryan/Source/u-boot/nand_spl/u-boot-spl
 
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: NAND bootstrap too big
 
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: NAND bootstrap too big
 
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: section .resetvec loaded at 
 [fff00ffc,fff00fff] overlaps section .data loaded at [fff00e50,fff01017]
 
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: 
 /home/bryan/Source/u-boot/nand_spl/u-boot-spl: section .resetvec vma 
 0xfff00ffc overlaps previous sections
 
 make[1]: *** [/home/bryan/Source/u-boot/nand_spl/u-boot-spl] Error 1
 
 make[1]: Leaving directory 
 `/home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb'
 
 make: *** [nand_spl] Error 2
 
 
 
 I'm guessing that this might be similar to 
 a4bfc4cc466473b97c7fe84bdf261b2935887e3f
 
 But for mpc85xx...
 

I need to install the ELDK toolchains and look into this.  The newer gcc 4.4 
toolchains we use aren't showing this.  Which means we've probably added 
something that has caused a code size increase thus breaking the SPL build.

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Pull request: u-boot-atmel/at91

2010-09-03 Thread Reinhard Meyer
The following changes since commit bd2313078114c4b44c4a5ce149af43bcb7fc8854:
  Wolfgang Denk (1):
Merge branch 'master' of ssh://gemini/home/wd/git/u-boot/master

are available in the git repository at:

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

Alexander Stein (1):
  at91_pit: Fix AT91_PIT_MR_PIV_MASK macro

Daniel Gorsulowski (3):
  at91: Defined main clock frequency on esd at91 boards
  at91: Update meesc board to new SoC access
  at91: Enabeling USB host on meesc board

Eric Bénard (2):
  cpuat91: convert to new at91 soc architecture
  cpuat91: update defaut environement

Jens Scharsig (1):
  AT91 Fix: return value of get_tbclk

Reinhard Meyer (7):
  AT91: MCI: add SD/MMC driver using mmc framework
  AT91: add RTT and GPBR based RTC
  AT91SAM9XE: add embedded flash support
  AT91: fix at91sam9260.h for AT91SAM9XE
  AT91: reset.c: fix comments, add option
  AT91/AVR32: atmel_spi.c: flush RDR before next SPI transaction
  AT91: add option to enable pullups in at91sam9260_devices.c

 Makefile  |2 +-
 arch/arm/cpu/arm926ejs/at91/Makefile  |1 +
 arch/arm/cpu/arm926ejs/at91/at91sam9260_devices.c |   56 +++-
 arch/arm/cpu/arm926ejs/at91/eflash.c  |  271 
 arch/arm/cpu/arm926ejs/at91/reset.c   |   21 +-
 arch/arm/cpu/arm926ejs/at91/timer.c   |5 +-
 arch/arm/include/asm/arch-at91/at91_common.h  |1 +
 arch/arm/include/asm/arch-at91/at91_dbu.h |   41 +++
 arch/arm/include/asm/arch-at91/at91_eefc.h|   51 +++
 arch/arm/include/asm/arch-at91/at91_gpbr.h|   45 +++
 arch/arm/include/asm/arch-at91/at91_pit.h |2 +-
 arch/arm/include/asm/arch-at91/at91_rtt.h |   36 ++
 arch/arm/include/asm/arch-at91/at91sam9260.h  |   18 +-
 arch/arm/include/asm/arch-at91/clk.h  |5 +
 arch/arm/include/asm/arch-at91/hardware.h |1 +
 board/esd/meesc/meesc.c   |  116 
 board/eukrea/cpuat91/cpuat91.c|   53 ++--
 doc/README.atmel_mci  |   86 +
 drivers/mmc/Makefile  |5 +-
 drivers/mmc/atmel_mci.h   |   48 +++-
 drivers/mmc/gen_atmel_mci.c   |  353 +
 drivers/rtc/Makefile  |1 +
 drivers/rtc/at91sam9_rtt.c|  100 ++
 drivers/spi/atmel_spi.c   |   13 +-
 include/configs/cpuat91.h |   53 ++-
 include/configs/meesc.h   |   26 +-
 include/configs/otc570.h  |1 +
 include/mmc.h |   15 +-
 28 files changed, 1278 insertions(+), 148 deletions(-)
 create mode 100644 arch/arm/cpu/arm926ejs/at91/eflash.c
 create mode 100644 arch/arm/include/asm/arch-at91/at91_dbu.h
 create mode 100644 arch/arm/include/asm/arch-at91/at91_eefc.h
 create mode 100644 arch/arm/include/asm/arch-at91/at91_gpbr.h
 create mode 100644 arch/arm/include/asm/arch-at91/at91_rtt.h
 create mode 100644 doc/README.atmel_mci
 create mode 100644 drivers/mmc/gen_atmel_mci.c
 create mode 100644 drivers/rtc/at91sam9_rtt.c

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 1/3] avr32: Print unrelocated PC on exception

2010-09-03 Thread Reinhard Meyer
Haavard Skinnemoen schrieb:
 In addition to the real PC value, also print the value of PC after
 subtracting the relocation offset. This value will match the address in
 the ELF file so it's much easier to figure out where things went wrong.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
 ---
  arch/avr32/cpu/exception.c |3 ++-
  1 files changed, 2 insertions(+), 1 deletions(-)
Applied to u-boot-atmel/avr32
Thanks,
Reinhard
attachment: reinhard_meyer.vcf___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 2/3] avr32: Use uncached() macro to get an address for SDRAM init

2010-09-03 Thread Reinhard Meyer
Haavard Skinnemoen schrieb:
 The paging system which is required to set up caching properties has not
 yet been initialized when the SDRAM is initialized. So when the
 map_physmem() function is converted to return the physical address
 unchanged, the SDRAM initialization will break on some boards.
 
 The avr32-specific uncached() macro will return an address which will
 always cause uncached accessed to be made. Since this happens in the
 board code, using avr32-specific features should be ok, and will allow
 the SDRAM initialization to keep working.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
 ---
  board/atmel/atngw100/atngw100.c  |4 +---
  board/atmel/atstk1000/atstk1000.c|4 +---
  board/earthlcd/favr-32-ezkit/favr-32-ezkit.c |4 +---
  board/mimc/mimc200/mimc200.c |4 +---
  board/miromico/hammerhead/hammerhead.c   |4 +---
  5 files changed, 5 insertions(+), 15 deletions(-)
Applied to u-boot-atmel/avr32
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 3/3] avr32: Add simple paging support

2010-09-03 Thread Reinhard Meyer
Haavard Skinnemoen schrieb:
 Use the MMU hardware to set up 1:1 mappings between physical and virtual
 addresses. This allows us to bypass the cache when accessing the flash
 without having to do any physical-to-virtual address mapping in the CFI
 driver.
 
 The virtual memory mappings are defined at compile time through a sorted
 array of virtual memory range objects. When a TLB miss exception
 happens, the exception handler does a binary search through the array
 until it finds a matching entry and loads it into the TLB. The u-boot
 image itself is covered by a fixed TLB entry which is never replaced.
 
 This makes the 'saveenv' command work again on ATNGW100 and other boards
 using the CFI driver, hopefully without breaking any rules.
 
 Signed-off-by: Haavard Skinnemoen haavard.skinnem...@atmel.com
 ---
  arch/avr32/cpu/at32ap700x/Makefile |2 +-
  arch/avr32/cpu/at32ap700x/mmu.c|   78 
 
  arch/avr32/cpu/start.S |   19 +++--
  arch/avr32/include/asm/arch-at32ap700x/addrspace.h |5 +-
  arch/avr32/include/asm/arch-at32ap700x/mmu.h   |   66 +
  arch/avr32/lib/board.c |4 +
  board/atmel/atngw100/atngw100.c|   15 
  board/atmel/atstk1000/atstk1000.c  |   15 
  board/earthlcd/favr-32-ezkit/favr-32-ezkit.c   |   15 
  board/mimc/mimc200/mimc200.c   |   20 +
  board/miromico/hammerhead/hammerhead.c |   15 
  include/configs/atngw100.h |3 +
  include/configs/atstk1002.h|3 +
  include/configs/atstk1003.h|3 +
  include/configs/atstk1004.h|3 +
  include/configs/atstk1006.h|3 +
  include/configs/favr-32-ezkit.h|3 +
  include/configs/hammerhead.h   |3 +
  include/configs/mimc200.h  |3 +
  19 files changed, 267 insertions(+), 11 deletions(-)
  create mode 100644 arch/avr32/cpu/at32ap700x/mmu.c
  create mode 100644 arch/avr32/include/asm/arch-at32ap700x/mmu.h
Applied to u-boot-atmel/avr32
Thanks,
Reinhard

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Pull request: u-boot-atmel/avr32

2010-09-03 Thread Reinhard Meyer
The following changes since commit bd2313078114c4b44c4a5ce149af43bcb7fc8854:
  Wolfgang Denk (1):
Merge branch 'master' of ssh://gemini/home/wd/git/u-boot/master

are available in the git repository at:

  git://git.denx.de/u-boot-atmel avr32

Haavard Skinnemoen (3):
  avr32: Print unrelocated PC on exception
  avr32: Use uncached() macro to get an address for SDRAM init
  avr32: Add simple paging support

 arch/avr32/cpu/at32ap700x/Makefile |2 +-
 arch/avr32/cpu/at32ap700x/mmu.c|   78 
 arch/avr32/cpu/exception.c |3 +-
 arch/avr32/cpu/start.S |   19 +++--
 arch/avr32/include/asm/arch-at32ap700x/addrspace.h |5 +-
 arch/avr32/include/asm/arch-at32ap700x/mmu.h   |   66 +
 arch/avr32/lib/board.c |4 +
 board/atmel/atngw100/atngw100.c|   19 -
 board/atmel/atstk1000/atstk1000.c  |   19 -
 board/earthlcd/favr-32-ezkit/favr-32-ezkit.c   |   19 -
 board/mimc/mimc200/mimc200.c   |   24 +-
 board/miromico/hammerhead/hammerhead.c |   19 -
 include/configs/atngw100.h |3 +
 include/configs/atstk1002.h|3 +
 include/configs/atstk1003.h|3 +
 include/configs/atstk1004.h|3 +
 include/configs/atstk1006.h|3 +
 include/configs/favr-32-ezkit.h|3 +
 include/configs/hammerhead.h   |3 +
 include/configs/mimc200.h  |3 +
 20 files changed, 274 insertions(+), 27 deletions(-)
 create mode 100644 arch/avr32/cpu/at32ap700x/mmu.c
 create mode 100644 arch/avr32/include/asm/arch-at32ap700x/mmu.h

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] P1011RDB P1020RDB: NAND bootstrap too big

2010-09-03 Thread Kumar Gala

On Sep 2, 2010, at 9:25 PM, Bryan Hundven wrote:

 This was a git clone from today.
 Built with a pretty standard e500v2 gnuspe timesys toolchain.
 
 I was able to build and run P1020RDB_config on the freescale demo board.
 
 ---
 From: Bryan Hundven
 Sent: Thursday, September 02, 2010 2:42 PM
 To: u-boot@lists.denx.de
 Subject: P1011RDB  P1020RDB: NAND bootstrap too big
 
 When I build either P1011RDB_NAND_config or P1020RDB_NAND_config, I get:
 
 make[1]: Entering directory 
 `/home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb'

What git commit are you on?  What toolchain are you using?

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] P1011RDB P1020RDB: NAND bootstrap too big

2010-09-03 Thread Bryan Hundven
Ah. Ok.
Thanks Kumar,
I will get with timesys.

From: Kumar Gala [ga...@kernel.crashing.org]
Sent: Friday, September 03, 2010 7:25 AM
To: Bryan Hundven
Cc: u-boot@lists.denx.de
Subject: Re: [U-Boot] P1011RDB  P1020RDB: NAND bootstrap too big


 cd /home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb  
 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld -Bstatic -T 
 /home/bryan/Source/u-boot/arch/powerpc/cpu/mpc85xx/u-boot-nand_spl.lds -Ttext 
 0xfff0  -n start.o resetvec.o cache.o cpu_init_early.o cpu_init_nand.o
 fsl_law.o law.o nand_boot.o nand_boot_fsl_elbc.o ns16550.o tlb.o tlb_table.o 
 -L 
 /opt/WG/ppc-linux26-glibc211/toolchain/bin/../lib/gcc/powerpc-timesys-linux-gnuspe/4.4.4
  -lgcc \

-Map /home/bryan/Source/u-boot/nand_spl/u-boot-spl.map \

-o /home/bryan/Source/u-boot/nand_spl/u-boot-spl

 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: NAND bootstrap too big

 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: NAND bootstrap too big

 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: section .resetvec loaded at 
 [fff00ffc,fff00fff] overlaps section .data loaded at [fff00e50,fff01017]

 /opt/WG/ppc-linux26-glibc211/bin/ppc-ld: 
 /home/bryan/Source/u-boot/nand_spl/u-boot-spl: section .resetvec vma 
 0xfff00ffc overlaps previous sections

 make[1]: *** [/home/bryan/Source/u-boot/nand_spl/u-boot-spl] Error 1

 make[1]: Leaving directory 
 `/home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb'

 make: *** [nand_spl] Error 2



 I'm guessing that this might be similar to 
 a4bfc4cc466473b97c7fe84bdf261b2935887e3f

 But for mpc85xx...


I need to install the ELDK toolchains and look into this.  The newer gcc 4.4 
toolchains we use aren't showing this.  Which means we've probably added 
something that has caused a code size increase thus breaking the SPL build.

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Severals questions How to submit new u-boot features

2010-09-03 Thread Christophe Henri RICARD
Hi,

I am actually working to add a new driver to u-boot related to Trusted Platform 
Module (TPM) for embedded system. (On a u-boot version 2009.11)
I would be interested to know:

-   what is the process to do the submission ?

-  Because the available i2c interfaces looks to be only for i2c memory 
(such as eeprom memory), I had to rework some i2c interfaces to be able to send 
directly my TPM command/data through the i2c bus. This mean write or read some 
data directly on the i2c bus without sending any offset byte.

Will it be acceptable for the community to add those two new interfaces 
(i2c_write_direct/i2c_read_direct) for each i2c driver ?

-  Is there any coding style specification ?


Thanks for your help
Best Regards
Christophe RICARD
STMicroelectronics.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] P1011RDB P1020RDB: NAND bootstrap too big

2010-09-03 Thread Bryan Hundven
git commit: bd2313078114c4b44c4a5ce149af43bcb7fc8854

toolchain: is provided by the timesys build factory. I'm not sure what 
information I can give about it, so I'll give you some version numbers and if 
you have more questions, please ask.

GNU Binutils 2.20.1.20100303

Target: powerpc-timesys-linux-gnuspe
Configured with: 
/here/workdir/factory/build_powerpc-timesys-linux-gnuspe/gcc-4.4.4/gcc-4.4.4/configure
 --target=powerpc-timesys-linux-gnuspe --host=i386-redhat-linux 
--build=i386-redhat-linux --prefix=/here/w
orkdir/factory/build_powerpc-timesys-linux-gnuspe/toolchain 
--with-sysroot=/here/workdir/factory/build_powerpc-timesys-linux-gnuspe/toolchain
 --with-headers=/here/workdir/factory/build_powerpc-timesys-linux-gn
uspe/toolchain/include --enable-languages=c,c++ 
--with-gmp=/here/workdir/factory/build_powerpc-timesys-linux-gnuspe/gmp-5.0.1/gmp-host-install
 --with-mpfr=/here/workdir/factory/build_powerpc-timesys-linux-gnus
pe/mpfr-2.4.2/mpfr-host-install --disable-nls --disable-libgcj 
--disable-libgomp --disable-libmudflap --disable-libssp --with-gnu-ld 
--with-gnu-as --enable-shared --enable-checking=release --with-system-zlib -
-enable-__cxa_atexit --disable-libunwind-exceptions --enable-threads=posix 
--with-long-double-128 --enable-secureplt --with-cpu=8540 --enable-e500_double 
--disable-decimal-float --disable-multilib --enable-cxx-flags=-mcpu=8540
Thread model: posix
gcc version 4.4.4 (GCC)

glibc-2.11

There are a few patches that timesys applies.
I also know that there are updates to the current factory.
Let me build a new one and compare the output and see what happens.

-Bryan

From: Kumar Gala [ga...@kernel.crashing.org]
Sent: Friday, September 03, 2010 9:29 AM
To: Bryan Hundven
Cc: u-boot@lists.denx.de
Subject: Re: [U-Boot] P1011RDB  P1020RDB: NAND bootstrap too big

On Sep 2, 2010, at 9:25 PM, Bryan Hundven wrote:

 This was a git clone from today.
 Built with a pretty standard e500v2 gnuspe timesys toolchain.

 I was able to build and run P1020RDB_config on the freescale demo board.

 ---
 From: Bryan Hundven
 Sent: Thursday, September 02, 2010 2:42 PM
 To: u-boot@lists.denx.de
 Subject: P1011RDB  P1020RDB: NAND bootstrap too big

 When I build either P1011RDB_NAND_config or P1020RDB_NAND_config, I get:

 make[1]: Entering directory 
 `/home/bryan/Source/u-boot/nand_spl/board/freescale/p1_p2_rdb'

What git commit are you on?  What toolchain are you using?

- k
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Reinhard Meyer
Dear Mike Frysinger,

 pass the bus/cs/speed/mode in to the initialize function and store it in the
 per-device state.  then you can work fine with multiple enc28j60 devices in
 one board.

 along those lines, all of the local enc funcs should be changed to take the
 private state and operate off that instead of random global variables.

Something like this?

typedef struct enc_device {
struct eth_device   netdev;
struct spi_slave*slave;
u16 next_pointer;
int rx_reset_counter;
u8  bank;   /* current bank in enc28j60 */
} enc_dev_t;
#define to_enc(_nd) container_of(_nd, struct enc_device, netdev)

...

static u8 enc_r8(enc_dev_t *, u16 reg);
static u16 enc_r16(enc_dev_t *, u16 reg);
static void enc_w8(enc_dev_t *, u16 reg, u8 data);
static void enc_w16(enc_dev_t *, u16 reg, u16 data);
static void enc_w8_retry(enc_dev_t *, u16 reg, u8 data, int c);

I agree that's more perfect but kind of overkill, too;
assuming that only one enc can be active at any given time;)

Best Regards,
Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Severals questions How to submit new u-boot features

2010-09-03 Thread Albert ARIBAUD
Hi,

Le 03/09/2010 19:06, Christophe Henri RICARD a écrit :
 Hi,

 I am actually working to add a new driver to u-boot related to Trusted 
 Platform Module (TPM) for embedded system. (On a u-boot version 2009.11)
 I would be interested to know:

 -   what is the process to do the submission ?

Read up http://www.denx.de/wiki/U-Boot/Patches

 -  Because the available i2c interfaces looks to be only for i2c 
 memory (such as eeprom memory), I had to rework some i2c interfaces to be 
 able to send directly my TPM command/data through the i2c bus. This mean 
 write or read some data directly on the i2c bus without sending any offset 
 byte.

You can do that with the current i2c drivers by making sure that adrlen=0.

 Will it be acceptable for the community to add those two new interfaces 
 (i2c_write_direct/i2c_read_direct) for each i2c driver ?

I would think they are unneeded, see the adrlen=0 trick. (If I were to 
suggest an I2C overhaul [hi Heiko :)], I'd rather suggest adding a 
generic i2c transaction API which would allow mixing elementary writes 
and reads with optional STARTs and STOPs inbetween).

 -  Is there any coding style specification ?

Look up http://www.denx.de/wiki/U-Boot/CodingStyle (actually many 
topics are covered, see left column).

 Thanks for your help

np :)

 Best Regards
 Christophe RICARD
 STMicroelectronics.

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Mike Frysinger
On Friday, September 03, 2010 13:50:28 Reinhard Meyer wrote:
 Something like this?

yes

 I agree that's more perfect but kind of overkill, too;
 assuming that only one enc can be active at any given time;)

yes and no.  the spi bus/cs/mode/speed is established at initialize() time, 
not init(), so you'd need to store that per-instance information somewhere.  
and how the discussion about netconsole handling falls out (not calling 
init/halt after every transaction), this base assumption may not be valid.
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Reinhard Meyer
On 03.09.2010 21:51, Mike Frysinger wrote:
 yes and no.  the spi bus/cs/mode/speed is established at initialize() time,
 not init(), so you'd need to store that per-instance information somewhere.
 and how the discussion about netconsole handling falls out (not calling
 init/halt after every transaction), this base assumption may not be valid.
 -mike

/*
  * This is the only exported function.
  *
  * It may be called several times with different bus:cs combinations.
  */
int enc_initialize(int bus, int cs, int speed)
{
enc_dev_t *enc;

enc = malloc(sizeof(*enc));
if (!enc) {
return -1;
}
memset(enc, 0, sizeof(*enc));
enc-slave = spi_setup_slave(bus, cs, speed, SPI_MODE_0);
if (!enc-slave) {
free(enc);
return -1;
}
enc-netdev.init = enc_init;
enc-netdev.halt = enc_halt;
enc-netdev.send = enc_send;
enc-netdev.recv = enc_recv;
sprintf(enc-netdev.name, enc%i.%i, bus, cs);
eth_register((enc-netdev));
return 0;
}
(not compiled yet)
not sure, however, if calling spi_setup_slave() is ok at this point.

Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Mike Frysinger
On Friday, September 03, 2010 16:31:47 Reinhard Meyer wrote:
 int enc_initialize(int bus, int cs, int speed)

should take the same parameters (including type) as spi_setup_slave

   eth_register((enc-netdev));

no need for the () before the 

 not sure, however, if calling spi_setup_slave() is ok at this point.

i believe the func should only doing validation on the arguments.  it shouldnt 
need to talk to any actual hardware.  maybe this needs codifying in the API 
documentation.
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Reinhard Meyer
On 03.09.2010 23:03, Mike Frysinger wrote:
 not sure, however, if calling spi_setup_slave() is ok at this point.

 i believe the func should only doing validation on the arguments.  it shouldnt
 need to talk to any actual hardware.  maybe this needs codifying in the API
 documentation.
In atmel_spi.c it sets up the csr[indexed by cs=0..3] register with clock,
phase and polarity.
That's harmless, but might need some rethinking there.

Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] zImage on ARM

2010-09-03 Thread Loïc Minier
On Fri, Sep 03, 2010, Wolfgang Denk wrote:
  I have a client that has asked me to look into loading zImage files
  instead of uImage for some ARM based boards.
 
 Why?

 It's the direct output of the kernel build process; typically,
 distributions ship a /boot/zImage-foo which gets converted into an
 uImage and flashed if the bootloader is u-boot.  If that step could be
 skipped, it would make things easier!

  Anyone familiar with the history of why uImage is the preferred
  format?  Just trying to learn . . .
 
 uImage, when done correctly, avoids that you have to add a pre-loader
 (including all of gzip and what else is needed to copy the uncom-
 pressed image to it's final location in RAM) to each and every kernel
 image - which is just a waste of time and resources as we already
 have all that code present in U-Boot.
 
 Note that the mainline ARM kernel does not make use of this; instead,
 it adds the preloader anyway.  I never understood why.

 Seems zImage is quite widespread now; would it make sense to allow
 builing u-boot without that code and rely on the kernel code to unpack?
 Or should u-boot just gain a new image type for zImage?

-- 
Loïc Minier
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Mike Frysinger
On Friday, September 03, 2010 17:23:43 Reinhard Meyer wrote:
 On 03.09.2010 23:03, Mike Frysinger wrote:
  not sure, however, if calling spi_setup_slave() is ok at this point.
  
  i believe the func should only doing validation on the arguments.  it
  shouldnt need to talk to any actual hardware.  maybe this needs
  codifying in the API documentation.
 
 In atmel_spi.c it sets up the csr[indexed by cs=0..3] register with clock,
 phase and polarity.
 That's harmless, but might need some rethinking there.

i'm not familiar with the Atmel SPI controller, but my gut reaction is that 
register values should be calculated in spi_setup_slave(), stored in the 
internal xxx_spi_slave struct, and then written at spi_claim_bus() time.
-mike


signature.asc
Description: This is a digitally signed message part.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [GIT PULL] MIPS patch

2010-09-03 Thread Shinya Kuribayashi
The following changes since commit bd2313078114c4b44c4a5ce149af43bcb7fc8854:

  Merge branch 'master' of ssh://gemini/home/wd/git/u-boot/master (2010-08-18 
21:16:35 +0200)

are available in the git repository at:

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

Xiangfu Liu (1):
  MIPS: update the MIPS u-boot.lds

 board/dbau1x00/u-boot.lds|2 +-
 board/gth2/u-boot.lds|2 +-
 board/incaip/u-boot.lds  |2 +-
 board/pb1x00/u-boot.lds  |2 +-
 board/purple/u-boot.lds  |2 +-
 board/qemu-mips/u-boot.lds   |2 +-
 examples/standalone/mips.lds |2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] NET: add ENC28J60 driver using SPI framework

2010-09-03 Thread Reinhard Meyer
On 04.09.2010 00:14, Mike Frysinger wrote:
 On Friday, September 03, 2010 17:23:43 Reinhard Meyer wrote:
 On 03.09.2010 23:03, Mike Frysinger wrote:
 not sure, however, if calling spi_setup_slave() is ok at this point.

 i believe the func should only doing validation on the arguments.  it
 shouldnt need to talk to any actual hardware.  maybe this needs
 codifying in the API documentation.

 In atmel_spi.c it sets up the csr[indexed by cs=0..3] register with clock,
 phase and polarity.
 That's harmless, but might need some rethinking there.

 i'm not familiar with the Atmel SPI controller, but my gut reaction is that
 register values should be calculated in spi_setup_slave(), stored in the
 internal xxx_spi_slave struct, and then written at spi_claim_bus() time.
 -mike
Well, the SPI hardware can store the settings for 4 chip selects, and quite
automagically switch between them, including programmable switch times. When
using DMA it would be possible to setup transfers to/from different chip
selects without software doing the switches, ex:
15 words of 13 bits using 199kHz and mode 3 via CS2,
2 words of 8 bits using 10MHz and mode 0 via CS3, etc...
All this, of course, is not needed in u-boot and currently only half used
anyway. Therefore I tend to rework the atmel_spi.c not to make use of that
feature at all.

For codig the ENC driver I will just postulate its save to setup the slave
at initialize time.

Reinhard
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot