[dpdk-dev,3/5] net/virtio: remove a redundant macro definition for ctrl vq
Checks
Commit Message
VIRTIO_NET_CTRL_MAC_ADDR_SET is defined two times in
virtqueue.h, the second one is obviously not wanted.
Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
---
drivers/net/virtio/virtqueue.h | 2 --
1 file changed, 2 deletions(-)
Comments
On 12/07/2017 06:30 AM, Tiwei Bie wrote:
> VIRTIO_NET_CTRL_MAC_ADDR_SET is defined two times in
> virtqueue.h, the second one is obviously not wanted.
>
> Signed-off-by: Tiwei Bie <tiwei.bie@intel.com>
> ---
> drivers/net/virtio/virtqueue.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/net/virtio/virtqueue.h b/drivers/net/virtio/virtqueue.h
> index ab466c2db..eaf9de13e 100644
> --- a/drivers/net/virtio/virtqueue.h
> +++ b/drivers/net/virtio/virtqueue.h
> @@ -226,8 +226,6 @@ struct virtqueue {
> #define VIRTIO_NET_CTRL_MQ_VQ_PAIRS_MIN 1
> #define VIRTIO_NET_CTRL_MQ_VQ_PAIRS_MAX 0x8000
>
> -#define VIRTIO_NET_CTRL_MAC_ADDR_SET 1
> -
> /**
> * This is the first element of the scatter-gather list. If you don't
> * specify GSO or CSUM features, you can simply ignore the header.
>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
Thanks,
Maxime
@@ -226,8 +226,6 @@ struct virtqueue {
#define VIRTIO_NET_CTRL_MQ_VQ_PAIRS_MIN 1
#define VIRTIO_NET_CTRL_MQ_VQ_PAIRS_MAX 0x8000
-#define VIRTIO_NET_CTRL_MAC_ADDR_SET 1
-
/**
* This is the first element of the scatter-gather list. If you don't
* specify GSO or CSUM features, you can simply ignore the header.