[U-Boot] [PATCH 02/17] GRETH: Added support for selecting PHY address from config, PHY address was always set to zero before.

Wolfgang Denk wd at denx.de
Wed May 26 22:42:21 CEST 2010


Dear Daniel Hellstrom,

In message <1274194143-8994-1-git-send-email-daniel at gaisler.com> you wrote:
> Signed-off-by: Daniel Hellstrom <daniel at gaisler.com>
> ---
>  drivers/net/greth.c |   69 +++++++++++++++++++++++++++++++-------------------
>  1 files changed, 43 insertions(+), 26 deletions(-)

Sorry for the late review...

> +#ifdef CONFIG_SYS_GRLIB_GRETH_PHYADDR
> +	int phyaddr = CONFIG_SYS_GRLIB_GRETH_PHYADDR;
> +#else
> +	int phyaddr = 0;
> +#endif

Please make this:

#ifndef	CONFIG_SYS_GRLIB_GRETH_PHYADDR
#define CONFIG_SYS_GRLIB_GRETH_PHYADDR	0
#endif
int phyaddr = CONFIG_SYS_GRLIB_GRETH_PHYADDR;


> +#ifdef DEBUG
> +			printf("greth_init_phy: PHY read 1 failed\n");
> +#endif
...
> +#ifdef DEBUG
> +			printf("greth_init_phy: PHY read 2 failed\n");
> +#endif

Please change all these into debug() calls and drop the #ifdef's.

Also, this patch needs to go through Ben's networking repository.

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
Do you suppose the reason the ends of the `Intel Inside'  logo  don't
match up is that it was drawn on a Pentium?


More information about the U-Boot mailing list