[U-Boot] [PATCH 3/4] ELPPC: Don't write magic numbers via short.

Wolfgang Denk wd at denx.de
Tue May 14 00:40:01 CEST 2013


Dear Tom Rini,

In message <1368477117-32669-3-git-send-email-trini at ti.com> you wrote:
> We need to write each segment of the buffer directly rather than using
> shorts.
> 
> Cc: Frank Gottschling <fgottschling at eltec.de>
> Signed-off-by: Tom Rini <trini at ti.com>
> ---
>  board/eltec/elppc/misc.c |    9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/board/eltec/elppc/misc.c b/board/eltec/elppc/misc.c
> index 89f1b1d..a7d8486 100644
> --- a/board/eltec/elppc/misc.c
> +++ b/board/eltec/elppc/misc.c
> @@ -207,9 +207,12 @@ int misc_init_r (void)
>  		buf[4] = eerev.etheraddr[5];
>  		buf[5] = eerev.etheraddr[4];
>  
> -		*(unsigned short *) &buf[20] = 0x48B2;
> -		*(unsigned short *) &buf[22] = 0x0004;
> -		*(unsigned short *) &buf[24] = 0x1433;
> +		buf[20] = 0x48;
> +		buf[21] = 0xB2;
> +		buf[22] = 0x00;
> +		buf[23] = 0x04;
> +		buf[24] = 0x14;
> +		buf[25] = 0x33;

I guess the 16 bit numbers originally written had some meaning, so
there should be at least a comment explaining the code.

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
My challenge to the goto-less programmer  is  to  recode  tcp_input()
without any gotos ... without any loss of efficiency (there has to be
a catch).                                             - W. R. Stevens


More information about the U-Boot mailing list