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

[Marek Vasut]

On 12/18/22 05:16, Adam Ford wrote:
> 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.

Let's do that