Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752341AbbBBO4F (ORCPT ); Mon, 2 Feb 2015 09:56:05 -0500 Received: from mail-qa0-f53.google.com ([209.85.216.53]:52847 "EHLO mail-qa0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751087AbbBBO4B (ORCPT ); Mon, 2 Feb 2015 09:56:01 -0500 MIME-Version: 1.0 In-Reply-To: <1561014.NTLJeH8lPH@dabox> References: <1561014.NTLJeH8lPH@dabox> From: Bjorn Helgaas Date: Mon, 2 Feb 2015 08:55:33 -0600 Message-ID: Subject: Re: [PATCH] 3.19-rc7: add quirk for 1c28:0122 (rev 14) SATA controller To: Tim Sander Cc: "linux-kernel@vger.kernel.org" , Linus Torvalds , "linux-pci@vger.kernel.org" , "alex.williamson@redhat.com" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2597 Lines: 73 [+cc Alex] On Mon, Feb 2, 2015 at 5:29 AM, Tim Sander wrote: > > The long name for this device is > Lite-On IT Corp. / Plextor M6e PCI Express SSD [Marvell 88SS9183] (rev 14) > > Background: the error description on bugzilla > https://bugzilla.kernel.org/show_bug.cgi?id=42679 > inspired me to check if my brand new m4 pci ssd could work with a quirk. > Its device representation is not visible before this patch. > After this patch it appears as a normal device. Formatting > and mounting worked so far. > > Signed-off-by: Tim Sander Hi Tim, Thanks a lot for the patch. Can you open a new report at http://bugzilla.kernel.org and attach the complete dmesg and "lspci -vv" output without this patch, and also a dmesg log *with* this patch? I like to keep a little documentation around in case we restructure this code in the future. Also, since the vendor and device IDs are used only once, we probably won't add them to pci_ids.h (see the comment at the top of the file). Just use the bare constants in the quirk. Bjorn > --- > drivers/pci/quirks.c | 4 ++++ > include/linux/pci_ids.h | 3 +++ > 2 files changed, 7 insertions(+) > > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c > index e52356a..fad7383 100644 > --- a/drivers/pci/quirks.c > +++ b/drivers/pci/quirks.c > @@ -3540,6 +3540,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642, > DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON, > PCI_DEVICE_ID_JMICRON_JMB388_ESD, > quirk_dma_func1_alias); > +/* https://bugzilla.kernel.org/show_bug.cgi?id=42679 */ > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_LITE_ON, > + PCI_DEVICE_ID_PLEXTOR_M6E, > + quirk_dma_func1_alias); > > /* > * A few PCIe-to-PCI bridges fail to expose a PCIe capability, resulting in > diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h > index e63c02a..1607b20 100644 > --- a/include/linux/pci_ids.h > +++ b/include/linux/pci_ids.h > @@ -2487,6 +2487,9 @@ > > #define PCI_VENDOR_ID_ASMEDIA 0x1b21 > > +#define PCI_VENDOR_ID_LITE_ON 0x1c28 > +#define PCI_DEVICE_ID_PLEXTOR_M6E 0x0122 > + > #define PCI_VENDOR_ID_CIRCUITCO 0x1cc8 > #define PCI_SUBSYSTEM_ID_CIRCUITCO_MINNOWBOARD 0x0001 > > -- > 1.9.1 > > -- 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/