[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