Re: [U-Boot-Users] U-Boot-Users Digest, Vol 23, Issue 48

2008-04-10 Thread Peter.wang
hi, I recently read the code of u-boot, but I have a problem about its 
relocation and uboot how to pass the paramter to the linux kernel. could 
you help me?, at least ,tell where i can find the code of these function 
,thanks .

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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] PPC440 Power Management Registers

2008-04-10 Thread Stefan Roese
On Thursday 10 April 2008, Wolfgang Denk wrote:
> In message <[EMAIL PROTECTED]> you wrote:
> > Unfortunately now your patch is line wrapped. And even if it wasn't line
> > wrapped, it couldn't be applied using git-am (which makes life so much
> > easier for me) because of the lines above and below the real patch.
>
> This is not a big problem; "git-am -i" will allow you to edit the
> commit message if needed.

Right. But it would be better if it could be avoided. Less work for us 
custodians. ;)

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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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] ppc: Revert patch 70431e8a that used _start instead of CFG_MONITOR_BASE

2008-04-10 Thread Kim Phillips
On Wed,  9 Apr 2008 12:56:42 +0200
Stefan Roese <[EMAIL PROTECTED]> wrote:

> The patch 70431e8a7393b6b793f77957f95b999fc9a269b8 (Make MPC83xx one step
> closer to full relocation.) doesn't use CFG_MONITOR_BASE anymore. But
> on 4xx systems _start currently cannot be used for this calculation.
> So revert back to the original version for now.
> 
> Signed-off-by: Stefan Roese <[EMAIL PROTECTED]>

Acked-by: Kim Phillips <[EMAIL PROTECTED]>

Kim

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] drivers MMCplus for at91sam9x

2008-04-10 Thread Ken.Fuchs
Pierre Savary wrote:

> I use a MMCplus 4GB on my design (with at91sam9260). It's 
> wired with 4 bits.

The MCI controller on the AT91SAM926x family does not support
MMCplus.  There is no way to support a 4 bit bus, since the
MCI controller supports only 1 bit to an MMC chip.  The MCI will
support a 4 bit SD chip, but I don't think it can be tricked
into working with a 4 bit MMC chip (at least not via software
alone).

The only reasonable solution is switching to a processor that
has MMCplus support.  Maybe Atmel has been working on one?

> Currently U-boot (1.1.5) can't detect correctly the MMC and 
> so I can't read my Linux kernel Image on the ext3 part of this
> MMC. If I use MMC 1GB, it works.

Did you add MMCplus commands to the MCI U-Boot driver?  You can
do this to the extent that these changes do not require the MCI
controller to be MMCplus compliant.  (The MCI U-Boot driver I'm
aware of contains no MMCplus support.)

I suggest that you use the u-boot sources available via git.
You will get very little support (if any) from the U-Boot ML for
two reasons:

1) U-Boot 1.1.5 is extremely old and no one on the list is
   interested in supporting it.

2) The version of U-Boot 1.1.5 you are using almost certainly
   has an Atmel patch applied to it that was never accepted
   into the "official" U-Boot tree.

However, ...

