[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAL-B5D38sNcvpR+XxjLLbAm279v5LuCGQttai-Ek6zOUWCbv3g@mail.gmail.com>
Date: Thu, 21 Mar 2013 09:50:47 -0600
From: Myron Stowe <myron.stowe@...il.com>
To: Andrew Cooks <acooks@...il.com>
Cc: Bjorn Helgaas <bhelgaas@...gle.com>, yxlraid@...il.com,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Xiangliang Yu <yuxiangl@...vell.com>
Subject: Re: [PATCH 1/2] PCI: define macro for marvell vendor ID
On Mon, Mar 11, 2013 at 11:23 PM, Andrew Cooks <acooks@...il.com> wrote:
> On Fri, Mar 8, 2013 at 12:24 AM, Bjorn Helgaas <bhelgaas@...gle.com> wrote:
>> On Thu, Mar 7, 2013 at 7:28 AM, <yxlraid@...il.com> wrote:
>>> From: Xiangliang Yu <yuxiangl@...vell.com>
>>>
>>> Define PCI_VENDOR_MARVELL_ID_EXT macro for 0x1b4b vendor ID
>>
>> "PCI_VENDOR_MARVELL_ID_EXT" is not actually the symbol you defined below.
>>
>> Per the comment at the top of pci_ids.h, we don't add things here
>> unless they are shared between multiple drivers.
>>
>> That said, the 0x1b4b vendor ID is used in mv_init.c, mvumi.h,
>> mvumi.c, and ahci.c. If you want to unify all those so they all use
>> same symbol, and define that symbol here in pci_ids.h, that would be
>> fine with me.
>>
>
> This symbol will also be useful for the quirk that creates iommu
> context entries for Marvell devices that use incorrect dma source
> tags. I'll update and resend that patch when this change has been
> merged.
Andrew -
A new series that includes the 0x1b4b vendor ID change was posted
last night so you may want to follow that:
https://lkml.org/lkml/2013/3/21/12
>
> Thanks
> Andrew
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists