On 9/16/26 9:19 AM, Julien Stephan wrote:
> On Fri, 04 Sep 2026 09:46:32 -0500, David Lechner <[email protected]>
> wrote:
>> diff --git a/arch/arm/dts/mt8189.dtsi b/arch/arm/dts/mt8189.dtsi
>> index c965b00bb5c..f3a350c51b8 100644
>> --- a/arch/arm/dts/mt8189.dtsi
>> +++ b/arch/arm/dts/mt8189.dtsi
>> @@ -144,6 +145,45 @@
>> [ ... skip 14 lines ... ]
>> + power-domain@MT8189_POWER_DOMAIN_MM_INFRA {
>> + reg = <MT8189_POWER_DOMAIN_MM_INFRA>;
>> + clocks = <&topckgen_clk
>> CLK_TOP_MMINFRA_SEL>,
>> + <&mminfra_config_clk
>> CLK_MMINFRA_SMI>;
>> + clock-names = "mminfra", "ss-mminfra";
>> + mediatek,infracfg = <&infracfg_ao>;
>
> Hello David,
>
> mediatek,infracfg is marked as deprecated in
> bindings/power/mediatek,power-controller.yaml
> so It will probably not be accepted upstream, it should be replaced with
> access-controllers.
>
> IMHO the current driver does not support access-controllers, so we
> should add support for it.
Thanks for the heads up. It looks like upstream hasn't actually made the
conversion yet though. We'll wait and see what actually lands upstream for
mt8189.
>
>> @@ -344,13 +387,18 @@
>> status = "disabled";
>> };
>>
>> + infracfg_ao: syscon@10001000 {
>> + compatible = "mediatek,mt8189-infra-ao", "syscon";
>
> not used in the codebase right now, but I think it should be
> mediatek,mt8189-infracfg-ao to stay consistent with other platform
>
I went ahead and merged this series so that we don't block other power
domain work, but will keep this in mind for the next series.
