Re: [PATCH] irqchip: fixed S3 could not resume after suspend

[Marc Zyngier]

On Fri, 09 Apr 2021 11:10:09 +0100,
414777006@xxxxxx wrote:
> 
> From: Mengguang Peng <pengmengguang@xxxxxxxxxxxxxx>
> 
> On arm64 platform, found that the machine could not wake up after suspend,
> this patch updates the its suspend and resume handling code.
> 
> - Add a variable named ctlr_save in struct rdists.
> - When suspend, save the value of GICR_CTLR to memmory
> 	in its_save_disable().
> - When resume, write the value of memory saved to GICR_CTLR
> 	in its_restore_enable().

This really is the kind of things that must be handled by firmware.
How comes yours doesn't handle it?

That is what you should be fixing. See the ATF code for a (working)
reference implementation

Thanks,

	M.

-- 
Without deviation from the norm, progress is not possible.