[PATCH v1] arm64: dts: imx8mm: fix missing pgc_vpu_* power domain parent

Lucas Stach l.stach at pengutronix.de
Tue Apr 9 02:13:13 PDT 2024


Hi Vitor,

Am Dienstag, dem 09.04.2024 um 09:58 +0100 schrieb Vitor Soares:
> From: Vitor Soares <vitor.soares at toradex.com>
> 
> The pgc_vpu_* nodes miss the reference to the power domain parent,
> leading the system to hang during the resume.
> 
This change is not correct. The vpumix domain is controlled through the
imx8mm-vpu-blk-ctrl and must not be directly triggered by the child
domains in order to guarantee proper power sequencing.

If the sequencing is incorrect for resume, it needs to be fixed in the
blk-ctrl driver. I'll happily assist if you have any questions about
this intricate mix between GPC and blk-ctrl hardware/drivers.

Regards,
Lucas

> As these PU domains are nested inside the vpumix domain, let's reference
> it accordingly. After this change, the suspend/resume is working.
> 
> Cc: Lucas Stach <l.stach at pengutronix.de>
> Cc: <stable at vger.kernel.org>
> Closes: https://lore.kernel.org/all/fccbb040330a706a4f7b34875db1d896a0bf81c8.camel@gmail.com/
> Fixes: d39d4bb15310 ("arm64: dts: imx8mm: add GPC node")
> Signed-off-by: Vitor Soares <vitor.soares at toradex.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8mm.dtsi | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> index 8a1b42b94dce..97d0c6d23ad8 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> @@ -739,16 +739,19 @@ pgc_vpumix: power-domain at 6 {
>  					pgc_vpu_g1: power-domain at 7 {
>  						#power-domain-cells = <0>;
>  						reg = <IMX8MM_POWER_DOMAIN_VPUG1>;
> +						power-domains = <&pgc_vpumix>;
>  					};
>  
>  					pgc_vpu_g2: power-domain at 8 {
>  						#power-domain-cells = <0>;
>  						reg = <IMX8MM_POWER_DOMAIN_VPUG2>;
> +						power-domains = <&pgc_vpumix>;
>  					};
>  
>  					pgc_vpu_h1: power-domain at 9 {
>  						#power-domain-cells = <0>;
>  						reg = <IMX8MM_POWER_DOMAIN_VPUH1>;
> +						power-domains = <&pgc_vpumix>;
>  					};
>  
>  					pgc_dispmix: power-domain at 10 {




More information about the linux-arm-kernel mailing list