| Message ID | 20250502150513.4169098-6-m.felsch@pengutronix.de |
|---|---|
| State | New |
| Headers | show |
| Series | VC8000E H.264 V4L2 Stateless Encoder | expand |
On 25-05-02, Adam Ford wrote: > On Fri, May 2, 2025 at 10:10 AM Marco Felsch <m.felsch@pengutronix.de> wrote: > > > > The GPCv2 G1, G2 and VC8000E power-domain don't need to reference the > > VPUMIX power-domain nor their module clocks since the power and reset > > handling is done by the VPUMIX blkctrl driver. > > > It was my understanding that having this dependency ensures the order > of the bring-up, but maybe I am wrong. Do you know if the 8MP If that is true, the 8MM should be broken. > suspend-resume works properly? No I didn't test suspend/resume. > Should this get a fixes tag? It's just a cleanup, therefore I didn't add the fixes-tag. Regards, Marco > > Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> > > --- > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 7 ------- > > 1 file changed, 7 deletions(-) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > index e0d3b8cba221..cf9b6c487bd5 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi > > @@ -879,24 +879,17 @@ pgc_mediamix: power-domain@10 { > > > > pgc_vpu_g1: power-domain@11 { > > #power-domain-cells = <0>; > > - power-domains = <&pgc_vpumix>; > > reg = <IMX8MP_POWER_DOMAIN_VPU_G1>; > > - clocks = <&clk IMX8MP_CLK_VPU_G1_ROOT>; > > }; > > > > pgc_vpu_g2: power-domain@12 { > > #power-domain-cells = <0>; > > - power-domains = <&pgc_vpumix>; > > reg = <IMX8MP_POWER_DOMAIN_VPU_G2>; > > - clocks = <&clk IMX8MP_CLK_VPU_G2_ROOT>; > > - > > }; > > > > pgc_vpu_vc8000e: power-domain@13 { > > #power-domain-cells = <0>; > > - power-domains = <&pgc_vpumix>; > > reg = <IMX8MP_POWER_DOMAIN_VPU_VC8000E>; > > - clocks = <&clk IMX8MP_CLK_VPU_VC8KE_ROOT>; > > }; > > > > pgc_hdmimix: power-domain@14 { > > -- > > 2.39.5 > > > > >
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index e0d3b8cba221..cf9b6c487bd5 100644 --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi @@ -879,24 +879,17 @@ pgc_mediamix: power-domain@10 { pgc_vpu_g1: power-domain@11 { #power-domain-cells = <0>; - power-domains = <&pgc_vpumix>; reg = <IMX8MP_POWER_DOMAIN_VPU_G1>; - clocks = <&clk IMX8MP_CLK_VPU_G1_ROOT>; }; pgc_vpu_g2: power-domain@12 { #power-domain-cells = <0>; - power-domains = <&pgc_vpumix>; reg = <IMX8MP_POWER_DOMAIN_VPU_G2>; - clocks = <&clk IMX8MP_CLK_VPU_G2_ROOT>; - }; pgc_vpu_vc8000e: power-domain@13 { #power-domain-cells = <0>; - power-domains = <&pgc_vpumix>; reg = <IMX8MP_POWER_DOMAIN_VPU_VC8000E>; - clocks = <&clk IMX8MP_CLK_VPU_VC8KE_ROOT>; }; pgc_hdmimix: power-domain@14 {
The GPCv2 G1, G2 and VC8000E power-domain don't need to reference the VPUMIX power-domain nor their module clocks since the power and reset handling is done by the VPUMIX blkctrl driver. Signed-off-by: Marco Felsch <m.felsch@pengutronix.de> --- arch/arm64/boot/dts/freescale/imx8mp.dtsi | 7 ------- 1 file changed, 7 deletions(-)