From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Date: Mon, 16 Feb 2009 14:54:06 -0600 Subject: [U-Boot] [PATCH 27/27] remove bi_enet*addr from global data for all arches In-Reply-To: <200902161549.48687.vapier@gentoo.org> References: <1234596190-524-1-git-send-email-vapier@gentoo.org> <200902161459.45065.vapier@gentoo.org> <20090216201432.C1646832E893@gemini.denx.de> <200902161549.48687.vapier@gentoo.org> Message-ID: <4999D26E.1080303@freescale.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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