On Wed, Oct 09, 2002 at 11:47:16AM -0700, Linus Torvalds wrote:
>
> On Wed, 9 Oct 2002, Sam Ravnborg wrote:
> >
> > ls rrunner*
> > should show me not only the implementation [.c + .h] but also
> > the configuration.
>
> I agree with you, but only if we _always_ have one config file per driver.
>
> Which is not necessarily the wrong thing to do.
The question here is what we aim at.
And we aim at getting the configuration less centralized as compared with
what we have today.
I still remember the thread were you suggested the drivers.conf principle.
And I liked it then, and I like it today.
What I had to add a new driver to the kernel was to add the following
three files:
driver.c, driver,h driver.conf
Even the makefile stuff could be retreived from driver.conf.
>
> But as long as most drivers are in "grouped" config files, they should be
> things like "Config.3com" etc.
>
> Looking at existing big config files (video, networking), they do _not_
> group according to driver, but according to other criteria (ie "PCI card",
> "100/1000 Mbit", "3com", "Sparc-specific" etc).
But there is a good reason why they do it.
Take a look at dirvers/video/Config.in for example.
See the size of the big if's. They span several pages if not the whole file.
Why they do this is simple. Only check for PCI once, and group all
PCI stuff there.
With the syntax Roman propose we see instead that _each_ config option
check for PCI. Which is good IMHO.
The most logical grouping should be utilised, and grouping based on
bustype is not the grouping that we aim for.
We aim to group similar drivers together, if they happens to be for the
same bus or not should not matter.
But the whole argumentation boils down to something rahter simple:
1) Shall we group configuration files together
2) Shall we group related files together
And in mu opinion, if we decide not to use the filesystem namespace to
group similar files, then the incentive to break things out in smaller
files are mostly gone.
Except in the case where I sumbit a new driver and want to keep my
changes to the kernel file to a bare minimum, but then why not
let the config file be grouped with the rest of the driver.
Well I have raised my points, if you still prefer Config.* then let's
stick to that. This should but be the reason to delay lkc-roman.
Sam
-
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 : Tue Oct 15 2002 - 22:00:33 EST