On 5/6/2024 3:55 PM, Bryan O'Donoghue wrote: > On 06/05/2024 10:57, Marc Gonzalez wrote: >> On 30/04/2024 17:28, Marc Gonzalez wrote: >> >>> Changes in v2 >>> - Add Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> for patches >>> 2 & 3 >>> - Replace qcom,msm8998-venus.yaml (copy of qcom,msm8996-venus.yaml) with item >>> in qcom,msm8996-venus.yaml >>> >>> Marc Gonzalez (3): >>> dt-bindings: media: add qcom,msm8998-venus >>> arm64: dts: qcom: msm8998: add venus node >>> media: venus: add MSM8998 support >>> >>> Documentation/devicetree/bindings/media/qcom,msm8996-venus.yaml | 4 ++- >>> arch/arm64/boot/dts/qcom/msm8998.dtsi | 48 >>> +++++++++++++++++++++++++++++ >>> drivers/media/platform/qcom/venus/core.c | 42 >>> +++++++++++++++++++++++++ >>> 3 files changed, 93 insertions(+), 1 deletion(-) >> >> Not sure what's holding up this series? >> Can it be merged before the 6.10 merge window opens? >> (Whose tree is it supposed to go through?) >> >> Been working on this feature since Feb 19 with >> [RFC WIP PATCH] venus: add qcom,no-low-power property >> (First try turned out to be incorrect) >> >> Regards >> > > Vikash, Stan. > > I think this is ready to go, can we get an Acked-by so Hans can do the merge ? Thanks Marc/Bryan for getting this through and getting rid of that hack "no-low-power". There is a minor comment in patch 3/3, otherwise looks good. Regards, Vikash