diff mbox series

[01/14] arm64: dts: qcom: sc7280: Don't disable MDP explicitly

Message ID 20230531011623.3808538-2-dmitry.baryshkov@linaro.org (mailing list archive)
State Accepted
Commit 650c09daca4cac35ffd264ee1ca92521bdd9a5a9
Headers show
Series arm/arm64: dts: qcom: MDSS nodes cleanup | expand

Commit Message

Dmitry Baryshkov May 31, 2023, 1:16 a.m. UTC
MDSS and all its subdevices are useless without DPU/MDP, so disabling
MDP doesn't make any sense. Remove explicit disabling of the DPU device.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 4 ----
 arch/arm64/boot/dts/qcom/sc7280.dtsi           | 2 --
 2 files changed, 6 deletions(-)

Comments

Konrad Dybcio May 31, 2023, 9:38 a.m. UTC | #1
On 31.05.2023 03:16, Dmitry Baryshkov wrote:
> MDSS and all its subdevices are useless without DPU/MDP, so disabling
> MDP doesn't make any sense. Remove explicit disabling of the DPU device.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>

Konrad
>  arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi | 4 ----
>  arch/arm64/boot/dts/qcom/sc7280.dtsi           | 2 --
>  2 files changed, 6 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> index 5b1c175c47f1..9ea6636125ad 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
> @@ -467,10 +467,6 @@ &mdss_dp_out {
>  	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
>  };
>  
> -&mdss_mdp {
> -	status = "okay";
> -};
> -
>  /* NVMe drive, enabled on a per-board basis */
>  &pcie1 {
>  	pinctrl-names = "default";
> diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> index 00547192215d..1aa3c6bf81f0 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
> @@ -3881,8 +3881,6 @@ mdss_mdp: display-controller@ae01000 {
>  				interrupt-parent = <&mdss>;
>  				interrupts = <0>;
>  
> -				status = "disabled";
> -
>  				ports {
>  					#address-cells = <1>;
>  					#size-cells = <0>;
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
index 5b1c175c47f1..9ea6636125ad 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-herobrine.dtsi
@@ -467,10 +467,6 @@  &mdss_dp_out {
 	link-frequencies = /bits/ 64 <1620000000 2700000000 5400000000>;
 };
 
-&mdss_mdp {
-	status = "okay";
-};
-
 /* NVMe drive, enabled on a per-board basis */
 &pcie1 {
 	pinctrl-names = "default";
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index 00547192215d..1aa3c6bf81f0 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -3881,8 +3881,6 @@  mdss_mdp: display-controller@ae01000 {
 				interrupt-parent = <&mdss>;
 				interrupts = <0>;
 
-				status = "disabled";
-
 				ports {
 					#address-cells = <1>;
 					#size-cells = <0>;