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.

> @@ -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

-- 
Julien Stephan <[email protected]>

Reply via email to