2004-10-23 23:14:41

by Seiichi Nakashima

[permalink] [raw]
Subject: linux-2.6.9 eepro100 warning

Hi.

I update linux kernel form 2.6.8.1 to 2.6.9.
I found eepro100 warning, then I update 2.6.10-rc1, but same result.
It is only warning, I think pro100 may work fine.

==

drivers/net/eepro100.c: In function `wait_for_cmd_done':
drivers/net/eepro100.c:564: warning: passing arg 1 of `readb' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_found1':
drivers/net/eepro100.c:749: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:750: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c:816: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:840: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:841: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `do_slow_command':
drivers/net/eepro100.c:919: warning: passing arg 1 of `readb' makes pointer from integer without a cast
drivers/net/eepro100.c:923: warning: passing arg 1 of `readb' makes pointer from integer without a cast
drivers/net/eepro100.c:925: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:928: warning: passing arg 1 of `readb' makes pointer from integer without a cast
drivers/net/eepro100.c:930: warning: passing arg 1 of `readb' makes pointer from integer without a cast
drivers/net/eepro100.c:934: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `mdio_read':
drivers/net/eepro100.c:982: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:984: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `mdio_write':
drivers/net/eepro100.c:998: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1000: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_open':
drivers/net/eepro100.c:1056: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c:1075: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_resume':
drivers/net/eepro100.c:1106: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1110: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1111: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c:1119: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1120: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c:1122: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:1132: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1133: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c:1158: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1161: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_rx_soft_reset':
drivers/net/eepro100.c:1190: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:1201: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1202: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_timer':
drivers/net/eepro100.c:1234: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_tx_timeout':
drivers/net/eepro100.c:1403: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c:1409: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c:1431: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1436: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_start_xmit':
drivers/net/eepro100.c:1504: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:1514: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_interrupt':
drivers/net/eepro100.c:1614: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c:1618: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c:1678: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c:1685: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_close':
drivers/net/eepro100.c:1913: warning: passing arg 1 of `readw' makes pointer from integer without a cast
drivers/net/eepro100.c:1918: warning: passing arg 2 of `writew' makes pointer from integer without a cast
drivers/net/eepro100.c:1921: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c:1922: warning: passing arg 1 of `readl' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `speedo_get_stats':
drivers/net/eepro100.c:2006: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `set_rx_mode':
drivers/net/eepro100.c:2200: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:2238: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c:2314: warning: passing arg 2 of `writeb' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `eepro100_suspend':
drivers/net/eepro100.c:2345: warning: passing arg 2 of `writel' makes pointer from integer without a cast
drivers/net/eepro100.c: In function `eepro100_resume':
drivers/net/eepro100.c:2369: warning: passing arg 2 of `writew' makes pointer from integer without a cast
Root device is (3, 3)
Boot sector 512 bytes.
Setup is 4735 bytes.
System is 1026 kB

--------------------------------
Seiichi Nakashima
Email [email protected]
--------------------------------


2004-10-25 06:17:16

by Jeff Garzik

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Seiichi Nakashima wrote:
> Hi.
>
> I update linux kernel form 2.6.8.1 to 2.6.9.
> I found eepro100 warning, then I update 2.6.10-rc1, but same result.
> It is only warning, I think pro100 may work fine.

This should be fixed in the 2.6.9-mm tree (via my netdev-2.6 tree).

Note that eepro100 driver will be deleted soon.

Jeff



2004-10-25 06:26:00

by Nick Piggin

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Jeff Garzik wrote:
> Seiichi Nakashima wrote:
>
>> Hi.
>>
>> I update linux kernel form 2.6.8.1 to 2.6.9.
>> I found eepro100 warning, then I update 2.6.10-rc1, but same result.
>> It is only warning, I think pro100 may work fine.
>
>
> This should be fixed in the 2.6.9-mm tree (via my netdev-2.6 tree).
>
> Note that eepro100 driver will be deleted soon.
>

By soon, do you mean in 5 years?
Or by deleted, do you mean deprecated?

Ignore me, I'm just being a smart arse ;)

2004-10-25 06:29:36

by YOSHIFUJI Hideaki

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

In article <[email protected]> (at Mon, 25 Oct 2004 02:16:46 -0400), Jeff Garzik <[email protected]> says:

> Note that eepro100 driver will be deleted soon.

I'm afraid that e100 does not work well on some systems (e.g. my ThinkPad
T30 on 2.4.x) while eepro100 does.
(I also know there're systems which does not work well
with eepro100 but e100.)

Has all of such kind of issues been solved?

--yoshfuji

2004-10-25 19:57:36

by Terje Kvernes

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Jeff Garzik <[email protected]> writes:

[ ... ]

> Note that eepro100 driver will be deleted soon.

we have several systems where e100 produces netdev watchdog errors,
while eepro100 works without problems. I can understand the desire
to migrate, but what timeframe are we looking at? roughly
translated, when do I _have_ to start help debugging the e100
driver? ;-)

I can do another attempt at rolling out the e100 driver, see what
breaks, and try to report it around 2.6.1[01] or something like that
if that helps?


--
Terje

2004-10-28 04:38:37

by Jeff Garzik

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Terje Kvernes wrote:
> Jeff Garzik <[email protected]> writes:
>
> [ ... ]
>
>
>>Note that eepro100 driver will be deleted soon.
>
>
> we have several systems where e100 produces netdev watchdog errors,
> while eepro100 works without problems. I can understand the desire
> to migrate, but what timeframe are we looking at? roughly
> translated, when do I _have_ to start help debugging the e100
> driver? ;-)
>
> I can do another attempt at rolling out the e100 driver, see what
> breaks, and try to report it around 2.6.1[01] or something like that
> if that helps?


If there are e100 problems, report them to the maintainers so we can get
them resolved ASAP...

INTEL PRO/100 ETHERNET SUPPORT
P: John Ronciak
M: [email protected]
P: Ganesh Venkatesan
M: [email protected]
P: Scott Feldman
M: [email protected]
W: http://sourceforge.net/projects/e1000/
S: Supported

(and of course [email protected] as well)

2004-10-28 15:52:35

by Terje Kvernes

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Jeff Garzik <[email protected]> writes:

[ ... ]

> If there are e100 problems, report them to the maintainers so we can
> get them resolved ASAP...

I got a very nice mail from Jesse Brandeburg saying pretty much the
same thing... so, I've pushed kernels for around 75 boxen over to
e100 today, from eepro100. I'm not _quite_ sure when I'll have the
chance to boot them all, but when they do, and if they have
problems, I'll report back.

> INTEL PRO/100 ETHERNET SUPPORT
> P: John Ronciak
> M: [email protected]
> P: Ganesh Venkatesan
> M: [email protected]
> P: Scott Feldman
> M: [email protected]
> W: http://sourceforge.net/projects/e1000/
> S: Supported
>
> (and of course [email protected] as well)

ack.

--
Terje - still waiting to test hotswap SATA. =)

2004-10-30 05:40:48

by Seiichi Nakashima

[permalink] [raw]
Subject: Re: linux-2.6.9 eepro100 warning

Thanks.

I update a kernel's parameter from eepro100 to PRO/100.
linux-2.6.9 work fine.

==

>
> [ ... ]
>
>> If there are e100 problems, report them to the maintainers so we can
>> get them resolved ASAP...
>
> I got a very nice mail from Jesse Brandeburg saying pretty much the
> same thing... so, I've pushed kernels for around 75 boxen over to
> e100 today, from eepro100. I'm not _quite_ sure when I'll have the
> chance to boot them all, but when they do, and if they have
> problems, I'll report back.
>
>> INTEL PRO/100 ETHERNET SUPPORT
>> P: John Ronciak
>> M: [email protected]
>> P: Ganesh Venkatesan
>> M: [email protected]
>> P: Scott Feldman
>> M: [email protected]
>> W: http://sourceforge.net/projects/e1000/
>> S: Supported
>>
>> (and of course [email protected] as well)
>
> ack.
>
>--
>Terje - still waiting to test hotswap SATA. =)
>
>
--------------------------------
Seiichi Nakashima
Email [email protected]
--------------------------------