[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8680b702-7890-4abc-a4c0-49bce5a11177@oss.qualcomm.com>
Date: Thu, 26 Jun 2025 20:24:09 +0200
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: Jorge Ramirez <jorge.ramirez@....qualcomm.com>,
Krzysztof Kozlowski <krzk@...nel.org>
Cc: krzk+dt@...nel.org, bryan.odonoghue@...aro.org, quic_vgarodia@...cinc.com,
quic_dikshita@...cinc.com, mchehab@...nel.org, robh@...nel.org,
conor+dt@...nel.org, konradybcio@...nel.org, andersson@...nel.org,
linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 5/5] arm64: dts: qcom: qcm2290: Add venus video node
On 6/26/25 4:25 PM, Jorge Ramirez wrote:
> On 26/06/25 16:05:00, Krzysztof Kozlowski wrote:
>> On 26/06/2025 15:59, Jorge Ramirez-Ortiz wrote:
>>> +
>>> + interconnects = <&mmnrt_virt MASTER_VIDEO_P0 RPM_ALWAYS_TAG
>>> + &bimc SLAVE_EBI1 RPM_ALWAYS_TAG>,
>>> + <&bimc MASTER_APPSS_PROC RPM_ACTIVE_TAG
>>> + &config_noc SLAVE_VENUS_CFG RPM_ACTIVE_TAG>;
>>> + interconnect-names = "video-mem",
>>> + "cpu-cfg";
>>> +
>>> + status = "okay";
>>
>> Drop, unless you override existing node, but then this should follow
>> standard override-label/phandle syntax.
>>
>
> yep
With that taken care of:
Reviewed-by: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
Konrad
Powered by blists - more mailing lists