* [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a
@ 2024-07-30 1:38 Richard Acayan
2024-07-30 1:38 ` [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible Richard Acayan
` (4 more replies)
0 siblings, 5 replies; 9+ messages in thread
From: Richard Acayan @ 2024-07-30 1:38 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Richard Acayan
This adds support for the speed-binned A615 GPU on SDM670.
Richard Acayan (4):
dt-bindings: display/msm/gmu: Add SDM670 compatible
drm/msm/adreno: add a615 support
arm64: dts: qcom: sdm670: add gpu
arm64: dts: qcom: sdm670-google-common: enable gpu
.../devicetree/bindings/display/msm/gmu.yaml | 1 +
.../boot/dts/qcom/sdm670-google-sargo.dts | 13 ++
arch/arm64/boot/dts/qcom/sdm670.dtsi | 168 ++++++++++++++++++
drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 27 +++
4 files changed, 209 insertions(+)
--
2.45.2
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
@ 2024-07-30 1:38 ` Richard Acayan
2024-07-30 6:22 ` Krzysztof Kozlowski
2024-07-30 1:38 ` [PATCH 2/4] drm/msm/adreno: add a615 support Richard Acayan
` (3 subsequent siblings)
4 siblings, 1 reply; 9+ messages in thread
From: Richard Acayan @ 2024-07-30 1:38 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Richard Acayan
The Snapdragon 670 has a GMU. Add its compatible.
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
Documentation/devicetree/bindings/display/msm/gmu.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/display/msm/gmu.yaml b/Documentation/devicetree/bindings/display/msm/gmu.yaml
index b1bd372996d5..1c055ba64038 100644
--- a/Documentation/devicetree/bindings/display/msm/gmu.yaml
+++ b/Documentation/devicetree/bindings/display/msm/gmu.yaml
@@ -91,6 +91,7 @@ allOf:
compatible:
contains:
enum:
+ - qcom,adreno-gmu-615.0
- qcom,adreno-gmu-618.0
- qcom,adreno-gmu-630.2
then:
--
2.45.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/4] drm/msm/adreno: add a615 support
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
2024-07-30 1:38 ` [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible Richard Acayan
@ 2024-07-30 1:38 ` Richard Acayan
2024-07-30 1:38 ` [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu Richard Acayan
` (2 subsequent siblings)
4 siblings, 0 replies; 9+ messages in thread
From: Richard Acayan @ 2024-07-30 1:38 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Richard Acayan
The Adreno A615 is used in SDM670. Add an entry to support it along with
the speed bins.
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 27 +++++++++++++++++++++++
1 file changed, 27 insertions(+)
diff --git a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
index 68ba9aed5506..fc4fa2a9547d 100644
--- a/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
+++ b/drivers/gpu/drm/msm/adreno/a6xx_catalog.c
@@ -651,6 +651,33 @@ static const struct adreno_info a6xx_gpus[] = {
{ 157, 3 },
{ 127, 4 },
),
+ }, {
+ .chip_ids = ADRENO_CHIP_IDS(0x06010500),
+ .family = ADRENO_6XX_GEN1,
+ .revn = 615,
+ .fw = {
+ [ADRENO_FW_SQE] = "a630_sqe.fw",
+ [ADRENO_FW_GMU] = "a630_gmu.bin",
+ },
+ .gmem = SZ_512K,
+ .inactive_period = DRM_MSM_INACTIVE_PERIOD,
+ .init = a6xx_gpu_init,
+ .zapfw = "a615_zap.mdt",
+ .a6xx = &(const struct a6xx_info) {
+ .hwcg = a615_hwcg,
+ .protect = &a630_protect,
+ },
+ .speedbins = ADRENO_SPEEDBINS(
+ /*
+ * The default speed bin (0) has the same values as
+ * speed bin 90 which goes up to 432 MHz.
+ */
+ { 0, 0 },
+ { 90, 0 },
+ { 105, 1 },
+ { 146, 2 },
+ { 163, 3 },
+ ),
}, {
.machine = "qcom,sm7150",
.chip_ids = ADRENO_CHIP_IDS(0x06010800),
--
2.45.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
2024-07-30 1:38 ` [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible Richard Acayan
2024-07-30 1:38 ` [PATCH 2/4] drm/msm/adreno: add a615 support Richard Acayan
@ 2024-07-30 1:38 ` Richard Acayan
2024-07-31 13:35 ` Dmitry Baryshkov
2024-08-03 2:37 ` Richard Acayan
2024-07-30 1:38 ` [PATCH 4/4] arm64: dts: qcom: sdm670-google-common: enable gpu Richard Acayan
2024-07-31 14:25 ` [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Rob Herring (Arm)
4 siblings, 2 replies; 9+ messages in thread
From: Richard Acayan @ 2024-07-30 1:38 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Richard Acayan
The Snapdragon 670 has the Adreno A615 GPU. Add it along with its device
tree dependencies.
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
arch/arm64/boot/dts/qcom/sdm670.dtsi | 168 +++++++++++++++++++++++++++
1 file changed, 168 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sdm670.dtsi b/arch/arm64/boot/dts/qcom/sdm670.dtsi
index 187c6698835d..467006ab2bcb 100644
--- a/arch/arm64/boot/dts/qcom/sdm670.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm670.dtsi
@@ -8,6 +8,7 @@
#include <dt-bindings/clock/qcom,dispcc-sdm845.h>
#include <dt-bindings/clock/qcom,gcc-sdm845.h>
+#include <dt-bindings/clock/qcom,gpucc-sdm845.h>
#include <dt-bindings/clock/qcom,rpmh.h>
#include <dt-bindings/dma/qcom-gpi.h>
#include <dt-bindings/gpio/gpio.h>
@@ -617,6 +618,11 @@ qfprom: qfprom@784000 {
#address-cells = <1>;
#size-cells = <1>;
+ gpu_speed_bin: gpu_speed_bin@1a2 {
+ reg = <0x1a2 0x2>;
+ bits = <5 8>;
+ };
+
qusb2_hstx_trim: hstx-trim@1eb {
reg = <0x1eb 0x1>;
bits = <1 4>;
@@ -1299,6 +1305,168 @@ rclk-pins {
};
};
+ gpu: gpu@5000000 {
+ compatible = "qcom,adreno-615.0", "qcom,adreno";
+
+ reg = <0 0x05000000 0 0x40000>, <0 0x0509e000 0 0x10>;
+ reg-names = "kgsl_3d0_reg_memory", "cx_mem";
+
+ /*
+ * Look ma, no clocks! The GPU clocks and power are
+ * controlled entirely by the GMU
+ */
+
+ interrupts = <GIC_SPI 300 IRQ_TYPE_LEVEL_HIGH>;
+
+ iommus = <&adreno_smmu 0>;
+
+ operating-points-v2 = <&gpu_opp_table>;
+
+ qcom,gmu = <&gmu>;
+
+ interconnects = <&mem_noc MASTER_GRAPHICS_3D 0 &mem_noc SLAVE_EBI_CH0 0>;
+ interconnect-names = "gfx-mem";
+
+ nvmem-cells = <&gpu_speed_bin>;
+ nvmem-cell-names = "speed_bin";
+
+ status = "disabled";
+
+ gpu_opp_table: opp-table {
+ compatible = "operating-points-v2";
+
+ opp-780000000 {
+ opp-hz = /bits/ 64 <780000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_TURBO_L1>;
+ opp-peak-kBps = <7216000>;
+ opp-supported-hw = <0x8>;
+ };
+
+ opp-750000000 {
+ opp-hz = /bits/ 64 <750000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_TURBO>;
+ opp-peak-kBps = <7216000>;
+ opp-supported-hw = <0xc>;
+ };
+
+ opp-650000000 {
+ opp-hz = /bits/ 64 <650000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_NOM_L1>;
+ opp-peak-kBps = <7216000>;
+ opp-supported-hw = <0xc>;
+ };
+
+ opp-565000000 {
+ opp-hz = /bits/ 64 <565000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_NOM>;
+ opp-peak-kBps = <7216000>;
+ opp-supported-hw = <0xe>;
+ };
+
+ opp-430000000 {
+ opp-hz = /bits/ 64 <430000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_SVS_L1>;
+ opp-peak-kBps = <7216000>;
+ opp-supported-hw = <0xf>;
+ };
+
+ opp-355000000 {
+ opp-hz = /bits/ 64 <355000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_SVS>;
+ opp-peak-kBps = <6220000>;
+ opp-supported-hw = <0xf>;
+ };
+
+ opp-267000000 {
+ opp-hz = /bits/ 64 <267000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS>;
+ opp-peak-kBps = <4068000>;
+ opp-supported-hw = <0xf>;
+ };
+
+ opp-180000000 {
+ opp-hz = /bits/ 64 <180000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
+ opp-peak-kBps = <1804000>;
+ opp-supported-hw = <0xf>;
+ };
+ };
+ };
+
+ adreno_smmu: iommu@5040000 {
+ compatible = "qcom,sdm670-smmu-v2", "qcom,adreno-smmu", "qcom,smmu-v2";
+ reg = <0 0x05040000 0 0x10000>;
+ #iommu-cells = <1>;
+ #global-interrupts = <2>;
+ interrupts = <GIC_SPI 229 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 231 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 364 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 365 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 366 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 367 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 368 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 369 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 370 IRQ_TYPE_EDGE_RISING>,
+ <GIC_SPI 371 IRQ_TYPE_EDGE_RISING>;
+ clocks = <&gcc GCC_GPU_MEMNOC_GFX_CLK>,
+ <&gcc GCC_GPU_CFG_AHB_CLK>;
+ clock-names = "bus", "iface";
+
+ power-domains = <&gpucc GPU_CX_GDSC>;
+ };
+
+ gmu: gmu@506a000 {
+ compatible = "qcom,adreno-gmu-615.0", "qcom,adreno-gmu";
+
+ reg = <0 0x0506a000 0 0x30000>,
+ <0 0x0b280000 0 0x10000>,
+ <0 0x0b480000 0 0x10000>;
+ reg-names = "gmu", "gmu_pdc", "gmu_pdc_seq";
+
+ interrupts = <GIC_SPI 304 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 305 IRQ_TYPE_LEVEL_HIGH>;
+ interrupt-names = "hfi", "gmu";
+
+ clocks = <&gpucc GPU_CC_CX_GMU_CLK>,
+ <&gpucc GPU_CC_CXO_CLK>,
+ <&gcc GCC_DDRSS_GPU_AXI_CLK>,
+ <&gcc GCC_GPU_MEMNOC_GFX_CLK>;
+ clock-names = "gmu", "cxo", "axi", "memnoc";
+
+ power-domains = <&gpucc GPU_CX_GDSC>,
+ <&gpucc GPU_GX_GDSC>;
+ power-domain-names = "cx", "gx";
+
+ iommus = <&adreno_smmu 5>;
+
+ operating-points-v2 = <&gmu_opp_table>;
+
+ status = "disabled";
+
+ gmu_opp_table: opp-table {
+ compatible = "operating-points-v2";
+
+ opp-200000000 {
+ opp-hz = /bits/ 64 <200000000>;
+ opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
+ };
+ };
+ };
+
+ gpucc: clock-controller@5090000 {
+ compatible = "qcom,sdm845-gpucc";
+ reg = <0 0x05090000 0 0x9000>;
+ #clock-cells = <1>;
+ #reset-cells = <1>;
+ #power-domain-cells = <1>;
+ clocks = <&rpmhcc RPMH_CXO_CLK>,
+ <&gcc GCC_GPU_GPLL0_CLK_SRC>,
+ <&gcc GCC_GPU_GPLL0_DIV_CLK_SRC>;
+ clock-names = "bi_tcxo",
+ "gcc_gpu_gpll0_clk_src",
+ "gcc_gpu_gpll0_div_clk_src";
+ };
+
usb_1_hsphy: phy@88e2000 {
compatible = "qcom,sdm670-qusb2-phy", "qcom,qusb2-v2-phy";
reg = <0 0x088e2000 0 0x400>;
--
2.45.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 4/4] arm64: dts: qcom: sdm670-google-common: enable gpu
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
` (2 preceding siblings ...)
2024-07-30 1:38 ` [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu Richard Acayan
@ 2024-07-30 1:38 ` Richard Acayan
2024-07-31 14:25 ` [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Rob Herring (Arm)
4 siblings, 0 replies; 9+ messages in thread
From: Richard Acayan @ 2024-07-30 1:38 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter,
Richard Acayan
Enable the A615 GPU and GMU for the Pixel 3a. It has zap firmware, so
add that in as well.
Signed-off-by: Richard Acayan <mailingradian@gmail.com>
---
arch/arm64/boot/dts/qcom/sdm670-google-sargo.dts | 13 +++++++++++++
1 file changed, 13 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/sdm670-google-sargo.dts b/arch/arm64/boot/dts/qcom/sdm670-google-sargo.dts
index 176b0119fe6d..7acbcb5b69e1 100644
--- a/arch/arm64/boot/dts/qcom/sdm670-google-sargo.dts
+++ b/arch/arm64/boot/dts/qcom/sdm670-google-sargo.dts
@@ -403,10 +403,23 @@ &gcc {
<GCC_QSPI_CNOC_PERIPH_AHB_CLK>;
};
+&gmu {
+ status = "okay";
+};
+
&gpi_dma1 {
status = "okay";
};
+&gpu {
+ status = "okay";
+
+ zap-shader {
+ memory-region = <&gpu_mem>;
+ firmware-name = "qcom/sdm670/sargo/a615_zap.mbn";
+ };
+};
+
&i2c9 {
clock-frequency = <100000>;
status = "okay";
--
2.45.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible
2024-07-30 1:38 ` [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible Richard Acayan
@ 2024-07-30 6:22 ` Krzysztof Kozlowski
0 siblings, 0 replies; 9+ messages in thread
From: Krzysztof Kozlowski @ 2024-07-30 6:22 UTC (permalink / raw)
To: Richard Acayan, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter
On 30/07/2024 03:38, Richard Acayan wrote:
> The Snapdragon 670 has a GMU. Add its compatible.
>
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu
2024-07-30 1:38 ` [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu Richard Acayan
@ 2024-07-31 13:35 ` Dmitry Baryshkov
2024-08-03 2:37 ` Richard Acayan
1 sibling, 0 replies; 9+ messages in thread
From: Dmitry Baryshkov @ 2024-07-31 13:35 UTC (permalink / raw)
To: Richard Acayan
Cc: Rob Clark, Abhinav Kumar, Sean Paul, Marijn Suijten,
Maarten Lankhorst, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Bjorn Andersson, Konrad Dybcio, linux-arm-msm, dri-devel,
freedreno, devicetree, Maxime Ripard, Thomas Zimmermann,
David Airlie, Daniel Vetter
On Mon, Jul 29, 2024 at 09:38:48PM GMT, Richard Acayan wrote:
> The Snapdragon 670 has the Adreno A615 GPU. Add it along with its device
> tree dependencies.
>
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> ---
> arch/arm64/boot/dts/qcom/sdm670.dtsi | 168 +++++++++++++++++++++++++++
> 1 file changed, 168 insertions(+)
> +
> + gmu: gmu@506a000 {
> + compatible = "qcom,adreno-gmu-615.0", "qcom,adreno-gmu";
> +
[...]
> +
> + status = "disabled";
Probably no need to. The rest LGTM.
> +
> + gmu_opp_table: opp-table {
> + compatible = "operating-points-v2";
> +
> + opp-200000000 {
> + opp-hz = /bits/ 64 <200000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
> + };
> + };
> + };
> +
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
` (3 preceding siblings ...)
2024-07-30 1:38 ` [PATCH 4/4] arm64: dts: qcom: sdm670-google-common: enable gpu Richard Acayan
@ 2024-07-31 14:25 ` Rob Herring (Arm)
4 siblings, 0 replies; 9+ messages in thread
From: Rob Herring (Arm) @ 2024-07-31 14:25 UTC (permalink / raw)
To: Richard Acayan
Cc: Maarten Lankhorst, Abhinav Kumar, Conor Dooley, Bjorn Andersson,
freedreno, dri-devel, David Airlie, Marijn Suijten, devicetree,
linux-arm-msm, Dmitry Baryshkov, Sean Paul, Krzysztof Kozlowski,
Maxime Ripard, Daniel Vetter, Rob Clark, Thomas Zimmermann,
Konrad Dybcio
On Mon, 29 Jul 2024 21:38:45 -0400, Richard Acayan wrote:
> This adds support for the speed-binned A615 GPU on SDM670.
>
> Richard Acayan (4):
> dt-bindings: display/msm/gmu: Add SDM670 compatible
> drm/msm/adreno: add a615 support
> arm64: dts: qcom: sdm670: add gpu
> arm64: dts: qcom: sdm670-google-common: enable gpu
>
> .../devicetree/bindings/display/msm/gmu.yaml | 1 +
> .../boot/dts/qcom/sdm670-google-sargo.dts | 13 ++
> arch/arm64/boot/dts/qcom/sdm670.dtsi | 168 ++++++++++++++++++
> drivers/gpu/drm/msm/adreno/a6xx_catalog.c | 27 +++
> 4 files changed, 209 insertions(+)
>
> --
> 2.45.2
>
>
>
My bot found new DTB warnings on the .dts files added or changed in this
series.
Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.
If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:
pip3 install dtschema --upgrade
New warnings running 'make CHECK_DTBS=y qcom/sdm670-google-sargo.dtb' for 20240730013844.41951-6-mailingradian@gmail.com:
arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb: iommu@5040000: compatible: 'oneOf' conditional failed, one must be fixed:
['qcom,sdm670-smmu-v2', 'qcom,adreno-smmu', 'qcom,smmu-v2'] is too long
['qcom,sdm670-smmu-v2', 'qcom,adreno-smmu', 'qcom,smmu-v2'] is too short
'qcom,sdm670-smmu-v2' is not one of ['qcom,msm8996-smmu-v2', 'qcom,msm8998-smmu-v2', 'qcom,sdm630-smmu-v2', 'qcom,sm6375-smmu-v2']
'qcom,sdm670-smmu-v2' is not one of ['qcom,qcm2290-smmu-500', 'qcom,qdu1000-smmu-500', 'qcom,sa8775p-smmu-500', 'qcom,sc7180-smmu-500', 'qcom,sc7280-smmu-500', 'qcom,sc8180x-smmu-500', 'qcom,sc8280xp-smmu-500', 'qcom,sdm670-smmu-500', 'qcom,sdm845-smmu-500', 'qcom,sdx55-smmu-500', 'qcom,sdx65-smmu-500', 'qcom,sdx75-smmu-500', 'qcom,sm6115-smmu-500', 'qcom,sm6125-smmu-500', 'qcom,sm6350-smmu-500', 'qcom,sm6375-smmu-500', 'qcom,sm8150-smmu-500', 'qcom,sm8250-smmu-500', 'qcom,sm8350-smmu-500', 'qcom,sm8450-smmu-500', 'qcom,sm8550-smmu-500', 'qcom,sm8650-smmu-500', 'qcom,x1e80100-smmu-500']
'qcom,sdm670-smmu-v2' is not one of ['qcom,qcm2290-smmu-500', 'qcom,sc7180-smmu-500', 'qcom,sc7280-smmu-500', 'qcom,sc8180x-smmu-500', 'qcom,sc8280xp-smmu-500', 'qcom,sdm845-smmu-500', 'qcom,sm6115-smmu-500', 'qcom,sm6350-smmu-500', 'qcom,sm6375-smmu-500', 'qcom,sm8150-smmu-500', 'qcom,sm8250-smmu-500', 'qcom,sm8350-smmu-500', 'qcom,sm8450-smmu-500']
'qcom,sdm670-smmu-v2' is not one of ['qcom,qcm2290-smmu-500', 'qcom,sa8775p-smmu-500', 'qcom,sc7280-smmu-500', 'qcom,sc8180x-smmu-500', 'qcom,sc8280xp-smmu-500', 'qcom,sm6115-smmu-500', 'qcom,sm6125-smmu-500', 'qcom,sm8150-smmu-500', 'qcom,sm8250-smmu-500', 'qcom,sm8350-smmu-500', 'qcom,sm8450-smmu-500', 'qcom,sm8550-smmu-500', 'qcom,sm8650-smmu-500', 'qcom,x1e80100-smmu-500']
'qcom,sdm670-smmu-v2' is not one of ['qcom,sc7280-smmu-500', 'qcom,sm8150-smmu-500', 'qcom,sm8250-smmu-500']
'qcom,sdm670-smmu-v2' is not one of ['qcom,msm8996-smmu-v2', 'qcom,sc7180-smmu-v2', 'qcom,sdm630-smmu-v2', 'qcom,sdm845-smmu-v2', 'qcom,sm6350-smmu-v2', 'qcom,sm7150-smmu-v2']
'qcom,sdm845-smmu-v2' was expected
'marvell,ap806-smmu-500' was expected
'qcom,sdm670-smmu-v2' is not one of ['nvidia,tegra186-smmu', 'nvidia,tegra194-smmu', 'nvidia,tegra234-smmu']
'arm,mmu-500' was expected
'qcom,sdm670-smmu-v2' is not one of ['arm,mmu-400', 'arm,mmu-401']
'qcom,sdm670-smmu-v2' is not one of ['arm,smmu-v1', 'arm,smmu-v2', 'arm,mmu-400', 'arm,mmu-401', 'arm,mmu-500', 'cavium,smmu-v2']
'qcom,smmu-v2' was expected
'qcom,smmu-500' was expected
'nvidia,smmu-500' was expected
'arm,smmu-v2' was expected
'arm,smmu-v1' was expected
from schema $id: http://devicetree.org/schemas/iommu/arm,smmu.yaml#
arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb: /soc@0/iommu@5040000: failed to match any schema with compatible: ['qcom,sdm670-smmu-v2', 'qcom,adreno-smmu', 'qcom,smmu-v2']
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu
2024-07-30 1:38 ` [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu Richard Acayan
2024-07-31 13:35 ` Dmitry Baryshkov
@ 2024-08-03 2:37 ` Richard Acayan
1 sibling, 0 replies; 9+ messages in thread
From: Richard Acayan @ 2024-08-03 2:37 UTC (permalink / raw)
To: Rob Clark, Abhinav Kumar, Dmitry Baryshkov, Sean Paul,
Marijn Suijten, Maarten Lankhorst, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson, Konrad Dybcio,
linux-arm-msm, dri-devel, freedreno, devicetree
Cc: Maxime Ripard, Thomas Zimmermann, David Airlie, Daniel Vetter
On Mon, Jul 29, 2024 at 09:38:48PM -0400, Richard Acayan wrote:
> The Snapdragon 670 has the Adreno A615 GPU. Add it along with its device
> tree dependencies.
>
> Signed-off-by: Richard Acayan <mailingradian@gmail.com>
> ---
> arch/arm64/boot/dts/qcom/sdm670.dtsi | 168 +++++++++++++++++++++++++++
> 1 file changed, 168 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm670.dtsi b/arch/arm64/boot/dts/qcom/sdm670.dtsi
> index 187c6698835d..467006ab2bcb 100644
> --- a/arch/arm64/boot/dts/qcom/sdm670.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm670.dtsi
[snip]
> +
> + gpu_opp_table: opp-table {
> + compatible = "operating-points-v2";
> +
> + opp-780000000 {
> + opp-hz = /bits/ 64 <780000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_TURBO_L1>;
> + opp-peak-kBps = <7216000>;
> + opp-supported-hw = <0x8>;
> + };
> +
> + opp-750000000 {
> + opp-hz = /bits/ 64 <750000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_TURBO>;
> + opp-peak-kBps = <7216000>;
> + opp-supported-hw = <0xc>;
> + };
> +
> + opp-650000000 {
> + opp-hz = /bits/ 64 <650000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_NOM_L1>;
> + opp-peak-kBps = <7216000>;
> + opp-supported-hw = <0xc>;
> + };
> +
> + opp-565000000 {
> + opp-hz = /bits/ 64 <565000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_NOM>;
> + opp-peak-kBps = <7216000>;
> + opp-supported-hw = <0xe>;
The speed bins aren't entirely accurate. There is a single speed bin
that can reach exactly 504 MHz, but no higher, and one other speed bin
that can reach exactly 700 MHz. Let's add their exclusive OPPs.
> + };
> +
> + opp-430000000 {
> + opp-hz = /bits/ 64 <430000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_SVS_L1>;
> + opp-peak-kBps = <7216000>;
> + opp-supported-hw = <0xf>;
> + };
> +
> + opp-355000000 {
> + opp-hz = /bits/ 64 <355000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_SVS>;
> + opp-peak-kBps = <6220000>;
> + opp-supported-hw = <0xf>;
> + };
> +
> + opp-267000000 {
> + opp-hz = /bits/ 64 <267000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_LOW_SVS>;
> + opp-peak-kBps = <4068000>;
> + opp-supported-hw = <0xf>;
> + };
> +
> + opp-180000000 {
> + opp-hz = /bits/ 64 <180000000>;
> + opp-level = <RPMH_REGULATOR_LEVEL_MIN_SVS>;
> + opp-peak-kBps = <1804000>;
> + opp-supported-hw = <0xf>;
> + };
> + };
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2024-08-03 2:37 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-30 1:38 [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Richard Acayan
2024-07-30 1:38 ` [PATCH 1/4] dt-bindings: display/msm/gmu: Add SDM670 compatible Richard Acayan
2024-07-30 6:22 ` Krzysztof Kozlowski
2024-07-30 1:38 ` [PATCH 2/4] drm/msm/adreno: add a615 support Richard Acayan
2024-07-30 1:38 ` [PATCH 3/4] arm64: dts: qcom: sdm670: add gpu Richard Acayan
2024-07-31 13:35 ` Dmitry Baryshkov
2024-08-03 2:37 ` Richard Acayan
2024-07-30 1:38 ` [PATCH 4/4] arm64: dts: qcom: sdm670-google-common: enable gpu Richard Acayan
2024-07-31 14:25 ` [PATCH 0/4] drm/msm/adreno: Add A615 GPU for SDM670 and Pixel 3a Rob Herring (Arm)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).