On 10/19/2023 12:48 PM, Ivan Vecera wrote: > Dne čt 19. 10. 2023 21:24 uživatel Jacob Keller <[email protected]> > napsal: > >> >> >> On 10/18/2023 5:30 AM, Przemek Kitszel wrote: >>> On 10/18/23 13:26, Ivan Vecera wrote: >>>> Commit c87c938f62d8f1 ("i40e: Add VF VLAN pruning") added new >>>> PF flag I40E_FLAG_VF_VLAN_PRUNING but its value collides with >>>> existing I40E_FLAG_TOTAL_PORT_SHUTDOWN_ENABLED flag. >>>> >>>> Move the affected flag at the end of the flags and fix its value. >>>> >>>> Cc: Mateusz Palczewski <[email protected]> >>>> Signed-off-by: Ivan Vecera <[email protected]> >>>> --- >>>> drivers/net/ethernet/intel/i40e/i40e.h | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/drivers/net/ethernet/intel/i40e/i40e.h >> b/drivers/net/ethernet/intel/i40e/i40e.h >>>> index 6e310a53946782..55bb0b5310d5b4 100644 >>>> --- a/drivers/net/ethernet/intel/i40e/i40e.h >>>> +++ b/drivers/net/ethernet/intel/i40e/i40e.h >>>> @@ -580,7 +580,6 @@ struct i40e_pf { >>>> #define I40E_FLAG_DISABLE_FW_LLDP BIT(24) >>>> #define I40E_FLAG_RS_FEC BIT(25) >>>> #define I40E_FLAG_BASE_R_FEC BIT(26) >>>> -#define I40E_FLAG_VF_VLAN_PRUNING BIT(27) >>>> /* TOTAL_PORT_SHUTDOWN >>>> * Allows to physically disable the link on the NIC's port. >>>> * If enabled, (after link down request from the OS) >>>> @@ -603,6 +602,7 @@ struct i40e_pf { >>> >>> such mistake happened only because list of flags is dispersed so much :/ >> >> Better yet if we didn't hard-code the bits, and instead defined them via >> an enumeration so that its not possible :D These aren't even ABI so >> there's not a backwards compatibility risk either. >> >> Thanks, >> Jake >> > > Hi Jake, > I have been preparing another series for iwl-next that covers this > conversion. I will submit it tomorrow or on the weekend. > > Ivan >
Great, thanks! -Jake >> > _______________________________________________ Intel-wired-lan mailing list [email protected] https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
