From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Mon, 10 Mar 2008 10:17:26 -0400 Subject: [U-Boot-Users] [PATCH] Add support for new AVR32 board In-Reply-To: <47D52E72.5000201@miromico.ch> References: <47D50F07.607@miromico.ch> <200803101216.47666.sr@denx.de> <47D5256F.9040803@miromico.ch> <200803101327.01970.sr@denx.de> <47D52E72.5000201@miromico.ch> Message-ID: <47D542F6.9060006@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Alex, Alex wrote: >> It is again. I suggest that you first send this message to yourself, until you >> figured out how to do it correctly. And if you are working with git, I >> strongly encourage you to use git-send-email to send patches. >> > > Now tested as you suggested: > > > diff -Nur old/u-boot-1.3.0/board/miromico/hammerhead/config.mk > new/u-boot-1.3.0/board/miromico/hammerhead/config.mk > --- old/u-boot-1.3.0/board/miromico/hammerhead/config.mk 1970-01-01 > 01:00:00.000000000 +0100 > +++ new/u-boot-1.3.0/board/miromico/hammerhead/config.mk 2008-03-10 > 11:18:50.000000000 +0100 > @@ -0,0 +1,3 @@ > +TEXT_BASE = 0x00000000 > +PLATFORM_RELFLAGS += -ffunction-sections -fdata-sections > +PLATFORM_LDFLAGS += --gc-sections > diff -Nur old/u-boot-1.3.0/board/miromico/hammerhead/eth.c > new/u-boot-1.3.0/board/miromico/hammerhead/eth.c > --- old/u-boot-1.3.0/board/miromico/hammerhead/eth.c 1970-01-01 > 01:00:00.000000000 +0100 > +++ new/u-boot-1.3.0/board/miromico/hammerhead/eth.c 2008-03-10 > 11:18:50.000000000 +0100 > @@ -0,0 +1,35 @@ > +/* > + * Copyright (C) 2008 Miromico AG > + * > + * Ethernet initialization for the AVR32 on Hammerhead > + * > + * See file CREDITS for list of people who contributed to this > + * project. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License as > + * published by the Free Software Foundation; either version 2 of > + * the License, or (at your option) any later version. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, write to the Free Software > + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, > + * MA 02111-1307 USA > + */ > +#include > + > +#include > + > +extern int macb_eth_initialize(int id, void *regs, unsigned int phy_addr); > + > +#ifdef CONFIG_CMD_NET > +void hammerhead_eth_initialize(bd_t *bi) > +{ > + macb_eth_initialize(0, (void *)MACB0_BASE, bi->bi_phy_id[0]); > +} > +#endif > Would you mind changing this name to board_eth_initialize(bd_t *bi)? I'm working on a rework of the net library and making this name change now will avoid doing it later. I think you'll be able to figure out where this is going... > diff -Nur old/u-boot-1.3.0/net/eth.c new/u-boot-1.3.0/net/eth.c > --- old/u-boot-1.3.0/net/eth.c 2008-03-06 07:56:04.000000000 +0100 > +++ new/u-boot-1.3.0/net/eth.c 2008-03-10 11:18:50.000000000 +0100 > @@ -61,6 +61,7 @@ > extern int bfin_EMAC_initialize(bd_t *); > extern int atstk1000_eth_initialize(bd_t *); > extern int atngw100_eth_initialize(bd_t *); > +extern int hammerhead_eth_initialize(bd_t *); > Change this to board_eth_initialize() and you can be the first soldier in the revolution! > extern int mcffec_initialize(bd_t*); > > static struct eth_device *eth_devices, *eth_current; > @@ -258,6 +259,9 @@ > #if defined(CONFIG_ATNGW100) > atngw100_eth_initialize(bis); > #endif > +#if defined(CONFIG_HAMMERHEAD) > + hammerhead_eth_initialize(bis); > +#endif > #if defined(CONFIG_MCFFEC) > mcffec_initialize(bis); > #endif > @@ -522,6 +526,7 @@ > extern int mcf52x2_miiphy_initialize(bd_t *bis); > extern int ns7520_miiphy_initialize(bd_t *bis); > extern int dm644x_eth_miiphy_initialize(bd_t *bis); > +extern int hammerhead_eth_initialize(bd_t *); > This isn't needed twice (They're probably #ifdef'd, but you don't need to #ifdef extern'd prototypes) > > int eth_initialize(bd_t *bis) > @@ -546,6 +551,9 @@ > #if defined(CONFIG_DRIVER_TI_EMAC) > dm644x_eth_miiphy_initialize(bis); > #endif > +#if defined(CONFIG_HAMMERHEAD) > + hammerhead_eth_initialize(bis); > +#endif > return 0; > } > #endif > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > U-Boot-Users mailing list > U-Boot-Users at lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/u-boot-users > > regards, Ben