[U-Boot] [PATCH] omap3evm: Clean-up EVM detection code.

Wolfgang Denk wd at denx.de
Thu Dec 2 12:37:26 CET 2010


Dear Sanjeev Premi,

In message <1291288812-12653-1-git-send-email-premi at ti.com> you wrote:
> This patch does following changes:
>   * Change the type (u8 -> int) for omap3_evm_version.
>   * Introduce an 'undefined' board revision for init
>     value.
>   * Use of #define instead of magic numbers
> 
> Signed-off-by: Sanjeev Premi <premi at ti.com>
> ---
>  board/ti/evm/evm.c |   39 +++++++++++++++++++++++----------------
>  board/ti/evm/evm.h |   17 +++++++----------
>  2 files changed, 30 insertions(+), 26 deletions(-)
> 
> diff --git a/board/ti/evm/evm.c b/board/ti/evm/evm.c
> index 09d14f7..8d9ce73 100644
> --- a/board/ti/evm/evm.c
> +++ b/board/ti/evm/evm.c
> @@ -37,36 +37,43 @@
>  #include <asm/mach-types.h>
>  #include "evm.h"
>  
> -static u8 omap3_evm_version;
> +static int omap3_evm_version = OMAP3EVM_BOARD_UNDEF;
...
> +#define OMAP3EVM_BOARD_UNDEF	-1	/* EVM revision not detected */

Sorry, but I will not accept this patch.

The only purpose of this initialization with a non-zero value is to
paper over a problem.  As a result, the problem will be left unsolved,
so it bites us again elsewhere, and we increase the memory footprint
of the U-Boot image without need.

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: wd at denx.de
You get a wonderful view from the point of no return.
                                    - Terry Pratchett, _Making_Money_


More information about the U-Boot mailing list