[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <IA1PR11MB6241346475B29A59D4FDA6638B61A@IA1PR11MB6241.namprd11.prod.outlook.com>
Date: Fri, 30 May 2025 06:52:33 +0000
From: "Rinitha, SX" <sx.rinitha@...el.com>
To: "Keller, Jacob E" <jacob.e.keller@...el.com>, Intel Wired LAN
<intel-wired-lan@...ts.osuosl.org>, "Nguyen, Anthony L"
<anthony.l.nguyen@...el.com>, netdev <netdev@...r.kernel.org>
CC: "Keller, Jacob E" <jacob.e.keller@...el.com>, "Kitszel, Przemyslaw"
<przemyslaw.kitszel@...el.com>, "Loktionov, Aleksandr"
<aleksandr.loktionov@...el.com>, Simon Horman <horms@...nel.org>
Subject: RE: [Intel-wired-lan] [PATCH iwl-next v2 1/2] net: intel: rename
'hena' to 'hashcfg' for clarity
> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@...osl.org> On Behalf Of Jacob Keller
> Sent: 06 May 2025 01:44
> To: Intel Wired LAN <intel-wired-lan@...ts.osuosl.org>; Nguyen, Anthony L <anthony.l.nguyen@...el.com>; netdev <netdev@...r.kernel.org>
> Cc: Keller, Jacob E <jacob.e.keller@...el.com>; Kitszel, Przemyslaw <przemyslaw.kitszel@...el.com>; Loktionov, Aleksandr <aleksandr.loktionov@...el.com>; Simon Horman <horms@...nel.org>
> Subject: [Intel-wired-lan] [PATCH iwl-next v2 1/2] net: intel: rename 'hena' to 'hashcfg' for clarity
>
> i40e, ice, and iAVF all use 'hena' as a shorthand for the "hash enable"
configuration. This comes originally from the X710 datasheet 'xxQF_HENA'
registers. In the context of the registers the meaning is fairly clear.
>
> However, on its own, hena is a weird name that can be more difficult to understand. This is especially true in ice. The E810 hardware doesn't even have registers with HENA in the name.
>
> Replace the shorthand 'hena' with 'hashcfg'. This makes it clear the variables deal with the Hash configuration, not just a single boolean on/off for all hashing.
>
> Do not update the register names. These come directly from the datasheet for X710 and X722, and it is more important that the names can be searched.
>
> Suggested-by: Przemek Kitszel <przemyslaw.kitszel@...el.com>
> Reviewed-by: Aleksandr Loktionov <aleksandr.loktionov@...el.com>
> Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@...el.com>
> Reviewed-by: Simon Horman <horms@...nel.org>
> Signed-off-by: Jacob Keller <jacob.e.keller@...el.com>
> ---
> drivers/net/ethernet/intel/i40e/i40e_txrx.h | 8 ++--
> drivers/net/ethernet/intel/iavf/iavf.h | 10 ++---
> drivers/net/ethernet/intel/iavf/iavf_txrx.h | 4 +-
> drivers/net/ethernet/intel/ice/ice_flow.h | 4 +-
> drivers/net/ethernet/intel/ice/ice_virtchnl.h | 4 +-
> include/linux/avf/virtchnl.h | 22 +++++------
> drivers/net/ethernet/intel/i40e/i40e_main.c | 2 +-
> drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 46 +++++++++++-----------
> drivers/net/ethernet/intel/iavf/iavf_main.c | 17 ++++----
> drivers/net/ethernet/intel/iavf/iavf_virtchnl.c | 33 ++++++++--------
> drivers/net/ethernet/intel/ice/ice_lib.c | 2 +-
> drivers/net/ethernet/intel/ice/ice_virtchnl.c | 44 ++++++++++-----------
> .../ethernet/intel/ice/ice_virtchnl_allowlist.c | 2 +-
> 13 files changed, 101 insertions(+), 97 deletions(-)
>
Tested-by: Rinitha S <sx.rinitha@...el.com> (A Contingent worker at Intel)
Powered by blists - more mailing lists