[PATCH 2/4] microblaze: reset: Call POWER_OFF handlers

[Thomas Weißschuh]

System power off might be implemented through sys_off handlers.

Currently these are not respected on microblaze.

On power_off call into the generic power off function which will execute
all regustered handlers.

Signed-off-by: Thomas Weißschuh <li...@weissschuh.net>
---
arch/microblaze/kernel/reset.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/microblaze/kernel/reset.c b/arch/microblaze/kernel/reset.c
index 2f66c7963084..3612a20ca16d 100644
--- a/arch/microblaze/kernel/reset.c
+++ b/arch/microblaze/kernel/reset.c
@@ -27,6 +27,7 @@ void machine_halt(void)

void machine_power_off(void)
{
+ do_kernel_power_off();
pr_notice("Machine power off...\n");
while (1)
;

--
2.55.0

[David Gow]

Le 04/08/2026 à 13:32, Thomas Weißschuh a écrit :
> System power off might be implemented through sys_off handlers.
>
> Currently these are not respected on microblaze.
>
> On power_off call into the generic power off function which will execute
> all regustered handlers.
>
> Signed-off-by: Thomas Weißschuh <li...@weissschuh.net>
> ---

Seems sensible enough.

Acked-by: David Gow <da...@davidgow.net>

Cheers,
-- David

[Michal Simek]

Likely it should be here. Or you will never see this notice.


> while (1)
> ;
>

M

--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP/Versal ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal/Versal NET SoCs
TF-A maintainer - Xilinx ZynqMP/Versal/Versal NET SoCs

[Thomas Weißschuh]

On 2026-08-31 10:13:52+0200, Michal Simek wrote:
> On 8/4/26 07:32, Thomas Weißschuh wrote:
> > System power off might be implemented through sys_off handlers.
> >
> > Currently these are not respected on microblaze.
> >
> > On power_off call into the generic power off function which will execute
> > all regustered handlers.
> >
> > Signed-off-by: Thomas Weißschuh <li...@weissschuh.net>
> > ---
> > arch/microblaze/kernel/reset.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/microblaze/kernel/reset.c b/arch/microblaze/kernel/reset.c
> > index 2f66c7963084..3612a20ca16d 100644
> > --- a/arch/microblaze/kernel/reset.c
> > +++ b/arch/microblaze/kernel/reset.c
> > @@ -27,6 +27,7 @@ void machine_halt(void)
> > void machine_power_off(void)
> > {
> > + do_kernel_power_off();
> > pr_notice("Machine power off...\n");
>
> Likely it should be here. Or you will never see this notice.

Ack, will change.

> > while (1)
> > ;
> >