[PATCH] firmware: psci: Fix return value from psci_system_suspend()

Sudeep Holla sudeep.holla at arm.com
Mon Jan 8 11:53:33 PST 2024


Hi Mark/Lorenzo,

On Mon, Nov 13, 2023 at 03:34:46PM +0000, Sudeep Holla wrote:
> Currently we return the value from invoke_psci_fn() directly as return
> value from psci_system_suspend(). It is wrong to send the PSCI interface
> return value directly. psci_to_linux_errno() provide the mapping from
> PSCI return value to the one that can be returned to the callers within
> the kernel.
> 
> Use psci_to_linux_errno() to convert and return the correct value from
> psci_system_suspend().
> 
> Fixes: faf7ec4a92c0 ("drivers: firmware: psci: add system suspend support")
> Signed-off-by: Sudeep Holla <sudeep.holla at arm.com>
> ---
>  drivers/firmware/psci/psci.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> Hi,
> 
> For some reason, this has gone unnoticed for years. I bumped into this when
> I was trying to test suspend on FVP which claims to support but returns error
> when called. The error was getting not communicated properly(incorrect
> error code) before this patch.
> 

Gentle ping! I had forgotten about this, if you are happy with the change,
I can ask Arnd to pick it up as fix during v6.8 cycle.

-- 
Regards,
Sudeep



More information about the linux-arm-kernel mailing list