[PATCH v1 5/5] realtek: remove hardcoded sys-led configurations
Sander Vanheule
sander at svanheule.net
Tue Jun 7 02:10:47 PDT 2022
On Tue, 2022-06-07 at 10:24 +0200, Birger Koblitz wrote:
> Hi,
>
> at least for the RTL931x, removing the rtl931x_setup() is not a good idea as the WDT reset does
> not work for that architecture.
> The only way to get a working reset is via registering a reset handler:
>
> static void __init rtl931x_setup(void)
> {
> pr_info("Registering _machine_restart\n");
> _machine_restart = rtl931x_restart;
> _machine_halt = rtl931x_halt;
We've gotten rid of _machine_restart and _machine_halt already, let's not reintroduce them.
If you don't feel like writing a driver for the reset controller, perhaps syscon-reboot [1] can be
useful?
Best,
Sander
[1] https://mjmwired.net/kernel/Documentation/devicetree/bindings/power/reset/syscon-reboot.yaml
More information about the openwrt-devel
mailing list