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

ramips-mt7621: migration of sysfs paths in /etc/config/wireless necessary #2391

Closed
neocturne opened this issue Feb 6, 2022 · 1 comment
Closed
Labels
0. type: bug This is a bug 1. severity: blocker This issue/pr is required for the next release 3. topic: hardware Topic: Hardware Support 3. topic: wireless Topic: Wireless
Milestone

Comments

@neocturne
Copy link
Member

Bug report

What is the problem?

On ramips targets (observed on D-Link DIR860L), the sysfs paths of the WLAN interfaces changed from OpenWrt 19.07 to 21.02:

Before:

pci0000:00/0000:00:00.0/0000:01:00.0
pci0000:00/0000:00:01.0/0000:02:00.0

After:

1e140000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0
1e140000.pcie/pci0000:00/0000:00:01.0/0000:02:00.0

/etc/config/wireless is currently not covered by #2372, so some config scripts fail after the upgrade.

Possible solutions:

  • Regenerate /etc/config/wireless on reconfig
    • Requires to move more user intent to /etc/config/gluon
    • Might lead to unstable radio index assignments on some devices?
      • Regenerate only when the old config is obviously broken?
      • Ignore radio indices for intent configuration, instead use more generic "config for all 2.4GHz", "config for all 5GHz"?
  • Target-specific migration script
@neocturne neocturne added 0. type: bug This is a bug 3. topic: hardware Topic: Hardware Support 1. severity: blocker This issue/pr is required for the next release 3. topic: wireless Topic: Wireless labels Feb 6, 2022
@neocturne neocturne added this to the 2022.1 milestone Feb 6, 2022
@rotanid rotanid changed the title ramips: migration of sysfs paths in /etc/config/wireless necessary ramips-mt7621: migration of sysfs paths in /etc/config/wireless necessary Jun 14, 2022
@blocktrron
Copy link
Member

Closed by #2560

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: bug This is a bug 1. severity: blocker This issue/pr is required for the next release 3. topic: hardware Topic: Hardware Support 3. topic: wireless Topic: Wireless
Projects
None yet
Development

No branches or pull requests

2 participants