From: ambx1@neo.rr.com (Adam Belay)
To: Dmitry Torokhov <dtor_core@ameritech.net>
Cc: Adam Belay <ambx1@neo.rr.com>,
linux-kernel@vger.kernel.org,
matthieu castet <castet.matthieu@free.fr>,
bjorn.helgaas@hp.com, vojtech@suse.cz, "Brown,
Len" <len.brown@intel.com>,
greg@kroah.com
Subject: Re: [PATCH] PNP support for i8042 driver
Date: Tue, 16 Nov 2004 01:24:33 -0500 [thread overview]
Message-ID: <20041116062433.GH29574@neo.rr.com> (raw)
In-Reply-To: <200411160106.46673.dtor_core@ameritech.net>
On Tue, Nov 16, 2004 at 01:06:46AM -0500, Dmitry Torokhov wrote:
> Adam,
>
> I agree with your point that every device in the system should have a
> driver attached. And i8042 does have one bound to it. It is i8042 platform
> driver that does power management and ensures proper integration into driver
> model.
>
> There is no need to keep secondary "drivers" around, their sole purpose is
> to provide information about avalilable resources. It would be ok if the
> code was shared among several devices on a bus but for most (all?) legacy
> devices it has to be programmed explicitely and will not be reused.
Platform drivers are secondary drivers. They're crude hacks used to get a
minimal device into the driver model. They're necessary because ISA devices
cannot be detected like other types of devices. Why not use a system with
actual complete bus functionality like ACPI? If ACPI is available, then there
should be no need to create a platform device.
>
> Also i8042 should not rely on either ACPI or PNP simply because the driver/
> chip works on boxes other than x86/ia64 so we can't make ACPI or PNP drivers
> "main" ones.
No, but they should take priority when they are available. Don't forget that
there is also Open Firmware, which has similar functionalility to ACPI if I
understand correctly.
>
> As far as binding/rebinding goes I guess sysdevs and platform devices will
> just disable this functionality.
Exactly, they have limits because they are not real devices. ACPI devices are
real devices. They have resource management capabilities, power dependencies,
physical parents, and other features found in buses like PCI.
Thanks,
Adam
next prev parent reply other threads:[~2004-11-16 6:26 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-13 13:23 [PATCH] PNP support for i8042 driver matthieu castet
2004-11-14 6:48 ` Dmitry Torokhov
2004-11-14 12:22 ` matthieu castet
2004-11-15 14:41 ` Dmitry Torokhov
2004-11-15 19:51 ` matthieu castet
2004-11-15 20:28 ` Dmitry Torokhov
2004-11-15 22:52 ` matthieu castet
2004-11-15 23:09 ` matthieu castet
2004-11-16 5:52 ` Adam Belay
2004-11-16 6:27 ` Dmitry Torokhov
2004-11-16 5:37 ` Adam Belay
2004-11-16 5:44 ` Greg KH
2004-11-16 6:06 ` Dmitry Torokhov
2004-11-16 6:24 ` Adam Belay [this message]
2004-11-17 10:07 ` Vojtech Pavlik
2005-02-04 17:37 ` matthieu castet
2005-02-04 18:28 ` Vojtech Pavlik
2005-02-04 22:54 ` matthieu castet
2005-02-05 13:48 ` matthieu castet
2005-02-05 18:51 ` Dmitry Torokhov
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=20041116062433.GH29574@neo.rr.com \
--to=ambx1@neo.rr.com \
--cc=bjorn.helgaas@hp.com \
--cc=castet.matthieu@free.fr \
--cc=dtor_core@ameritech.net \
--cc=greg@kroah.com \
--cc=len.brown@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=vojtech@suse.cz \
/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