Re: [2.6 patch] drivers/pnp/: possible cleanups

From: Adrian Bunk (bunk_at_stusta.de)
Date: 03/12/05

  • Next message: Dmitry Yusupov: "Re: [ANNOUNCE 0/6] Open-iSCSI High-Performance Initiator for Linux"
    Date:	Sat, 12 Mar 2005 18:07:06 +0100
    To: Adam Belay <abelay@novell.com>
    
    

    On Fri, Mar 11, 2005 at 07:44:56PM -0500, Adam Belay wrote:
    > On Fri, 2005-03-11 at 16:23 -0800, Andrew Morton wrote:
    > > Adam Belay <abelay@novell.com> wrote:
    > > >
    > > > This patch essential makes it impossible for PnP protocols to be
    > > > modules. Currently, they are all in-kernel. If that is acceptable...,
    > > > then this patch looks fine to me. Any comments?
    > >
    > > You're the maintainer...
    >
    > I've been holding off on making many changes to PnP at the moment,
    > because I have been considering replacing it with a new (more modern and
    > ACPI capable) ISA/LPC bridge driver. This work would likely begin after
    > my PCI bridge driver rewrite is finished and merged (as the PCI work is
    > in some ways a prerequisite).
    >
    > http://marc.theaimsgroup.com/?l=linux-kernel&m=111023821617705&w=2
    >
    > Still, if there are changes to fix actual bugs, then I'm all for them.
    >
    > Also a few features could be added. Specifically PnPBIOS
    > hotplug/docking station support. If anyone's interested, I may
    > implement it (and it would use some functions that were removed by this
    > patch). Furthermore, ISAPnP could be made a module. PnPBIOS probably
    > couldn't.
    >...

    Note that my patch #if 0's exactly one functions and removes no
    functions. Most it does is the removal of EXPORT_SYMBOL's, so if any
    modular code will use any of them, re-adding will be trivial.

    Modular ISAPnP might be interesting in some cases, but this is more
    legacy code. If someone would work on it to sort all the issues out
    (starting with the point that most users of __ISAPNP__ will have to be
    fixed) re-adding the required EXPORT_SYMBOL's won't be hard for him.

    > Thanks,
    > Adam

    cu
    Adrian

    -- 
           "Is there not promise of rain?" Ling Tan asked suddenly out
            of the darkness. There had been need of rain for many days.
           "Only a promise," Lao Er said.
                                           Pearl S. Buck - Dragon Seed
    -
    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/
    

  • Next message: Dmitry Yusupov: "Re: [ANNOUNCE 0/6] Open-iSCSI High-Performance Initiator for Linux"

    Relevant Pages

    • Re: [2.6 patch] let W1 select NET
      ... > selecting the required dependencies. ... There had been need of rain for many days. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [2.6 patch] drivers/acpi: remove unused exported functions
      ... > future patches, but sometimes Real Life gets in the way and the ... > programmer stalls development for some time, no problem, just ifdef it. ... There had been need of rain for many days. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.12 Performance problems
      ... Linux has long had a philisophical ... debug your problem. ... There had been need of rain for many days. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: A users thoughts on the new dev. model
      ... >> when the stable tree wasn't. ... Andrew said explicitely in a mail to linux-kernel that he'd consider ... There had been need of rain for many days. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [PATCH] sched.c gratious export removal
      ... sched.c exports a few internal helpers although none of them is used in ... modular code or code that could easily be made modular, ... * Return a low guess at the load of a migration-source cpu. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)