From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Warren Date: Mon, 19 Oct 2009 09:41:22 -0700 Subject: [U-Boot] [PATCH] qemu_mips: Fix CONFIG_NET_MULTI build warning In-Reply-To: <4ADC63E7.4000901@pobox.com> References: <4ADC63E7.4000901@pobox.com> Message-ID: <4ADC96B2.9030305@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 Shinya, Shinya Kuribayashi wrote: > eth.c:497:2: warning: #warning Ethernet driver is deprecated. Please update to use CONFIG_NET_MULTI > > Signed-off-by: Shinya Kuribayashi > --- > > I have a few concerns about this fix: > > First. I'm not sure why CONFIG_NET_MULTI is "undef"ed for qemu_mips, > while CONFIG_DRIVER_NE2000 has been enabled for qemu_mips at an early > stage. > > I don't follow recent changes around eth.c and CONFIG_NET_MULTI, but > it's probably CONFIG_NET_MULTI used to be used strictly for multi > ports, isn't it? > > Currently, there are two incompatible networking APIs. One that uses CONFIG_NET_MULTI, and one that doesn't. I'm trying to move everything towards the former (single-port applications are of course a degenerate instance of multi-port ones). Once all drivers have been ported to the MULTI API, that config option will magically disappear. > Next. As far as looking at drivers/net/ne2000*.[ch], NE2000 driver > doesn't seem to require board_eth_init() or cpu_eth_init(). Right? > Therefore I've not added a corresponding hook in board/qemu_mips/ > qemu_mips.c. If my understanding is wrong, please let me know. > > The NE2000 driver hasn't been ported yet. It's on my short term to-do list, and will be in the next release (01.2010, I guess?) > include/configs/qemu-mips.h | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/include/configs/qemu-mips.h b/include/configs/qemu-mips.h > index cbacdf9..f419174 100644 > --- a/include/configs/qemu-mips.h > +++ b/include/configs/qemu-mips.h > @@ -157,7 +157,7 @@ > > #define CONFIG_ENV_OVERWRITE 1 > > -#undef CONFIG_NET_MULTI > +#define CONFIG_NET_MULTI > > This won't do anything other than disabling networking. Since QEMU is an emulator, though, maybe it would make sense to use a device driver that has CONFIG_NET_MULTI support? > #define MEM_SIZE 128 > > If the warning isn't more than a nuisance, please live with it for now. regards, Ben