Re: [PATCH v5 4/5] arm64: dts: qcom: qcs8300: add support for video node

[Bryan O'Donoghue <bryan.odonoghue@linaro.org>]

On 24/04/2025 13:20, Bryan O'Donoghue wrote:
> On 24/04/2025 11:28, Vikash Garodia wrote:
>>
>> On 4/24/2025 2:51 PM, Dmitry Baryshkov wrote:
>>> On Thu, Apr 24, 2025 at 02:20:48PM +0530, Vikash Garodia wrote:
>>>> Add the IRIS video-codec node on QCS8300.
>>>
>>> Nit: you can not "add support for the video node". You can either add
>>> video node or add support for video en/decoding.
>> Makes sense. Will wait for any other comments, before resending.
>>
>> Regards,
>> Vikash
>>>
>>>>
>>>> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
>>>> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
>>>> ---
>>>>   arch/arm64/boot/dts/qcom/qcs8300.dtsi | 71 +++++++++++++++++++++++ 
>>>> ++++++++++++
>>>>   1 file changed, 71 insertions(+)
>>>>
>>>
> 
> Unless you get another comment, there's no need to resend.
> 
> I can fix the commit log for you on the way in.
> 
> ---
> bod

Oops this is isn't one I can fix for you - dtsi.

---
bod