[kmods,v2] windows/netuio: add Intel device ID

Message ID 20211019190102.1903-1-u9012063@gmail.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series [kmods,v2] windows/netuio: add Intel device ID |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing warning apply patch failure
ci/Intel-compilation warning apply issues

Commit Message

William Tu Oct. 19, 2021, 7:01 p.m. UTC
  The patch adds three Intel device IDs,
I350 (0x1521), 82574L (0x10D3), and 82599 (0x10ED).

Signed-off-by: William Tu <u9012063@gmail.com>
Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
Acked-by: Pallavi Kadam <pallavi.kadam@intel.com>
---
v2:
* fix the device string suggested by Pallavi
* combine this patch with the 82599 ixgbevf

---
 windows/netuio/netuio.inf | 6 ++++++
 1 file changed, 6 insertions(+)
  

Comments

William Tu Nov. 24, 2021, 4:11 p.m. UTC | #1
Hi,
Can we merge this patch? Thank you!
William

On Tue, Oct 19, 2021 at 12:02 PM William Tu <u9012063@gmail.com> wrote:
>
> The patch adds three Intel device IDs,
> I350 (0x1521), 82574L (0x10D3), and 82599 (0x10ED).
>
> Signed-off-by: William Tu <u9012063@gmail.com>
> Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
> Acked-by: Pallavi Kadam <pallavi.kadam@intel.com>
> ---
> v2:
> * fix the device string suggested by Pallavi
> * combine this patch with the 82599 ixgbevf
>
> ---
>  windows/netuio/netuio.inf | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/windows/netuio/netuio.inf b/windows/netuio/netuio.inf
> index 816ff17c7936..99dd53c6953f 100644
> --- a/windows/netuio/netuio.inf
> +++ b/windows/netuio/netuio.inf
> @@ -29,6 +29,7 @@ HKR,,Icon,,-5
>  [Standard.NT$ARCH$]
>  %netuio.DeviceDesc%=netuio_Device, Root\netuio
>  %netuio.nvme.DeviceDesc%=netuio_Device, PCI\CC_010802
> +%Intel.F1521.Description%=netuio_Device, PCI\VEN_8086&DEV_1521
>  %Intel.F1572.Description%=netuio_Device, PCI\VEN_8086&DEV_1572
>  %Intel.F1580.Description%=netuio_Device, PCI\VEN_8086&DEV_1580
>  %Intel.F1581.Description%=netuio_Device, PCI\VEN_8086&DEV_1581
> @@ -43,6 +44,8 @@ HKR,,Icon,,-5
>  %Intel.F101F.Description%=netuio_Device, PCI\VEN_8086&DEV_101F
>  %Intel.F104F.Description%=netuio_Device, PCI\VEN_8086&DEV_104F
>  %Intel.F104E.Description%=netuio_Device, PCI\VEN_8086&DEV_104E
> +%Intel.F10D3.Description%=netuio_Device, PCI\VEN_8086&DEV_10D3
> +%Intel.F10ED.Description%=netuio_Device, PCI\VEN_8086&DEV_10ED
>  %Intel.F1591.Description%=netuio_Device, PCI\VEN_8086&DEV_1591
>  %Intel.F1592.Description%=netuio_Device, PCI\VEN_8086&DEV_1592
>  %Intel.F1593.Description%=netuio_Device, PCI\VEN_8086&DEV_1593
> @@ -100,6 +103,7 @@ ClassName = "Windows UIO"
>  DiskName = "DPDK netUIO Installation Disk"
>  netuio.DeviceDesc = "netuio Device"
>  netuio.nvme.DeviceDesc = "netuio NVM Express Controller"
> +Intel.F1521.Description = "Intel(R) I350 Gigabit Network Connection"
>  Intel.F1572.Description = "Intel(R) Ethernet Controller X710 for 10GbE SFP+"
>  Intel.F1580.Description = "Intel(R) Ethernet Controller XL710 for 40GbE backplane"
>  Intel.F1581.Description = "Intel(R) Ethernet Controller X710 for 10GbE backplane"
> @@ -114,6 +118,8 @@ Intel.F15FF.Description = "Intel(R) Ethernet Controller X710 for 10GBASE-T"
>  Intel.F101F.Description = "Intel(R) Ethernet Controller V710 for 5GBASE-T"
>  Intel.F104F.Description = "Intel(R) Ethernet Controller X710 for 10 Gigabit backplane"
>  Intel.F104E.Description = "Intel(R) Ethernet Controller X710 for 10 Gigabit SFP+"
> +Intel.F10D3.Description = "Intel(R) 82574L Gigabit Network Connection"
> +Intel.F10ED.Description = "Intel(R) 82599 Virtual Function"
>  Intel.F1591.Description = "Intel(R) Ethernet Controller E810-C for backplane"
>  Intel.F1592.Description = "Intel(R) Ethernet Controller E810-C for QSFP"
>  Intel.F1593.Description = "Intel(R) Ethernet Controller E810-C for SFP"
> --
> 2.33.0.windows.2
>
  
