[Intel-wired-lan] [PATCH S55 09/14] ice: remove unnecessary duplicated AQ command flag setting

Brelinski, TonyX tonyx.brelinski at intel.com
Thu Mar 11 00:33:30 UTC 2021


> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces at osuosl.org> On Behalf Of
> Tony Nguyen
> Sent: Tuesday, March 2, 2021 10:12 AM
> To: intel-wired-lan at lists.osuosl.org
> Subject: [Intel-wired-lan] [PATCH S55 09/14] ice: remove unnecessary
> duplicated AQ command flag setting
> 
> From: Bruce Allan <bruce.w.allan at intel.com>
> 
> Commit a012dca9f7a2 ("ice: add ethtool -m support for reading i2c eeprom
> modules") unnecessarily added the ICE_AQ_FLAG_BUF flag to the descriptor
> when sending the indirect Read/Write SFF EEPROM AQ command. The flag is
> already added later in the code flow for all indirect AQ commands, i.e.
> commands that provide an additional data buffer.
> 
> Signed-off-by: Bruce Allan <bruce.w.allan at intel.com>
> Signed-off-by: Tony Nguyen <anthony.l.nguyen at intel.com>
> ---
>  drivers/net/ethernet/intel/ice/ice_common.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Tested-by: Tony Brelinski <tonyx.brelinski at intel.com> A Contingent Worker at Intel




More information about the Intel-wired-lan mailing list