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

FS#1965 - BT Home Hub 5A - 18.06 snapshots- Router crashes when the soft reset "Restart" key is pressed #7168

Closed
openwrt-bot opened this issue Nov 23, 2018 · 4 comments
Labels

Comments

@openwrt-bot
Copy link

VaxMan:

BT HomeHub 5A
Latest snapshot dated Fri Nov 23 03:38:09 2018

After pressing the "Restart" button of the top left of the router, all LEDs go dark and the router becomes unresponsive.
Power Cycling the router will bring everything back up though.

@openwrt-bot
Copy link
Author

VaxMan:

Update:
Well, this is a little awkward. Looking at the code, it seems that the “RESTART” key is and was always intended to execute ’ /sbin/poweroff’ just that executing ‘sbin/poweroff’ has never actually resulted in powering off / halting the router before and has always resulted in the router restarting. Why anyone would want to power the BTHOMEHUB V5 off using the big ass button, labeled "RESTART", on the top left of the unit, which is so easily deployed accidentally by a cat or subtle fluctuations in the earth’s gravitational field, when there is a perfectly decent off/on switch at the back, astounds me and leaves me wondering, have I been in the matrix all this time?
Anyways, my opinion is not what is important here and I have also seen many people who own different routers complaining bitterly that executing ‘sbin/poweroff’ does not HALT or power their units off. I will leave it to those who are more qualified to make those kinds of decisions as to whether it is correct or not. I have modified my version of the code so that, for me at least, the router continues to do what it was created to do... RUN.

Regards,
Alex

@openwrt-bot
Copy link
Author

Reiver:

There aren't separate restart/reset button event codes so BTN_RESTART [0x198] had been multitasked to both restart and reset buttons. To prevent accidental (or malicious) factory resets, the following commit changed the restart button for Home Hub 5;
[[https://git.openwrt.org/?p=openwrt/openwrt.git;a=commitdiff;h=153b3f05d40aa558dfffdc617c0d0d285be26165]]

I assume you changed /etc/rc.button/power [package/base-files/files/etc/rc.button/power] to do a reboot preferably with a 5 second SEEN delay such as in /etc/rc.button/reset?

Might be useful to map a new button with corresponding rc.button entry with a 5 second SEEN delay till reboot for devices where a power off isn't useful. Perhaps KEY_POWER2 [0x164].

@openwrt-bot
Copy link
Author

VaxMan:

This is kinda old now but yes I redirected to reset with a short delay.

Faulty memory^^ and a little confusion between projects. It seems that in the end I simply reversed the commit you referred to.

TBH I have not used anything other than the code I last generated at the end of 2018 for any of my routers (currently 6) as I don't see any need to interfere with something that has been working solidly and I am happy with.
I have no idea what the latest code does but if the 'Restart' key still powers the router down, my opinion remains the same. A 'Restart' button IS handy but having this button power the router down when it is so easily pressed by accident or by a family pet is Trumpesque. There certainly should be some provision for maintaining the ORIGINAL function of any keys on this/any router when it is possible to do so.

@openwrt-bot
Copy link
Author

Reiver:

Created pull request adding reboot button script to resolve;
[[https://github.com//pull/2028]]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant