From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Sat, 22 Mar 2008 11:14:40 +0100 Subject: [U-Boot-Users] [RFC][PATCH 1/1] Add board_eth_init() function In-Reply-To: <20080322095956.1C8CA24A91@gemini.denx.de> References: <20080322095956.1C8CA24A91@gemini.denx.de> Message-ID: <200803221114.40956.sr@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Saturday 22 March 2008, Wolfgang Denk wrote: > In message <200803220731.23229.sr@denx.de> you wrote: > > I would prefer to move this init code not into the board code but into > > the platform/cpu code. Speaking for PPC4xx, here all configuration I know > > of have the same init code. And copying this into all board ports instead > > on one place in the cpu directory seems "imperfect". > > But the ethernet configuration *is* actually bord dependent. A board > may or may not use all the ethernet interfaces present on a > processor, or it may have additional network controllers attached for > example to the PCI bus, etc. So you vote for adding this code to more than 80 4xx boards that all would use the same code here? I don't like this. I would prefer one instance in the cpu directory which could be overridden if needed by a board specific version. Best regards, Stefan ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de =====================================================================