diff mbox series

[v2,net-next] ice: Remove unused ICE_FLOW_SEG_HDRS_L2_MASK

Message ID 20211117235505.70783-1-anthony.l.nguyen@intel.com
State Accepted
Delegated to: Anthony Nguyen
Headers show
Series [v2,net-next] ice: Remove unused ICE_FLOW_SEG_HDRS_L2_MASK | expand

Commit Message

Tony Nguyen Nov. 17, 2021, 11:55 p.m. UTC
Remove the unused define ICE_FLOW_SEG_HDRS_L2_MASK.

Reported-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
---
v2: Reword title and description to be imperative

 drivers/net/ethernet/intel/ice/ice_flow.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Paul Menzel Nov. 18, 2021, 8:32 a.m. UTC | #1
Dear Tony,


Am 18.11.21 um 00:55 schrieb Tony Nguyen:
> Remove the unused define ICE_FLOW_SEG_HDRS_L2_MASK.
> 
> Reported-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
> ---
> v2: Reword title and description to be imperative
> 
>   drivers/net/ethernet/intel/ice/ice_flow.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/ice/ice_flow.c b/drivers/net/ethernet/intel/ice/ice_flow.c
> index 2bfe9d9d7edc..ef10f0941231 100644
> --- a/drivers/net/ethernet/intel/ice/ice_flow.c
> +++ b/drivers/net/ethernet/intel/ice/ice_flow.c
> @@ -609,8 +609,6 @@ struct ice_flow_prof_params {
>   	ICE_FLOW_SEG_HDR_ESP | ICE_FLOW_SEG_HDR_AH | \
>   	ICE_FLOW_SEG_HDR_NAT_T_ESP)
>   
> -#define ICE_FLOW_SEG_HDRS_L2_MASK	\
> -	(ICE_FLOW_SEG_HDR_ETH | ICE_FLOW_SEG_HDR_VLAN)
>   #define ICE_FLOW_SEG_HDRS_L3_MASK	\
>   	(ICE_FLOW_SEG_HDR_IPV4 | ICE_FLOW_SEG_HDR_IPV6 | ICE_FLOW_SEG_HDR_ARP)
>   #define ICE_FLOW_SEG_HDRS_L4_MASK	\

Acked-by: Paul Menzel <pmenzel@molgen.mpg.de>


Kind regards,

Paul
G, GurucharanX Nov. 24, 2021, 12:42 p.m. UTC | #2
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of Tony
> Nguyen
> Sent: Thursday, November 18, 2021 5:25 AM
> To: intel-wired-lan@lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH v2 net-next] ice: Remove unused
> ICE_FLOW_SEG_HDRS_L2_MASK
> 
> Remove the unused define ICE_FLOW_SEG_HDRS_L2_MASK.
> 
> Reported-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
> ---
> v2: Reword title and description to be imperative
> 
>  drivers/net/ethernet/intel/ice/ice_flow.c | 2 --
>  1 file changed, 2 deletions(-)
> 

Tested-by: Gurucharan G <gurucharanx.g@intel.com> (A Contingent worker at Intel)
diff mbox series

Patch

diff --git a/drivers/net/ethernet/intel/ice/ice_flow.c b/drivers/net/ethernet/intel/ice/ice_flow.c
index 2bfe9d9d7edc..ef10f0941231 100644
--- a/drivers/net/ethernet/intel/ice/ice_flow.c
+++ b/drivers/net/ethernet/intel/ice/ice_flow.c
@@ -609,8 +609,6 @@  struct ice_flow_prof_params {
 	ICE_FLOW_SEG_HDR_ESP | ICE_FLOW_SEG_HDR_AH | \
 	ICE_FLOW_SEG_HDR_NAT_T_ESP)
 
-#define ICE_FLOW_SEG_HDRS_L2_MASK	\
-	(ICE_FLOW_SEG_HDR_ETH | ICE_FLOW_SEG_HDR_VLAN)
 #define ICE_FLOW_SEG_HDRS_L3_MASK	\
 	(ICE_FLOW_SEG_HDR_IPV4 | ICE_FLOW_SEG_HDR_IPV6 | ICE_FLOW_SEG_HDR_ARP)
 #define ICE_FLOW_SEG_HDRS_L4_MASK	\