[U-Boot] [PATCH 2/5] smc911x: use dev->name in printfs
Mike Rapoport
mike.rapoport at gmail.com
Wed Nov 11 23:24:27 CET 2009
On Thu, Nov 12, 2009 at 12:11 AM, Mike Frysinger <vapier at gentoo.org> wrote:
> 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 ...
It gives pretty long list of compile errors. The smc911x.h header has
now 'struct eth_device *dev' parameter in all the functions.
>> 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.
Ok, will try.
> -mike
>
--
Sincerely Yours,
Mike.
More information about the U-Boot
mailing list