Re: [PATCH] atl1_main: pci_module_int returns to haunt us



Alan wrote:
Perhaps Adrian would care to simply delete the pci_module_init ancient compat
code so nobody else can inadvertently merge a driver that uses it.

Signed-off-by: Alan Cox <alan@xxxxxxxxxx>


diff -u --new-file --recursive --exclude-from /usr/src/exclude linux.vanilla-2.6.20-rc6-mm3/drivers/net/atl1/atl1_main.c linux-2.6.20-rc6-mm3/drivers/net/atl1/atl1_main.c
--- linux.vanilla-2.6.20-rc6-mm3/drivers/net/atl1/atl1_main.c 2007-01-31 14:20:41.000000000 +0000
+++ linux-2.6.20-rc6-mm3/drivers/net/atl1/atl1_main.c 2007-01-31 15:25:29.083361224 +0000
@@ -2482,12 +2482,9 @@
**/
static int __init atl1_init_module(void)
{
- int ret;
printk(KERN_INFO "%s - version %s\n", atl1_driver_string, DRIVER_VERSION);
printk(KERN_INFO "%s\n", atl1_copyright);
- ret = pci_module_init(&atl1_driver);
-
- return ret;
+ return pci_register_driver(&atl1_driver);
}
module_init(atl1_init_module);

Ack. As expected, works fine. Thanks.

Jay

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: [PATCH] remove old broken Cobalt LCD driver
    ... rather than unbreaking it? ... Did some other driver replace it? ... Does nobody need the driver any ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: 2.6.0-test1-ac3 still broken on Adaptec I2O
    ... Nobody has ported this driver to 2.6 yet ... send the line "unsubscribe linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: 2.6.0-test1-ac3 still broken on Adaptec I2O
    ... Nobody has ported this driver to 2.6 yet ... send the line "unsubscribe linux-kernel" in ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: Snape-related questions about Alan Rickman
    ... That's very different from what you're arguing NOW, ... I argued nobody knew ... I don't think Alan Rickman new anything; ... There's no way to misread read, "Alan Rickman knew nothing". ...
    (alt.fan.harry-potter)
  • Re: Closed source modules will be banned from kernel?
    ... kernel break them by accident anyway so I would assume that if a binary ... driver vendor goes under the clock is ticking. ... Why is it considered OK to insult people on a mailing list, when you wouldn't to their face? ... Alan did append a :-) which automatically absolves the writer of any ill intent or presumed intent. ...
    (Fedora)