igc: Rename qbv_enable to taprio_offload_enable
authorFlorian Kauer <florian.kauer@linutronix.de>
Wed, 14 Jun 2023 14:07:09 +0000 (16:07 +0200)
committerTony Nguyen <anthony.l.nguyen@intel.com>
Mon, 10 Jul 2023 15:40:22 +0000 (08:40 -0700)
commit8046063df887bee35c002224267ba46f41be7cf6
tree665b88b7e55e0d34f420247badc5a226f37b974e
parent9d0aba98316d00f9c0a4506fc15f5ed9241bc1fd
igc: Rename qbv_enable to taprio_offload_enable

In the current implementation the flags adapter->qbv_enable
and IGC_FLAG_TSN_QBV_ENABLED have a similar name, but do not
have the same meaning. The first one is used only to indicate
taprio offload (i.e. when igc_save_qbv_schedule was called),
while the second one corresponds to the Qbv mode of the hardware.
However, the second one is also used to support the TX launchtime
feature, i.e. ETF qdisc offload. This leads to situations where
adapter->qbv_enable is false, but the flag IGC_FLAG_TSN_QBV_ENABLED
is set. This is prone to confusion.

The rename should reduce this confusion. Since it is a pure
rename, it has no impact on functionality.

Fixes: e17090eb2494 ("igc: allow BaseTime 0 enrollment for Qbv")
Signed-off-by: Florian Kauer <florian.kauer@linutronix.de>
Reviewed-by: Kurt Kanzenbach <kurt@linutronix.de>
Tested-by: Naama Meir <naamax.meir@linux.intel.com>
Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>
drivers/net/ethernet/intel/igc/igc.h
drivers/net/ethernet/intel/igc/igc_main.c
drivers/net/ethernet/intel/igc/igc_tsn.c