> -----Original Message----- > From: Intel-wired-lan <[email protected]> On Behalf Of Jacob > Keller > Sent: Monday, May 5, 2025 10:14 PM > To: Intel Wired LAN <[email protected]>; Nguyen, Anthony L > <[email protected]>; netdev <[email protected]> > Cc: Keller, Jacob E <[email protected]>; Kitszel, Przemyslaw > <[email protected]>; Loktionov, Aleksandr > <[email protected]>; Simon Horman <[email protected]> > 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 <[email protected]> > Reviewed-by: Aleksandr Loktionov <[email protected]> > Reviewed-by: Przemek Kitszel <[email protected]> > Reviewed-by: Simon Horman <[email protected]> > Signed-off-by: Jacob Keller <[email protected]>
Tested-by: Rafal Romanowski <[email protected]>
