lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAGXv+5EbYBLt84Gx0mhTt9SqH8iMW87Y=_q=R2sHmJ4fygUQnQ@mail.gmail.com>
Date: Wed, 29 May 2024 14:34:54 +0800
From: Chen-Yu Tsai <wenst@...omium.org>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Cc: linux-mediatek@...ts.infradead.org, lee@...nel.org, robh@...nel.org, 
	krzk+dt@...nel.org, conor+dt@...nel.org, matthias.bgg@...il.com, 
	mandyjh.liu@...iatek.com, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	kernel@...labora.com, jpanis@...libre.com
Subject: Re: [PATCH 5/5] arm64: dts: mediatek: mt8188: Add support for Mali
 GPU on Panfrost

On Mon, May 27, 2024 at 5:40 PM AngeloGioacchino Del Regno
<angelogioacchino.delregno@...labora.com> wrote:
>
> Add the necessary OPP table for the GPU and also add a GPU node
> to enable support for the Valhall-JM G57 MC3 found on this SoC,
> using the Panfrost driver.
>
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
> ---
>  arch/arm64/boot/dts/mediatek/mt8188.dtsi | 123 +++++++++++++++++++++++
>  1 file changed, 123 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> index 0bca6c9f15fe..29d012d28edb 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> @@ -294,6 +294,112 @@ clk32k: oscillator-32k {
>                 clock-output-names = "clk32k";
>         };
>
> +       gpu_opp_table: opp-table-gpu {
> +               compatible = "operating-points-v2";
> +               opp-shared;
> +
> +               opp-390000000 {
> +                       opp-hz = /bits/ 64 <390000000>;
> +                       opp-microvolt = <575000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-431000000 {
> +                       opp-hz = /bits/ 64 <431000000>;
> +                       opp-microvolt = <587500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-473000000 {
> +                       opp-hz = /bits/ 64 <473000000>;
> +                       opp-microvolt = <600000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-515000000 {
> +                       opp-hz = /bits/ 64 <515000000>;
> +                       opp-microvolt = <612500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-556000000 {
> +                       opp-hz = /bits/ 64 <556000000>;
> +                       opp-microvolt = <625000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-598000000 {
> +                       opp-hz = /bits/ 64 <598000000>;
> +                       opp-microvolt = <637500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-640000000 {
> +                       opp-hz = /bits/ 64 <640000000>;
> +                       opp-microvolt = <650000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-670000000 {
> +                       opp-hz = /bits/ 64 <670000000>;
> +                       opp-microvolt = <662500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-700000000 {
> +                       opp-hz = /bits/ 64 <700000000>;
> +                       opp-microvolt = <675000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-730000000 {
> +                       opp-hz = /bits/ 64 <730000000>;
> +                       opp-microvolt = <687500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-760000000 {
> +                       opp-hz = /bits/ 64 <760000000>;
> +                       opp-microvolt = <700000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-790000000 {
> +                       opp-hz = /bits/ 64 <790000000>;
> +                       opp-microvolt = <712500>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-835000000 {
> +                       opp-hz = /bits/ 64 <835000000>;
> +                       opp-microvolt = <731250>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-880000000 {
> +                       opp-hz = /bits/ 64 <880000000>;
> +                       opp-microvolt = <750000>;
> +                       opp-supported-hw = <0xff>;
> +               };
> +               opp-915000000 {
> +                       opp-hz = /bits/ 64 <915000000>;
> +                       opp-microvolt = <775000>;
> +                       opp-supported-hw = <0x8f>;
> +               };
> +               opp-915000000-5 {
> +                       opp-hz = /bits/ 64 <915000000>;
> +                       opp-microvolt = <762500>;
> +                       opp-supported-hw = <0x30>;
> +               };
> +               opp-915000000-6 {
> +                       opp-hz = /bits/ 64 <915000000>;
> +                       opp-microvolt = <750000>;
> +                       opp-supported-hw = <0x70>;
> +               };
> +               opp-950000000 {
> +                       opp-hz = /bits/ 64 <950000000>;
> +                       opp-microvolt = <800000>;
> +                       opp-supported-hw = <0x8f>;
> +               };
> +               opp-950000000-5 {
> +                       opp-hz = /bits/ 64 <950000000>;
> +                       opp-microvolt = <775000>;
> +                       opp-supported-hw = <0x30>;
> +               };
> +               opp-950000000-6 {
> +                       opp-hz = /bits/ 64 <950000000>;
> +                       opp-microvolt = <750000>;
> +                       opp-supported-hw = <0x70>;
> +               };
> +       };
> +
>         pmu-a55 {
>                 compatible = "arm,cortex-a55-pmu";
>                 interrupt-parent = <&gic>;
> @@ -1167,6 +1273,23 @@ imp_iic_wrap_en: clock-controller@...c2000 {
>                         #clock-cells = <1>;
>                 };
>
> +               gpu: gpu@...00000 {
> +                       compatible = "mediatek,mt8188-mali", "arm,mali-valhall-jm";
> +                       reg = <0 0x13000000 0 0x4000>;
> +
> +                       clocks = <&mfgcfg CLK_MFGCFG_BG3D>;
> +                       interrupts = <GIC_SPI 383 IRQ_TYPE_LEVEL_HIGH 0>,
> +                                    <GIC_SPI 382 IRQ_TYPE_LEVEL_HIGH 0>,
> +                                    <GIC_SPI 381 IRQ_TYPE_LEVEL_HIGH 0>;
> +                       interrupt-names = "job", "mmu", "gpu";
> +                       operating-points-v2 = <&gpu_opp_table>;
> +                       power-domains = <&spm MT8188_POWER_DOMAIN_MFG2>,
> +                                       <&spm MT8188_POWER_DOMAIN_MFG3>,
> +                                       <&spm MT8188_POWER_DOMAIN_MFG4>;
> +                       power-domain-names = "core0", "core1", "core2";
> +                       status = "disabled";
> +               };
> +

This block no longer applies cleanly on the MTK tree because of
"arm64: dts: mediatek: mt8188: add lvts definitions" being applied.

ChenYu

>                 mfgcfg: clock-controller@...bf000 {
>                         compatible = "mediatek,mt8188-mfgcfg";
>                         reg = <0 0x13fbf000 0 0x1000>;
> --
> 2.45.1
>
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