linux-kernel - Re: [PATCH v1 00/13] pmdomain: Partial refactor, add MT8196 support

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] [day] [month] [year] [list]

Message-ID: <2762e08674df39b79dc169ef3791eaeeaff17d17.camel@collabora.com>
Date: Tue, 24 Jun 2025 10:24:21 -0400
From: Nícolas "F. R. A. Prado" <nfraprado@...labora.com>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, 
	linux-mediatek@...ts.infradead.org
Cc: robh@...nel.org, conor+dt@...nel.org, mbrugger@...e.com, 
 y.oudjana@...tonmail.com, linux-pm@...r.kernel.org, ulf.hansson@...aro.org,
  linux-kernel@...r.kernel.org, devicetree@...r.kernel.org, 
 mandyjh.liu@...iatek.com, lihongbo22@...wei.com, wenst@...omium.org, 
 matthias.bgg@...il.com, krzk+dt@...nel.org, kernel@...labora.com, 
 linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v1 00/13] pmdomain: Partial refactor, add MT8196 support

On Mon, 2025-06-23 at 14:01 +0200, AngeloGioacchino Del Regno wrote:
> This series refactors the bus protection regmaps retrieval to avoid
> searching in all power domain devicetree subnodes for vendor
> properties
> to get syscons for different busses, and adds a new property which is
> located in the power controller root node containing handles to the
> same.
> 
> Retrocompatibility is retained and was tested on multiple SoCs in the
> Collabora lab - specifically, on Genio 350/510/700/1200, and manually
> on MT6795 Helio (Xperia M5 Smartphone), MT8186, MT8192 and MT8195
> Chromebooks.
> 
> This was tested *three times*:
>  - Before the per-SoC conversion in drivers/pmdomain/mediatek
>  - With per-SoC conversion code but with *legacy* devicetree
>  - With per-SoC conversion code and with *new* devicetree conversion
> 
> All of those tests were successful on all of the aforementioned SoCs.
> 
> This also adds support for:
>  - Modem power domain for both old and new MediaTek SoCs, useful for
>    bringing up the GSM/3G/4G/5G modem for both laptop and smartphone
> use
>  - RTFF MCU HW, as found in MT8196 Chromebooks and MT6991 Dimensity
> 9400
>  - Hardware Voter (MT8196/MT6991), allowing ATF, remote processors
> and
>    the AP (Linux) to manage the same power domains through a voter
> MCU,
>    avoiding power racing
>  - Directly controlled power domains for MT8196
>  - Voted power domains for MT8196
>  - Multimedia (voted) power domains for MT8196.
> 
> Note that all of the power domains for MT8196 should also work on
> MT6991
> but since I have no Dimensity 9400 boards, even though I'm 99.5% sure
> that
> it will simply work as those are the same, I avoided to add
> compatibles
> for 6991 as it's impossible for me to test.
> 
> AngeloGioacchino Del Regno (13):
>   dt-bindings: power: mediatek: Document mediatek,bus-protection
>   pmdomain: mediatek: Refactor bus protection regmaps retrieval
>   pmdomain: mediatek: Handle SoCs with inverted SRAM power-down bits
>   pmdomain: mediatek: Move ctl sequences out of power_on/off
> functions
>   pmdomain: mediatek: Add support for modem power sequences
>   pmdomain: mediatek: Add support for RTFF Hardware in MT8196/MT6991
>   pmdomain: mediatek: Add support for Hardware Voter power domains
>   pmdomain: mediatek: Add support for secure HWCCF infra power on
>   pmdomain: mediatek: Convert all SoCs to new style regmap retrieval
>   arm64: dts: mediatek: Convert all SoCs to use mediatek,bus-
> protection
>   dt-bindings: power: Add support for MT8196 power controllers
>   pmdomain: mediatek: Add support for MT8196 SCPSYS power domains
>   pmdomain: mediatek: Add support for MT8196 HFRPSYS power domains

For the entire series,

Reviewed-by: Nícolas F. R. A. Prado <nfraprado@...labora.com>

(as I've reviewed this internally before submission)

-- 
Thanks,

Nícolas