From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Garrett Subject: Re: [git patches] libata updates for 3.3 Date: Tue, 17 Jan 2012 00:18:12 +0000 Message-ID: <20120117001812.GA9588@srcf.ucam.org> References: <1326691403.13517.21.camel@minggr> <20120116194211.2cd09fb3@pyramind.ukuu.org.uk> <20120116195427.64edd96f@pyramind.ukuu.org.uk> <4F14881C.8040601@garzik.org> <20120116235427.168e13f2@pyramind.ukuu.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from cavan.codon.org.uk ([93.93.128.6]:60860 "EHLO cavan.codon.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751258Ab2AQASX (ORCPT ); Mon, 16 Jan 2012 19:18:23 -0500 Content-Disposition: inline In-Reply-To: Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Linus Torvalds Cc: Alan Cox , Jeff Garzik , Lin Ming , Andrew Morton , linux-ide@vger.kernel.org, LKML On Mon, Jan 16, 2012 at 04:02:31PM -0800, Linus Torvalds wrote: > On Mon, Jan 16, 2012 at 3:54 PM, Alan Cox = wrote: > >> Part of the problem with force-enable is that the MMIO BAR may nee= d a > >> value, and not have it. =A0That, and an expectations mismatch betw= een BIOS > > > > That's fine - we can happily assign one at boot time. >=20 > Actually, I think the reason that Matthews patch doesn't work for me > is that on my device, BAR #5 really is a _port_ BAR. >=20 > I didn't play with it much - busy merging and looking at various othe= r > issues - but I'm starting to wonder whether maybe that 8086:1c01 chip > doesn't support AHCI at all. Or maybe it does something differently. >=20 > Other reports of this have BAR#5 either clear, or an MMIO BAR. That >=20 > Region 5: I/O ports at ffe0 [size=3D16] >=20 > looks really odd. Matthew's patch uses >=20 > pci_assign_resource(pdev, 5); >=20 > but if it is a PIO region, that won't help anything.. More recent ICHs may need different setup here. The datasheet should=20 have enough to figure it out, but I can take a look once I get back fro= m=20 LCA. --=20 Matthew Garrett | mjg59@srcf.ucam.org