public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Dave Jones <davej@redhat.com>
Cc: "Randy.Dunlap" <rddunlap@osdl.org>,
	greg@kroah.com, willy@debian.org, davem@redhat.com,
	linux-kernel@vger.kernel.org,
	kernel-janitor-discuss@lists.sourceforge.net
Subject: Re: C99 Initialisers
Date: Tue, 12 Aug 2003 21:25:56 -0400	[thread overview]
Message-ID: <3F3993A4.40108@pobox.com> (raw)
In-Reply-To: <20030813004941.GD2184@redhat.com>

Dave Jones wrote:
> On Tue, Aug 12, 2003 at 05:37:42PM -0700, Randy.Dunlap wrote:
>  > | I would much rather move the PCI ids out of the 
>  > | drivers altogether, into some metadata file(s) in the kernel source 
>  > | tree, than bloat up tg3, tulip, e100, and the other PCI id-heavy 
>  > | drivers' source code.
>  > 
>  > That last few lines certainly sounds desirable.
> 
> What exactly would be the benefit of this ?
> The only thing I could think of was out-of-kernel tools to do
> things like matching modules to pci IDs, but that seems to be
> done mechanically by various distros already reading the pci_driver
> structs.


Fundamentally, the PCI ID list is not C code.  And if anyone ever wants 
to get to the PCI ID lists at the _source code_ level, they have to 
parse C or assembler :)  It's data, so I say, put it in a data file. 
Stuffing the PCI ID list in C code is a sometimes convenient, sometimes 
inconvenient form of packaging, nothing more :)

I would rather store the PCI ID list in a more natural form, and then 
use small tool to generate the pci_device_id tables that are linked into 
the kernel.

	Jeff




  reply	other threads:[~2003-08-13  1:26 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-08-12  2:02 C99 Initialisers CaT
2003-08-12  2:18 ` Robert Love
2003-08-12  2:39   ` Matthew Wilcox
2003-08-12  2:45     ` Robert Love
2003-08-12  2:57     ` Dagfinn Ilmari Mannsåker
2003-08-12  5:38     ` Greg KH
2003-08-12  9:01       ` Maciej Soltysiak
2003-08-12 10:03         ` Geert Uytterhoeven
2003-08-12 10:19           ` Jakub Jelinek
2003-08-12 11:27       ` Matthew Wilcox
2003-08-12 16:54         ` Ian Hastie
2003-08-12 18:01         ` Greg KH
2003-08-12 23:53           ` Dave Jones
2003-08-13  0:08             ` Matthew Wilcox
2003-08-13  0:23               ` Greg KH
2003-08-13  0:31                 ` Matthew Wilcox
2003-08-14  5:45               ` H. Peter Anvin
2003-08-13 15:52             ` Timothy Miller
2003-08-13 17:50               ` Jeff Garzik
2003-08-13  0:02           ` Jeff Garzik
2003-08-13  0:14             ` Randy.Dunlap
2003-08-13  0:31               ` Jeff Garzik
2003-08-13  0:37                 ` Randy.Dunlap
2003-08-13  0:49                   ` Dave Jones
2003-08-13  1:25                     ` Jeff Garzik [this message]
2003-08-13  3:02                     ` Randy.Dunlap
2003-08-13  3:26                       ` Jeff Garzik
2003-08-13 10:14                         ` David S. Miller
2003-08-13 17:31                           ` Greg KH
2003-08-13 17:36                             ` David S. Miller
2003-08-13 17:47                             ` Jeff Garzik
2003-08-13 18:02                               ` Greg KH
2003-08-13 18:26                                 ` Jeff Garzik
2003-08-13 18:38                                   ` Russell King
2003-08-13 19:44                                     ` Jeff Garzik
2003-08-13 19:54                                       ` Matthew Wilcox
2003-08-13 20:15                                         ` Greg KH
2003-08-13 20:16                                         ` Dave Jones
2003-08-13 20:30                                           ` Matt Domsch
2003-08-13 20:29                                         ` Jeff Garzik
2003-08-13 21:05                                           ` Sam Ravnborg
2003-08-13 22:24                                             ` Roman Zippel
2003-08-14 20:31                                               ` Sam Ravnborg
2003-08-14 10:05                                           ` Geert Uytterhoeven
2003-08-14 10:25                                             ` Gene Heskett
2003-08-14 10:52                                             ` jw schultz
2003-08-14 12:34                                               ` Geert Uytterhoeven
2003-08-14 12:57                                             ` Andrey Panin
2003-08-14 18:45                                             ` H. Peter Anvin
2003-08-13 21:06                                       ` Russell King
2003-08-13 21:17                                       ` Eduardo Pereira Habkost
2003-08-13 17:50                             ` Sam Ravnborg
2003-08-13 17:54                               ` Jeff Garzik
2003-08-13 17:54                               ` Matthew Wilcox
2003-08-13 17:58                                 ` Jeff Garzik
2003-08-13 18:03                                 ` Greg KH
2003-08-13 17:58                               ` Greg KH
2003-08-13 18:21                                 ` Sam Ravnborg
2003-08-13 18:09                               ` Russell King
2003-08-13 20:21                             ` Krzysztof Halasa
2003-08-13 21:17                               ` David S. Miller
2003-08-13 21:26                               ` Greg KH
2003-08-14 22:46                                 ` Krzysztof Halasa
2003-08-12 17:37     ` Dave Jones
2003-08-12 17:48       ` Matthew Wilcox
2003-08-12 22:06         ` Ian Hastie
2003-08-13 15:54   ` CaT
2003-08-14  6:57     ` Maciej Soltysiak
  -- strict thread matches above, loose matches on Subject: below --
2003-08-12 16:52 Shureih, Tariq
     [not found] <jFFu.7t8.15@gated-at.bofh.it>
     [not found] ` <jLKX.4KI.13@gated-at.bofh.it>
     [not found]   ` <jRnj.2dx.11@gated-at.bofh.it>
     [not found]     ` <jRwZ.2kJ.15@gated-at.bofh.it>
     [not found]       ` <jRQi.2zQ.5@gated-at.bofh.it>
     [not found]         ` <jRZY.2Hw.5@gated-at.bofh.it>
     [not found]           ` <jS9J.2Np.5@gated-at.bofh.it>
     [not found]             ` <jUbt.57S.7@gated-at.bofh.it>
     [not found]               ` <jUuT.5kZ.13@gated-at.bofh.it>
     [not found]                 ` <k13k.22O.3@gated-at.bofh.it>
     [not found]                   ` <k7Lq.7Gr.7@gated-at.bofh.it>
2003-08-13 21:19                     ` junkio
2003-08-13 22:18                       ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3F3993A4.40108@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=davej@redhat.com \
    --cc=davem@redhat.com \
    --cc=greg@kroah.com \
    --cc=kernel-janitor-discuss@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rddunlap@osdl.org \
    --cc=willy@debian.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox