diff mbox series

[1/4] ARM: dts: qcom: msm8974: add gpu support

Message ID 20201230155132.3661292-1-iskren.chernev@gmail.com
State Superseded
Headers show
Series [1/4] ARM: dts: qcom: msm8974: add gpu support | expand

Commit Message

Iskren Chernev Dec. 30, 2020, 3:51 p.m. UTC
From: Brian Masney <masneyb@onstation.org>

Add support for the a3xx GPU

Signed-off-by: Brian Masney <masneyb@onstation.org>
---
 arch/arm/boot/dts/qcom-msm8974.dtsi | 45 +++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)


base-commit: d7a03a44a5e93f39ece70ec75d25c6088caa0fdb
prerequisite-patch-id: aba6f684932cab35d98457c21e4ff7a5ac75c753
prerequisite-patch-id: 4884d57df1bd197896b69e115d9002d6c26ae2e2
prerequisite-patch-id: 4f1aba3c3675236b18578eedbe71b0cdca01ed77
prerequisite-patch-id: cbfe6ccfebb142370baff15bbdf3cf2f34ee77df

Comments

Brian Masney Dec. 30, 2020, 10:27 p.m. UTC | #1
On Wed, Dec 30, 2020 at 05:51:29PM +0200, Iskren Chernev wrote:
> From: Brian Masney <masneyb@onstation.org>
> 
> Add support for the a3xx GPU
> 
> Signed-off-by: Brian Masney <masneyb@onstation.org>

Reviewed-by: Brian Masney <masneyb@onstation.org>
Alexey Minnekhanov Jan. 3, 2021, 5:59 a.m. UTC | #2
This indeed fixes device freeze+reboot issue when display powers off.

Tested-by: Alexey Minnekhanov <alexeymin@postmarketos.org>

On 12/30/20 6:51 PM, Iskren Chernev wrote:
> s1 and l12 regulators are used for the memory and cache on the Samsung
> S5 (klte). If they are turned off the phone shuts down. So mark them as
> always-on to prevent that from happening.
> 
> Signed-off-by: Iskren Chernev <iskren.chernev@gmail.com>
> ---
>   arch/arm/boot/dts/qcom-msm8974-samsung-klte.dts | 2 ++
>   1 file changed, 2 insertions(+)
>
Alexey Minnekhanov Jan. 3, 2021, 6:16 a.m. UTC | #3
Tested these patches on Samsung Galaxy S5 along with other patches that 
add panel driver and enable vram carve-out option on this device.

Tested-by: Alexey Minnekhanov <alexeymin@postmarketos.org>


On 12/30/20 6:51 PM, Iskren Chernev wrote:
> From: Brian Masney <masneyb@onstation.org>

> 

> Add support for the a3xx GPU

> 

> Signed-off-by: Brian Masney <masneyb@onstation.org>

> ---

>   arch/arm/boot/dts/qcom-msm8974.dtsi | 45 +++++++++++++++++++++++++++++

>   1 file changed, 45 insertions(+)

>
Bjorn Andersson Jan. 22, 2021, 6:33 p.m. UTC | #4
On Wed 30 Dec 09:51 CST 2020, Iskren Chernev wrote:

> From: Brian Masney <masneyb@onstation.org>

> 

> Add support for the a3xx GPU

> 

> Signed-off-by: Brian Masney <masneyb@onstation.org>


As discussed on IRC I'm waiting for a respin of this with your S-o-b
added after Brian's.

Thanks,
Bjorn

> ---

>  arch/arm/boot/dts/qcom-msm8974.dtsi | 45 +++++++++++++++++++++++++++++

>  1 file changed, 45 insertions(+)

> 

> diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi

> index 51f5f904f9eb9..c399446d8154e 100644

> --- a/arch/arm/boot/dts/qcom-msm8974.dtsi

> +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi

> @@ -1399,6 +1399,51 @@ cnoc: interconnect@fc480000 {

>  			         <&rpmcc RPM_SMD_CNOC_A_CLK>;

>  		};

>  

> +		gpu_opp_table: opp_table {

> +			status = "disabled";

> +

> +			compatible = "operating-points-v2";

> +

> +			opp-800000000 {

> +				opp-hz = /bits/ 64 <800000000>;

> +			};

> +

> +			opp-500000000 {

> +				opp-hz = /bits/ 64 <500000000>;

> +			};

> +

> +			opp-275000000 {

> +				opp-hz = /bits/ 64 <275000000>;

> +			};

> +		};

