2023-07-07 10:21:08

by Benjamin Gaignard

Subject: [PATCH] ARM64: dts: freescale: Fix VPU G2 clock

Set VPU G2 clock to 300MHz like described in documentation.
This fix pixels error occurring with large resolution ( >= 2560x1600)
HEVC test stream when using the postprocessor to produce NV12.

Fixes: 4ac7e4a81272 ("arm64: dts: imx8mq: Enable both G1 and G2 VPU's with vpu-blk-ctrl")
Signed-off-by: Benjamin Gaignard <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 0492556a10db..345c70c6c697 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -770,7 +770,7 @@ pgc_vpu: power-domain@6 {
<&clk IMX8MQ_SYS1_PLL_800M>,
<&clk IMX8MQ_VPU_PLL>;
assigned-clock-rates = <600000000>,
- <600000000>,
+ <300000000>,
<800000000>,
<0>;
};
--
2.39.2

2023-07-18 07:22:51

by Shawn Guo

Subject: Re: [PATCH] ARM64: dts: freescale: Fix VPU G2 clock

On Fri, Jul 07, 2023 at 12:08:34PM +0200, Markus Elfring wrote:
> > Set VPU G2 clock to 300MHz like described in documentation.
> > This fix pixels error …
>
> Would you like to avoid typos in such a change description?

I assume you are suggesting s/fix/fixes? I can fix it up when applying.
Thanks!

Shawn

2023-07-18 07:58:13

by Shawn Guo

Subject: Re: [PATCH] ARM64: dts: freescale: Fix VPU G2 clock

On Fri, Jul 07, 2023 at 11:42:00AM +0200, Benjamin Gaignard wrote:
> Set VPU G2 clock to 300MHz like described in documentation.
> This fix pixels error occurring with large resolution ( >= 2560x1600)
> HEVC test stream when using the postprocessor to produce NV12.
>
> Fixes: 4ac7e4a81272 ("arm64: dts: imx8mq: Enable both G1 and G2 VPU's with vpu-blk-ctrl")
> Signed-off-by: Benjamin Gaignard <[email protected]>

We use prefix "arm64: dts: ..." for arm64 DTS changes. I fixed it up
and applied the patch.

Shawn