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


_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to