It appears that someone is working on the AT91SAM9260 within the
official U-Boot (git) tree, since the ./include/configs/at91sam9260ek.h
file is there.  Other file structures like ./cpu/arm926ejs/* seem to be
missing some drivers and other support files.  I know that the
AT91SAM926x specific U-Boot files were never an official part of
U-Boot, but that appears to be changing.  It is my understanding
that the AT91SAM926x support is being completely reworked and
integrated with the AT91CAP9 code that has been present in git
for over a month already.  The AT91CAP9 and AT91SAM926x are very
similar and should share a lot of code.

Even if the AT91SAM9260 is not quite ready, the AT91CAP9 code
should provide the basis of a current U-Boot for the AT91SAM926x
family.

> Somebody have already use it? Or somebody have already implemented
> the ext_csd and high capacity with MMC on another platform?

I know that the MCI controller can be used to access 1GB MMCplus chips,
but I'm not sure it can be used to access MMCplus chips larger than
that.
There may be a special MMCplus command that will allow larger chips to
be accessed.  Try looking for it in your MMCplus chip's manual.

Sincerely,

Ken Fuchs

> -Original Message-
> From: [EMAIL PROTECTED] 
> [mailto:[EMAIL PROTECTED] On Behalf 
> Of Pierre Savary
> Sent: Tuesday, April 08, 2008 09:32
> To: u-boot-users@lists.sourceforge.net
> Cc: 'Pierre Ossman'
> Subject: [U-Boot-Users] drivers MMCplus for at91sam9x
> 
> 
> Hi,
> I use a MMCplus 4GB on my design (with at91sam9260). It's 
> wired with 4 bits.
> Currently U-boot (1.1.5) can't detect correctly the MMC and 
> so I can't read
> my Linux kernel Image on the ext3 part of this MMC. If I use 
> MMC 1GB, it
> works.
> Somebody have already use it? Or somebody have already implemented the
> ext_csd and high capacity with MMC on another platform?
> I need help. Thanks in advance to help me.
> 
> Regards,
> 
> Pierre
> 
> 
> --
> ---
> This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
> Register now and save $200. Hurry, offer ends at 11:59 p.m., 
> Monday, April 7! Use priority code J8TLD2. 
> http://ad.doubleclick.net/clk;198757673;13503038;p?http://java
.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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 MPC8343 based board mvBlueLYNX-M7 aka mvblm7

2008-04-10 Thread André Schwarz

Wolfgang,

that's absolutely fine since there are some open issues regarding docs 
and coding style.

I didn't dare to believe getting the board in in the first run anyway ;-)

Sometimes the rules are a bit annoying - but they are definitely necessary !


Cheers,
André

Wolfgang Denk wrote:

Dear Kim,

in message <[EMAIL PROTECTED]> you wrote:
  

Wolfgang, I believe the window for new board support is closed; would



Correct.

  

you rather I maintain this on a "next" branch on mpc83xx, or are you
willing to let this in for 1.3.3?



It's all new stuff, so it should go to "next".

Thanks.

Best regards,

Wolfgang Denk

  




MATRIX VISION GmbH, Talstraße 16, DE-71570 Oppenweiler  - Registergericht: 
Amtsgericht Stuttgart, HRB 271090
Geschäftsführer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
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 MPC8343 based board mvBlueLYNX-M7 aka mvblm7

2008-04-10 Thread André Schwarz

Kim,

I'll fix all your issues mentioned below in the next days.
We're not in a hurry. Getting the board into 1.3.3 is absolutely fine.

Thanks,
André

Kim Phillips wrote:

On Wed, 09 Apr 2008 16:16:28 +0200
Andre Schwarz <[EMAIL PROTECTED]> wrote:

  

MPC8343 based stereo camera system with Cyclone-II FPGA and miniPCI Slot.
CPU utilizes dual 10/100/1000 Ethernet using Vitesse VSC8601 RGMII Phys 
and USB over ULPI.
512MB Micron DDR-II memory, 8MB Flash on LocalBus, SD over SPU and 32MB 
NAND @ FPGA.


Signed-off-by: Andre Schwarz <[EMAIL PROTECTED]>
--



Wolfgang, I believe the window for new board support is closed; would
you rather I maintain this on a "next" branch on mpc83xx, or are you
willing to let this in for 1.3.3?

Andre, thanks for the new board contribution - please include diffstats
in your patches.

  

diff --git a/CREDITS b/CREDITS
index e84ef38..713f58a 100644
--- a/CREDITS
+++ b/CREDITS
@@ -424,6 +424,11 @@ N: Paolo Scaffardi
 E: [EMAIL PROTECTED]
 D: FADS823 configuration, MPC823 video support, I2C, wireless keyboard, lots 
more
 
+N: Andre Schwarz

+E: [EMAIL PROTECTED]
+D: Support for BlueLYNX and BlueCOUGAR series
+W: www.matrix-vision.com
+
 N: Robert Schwebel
 E: [EMAIL PROTECTED]
 D: Support for csb226, logodl and innokom boards (PXA2xx)
diff --git a/MAKEALL b/MAKEALL
index 2a872ac..f21c34e 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -327,6 +327,7 @@ LIST_83xx="\
MPC8360ERDK_66  \
MPC837XEMDS \
MPC837XERDB \
+   MVBLM7  \
sbc8349 \
TQM834x \
 "
diff --git a/Makefile b/Makefile
index a7f886b..3f217fe 100644
--- a/Makefile
+++ b/Makefile
@@ -2084,6 +2084,9 @@ sbc8349_config:   unconfig
 TQM834x_config:unconfig
@$(MKCONFIG) $(@:_config=) ppc mpc83xx tqm834x
 
+MVBLM7_config: unconfig

+   @$(MKCONFIG) $(@:_config=) ppc mpc83xx mvblm7
+



I believe intra-family targets are kept in alpha order in the Makefile
(as you did in MAKEALL).

please add a MAINTAINERS entry. A doc/README.mvblm7 would be nice.


  

diff --git a/board/mvblm7/config.mk b/board/mvblm7/config.mk
new file mode 100644
index 000..a659203
--- /dev/null
+++ b/board/mvblm7/config.mk



  

+#
+# MPC8349E-mITX and MPC8349E-mITX-GP
+#



I'd leave this out :)

  

+
+sinclude $(OBJTREE)/board/$(BOARDDIR)/config.tmp
+
+TEXT_BASE  = 0xFFF0
diff --git a/board/mvblm7/fpga.c b/board/mvblm7/fpga.c
new file mode 100644
index 000..d286ef1
--- /dev/null
+++ b/board/mvblm7/fpga.c



  

+#if (CONFIG_FPGA)



#if defined?


  

+int mvblm7_init_fpga (void)



we don't put spaces between function name and open parenthesis, do we?

All your functions are like this.

  

+{
+   DECLARE_GLOBAL_DATA_PTR;



this needs to be put somewhere global.

  

+
+   fpga_debug("%s:%d: Initialize FPGA interface (relocation offset = 
0x%.8lx)\n",
+   __FUNCTION__, __LINE__, gd->reloc_off);
+   fpga_init (gd->reloc_off);
+
+   fpga_debug("%s:%d: Adding fpga 0\n", __FUNCTION__, __LINE__);
+   fpga_add (fpga_altera, &cyclone2);
+   return 1;
+}
+
+int fpga_null_fn (int cookie)
+{
+return 0;
+}
+
+int fpga_config_fn (int assert, int flush, int cookie)
+{
+volatile immap_t*im = (volatile immap_t *)CFG_IMMR;
+volatile gpio83xx_t*gpio = (volatile gpio83xx_t *)&im->gpio[0];



fix indentation; use tabs, not spaces.

  

+
+   u32 dvo = gpio->dat;



keep declarations together, leave blank space between declarations and
code - i.e, mv above blank line to before this line:

  

+   fpga_debug("SET config : %s\n", assert ? "low" : "high");
+	if ( assert ) 



no spaces around assert

  

+   dvo |= FPGA_CONFIG;
+	else 
+		dvo &= ~FPGA_CONFIG;

+   
+   if ( flush )
+   gpio->dat = dvo;
+   return assert;



can you also get in the habit of leaving blank lines before return
statements?  thanks.



  

+   for ( i=0; i<8; i++ ) {



please read CodingStyle!  this should look like:

for (i = 0; i < 8; i++) {



  

diff --git a/board/mvblm7/mvblm7.c b/board/mvblm7/mvblm7.c



  

+im->ddr.sdram_interval = CFG_DDR_INTERVAL;
+im->ddr.sdram_clk_cntl = CFG_DDR_CLK_CNTL;



use tabs, not spaces.


  

+u8 *dhcp_vendorex_prep (u8 * e)



u8 *dhcp_vendorex_prep(u8 *e)

  

+{
+char *ptr;
+
+/* DHCP vendor-class-identifier = 60 */
+if ((ptr = getenv ("dhcp_vendor-class-identifier"))) {



fix indentation.  Use tabs, not spaces.

  

+*e++ = 60;
+*e++ = strlen (ptr);



strlen(ptr);

  

+while (*ptr)
+*e++ = *ptr++;
+}
+/* DHCP_CLIENT_IDENTIFIER = 61 */
+if ((ptr = getenv ("dhcp_client_id"))) {



..


  

diff --git a/board/mvblm7/pci.c b/board/mvblm7/pci.c



  

+   tmp[0] = cpu_to_be32(gd->pci_clk);
+   do_fixup_by_path(blob, path, "clock-freq

Re: [U-Boot-Users] question about environment variable in U-BOOT

2008-04-10 Thread Jerry Van Baren
sonicss wrote:
> U-BOOT:
> I want to modify the default environment variable in U-BOOT. The 
> following info are about the hardware:
>  
> 
> ARM: at91rm9200
> platform: at91rm9200dk
> 
> U-BOOT: CFG_ENV_IS_IN_DATAFLASH
>  
> I modifed /include/configs/at91rm9200dk.h and add the following lines 
> after "#define CONFIG_BAUDRATE 115200":
>  
> #define CONFIG_ETHADDR 12.24.96.78.91.11
> #define CONFIG_IPADDR 172.19.71.10
> #define CONFIG_SERVERIP 172.19.71.40
> *#define CONFIG_BOOTCMD = tftp 20008000 zImage; tftp 2100 Ramdisk.gz; go 
> 20008000*
>  
> the first three commands work well, but the last command is not in the 
> environment. I use printenv command to show the current

1) Your #define for CONFIG_BOOTCMD is not a valid C preprocessor define
2) You probably wanted to #define CONFIG_BOOTCOMMAND, not CONFIG_BOOTCMD

Something closer (but not necessarily correct) would be:
#define CONFIG_BOOTCOMMAND "tftp 20008000 zImage; tftp 2100 
Ramdisk.gz; go 20008000"

[snip]

> There is not bootcmd command. How can I implement this function? ( just 
> add bootcmd before compile u-boot)
>  
> best regards
>  
> 2008-04-11
> 
> sonicss

Good luck,
gvb

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] Request to remove SEARCH_DIR from lds files

2008-04-10 Thread Jason Wessel
Unless someone has an objection, would it be possible to remove all
the SEARCH_DIR() directives from the lds files with the attached
patch?

Modern cross compilers will automatically find the correct link time
libraries so there is no need to go searching in places that might
have libraries which you do not actually want to link in, such as
probing /usr/lib on an x86 host for powerpc link objects.

Thanks,
Jason.


remove_SEARCH_DIR_from_lds_files.patch.gz
Description: GNU Zip compressed data
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] Linker scripts, SEARCH_DIR and host directories

2008-04-10 Thread kenneth johansson
On Thu, 2008-04-10 at 20:34 +0100, Richard Danter wrote:
> Hi all,
> 
> Using a new toolchain I am seeing the following warnings at the final
> link stage of U-Boot:
> 
> warning: library search path "/lib" is unsafe for cross-compilation
> warning: library search path "/usr/lib" is unsafe for cross-compilation
> warning: library search path "/usr/local/lib" is unsafe for cross-compilation
> 
> Looking further I can see in the u-boot.lds scripts that SEARCH_DIR is
> specifying each of these directories. I am unsure why a cross-compiled
> bootloader would specify that the linker should look in the hosts
> directories for libraries.
> 
> Can anyone explain this?
> 
> Thanks
> Rich

You should delete that from the linker script. There is several scripts
that has this error it's not needed and wrong as you suspected.





-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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 MPC8343 based board mvBlueLYNX-M7 aka mvblm7

2008-04-10 Thread Wolfgang Denk
Dear Kim,

in message <[EMAIL PROTECTED]> you wrote:
>
> Wolfgang, I believe the window for new board support is closed; would

Correct.

> you rather I maintain this on a "next" branch on mpc83xx, or are you
> willing to let this in for 1.3.3?

It's all new stuff, so it should go to "next".

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]
"It is better to have tried and failed than to have  failed  to  try,
but the result's the same."   - Mike Dennison

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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] PPC440 Power Management Registers

