Re: [PATCH v5 3/4] arm64: dts: qcom: qcs615: add venus node

From: Bryan O'Donoghue
Date: Tue Dec 17 2024 - 04:38:29 EST


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

I gave you feedback on this in v4.

Could you please provide some commentary on why you're persisting with this ?

- Driver configuration should not live in dts
- A patchset exists to mitigate this
- If you don't want to use that series, what do you propose
to resolve this ?

Please don't just ignore feedback, either act on it or add to your commit log _why_ you didn't act on it.

---
bod