netdev
[Top] [All Lists]

Re: [BK PATCHES] 2.6.x experimental net driver queue

To: Jeff Garzik <jgarzik@xxxxxxxxx>, Sam Creasey <sammy@xxxxxxxxx>
Subject: Re: [BK PATCHES] 2.6.x experimental net driver queue
From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
Date: Tue, 18 Nov 2003 13:27:52 +0100 (MET)
Cc: netdev@xxxxxxxxxxx, Linux Kernel <linux-kernel@xxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx>, Linux/m68k <linux-m68k@xxxxxxxxxxxxxxxxxxxx>
In-reply-to: <Pine.GSO.4.21.0311171812000.5421-100000@xxxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Mon, 17 Nov 2003, Geert Uytterhoeven wrote:
> On Sun, 16 Nov 2003, Jeff Garzik wrote:
> > Yet more updates.  Syncing with Andrew Morton, and more syncing with Al 
> > Viro.
> > 
> > No users of init_etherdev remain in the tree.  (yay!)
> 
> Here are some (untested, except for cross-gcc) fixes for the m68k-related
> drivers:

I forget to test the Sun-3 drivers:
  - sun3_82586.c:
      o add missing casts to iounmap() calls
      o fix parameter of free_netdev()
  - sun3lance.c: add missing casts to iounmap() calls
      
Note that sun3_82586.c no longer compiles since SUN3_82586_TOTAL_SIZE is not
defined. Sammy, is it OK to use PAGE_SIZE for that, since that's what's passed
to ioremap()?

--- linux-net2-2.6.0-test9/drivers/net/sun3_82586.c     2003-11-17 
11:06:18.000000000 +0100
+++ linux-net2-2.6.0-test9/drivers/net/sun3_82586.c     2003-11-18 
13:03:01.000000000 +0100
@@ -327,7 +327,7 @@
 out1:
        free_netdev(dev);
 out:
-       iounmap(ioaddr);
+       iounmap((void *)ioaddr);
        return ERR_PTR(err);
 }
 
@@ -1163,8 +1163,8 @@
        unsigned long ioaddr = dev_sun3_82586->base_addr;
        unregister_netdev(dev_sun3_82586);
        release_region(ioaddr, SUN3_82586_TOTAL_SIZE);
-       iounmap(ioaddr);
-       free_netdev(dev);
+       iounmap((void *)ioaddr);
+       free_netdev(dev_sun3_82586);
 }
 #endif /* MODULE */
 
--- linux-net2-2.6.0-test9/drivers/net/sun3lance.c      2003-11-17 
11:06:18.000000000 +0100
+++ linux-net2-2.6.0-test9/drivers/net/sun3lance.c      2003-11-18 
13:02:32.000000000 +0100
@@ -287,7 +287,7 @@
 
 out1:
 #ifdef CONFIG_SUN3
-       iounmap(dev->base_addr);
+       iounmap((void *)dev->base_addr);
 #endif
 out:
        free_netdev(dev);
@@ -327,7 +327,7 @@
                ioaddr_probe[1] = tmp2;
 
 #ifdef CONFIG_SUN3
-               iounmap(ioaddr);
+               iounmap((void *)ioaddr);
 #endif
                return 0;
        }
@@ -957,7 +957,7 @@
 {
        unregister_netdev(sun3lance_dev);
 #ifdef CONFIG_SUN3
-       iounmap(sun3lance_dev->base_addr);
+       iounmap((void *)sun3lance_dev->base_addr);
 #endif
        free_netdev(sun3lance_dev);
 }

Gr{oetje,eeting}s,

                                                Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                                            -- Linus Torvalds


<Prev in Thread] Current Thread [Next in Thread>