Date: Fri, 4 Feb 2005 19:28:16 +0100
From: Vojtech Pavlik <vojtech@suse.cz>
To: matthieu castet <castet.matthieu@free.fr>
Cc: linux-kernel@vger.kernel.org, Adam Belay <ambx1@neo.rr.com>,
       bjorn.helgaas@hp.com, Dmitry Torokhov <dtor_core@ameritech.net>
Subject: Re: [PATCH] PNP support for i8042 driver
Message-ID: <20050204182816.GA3573@ucw.cz>
References: <41960AE9.8090409@free.fr> <20041117100745.GA1387@ucw.cz> <4203B2D9.7080904@free.fr>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4203B2D9.7080904@free.fr>
User-Agent: Mutt/1.5.6i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1377
Lines: 41

On Fri, Feb 04, 2005 at 06:37:29PM +0100, matthieu castet wrote:
> Hi,
> 
> Vojtech Pavlik wrote:
> >On Sat, Nov 13, 2004 at 02:23:53PM +0100, matthieu castet wrote:
> >
> >>Hi,
> >>this patch add PNP support for the i8042 driver in 2.6.10-rc1-mm5. Acpi 
> >>is try before the pnp driver so if you don't disable ACPI or apply 
> >>others pnpacpi patches, it won't change anything.
> >>
> >>Please review it and apply if possible
> >
> >
> >Ok, my thoughts on this:
> >
> >	It's OK to keep the device allocated to this driver via the PnP
> >        subsystem, and not bother with releasing the code via
> >	__initcall.
> >
> >	I agree that if there is a way to enumerate the device, (like
> >	PnP, ACPI or OpenFirmware), we should use that instead of
> >	probing and using a platform device for the controller.
> >
> >	I think that we should drop the ACPI support from i8042, in
> >	favor of pnpacpi, because PnP is more generic and if the
> > 	keyboard device was listed in PnPBIOS instead of ACPI, it'll
> >	still work.
> >
> Any news about this ?
 
Sort of fell off my radar, can you resend?

-- 
Vojtech Pavlik
SuSE Labs, SuSE CR
-
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/