Re: [dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-20 Thread Jerin Jacob
On Thu, Mar 19, 2020 at 1:41 PM David Marchand wrote: > > On Mon, Mar 16, 2020 at 8:08 PM David Christensen > wrote: > > On 3/16/20 10:49 AM, David Christensen wrote: > > > > > >> The rte_pci.h file includes more header files than are actually needed, > > >> which means that all users of it also

Re: [dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-19 Thread David Marchand
On Mon, Mar 16, 2020 at 8:08 PM David Christensen wrote: > On 3/16/20 10:49 AM, David Christensen wrote: > > > >> The rte_pci.h file includes more header files than are actually needed, > >> which means that all users of it also include those headers. This patch > >> removes the unneeded headers -

Re: [dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-16 Thread David Christensen
On 3/16/20 10:49 AM, David Christensen wrote: The rte_pci.h file includes more header files than are actually needed, which means that all users of it also include those headers. This patch removes the unneeded headers - adding them elsewhere where other components were requiring them but

Re: [dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-16 Thread David Christensen
The rte_pci.h file includes more header files than are actually needed, which means that all users of it also include those headers. This patch removes the unneeded headers - adding them elsewhere where other components were requiring them but not including them directly. Fixes: c752998b5e2e (

Re: [dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-16 Thread Thomas Monjalon
16/03/2020 12:29, Bruce Richardson: > The rte_pci.h file includes more header files than are actually needed, > which means that all users of it also include those headers. This patch > removes the unneeded headers - adding them elsewhere where other components > were requiring them but not includi

[dpdk-dev] [PATCH] pci: fix unneeded includes in public header file

2020-03-16 Thread Bruce Richardson
The rte_pci.h file includes more header files than are actually needed, which means that all users of it also include those headers. This patch removes the unneeded headers - adding them elsewhere where other components were requiring them but not including them directly. Fixes: c752998b5e2e ("pci