son_master_thread: remove unneeded use of set_hostap_supported_channels

In handle_intel_slave_join, there is a call to set_hostap_supported_channels, the function is also used as part of add_hostap_supported_operating_class which is called by autoconfig_wsc_parse_radio_caps, which runs in the handle_intel_slave_join making the first use of set_hostap_supported_channels redundant. Remove set_hostap_supported_channels from handle_intel_slave_join Signed-off-by: Itay Elenzweig <[email protected]>
prplfoundation · Apr 12, 2020 · 2c2f23c · 2c2f23c
1 parent c91a301
commit 2c2f23c
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/controller/src/beerocks/master/son_master_thread.cpp b/controller/src/beerocks/master/son_master_thread.cpp
@@ -2101,8 +2101,6 @@ bool master_thread::handle_intel_slave_join(
     database.set_node_ipv4(radio_mac, bridge_ipv4);
     database.set_node_manufacturer(radio_mac, "Intel");
 
-    database.set_hostap_supported_channels(radio_mac, notification->hostap().supported_channels,
-                                           message::SUPPORTED_CHANNELS_LENGTH);
 
     if (database.get_node_5ghz_support(radio_mac)) {
         if (notification->low_pass_filter_on()) {
@@ -2272,7 +2270,7 @@ bool master_thread::autoconfig_wsc_parse_radio_caps(
             non_operable_channels.push_back(*channel);
         }
         ss << " }" << std::endl;
-        //        LOG(DEBUG) << ss.str();
+        LOG(DEBUG) << ss.str();
         // store operating class in the DB for this hostap
         database.add_hostap_supported_operating_class(
             radio_mac, operating_class, maximum_transmit_power_dbm, non_operable_channels);