On 19/12/2024 03:45, Renjiang Han wrote: > > On 12/18/2024 8:38 PM, Krzysztof Kozlowski wrote: >>>> @@ -132,12 +132,4 @@ examples: >>>> resets = <&gcc GCC_VIDEO_AXI0_CLK_ARES>, >>>> <&videocc VIDEO_CC_MVS0C_CLK_ARES>; >>>> reset-names = "bus", "core"; >>>> - >>>> - video-decoder { >>>> - compatible = "venus-decoder"; >>>> - }; >>>> - >>>> - video-encoder { >>>> - compatible = "venus-encoder"; >>>> - }; >>>> }; >>> It is working fine on QCS615. >>> Tested-by: Renjiang Han<quic_renjiang@xxxxxxxxxxx> >> How? How is it possible to test a binding on real hardware? To my >> knowledge it is impossible, so I really would like to see explanation of >> this tag. > I used this binding and modified the driver to verify the video > function, and the result was fine. Also, I used this binding to > verify the dtb, and there was no related error, so I added > Tested-by. This is not a testing of binding. You tested driver. Don't add fake tags. Best regards, Krzysztof