Devicetree
 help / color / mirror / Atom feed
* [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU
@ 2026-11-02 14:28 Lukas Märdian
  2026-11-02 14:38 ` sashiko-bot
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Lukas Märdian @ 2026-11-02 14:28 UTC (permalink / raw)
  To: andersson, konradybcio
  Cc: abelvesa, robh, krzk+dt, conor+dt, srinivas.kandagatla, akhilpo,
	jie.zhang, linux-arm-msm, devicetree, linux-kernel,
	Lukas Märdian

The GPU and GMU were disabled when the ThinkPad T14s Gen 7 device tree
was introduced, as the Adreno X2-85 found in the Mahua SoC was not
supported yet. Until then, mahua.dtsi inherited the GPU and GMU nodes
from glymur.dtsi unchanged, describing them as Glymur's larger Adreno
X2-90, which is not what the Mahua hardware is.

Mahua's own GPU is now described: mahua.dtsi overrides the compatibles
with "qcom,adreno-44060000" and "qcom,adreno-gmu-x285.0", and the msm
driver carries a matching Adreno X2-85 entry. With that, the nodes
work as-is on this laptop, so drop the board-level overrides and let
them inherit the default enabled status.

Tested on a ThinkPad T14s Gen 7 (LCD): the GPU and GMU bind, the
gen80100 SQE/AQE/GMU firmware loads (GMU firmware v5.2.38), and Mesa
26.3.0-rc2 reports "Adreno (TM) X2-85" for both OpenGL 4.6 (freedreno)
and Vulkan (turnip).

Assisted-by: LLM
Signed-off-by: Lukas Märdian <lukas.maerdian@oss.qualcomm.com>
---
This depends on the Mahua GPU support series, specifically
"drm/msm/a8xx: Add Mahua GPU support" and "arm64: dts: qcom: mahua:
Support GPU and GMU", which are not merged yet:
https://lore.kernel.org/all/20260925-mahua-gpu-v1-0-0fa0bfd8d315@oss.qualcomm.com/

 .../dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts     | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
index 255857cda9b..189dc3fe542 100644
--- a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
+++ b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
@@ -693,15 +693,6 @@ vreg_l4h_e0_1p2: ldo4 {
 	};
 };
 
-/* GPU/GMU not fully functional yet on this platform. */
-&gmu {
-	status = "disabled";
-};
-
-&gpu {
-	status = "disabled";
-};
-
 &i2c0 {
 	clock-frequency = <400000>;
 
-- 
2.53.0


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

* Re: [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU
  2026-11-02 14:28 [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU Lukas Märdian
@ 2026-11-02 14:38 ` sashiko-bot
  2026-11-02 15:11 ` Akhil P Oommen
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: sashiko-bot @ 2026-11-02 14:38 UTC (permalink / raw)
  To: Lukas Märdian; +Cc: devicetree, robh, conor+dt

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [High] arm64: dts: qcom: mahua-t14s enables GPU/GMU without mahua.dtsi overrides
--

commit 9b4a7ddbb7e3ff673d119429fdec3506da117877
Author: Lukas Märdian <lukas.maerdian@oss.qualcomm.com>

arm64: dts: qcom: mahua-t14s: Enable GPU and GMU

This commit removes the board-level disabled status overrides for the GPU and
GMU nodes in the Lenovo ThinkPad T14s Gen 7 (LCD) device tree so that they
inherit the default enabled status from the SoC device tree.

> diff --git a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> index 26d6d8285e040..fb913da7343fd 100644
> --- a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> +++ b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> @@ -634,15 +634,6 @@ vreg_l4h_e0_1p2: ldo4 {
>  	};
>  };
>  
> -/* GPU/GMU not fully functional yet on this platform. */
> -&gmu {
> -	status = "disabled";
> -};
> -
> -&gpu {
> -	status = "disabled";
> -};

[Severity: High]
Does enabling &gpu and &gmu in
arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts cause the
ThinkPad T14s Gen 7 to probe the GPU with the wrong hardware configuration
when the prerequisite arch/arm64/boot/dts/qcom/mahua.dtsi overrides are not
yet applied?

The commit message notes that mahua.dtsi overrides the compatibles with
"qcom,adreno-44060000" and "qcom,adreno-gmu-x285.0", and the note below the
tearline mentions a dependency on "arm64: dts: qcom: mahua: Support GPU and
GMU" (https://lore.kernel.org/all/20260925-mahua-gpu-v1-0-0fa0bfd8d315@oss.qualcomm.com/).

In the current tree, drivers/gpu/drm/msm/adreno/a8xx_catalog.c already has
the a8xx_gpus[] entry for 0x44060000, but arch/arm64/boot/dts/qcom/mahua.dtsi
only includes glymur.dtsi without overriding &gpu or &gmu:

arch/arm64/boot/dts/qcom/mahua.dtsi:
    #include "glymur.dtsi"

Without those overrides in mahua.dtsi, removing status = "disabled" here
exposes the unchanged nodes from arch/arm64/boot/dts/qcom/glymur.dtsi:

arch/arm64/boot/dts/qcom/glymur.dtsi:
    gpu: gpu@3d00000 {
            compatible = "qcom,adreno-44070001", "qcom,adreno";
    ...
    gmu: gmu@3d6c000 {
            compatible = "qcom,adreno-gmu-x285.1", "qcom,adreno-gmu";

When the drm/msm driver probes gpu@3d00000 via adreno_bind() -> find_chipid(),
it matches the 0x44070001 (Adreno X2-90) entry in a8xx_gpus[] rather than the
0x44060000 (Adreno X2-85) entry in drivers/gpu/drm/msm/adreno/a8xx_catalog.c:

drivers/gpu/drm/msm/adreno/a8xx_catalog.c:
    .chip_ids = ADRENO_CHIP_IDS(0x44070001),
    .family = ADRENO_8XX_GEN2,
    .fw = {
            [ADRENO_FW_SQE] = "gen80100_sqe.fw",
            [ADRENO_FW_GMU] = "gen80100_gmu.bin",
    },
    .gmem = 21 * SZ_1M,
    ...
    .a6xx = &(const struct a6xx_info) {
            ...
            .max_slices = 4,
            .gmu_chipid = 0x8010100,

Instead of Mahua's 15.75 MiB GMEM, max_slices = 3, gmu_chipid = 0x80a0000,
and gen80100_aqe.fw, the driver will configure 21 MiB GMEM, 4 slices, and
write 0x8010100 to REG_A8XX_GMU_GENERAL_10 in a6xx_gmu_fw_start(), while also
leaving the undocumented "qcom,adreno-gmu-x285.1" compatible enabled on
gmu@3d6c000.

Should the mahua.dtsi GPU and GMU overrides be applied before enabling &gpu
and &gmu on this board?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261102142812.1353199-1-lukas.maerdian@oss.qualcomm.com?part=1

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

* Re: [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU
  2026-11-02 14:28 [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU Lukas Märdian
  2026-11-02 14:38 ` sashiko-bot
@ 2026-11-02 15:11 ` Akhil P Oommen
  2026-11-02 15:34 ` Christopher Obbard
  2026-11-02 15:50 ` Dmitry Baryshkov
  3 siblings, 0 replies; 5+ messages in thread
From: Akhil P Oommen @ 2026-11-02 15:11 UTC (permalink / raw)
  To: Lukas Märdian, andersson, konradybcio
  Cc: abelvesa, robh, krzk+dt, conor+dt, srinivas.kandagatla, jie.zhang,
	linux-arm-msm, devicetree, linux-kernel



On 11/2/2026 7:58 PM, Lukas Märdian wrote:
> The GPU and GMU were disabled when the ThinkPad T14s Gen 7 device tree
> was introduced, as the Adreno X2-85 found in the Mahua SoC was not
> supported yet. Until then, mahua.dtsi inherited the GPU and GMU nodes
> from glymur.dtsi unchanged, describing them as Glymur's larger Adreno
> X2-90, which is not what the Mahua hardware is.
> 
> Mahua's own GPU is now described: mahua.dtsi overrides the compatibles
> with "qcom,adreno-44060000" and "qcom,adreno-gmu-x285.0", and the msm
> driver carries a matching Adreno X2-85 entry. With that, the nodes
> work as-is on this laptop, so drop the board-level overrides and let
> them inherit the default enabled status.
> 
> Tested on a ThinkPad T14s Gen 7 (LCD): the GPU and GMU bind, the
> gen80100 SQE/AQE/GMU firmware loads (GMU firmware v5.2.38), and Mesa
> 26.3.0-rc2 reports "Adreno (TM) X2-85" for both OpenGL 4.6 (freedreno)
> and Vulkan (turnip).
> 
> Assisted-by: LLM
> Signed-off-by: Lukas Märdian <lukas.maerdian@oss.qualcomm.com>
> ---
> This depends on the Mahua GPU support series, specifically
> "drm/msm/a8xx: Add Mahua GPU support" and "arm64: dts: qcom: mahua:
> Support GPU and GMU", which are not merged yet:
> https://lore.kernel.org/all/20260925-mahua-gpu-v1-0-0fa0bfd8d315@oss.qualcomm.com/
> 
>  .../dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts     | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> index 255857cda9b..189dc3fe542 100644
> --- a/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> +++ b/arch/arm64/boot/dts/qcom/mahua-lenovo-thinkpad-t14s-gen7-lcd.dts
> @@ -693,15 +693,6 @@ vreg_l4h_e0_1p2: ldo4 {
>  	};
>  };
>  
> -/* GPU/GMU not fully functional yet on this platform. */
> -&gmu {
> -	status = "disabled";
> -};
> -
> -&gpu {
> -	status = "disabled";
> -};
> -
>  &i2c0 {
>  	clock-frequency = <400000>;
>  

Reviewed-by: Akhil P Oommen <akhilpo@oss.qualcomm.com>

-Akhil.


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

* Re: [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU
  2026-11-02 14:28 [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU Lukas Märdian
  2026-11-02 14:38 ` sashiko-bot
  2026-11-02 15:11 ` Akhil P Oommen
@ 2026-11-02 15:34 ` Christopher Obbard
  2026-11-02 15:50 ` Dmitry Baryshkov
  3 siblings, 0 replies; 5+ messages in thread
From: Christopher Obbard @ 2026-11-02 15:34 UTC (permalink / raw)
  To: Lukas Märdian
  Cc: andersson, konradybcio, abelvesa, robh, krzk+dt, conor+dt,
	srinivas.kandagatla, akhilpo, jie.zhang, linux-arm-msm,
	devicetree, linux-kernel

On Mon, 02 Nov 2026 15:28:12 +0100, Lukas Märdian <lukas.maerdian@oss.qualcomm.com> wrote:
> The GPU and GMU were disabled when the ThinkPad T14s Gen 7 device tree
> was introduced, as the Adreno X2-85 found in the Mahua SoC was not
> supported yet. Until then, mahua.dtsi inherited the GPU and GMU nodes
> from glymur.dtsi unchanged, describing them as Glymur's larger Adreno
> X2-90, which is not what the Mahua hardware is.
> 
> Mahua's own GPU is now described: mahua.dtsi overrides the compatibles
> with "qcom,adreno-44060000" and "qcom,adreno-gmu-x285.0", and the msm
> driver carries a matching Adreno X2-85 entry. With that, the nodes
> work as-is on this laptop, so drop the board-level overrides and let
> them inherit the default enabled status.
> 
> [...]

Tested-by: Christopher Obbard <chris.obbard@oss.qualcomm.com>

-- 
Christopher Obbard


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

* Re: [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU
  2026-11-02 14:28 [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU Lukas Märdian
                   ` (2 preceding siblings ...)
  2026-11-02 15:34 ` Christopher Obbard
@ 2026-11-02 15:50 ` Dmitry Baryshkov
  3 siblings, 0 replies; 5+ messages in thread
From: Dmitry Baryshkov @ 2026-11-02 15:50 UTC (permalink / raw)
  To: Lukas Märdian
  Cc: andersson, konradybcio, abelvesa, robh, krzk+dt, conor+dt,
	srinivas.kandagatla, akhilpo, jie.zhang, linux-arm-msm,
	devicetree, linux-kernel

On Mon, Nov 02, 2026 at 03:28:12PM +0100, Lukas Märdian wrote:
> The GPU and GMU were disabled when the ThinkPad T14s Gen 7 device tree
> was introduced, as the Adreno X2-85 found in the Mahua SoC was not
> supported yet. Until then, mahua.dtsi inherited the GPU and GMU nodes
> from glymur.dtsi unchanged, describing them as Glymur's larger Adreno
> X2-90, which is not what the Mahua hardware is.
> 
> Mahua's own GPU is now described: mahua.dtsi overrides the compatibles
> with "qcom,adreno-44060000" and "qcom,adreno-gmu-x285.0", and the msm
> driver carries a matching Adreno X2-85 entry. With that, the nodes
> work as-is on this laptop, so drop the board-level overrides and let
> them inherit the default enabled status.
> 
> Tested on a ThinkPad T14s Gen 7 (LCD): the GPU and GMU bind, the
> gen80100 SQE/AQE/GMU firmware loads (GMU firmware v5.2.38), and Mesa
> 26.3.0-rc2 reports "Adreno (TM) X2-85" for both OpenGL 4.6 (freedreno)
> and Vulkan (turnip).
> 
> Assisted-by: LLM
> Signed-off-by: Lukas Märdian <lukas.maerdian@oss.qualcomm.com>
> ---
> This depends on the Mahua GPU support series, specifically
> "drm/msm/a8xx: Add Mahua GPU support" and "arm64: dts: qcom: mahua:
> Support GPU and GMU", which are not merged yet:
> https://lore.kernel.org/all/20260925-mahua-gpu-v1-0-0fa0bfd8d315@oss.qualcomm.com/


With the dependency landing first:

Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>



-- 
With best wishes
Dmitry

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

end of thread, other threads:[~2026-11-02 15:50 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-11-02 14:28 [PATCH] arm64: dts: qcom: mahua-t14s: Enable GPU and GMU Lukas Märdian
2026-11-02 14:38 ` sashiko-bot
2026-11-02 15:11 ` Akhil P Oommen
2026-11-02 15:34 ` Christopher Obbard
2026-11-02 15:50 ` Dmitry Baryshkov

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