[dpdk-dev,[PATCH,v2] 04/13] virtio_pci: use rte_io.h for non-x86 arch

Message ID 1450098032-21198-5-git-send-email-sshukla@mvista.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Santosh Shukla Dec. 14, 2015, 1 p.m. UTC
  Use rte_io.h for non-x86 arch.

Signed-off-by: Santosh Shukla <sshukla@mvista.com>
---
 drivers/net/virtio/virtio_pci.h |    4 ++++
 1 file changed, 4 insertions(+)
  

Comments

Jerin Jacob Dec. 14, 2015, 2:28 p.m. UTC | #1
On Mon, Dec 14, 2015 at 06:30:23PM +0530, Santosh Shukla wrote:
> Use rte_io.h for non-x86 arch.
> 
> Signed-off-by: Santosh Shukla <sshukla@mvista.com>
> ---
>  drivers/net/virtio/virtio_pci.h |    4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/net/virtio/virtio_pci.h b/drivers/net/virtio/virtio_pci.h
> index 47f722a..3f4ff80 100644
> --- a/drivers/net/virtio/virtio_pci.h
> +++ b/drivers/net/virtio/virtio_pci.h
> @@ -40,7 +40,11 @@
>  #include <sys/types.h>
>  #include <machine/cpufunc.h>
>  #else
> +#if defined(RTE_ARCH_X86_64) || defined(RTE_ARCH_I686)
>  #include <sys/io.h>
> +#else
> +#include <rte_io.h>

I believe this patch can be squashed with patch 08/13

> +#endif
>  #endif
>  
>  #include <rte_ethdev.h>
> -- 
> 1.7.9.5
>
  
Santosh Shukla Dec. 14, 2015, 3:29 p.m. UTC | #2
On Mon, Dec 14, 2015 at 7:58 PM, Jerin Jacob
<jerin.jacob@caviumnetworks.com> wrote:
> On Mon, Dec 14, 2015 at 06:30:23PM +0530, Santosh Shukla wrote:
>> Use rte_io.h for non-x86 arch.
>>
>> Signed-off-by: Santosh Shukla <sshukla@mvista.com>
>> ---
>>  drivers/net/virtio/virtio_pci.h |    4 ++++
>>  1 file changed, 4 insertions(+)
>>
>> diff --git a/drivers/net/virtio/virtio_pci.h b/drivers/net/virtio/virtio_pci.h
>> index 47f722a..3f4ff80 100644
>> --- a/drivers/net/virtio/virtio_pci.h
>> +++ b/drivers/net/virtio/virtio_pci.h
>> @@ -40,7 +40,11 @@
>>  #include <sys/types.h>
>>  #include <machine/cpufunc.h>
>>  #else
>> +#if defined(RTE_ARCH_X86_64) || defined(RTE_ARCH_I686)
>>  #include <sys/io.h>
>> +#else
>> +#include <rte_io.h>
>
> I believe this patch can be squashed with patch 08/13
>

Yes. Thanks.
>> +#endif
>>  #endif
>>
>>  #include <rte_ethdev.h>
>> --
>> 1.7.9.5
>>
  

Patch

diff --git a/drivers/net/virtio/virtio_pci.h b/drivers/net/virtio/virtio_pci.h
index 47f722a..3f4ff80 100644
--- a/drivers/net/virtio/virtio_pci.h
+++ b/drivers/net/virtio/virtio_pci.h
@@ -40,7 +40,11 @@ 
 #include <sys/types.h>
 #include <machine/cpufunc.h>
 #else
+#if defined(RTE_ARCH_X86_64) || defined(RTE_ARCH_I686)
 #include <sys/io.h>
+#else
+#include <rte_io.h>
+#endif
 #endif
 
 #include <rte_ethdev.h>