On 25/11/2024 16:49, Renjiang Han (QUIC) wrote: >>> + video-decoder: >>> + type: object >>> + >>> + additionalProperties: false >>> + >>> + properties: >>> + compatible: >>> + const: venus-decoder >>> + >>> + required: >>> + - compatible >>> + >>> + video-encoder: >>> + type: object > >> Both nodes are useless - no resources here, nothing to control. >> Do not add nodes just to instantiate Linux drivers. Drop them. > Do you mean I should remove video-decoder and video-encoder from here? Yes, that's my suggestion. > If so, do I also need to remove these two nodes from the dtsi file and add Yes > them in the qcs615-ride.dts file? Well, no, how would it pass dtbs_check? Don't add nodes purely for Linux driver instantiation. Best regards, Krzysztof
