On 2018-09-04 19:43, Kalle Valo wrote:
Wen Gong <[email protected]> writes:

> +enum wmi_nlo_cipher_algorithm {
> +  WMI_NLO_CIPHER_ALGO_NONE           = 0x00,
> +  WMI_NLO_CIPHER_ALGO_WEP40          = 0x01,
> +  WMI_NLO_CIPHER_ALGO_TKIP           = 0x02,
> +  WMI_NLO_CIPHER_ALGO_CCMP           = 0x04,
> +  WMI_NLO_CIPHER_ALGO_WEP104         = 0x05,
> +  WMI_NLO_CIPHER_ALGO_BIP            = 0x06,
> +  WMI_NLO_CIPHER_ALGO_WPA_USE_GROUP  = 0x100,
> +  WMI_NLO_CIPHER_ALGO_RSN_USE_GROUP  = 0x100,

Two defines with the same value 0x100, is this really correct?

Yes, it is an interface between host driver /firmware.

Yeah, I know it's an interface between the host driver and the firmware
but it does not make any sense and is just confusing. Either of them
should be dropped.
yes, either of them can be dropped.

_______________________________________________
ath10k mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/ath10k

Reply via email to