2022-01-26 02:58:08

by Adam Ford

Subject: [PATCH V4 08/11] arm64: dts: imx8mm: Fix VPU Hanging

The vpumix power domain has a reset assigned to it, however
when used, it causes a system hang. Testing has shown that
it does not appear to be needed anywhere.

Fixes: d39d4bb15310 ("arm64: dts: imx8mm: add GPC node")
Signed-off-by: Adam Ford <[email protected]>

diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index f77f90ed416f..0c7a72c51a31 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -707,7 +707,6 @@ pgc_vpumix: power-domain@6 {
clocks = <&clk IMX8MM_CLK_VPU_DEC_ROOT>;
assigned-clocks = <&clk IMX8MM_CLK_VPU_BUS>;
assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_800M>;
- resets = <&src IMX8MQ_RESET_VPU_RESET>;
};

pgc_vpu_g1: power-domain@7 {
--
2.32.0

2022-01-26 03:29:00

by Lucas Stach

Subject: Re: [PATCH V4 08/11] arm64: dts: imx8mm: Fix VPU Hanging

Am Dienstag, dem 25.01.2022 um 11:11 -0600 schrieb Adam Ford:
> The vpumix power domain has a reset assigned to it, however
> when used, it causes a system hang. Testing has shown that
> it does not appear to be needed anywhere.
>
> Fixes: d39d4bb15310 ("arm64: dts: imx8mm: add GPC node")
> Signed-off-by: Adam Ford <[email protected]>

Reviewed-by: Lucas Stach <[email protected]>

>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> index f77f90ed416f..0c7a72c51a31 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
> @@ -707,7 +707,6 @@ pgc_vpumix: power-domain@6 {
> clocks = <&clk IMX8MM_CLK_VPU_DEC_ROOT>;
> assigned-clocks = <&clk IMX8MM_CLK_VPU_BUS>;
> assigned-clock-parents = <&clk IMX8MM_SYS_PLL1_800M>;
> - resets = <&src IMX8MQ_RESET_VPU_RESET>;
> };
>
> pgc_vpu_g1: power-domain@7 {

2022-02-09 12:49:26

by Shawn Guo

Subject: Re: [PATCH V4 08/11] arm64: dts: imx8mm: Fix VPU Hanging

On Tue, Jan 25, 2022 at 11:11:25AM -0600, Adam Ford wrote:
> The vpumix power domain has a reset assigned to it, however
> when used, it causes a system hang. Testing has shown that
> it does not appear to be needed anywhere.
>
> Fixes: d39d4bb15310 ("arm64: dts: imx8mm: add GPC node")
> Signed-off-by: Adam Ford <[email protected]>

Applied, thanks!