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(-)
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.5On 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.
>
Using fluster, I wanted to test this. I didn't test fluster before this:
./fluster.py run -d GStreamer-VP8-V4L2SL-Gst1.0
Ran 57/61 tests successfully in 7.059 secs
./fluster.py run -dGStreamer-H.264-V4L2SL-Gst1.0
Ran 129/135 tests successfully in 45.741 secs
Both of these outperformed the tests I ran a while ago on the imx8mm
running at 600MHz. I haven't updated my clocks to run at overdrive
rates yet, but if you want, I could help with that. Either way, I
think you could push submit patch as a stand-alone or with some clock
updates even before the adding the encoder stuff.
Tested-by: Adam Ford <aford173@gmail.com> #imx8mp-beacon-kit
> 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
>
>
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
suspend-resume works properly?
Should this get a fixes tag?
> 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
>
>
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
> >
> >
>
© 2016 - 2026 Red Hat, Inc.