commit 5e9ebe22503f55d5c9c0cf1a17881a4f130d2c22
author Charles Celerier <chcl@google.com> Wed Jul 08 20:56:26 2020 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Wed Jul 08 20:56:26 2020 +0000
tree d0540a88eb03c4a8a0b41435d1e3da6f58eab37d
parent 4e07c3c0dd4c22601f250dadab3510a7cc2bf48a

[wlan][brcmfmac] Remove extraneous error logging after call to brcmf_fil_iovar_data_get

The brcmf_fil_iovar_data_get function does its own logging if an iovar
could not be retrieved, so it is not necessary to log the error twice.

Bug: 52941
Test: None necessary
Change-Id: Ied970a16a9972c7974145d04f5e7c212490de829
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/404143
Commit-Queue: Charles Celerier <chcl@google.com>
Reviewed-by: Joshua Conner <joshconner@google.com>
Reviewed-by: Karthik Krishnaswami <karthikrish@google.com>
Testability-Review: Joshua Conner <joshconner@google.com>
Testability-Review: Karthik Krishnaswami <karthikrish@google.com>

src/connectivity/wlan/drivers/third_party/broadcom/brcmfmac/feature.cc

diff --git a/src/connectivity/wlan/drivers/third_party/broadcom/brcmfmac/feature.cc b/src/connectivity/wlan/drivers/third_party/broadcom/brcmfmac/feature.cc
index 912fbde..d1eb3d1 100644
--- a/src/connectivity/wlan/drivers/third_party/broadcom/brcmfmac/feature.cc
+++ b/src/connectivity/wlan/drivers/third_party/broadcom/brcmfmac/feature.cc
@@ -102,8 +102,6 @@
 
   err = brcmf_fil_iovar_data_get(ifp, "cap", caps, sizeof(caps), &fw_err);
   if (err != ZX_OK) {
-    BRCMF_ERR("could not get firmware cap: %s, fw err %s", zx_status_get_string(err),
-              brcmf_fil_get_errstr(fw_err));
     return;
   }