Linux-Media Archive mirror
 help / color / mirror / Atom feed
From: Vikash Garodia <quic_vgarodia@quicinc.com>
To: Bryan O'Donoghue <bryan.odonoghue@linaro.org>,
	Marc Gonzalez <mgonzalez@freebox.fr>,
	Bjorn Andersson <andersson@kernel.org>,
	Jeffrey Hugo <quic_jhugo@quicinc.com>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	"Stanimir Varbanov" <stanimir.k.varbanov@gmail.com>
Cc: MSM <linux-arm-msm@vger.kernel.org>,
	linux-media <linux-media@vger.kernel.org>,
	DT <devicetree@vger.kernel.org>,
	"Pierre-Hugues Husson" <phhusson@freebox.fr>,
	Arnaud Vrac <avrac@freebox.fr>, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>
Subject: Re: [PATCH v2 0/3] Add support for qcom msm8998-venus (HW vdec / venc)
Date: Mon, 6 May 2024 17:52:01 +0530	[thread overview]
Message-ID: <0bce5d7c-8133-f3c7-55af-c5b8f89a5371@quicinc.com> (raw)
In-Reply-To: <12daeed1-0075-4a6c-bd02-dc70a0c0d721@linaro.org>


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

      reply	other threads:[~2024-05-06 12:22 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-30 15:28 [PATCH v2 0/3] Add support for qcom msm8998-venus (HW vdec / venc) Marc Gonzalez
2024-04-30 15:30 ` [PATCH v2 1/3] dt-bindings: media: add qcom,msm8998-venus Marc Gonzalez
2024-04-30 16:13   ` Bryan O'Donoghue
2024-04-30 16:37   ` Krzysztof Kozlowski
2024-04-30 15:32 ` [PATCH v2 2/3] arm64: dts: qcom: msm8998: add venus node Marc Gonzalez
2024-04-30 15:34 ` [PATCH v2 3/3] media: venus: add msm8998 support Marc Gonzalez
2024-05-06 12:19   ` Vikash Garodia
2024-05-06 13:18     ` Marc Gonzalez
2024-05-06  9:57 ` [PATCH v2 0/3] Add support for qcom msm8998-venus (HW vdec / venc) Marc Gonzalez
2024-05-06 10:25   ` Bryan O'Donoghue
2024-05-06 12:22     ` Vikash Garodia [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=0bce5d7c-8133-f3c7-55af-c5b8f89a5371@quicinc.com \
    --to=quic_vgarodia@quicinc.com \
    --cc=andersson@kernel.org \
    --cc=avrac@freebox.fr \
    --cc=bryan.odonoghue@linaro.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mgonzalez@freebox.fr \
    --cc=phhusson@freebox.fr \
    --cc=quic_jhugo@quicinc.com \
    --cc=robh+dt@kernel.org \
    --cc=stanimir.k.varbanov@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).