Thomas Monjalon Jan. 11, 2022, 2:21 p.m. UTC | #2
19/10/2021 21:01, William Tu:
> The patch adds three Intel device IDs,
> I350 (0x1521), 82574L (0x10D3), and 82599 (0x10ED).
> 
> Signed-off-by: William Tu <u9012063@gmail.com>
> Acked-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
> Acked-by: Pallavi Kadam <pallavi.kadam@intel.com>

Applied, thanks.
  

Patch

diff --git a/windows/netuio/netuio.inf b/windows/netuio/netuio.inf
index 816ff17c7936..99dd53c6953f 100644
--- a/windows/netuio/netuio.inf
+++ b/windows/netuio/netuio.inf
@@ -29,6 +29,7 @@  HKR,,Icon,,-5
 [Standard.NT$ARCH$]
 %netuio.DeviceDesc%=netuio_Device, Root\netuio
 %netuio.nvme.DeviceDesc%=netuio_Device, PCI\CC_010802
+%Intel.F1521.Description%=netuio_Device, PCI\VEN_8086&DEV_1521
 %Intel.F1572.Description%=netuio_Device, PCI\VEN_8086&DEV_1572
 %Intel.F1580.Description%=netuio_Device, PCI\VEN_8086&DEV_1580
 %Intel.F1581.Description%=netuio_Device, PCI\VEN_8086&DEV_1581
@@ -43,6 +44,8 @@  HKR,,Icon,,-5
 %Intel.F101F.Description%=netuio_Device, PCI\VEN_8086&DEV_101F
 %Intel.F104F.Description%=netuio_Device, PCI\VEN_8086&DEV_104F
 %Intel.F104E.Description%=netuio_Device, PCI\VEN_8086&DEV_104E
+%Intel.F10D3.Description%=netuio_Device, PCI\VEN_8086&DEV_10D3
+%Intel.F10ED.Description%=netuio_Device, PCI\VEN_8086&DEV_10ED
 %Intel.F1591.Description%=netuio_Device, PCI\VEN_8086&DEV_1591
 %Intel.F1592.Description%=netuio_Device, PCI\VEN_8086&DEV_1592
 %Intel.F1593.Description%=netuio_Device, PCI\VEN_8086&DEV_1593
@@ -100,6 +103,7 @@  ClassName = "Windows UIO"
 DiskName = "DPDK netUIO Installation Disk"
 netuio.DeviceDesc = "netuio Device"
 netuio.nvme.DeviceDesc = "netuio NVM Express Controller"
+Intel.F1521.Description = "Intel(R) I350 Gigabit Network Connection"
 Intel.F1572.Description = "Intel(R) Ethernet Controller X710 for 10GbE SFP+"
 Intel.F1580.Description = "Intel(R) Ethernet Controller XL710 for 40GbE backplane"
 Intel.F1581.Description = "Intel(R) Ethernet Controller X710 for 10GbE backplane"
@@ -114,6 +118,8 @@  Intel.F15FF.Description = "Intel(R) Ethernet Controller X710 for 10GBASE-T"
 Intel.F101F.Description = "Intel(R) Ethernet Controller V710 for 5GBASE-T"
 Intel.F104F.Description = "Intel(R) Ethernet Controller X710 for 10 Gigabit backplane"
 Intel.F104E.Description = "Intel(R) Ethernet Controller X710 for 10 Gigabit SFP+"
+Intel.F10D3.Description = "Intel(R) 82574L Gigabit Network Connection"
+Intel.F10ED.Description = "Intel(R) 82599 Virtual Function"
 Intel.F1591.Description = "Intel(R) Ethernet Controller E810-C for backplane"
 Intel.F1592.Description = "Intel(R) Ethernet Controller E810-C for QSFP"
 Intel.F1593.Description = "Intel(R) Ethernet Controller E810-C for SFP"