[U-Boot-Users] broken changes @ MPC83xx

[U-Boot-Users] broken changes @ MPC83xx

[Andre Schwarz]

Kim,

after being back from holiday and pulling the lates u-boot today
(v1.3.3) my MPC8343 based board (mvBL-M7) doesn't work any longer.
Not a single character is put out on serial - yet build is not
complaining about anything and System.map looks quite normal.

Any ideas what commit can cause this ?

Am I missing any #defines introduced lately ?


regards,
Andre

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

[U-Boot-Users] broken changes @ MPC83xx

[Tor Krill]

Hi Andre,

I had a similar problem a while back.

Adding the new define:

#define CONFIG_HIGH_BATS	1	/* High BATs supported */

To my config solved the problem.

/Tor

On 6/23/2008, "Andre Schwarz" <andre.schwarz@matrix-vision.de> wrote:

>Kim,
>
>after being back from holiday and pulling the lates u-boot today
>(v1.3.3) my MPC8343 based board (mvBL-M7) doesn't work any longer.
>Not a single character is put out on serial - yet build is not
>complaining about anything and System.map looks quite normal.
>
>Any ideas what commit can cause this ?
>
>Am I missing any #defines introduced lately ?
>
>
>regards,
>Andre
>
>MATRIX VISION GmbH, Talstra?e 16, DE-71570 Oppenweiler  - Registergericht: Amtsgericht Stuttgart, HRB 271090
>Gesch?ftsf?hrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
>
>-------------------------------------------------------------------------
>Check out the new SourceForge.net Marketplace.
>It's the best place to buy or sell services for
>just about anything Open Source.
>http://sourceforge.net/services/buy/index.php
>_______________________________________________
>U-Boot-Users mailing list
>U-Boot-Users at lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/u-boot-users
>

[U-Boot-Users] broken changes @ MPC83xx

[Andre Schwarz]

gotcha - that's it !

It's _really_ hard to keep things working these days ...


many thanks,
Andre

Tor Krill schrieb:
> Hi Andre,
>
> I had a similar problem a while back.
>
> Adding the new define:
>
> #define CONFIG_HIGH_BATS	1	/* High BATs supported */
>
> To my config solved the problem.
>
> /Tor
>
> On 6/23/2008, "Andre Schwarz" <andre.schwarz@matrix-vision.de> wrote:
>
>   
>> Kim,
>>
>> after being back from holiday and pulling the lates u-boot today
>> (v1.3.3) my MPC8343 based board (mvBL-M7) doesn't work any longer.
>> Not a single character is put out on serial - yet build is not
>> complaining about anything and System.map looks quite normal.
>>
>> Any ideas what commit can cause this ?
>>
>> Am I missing any #defines introduced lately ?
>>
>>
>> regards,
>> Andre
>>
>> MATRIX VISION GmbH, Talstra??e 16, DE-71570 Oppenweiler  - Registergericht: Amtsgericht Stuttgart, HRB 271090
>> Gesch??ftsf??hrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
>>
>> -------------------------------------------------------------------------
>> Check out the new SourceForge.net Marketplace.
>> It's the best place to buy or sell services for
>> just about anything Open Source.
>> http://sourceforge.net/services/buy/index.php
>> _______________________________________________
>> U-Boot-Users mailing list
>> U-Boot-Users at lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/u-boot-users
>>
>>     


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

[U-Boot-Users] [PATCH] fix non-working mvBL-M7

[Andre Schwarz]

Add missing #define CONFIG_HIGH_BATS in mvBL-M7 board config file.
Add default MAC to enable netboot during production.

Signed-off-by: Andre Schwarz <andre.schwarz@matrix-vision.de>
---





MATRIX VISION GmbH, Talstra?e 16, DE-71570 Oppenweiler  - Registergericht: Amtsgericht Stuttgart, HRB 271090
Gesch?ftsf?hrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: patch_m7
Url: http://lists.denx.de/pipermail/u-boot/attachments/20080623/a2ecd34d/attachment.txt 

[U-Boot-Users] [PATCH] fix non-working mvBL-M7

[Kim Phillips]

On Mon, 23 Jun 2008 13:25:34 +0200
Andre Schwarz <andre.schwarz@matrix-vision.de> wrote:

Hello Andre,

> diff --git a/include/configs/MVBLM7.h b/include/configs/MVBLM7.h
> index 349ca14..8c0e8b6 100644
> --- a/include/configs/MVBLM7.h
> +++ b/include/configs/MVBLM7.h
> @@ -27,7 +27,7 @@
>  #ifndef __CONFIG_H
>  #define __CONFIG_H
>  
> -#define MV_VERSION  "v1.0.1"
> +#define MV_VERSION  "v1.0.2"

instead of incrementing this for each patch, can we just use
U_BOOT_VERSION instead?

>  #define CONFIG_HAS_ETH0
> +#define CONFIG_ETHADDR          b6:b4:45:eb:fb:c0
>  #define CONFIG_TSEC1_NAME	"TSEC0"
>  #define CONFIG_FEC1_PHY_NORXERR
>  #define CFG_TSEC1_OFFSET	0x24000
> @@ -224,6 +225,7 @@
>  #define TSEC1_FLAGS		(TSEC_GIGABIT|TSEC_REDUCED)
>  
>  #define CONFIG_HAS_ETH1
> +#define CONFIG_ETH1ADDR         b6:b4:45:eb:fb:c2
>  #define CONFIG_TSEC2_NAME  	"TSEC1"

I had removed this from your original board port because this goes
against recommended practice; see:

http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/41178

> +#define CONFIG_HIGH_BATS       1

this line applied.

Kim

[U-Boot-Users] [PATCH] fix non-working mvBL-M7

[André Schwarz]

Kim,

thanks - ok.

Switching to U_BOOT_VERSION should be no problem.

regards,
Andr?

Kim Phillips schrieb:
> On Mon, 23 Jun 2008 13:25:34 +0200
> Andre Schwarz <andre.schwarz@matrix-vision.de> wrote:
> 
> Hello Andre,
> 
>> diff --git a/include/configs/MVBLM7.h b/include/configs/MVBLM7.h
>> index 349ca14..8c0e8b6 100644
>> --- a/include/configs/MVBLM7.h
>> +++ b/include/configs/MVBLM7.h
>> @@ -27,7 +27,7 @@
>>  #ifndef __CONFIG_H
>>  #define __CONFIG_H
>>  
>> -#define MV_VERSION  "v1.0.1"
>> +#define MV_VERSION  "v1.0.2"
> 
> instead of incrementing this for each patch, can we just use
> U_BOOT_VERSION instead?
> 
>>  #define CONFIG_HAS_ETH0
>> +#define CONFIG_ETHADDR          b6:b4:45:eb:fb:c0
>>  #define CONFIG_TSEC1_NAME	"TSEC0"
>>  #define CONFIG_FEC1_PHY_NORXERR
>>  #define CFG_TSEC1_OFFSET	0x24000
>> @@ -224,6 +225,7 @@
>>  #define TSEC1_FLAGS		(TSEC_GIGABIT|TSEC_REDUCED)
>>  
>>  #define CONFIG_HAS_ETH1
>> +#define CONFIG_ETH1ADDR         b6:b4:45:eb:fb:c2
>>  #define CONFIG_TSEC2_NAME  	"TSEC1"
> 
> I had removed this from your original board port because this goes
> against recommended practice; see:
> 
> http://thread.gmane.org/gmane.comp.boot-loaders.u-boot/41178
> 
>> +#define CONFIG_HIGH_BATS       1
> 
> this line applied.
> 
> Kim


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