2005-10-07 15:56:44

by Bill Waddington

[permalink] [raw]
Subject: [RFClue] pci_get_device, new driver model

I'm missing something fundamental, and beg your indulgence.

My current 2.6 drivers support multiple indentical PCI boards.
The init code spins on pci_find_device and assigns instance/minor
numbers as boards are found. Load script insmods the driver,
gets the major # from /proc/devices, and creates the /dev/ entries.




Not subscribed but lurking,
Bill
--
--------------------------------------------
William D Waddington
Bainbridge Island, WA, USA
[email protected]
--------------------------------------------
"Even bugs...are unexpected signposts on
the long road of creativity..." - Ken Burtch


2005-10-07 16:16:05

by Steven Rostedt

[permalink] [raw]
Subject: Dell firmware default config options?


I'm just curious why the Dell firmware configuration options are default
to "m" instead of "n". Since it only matters if you have a Dell System.

So for the huge number of systems that are not Dell Systems, they are
probably wasting CPU cycles compiling these as modules, and taking up
space in loads of /lib/modules directories throughout the world ;-)

DCDBAS explicitly states default of "m", and DELL_RBU has no default which
just makes it automatically on.

Is there any reason that these shouldn't be turned off by default?

Thanks,

-- Steve


2005-10-07 16:20:29

by Randy Dunlap

[permalink] [raw]
Subject: Re: Dell firmware default config options?

On Fri, 7 Oct 2005, Steven Rostedt wrote:

>
> I'm just curious why the Dell firmware configuration options are default
> to "m" instead of "n". Since it only matters if you have a Dell System.
>
> So for the huge number of systems that are not Dell Systems, they are
> probably wasting CPU cycles compiling these as modules, and taking up
> space in loads of /lib/modules directories throughout the world ;-)
>
> DCDBAS explicitly states default of "m", and DELL_RBU has no default which
> just makes it automatically on.
>
> Is there any reason that these shouldn't be turned off by default?

Agreed, and there's already been a patch to do that.
It's probably sitting in -mm... (just guessing), but could
easily move upward IMO.

--
~Randy