ap_wlan_hal_dwpal: override refresh_radio_info

The read_supported_channels functions, which fills the supported_channels vector in m_radio_info, is never called, it should be called as part of the refresh_radio_info. In ap_wlan_hal_dwpal override refresh_radio_info, and add a call to read_supported_channels. Signed-off-by: Itay Elenzweig <[email protected]>
prplfoundation · Apr 12, 2020 · c91a301 · c91a301
1 parent 2e73e20
commit c91a301
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/common/beerocks/bwl/dwpal/ap_wlan_hal_dwpal.cpp b/common/beerocks/bwl/dwpal/ap_wlan_hal_dwpal.cpp
@@ -1436,6 +1436,12 @@ bool ap_wlan_hal_dwpal::restricted_channels_get(char *channel_list)
     return true;
 }
 
+bool ap_wlan_hal_dwpal::refresh_radio_info()
+{
+    auto ret = base_wlan_hal_dwpal::refresh_radio_info();
+    return ret && read_supported_channels();
+}
+
 bool ap_wlan_hal_dwpal::read_acs_report()
 {
     LOG(TRACE) << __func__ << " for interface: " << get_radio_info().iface_name;

diff --git a/common/beerocks/bwl/dwpal/ap_wlan_hal_dwpal.h b/common/beerocks/bwl/dwpal/ap_wlan_hal_dwpal.h
@@ -66,6 +66,7 @@ class ap_wlan_hal_dwpal : public base_wlan_hal_dwpal, public ap_wlan_hal {
     virtual bool failsafe_channel_get(int &chan, int &bw) override;
     virtual bool restricted_channels_set(char *channel_list) override;
     virtual bool restricted_channels_get(char *channel_list) override;
+    virtual bool refresh_radio_info() override;
     virtual bool read_acs_report() override;
     virtual bool read_supported_channels() override;
     virtual bool update_preference_channels_from_supoorted_channels() override;