-
Notifications
You must be signed in to change notification settings - Fork 333
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
wifi: mt76: mt7921: update the channel usage when the regd domain cha…
…nged The 5.9/6GHz channel license of a certain platform device has been regulated in various countries. That may be difference with standard Liunx regulatory domain settings. In this case, when .reg_notifier() called for regulatory change, mt792x chipset should update the channel usage based on clc or dts configurations. Channel would be disabled by following cases. * clc report the particular UNII-x is disabled. * dts enabled and the channel is not configured. Signed-off-by: Ming Yen Hsieh <mingyen.hsieh@mediatek.com> Co-developed-by: Deren Wu <deren.wu@mediatek.com> Signed-off-by: Deren Wu <deren.wu@mediatek.com> Signed-off-by: Felix Fietkau <nbd@nbd.name>
- Loading branch information
Showing
4 changed files
with
64 additions
and
2 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
4a0f839
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80/mt7921/mcu.c:16:52: error: expected ')' before 'bool'
16 | module_param_named(disable_clc, mt7921_disable_clc, bool, 0644);
| ^~~~~
| )
/home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80/mt7921/mcu.c:17:30: error: expected ')' before string constant
17 | MODULE_PARM_DESC(disable_clc, "disable CLC support");
| ^~~~~~~~~~~~~~~~~~~~~~
| )
make[6]: *** [scripts/Makefile.build:262: /home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80/mt7921/mcu.o] Error 1
make[5]: *** [scripts/Makefile.build:497: /home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80/mt7921] Error 2
make[4]: *** [Makefile:1750: /home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80] Error 2
make[3]: *** [Makefile:585: /home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/mt76-2022-12-22-5b509e80/.built] Error 2
make[4]: Leaving directory '/home/runner/work/test/test/openwrt/build_dir/target-x86_64_musl/linux-x86_64/linux-5.4.257'
make[3]: Leaving directory '/home/runner/work/test/test/openwrt/package/kernel/mt76'
time: package/kernel/mt76/compile#2.20#0.46#2.65
ERROR: package/kernel/mt76 failed to build.
4a0f839
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error message indicates that there's a problem on line 16 of the mcu.c file, specifically saying: "expected ')' before 'bool'." This error suggests there's a problem with how the parameter is defined. In this case, it might be related to the module_param_named line where the parameter is declared.Would it be better to modify it as follows?
static bool mt7921_disable_clc;
module_param_named(disable_clc, mt7921_disable_clc, bool, 0644);
MODULE_PARM_DESC(disable_clc, "Disable CLC support (0/1)");