> +

> +		gpu: adreno@fdb00000 {

> +			status = "disabled";

> +

> +			compatible = "qcom,adreno-330.2",

> +			             "qcom,adreno";

> +			reg = <0xfdb00000 0x10000>;

> +			reg-names = "kgsl_3d0_reg_memory";

> +			interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>;

> +			interrupt-names = "kgsl_3d0_irq";

> +			clock-names = "core",

> +			              "iface",

> +			              "mem_iface";

> +			clocks = <&mmcc OXILI_GFX3D_CLK>,

> +			         <&mmcc OXILICX_AHB_CLK>,

> +			         <&mmcc OXILICX_AXI_CLK>;

> +			sram = <&gmu_sram>;

> +			power-domains = <&mmcc OXILICX_GDSC>;

> +			operating-points-v2 = <&gpu_opp_table>;

> +

> +			interconnects = <&mmssnoc MNOC_MAS_GRAPHICS_3D &bimc BIMC_SLV_EBI_CH0>,

> +			                <&ocmemnoc OCMEM_VNOC_MAS_GFX3D &ocmemnoc OCMEM_SLV_OCMEM>;

> +			interconnect-names = "gfx-mem",

> +			                     "ocmem";

> +

> +			// iommus = <&gpu_iommu 0>;

> +		};

> +

>  		mdss: mdss@fd900000 {

>  			status = "disabled";

>  

> 

> base-commit: d7a03a44a5e93f39ece70ec75d25c6088caa0fdb

> prerequisite-patch-id: aba6f684932cab35d98457c21e4ff7a5ac75c753

> prerequisite-patch-id: 4884d57df1bd197896b69e115d9002d6c26ae2e2

> prerequisite-patch-id: 4f1aba3c3675236b18578eedbe71b0cdca01ed77

> prerequisite-patch-id: cbfe6ccfebb142370baff15bbdf3cf2f34ee77df

> -- 

> 2.29.2

>
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi
index 51f5f904f9eb9..c399446d8154e 100644
--- a/arch/arm/boot/dts/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8974.dtsi
@@ -1399,6 +1399,51 @@  cnoc: interconnect@fc480000 {
 			         <&rpmcc RPM_SMD_CNOC_A_CLK>;
 		};
 
+		gpu_opp_table: opp_table {
+			status = "disabled";
+
+			compatible = "operating-points-v2";
+
+			opp-800000000 {
+				opp-hz = /bits/ 64 <800000000>;
+			};
+
+			opp-500000000 {
+				opp-hz = /bits/ 64 <500000000>;
+			};
+
+			opp-275000000 {
+				opp-hz = /bits/ 64 <275000000>;
+			};
+		};
+
+		gpu: adreno@fdb00000 {
+			status = "disabled";
+
+			compatible = "qcom,adreno-330.2",
+			             "qcom,adreno";
+			reg = <0xfdb00000 0x10000>;
+			reg-names = "kgsl_3d0_reg_memory";
+			interrupts = <GIC_SPI 33 IRQ_TYPE_LEVEL_HIGH>;
+			interrupt-names = "kgsl_3d0_irq";
+			clock-names = "core",
+			              "iface",
+			              "mem_iface";
+			clocks = <&mmcc OXILI_GFX3D_CLK>,
+			         <&mmcc OXILICX_AHB_CLK>,
+			         <&mmcc OXILICX_AXI_CLK>;
+			sram = <&gmu_sram>;
+			power-domains = <&mmcc OXILICX_GDSC>;
+			operating-points-v2 = <&gpu_opp_table>;
+
+			interconnects = <&mmssnoc MNOC_MAS_GRAPHICS_3D &bimc BIMC_SLV_EBI_CH0>,
+			                <&ocmemnoc OCMEM_VNOC_MAS_GFX3D &ocmemnoc OCMEM_SLV_OCMEM>;
+			interconnect-names = "gfx-mem",
+			                     "ocmem";
+
+			// iommus = <&gpu_iommu 0>;
+		};
+
 		mdss: mdss@fd900000 {
 			status = "disabled";