Presently, htt_frag_desc is not getting zeroed out explicitly.
So fw can take the invalid/garbage data in the frag information
and treat it as a valid address which may lead to crash.
Memzero htt_frag_desc before use.
Change-Id: I9dadcb883a65c43f96e810a12b4fb491c3a2e734
CRs-Fixed: 2088448
When no channel is selected from ACS and fallback channel
is invalid, a default channel is selected but the channel
width remains invalid due to which ASSERT is happening.
Whenever default channel is selected, select default channel
width of 20 MHz as well.
Change-Id: I885e01d1324484b84e04675238d2e1f8cd10e30c
CRS-Fixed: 2112806
Add logic to mark first wakeup packet even if offload bit is not set in
the htt rx indication message sent by fw.
Change-Id: Ide50dcfcf88ecd0c71b32b0e746742dc49d08567
CRs-Fixed: 2110627
Add ini items for current and candidate rssi thresholds used in
mbo bss transition context.
Change-Id: I834c87aaaf776cdc293718fb994b4aae38af6a8b
CRs-Fixed: 2007107
During initialization, bug report lock is initialized after logger
thread is created and during deinitialization, bug report lock is
destroyed before logger thread exits. As logger thread uses this
lock there is a possibility of logger thread to access uninitialized
lock.
To mitigate this issue initialize/deinitialize lock after/before
creating logging thread.
Change-Id: I973c9b6c7eed38cbbc09258a54e587078ddb37e1
CRs-Fixed: 2094510
Device crashed while accessing the unintialized
memory as part of scan request processing.
Initialize the scan params structure to zero.
Change-Id: Iaf430c6edb485a009f8d932ce0c2d033b9ec6137
CRs-Fixed: 2116256
Moved unnecessary kernel message to debug logs where ever not
required in kernel logs
Change-Id: If7b69fbdc4afea4597d38a44f786ee221ee917b2
CRs-Fixed: 2042092
As part of start_ap new beacon memory is allocated and filled with
the parameters sent by the upper layer. If there is any failure
during the start the bss, the memory is not freed resulting in
the leak.
Free the allocated beacon memory if there is any failure in starting
the bss.
Change-Id: Idc263ffbb352e56d65d397b200facb23b7ab207c
CRs-fixed: 2033325
qcacld-2.0 to qcacld-3.0 propagation
Cleanup target dump code and add support to dump IRAM region
for different platforms. Update the Target Memory Region for
different platforms.
Change-Id: Ie29fb62e0f1bc279311f77400e8be490ccf987a4
CRs-Fixed: 2088394
In LFR3 roaming, session->nss is recomputed by starting with mac_ctx
configured value, then lim_fill_ft_session() trims it based on
AP's capability in the beacon. Then lim_ft_prepare_add_bss_req() modifies
it based on dot11mode if necessary.
CRs-Fixed: 2082240
Change-Id: If9841ea8d10cc1269ec118c9e7f75fa0444abe3e
Currently logs are printed from HDD during scan and connect for every
retry, which is causing MCThread to get stuck at printk and crash
Rate limit the hdd logs to print for every 5 retries to avoid filling
the log buffer. Also change some error logs to debug level.
Change-Id: I09ad38cd89bbd20081673142bf5a350c55b0f005
CRs-Fixed: 2090761
This is qcacld-2.0 to qcacld-3.0 propagation
In get_container_ies_len size type for len is uint8_t.
len copies values from pBufRemaining.
There can be chance for integer overflow.
To avoid that make size type for len as uint32_t.
Change-Id: I305321a6631719808ef213571974ae23b0e61bb3
CRs-Fixed: 2064580
With the latest logging guidelines info logs will be routed to
console there are lot of redundant logs causing the watchdog
bark during driver initialization.
Reduce the loglevel from info to debug so these will be logged via
Loggerapp.
CRs-Fixed: 2040730
Change-Id: I2f073e3ab2fe9c129867cadd54c9f27a6b0e65c8
In hdd_send_re_assoc_event api to get ieee80211_channel,
channel number is passed to ieee80211_get_channel kernel
API which expect frequency as an argument.
Pass frequency to ieee80211_get_channel API instead of
channel number.
CRs-fixed: 2053536
Change-Id: I216889d13252c56e817c4e09e77459cc38f8e9ec
Currently hdd_clear_fils_connection_info is called for all device
modes which is leading to crash in SAP mode due to illegal memory
access.
Add check to invoke hdd_clear_fils_connection_info only if the
device mode is STA.
Change-Id: Ia0e28c29d809865169c254b0d45013aad39e13d2
CRs-Fixed: 2106324
Currently, Host performs validation of NLA attribute
QCA_WLAN_VENDOR_ATTR_OCB_SET_CONFIG_CHANNEL_ARRAY twice
which leads to memory leak.
As a part of fix, Check for SET_CONFIG_CHANNEL_ARRAY
only once and free memory in error scenario.
Change-Id: I5c90c937682417b8181fda7d499908b9425d4c6a
CRs-Fixed: 2103954
When host receives roam sync indication from firmware
and if state in PE is not equal to eLIM_SME_LINK_EST_STATE
then roam is aborted. As part of eCSR_ROAM_FT_START,
ft_carrier_on flag is set to true but is not set to false
as part of roam abort handling in hdd_sme_roam_callback api.
This results in not sending subsequent connect result to nl.
Fix is to set ft_carrier_on to false on roam abort.
Change-Id: I43c65730fd439145c22dbf77b0deb47a1bf2ef45
CRs-Fixed: 2105072
Currently, Host configures the NS entries through vendor command
where host do not check the active offload is enable or disabled.
Hence it is not configuring according to active offload ini.
As part of fix, Cache the vendor command value and check whether
active offload is enabled or not before configuration to fwr.
Change-Id: Icb11e43198a75975418a2c9ed95d5f1df05ce0c5
CRs-fixed: 2105071
NULL constant is unsuitable for sapEvent parameter since following
may be called:
sapEventInit(sapEvent);
vosStatus = sapFsm(sapContext, sapEvent);
Change-Id: I2ffc1be27f96bcbd793b2302e6c5920b5d30073b
CRs-Fixed: 2111290
Presently in the sta mode locally administered is set and it
is not logged with new macaddress.
Print the macaddress of the STA once the LA bit is reset.
Change-Id: I5ab5c67c622723f32500bc07d815efb02bd834a7
CRs-Fixed: 2069219
Currently the mpdu_data_len in Rx pkt meta is not checked for
upper bound in wma_form_rx_packet.
Add sanity check to drop the packet if mpdu_data_len is
greater than 2000 bytes. Also add upper bound check for
frame_len in lim_process_auth_frame function.
Change-Id: I7ab454045e2f6d278351dcabde6da556f9f741e0
CRs-Fixed: 2093392
SIR_MAC_AUTH_CHALLENGE_LENGTH is updated to 253 from
128 as per IEEE spec due to connection fails between
DUT-SAP and old ref-STA. Auth failure occurs as encrypted
data sent by ref-STA is only 128 bytes instead of 253
bytes.
Fix is to set length of challenge text sent by SAP
to 128 bytes.
Change-Id: I81409bb58ad34e469c54e2909f45b8a6826eb06a
CRs-Fixed: 2096512
In function ol_configure_target cap is used uninitialized when call
pld_get_platform_cap fail, although pld_get_bus_type and
hif_get_bus_type return same bus type, so actually it has no issue here,
but it look not clear here, init it explicitly.
Change-Id: I001956521c62c14b3537b399bbca1791b274a4c7
CRs-Fixed: 2115553
In a noisy enviornment assoc indication in SME queueu
is not getting processed due to multiple beacons/probes
getting processed in PE queue which is high priority
than SME queue.
Instead of posting a message to SME queue, assoc indication
will run to completion using sme callback mechanism.
Change-Id: I4dc7e3ef0c91964ebc842afa38b7815464e0e406
CRs-Fixed: 2114425
Before sending tdls peer state update command to FW make sure
that tdls peer exists.
Change-Id: I26b5daf9896b0f57fbcfedadcabd67ddd000b257
CRs-Fixed: 2032770
In order to support multiple nla_parse function signatures across
different kernel versions, a new hdd_nla_parse wrapper was introduced.
However, some propagations were made that reintroduced nla_parse.
Convert these calls to nla_parse to hdd_nla_parse instead.
Change-Id: Ib60d51252d943c59648682fb2794c3dcbae7cbf4
CRs-Fixed: 2113637
Power save checks for user configuration and STA connection state,
if either of these fails then do not restart the auto power save
timer.
Change-Id: I83a337b828677464c74800adf690c00fda68fadc
CRs-Fixed: 2112585
When uevent is introduced for integrated devices, it breaks
the PLD handling for PCIe devices. Hence correct it by using
proper enum.
Change-Id: Iacb5037932065f1822c1408d74f8aa3824ff0e71
CRs-fixed: 2114385
On framework initiated scan, ies are not sent in
scan request from upper layer and default scan ies are
not sent as part of scan request sent to firmware. This
results in not including oce ie in probe requests for
framework initated scans
Fix is to copy default ies to ie field in scan request
sent to firmware.
Change-Id: I895563bafc2a2b6e483d08cb28ee44ad34f5af70
CRs-Fixed: 2087124
Default scan ies are sent to fw after interface
is up via set wifi configuration vendor subcommand.
In case of PDR, default ies are not sent again to
fw.
Fix is to send default scan ies to fw after PDR.
Change-Id: Ia42f5b49711ec77b84ae3031aa7998283c1d4ec8
CRs-Fixed: 2100739
Currently the SAP has logic to extract and locally save a collection
of Additional IEs. But this is legacy code inherited from a version
of the driver that supported in-driver SAP, and now the SAP
functionality is handled primarily by hostapd. Since this local copy
of the Additional IEs is never used, remove it.
Change-Id: I9bb1f7d44dfdeb4c908016aad365ee1b94a97b29
CRs-Fixed: 2114375
Currently the SAP has logic to extract and locally save the WAPI
IE. But this is legacy code inherited from a version of the driver
that supported in-driver WAPI, and now the SAP WAPI functionality is
handled by hostapd. Since this local copy of the WAPI IE is never
used, remove it.
Change-Id: I57cad0d8e44ff1ca5a7a61454a21aac61bf873c0
CRs-Fixed: 2114153
Currently, in monitor mode, beacon frames are showing an incorrect phy
rate (0). This is because the values in the RX descriptor for
interpreting the legacy rates have changed in ihelium.
Make changes in host to interpret the new values into data rates correctly.
Change-Id: Ibf845666943fd718b657c0128758dabfb64a38ef
CRs-Fixed: 2082456
Add a new dumpStats parameter (25) to dump disconnect stats.
The following command will dump disconnect Stats into QXDM.
>iwpriv wlan0 dumpStats 25
Change-Id: I0908c947b5c829f51b087871ad11d5674908dca2
CRs-Fixed: 2058958
There is a race condition bewteen CLIENT_DISCONNECT and AP_DISCONNECT,
which causes short HDD IPA interfaces, so AP_CONNECT failed afterward.
HDD IPA event handler waits IPA resource load/unload complete.
Once timed out, add the event in a pending queue for loading icase or
ignore it for unloading case.
Remove redundant CLIENT_DISCONNECT event from del_sta.
Change-Id: I671ecec6f236d52a1b51cb78b9c7466f2cde82aa
CRs-Fixed: 2027714
In order to support Linux kernel versions 4.7+, nl80211_band must be
used instead of ieee80211_band. Convert all usages of ieee80211_band
to nl80211_band.
Change-Id: Iaf7a1efbaf54166b4d6e52a0d6b68b32c871028b
CRs-Fixed: 2114157
The SAP no longer maintains WPS information internally and the
APWPSIEs struct in the SAP context is no longer referenced, so remove
it.
Change-Id: If4a30626913fdacb751a434458a2e3918665a352
CRs-Fixed: 2113553
Change "qcacld-3.0: Remove obsolete wlansap_get_wps_state()" removed
the last consumer of the SapGlobalLock infrastructure. Since this
infrastructure is now unused, remove it.
Change-Id: I4646a7796073b259351d8cff48a424ef8741c9cf
CRs-Fixed: 2113552
Some of the logs wrongly use dev->ifindex to print net device name. Fix
those prints with correct device name.
Change-Id: I0cd7cbadcc69cafa05207578b66a2e07ce9e0f4e
CRs-Fixed: 2113484
Add adapter validity check for get NUD stats vendor command such that
after adapter down/close request is not sent down to FW over WMI which
can trigger FW assert.
Change-Id: Ia4809ce50f131d2a85e668681711d89c985203a0
CRs-Fixed: 2098149
Currently in hdd_hostapd_sap_event_cb() when processing the
eSAP_STA_ASSOC_EVENT and eSAP_STA_REASSOC_EVENT events a call is made
to wlansap_get_wps_state(). This is a legacy call inherited from a
version of the driver that supported in-driver WPS. But now that WPS
is handled by hostapd this functionality is obsolete and should be
removed.
Change-Id: Ia52d9554ed4b3a44726a4cac268eae77856cbd18
CRs-Fixed: 2113551
Change attribute type in policy to NLA_UNSPEC, so nla API will take
a more strict length check.
Change-Id: I9faf60702a9efba5e76fb1e82c7b020812687cad
CRs-Fixed: 2092378
Add new WCNSS_qcom_cfg.ini item to configure offset from bad RSSI
threshold ini for 2G to 5G band roam. This offset is used to
calculate the RSSI to be used as trigger for device to roam from
2G to 5G band when it is connected to a bad RSSI 2G AP and a 5G AP
is available in the environment.
New ini added : roam_bad_rssi_thresh_offset_2g
Change-Id: If2285317d1d01bb2faae2cf1928ad7adae8204d4
CRs-Fixed: 2105894
struct sap_WPSIE_s, as well as the structs it contains, were inherited
from a prior version of the driver where SAP WPS was implemented in
the driver. Since SAP WPS is now handled by hostapd these structs are
unused, so remove them.
Change-Id: Iabdd7f3b83668cb203614353b4766fb0b40fbd05
CRs-Fixed: 2110167
Fix the regression issue introduced by change
"Change-Id: Ica336398865a8b5e16297e4555dbb7de5e79567e".
The issue is that it will always include driver default rates.
In some case it only needs to calculate supported rates from hostapd.conf
file. Define ini item gChanSwitchHostapdRateEnabled to configure supported
rates calculated from hostapd.conf file or driver default rates when doing
SAP channel switch.
Change-Id: I45eeea5134dd80929bdd1f61246bdc66e1857e22
CRs-fixed: 2103714
qcacld-2.0 to qcacld-3.0 propagation
Current driver implementation doesn't support the multi beacon
interval in case concurrent AP's are operating on the same
channel. A new parameter 'beacon_int_min_gcd' is added in the
kernel to support the multi beacon interval.
Updating parameter 'beacon_int_min_gcd' to support multi beacon
interval. Related changs are
"Change-Id: I997059f0cad77b914a1296ffe7f1a51a511adac2" and
"Change-Id: Ibd2557080521276aa86a83089dfaa9b445de05af".
Change-Id: Iaa1bf7b7ead05bb400fbd8e28ef3fc1309c6b567
CRs-Fixed: 2111272
Copy the contents of auth frame in a proper manner
in lim_send_auth_mgmt_frame().
Change-Id: I5197634e36fa5b4344fc71fc22e1de6a2147744d
CRS-Fixed: 2112170
At init time, program DFS region before programming the init
channel list. Programming DFS region would also initialize the
channel map.
CRs-Fixed: 2104133
Change-Id: I0ab3607d020086d226a99814dada5ac820880f1f
In Rome platform, it use invalid radio id in FW event
WMI_RADIO_TX_POWER_LEVEL_STATS_EVENTID and cause crash. So check
radio id when handling this event.
Change-Id: Id720ca94ef496ea883f5ba1848fb4e28af57002f
CRs-Fixed: 2112322
Add data structures to save beacon tx rate.
The beacon data rate is multiples of 100 Kbps.
Firmware expects the data rate in the form of hw rate codes.
So convert the data rates to hw rate code.
And send it to firmware.
Change-Id: Ia39fd4c14defa729f75f2c45748fe5b04b909647
CRs-Fixed: 2099052
qcacld-2.0 to qcacld-3.0 Propagation.
extn capabilities can be controlled by user, which can
be sent greater than the max supported value. This results
in stack overflow in change station command.
Add check to validate extn capability param given by user
and if it exceeds max supported value, set it to max supported
value.
CRs-Fixed: 2076664
Change-Id: I531799dd06c41069e85ad969de6182363dbf9f05
qcacld-2.0 to qcacld-3.0 propagation
Currently the crash can be injected by iwpriv command and FW
gets crashed.
Changes are done to add the gEnableCrashInject ini parameter
1) This ini param is disabled by default.
2) If this param is disabled the crash inject is ignored.
Change-Id: I7e908be1e37090a9d343dc04411fe387f776a937
CRs-Fixed: 864932
The Linux Coding Style enumerates a few special cases where typedefs
are useful, but stresses "NEVER EVER use a typedef unless you can
clearly match one of those rules." The ptSapContext typedef does not
meet any of those criteria, so replace it.
Change-Id: Ia08bea35b7c0d3b3dfad4641c1c79a7c0990cd8d
CRs-Fixed: 2109993
Currently wlansap_open() takes a p_cds_gctx parameter. At one time
this was needed, but that is no longer the case, so remove this unused
parameter.
Change-Id: Ie110b3cee9898233fb921aaab223caa4cf93de4e
CRs-Fixed: 2109961
The Linux Coding Style enumerates a few special cases where typedefs
are useful, but stresses "NEVER EVER use a typedef unless you can
clearly match one of those rules." The tSapContext typedef does not
meet any of those criteria, so replace it.
Change-Id: If237edee7638c1c6b35413312cb3dd934554431a
CRs-Fixed: 2109960
The Linux Coding Style frowns upon mixed-case names and so-called
Hungarian notation, so rename struct sSapContext to align with the
Coding Style. The associated tSapContext and ptSapContext typedefs
will be cleaned up as a subquent effort.
Change-Id: I83962e1c574491039597559fb8b16f3339d41fbd
CRs-Fixed: 2109959
csr_roam_stats_rsp_processor() calls cds_get_global_context() but then
doesn't do anything with the result, so remove the unnecessary call.
Change-Id: Ic47dbf971d1eaea4c7c33ff62f9bfb848d9d654b
CRs-Fixed: 2109956
Macro CDS_GET_HAL_CB() currently takes a ctx parameter. At one time
cds_get_context() needed a pointer to the global context, and hence
the reason for the parameter. But now that cds_get_context() no longer
needs the global context, remove the parameter from the macro. This is
a precursor to removing the p_cds_gctx from the SAP config.
Change-Id: Ic75bab55fd8b87710aee99b7e2d3783361dc3d87
CRs-Fixed: 2109955
Modify pHddCtx to hdd_ctx to avoid camel case coding style
as per the coding guidelines
Change-Id: I55767e0efdd803b3a8bf2bc7c52c96bbdc6d65bd
CRs-Fixed: 2111749
Introduce the below WCNSS_qcom_cfg.ini items to configure
for bad RSSI roaming. These parameters would be used when
the device is connected to an AP with weak signal and has
to roam to a better AP if it is stationary but has found
some other better AP in the vicinity
roam_bg_scan_bad_rssi_thresh:
If the DUT is connected to an AP with weak signal, then the bad RSSI
threshold will be used as an opportunity to use the scan results
from other scan clients and try to roam if there is a better AP
available in the environment.
roam_bg_scan_client_bitmap:
This bitmap is used to define the client scans that need to be used
by the roaming module to perform a background roaming.
Change-Id: I3b9f737ea389d35f8be173ce83b2237c2375fb88
CRs-Fixed: 2082904
Cleanup duplicate APIs for get ie pointer from EID or OUI or EXT_ID
and use wlan_utils exposed by qcacmn.
Change-Id: Iacbf041c9c49b2aedeffecba3427e9334cb70dcc
CRs-Fixed: 2105377
Linux kernel version 4.12 changes the way net_device destruction works.
Add conditional compilation to properly setup the net_device destruction
based on which version of the Linux kernel is being compiled against.
Change-Id: Idcc2186ff8593ed1467e7c0b08ece7c4dd70843a
CRs-Fixed: 2093367
Fix misleading ACS logging that indicates best channel and final
channel selected are different.
Change-Id: I00a2a15fa9d709c3e78ebf580d599ec10deefb28
CRs-Fixed: 2105204
Currently in the function lim_process_action_vendor_specific, mem_cmp
is done on the received frame pointer without validating the frame_len
which could lead to out-of-bounds memory access if the frame_len is
not matching the size of action_hdr.
Add check to validate the frame_len with action_hdr size before doing
mem_cmp for the p2p oui.
Change-Id: I39329d1a9ef45614d3c617db11a7a7f5ec2aaaec
CRs-Fixed: 2101439
Enable support to use SKB Buffers allocated during bootup for
low memory targets.
CRs-Fixed: 2106466
Change-Id: Ia55c38767804c5915b9d29d3c61bff236d12cacd
qcacld-2.0 to qcacld-3.0 propagation
Add changes to support PNO scan in connected state to find better Ap's
based on rssi threshold, band and rssi preference.
Change-Id: I17f1830c07821b577473fc2d8f50c0867572fe0c
CRs-Fixed: 2012702
Use qdf_do_div() for 64 bit division, because '/'
can't be used for 64 bit division on arm32 platform.
Change-Id: I047334565643e695ac343a5da06e030d8e37d7e4
CRs-Fixed: 2110736
There is an incorrect mapping for trace level - bitmask in api
hdd_qdf_trace_enable trace level is getting started with
QDF_TRACE_LEVEL_FATAL here which should be actually
QDF_TRACE_LEVEL_NONE.
Change-Id: I63e171bf83d3d4f039f7f3170215f036e778de7c
CRs-Fixed: 2110797
As WAPI spec, once USK rekey, PN need reset to 0x5c37, but IWNCOM AP
will send 1st packet with old PN larger than 0x5c37 after rekey, So PN
check failed for later packets.
Fix: Don't update Last_PN when receive the 1st packet from AP after USK
rekey, Last_PN keeps 0x5c37.
Analysis:
1. Host installed new unicast key after rekey handshake with AP.
2. Host Last_PN was updated to 0x5c37 as spec (expect PN of next
packet from AP is: 0x5c39)
3. Host received ping request from AP with old PN 0x5cb9
4. Host PN checked passed, driver Last_PN was updated as 0x5cb9
5. Host received ping request from AP with new PN 0x5c39
6. Host PN checked failed since 0x5c39 < 0x5cb9, ping request
dropped, no response.
Change-Id: Ifd0269193ffbe53889191269a0e985d13ea2ecf7
CRs-Fixed: 2101217
RRM beacon report scan may find many neighbor BSSIDs on a channel
but it currently sends only 1 beacon report frame with maximum 4
measurement reports. Add a loop to go send all BSSIDs by generating
multiple frames.
CRs-Fixed: 2065920
Change-Id: Iabf889e092b4818ca9c31969eea38e92028209c0
Join failure timeout value is set to 10 times beacon
interval. If 10 times beacon interval is greater than
max join failure timeout value, global join failure
timeout cfg value is not set which results in connection
failure.
Set join failure timeout value in bss config to the
default join failure timeout value if 10 times beacon
value exceeds default value.
Change-Id: I835256ba07a714cb0606c6e90964475fc112c4b4
CRs-Fixed: 2100633
Introduce the following items in WCNSS_qcom_cfg.ini for
user to modify them for MAWC based roaming.
mawc_roam_enabled
mawc_roam_traffic_threshold
mawc_roam_ap_rssi_threshold
mawc_roam_rssi_high_adjust
mawc_roam_rssi_low_adjust
Change-Id: Ief51e3e91603bfd7f6c6deed3ca48bc717b2b6fb
CRs-Fixed: 2081413
Currently cds_sched_close() takes a cds_context param. However CDS
already maintains its own context, hence this param introduces an
unnecessary coupling, so remove it.
Change-Id: Ic7ceb85653acd2acdb700ae9d2ef15f4e09b81c7
CRs-Fixed: 2109435
Currently struct sAniGetTsmStatsReq contains a p_cds_context field,
but this field is unused. Remove the field along with all of the code
that is present to populate it.
Change-Id: I1335034b959b77223834d7cf77e33c1f9bd1b358
CRs-Fixed: 2109433
Currently sme_set_idle_powersave_config() takes a cds_ctx
parameter. At one time this was needed for the call to
cds_get_context(), but that is no longer the case, so remove this
unused parameter.
Change-Id: Iacef39eded8d87a22a30fe62b0305f93e604dc4c
CRs-Fixed: 2109431
Currently cds_close() takes a cds_context param. However CDS already
maintains its own context, hence this param introduces an unnecessary
coupling, so remove it.
Change-Id: I9c5231ef5d1e2ca969ff01c3fa0dc9b202e1aabd
CRs-Fixed: 2109430
Currently cds_disable() takes a cds_context param. However CDS already
maintains its own context, hence this param introduces an unnecessary
coupling, so remove it.
Change-Id: I539b3c19a52a9c663268bade6a0acfaa411906dd
CRs-Fixed: 2109429
Currently cds_enable() takes a cds_context param. However CDS already
maintains its own context, hence this param introduces an unnecessary
coupling, so remove it.
Change-Id: I13a9a0cbdb8b869f8f543c4f7f89cffbcf4fd210
CRs-Fixed: 2109427
Over time the wlansap() APIs changed from being passed the global
context to being passed the SAP context. Apparently two invocations of
wlansap_cancel_remain_on_channel() were not correctly updated, so
update them.
Change-Id: I53c54c639ed2695439e4f5cf506e7f23732ab8de
CRs-Fixed: 2109408
Channel 13 and channel 9 can be combined to HT40. The channel weight of
13 is updated with channel 9 when chStartNum of acs_ht40_channels24_g
is 9 in sap_sort_chl_weight_ht40_24_g, however at this time the channel
weight of 9 may be already updated to SAP_ACS_WEIGHT_MAX * 2 when the
chStartNum of acs_ht40_channels24_g is 1. In such case channel weight of
13 is updated incorrectly.
When update the channel weight of 13, use original channel weight of
channel 9 and channel 13.
Change-Id: Iab16cd5b1ab45fdf2f9805767d81c6fc997ee4be
CRs-Fixed: 2105213
Currently cds_alloc_context() & cds_free_context() take a CDS context
parameter. However CDS already maintains its own context, hence this
parameter introduces an unnecessary coupling, so remove it.
Change-Id: I4501a0fea7acc7b4a764b0d425da386922b93bcb
CRs-Fixed: 2109265
Currently wma_open() takes a cds_ctx parameter. All of the other WMA
functions which previously took a cds_ctx no longer use that
parameter, and those functions have been modified to remove the unused
parameter. wma_open() still needs the cds_ctx, but it can get that
context by calling cds_get_global_context(), so remove the cds_ctx
parameter to be consistent with the other WMA functions.
Note: the fact that wma_open() needs the cds_ctx, and then
dereferences it, is a layering violation that needs to be addressed in
the future.
Change-Id: I176e2ac68cc2e8081645a4ce3c158b41d3018587
CRs-Fixed: 2109263
