diff mbox series

ACS: include frequency in info messages

Message ID 20220830114511.2460483-1-stijn@linux-ipv6.be
State Accepted
Headers show
Series ACS: include frequency in info messages | expand

Commit Message

Stijn Tintel Aug. 30, 2022, 11:45 a.m. UTC 
The ACS info messages frequently appear for multiple channels. Without
the actual frequency in the messages, they are not very informative.
Add the frequency to them to improve this.

Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>
---
 src/ap/acs.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Jouni Malinen Sept. 16, 2022, 7:21 p.m. UTC | #1 
On Tue, Aug 30, 2022 at 02:45:11PM +0300, stijn@linux-ipv6.be wrote:
> The ACS info messages frequently appear for multiple channels. Without
> the actual frequency in the messages, they are not very informative.
> Add the frequency to them to improve this.

Thanks, applied.
diff mbox series

Patch

diff --git a/src/ap/acs.c b/src/ap/acs.c
index 7708bc283..e8d73aa22 100644
--- a/src/ap/acs.c
+++ b/src/ap/acs.c
@@ -420,19 +420,19 @@  static int acs_usable_bw160_chan(const struct hostapd_channel_data *chan)
 static int acs_survey_is_sufficient(struct freq_survey *survey)
 {
 	if (!(survey->filled & SURVEY_HAS_NF)) {
-		wpa_printf(MSG_INFO, "ACS: Survey is missing noise floor");
+		wpa_printf(MSG_INFO, "ACS: Survey for freq '%d' is missing noise floor", survey->freq);
 		return 0;
 	}
 
 	if (!(survey->filled & SURVEY_HAS_CHAN_TIME)) {
-		wpa_printf(MSG_INFO, "ACS: Survey is missing channel time");
+		wpa_printf(MSG_INFO, "ACS: Survey for freq '%d' is missing channel time", survey->freq);
 		return 0;
 	}
 
 	if (!(survey->filled & SURVEY_HAS_CHAN_TIME_BUSY) &&
 	    !(survey->filled & SURVEY_HAS_CHAN_TIME_RX)) {
 		wpa_printf(MSG_INFO,
-			   "ACS: Survey is missing RX and busy time (at least one is required)");
+			   "ACS: Survey for freq '%d' is missing RX and busy time (at least one is required)", survey->freq);
 		return 0;
 	}