Re: [PATCH][TRIVIAL] de2104x.c missing __devexit_p in 2.5.43

From: Adam J. Richter (adam@yggdrasil.com)
Date: Thu Oct 17 2002 - 12:14:03 EST


Jeff Garzik wrote:
>Andrey Panin wrote:
>> diff -urN -X /usr/share/dontdiff linux-vanilla/drivers/net/tulip/de2104x.c \
>> linux/drivers/net/tulip/de2104x.c
>> --- linux-vanilla/drivers/net/tulip/de2104x.c Sun Sep 1 02:04:53 2002
>> +++ linux/drivers/net/tulip/de2104x.c Thu Oct 17 04:10:19 2002
>> @@ -2216,7 +2216,7 @@
>> .name = DRV_NAME,
>> .id_table = de_pci_tbl,
>> .probe = de_init_one,
>> - .remove = de_remove_one,
>> + .remove = __devexit_p(de_remove_one),
>> #ifdef CONFIG_PM
>> .suspend = de_suspend,
>> .resume = de_resume,
>
>
>alas, it is incorrect, as no one hotplugs this hardware.

        I believe that there are motherboards that use a chipset from
Compaq that allows hot plugging and unplugging of ordinary PCI cards,
supported by drivers in linux-2.5.43/drivers/hotplug/cpq*.[ch]. At a
trade show, I saw a demo of a motherboard with such a capability (not
running Linux, but I think from Compaq).

        So, I believe that all ordinary PCI cards (as opposed to
devices soldered onto motherboards, for example) are now hot plug
capable, although their Linux drivers may not yet be.

        Do I misunderstand the situation?

        As a side note, I also either do not agree or somehow
misunderstand Jeff Garzik's opposition to devexit_p in non-hotplug
drivers, but that issue will irrelevant in the case of de2104.c if it
is indeed possible for all ordinary PCI form factor cards to be
deployed with the hot plug motherboard chipset that I described.

Adam J. Richter __ ______________ 575 Oroville Road
adam@yggdrasil.com \ / Milpitas, California 95035
+1 408 309-6081 | g g d r a s i l United States of America
                         "Free Software For The Rest Of Us."
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Oct 23 2002 - 22:00:36 EST