qcacld-3.0: Refine target_if_ext_twt_evt.c
Refine target_if_ext_twt_evt.c to remove duplicate conditional checks. Also fix some code style errors by adding a blank line return to last return in the API Change-Id: I759becd864293b8bec35cb54763bdf4b047537c2 CRs-Fixed: 3402852
This commit is contained in:

committed by
Madan Koyyalamudi

parent
38dbf0dd0c
commit
26ab632e89
@@ -80,6 +80,7 @@ target_if_twt_setup_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(data);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
@@ -129,6 +130,7 @@ target_if_twt_teardown_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(data);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
|
||||
}
|
||||
@@ -187,6 +189,7 @@ target_if_twt_pause_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(param);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
@@ -244,6 +247,7 @@ target_if_twt_resume_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(param);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
@@ -301,6 +305,7 @@ target_if_twt_nudge_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(param);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
@@ -349,8 +354,8 @@ target_if_twt_notify_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(data);
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
static int
|
||||
@@ -398,6 +403,7 @@ target_if_twt_ack_complete_event_handler(ol_scn_t scn, uint8_t *event,
|
||||
|
||||
done:
|
||||
qdf_mem_free(data);
|
||||
|
||||
return qdf_status_to_os_return(qdf_status);
|
||||
}
|
||||
|
||||
@@ -407,11 +413,6 @@ target_if_twt_register_ext_events(struct wlan_objmgr_psoc *psoc)
|
||||
QDF_STATUS status;
|
||||
struct wmi_unified *wmi_handle;
|
||||
|
||||
if (!psoc) {
|
||||
target_if_err("psoc obj is null!");
|
||||
return QDF_STATUS_E_NULL_VALUE;
|
||||
}
|
||||
|
||||
wmi_handle = get_wmi_unified_hdl_from_psoc(psoc);
|
||||
if (!wmi_handle) {
|
||||
target_if_err("wmi_handle is null!");
|
||||
@@ -497,11 +498,6 @@ target_if_twt_deregister_ext_events(struct wlan_objmgr_psoc *psoc)
|
||||
QDF_STATUS status;
|
||||
struct wmi_unified *wmi_handle;
|
||||
|
||||
if (!psoc) {
|
||||
target_if_err("psoc is NULL!");
|
||||
return QDF_STATUS_E_INVAL;
|
||||
}
|
||||
|
||||
wmi_handle = get_wmi_unified_hdl_from_psoc(psoc);
|
||||
if (!wmi_handle) {
|
||||
target_if_err("wmi_handle is null");
|
||||
|
Reference in New Issue
Block a user