Re: [PATCH] arm64: dts: mediatek: mt8183: Remove clock from mfg_async power domain

[AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>]

Il 05/08/24 08:50, Pin-yen Lin ha scritto:
> This clock dependency introduced a mutual dependency between mfg_async
> power domain and mt8183-mfgcfg clock, and Mediatek has confirmed that
> this dependency is not needed. Remove this to avoid potential deadlock.
> 

Uhm, have you tested this upstream with Panfrost, or only downstream with the
proprietary Mali stack?

Regards,
Angelo

> Signed-off-by: Pin-yen Lin <treapking@chromium.org>
> Fixes: 37fb78b9aeb7 ("arm64: dts: mediatek: Add mt8183 power domains controller")
> 
> ---
> 
>   arch/arm64/boot/dts/mediatek/mt8183.dtsi | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> index fbf145639b8c..267378fa46c0 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> @@ -872,8 +872,6 @@ power-domain@MT8183_POWER_DOMAIN_CONN {
>   
>   				mfg_async: power-domain@MT8183_POWER_DOMAIN_MFG_ASYNC {
>   					reg = <MT8183_POWER_DOMAIN_MFG_ASYNC>;
> -					clocks = <&topckgen CLK_TOP_MUX_MFG>;
> -					clock-names = "mfg";
>   					#address-cells = <1>;
>   					#size-cells = <0>;
>   					#power-domain-cells = <1>;