2008-04-10 Thread Wolfgang Denk
In message <[EMAIL PROTECTED]> you wrote:
> 
> Unfortunately now your patch is line wrapped. And even if it wasn't line 
> wrapped, it couldn't be applied using git-am (which makes life so much easier 
> for me) because of the lines above and below the real patch.

This is not a big problem; "git-am -i" will allow you to edit the
commit message if needed.

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]
HANDLE WITH EXTREME CARE:  This Product Contains  Minute Electrically
Charged  Particles  Moving  at  Velocities  in Excess of Five Hundred
Million Miles Per Hour.

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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/review] Blackfin: resurrect BF533-STAMP video splash driver

2008-04-10 Thread Wolfgang Denk
In message <[EMAIL PROTECTED]> you wrote:
>
...
> > +typedef struct {
> > +   unsigned int SAV;
> > +   unsigned int EAV;
> > +} SystemCodeType;
 ^
> > +
> > +const SystemCodeType SystemCodeMap[4] = {
 ^
> > +   {0xFF80, 0xFF9D},
> > +   {0xFFAB, 0xFFB6},
> > +   {0xFFC7, 0xFFDA},
> > +   {0xFFEC, 0xFFF1}
> > +};
> > -- 
> Personnaly, I do not like var name that strart with UPPERCASE

It's more than not liking. It's a clear violation of the coding style:

C is a Spartan language, and so should your naming be.  Unlike Modula-2
and Pascal programmers, C programmers do not use cute names like
ThisVariableIsATemporaryCounter.  A C programmer would call that
variable "tmp", which is much easier to write, and not the least more
difficult to understand.

So this is a clear NAK.

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]
Nothing is easier than to denounce  the  evildoer;  nothing  is  more
difficult than to understand him. - Fyodor Dostoevski

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Linker scripts, SEARCH_DIR and host directories

2008-04-10 Thread Richard Danter
Hi all,

Using a new toolchain I am seeing the following warnings at the final
link stage of U-Boot:

warning: library search path "/lib" is unsafe for cross-compilation
warning: library search path "/usr/lib" is unsafe for cross-compilation
warning: library search path "/usr/local/lib" is unsafe for cross-compilation

Looking further I can see in the u-boot.lds scripts that SEARCH_DIR is
specifying each of these directories. I am unsure why a cross-compiled
bootloader would specify that the linker should look in the hosts
directories for libraries.

Can anyone explain this?

Thanks
Rich

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] question about environment variable in U-BOOT

2008-04-10 Thread sonicss
U-BOOT:
I want to modify the default environment variable in U-BOOT. The following info 
are about the hardware:

ARM: at91rm9200
platform: at91rm9200dk
U-BOOT: CFG_ENV_IS_IN_DATAFLASH

I modifed /include/configs/at91rm9200dk.h and add the following lines after 
"#define CONFIG_BAUDRATE 115200":

#define CONFIG_ETHADDR 12.24.96.78.91.11
#define CONFIG_IPADDR 172.19.71.10
#define CONFIG_SERVERIP 172.19.71.40
#define CONFIG_BOOTCMD = tftp 20008000 zImage; tftp 2100 Ramdisk.gz; go 
20008000

the first three commands work well, but the last command is not in the 
environment. I use printenv command to show the current

environment:
U-Boot> printenv
bootdelay=3
baudrate=115200
ethaddr=12.24.96.78.91.11
ipaddr=172.19.71.10
serverip=172.19.71.40
stdin=serial
stdout=serial
stderr=serial

There is not bootcmd command. How can I implement this function? ( just add 
bootcmd before compile u-boot)

best regards

2008-04-11 



sonicss 
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
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 MPC8343 based board mvBlueLYNX-M7 aka mvblm7

2008-04-10 Thread Kim Phillips
On Wed, 09 Apr 2008 16:16:28 +0200
Andre Schwarz <[EMAIL PROTECTED]> wrote:

> MPC8343 based stereo camera system with Cyclone-II FPGA and miniPCI Slot.
> CPU utilizes dual 10/100/1000 Ethernet using Vitesse VSC8601 RGMII Phys 
> and USB over ULPI.
> 512MB Micron DDR-II memory, 8MB Flash on LocalBus, SD over SPU and 32MB 
> NAND @ FPGA.
> 
> Signed-off-by: Andre Schwarz <[EMAIL PROTECTED]>
> --

Wolfgang, I believe the window for new board support is closed; would
you rather I maintain this on a "next" branch on mpc83xx, or are you
willing to let this in for 1.3.3?

Andre, thanks for the new board contribution - please include diffstats
in your patches.

> diff --git a/CREDITS b/CREDITS
> index e84ef38..713f58a 100644
> --- a/CREDITS
> +++ b/CREDITS
> @@ -424,6 +424,11 @@ N: Paolo Scaffardi
>  E: [EMAIL PROTECTED]
>  D: FADS823 configuration, MPC823 video support, I2C, wireless keyboard, lots 
> more
>  
> +N: Andre Schwarz
> +E: [EMAIL PROTECTED]
> +D: Support for BlueLYNX and BlueCOUGAR series
> +W: www.matrix-vision.com
> +
>  N: Robert Schwebel
>  E: [EMAIL PROTECTED]
>  D: Support for csb226, logodl and innokom boards (PXA2xx)
> diff --git a/MAKEALL b/MAKEALL
> index 2a872ac..f21c34e 100755
> --- a/MAKEALL
> +++ b/MAKEALL
> @@ -327,6 +327,7 @@ LIST_83xx="   \
>   MPC8360ERDK_66  \
>   MPC837XEMDS \
>   MPC837XERDB \
> + MVBLM7  \
>   sbc8349 \
>   TQM834x \
>  "
> diff --git a/Makefile b/Makefile
> index a7f886b..3f217fe 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2084,6 +2084,9 @@ sbc8349_config: unconfig
>  TQM834x_config:  unconfig
>   @$(MKCONFIG) $(@:_config=) ppc mpc83xx tqm834x
>  
> +MVBLM7_config: unconfig
> + @$(MKCONFIG) $(@:_config=) ppc mpc83xx mvblm7
> +

I believe intra-family targets are kept in alpha order in the Makefile
(as you did in MAKEALL).

please add a MAINTAINERS entry. A doc/README.mvblm7 would be nice.


> diff --git a/board/mvblm7/config.mk b/board/mvblm7/config.mk
> new file mode 100644
> index 000..a659203
> --- /dev/null
> +++ b/board/mvblm7/config.mk

> +#
> +# MPC8349E-mITX and MPC8349E-mITX-GP
> +#

I'd leave this out :)

> +
> +sinclude $(OBJTREE)/board/$(BOARDDIR)/config.tmp
> +
> +TEXT_BASE  = 0xFFF0
> diff --git a/board/mvblm7/fpga.c b/board/mvblm7/fpga.c
> new file mode 100644
> index 000..d286ef1
> --- /dev/null
> +++ b/board/mvblm7/fpga.c

> +#if (CONFIG_FPGA)

#if defined?


> +int mvblm7_init_fpga (void)

we don't put spaces between function name and open parenthesis, do we?

All your functions are like this.

