[U-Boot] [PATCH 27/27] remove bi_enet*addr from global data for all arches

Wolfgang Denk wd at denx.de
Mon Feb 16 20:24:52 CET 2009


Dear Mike,

In message <200902161357.54008.vapier at gentoo.org> you wrote:
>
> > > diff --git a/include/asm-ppc/u-boot.h b/include/asm-ppc/u-boot.h
> > > index 7451905..c16f32b 100644
> > > --- a/include/asm-ppc/u-boot.h
> > > +++ b/include/asm-ppc/u-boot.h
> > > @@ -64,7 +64,6 @@ typedef struct bd_info {
> > > -	unsigned char	bi_enetaddr[6];	/* Ethernet adress */
> > > -#ifdef CONFIG_HAS_ETH1
> > > -	/* second onboard ethernet port */
> > > -	unsigned char   bi_enet1addr[6];
> > > -#endif
> > > -#ifdef CONFIG_HAS_ETH2
> > > -	/* third onboard ethernet port */
> > > -	unsigned char	bi_enet2addr[6];
> > > -#endif
> > > -#ifdef CONFIG_HAS_ETH3
> > > -	unsigned char   bi_enet3addr[6];
> > > -#endif
> > > -#ifdef CONFIG_HAS_ETH4
> > > -	unsigned char   bi_enet4addr[6];
> > > -#endif
> > > -#ifdef CONFIG_HAS_ETH5
> > > -	unsigned char   bi_enet5addr[6];
> > > -#endif
> >
> > This will break compatibility with pre-device-tree kernels.
> >
> > bd_t is a public interface, albeit a deprecated one.

Crect - and one that must be kept untouched.

> at any rate, is there a define that can be keyed off of ?  CONFIG_HAS_FDT or 
> some such junk ?  then the behavior can be changed to like:
> #ifndef CONFIG_HAS_FDT
> 	/* ABI compat junk */
> 	uchar bi_padding_was_enetaddr[6];
> #endif

No. This interface must be kept as is. 

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
Motto of the Electrical Engineer: Working computer hardware is a  lot
like an erect penis: it stays up as long as you don't fuck with it.


More information about the U-Boot mailing list