mbox series

[v2,0/3] Add support for qcom msm8998-venus (HW vdec / venc)

Message ID ff646f97-68e3-4fef-9b56-2bd98f0cbe7d@freebox.fr (mailing list archive)
Headers show
Series Add support for qcom msm8998-venus (HW vdec / venc) | expand

Message

Marc Gonzalez April 30, 2024, 3:28 p.m. UTC
Changes in v2
- Add Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> 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(-)

Comments

Marc Gonzalez May 6, 2024, 9:57 a.m. UTC | #1
On 30/04/2024 17:28, Marc Gonzalez wrote:

> Changes in v2
> - Add Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> 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
Bryan O'Donoghue May 6, 2024, 10:25 a.m. UTC | #2
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@linaro.org> 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 ?

---
bod
Vikash Garodia May 6, 2024, 12:22 p.m. UTC | #3
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@linaro.org> 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