Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core: rtw_ieee80211.c: define CONFIG_GET_MAC_ADDRESS_FROM_FILE #1

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

RadxaMitchell
Copy link

Support for reading MAC addresses from files by adding CONFIG_GET_MAC_ADDRESS_FROM_FILE

Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

另外我没有看到如果这个文件不存在又没有读取到efuse时的生成随机mac地址以及保存的代码

debian/patches/get-mac-address-from-file.patch Outdated Show resolved Hide resolved
debian/patches/get-mac-address-from-file.patch Outdated Show resolved Hide resolved
debian/patches/get-mac-address-from-file.patch Outdated Show resolved Hide resolved
@RadxaMitchell
Copy link
Author

另外我没有看到如果这个文件不存在又没有读取到efuse时的生成随机mac地址以及保存的代码

这个要用户在 /etc 下添加文件添加生成的 mac 地址

Copy link
Member

@RadxaYuntian RadxaYuntian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

两个补丁合一起

@RadxaYuntian
Copy link
Member

另外应该要加上保存随机生成的mac地址功能,用户在使用时不应该需要自己去手动添加这个文件才能正常工作

What this patch does is to save the randomly generated MAC address to a file,
and read the written MAC address from the file right after the next boot.

Signed-off-by: Chuang Ma <[email protected]>
@RadxaYuntian
Copy link
Member

有自己生成deb包尝试安装么?能在产品上编译通过并且mac地址工作正常么?

@RadxaMitchell
Copy link
Author

有自己生成deb包尝试安装么?能在产品上编译通过并且mac地址工作正常么?

试了,我用生成的 deb 包在 rock-pi-s 上安装后,可以编译通过,mac 地址也工作正常

@RadxaYuntian RadxaYuntian merged commit bbcfcea into radxa-pkg:main Dec 27, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants