[U-Boot] PATCH][v0][85xx/P1_P2] Fix wrong CONFIG_SYS_MPC85xx_SERDES1_ADDR #define

Wolfgang Denk wd at denx.de
Thu Jan 13 07:12:09 CET 2011


Dear Kushwaha Prabhakar-B32579,

In message <071A08F2C6A57E4E94D980ECA553F874070BF5 at 039-SN1MPN1-004.039d.mgd.msft.net> you wrote:
> 
> CONFIG_SYS_MPC85xx_SERDES1_ADDR was defined wrong as CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET. It should be CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET.

Please restrict the line length of the commit message to < 70
characters.

> 
> diff --git a/arch/powerpc/include/asm/immap_85xx.h b/arch/powerpc/include/asm/immap_85xx.h
> index 81f215b..7b371e3 100644
> --- a/arch/powerpc/include/asm/immap_85xx.h
> +++ b/arch/powerpc/include/asm/immap_85xx.h
> @@ -2308,7 +2308,7 @@ typedef struct ccsr_pme {  #define CONFIG_SYS_MPC85xx_CPM_ADDR \
>  	(CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_CPM_OFFSET)  #define CONFIG_SYS_MPC85xx_SERDES1_ADDR \
> -	(CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET)
> +	(CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES1_OFFSET)
>  #define CONFIG_SYS_MPC85xx_SERDES2_ADDR \
>  	(CONFIG_SYS_IMMR + CONFIG_SYS_MPC85xx_SERDES2_OFFSET)
>  #define CONFIG_SYS_FSL_CORENET_SERDES_ADDR \

This patch looks corrupted and does not apply.

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: wd at denx.de
Microsoft Multitasking:
                     several applications can crash at the same time.


More information about the U-Boot mailing list