Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754697Ab0HPP4h (ORCPT ); Mon, 16 Aug 2010 11:56:37 -0400 Received: from cavan.codon.org.uk ([93.93.128.6]:47269 "EHLO cavan.codon.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754634Ab0HPP4g (ORCPT ); Mon, 16 Aug 2010 11:56:36 -0400 Date: Mon, 16 Aug 2010 16:56:27 +0100 From: Matthew Garrett To: Joey Lee Cc: gregkh@suse.de, trenn@suse.de, linux-kernel@vger.kernel.org, platform-driver-x86@vger.kernel.org, Dennis.Jansen@web.de Subject: Re: [PATCH] Support MSI Poulsbo netbook U110/U115 Message-ID: <20100816155627.GA2989@srcf.ucam.org> References: <4C659796020000230001EA4E@novprvlin0050.provo.novell.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4C659796020000230001EA4E@novprvlin0050.provo.novell.com> User-Agent: Mutt/1.5.18 (2008-05-17) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: mjg59@cavan.codon.org.uk X-SA-Exim-Scanned: No (on cavan.codon.org.uk); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 873 Lines: 20 On Fri, Aug 13, 2010 at 03:05:58AM -0600, Joey Lee wrote: > Hi Matthew, > > Dennis help me to ported this patch to 2.6.35, and it works as expected. > Could you please give me any suggestions for this patch? I've thought about this a bit more and now lean towards thinking that putting the PCI IDs in the ACPI video driver is the best way to go. The alternative is to only fix it for MSI, which isn't the best option. On the other hand, I looked at adding support to the Poulsbo driver and it actually seems pretty easy, so to be honest I'd prefer it if that got done instead. -- Matthew Garrett | mjg59@srcf.ucam.org -- 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/