[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-23 Thread Thomas Monjalon
2016-08-08 08:20, David Marchand:
> On Sat, Aug 6, 2016 at 2:55 PM, Thomas Monjalon
>  wrote:
> > What do you think about this list? I would say we can remove it.
> 
> Yes, we already discussed this with John.
> http://dpdk.org/ml/archives/dev/2016-January/031595.html

Done: http://dpdk.org/commit/562ad630

This patch will be rebased when applying.


[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-08 Thread David Marchand
On Sat, Aug 6, 2016 at 2:55 PM, Thomas Monjalon
 wrote:
> What do you think about this list? I would say we can remove it.

Yes, we already discussed this with John.
http://dpdk.org/ml/archives/dev/2016-January/031595.html


-- 
David Marchand


[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-06 Thread Thomas Monjalon
2016-08-05 15:09, Ferruh Yigit:
> All PCI device ids moved to drivers, it is safe to delete
> rte_pci_dev_ids.h

Good move. Thanks Ferruh!

> --- a/doc/guides/prog_guide/dev_kit_build_system.rst
> +++ b/doc/guides/prog_guide/dev_kit_build_system.rst
> @@ -93,20 +93,20 @@ Each build directory contains include files, libraries, 
> and applications:
>  cmdline_cirbuf.h   rte_cycles.h rte_memory.h
>  cmdline.h  rte_debug.h  rte_mempool.h
>  cmdline_parse_etheraddr.h  rte_eal.hrte_memzone.h
> -cmdline_parse.hrte_errno.h  rte_pci_dev_ids.h
> -cmdline_parse_ipaddr.h rte_ethdev.h rte_pci.h
> -cmdline_parse_num.hrte_ether.h  rte_per_lcore.h
> -cmdline_parse_portlist.h   rte_fbk_hash.h   rte_prefetch.h
> -cmdline_parse_string.h rte_hash_crc.h   rte_random.h
> -cmdline_rdline.h   rte_hash.h   rte_ring.h
> -cmdline_socket.h   rte_interrupts.h rte_rwlock.h
> -cmdline_vt100.hrte_ip.h rte_sctp.h
> -exec-env   rte_jhash.h  rte_spinlock.h
> -rte_alarm.hrte_launch.h rte_string_fns.h
> -rte_atomic.h   rte_lcore.h  rte_tailq.h
> -rte_branch_prediction.hrte_log.hrte_tcp.h
> -rte_byteorder.hrte_lpm.hrte_timer.h
> -rte_common.h   rte_malloc.h rte_udp.h
> +cmdline_parse.hrte_errno.h  rte_pci.h
> +cmdline_parse_ipaddr.h rte_ethdev.h rte_per_lcore.h
> +cmdline_parse_num.hrte_ether.h  rte_prefetch.h
> +cmdline_parse_portlist.h   rte_fbk_hash.h   rte_random.h
> +cmdline_parse_string.h rte_hash_crc.h   rte_ring.h
> +cmdline_rdline.h   rte_hash.h   rte_rwlock.h
> +cmdline_socket.h   rte_interrupts.h rte_sctp.h
> +cmdline_vt100.hrte_ip.h rte_spinlock.h
> +exec-env   rte_jhash.h  rte_string_fns.h
> +rte_alarm.hrte_launch.h rte_tailq.h
> +rte_atomic.h   rte_lcore.h  rte_tcp.h
> +rte_branch_prediction.hrte_log.hrte_timer.h
> +rte_byteorder.hrte_lpm.hrte_udp.h
> +rte_common.h   rte_malloc.h
>  rte_config.h   rte_mbuf.h

What do you think about this list? I would say we can remove it.


[dpdk-dev] [PATCH 3/3] eal: remove rte_pci_dev_ids.h

2016-08-05 Thread Ferruh Yigit
All PCI device ids moved to drivers, it is safe to delete
rte_pci_dev_ids.h

Signed-off-by: Ferruh Yigit 
---
 doc/guides/prog_guide/dev_kit_build_system.rst  | 28 +--
 lib/librte_eal/common/Makefile  |  2 +-
 lib/librte_eal/common/include/rte_pci_dev_ids.h | 63 -
 3 files changed, 15 insertions(+), 78 deletions(-)
 delete mode 100644 lib/librte_eal/common/include/rte_pci_dev_ids.h

diff --git a/doc/guides/prog_guide/dev_kit_build_system.rst 
b/doc/guides/prog_guide/dev_kit_build_system.rst
index fa2411f..e657e5f 100644
--- a/doc/guides/prog_guide/dev_kit_build_system.rst
+++ b/doc/guides/prog_guide/dev_kit_build_system.rst
@@ -93,20 +93,20 @@ Each build directory contains include files, libraries, and 
applications:
 cmdline_cirbuf.h   rte_cycles.h rte_memory.h
 cmdline.h  rte_debug.h  rte_mempool.h
 cmdline_parse_etheraddr.h  rte_eal.hrte_memzone.h
-cmdline_parse.hrte_errno.h  rte_pci_dev_ids.h
-cmdline_parse_ipaddr.h rte_ethdev.h rte_pci.h
-cmdline_parse_num.hrte_ether.h  rte_per_lcore.h
-cmdline_parse_portlist.h   rte_fbk_hash.h   rte_prefetch.h
-cmdline_parse_string.h rte_hash_crc.h   rte_random.h
-cmdline_rdline.h   rte_hash.h   rte_ring.h
-cmdline_socket.h   rte_interrupts.h rte_rwlock.h
-cmdline_vt100.hrte_ip.h rte_sctp.h
-exec-env   rte_jhash.h  rte_spinlock.h
-rte_alarm.hrte_launch.h rte_string_fns.h
-rte_atomic.h   rte_lcore.h  rte_tailq.h
-rte_branch_prediction.hrte_log.hrte_tcp.h
-rte_byteorder.hrte_lpm.hrte_timer.h
-rte_common.h   rte_malloc.h rte_udp.h
+cmdline_parse.hrte_errno.h  rte_pci.h
+cmdline_parse_ipaddr.h rte_ethdev.h rte_per_lcore.h
+cmdline_parse_num.hrte_ether.h  rte_prefetch.h
+cmdline_parse_portlist.h   rte_fbk_hash.h   rte_random.h
+cmdline_parse_string.h rte_hash_crc.h   rte_ring.h
+cmdline_rdline.h   rte_hash.h   rte_rwlock.h
+cmdline_socket.h   rte_interrupts.h rte_sctp.h
+cmdline_vt100.hrte_ip.h rte_spinlock.h
+exec-env   rte_jhash.h  rte_string_fns.h
+rte_alarm.hrte_launch.h rte_tailq.h
+rte_atomic.h   rte_lcore.h  rte_tcp.h
+rte_branch_prediction.hrte_log.hrte_timer.h
+rte_byteorder.hrte_lpm.hrte_udp.h
+rte_common.h   rte_malloc.h
 rte_config.h   rte_mbuf.h


diff --git a/lib/librte_eal/common/Makefile b/lib/librte_eal/common/Makefile
index f5ea0ee..bb9810d 100644
--- a/lib/librte_eal/common/Makefile
+++ b/lib/librte_eal/common/Makefile
@@ -34,7 +34,7 @@ include $(RTE_SDK)/mk/rte.vars.mk
 INC := rte_branch_prediction.h rte_common.h
 INC += rte_debug.h rte_eal.h rte_errno.h rte_launch.h rte_lcore.h
 INC += rte_log.h rte_memory.h rte_memzone.h rte_pci.h
-INC += rte_pci_dev_ids.h rte_per_lcore.h rte_random.h
+INC += rte_per_lcore.h rte_random.h
 INC += rte_tailq.h rte_interrupts.h rte_alarm.h
 INC += rte_string_fns.h rte_version.h
 INC += rte_eal_memconfig.h rte_malloc_heap.h
diff --git a/lib/librte_eal/common/include/rte_pci_dev_ids.h 
b/lib/librte_eal/common/include/rte_pci_dev_ids.h
deleted file mode 100644
index 1aae6f7..000
--- a/lib/librte_eal/common/include/rte_pci_dev_ids.h
+++ /dev/null
@@ -1,63 +0,0 @@
-/*-
- * This file is provided under a dual BSD/GPLv2 license.  When using or
- *   redistributing this file, you may do so under either license.
- *
- *   GPL LICENSE SUMMARY
- *
- *   Copyright(c) 2010-2015 Intel Corporation. All rights reserved.
- *
- *   This program is free software; you can redistribute it and/or modify
- *   it under the terms of version 2 of the GNU General Public License as
- *   published by the Free Software Foundation.
- *
- *   This program is distributed in the hope that it will be useful, but
- *   WITHOUT ANY WARRANTY; without even the implied warranty of
- *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- *   General Public License for more details.
- *
- *   You should have received a copy of the GNU General Public License
- *   along with this program; if not, write to the Free Software
- *   Foundation, Inc., 51 Franklin St - Fifth Floor, Boston, MA 02110-1301 USA.
- *   The full GNU General Public License is included in this distribution
- *   in the file called LICENSE.GPL.
- *
- *   Contact Information:
- *   Intel Corporation
- *
- *   BSD LICENSE
- *
- *   Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
- *   All rights reserved.
- *
- *   Redistribution and use in source and binary