RDKCOM-5544: RDKBWIFI-322 Wifi 6/7 Radio capabilities - AP Capabiliti…#1216
Open
tinaelizabeth84 wants to merge 1 commit into
Open
RDKCOM-5544: RDKBWIFI-322 Wifi 6/7 Radio capabilities - AP Capabiliti…#1216tinaelizabeth84 wants to merge 1 commit into
tinaelizabeth84 wants to merge 1 commit into
Conversation
…es (rdkcentral#934) * Changes for Wifi Radio Capability data to be populated in OneWifi Reason for change: The OneWifi/Easymesh needs Radio Capabilities data which is missing. Hence implemented the method to pull the caps from Driver(Hostap) to OneWifi. Co-authored-by: sriraaman-c <sriraaman33@gmail.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds WiFi 6 (802.11ax/HE) and WiFi 7 (802.11be/EHT) radio capability propagation through the webconfig radiocap subdoc and translates those capabilities into EasyMesh radio capability structures, enabling OneWifi/EasyMesh consumers to access driver-reported radio capabilities.
Changes:
- Extend radiocap JSON encoding/decoding to distinguish
PhyIndexvsRadioIndexand to carry WiFi6/7 capability fields/arrays. - Add an EasyMesh translator helper to map OneWifi HE/EHT capabilities into
em_radio_cap_info_t. - Extend the external EasyMesh proto interface (
webconfig_external_easymesh_t) to provide aget_radio_capcallback.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| source/webconfig/wifi_encoder.c | Encodes PhyIndex + RadioIndex and adds WiFi6/7 capability fields into the radiocap JSON. |
| source/webconfig/wifi_decoder.c | Decodes the new index fields and WiFi6/7 capability fields from radiocap JSON. |
| source/webconfig/wifi_easymesh_translator.c | Translates OneWifi radiocap HE/EHT data into EasyMesh radio capability structures and wires new callback in proto init. |
| include/wifi_base.h | Adds HE capability bit position constants used during translation. |
| include/webconfig_external_proto_easymesh.h | Adds get_radio_cap callback type/member and extends init signature to accept it. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+4915
to
+4918
| decode_param_bool(object, "WiFi6Supported", value_object); | ||
| if (value_object != NULL) { | ||
| radio_cap->wifi6_supported = (value_object->type & cJSON_True) ? true : false; | ||
| } |
Comment on lines
+4983
to
+4986
| decode_param_bool(object, "WiFi7Supported", value_object); | ||
| if (value_object != NULL) { | ||
| radio_cap->wifi7_supported = (value_object->type & cJSON_True) ? true : false; | ||
| } |
| @@ -506,6 +623,19 @@ webconfig_error_t translate_radio_object_to_easymesh_for_dml(webconfig_subdoc_da | |||
| em_radio_info->associated_sta_link_mterics_inclusion_policy = 0; | |||
| strncpy (em_radio_info->chip_vendor, wifi_prop->manufacturer, strlen(em_radio_info->chip_vendor)); | |||
Comment on lines
+626
to
+630
| em_radio_cap_info_t *radio_cap = proto->get_radio_cap(proto->data_model, wifi_prop->radiocap[index].rdk_radio_index); | ||
| if (radio_cap == NULL) { | ||
| wifi_util_error_print(WIFI_WEBCONFIG, "%s:%d: radio_cap not found\n", __func__, __LINE__); | ||
| return webconfig_error_translate_to_easymesh; | ||
| } |
pradeeptakdas
approved these changes
Jun 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…es (#934)
Reason for change: The OneWifi/Easymesh needs Radio Capabilities data which is missing. Hence implemented the method to pull the caps from Driver(Hostap) to OneWifi.
Signed-off-by:sriraaman-c <sriraaman33@gmail.com