> +{
> + DECLARE_GLOBAL_DATA_PTR;

this needs to be put somewhere global.

> +
> + fpga_debug("%s:%d: Initialize FPGA interface (relocation offset = 
> 0x%.8lx)\n",
> + __FUNCTION__, __LINE__, gd->reloc_off);
> + fpga_init (gd->reloc_off);
> +
> + fpga_debug("%s:%d: Adding fpga 0\n", __FUNCTION__, __LINE__);
> + fpga_add (fpga_altera, &cyclone2);
> + return 1;
> +}
> +
> +int fpga_null_fn (int cookie)
> +{
> +return 0;
> +}
> +
> +int fpga_config_fn (int assert, int flush, int cookie)
> +{
> +volatile immap_t*im = (volatile immap_t *)CFG_IMMR;
> +volatile gpio83xx_t  *gpio = (volatile gpio83xx_t *)&im->gpio[0];

fix indentation; use tabs, not spaces.

> +
> + u32 dvo = gpio->dat;

keep declarations together, leave blank space between declarations and
code - i.e, mv above blank line to before this line:

> + fpga_debug("SET config : %s\n", assert ? "low" : "high");
> + if ( assert ) 

no spaces around assert

> + dvo |= FPGA_CONFIG;
> + else 
> + dvo &= ~FPGA_CONFIG;
> + 
> + if ( flush )
> + gpio->dat = dvo;
> + return assert;

can you also get in the habit of leaving blank lines before return
statements?  thanks.



> + for ( i=0; i<8; i++ ) {

please read CodingStyle!  this should look like:

for (i = 0; i < 8; i++) {



> diff --git a/board/mvblm7/mvblm7.c b/board/mvblm7/mvblm7.c

> +im->ddr.sdram_interval = CFG_DDR_INTERVAL;
> +im->ddr.sdram_clk_cntl = CFG_DDR_CLK_CNTL;

use tabs, not spaces.


> +u8 *dhcp_vendorex_prep (u8 * e)

u8 *dhcp_vendorex_prep(u8 *e)

> +{
> +char *ptr;
> +
> +/* DHCP vendor-class-identifier = 60 */
> +if ((ptr = getenv ("dhcp_vendor-class-identifier"))) {

fix indentation.  Use tabs, not spaces.

> +*e++ = 60;
> +*e++ = strlen (ptr);

strlen(ptr);

> +while (*ptr)
> +*e++ = *ptr++;
> +}
> +/* DHCP_CLIENT_IDENTIFIER = 61 */
> +if ((ptr = getenv ("dhcp_client_id"))) {

..


> diff --git a/board/mvblm7/pci.c b/board/mvblm7/pci.c

> + tmp[0] = cpu_to_be32(gd->pci_clk);
> + do_fixup_by_path(blob, path, "clock-frequency",
> + &tmp, sizeof(tmp[0]), 1);
> + }
> + }
> +}
> +#endif
 
please use generic 83xx pci code (cpu/mpc83xx/pci.c).  See the
mp

Re: [U-Boot-Users] [PATCH] qemu-mips: add CFI support

2008-04-10 Thread Shinya Kuribayashi
Hi Jean,

Jean-Christophe PLAGNIOL-VILLARD wrote:
> Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]>
> 
> diff --git a/include/configs/qemu-mips.h b/include/configs/qemu-mips.h
> index e164019..7ee6fd4 100644
> --- a/include/configs/qemu-mips.h
> +++ b/include/configs/qemu-mips.h
> @@ -33,7 +33,7 @@
>  #define CONFIG_MISC_INIT_R
>  
>  /*IP address is default used by Qemu*/
> -#define CONFIG_IPADDR10.0.2.15/* Our IP address 
> */
> +#define CONFIG_IPADDR10.0.2.15/* Our IP address 
> */
>  #define CONFIG_SERVERIP  10.0.2.2 /* Server IP 
> address*/
>  
>  #define CONFIG_BOOTDELAY 10  /* autoboot after 10 seconds*/
> @@ -74,16 +74,12 @@
>  #define CONFIG_CMD_ELF
>  #define CONFIG_CMD_FAT
>  #define CONFIG_CMD_EXT2
> -#undef  CONFIG_CMD_IMLS
> -#undef  CONFIG_CMD_FLASH
> -#undef  CONFIG_CMD_LOADB
> -#undef  CONFIG_CMD_LOADS
>  #define CONFIG_CMD_DHCP
> +#define CONFIG_CMD_PING
>  
>  #define CONFIG_DRIVER_NE2000

This patch contains as many fixes as CFI related changes (above).
In that case, please add some notes about them or change the subject.

I applied the patch below. If something wrong, please let me know.
Or will push in a week or two :-)

  Shinya


[MIPS] qemu-mips.h: Update config header

From: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]>

- Add CFI support
- Add CONFIG_CMD_PING since qemu-mips has NE2000 driver enabled
- Cleanup whitespaces and remove #undef CMD lines

Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]>
Signed-off-by: Shinya Kuribayashi <[EMAIL PROTECTED]>
---

 include/configs/qemu-mips.h |   24 +---
 1 files changed, 13 insertions(+), 11 deletions(-)


diff --git a/include/configs/qemu-mips.h b/include/configs/qemu-mips.h
index e164019..7ee6fd4 100644
--- a/include/configs/qemu-mips.h
+++ b/include/configs/qemu-mips.h
@@ -33,7 +33,7 @@
 #define CONFIG_MISC_INIT_R
 
 /*IP address is default used by Qemu*/
-#define CONFIG_IPADDR  10.0.2.15/* Our IP address */
+#define CONFIG_IPADDR  10.0.2.15/* Our IP address */
 #define CONFIG_SERVERIP10.0.2.2 /* Server IP 
address*/
 
 #define CONFIG_BOOTDELAY   10  /* autoboot after 10 seconds*/
@@ -74,16 +74,12 @@
 #define CONFIG_CMD_ELF
 #define CONFIG_CMD_FAT
 #define CONFIG_CMD_EXT2
-#undef  CONFIG_CMD_IMLS
-#undef  CONFIG_CMD_FLASH
-#undef  CONFIG_CMD_LOADB
-#undef  CONFIG_CMD_LOADS
 #define CONFIG_CMD_DHCP
+#define CONFIG_CMD_PING
 
 #define CONFIG_DRIVER_NE2000
 #define CONFIG_DRIVER_NE2000_BASE  (0xb4000300)
 
-#define CFG_NO_FLASH
 #define CFG_NS16550
 #define CFG_NS16550_SERIAL
 #define CFG_NS16550_REG_SIZE1
@@ -140,17 +136,23 @@
 
 /* The following #defines are needed to get flash environment right */
 #defineCFG_MONITOR_BASETEXT_BASE
-#defineCFG_MONITOR_LEN (192 << 10)
+#defineCFG_MONITOR_LEN (192 << 11)
 
 #define CFG_INIT_SP_OFFSET 0x40
 
 /* We boot from this flash, selected with dip switch */
 #define CFG_FLASH_BASE 0xbfc0
-
-#defineCFG_ENV_IS_NOWHERE  1
-
+#defineCFG_MAX_FLASH_BANKS 1
+#defineCFG_MAX_FLASH_SECT  128
+#defineCFG_FLASH_CFI   1   /* Flash memory is CFI 
compliant */
+#define CFG_FLASH_CFI_DRIVER   1
+#define CFG_FLASH_USE_BUFFER_WRITE1
+
+#defineCFG_ENV_IS_IN_FLASH 1
+#define CFG_ENV_ADDR   (CFG_FLASH_BASE+0x6)
 /* Address and size of Primary Environment Sector  */
-#define CFG_ENV_SIZE   0x1
+#define CFG_ENV_SIZE   0x8000
+
 #undef CONFIG_NET_MULTI
 
 #define MEM_SIZE 128

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] mpc83xx: Update DIMM data bus width test to support 40-bit width

2008-04-10 Thread Lee Nipper
32-bit wide ECC memory modules report 40-bit width.
Changed the DIMM data bus width test to 'less than 64' instead of 'equal 32'.

Signed-off-by: Lee Nipper <[EMAIL PROTECTED]>
---
 cpu/mpc83xx/spd_sdram.c |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpu/mpc83xx/spd_sdram.c b/cpu/mpc83xx/spd_sdram.c
index 0acca47..4ad5a3f 100644
--- a/cpu/mpc83xx/spd_sdram.c
+++ b/cpu/mpc83xx/spd_sdram.c
@@ -598,7 +598,7 @@ long int spd_sdram()
debug("DDR:timing_cfg_2=0x%08x\n", ddr->timing_cfg_2);
 
