[PATCH v2] arm64: dts: mt8192: Add vcodec lat and core nodes

[Allen-KH Cheng (程冠勳)]

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 at mediatek.com>
> > Tested-by: Chen-Yu Tsai <wenst at chromium.org>
> > ---
> > Change in v1:
> >    * Replace node names with video-codec
> >      [Allen-KH Cheng <allen-kh.cheng at mediatek.com>]
> > ---
> > ---
> >   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 at 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 at 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 at 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?

Thanks,
Allen