Hi,

On Fri, Sep 26, 2014 at 12:53:21PM +0800, Huang Rui wrote:
> On Thu, Sep 25, 2014 at 09:53:55AM -0500, Felipe Balbi wrote:
> > On Thu, Sep 25, 2014 at 03:21:44PM +0800, Huang Rui wrote:
> > > Add PCI device ID of AMD NL.
> > > 
> > > Signed-off-by: Huang Rui <ray.hu...@amd.com>
> > > ---
> > >  drivers/usb/dwc3/dwc3-pci.c | 2 ++
> > >  1 file changed, 2 insertions(+)
> > > 
> > > diff --git a/drivers/usb/dwc3/dwc3-pci.c b/drivers/usb/dwc3/dwc3-pci.c
> > > index 436fb08..cebbd01 100644
> > > --- a/drivers/usb/dwc3/dwc3-pci.c
> > > +++ b/drivers/usb/dwc3/dwc3-pci.c
> > > @@ -30,6 +30,7 @@
> > >  #define PCI_DEVICE_ID_SYNOPSYS_HAPSUSB3  0xabcd
> > >  #define PCI_DEVICE_ID_INTEL_BYT          0x0f37
> > >  #define PCI_DEVICE_ID_INTEL_MRFLD        0x119e
> > > +#define PCI_DEVICE_ID_AMD_NL             0x7912
> > >  
> > >  struct dwc3_pci {
> > >   struct device           *dev;
> > > @@ -183,6 +184,7 @@ static const struct pci_device_id dwc3_pci_id_table[] 
> > > = {
> > >   },
> > >   { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_BYT), },
> > >   { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_MRFLD), },
> > > + { PCI_DEVICE(PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_NL), },
> > 
> > this patch causes a build breakage.
> > 
> 
> That's strange, I tested it once again in my side based on your "next"
> branch and didn't find any build error. If I misunderstood your
> meaning, please correct me. :)

no, I just misread your patch. I'm sorry. I thought the PCI Device ID
was set to the quirk which was not defined here. I apologize, this patch
is 100% correct.

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to