Re: [PATCH v6 4/7] iwlwifi: integrate with iwlmei

[Kalle Valo <kvalo@codeaurora.org>]

Emmanuel Grumbach <emmanuel.grumbach@intel.com> writes:

> iwlmei needs to know about the follwing events:
>
> * Association
> * De-association
> * Country Code change
> * SW Rfkill change
> * SAR table changes
>
> iwlmei can take the device away from us, so report the new
> rfkill type when this happens.
> Advertise the required data from the CSME firmware to the
> usersapce: mostly, the AP that the CSME firmware is currently
> associated to in case there is an active link protection
> session.
> Generate the HOST_ASSOC / HOST_DISSASSOC messages.
>
> Don't support WPA1 (non-RSNA) for now.
> Don't support shared wep either.
> We can then determine the AUTH parameter by checking the AKM.
> Feed the cipher from the key installation.
>
> SW Rfkill will be implemented later when cfg80211 will
> allow us to read the SW Rfkill state.
>
> Co-Developed-by: Ayala Beker <ayala.beker@intel.com>
> Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>

[...]

> @@ -1932,6 +1952,17 @@ void iwl_mvm_enter_ctkill(struct iwl_mvm *mvm);
>  int iwl_mvm_send_temp_report_ths_cmd(struct iwl_mvm *mvm);
>  int iwl_mvm_ctdp_command(struct iwl_mvm *mvm, u32 op, u32 budget);
>  
> +#ifdef CONFIG_IWLMVM_VENDOR_CMDS
> +
> +/* vendor commands */
> +void iwl_mvm_vendor_cmds_register(struct iwl_mvm *mvm);
> +
> +#else // IWLMVM_VENDOR_CMDS
> +
> +static inline void iwl_mvm_vendor_cmds_register(struct iwl_mvm *mvm) {}
> +
> +#endif // IWLMVM_VENDOR_CMDS

Shouldn't this be in patch 6?

And no C++ style comments, please.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches