[Intel-wired-lan] [PATCH S18 10/15] ice: Separate if conditions for ice_set_features()

Bowers, AndrewX andrewx.bowers at intel.com
Fri Apr 26 20:36:57 UTC 2019


> -----Original Message-----
> From: Intel-wired-lan [mailto:intel-wired-lan-bounces at osuosl.org] On
> Behalf Of Anirudh Venkataramanan
> Sent: Tuesday, April 16, 2019 10:21 AM
> To: intel-wired-lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH S18 10/15] ice: Separate if conditions for
> ice_set_features()
> 
> From: Tony Nguyen <anthony.l.nguyen at intel.com>
> 
> Set features can have multiple features turned on|off in a single call.
> Grouping these all in an if/else means after one condition is met, other
> conditions/features will not be evaluated.  Break the if/else statements by
> feature to ensure all features will be handled properly.
> 
> Signed-off-by: Tony Nguyen <anthony.l.nguyen at intel.com>
> Signed-off-by: Anirudh Venkataramanan
> <anirudh.venkataramanan at intel.com>
> ---
>  drivers/net/ethernet/intel/ice/ice_main.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)

Tested-by: Andrew Bowers <andrewx.bowers at intel.com>




More information about the Intel-wired-lan mailing list