There are multiple trivial issues with the SME stats ext event
handling:
* sme_stats_ext_event() is exported as a global function even though
its use is local to sme_api.c
* As a local API sme_stats_ext_event() should take an internal MAC
context instead of an external HAL handle
* The dispatch code in sme_process_msg() incorporates conditional
compilation within the function body.
* There is replication of parameter checking.
Normally such trivial issues would not be addressed, however an
upcoming change will enforce type safety and no longer allow
tHalHandle and tpAniSirGlobal to be freely exchanged, so in
preparation perform the following cleanups:
* Make sme_stats_ext_event() static
* Change sme_stats_ext_event() to take a tpAniSirGlobal MAC context
* Provide a stub implementation of sme_stats_ext_event() when the
WLAN_FEATURE_STATS_EXT feature is not enabled
* Remove the WLAN_FEATURE_STATS_EXT featurization from
sme_process_msg()
* Remove the parameter checking from sme_process_msg()
Change-Id: I367e9d73ca50bd5c24e9261ceb2c3a504fb70e1a
CRs-Fixed: 2264796
fdecd51166