2019-04-04 12:11:15

by Rakesh Pillai

[permalink] [raw]
Subject: Re: FW: [PATCH v2] ath10k: Report low ack rssi based on the reason code

>> diff --git a/drivers/net/wireless/ath/ath10k/wmi.h
>> b/drivers/net/wireless/ath/ath10k/wmi.h
>> index e1c40bb..3ccd79e 100644
>> --- a/drivers/net/wireless/ath/ath10k/wmi.h
>> +++ b/drivers/net/wireless/ath/ath10k/wmi.h
>> @@ -6797,6 +6797,8 @@ struct wmi_vdev_start_ev_arg {
>>
>> struct wmi_peer_kick_ev_arg {
>> const u8 *mac_addr;
>> + u32 reason;
>> + bool reason_code_valid;
>> };
>>
> Adding extra members to this structure breaks structure consistency
> between FW and host driver since FW doesn't have such members.
>
> Peter
> _______________________________________________
> ath10k mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/ath10k

struct wmi_peer_sta_kickout_event is used for parsing event from non-tlv
firmwares and
struct wmi_tlv_peer_sta_kickout_event is used for parsing event from
tlv firmwares.
Both of the above structures maintain consistency with the respective
FW.

struct wmi_peer_kick_ev_arg is used only for internal ath10k operations
only.

Thanks,
Rakesh Pillai.