On Wednesday 11 November 2009 16:56:57 Mike Rapoport wrote: > On Wed, Nov 11, 2009 at 5:18 PM, Mike Frysinger wrote: > > On Wednesday 11 November 2009 03:03:01 Mike Rapoport wrote: > >> --- a/drivers/net/smc911x.h > >> +++ b/drivers/net/smc911x.h > >> @@ -480,8 +480,8 @@ static void smc911x_reset(struct eth_device *dev) > >> !(smc911x_reg_read(dev, PMT_CTRL) & > >> PMT_CTRL_READY)) udelay(10); > >> if (!timeout) { > >> - printf(DRIVERNAME > >> - ": timeout waiting for PM restore\n"); > >> + printf("%s: timeout waiting for PM restore\n", > >> + dev->name); > >> return; > >> } > >> } > > > > these changes in general look good, but if you're going to modify the > > common header, you need to update the eeprom code as well to set up the > > name field > > It seems that eeprom code is broken since commit > 736fead8fdbf8a8407048bebc373cd551d01ec98: "Convert SMC911X Ethernet > driver to CONFIG_NET_MULTI API".
broken how ? i recall it working ... > I'll try to come up with a fix, but I have no way to test it. any board that has a smc911x part can test this code easily. it doesnt require an eeprom to be hooked up to the smc911x part. you can still dump on- chip registers as well as poke the eeprorm interface. -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot