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

Scott Wood scottwood at freescale.com
Mon Feb 16 21:54:06 CET 2009


Mike Frysinger wrote:
> On Monday 16 February 2009 15:14:32 Wolfgang Denk wrote:
>> In message Mike Frysinger wrote:
>>>> As mentioned before, bd_info is a binary interfwace between U-Boot and
>>>> the Linux kernel, and it must not be changed.
>>> i'm not talking about changing the binary aspect of it.  i'm talking
>>> about changing the API.  two completely different things.
>> Then I don't understand what you are talking about or what your patch
>> is doing. To me it looks as if you were removing the bi_enetaddr[]
>> resp. bi_enet?addr[] from the bd_info structure.
> 
> the discussion has moved on.  the original patch removed the fields, but in 
> the follow ups it was proposed simply renaming them.  please review the 
> context of each reply.

What do we gain from renaming them?  We still need to fill in the MAC 
addresses for compatibility.  If you want to discourage new internal 
users, use a comment (preferably a giant do-not-touch sign over the 
entire bd_t).

-Scott


More information about the U-Boot mailing list