[U-Boot] [PATCH v2] powerpc: mpc5xxx: remove redundant CONFIG_MPC5xxx definition

Gerhard Sittig gsi at denx.de
Wed Jan 15 17:22:41 CET 2014


On Tue, Jan 14, 2014 at 17:26 +0900, Masahiro Yamada wrote:
> 
> diff --git a/include/configs/BC3450.h b/include/configs/BC3450.h
> index 377db7b..ad69d33 100644
> --- a/include/configs/BC3450.h
> +++ b/include/configs/BC3450.h
> @@ -22,7 +22,6 @@
>  /*
>   * High Level Configuration Options
>   */
> -#define CONFIG_MPC5xxx		1	/* This is an MPC5xxx CPU	    */
>  #define CONFIG_MPC5200		1	/* (more precisely a MPC5200 CPU)   */
>  #define CONFIG_TQM5200		1	/* ... on a TQM5200 module	    */
>  

Haven't checked all of the patch, just by coincidence spotted
this one.  The removal of one of the preprocessor lines loses
information in the right hand side comment.  After the patch the
remaining text looks odd.


virtually yours
Gerhard Sittig
-- 
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: office at denx.de


More information about the U-Boot mailing list