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