From a597e66070517edc5d1650812bbe282c82fc84d6 Mon Sep 17 00:00:00 2001
From: Alexander Lunev <alexanderlunev@mail.ru>
Date: Thu, 3 Feb 2022 06:53:01 +0300
Subject: [PATCH] bcm43xxx: fixed several warnings: warning: ISO C forbids
 'return' with expression, in function returning void [-Wpedantic] warning:
 format '%x' expects argument of type 'unsigned int', but argument 4 has type
 'uint8_t * {aka unsigned char *}' [-Wformat=] warning: too many arguments for
 format [-Wformat-extra-args]

---
 drivers/wireless/ieee80211/bcm43xxx/bcmf_driver.c | 2 +-
 drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.c   | 2 +-
 drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.h   | 4 ++--
 drivers/wireless/ieee80211/bcm43xxx/bcmf_sdpcm.c  | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/wireless/ieee80211/bcm43xxx/bcmf_driver.c b/drivers/wireless/ieee80211/bcm43xxx/bcmf_driver.c
index 15160491e0..718fab2b60 100644
--- a/drivers/wireless/ieee80211/bcm43xxx/bcmf_driver.c
+++ b/drivers/wireless/ieee80211/bcm43xxx/bcmf_driver.c
@@ -351,7 +351,7 @@ int bcmf_driver_download_clm(FAR struct bcmf_dev_s *priv)
     }
   else
     {
-      wlinfo("Download %d bytes @ 0x%08x\n", datalen, srcbuff);
+      wlinfo("Download %d bytes @ %p\n", datalen, srcbuff);
     }
 
   /* Divide CLM blob into chunks */
diff --git a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.c b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.c
index 39502706a3..9f0d2cc4a5 100644
--- a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.c
+++ b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.c
@@ -278,7 +278,7 @@ int bcmf_probe(FAR struct bcmf_sdio_dev_s *sbus)
     }
   else
     {
-      wlwarn("High-Speed mode is not supported by the chip!\n", value);
+      wlwarn("High-Speed mode is not supported by the chip!\n");
     }
 #endif
 
diff --git a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.h b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.h
index 3a6206f171..ac4214919d 100644
--- a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.h
+++ b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdio.h
@@ -148,8 +148,8 @@ int bcmf_bus_sdio_initialize(FAR struct bcmf_dev_s *priv,
  */
 
 int bcmf_transfer_bytes(FAR struct bcmf_sdio_dev_s *sbus, bool write,
-                         uint8_t function, uint32_t address,
-                         uint8_t *buf, unsigned int len);
+                        uint8_t function, uint32_t address,
+                        uint8_t *buf, unsigned int len);
 
 int bcmf_read_reg(FAR struct bcmf_sdio_dev_s *sbus, uint8_t function,
                   uint32_t address, uint8_t *reg);
diff --git a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdpcm.c b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdpcm.c
index 94ecd9a07f..5697989e19 100644
--- a/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdpcm.c
+++ b/drivers/wireless/ieee80211/bcm43xxx/bcmf_sdpcm.c
@@ -522,9 +522,9 @@ struct bcmf_frame_s *bcmf_sdpcm_alloc_frame(FAR struct bcmf_dev_s *priv,
 }
 
 void bcmf_sdpcm_free_frame(FAR struct bcmf_dev_s *priv,
-                     struct bcmf_frame_s *frame)
+                           struct bcmf_frame_s *frame)
 {
-  return bcmf_sdio_free_frame(priv, (struct bcmf_sdio_frame *)frame);
+  bcmf_sdio_free_frame(priv, (struct bcmf_sdio_frame *)frame);
 }
 
 struct bcmf_frame_s *bcmf_sdpcm_get_rx_frame(FAR struct bcmf_dev_s *priv)