/* Check DIMM data bus width */
-   if (spd.dataw_lsb == 0x20) {
+   if (spd.dataw_lsb < 64) {
if (spd.mem_type == SPD_MEMTYPE_DDR)
burstlen = 0x03; /* 32 bit data bus, burst len is 8 */
else
@@ -760,7 +760,7 @@ long int spd_sdram()
sdram_cfg |= SDRAM_CFG_RD_EN;
 
/* The DIMM is 32bit width */
-   if (spd.dataw_lsb == 0x20) {
+   if (spd.dataw_lsb < 64) {
if (spd.mem_type == SPD_MEMTYPE_DDR)
sdram_cfg |= SDRAM_CFG_32_BE | SDRAM_CFG_8_BE;
if (spd.mem_type == SPD_MEMTYPE_DDR2)
-- 
1.5.4.2



-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Ben Warren
Andre Schwarz wrote:
> Ben,
>
> I'm a little confused right now and need to have a closer look.
>
> The EEPRM is a M24C64 from ST with extended adressing modes. There 
> might be something going wrong during probe or initial adressing.
>
>
If it's an extended addressing eeprom, you probably need to do 16-bit 
reads, e.g.

i2c md 50.2 0 10

If that doesn't work, it's quite possible that something else has hosed 
the bus.

regards,
Ben

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Martin Krause
Hi Andre,

Andre Schwarz wrote on Thursday, April 10, 2008 2:14 PM:
>> The number of address bytes a device needs is varying. Your could
>> look up the correct address length in the datasheet of your device,
>> or try it manually:
>> 
>> imd 50.0 0 10
>> imd 50.1 0 10
>> imd 50.2 0 10
>> 
>> One of this should work.
> 
> 
> no - only 0xff.
> Scope shows valid I2C transactions with correct data.

If you see the correct data on the bus with your scope and U-Boot 
nevertheless shows only 0xff, this seems like a bug in U-Boot to mee.
If you set the address lenght (.x spezifier) wrong, then the data
you see on the bus will also be wrong.

>> One reason for a hanging bus could be a lost clock pulse. This could
>> happen, if the low->high rise time of the bus signal is longer than
>> the clock pulse width. For testing you could try a lower bus clock
>> (10 kHz for example).
> 
> 
> rise time is  ~200ns.

This should definitely be fast enough (for 100 kHz)

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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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] PPC440 Power Management Registers

2008-04-10 Thread Stefan Roese
Hi Eugene,

On Thursday 10 April 2008, Eugene O'Brien wrote:
> Here is an updated patch correcting only the pre-existing power
> management register definitions. (I might add that these definitions are
> not currently used in the U-Boot source tree either.)
>
> CHANGELOG:
>
> ppc440: Fix power mgt definitions for PPC440
>
> Corrected DCR addresses of PPC440 power management registers.
> All AMCC PPC440 processors conform to the same DCR address usage for
> these registers.
>
> Signed-off-by: Eugene O'Brien <[EMAIL PROTECTED]>

Unfortunately now your patch is line wrapped. And even if it wasn't line 
wrapped, it couldn't be applied using git-am (which makes life so much easier 
for me) because of the lines above and below the real patch.

I there a chance that you can create this patch by using git-format-patch? I 
know it is a lot of work for such a small patch, but once you've got all this 
git patch stuff working, you can send patches much easier in the future.

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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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] PPC440 Power Management Registers

2008-04-10 Thread Eugene O'Brien
Hello Stefan,

Here is an updated patch correcting only the pre-existing power
management register definitions. (I might add that these definitions are
not currently used in the U-Boot source tree either.)

CHANGELOG:

ppc440: Fix power mgt definitions for PPC440

Corrected DCR addresses of PPC440 power management registers.
All AMCC PPC440 processors conform to the same DCR address usage for
these registers.

Signed-off-by: Eugene O'Brien <[EMAIL PROTECTED]> 


diff --git a/include/ppc440.h b/include/ppc440.h
index 80dd332..34963c5 100644
--- a/include/ppc440.h
+++ b/include/ppc440.h
@@ -1726,17 +1726,10 @@
 #else
 #define CNTRL_DCR_BASE 0x0b0
 #endif
-#if defined(CONFIG_440GX) || \
-defined(CONFIG_440SP) || defined(CONFIG_440SPE) || \
-defined(CONFIG_460EX) || defined(CONFIG_460GT)
+
 #define cpc0_er(CNTRL_DCR_BASE+0x00)   /* CPM enable
register*/
 #define cpc0_fr(CNTRL_DCR_BASE+0x01)   /* CPM force
register*/
 #define cpc0_sr(CNTRL_DCR_BASE+0x02)   /* CPM status
register*/
-#else
-#define cpc0_sr(CNTRL_DCR_BASE+0x00)   /* CPM status
register*/
-#define cpc0_er(CNTRL_DCR_BASE+0x01)   /* CPM enable
register*/
-#define cpc0_fr(CNTRL_DCR_BASE+0x02)   /* CPM force
register*/
-#endif
 
 #define cpc0_sys0  (CNTRL_DCR_BASE+0x30)   /* System configuration
reg 0   */
 #define cpc0_sys1  (CNTRL_DCR_BASE+0x31)   /* System configuration
reg 1   */


Regards,
Eugene


-Original Message-
From: Stefan Roese [mailto:[EMAIL PROTECTED] 
Sent: April 8, 2008 7:30 AM
To: u-boot-users@lists.sourceforge.net
Cc: Eugene O'Brien
Subject: Re: [U-Boot-Users] [PATCH] PPC440 Power Management Registers

Hi Eugene,

On Monday 07 April 2008, Eugene O'Brien wrote:
> I had a look at all PPC440 processor manuals from AMCC and see that my
> patch applies to all of them. In other words the #else portion is
never
> used. Therefore I am submitting a patch that cleans up this code quite
> nicely.

Great, thanks.

> Another observation that I made is that the PPC440EPx and PPC440GPx
> require more than 32 bits to control the power management functions.
> Therefore I defined a second set of registers cpc1_er, cpc1_fr,
cpc1_sr
> for these processors. These can be used as placeholders for future
> development.

I would prefer to add those when really needed. Let's try to include
only 
defines that are used. This way the headers don't get "polluted" even
more.

> Let me know if this sounds good to you.

Could you please resend you patch without those new registers? And
please send 
it inline and add a proper Signed-off-by line as described here:

http://www.denx.de/wiki/UBoot/Patches

I suggest to use git-send-email for sending patches.

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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Andre Schwarz

Ben,

I'm a little confused right now and need to have a closer look.

The EEPRM is a M24C64 from ST with extended adressing modes. There might 
be something going wrong during probe or initial adressing.



I'll post as soon as there are more useful results ...

thanks,
Andre

Ben Warren schrieb:

Hi Andre,

On Thu, Apr 10, 2008 at 4:37 AM, Andre Schwarz
<[EMAIL PROTECTED]> wrote:
  

All,

 in my current system the I2C bus is not working properly on a MPC8343 in
 u-boot v1.3.2.

 i2c board config includes :

 #define CONFIG_HARD_I2C
 #undef CONFIG_SOFT_I2C
 #define CONFIG_FSL_I2C
 #define CONFIG_I2C_MULTI_BUS
 #define CONFIG_I2C_CMD_TREE
 #define CFG_I2C_OFFSET  0x3000
 #define CFG_I2C2_OFFSET 0x3100
 #define CFG_I2C_SPEED   10
 #define CFG_I2C_SLAVE   0x7F


 chip probing works fine.

 mvBL-M7> i2c probe
 Valid chip addresses: 30 48 50 68

 reading the Chips gives all "ff"

 mvBL-M7> i2c md 50 0 10
 : ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff


 Observing the I2C bus wires show that everything _works excellent_ :
 100kHz speed as well as all data seems ok - but u-boot shows "ff".




The fact that probing works and your scope shows bits toggling leads
me to believe that your command syntax is the problem.
Please try something like: "i2c md 50.1 0 10".  I2C syntax is a bit
goofy - you often have to specify the bus width.  I've used 1.3.2
successfully on an MPC8349 (same as yours) with and without Timur's
speed patches.

regards,
Ben
  




MATRIX VISION GmbH, Talstraße 16, DE-71570 Oppenweiler  - Registergericht: 
Amtsgericht Stuttgart, HRB 271090
Geschäftsführer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Andre Schwarz

Wolfgang,

I have indeed a LM75 on this bus - but it's not adressed.
The same I2C bus works fine on linux-2.6.25 including LM75 and EEPROM.

The oszi shows complete transactions with valid data.
I think it is as software issue.

Thanks,
Andre

[EMAIL PROTECTED] schrieb:

Hi,

On 10 Apr 2008 at 13:08, Martin Krause wrote:
  

After some tries (i2c md ..) the bus hangs and no more transactions
can 
be seen on the bus.
  

One reason for a hanging bus could be a lost clock pulse. This could
happen, if the low->high rise time of the bus signal is longer than
the clock pulse width. For testing you could try a lower bus clock 
(10 kHz for example).



sorry if I did not follow the discussion up to here. As I stumbled over
it yesterday, I just want to give another example how to hang a bus.

There are devices (namely LM75) that claim to be I2C devices but
do not care about I2C specification. In case of LM75, a read must
always be 16 bit (2 bytes), in case of reading only 1 byte the device
does not interpret the missing ACK correctly and, in case the
last byte read is '0', it will block the bus until some more (worst
case 8) SCL pulses follow.

Maybe something like this could also happen in your case?

Regards,
Wolfgang

  




MATRIX VISION GmbH, Talstraße 16, DE-71570 Oppenweiler  - Registergericht: 
Amtsgericht Stuttgart, HRB 271090
Geschäftsführer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Andre Schwarz

Martin,

thanks for your hints.


Martin Krause schrieb:

Hi Andre,

[EMAIL PROTECTED] wrote on :
  

All,

in my current system the I2C bus is not working properly on a MPC8343
in 
u-boot v1.3.2.


i2c board config includes :

#define CONFIG_HARD_I2C
#undef CONFIG_SOFT_I2C
#define CONFIG_FSL_I2C
#define CONFIG_I2C_MULTI_BUS
#define CONFIG_I2C_CMD_TREE
#define CFG_I2C_OFFSET  0x3000
#define CFG_I2C2_OFFSET 0x3100
#define CFG_I2C_SPEED   10
#define CFG_I2C_SLAVE   0x7F


chip probing works fine.

mvBL-M7> i2c probe
Valid chip addresses: 30 48 50 68

reading the Chips gives all "ff"

mvBL-M7> i2c md 50 0 10



Uh, it seems I lag behind in U-Boot evolution. I know this
"i2c md" command as "imd"?

  
: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff   
 



Devices with address 50 normally are EEPROMs. If this device is an
EEPROM, are you sure it contains data other than 0xff?

  

Yes - it's the configuration data of the CPU.
I can see the transaction on the bus - all data is correct. U-boot shows 
0xff.

The number of address bytes a device needs is varying. Your could
look up the correct address length in the datasheet of your device,
or try it manually:

imd 50.0 0 10
imd 50.1 0 10
imd 50.2 0 10

One of this should work.

  

no - only 0xff.
Scope shows valid I2C transactions with correct data.

Observing the I2C bus wires show that everything _works excellent_ :
100kHz speed as well as all data seems ok - but u-boot shows "ff".

BTW:  Fetching HRCW from I2C is also working fine.

After some tries (i2c md ..) the bus hangs and no more transactions
can 
be seen on the bus.



One reason for a hanging bus could be a lost clock pulse. This could
happen, if the low->high rise time of the bus signal is longer than
the clock pulse width. For testing you could try a lower bus clock 
(10 kHz for example).


  

rise time is  ~200ns.

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
  




MATRIX VISION GmbH, Talstraße 16, DE-71570 Oppenweiler  - Registergericht: 
Amtsgericht Stuttgart, HRB 271090
Geschäftsführer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm

2008-04-10 Thread Magnus Lilja
>  I would expect, that the state of the current repository represents
>  what we get after applying Sascha's and Guennadi's patches (in the
>  correct versions and order). Howver, the current tree looks totally
>  different to me.

There are some vital differences between the current git trees (both
the ARM tree and the main tree) and the tree one gets after applying
the patches manually (from the posts here on the list).

I had to make some changes in order to get the litekit to work with
the git source, it seems like the phycore is also affected but I don't
have such a board to test on.

The following changes to the current git will put the files into the
same state as in the patches posted on the list.

At the moment I'm using gmail to post this so the diff below may be
garbled, I can re-post tonight using another mailer if necessary.

Regards, Magnus Lilja

 board/imx31_litekit/Makefile|2 ++
 board/imx31_litekit/lowlevel_init.S |5 +
 board/imx31_phycore/Makefile|2 ++
 board/imx31_phycore/lowlevel_init.S |3 +++
 4 files changed, 12 insertions(+), 0 deletions(-)

diff --git a/board/imx31_litekit/Makefile b/board/imx31_litekit/Makefile
index aaaec69..76ed1ad 100644
--- a/board/imx31_litekit/Makefile
+++ b/board/imx31_litekit/Makefile
@@ -48,3 +48,5 @@ distclean:clean
 include $(SRCTREE)/rules.mk

 sinclude $(obj).depend
+
+#
diff --git a/board/imx31_litekit/lowlevel_init.S
b/board/imx31_litekit/lowlevel_init.S
index 74d6067..c778e88 100644
--- a/board/imx31_litekit/lowlevel_init.S
+++ b/board/imx31_litekit/lowlevel_init.S
@@ -101,3 +101,8 @@ lowlevel_init:
REG 0xB8001000, 0xb210
REG80x8033, 0xda
REG80x8100, 0xff
+   REG 0xB8001000, 0x82226080
+   REG 0x8000, 0xDEADBEEF
+   REG 0xB8001010, 0x000c
+
+   mov pc, lr
diff --git a/board/imx31_phycore/Makefile b/board/imx31_phycore/Makefile
index de37cca..f6c248d 100644
--- a/board/imx31_phycore/Makefile
+++ b/board/imx31_phycore/Makefile
@@ -47,3 +47,5 @@ distclean:clean
 include $(SRCTREE)/rules.mk

 sinclude $(obj).depend
+
+#
diff --git a/board/imx31_phycore/lowlevel_init.S
b/board/imx31_phycore/lowlevel_init.S
index b0a5389..61a500e 100644
--- a/board/imx31_phycore/lowlevel_init.S
+++ b/board/imx31_phycore/lowlevel_init.S
@@ -103,3 +103,6 @@ lowlevel_init:
REG80x8100, 0xff
REG 0xB8001000, 0x82226080
REG 0x8000, 0xDEADBEEF
+   REG 0xB8001010, 0x000c
+
+   mov pc, lr

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Ben Warren
Hi Andre,

On Thu, Apr 10, 2008 at 4:37 AM, Andre Schwarz
<[EMAIL PROTECTED]> wrote:
> All,
>
>  in my current system the I2C bus is not working properly on a MPC8343 in
>  u-boot v1.3.2.
>
>  i2c board config includes :
>
>  #define CONFIG_HARD_I2C
>  #undef CONFIG_SOFT_I2C
>  #define CONFIG_FSL_I2C
>  #define CONFIG_I2C_MULTI_BUS
>  #define CONFIG_I2C_CMD_TREE
>  #define CFG_I2C_OFFSET  0x3000
>  #define CFG_I2C2_OFFSET 0x3100
>  #define CFG_I2C_SPEED   10
>  #define CFG_I2C_SLAVE   0x7F
>
>
>  chip probing works fine.
>
>  mvBL-M7> i2c probe
>  Valid chip addresses: 30 48 50 68
>
>  reading the Chips gives all "ff"
>
>  mvBL-M7> i2c md 50 0 10
>  : ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff
>
>
>  Observing the I2C bus wires show that everything _works excellent_ :
>  100kHz speed as well as all data seems ok - but u-boot shows "ff".
>

The fact that probing works and your scope shows bits toggling leads
me to believe that your command syntax is the problem.
Please try something like: "i2c md 50.1 0 10".  I2C syntax is a bit
goofy - you often have to specify the bus width.  I've used 1.3.2
successfully on an MPC8349 (same as yours) with and without Timur's
speed patches.

regards,
Ben

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
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/review] Blackfin: resurrect BF533-STAMP video splash driver

2008-04-10 Thread Jean-Christophe PLAGNIOL-VILLARD
On 02:33 Wed 09 Apr , Mike Frysinger wrote:
> This video driver used to live in the Blackfin cpu directory, but it was
> lost during the unification process.  This brings it back.
> 
> Signed-off-by: Mike Frysinger <[EMAIL PROTECTED]>
> ---
>  board/bf533-stamp/Makefile |4 +-
>  board/bf533-stamp/video.c  |  276 
> 
>  board/bf533-stamp/video.h  |   25 
>  3 files changed, 303 insertions(+), 2 deletions(-)
>  create mode 100644 board/bf533-stamp/video.c
>  create mode 100644 board/bf533-stamp/video.h
> 
> diff --git a/board/bf533-stamp/Makefile b/board/bf533-stamp/Makefile
> index 1115df8..a759d9a 100644
> --- a/board/bf533-stamp/Makefile
> +++ b/board/bf533-stamp/Makefile
> @@ -1,7 +1,7 @@
>  #
>  # U-boot - Makefile
>  #
> -# Copyright (c) 2005-2007 Analog Device Inc.
> +# Copyright (c) 2005-2008 Analog Device Inc.
>  #
>  # (C) Copyright 2000-2006
>  # Wolfgang Denk, DENX Software Engineering, [EMAIL PROTECTED]
> @@ -29,7 +29,7 @@ include $(TOPDIR)/config.mk
>  
>  LIB  = $(obj)lib$(BOARD).a
>  
> -COBJS:= $(BOARD).o spi_flash.o
> +COBJS:= $(BOARD).o spi_flash.o video.o
>  
>  SRCS := $(SOBJS:.o=.S) $(COBJS:.o=.c)
>  OBJS := $(addprefix $(obj),$(COBJS))
> diff --git a/board/bf533-stamp/video.c b/board/bf533-stamp/video.c
> new file mode 100644
> index 000..6899930
> --- /dev/null
> +++ b/board/bf533-stamp/video.c
> @@ -0,0 +1,276 @@
> +/*
> + * (C) Copyright 2000
> + * Paolo Scaffardi, AIRVENT SAM s.p.a - RIMINI(ITALY), [EMAIL PROTECTED]
> + * (C) Copyright 2002
> + * Wolfgang Denk, [EMAIL PROTECTED]
> + * (C) Copyright 2006
> + * Aubrey Li, [EMAIL PROTECTED]
> + *
> + * 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 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
> + */
> +
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +#include 
> +
> +int gunzip(void *, int, unsigned char *, unsigned long *);
> +
> +#ifdef CONFIG_VIDEO
Plese move to Makefile
> +
> +#define DMA_SIZE16   2
> + for (i = 0; i < u_boot_logo.width / 2; i++) {
> + /* enlarge one pixel to m x n */
> + for (n = 0; n < HORIZONTAL; n++) {
> + *OddPtr32++ = *data;
> + *EvenPtr32++ = *data;
> + }
> + data++;
> + }
> + }
> + }
> +}
> +
> +static int video_init(void)
> +{
> + char *NTSCFrame;
> + NTSCFrame = (char *)NTSC_FRAME_ADDR;
> + NTSC_framebuffer_init(NTSCFrame);
> + fill_frame(NTSCFrame, BLUE);
> +
> + *pPPI_CONTROL = 0x0082;
> + *pPPI_FRAME = 0x020D;
> +
> + *pDMA0_START_ADDR = NTSCFrame;
> + *pDMA0_X_COUNT = 0x035A;
> + *pDMA0_X_MODIFY = 0x0002;
> + *pDMA0_Y_COUNT = 0x020D;
> + *pDMA0_Y_MODIFY = 0x0002;
> + *pDMA0_CONFIG = 0x1015;
> + *pPPI_CONTROL = 0x0083;
> + return 0;
> +}
> +
> + error = device_register(&videodev);
> +
> + return (error == 0) ? devices : error;
> +}
> +
> +#endif
> diff --git a/board/bf533-stamp/video.h b/board/bf533-stamp/video.h
> new file mode 100644
> index 000..d5a8bc8
> --- /dev/null
> +++ b/board/bf533-stamp/video.h
> @@ -0,0 +1,25 @@
> +#include 
> +#define write_dest_byte(val) {*dest++=val;}
> +#define BLACK   (0x01800180) /* black pixel pattern   */
> +#define BLUE(0x296E29F0) /* blue pixel pattern*/
> +#define RED (0x51F0515A) /* red pixel pattern */
> +#define MAGENTA (0x6ADE6ACA) /* magenta pixel pattern */
> +#define GREEN   (0x91229136) /* green pixel pattern   */
> +#define CYAN(0xAA10AAA6) /* cyan pixel pattern*/
> +#define YELLOW  (0xD292D210) /* yellow pixel pattern  */
> +#define WHITE   (0xFE80FE80) /* white pixel pattern   */
> +
> +#define true 1
WhiteSpace ^
> +#define false0
> +
> +typedef struct {
> + unsigned int SAV;
> + unsigned int EAV;
> +} SystemCodeType;
> +
> +const SystemCodeType SystemCodeMap[4] = {
> + {0xFF80, 0xFF9D},
> + {0xFFAB, 0xFFB6},
> + {0xFFC7, 0xFFDA},
> + {0xFFEC, 0xFFF1}
> +};
> -- 
Personnaly, I do not lik

Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread w . wegner
Hi,

