Dear Mike,

In message <200902161357.54008.vap...@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: w...@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.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to