[PATCH] arm64: dts: imx8mp: Add Hantro G1, G2, VC8000E DT nodes

[Adam Ford]

On Sat, Dec 17, 2022 at 8:37 PM Marek Vasut <marex at denx.de> wrote:
>
> On 12/18/22 02:36, Laurent Pinchart wrote:
>
> Hi,
>
> [...]
>
> >> diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> index 34c2c80c47eb5..5040b3a545c32 100644
> >> --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> >> @@ -1639,6 +1639,39 @@ gpu2d: gpu at 38008000 {
> >>                      power-domains = <&pgc_gpu2d>;
> >>              };
> >>
> >> +            vpu_g1: video-codec at 38300000 {
> >> +                    compatible = "nxp,imx8mm-vpu-g1";
> >> +                    reg = <0x38300000 0x10000>;
> >> +                    interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
> >> +                    clocks = <&clk IMX8MP_CLK_VPU_G1_ROOT>;
> >> +                    assigned-clocks = <&clk IMX8MP_CLK_VPU_G1>;
> >> +                    assigned-clock-parents = <&clk IMX8MP_VPU_PLL_OUT>;
> >> +                    assigned-clock-rates = <700000000>;
> >> +                    power-domains = <&vpumix_blk_ctrl IMX8MP_VPUBLK_PD_G1>;
> >> +            };
> >> +
> >> +            vpu_g2: video-codec at 38310000 {
> >> +                    compatible = "nxp,imx8mq-vpu-g2";
> >> +                    reg = <0x38310000 0x10000>;
> >> +                    interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
> >> +                    clocks = <&clk IMX8MP_CLK_VPU_G2_ROOT>;
> >> +                    assigned-clocks = <&clk IMX8MP_CLK_VPU_G2>;
> >> +                    assigned-clock-parents = <&clk IMX8MP_VPU_PLL_OUT>;
> >> +                    assigned-clock-rates = <700000000>;
> >> +                    power-domains = <&vpumix_blk_ctrl IMX8MP_VPUBLK_PD_G2>;
> >> +            };
> >> +
> >> +            vpu_vc8000e: vpu_vc8000e at 38320000 {
> >> +                    compatible = "nxp,imx8mp-vpu-vc8000e";
> >
> > I tried to look for the patch that defined this compatible value, but
> > couldn't find it. Same for driver support. Have they been posted
> > somewhere ?
>
> Not the vc8000e to my knowledge, the other two are upstream.

For what it's worth, I'm still working on getting my imx8mp up and
running with a mainline kernel, but the g1 and g2 enumerate:

[    7.039007] hantro-vpu 38300000.video-codec: registered
nxp,imx8mm-vpu-g1-dec as /dev/video0
[    7.054049] hantro-vpu 38310000.video-codec: registered
nxp,imx8mq-vpu-g2-dec as /dev/video1

I'm hoping to get a more mainline version of g-streamer to be able to
do video decoding and run some fluster tests, but I want to get HDMI
and DSI video working before I'll have more time to spend on the
decoders.
If there isn't a driver for the vc8000e, I wonder if it wouldn't make
sense to post a V2 with just the G1 and G2 decoders enabled.

adam