Re: 2.6.15-rc2 pci_ids.h cleanup is a pain

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 22 Nov 2005 23:19:17 -0500, Dave Jones <[email protected]> wrote:

>On Wed, Nov 23, 2005 at 12:51:45PM +1100, Grant Coady wrote:
> > Hi Andrew, Greg,
> > 
> > On Tue, 22 Nov 2005 16:25:58 -0800, Andrew Morton <[email protected]> wrote:
> > 
> > >Greg KH <[email protected]> wrote:
> > >>
> > >> On Mon, Nov 21, 2005 at 02:27:23PM -0800, David Fox wrote:
> > >> > I'm sure I'm not the only person that applies patches to the kernel that 
> > >> > use some of the 500 plus PCI IDS eliminated from pci_ids.h by rc2.  I 
> > >> > would like to see the PCI ids that were removed simply because the don't 
> > >> > occur in the main kernel source restored.  Is there a rationale for 
> > >> > removing them that I'm not aware of?
> > >> 
> > >> They were not being used.  Why would you want them in there?
> > >
> > >Because they contained useful information which had been accumulated by
> > >many people over a long period of time.
> > >
> > >Throwing that information away seemed rather pointless, especially as the
> > >cost of retaining it was so low.
> > 
> > There's an out-of-tree reference, the pci.ids website, that carries 
> > this information, do we need the reference info in the kernel as well?  
> > 
> > So far two people raised an objection, the other wants to maintain 
> > an out-of-tree driver, D. Fox didn't say why he needs the symbols.
>
>Three. I already mentioned we broke the compilation of the
>advansys driver because of this.

Nope, advansys.* don't appear to use PCI_*  Defines its own ASC_PCI* 
instead?

Checked:  linux-2.6.13.4, linux-2.6.15-rc2

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux