[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <edf25359-1804-445d-bd49-45c7238a3cd5@gmail.com>
Date: Fri, 13 Sep 2024 13:34:49 +0200
From: Matthias Brugger <matthias.bgg@...il.com>
To: Pablo Sun <pablo.sun@...iatek.com>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
<conor+dt@...nel.org>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH v2] arm64: dts: mediatek: mt8395-genio-1200-evk: Enable
GPU
On 12/09/2024 09:06, Pablo Sun wrote:
> Enable the Mali Valhall GPU on Genio 1200 EVK by providing regulator
> supply settingsi to gpu and mfg1, and enable the GPU node.
>
> In addition, set the GPU related regulator voltage range:
>
> 1. Set the recommended input voltage range of DVDD_GPU to (0.546V-0.787V),
> based on Table 5-3 of MT8395 Application Processor Datasheet.
> The regulator mt6315_7_vbuck1("Vgpu") connects to the DVDD_GPU input.
> Note that the minimum voltage in SoC eFuse data, which is read by
> MTK-SVS to adjust the regulator voltage, does not go below
> the recommended operating voltage in the datasheet.
>
> 2. Set the input voltage of DVDD_SRAM_GPU, supplied by
> mt6359_vsram_others_ldo_reg, to 0.75V, the recommended typical
> operating voltage in MT8395 Application Processor Datasheet.
>
> This patch is tested by enabling CONFIG_DRM_PANFROST and
> on Genio 1200 EVK it probed with following dmesg:
>
> ```
> panfrost 13000000.gpu: clock rate = 700000092
> panfrost 13000000.gpu: mali-g57 id 0x9093 major 0x0 minor 0x1 status 0x0
> panfrost 13000000.gpu: features: 00000000,000019f7,
> issues: 00000001,80000400
> panfrost 13000000.gpu: Features: L2:0x07120206 Shader:0x00000000
> Tiler:0x00000809 Mem:0x301
> MMU:0x00002830 AS:0xff JS:0x7
> panfrost 13000000.gpu: shader_present=0x50045 l2_present=0x1
> [drm] Initialized panfrost 1.2.0 for 13000000.gpu on minor 0
> ```
>
> Signed-off-by: Pablo Sun <pablo.sun@...iatek.com>
Reviewed-by: Matthias Brugger <matthias.bgg@...il.com>
> ---
> .../dts/mediatek/mt8395-genio-1200-evk.dts | 19 +++++++++++++++++--
> 1 file changed, 17 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8395-genio-1200-evk.dts b/arch/arm64/boot/dts/mediatek/mt8395-genio-1200-evk.dts
> index a06610fff8ad..4f7d66d6d785 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8395-genio-1200-evk.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8395-genio-1200-evk.dts
> @@ -194,6 +194,11 @@ eth_phy0: eth-phy0@1 {
> };
> };
>
> +&gpu {
> + mali-supply = <&mt6315_7_vbuck1>;
> + status = "okay";
> +};
> +
> &i2c0 {
> clock-frequency = <400000>;
> pinctrl-0 = <&i2c0_pins>;
> @@ -337,6 +342,10 @@ &mfg0 {
> domain-supply = <&mt6315_7_vbuck1>;
> };
>
> +&mfg1 {
> + domain-supply = <&mt6359_vsram_others_ldo_reg>;
> +};
> +
> &mmc0 {
> status = "okay";
> pinctrl-names = "default", "state_uhs";
> @@ -407,6 +416,12 @@ &mt6359_vrf12_ldo_reg {
> regulator-always-on;
> };
>
> +/* for GPU SRAM */
> +&mt6359_vsram_others_ldo_reg {
> + regulator-min-microvolt = <750000>;
> + regulator-max-microvolt = <750000>;
> +};
> +
> &mt6359codec {
> mediatek,mic-type-0 = <1>; /* ACC */
> mediatek,mic-type-1 = <3>; /* DCC */
> @@ -839,8 +854,8 @@ regulators {
> mt6315_7_vbuck1: vbuck1 {
> regulator-compatible = "vbuck1";
> regulator-name = "Vgpu";
> - regulator-min-microvolt = <300000>;
> - regulator-max-microvolt = <1193750>;
> + regulator-min-microvolt = <546000>;
> + regulator-max-microvolt = <787000>;
> regulator-enable-ramp-delay = <256>;
> regulator-allowed-modes = <0 1 2>;
> };
Powered by blists - more mailing lists