-
Notifications
You must be signed in to change notification settings - Fork 21
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
How to indicate latest flash on Belkin RT3200 #633
Comments
Unless anyone has better ideas, I propose the following color code for the "Internet LED": Year Ending: Color: Further, I propose the following for the power LED: Feedback welcomed and encouraged. |
Thanks @owendelong Im good with this proposal. The only thing I would want to confirm is that it doesnt conflict with the existing LED states of the AP. Things like |
Wouldn't context be sufficient for this as well? We know when we're flashing or trying to flash something, so we wouldn't expect to see those blinking states in the wild and they'd be pretty distinctive when we are flashing, I would think. Since we wouldn't take control of the LEDs to represent our intended state until our flashed-on firmware is loaded and running and networked, I don't think there's a conflict. Let me know if you think I'm missing something important. It does occur to me that I didn't have a state to represent Power good, internet available, all radios down. Perhaps we should make that Yellow and make Power + 2.4G OK, 5G down Green (0x00ff00) instead. |
I don't think there will be many cases where we have one or the other band down,
so I think all radios up or all radios down is sufficient.
that frees up a slot for other purposes
David Lang
…On Wed, 8 Nov 2023, Owen DeLong wrote:
Wouldn't context be sufficient for this as well? We know when we're flashing or trying to flash something, so we wouldn't expect to see those blinking states in the wild and they'd be pretty distinctive when we are flashing, I would think.
Since we wouldn't take control of the LEDs to represent our intended state until our flashed-on firmware is loaded and running and networked, I don't think there's a conflict. Let me know if you think I'm missing something important.
It does occur to me that I didn't have a state to represent Power good, internet available, all radios down. Perhaps we should make that Yellow and make Power + 2.4G OK, 5G down Green (0x00ff00) instead.
|
I don't think there's a "free slot" issue here. We have at least 10 easily distinguishable colors (likely considerably more) on each of 2 LEDs... If we're really worried, that gives us at least 100 different meanings we could convey, but I really don't want to make this that complex. However, if we want to indicate other things beyond what I've proposed for power, I don't think we need to give up the individual radio indications to do so. |
Closing since the Belkin 3200 mt-7622 has been superseded by the Openwrt One mt798x |
Description
From the discussion #532 (comment)
Acceptance Criteria
The text was updated successfully, but these errors were encountered: