Hi Wei,
> -----Original Message-----
> From: Zhao1, Wei
> Sent: Friday, June 2, 2017 2:36 PM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo; Zhao1, Wei
> Subject: [PATCH v2 06/11] net/e1000: parse ethertype filter
>
> check if the rule is a ethertype rule, and get the ethertype info.
>
> Signed-off-by: Wei Zhao <wei.zh...@intel.com>
> ---
> drivers/net/e1000/igb_flow.c | 280
> +++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 280 insertions(+)
> +
> + /* The first non-void item should be MAC. */
> + item = pattern + index;
> + while (item->type == RTE_FLOW_ITEM_TYPE_VOID) {
> + index++;
> + item = pattern + index;
> + }
You've created a macro to do it in the previous patch, why not use the macro?
The same below.
