Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: 0t4QUhTWtomT8UekP8Yqu7m3lWfWKbf3yfLS39m3kq3ZL/+7MJRmdSTxbqnBipdOtQf/0jlGTw
 Lm1tp8H1n69w==
IronPort-SDR: /HFmnbYNjhZx1YxmCSWAUvfud0DDF3sX7LduM5lISPlmbBaLvDEZLmCBL/EhQQGlKFr1ygCfhN
 hS4hd1XmHlyw==
Date:   Tue, 6 Apr 2021 08:42:04 -0700 (PDT)
From:   matthew.gerlach@linux.intel.com
To:     Moritz Fischer <mdf@kernel.org>
cc:     hao.wu@intel.com, trix@redhat.com, linux-fpga@vger.kernel.org,
        linux-kernel@vger.kernel.org, yilun.xu@intel.com,
        jdelvare@suse.com, linux@roeck-us.net, lee.jones@linaro.org,
        linux-hwmon@vger.kernel.org, russell.h.weight@linux.intel.com,
        Russ Weight <russell.h.weight@intel.com>
Subject: Re: [PATCH 1/3] fpga: dfl: pci: add DID for D5005 PAC cards
In-Reply-To: <YGuwDHuxQd01sLQu@epycbox.lan>
Message-ID: <alpine.DEB.2.22.394.2104060841350.208844@rhweight-WRK1>
References: <20210405235301.187542-1-matthew.gerlach@linux.intel.com> <20210405235301.187542-2-matthew.gerlach@linux.intel.com> <YGuwDHuxQd01sLQu@epycbox.lan>
User-Agent: Alpine 2.22 (DEB 394 2020-01-19)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII; format=flowed
Precedence: bulk


On Mon, 5 Apr 2021, Moritz Fischer wrote:

> On Mon, Apr 05, 2021 at 04:52:59PM -0700, matthew.gerlach@linux.intel.com wrote:
>> From: Russ Weight <russell.h.weight@intel.com>
>>
>> This patch adds the approved PCI Express Device IDs for the
>> PF and VF for the card for D5005 PAC cards.
>>
>> Signed-off-by: Russ Weight <russell.h.weight@intel.com>
>> Signed-off-by: Matthew Gerlach <matthew.gerlach@linux.intel.com>
>> ---
>>  drivers/fpga/dfl-pci.c | 18 +++++++++++-------
>>  1 file changed, 11 insertions(+), 7 deletions(-)
>>
>> diff --git a/drivers/fpga/dfl-pci.c b/drivers/fpga/dfl-pci.c
>> index 04e47e2..b44523e 100644
>> --- a/drivers/fpga/dfl-pci.c
>> +++ b/drivers/fpga/dfl-pci.c
>> @@ -69,14 +69,16 @@ static void cci_pci_free_irq(struct pci_dev *pcidev)
>>  }
>>
>>  /* PCI Device ID */
>> -#define PCIE_DEVICE_ID_PF_INT_5_X	0xBCBD
>> -#define PCIE_DEVICE_ID_PF_INT_6_X	0xBCC0
>> -#define PCIE_DEVICE_ID_PF_DSC_1_X	0x09C4
>> -#define PCIE_DEVICE_ID_INTEL_PAC_N3000	0x0B30
>> +#define PCIE_DEVICE_ID_PF_INT_5_X		0xBCBD
>> +#define PCIE_DEVICE_ID_PF_INT_6_X		0xBCC0
>> +#define PCIE_DEVICE_ID_PF_DSC_1_X		0x09C4
>> +#define PCIE_DEVICE_ID_INTEL_PAC_N3000		0x0B30
>> +#define PCIE_DEVICE_ID_INTEL_PAC_D5005		0x0B2B
>>  /* VF Device */
>> -#define PCIE_DEVICE_ID_VF_INT_5_X	0xBCBF
>> -#define PCIE_DEVICE_ID_VF_INT_6_X	0xBCC1
>> -#define PCIE_DEVICE_ID_VF_DSC_1_X	0x09C5
>> +#define PCIE_DEVICE_ID_VF_INT_5_X		0xBCBF
>> +#define PCIE_DEVICE_ID_VF_INT_6_X		0xBCC1
>> +#define PCIE_DEVICE_ID_VF_DSC_1_X		0x09C5
>> +#define PCIE_DEVICE_ID_INTEL_PAC_D5005_VF	0x0B2C
>>
>>  static struct pci_device_id cci_pcie_id_tbl[] = {
>>  	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_PF_INT_5_X),},
>> @@ -86,6 +88,8 @@ static void cci_pci_free_irq(struct pci_dev *pcidev)
>>  	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_PF_DSC_1_X),},
>>  	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_VF_DSC_1_X),},
>>  	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_INTEL_PAC_N3000),},
>> +	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_INTEL_PAC_D5005),},
>> +	{PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCIE_DEVICE_ID_INTEL_PAC_D5005_VF),},
>>  	{0,}
>>  };
>>  MODULE_DEVICE_TABLE(pci, cci_pcie_id_tbl);
>> --
>> 1.8.3.1
>>
>
> Applied to for-next,
>
> Thanks
>

Thank you for the review.

Matthew