On Thu, 8 Jun 2006, David Brownell wrote:
> On Thursday 08 June 2006 2:49 pm, Richard Purdie wrote:
> >
> > > One additional nuance: if
> > > the kernel doesn't have that driver configured, that's another reason not
> > > to bother registering its device.
> >
> > This is where you start to add ugly ifdefs and generally start making
> > the code look horrible. The device model separated the drivers and the
> > devices to deal with this issue as I see it.
>
> Enumeration is a separate issue. You wouldn't argue that every potential
> PCI or USB device must get registered, right? Only the ones that are
> actually _present_ get registered.
You are both saying the same thing so far.
> But here you argue that platform bus should not work that same way ... it
> should register devices that can't be present. If nothing else, that's
> an inconsistent aproach.
That's not what Richard is saying.
He replied to your assertion where you said: "if the kernel doesn't have
that driver configured, that's another reason not to bother registering
its device" to which he disagreed, and I disagree too.
> Plus, consider the common situation that a given pin could potentially
> be used with several different devices. On a given board, only one of
> those devices will be wired up. It's counterproductive to register any
> of the others ... error prone, waste-of-kernel-address-space, etc.
No one disagreed with that AFAICS.
> > Generally I'd say its
> > cleaner just to let the device register, then if a module comes along at
> > some later point, the device is there for it.
>
> Whether the device is there or not is a hardware issue. Board schematics
> will show which devices are relevant ... registering any others is just
> wastage.
But you originally talked about _driver_ configuration dictating if a
_device_ should be registered or not.
The _device_ should indeed be registered based on _hardware_ config, not
_driver_ config.
Nicolas
-
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]
