Skip to content

Comments

luci-app-ffwizard-falter: update to use option band instead of iwinfo#516

Open
pmelange wants to merge 1 commit intofreifunk-berlin:mainfrom
pmelange:wirless-lua-check-freq-alt
Open

luci-app-ffwizard-falter: update to use option band instead of iwinfo#516
pmelange wants to merge 1 commit intofreifunk-berlin:mainfrom
pmelange:wirless-lua-check-freq-alt

Conversation

@pmelange
Copy link
Collaborator

@pmelange pmelange commented Feb 19, 2026

Instead of trying to figure out which freq a wireless device uses
based on it's reported freq list using iwinfo, simply use the "band"
option from the wireless config (since OpenWrt 21.02.2). Also remove
deprecated hwmode option.

Kompiliert Nein:
Läuft live Ja: Falter 1.5.0-snapshot, runtest

@pmelange
Copy link
Collaborator Author

This is an alternative to #515

@pktpls
Copy link
Contributor

pktpls commented Feb 19, 2026

One less iwinfo dependency \o/

@pmelange pmelange force-pushed the wirless-lua-check-freq-alt branch from 3145b01 to c5da251 Compare February 19, 2026 14:55
Instead of trying to figure out which freq a wireless device uses
based on it's reported freq list using iwinfo, simply use the "band"
option from the wireless config (since OpenWrt 21.02.2).  Also remove
deprecated hwmode option.

Signed-off-by: Perry Melange <isprotejesvalkata@gmail.com>
@pmelange pmelange force-pushed the wirless-lua-check-freq-alt branch from c5da251 to 595ae2b Compare February 19, 2026 14:56
@pmelange
Copy link
Collaborator Author

updated to get rid of hwmode (deprecated option). Thereby we are 100% no longer using the iwinfo library in our lua code.

@pmelange pmelange requested a review from pktpls February 19, 2026 15:28
@pmelange pmelange changed the title luci-app-ffwizard-falter: getchannel update to use option band luci-app-ffwizard-falter: update to use option band instead of iwinfo Feb 19, 2026
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