mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes
@ 2025-03-03 12:37 Vikash Garodia
  2025-03-05 12:26 ` Bryan O'Donoghue
  2025-03-14 20:01 ` Bjorn Andersson
  0 siblings, 2 replies; 3+ messages in thread
From: Vikash Garodia @ 2025-03-03 12:37 UTC (permalink / raw)
  To: cros-qcom-dts-watchers, Bjorn Andersson, Konrad Dybcio,
	Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-msm, devicetree, linux-kernel, Vikash Garodia

Decoder and encoder nodes are already deprecated from bindings. Update
the venus node to align with bindings. The nodes were deprecated with
commit 459997e8990d9 ("media: dt-bindings: qcom-venus: Deprecate
video-decoder and video-encoder where applicable") and is part of
v6.14-rc1 and onwards.

Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
---
Changes in v2:
- Add history in commit message about the deprecated nodes.
- Link to v1: https://lore.kernel.org/r/20250303-b4-media-v1-1-ddc5c81cc2b3@quicinc.com
---
 arch/arm64/boot/dts/qcom/sc7280.dtsi | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 0f2caf36910b65c398c9e03800a8ce0a8a1f8fc7..31abb2b9555f37ecc9c7753509e95acd57acf015 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -4301,14 +4301,6 @@ venus: video-codec@aa00000 {
 
 			status = "disabled";
 
-			video-decoder {
-				compatible = "venus-decoder";
-			};
-
-			video-encoder {
-				compatible = "venus-encoder";
-			};
-
 			venus_opp_table: opp-table {
 				compatible = "operating-points-v2";
 

---
base-commit: d98e9213a768a3cc3a99f5e1abe09ad3baff2104
change-id: 20250303-b4-media-79ff0738b9cf

Best regards,
-- 
Vikash Garodia <quic_vgarodia@quicinc.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes
  2025-03-03 12:37 [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes Vikash Garodia
@ 2025-03-05 12:26 ` Bryan O'Donoghue
  2025-03-14 20:01 ` Bjorn Andersson
  1 sibling, 0 replies; 3+ messages in thread
From: Bryan O'Donoghue @ 2025-03-05 12:26 UTC (permalink / raw)
  To: Vikash Garodia, cros-qcom-dts-watchers, Bjorn Andersson,
	Konrad Dybcio, Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-msm, devicetree, linux-kernel

On 03/03/2025 12:37, Vikash Garodia wrote:
> Decoder and encoder nodes are already deprecated from bindings. Update
> the venus node to align with bindings. The nodes were deprecated with
> commit 459997e8990d9 ("media: dt-bindings: qcom-venus: Deprecate
> video-decoder and video-encoder where applicable") and is part of
> v6.14-rc1 and onwards.
> 
> Signed-off-by: Vikash Garodia <quic_vgarodia@quicinc.com>
> ---
> Changes in v2:
> - Add history in commit message about the deprecated nodes.
> - Link to v1: https://lore.kernel.org/r/20250303-b4-media-v1-1-ddc5c81cc2b3@quicinc.com
> ---
>   arch/arm64/boot/dts/qcom/sc7280.dtsi | 8 --------
>   1 file changed, 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> index 0f2caf36910b65c398c9e03800a8ce0a8a1f8fc7..31abb2b9555f37ecc9c7753509e95acd57acf015 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> @@ -4301,14 +4301,6 @@ venus: video-codec@aa00000 {
>   
>   			status = "disabled";
>   
> -			video-decoder {
> -				compatible = "venus-decoder";
> -			};
> -
> -			video-encoder {
> -				compatible = "venus-encoder";
> -			};
> -
>   			venus_opp_table: opp-table {
>   				compatible = "operating-points-v2";
>   
> 
> ---
> base-commit: d98e9213a768a3cc3a99f5e1abe09ad3baff2104
> change-id: 20250303-b4-media-79ff0738b9cf
> 
> Best regards,
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes
  2025-03-03 12:37 [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes Vikash Garodia
  2025-03-05 12:26 ` Bryan O'Donoghue
@ 2025-03-14 20:01 ` Bjorn Andersson
  1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Andersson @ 2025-03-14 20:01 UTC (permalink / raw)
  To: cros-qcom-dts-watchers, Konrad Dybcio, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Vikash Garodia
  Cc: linux-arm-msm, devicetree, linux-kernel


On Mon, 03 Mar 2025 18:07:01 +0530, Vikash Garodia wrote:
> Decoder and encoder nodes are already deprecated from bindings. Update
> the venus node to align with bindings. The nodes were deprecated with
> commit 459997e8990d9 ("media: dt-bindings: qcom-venus: Deprecate
> video-decoder and video-encoder where applicable") and is part of
> v6.14-rc1 and onwards.
> 
> 
> [...]

Applied, thanks!

[1/1] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes
      commit: 37bd695c16b1b443e357eb37c09ef30b401f8b83

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-03-14 20:02 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-03-03 12:37 [PATCH v2] arm64: dts: qcom: sc7280: drop video decoder and encoder nodes Vikash Garodia
2025-03-05 12:26 ` Bryan O'Donoghue
2025-03-14 20:01 ` Bjorn Andersson

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®