Il 27/09/22 04:19, Allen-KH Cheng (程冠勳) ha scritto:
Hi Angelo,
On Mon, 2022-09-26 at 17:29 +0200, AngeloGioacchino Del Regno wrote:
Il 26/09/22 12:50, Allen-KH Cheng ha scritto:
Add vcodec lat and core nodes for mt8192 SoC.
Signed-off-by: Allen-KH Cheng <allen-kh.cheng@xxxxxxxxxxxx>
Tested-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx>
---
Change in v1:
* Replace node names with video-codec
[Allen-KH Cheng <allen-kh.cheng@xxxxxxxxxxxx>]
---
---
arch/arm64/boot/dts/mediatek/mt8192.dtsi | 60
++++++++++++++++++++++++
1 file changed, 60 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
index 6b20376191a7..214dfc6b0ed1 100644
--- a/arch/arm64/boot/dts/mediatek/mt8192.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8192.dtsi
@@ -1449,6 +1449,66 @@
power-domains = <&spm
MT8192_POWER_DOMAIN_ISP2>;
};
+ vcodec_dec: video-codec@16000000 {
+ compatible = "mediatek,mt8192-vcodec-dec";
+ reg = <0 0x16000000 0 0x1000>;
+ mediatek,scp = <&scp>;
+ iommus = <&iommu0 M4U_PORT_L4_VDEC_MC_EXT>;
+ dma-ranges = <0x1 0x0 0x0 0x40000000 0x0
0xfff00000>;
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges = <0 0 0 0x16000000 0 0x26000>;
+
+ vcodec_lat: video-codec@10000 {
Allen, this won't work :-(
Check Documentation/devicetree/bindings/media/mediatek,vcodec-subdev-
decoder.yaml
...the schema requires vcodec-lat@[0-9a-f] and vcodec-core@[0-9a-f].
If you want to call them all video-codec@addr, you have to also fix
the schema.
P.S.: Did you try to run `make dtbs_check`?
Regards,
Angelo
Sorry, my bad. I alsways run `make dtbs_check` to confirm dtb with
bindings. I just think we didn't limit node names in mtk-vodec
bindings. I will pay attention next time.
Since currently the vcodec lat and core nodes are absent from the mtk
dts, do you think the child node name should be changed to something
more general (ex: video-codec) in mediatek,vcodec-subdev-decoder
bindings?
The video codec is mt8192-vcodec-dec, while the other nodes are describing
the VPU instances (and/or vpu cores)... I'm not sure.
Krzysztof, please, can you give your opinion on that?
Regards,
Angelo
Thanks,
Allen
