summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorspuligil <spuligil@codeaurora.org>2018-10-16 03:21:38 -0700
committerGerrit - the friendly Code Review server <code-review@localhost>2018-10-16 04:44:43 -0700
commitebf00a49fd29206d5fbdc10d511c6504182622b8 (patch)
treebac611e985f3e9e2be5822d5759c1a1976f5cce5
parentaa10ef55101f168cce8ae080e193e803a9d0e24a (diff)
downloadwlan-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.h47
-rw-r--r--fw/wmi_version.h2
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