On 10 Apr 2008 at 13:08, Martin Krause wrote:
> > After some tries (i2c md ..) the bus hangs and no more transactions
> > can 
> > be seen on the bus.
> 
> One reason for a hanging bus could be a lost clock pulse. This could
> happen, if the low->high rise time of the bus signal is longer than
> the clock pulse width. For testing you could try a lower bus clock 
> (10 kHz for example).

sorry if I did not follow the discussion up to here. As I stumbled over
it yesterday, I just want to give another example how to hang a bus.

There are devices (namely LM75) that claim to be I2C devices but
do not care about I2C specification. In case of LM75, a read must
always be 16 bit (2 bytes), in case of reading only 1 byte the device
does not interpret the missing ACK correctly and, in case the
last byte read is '0', it will block the bus until some more (worst
case 8) SCL pulses follow.

Maybe something like this could also happen in your case?

Regards,
Wolfgang


-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Martin Krause
Hi Andre,

[EMAIL PROTECTED] wrote on :
> All,
> 
> in my current system the I2C bus is not working properly on a MPC8343
> in 
> u-boot v1.3.2.
> 
> i2c board config includes :
> 
> #define CONFIG_HARD_I2C
> #undef CONFIG_SOFT_I2C
> #define CONFIG_FSL_I2C
> #define CONFIG_I2C_MULTI_BUS
> #define CONFIG_I2C_CMD_TREE
> #define CFG_I2C_OFFSET  0x3000
> #define CFG_I2C2_OFFSET 0x3100
> #define CFG_I2C_SPEED   10
> #define CFG_I2C_SLAVE   0x7F
> 
> 
> chip probing works fine.
> 
> mvBL-M7> i2c probe
> Valid chip addresses: 30 48 50 68
> 
> reading the Chips gives all "ff"
> 
> mvBL-M7> i2c md 50 0 10

