> -----Original Message----- > From: Intel-wired-lan <[email protected]> On Behalf Of Jesse > Brandeburg > Sent: Wednesday, December 6, 2023 6:31 AM > To: [email protected] > Cc: Kitszel, Przemyslaw <[email protected]>; Brandeburg, Jesse > <[email protected]>; Julia Lawall <[email protected]>; Lobakin, > Aleksander <[email protected]>; [email protected]; > [email protected]; [email protected] > Subject: [Intel-wired-lan] [PATCH iwl-next v2 06/15] ice: field prep > conversion > > Refactor ice driver to use FIELD_PREP(), which reduces lines of code > and adds clarity of intent. > > This code was generated by the following coccinelle/spatch script and > then manually repaired. > > Several places I changed to OR into a single variable with |= instead of > using a multi-line statement with trailing OR operators, as it > (subjectively) makes the code clearer. > > A local variable vmvf_and_timeout was created and used to avoid multiple > logical ORs being __le16 converted, which shortened some lines and makes > the code cleaner. > > Also clean up a couple of places where conversions were made to have the > code read more clearly/consistently. > > @prep2@ > constant shift,mask; > type T; > expression a; > @@ > -(((T)(a) << shift) & mask) > +FIELD_PREP(mask, a) > > @prep@ > constant shift,mask; > type T; > expression a; > @@ > -((T)((a) << shift) & mask) > +FIELD_PREP(mask, a) > > Cc: Julia Lawall <[email protected]> > CC: Alexander Lobakin <[email protected]> > Reviewed-by: Marcin Szycik <[email protected]> > Reviewed-by: Simon Horman <[email protected]> > Signed-off-by: Jesse Brandeburg <[email protected]> > --- > v2: added a couple more preps, some code cleanups found when looking for > le32_set/encode opportunities. > --- > drivers/net/ethernet/intel/ice/ice_base.c | 20 ++--- > drivers/net/ethernet/intel/ice/ice_common.c | 35 ++++----- > drivers/net/ethernet/intel/ice/ice_dcb.c | 3 +- > drivers/net/ethernet/intel/ice/ice_dcb_lib.c | 2 +- > drivers/net/ethernet/intel/ice/ice_eswitch.c | 4 +- > drivers/net/ethernet/intel/ice/ice_fdir.c | 69 ++++++----------- > .../net/ethernet/intel/ice/ice_flex_pipe.c | 8 +- > drivers/net/ethernet/intel/ice/ice_flow.c | 2 +- > drivers/net/ethernet/intel/ice/ice_lag.c | 7 +- > drivers/net/ethernet/intel/ice/ice_lib.c | 52 +++++-------- > drivers/net/ethernet/intel/ice/ice_ptp.c | 9 +-- > drivers/net/ethernet/intel/ice/ice_sriov.c | 38 ++++------ > drivers/net/ethernet/intel/ice/ice_switch.c | 75 +++++++++---------- > drivers/net/ethernet/intel/ice/ice_txrx.c | 6 +- > drivers/net/ethernet/intel/ice/ice_virtchnl.c | 8 +- > .../ethernet/intel/ice/ice_virtchnl_fdir.c | 2 +- > .../net/ethernet/intel/ice/ice_vsi_vlan_lib.c | 25 +++---- > 17 files changed, 147 insertions(+), 218 deletions(-) >
Tested-by: Pucha Himasekhar Reddy <[email protected]> (A Contingent worker at Intel) _______________________________________________ Intel-wired-lan mailing list [email protected] https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
