Re: [PATCH v4 3/4] arm64: dts: qcom: add venus node for the qcs615

From: Bryan O'Donoghue
Date: Fri Dec 13 2024 - 06:31:20 EST


On 13/12/2024 09:56, Renjiang Han wrote:
+ video-decoder {
+ compatible = "venus-decoder";
+ };
+
+ video-encoder {
+ compatible = "venus-encoder";
+ };

To be honest, I'd prefer not to continue on doing this.

Adding a compat string to existing yaml might work-around the issue but, it doesn't really _account_ for the issue.

I've posted a series to fix the problem

20241209-media-staging-24-11-25-rb3-hw-compat-string-v5-0-ef7e5f85f302@xxxxxxxxxx

Either that or deeper rewrite of venus to remove the above dependencies should go ahead.

I'm not in favour of willfully continuing to do the wrong thing, especially when per above, there's a working solution for it.

---
bod