diff options
author | spuligil <spuligil@codeaurora.org> | 2018-10-16 03:21:38 -0700 |
---|---|---|
committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2018-10-16 04:44:43 -0700 |
commit | ebf00a49fd29206d5fbdc10d511c6504182622b8 (patch) | |
tree | bac611e985f3e9e2be5822d5759c1a1976f5cce5 | |
parent | aa10ef55101f168cce8ae080e193e803a9d0e24a (diff) | |
download | wlan-fw-api-ebf00a49fd29206d5fbdc10d511c6504182622b8.tar.gz |
fw-api: CL 5574292 - update fw common interface files
Change-Id: Ia7f2989ebbf76ce7716d8d2f30854ab6a773ec13
WMI: provide unconditional defs of wmi_pdev_stats struct
CRs-Fixed: 2262693
-rw-r--r-- | fw/wmi_unified.h | 47 | ||||
-rw-r--r-- | fw/wmi_version.h | 2 |
2 files changed, 44 insertions, 5 deletions
diff --git a/fw/wmi_unified.h b/fw/wmi_unified.h index 59336da..44a740e 100644 --- a/fw/wmi_unified.h +++ b/fw/wmi_unified.h @@ -7118,8 +7118,23 @@ typedef struct { /** * PDEV statistics - * @todo - * add all PDEV stats here + * + * This struct incorporates the wlan_dbg_stats struct, which is + * conditionally defined, based on the AR900B flag. + * The below _v1 struct variant is the unconditional definition + * that matches what would be conditionally defined by builds that + * don't use the AR900B flag. The _v2 struct variant is the + * unconditional definition that matches what would be conditionally + * defined by builds that use the AR900B flag. + * The _v2 struct def can be used within host or target builds + * that don't use the AR900B flag, but needs to interoperate with a + * target or host build that does use the AR900B flag. + * Similarly, the _v1 struct def can be used by a host or target build + * that does use the AR900B flag, but needs to interoperate with a + * target or host build that doesn't use the AR900B flag. + * + * For backwards compatibility, wmi_pdev_stats is still (conditionally) + * defined, as an alias for either the _v1 or _v2 variant. */ typedef struct { /** Channel noise floor */ @@ -7137,9 +7152,33 @@ typedef struct { /** Channel Tx Power */ A_UINT32 chan_tx_pwr; /** WAL dbg stats */ - struct wlan_dbg_stats pdev_stats; + struct wlan_dbg_stats_v1 pdev_stats; +} wmi_pdev_stats_v1; -} wmi_pdev_stats; +typedef struct { + /** Channel noise floor */ + A_INT32 chan_nf; + /** TX frame count */ + A_UINT32 tx_frame_count; + /** RX frame count */ + A_UINT32 rx_frame_count; + /** rx clear count */ + A_UINT32 rx_clear_count; + /** cycle count */ + A_UINT32 cycle_count; + /** Phy error count */ + A_UINT32 phy_err_count; + /** Channel Tx Power */ + A_UINT32 chan_tx_pwr; + /** WAL dbg stats */ + struct wlan_dbg_stats_v2 pdev_stats; +} wmi_pdev_stats_v2; + +#if defined(AR900B) +#define wmi_pdev_stats wmi_pdev_stats_v2 +#else +#define wmi_pdev_stats wmi_pdev_stats_v1 +#endif /** * VDEV statistics diff --git a/fw/wmi_version.h b/fw/wmi_version.h index 00ab85e..157ddca 100644 --- a/fw/wmi_version.h +++ b/fw/wmi_version.h @@ -36,7 +36,7 @@ #define __WMI_VER_MINOR_ 0 /** WMI revision number has to be incremented when there is a * change that may or may not break compatibility. */ -#define __WMI_REVISION_ 583 +#define __WMI_REVISION_ 584 /** The Version Namespace should not be normally changed. Only * host and firmware of the same WMI namespace will work |