Uh, it seems I lag behind in U-Boot evolution. I know this
"i2c md" command as "imd"?

> : ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff   
>  

Devices with address 50 normally are EEPROMs. If this device is an
EEPROM, are you sure it contains data other than 0xff?

The number of address bytes a device needs is varying. Your could
look up the correct address length in the datasheet of your device,
or try it manually:

imd 50.0 0 10
imd 50.1 0 10
imd 50.2 0 10

One of this should work.

> Observing the I2C bus wires show that everything _works excellent_ :
> 100kHz speed as well as all data seems ok - but u-boot shows "ff".
> 
> BTW:  Fetching HRCW from I2C is also working fine.
> 
> After some tries (i2c md ..) the bus hangs and no more transactions
> can 
> be seen on the bus.

One reason for a hanging bus could be a lost clock pulse. This could
happen, if the low->high rise time of the bus signal is longer than
the clock pulse width. For testing you could try a lower bus clock 
(10 kHz for example).

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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] drivers MMCplus for at91sam9x

2008-04-10 Thread Pierre Savary
I have already test it but the version of the MMC driver is the same... And
it's very difficult for me to update my own version to the current.
Thanks for your help

Pierre

-Message d'origine-
De : [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] De la part de
Jean-Christophe PLAGNIOL-VILLARD
Envoyé : mercredi 9 avril 2008 19:49
À : Pierre Savary
Cc : u-boot-users@lists.sourceforge.net; 'Pierre Ossman'
Objet : Re: [U-Boot-Users] drivers MMCplus for at91sam9x

On 16:32 Tue 08 Apr , Pierre Savary wrote:
> Hi,
> I use a MMCplus 4GB on my design (with at91sam9260). It's wired with 4
bits.
> Currently U-boot (1.1.5) can't detect correctly the MMC and so I can't
read
1.1.5 is really old please update to the curent RC

Best Regards,
J.

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javao
ne
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


Re: [U-Boot-Users] [GIT PULL] Please pull u-boot-arm

2008-04-10 Thread Wolfgang Denk
Dear Peter,

in message <[EMAIL PROTECTED]> you wrote:
>
> These are the patches:
> 
> [PATCH v2] ARM: Davinci: Fix DM644x timer overflow handling and cleanup
> [PATCH] DM644x: This patch removes all boardspecific code from the arch part
> for DM644x (DaVinci) boards
> [PATCH] DM644x: (2nd try) This adds support fortheProdrivePMDRA board, based
> on a DM6441
> [PATCH 0/7] Respin of Sascha Hauer's i.MX31support plus MX31ADS
...
> Guennadi Liakhovetski (1):
>   Support for the MX31ADS evaluation board from Freescale
...
> Sascha Hauer (6):
>   Separate omap24xx specific code from arm1136
>   core support for Freescale mx31
>   add an i2c driver for mx31
>   add SMSC LAN9x1x Network driver
>   mx31 litekit support
>   Phytec Phycore-i.MX31 support

I would expect, that the state of the current repository represents
what we get after applying Sascha's and Guennadi's patches (in the
correct versions and order). Howver, the current tree looks totally
different to me.

Can you please check with Guennadi what happened, how this can be
cleaned up, and especially how this can be prevented in the future?

Note that there is not only not the expected state of the source code,
but there is also corruption of meta-information again; for example:

-> git-show --pretty=fuller 7a837b7310166ae8fc8b8d66d7ef01b60a80f9d6
commit 7a837b7310166ae8fc8b8d66d7ef01b60a80f9d6
Author: Guennadi Liakhovetski <[EMAIL PROTECTED]>
---
AuthorDate: Sun Mar 30 11:32:30 2008 +0100
Commit: Peter Pearse <[EMAIL PROTECTED]>
CommitDate: Sun Mar 30 11:32:30 2008 +0100

Support for the MX31ADS evaluation board from Freescale
...

None of Guennadis messages on the mailing list  ever  used  something
like "[EMAIL PROTECTED]" - this must be something that happend on your end
when  processing  his  patches.  Where  are the brackets coming from?
Don't you use git-am to apply the patches?

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]
If a man had a child who'd gone  anti-social,  killed  perhaps,  he'd
still tend to protect that child.
-- McCoy, "The Ultimate Computer", stardate 4731.3

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] Herbal solutions will change your life

2008-04-10 Thread Marcellais
A 9 inch organ is just months away http://www.mihugab.com/

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] I2C @ MPC8343

2008-04-10 Thread Andre Schwarz
All,

in my current system the I2C bus is not working properly on a MPC8343 in 
u-boot v1.3.2.

i2c board config includes :

#define CONFIG_HARD_I2C
#undef CONFIG_SOFT_I2C
#define CONFIG_FSL_I2C
#define CONFIG_I2C_MULTI_BUS
#define CONFIG_I2C_CMD_TREE
#define CFG_I2C_OFFSET  0x3000
#define CFG_I2C2_OFFSET 0x3100
#define CFG_I2C_SPEED   10
#define CFG_I2C_SLAVE   0x7F


chip probing works fine.

mvBL-M7> i2c probe
Valid chip addresses: 30 48 50 68

reading the Chips gives all "ff"

mvBL-M7> i2c md 50 0 10
: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff


Observing the I2C bus wires show that everything _works excellent_ :
100kHz speed as well as all data seems ok - but u-boot shows "ff".

BTW:  Fetching HRCW from I2C is also working fine.

After some tries (i2c md ..) the bus hangs and no more transactions can 
be seen on the bus.


regards,
Andre Schwarz
Matrix Vision


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

-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users


[U-Boot-Users] [PATCH] qemu-mips: add CFI support

2008-04-10 Thread Jean-Christophe PLAGNIOL-VILLARD
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <[EMAIL PROTECTED]>

diff --git a/include/configs/qemu-mips.h b/include/configs/qemu-mips.h
index e164019..7ee6fd4 100644
--- a/include/configs/qemu-mips.h
+++ b/include/configs/qemu-mips.h
@@ -33,7 +33,7 @@
 #define CONFIG_MISC_INIT_R
 
 /*IP address is default used by Qemu*/
-#define CONFIG_IPADDR  10.0.2.15/* Our IP address */
+#define CONFIG_IPADDR  10.0.2.15/* Our IP address */
 #define CONFIG_SERVERIP10.0.2.2 /* Server IP 
address*/
 
 #define CONFIG_BOOTDELAY   10  /* autoboot after 10 seconds*/
@@ -74,16 +74,12 @@
 #define CONFIG_CMD_ELF
 #define CONFIG_CMD_FAT
 #define CONFIG_CMD_EXT2
-#undef  CONFIG_CMD_IMLS
-#undef  CONFIG_CMD_FLASH
-#undef  CONFIG_CMD_LOADB
-#undef  CONFIG_CMD_LOADS
 #define CONFIG_CMD_DHCP
+#define CONFIG_CMD_PING
 
 #define CONFIG_DRIVER_NE2000
 #define CONFIG_DRIVER_NE2000_BASE  (0xb4000300)
 
-#define CFG_NO_FLASH
 #define CFG_NS16550
 #define CFG_NS16550_SERIAL
 #define CFG_NS16550_REG_SIZE1
@@ -140,17 +136,23 @@
 
 /* The following #defines are needed to get flash environment right */
 #defineCFG_MONITOR_BASETEXT_BASE
-#defineCFG_MONITOR_LEN (192 << 10)
+#defineCFG_MONITOR_LEN (192 << 11)
 
 #define CFG_INIT_SP_OFFSET 0x40
 
 /* We boot from this flash, selected with dip switch */
 #define CFG_FLASH_BASE 0xbfc0
-
-#defineCFG_ENV_IS_NOWHERE  1
-
+#defineCFG_MAX_FLASH_BANKS 1
+#defineCFG_MAX_FLASH_SECT  128
+#defineCFG_FLASH_CFI   1   /* Flash memory is CFI 
compliant */
+#define CFG_FLASH_CFI_DRIVER   1
+#define CFG_FLASH_USE_BUFFER_WRITE1
+
+#defineCFG_ENV_IS_IN_FLASH 1
+#define CFG_ENV_ADDR   (CFG_FLASH_BASE+0x6)
 /* Address and size of Primary Environment Sector  */
-#define CFG_ENV_SIZE   0x1
+#define CFG_ENV_SIZE   0x8000
+
 #undef CONFIG_NET_MULTI
 
 #define MEM_SIZE 128
-- 
1.5.4.5


-
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
___
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users