* Re: [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0
2025-07-21 9:59 [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0 Guoqing Jiang
@ 2025-07-21 12:13 ` AngeloGioacchino Del Regno
2025-07-24 5:40 ` Macpaul Lin (林智斌)
2025-07-24 5:56 ` Macpaul Lin (林智斌)
` (2 subsequent siblings)
3 siblings, 1 reply; 6+ messages in thread
From: AngeloGioacchino Del Regno @ 2025-07-21 12:13 UTC (permalink / raw)
To: Guoqing Jiang, robh, krzk+dt, conor+dt, matthias.bgg,
jianjun.wang, tinghan.shen
Cc: macpaul.lin, devicetree, linux-arm-kernel, linux-mediatek
Il 21/07/25 11:59, Guoqing Jiang ha scritto:
> When test suspend resume with 6.8 based kernel, system can't resume
> and I got below error which can be also reproduced with 6.16 rc6+
> kernel.
>
> mtk-pcie-gen3 112f0000.pcie: PCIe link down, current LTSSM state: detect.quiet (0x0)
> mtk-pcie-gen3 112f0000.pcie: PM: dpm_run_callback(): genpd_resume_noirq returns -110
> mtk-pcie-gen3 112f0000.pcie: PM: failed to resume noirq: error -110
>
> After investigation, looks pcie0 has the same problem as pcie1 as
> decribed in commit 3d7fdd8e38aa ("arm64: dts: mediatek: mt8195:
> Remove suspend-breaking reset from pcie1").
>
> Fixes: ecc0af6a3fe6 ("arm64: dts: mt8195: Add pcie and pcie phy nodes")
> Signed-off-by: Guoqing Jiang <guoqing.jiang@canonical.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0
2025-07-21 12:13 ` AngeloGioacchino Del Regno
@ 2025-07-24 5:40 ` Macpaul Lin (林智斌)
0 siblings, 0 replies; 6+ messages in thread
From: Macpaul Lin (林智斌) @ 2025-07-24 5:40 UTC (permalink / raw)
To: matthias.bgg@gmail.com, Jianjun Wang (王建军),
Guoqing Jiang, AngeloGioacchino Del Regno, robh@kernel.org,
TingHan Shen (沈廷翰), joying.guo@mediatek.com,
krzk+dt@kernel.org, Jieyy Yang (杨洁),
Chuanjia Liu (柳传嘉),
Jianguo Zhang (张建国), conor+dt@kernel.org,
Jingming Jiang (江经明)
Cc: linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
Macpaul Lin (林智斌),
devicetree@vger.kernel.org, macpaul@gmail.com,
Ramax Lo (羅明遠)
On Mon, 2025-07-21 at 14:13 +0200, AngeloGioacchino Del Regno wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
>
> Il 21/07/25 11:59, Guoqing Jiang ha scritto:
> > When test suspend resume with 6.8 based kernel, system can't resume
> > and I got below error which can be also reproduced with 6.16 rc6+
> > kernel.
> >
> > mtk-pcie-gen3 112f0000.pcie: PCIe link down, current LTSSM state:
> > detect.quiet (0x0)
> > mtk-pcie-gen3 112f0000.pcie: PM: dpm_run_callback():
> > genpd_resume_noirq returns -110
> > mtk-pcie-gen3 112f0000.pcie: PM: failed to resume noirq: error -110
> >
> > After investigation, looks pcie0 has the same problem as pcie1 as
> > decribed in commit 3d7fdd8e38aa ("arm64: dts: mediatek: mt8195:
> > Remove suspend-breaking reset from pcie1").
> >
> > Fixes: ecc0af6a3fe6 ("arm64: dts: mt8195: Add pcie and pcie phy
> > nodes")
> > Signed-off-by: Guoqing Jiang <guoqing.jiang@canonical.com>
>
> Reviewed-by: AngeloGioacchino Del Regno
> <angelogioacchino.delregno@collabora.com>
>
>
Reviewed-by: Macpaul Lin <macpaul.lin@mediatek.com>
Dear Guoqing, please help to add Cc: stable@vger.kernel.org next time.
:)
Thanks
Macpaul Lin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0
2025-07-21 9:59 [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0 Guoqing Jiang
2025-07-21 12:13 ` AngeloGioacchino Del Regno
@ 2025-07-24 5:56 ` Macpaul Lin (林智斌)
2025-09-09 12:36 ` Matthias Brugger
2025-09-09 12:38 ` Matthias Brugger
3 siblings, 0 replies; 6+ messages in thread
From: Macpaul Lin (林智斌) @ 2025-07-24 5:56 UTC (permalink / raw)
To: matthias.bgg@gmail.com, Jianjun Wang (王建军),
Joying Kuo (郭羽喬), Guoqing Jiang,
Jingming Jiang (江经明), robh@kernel.org,
krzk+dt@kernel.org, TingHan Shen (沈廷翰),
Jieyy Yang (杨洁),
Chuanjia Liu (柳传嘉),
Jianguo Zhang (张建国), conor+dt@kernel.org,
AngeloGioacchino Del Regno
Cc: linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org
On Mon, 2025-07-21 at 17:59 +0800, Guoqing Jiang wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
>
> When test suspend resume with 6.8 based kernel, system can't resume
> and I got below error which can be also reproduced with 6.16 rc6+
> kernel.
>
> mtk-pcie-gen3 112f0000.pcie: PCIe link down, current LTSSM state:
> detect.quiet (0x0)
> mtk-pcie-gen3 112f0000.pcie: PM: dpm_run_callback():
> genpd_resume_noirq returns -110
> mtk-pcie-gen3 112f0000.pcie: PM: failed to resume noirq: error -110
>
> After investigation, looks pcie0 has the same problem as pcie1 as
> decribed in commit 3d7fdd8e38aa ("arm64: dts: mediatek: mt8195:
> Remove suspend-breaking reset from pcie1").
>
> Fixes: ecc0af6a3fe6 ("arm64: dts: mt8195: Add pcie and pcie phy
> nodes")
> Signed-off-by: Guoqing Jiang <guoqing.jiang@canonical.com>
> ---
> arch/arm64/boot/dts/mediatek/mt8195.dtsi | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> index dd065b1bf94a..50cf01452208 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> @@ -1563,9 +1563,6 @@ pcie0: pcie@112f0000 {
>
> power-domains = <&spm
> MT8195_POWER_DOMAIN_PCIE_MAC_P0>;
>
> - resets = <&infracfg_ao
> MT8195_INFRA_RST2_PCIE_P0_SWRST>;
> - reset-names = "mac";
> -
> #interrupt-cells = <1>;
> interrupt-map-mask = <0 0 0 7>;
> interrupt-map = <0 0 0 1 &pcie_intc0 0>,
> --
> 2.34.1
>
Reviewed-by: Macpaul Lin <macpaul.lin@mediatek.com>
Please help to add Cc: stable@vger.kernel.org next time.
Thanks
Macpaul Lin
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0
2025-07-21 9:59 [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0 Guoqing Jiang
2025-07-21 12:13 ` AngeloGioacchino Del Regno
2025-07-24 5:56 ` Macpaul Lin (林智斌)
@ 2025-09-09 12:36 ` Matthias Brugger
2025-09-09 12:38 ` Matthias Brugger
3 siblings, 0 replies; 6+ messages in thread
From: Matthias Brugger @ 2025-09-09 12:36 UTC (permalink / raw)
To: Guoqing Jiang, robh, krzk+dt, conor+dt, matthias.bgg,
angelogioacchino.delregno, jianjun.wang, tinghan.shen
Cc: macpaul.lin, devicetree, linux-arm-kernel, linux-mediatek
On 21/07/2025 11:59, Guoqing Jiang wrote:
> When test suspend resume with 6.8 based kernel, system can't resume
> and I got below error which can be also reproduced with 6.16 rc6+
> kernel.
>
> mtk-pcie-gen3 112f0000.pcie: PCIe link down, current LTSSM state: detect.quiet (0x0)
> mtk-pcie-gen3 112f0000.pcie: PM: dpm_run_callback(): genpd_resume_noirq returns -110
> mtk-pcie-gen3 112f0000.pcie: PM: failed to resume noirq: error -110
>
> After investigation, looks pcie0 has the same problem as pcie1 as
> decribed in commit 3d7fdd8e38aa ("arm64: dts: mediatek: mt8195:
> Remove suspend-breaking reset from pcie1").
>
> Fixes: ecc0af6a3fe6 ("arm64: dts: mt8195: Add pcie and pcie phy nodes")
> Signed-off-by: Guoqing Jiang <guoqing.jiang@canonical.com>
Queued, thanks!
Matthias
> ---
> arch/arm64/boot/dts/mediatek/mt8195.dtsi | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> index dd065b1bf94a..50cf01452208 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> @@ -1563,9 +1563,6 @@ pcie0: pcie@112f0000 {
>
> power-domains = <&spm MT8195_POWER_DOMAIN_PCIE_MAC_P0>;
>
> - resets = <&infracfg_ao MT8195_INFRA_RST2_PCIE_P0_SWRST>;
> - reset-names = "mac";
> -
> #interrupt-cells = <1>;
> interrupt-map-mask = <0 0 0 7>;
> interrupt-map = <0 0 0 1 &pcie_intc0 0>,
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0
2025-07-21 9:59 [PATCH] arm64: dts: mediatek: mt8195: Remove suspend-breaking reset from pcie0 Guoqing Jiang
` (2 preceding siblings ...)
2025-09-09 12:36 ` Matthias Brugger
@ 2025-09-09 12:38 ` Matthias Brugger
3 siblings, 0 replies; 6+ messages in thread
From: Matthias Brugger @ 2025-09-09 12:38 UTC (permalink / raw)
To: Guoqing Jiang, robh, krzk+dt, conor+dt, matthias.bgg,
angelogioacchino.delregno, jianjun.wang, tinghan.shen
Cc: macpaul.lin, devicetree, linux-arm-kernel, linux-mediatek
On 21/07/2025 11:59, Guoqing Jiang wrote:
> When test suspend resume with 6.8 based kernel, system can't resume
> and I got below error which can be also reproduced with 6.16 rc6+
> kernel.
>
> mtk-pcie-gen3 112f0000.pcie: PCIe link down, current LTSSM state: detect.quiet (0x0)
> mtk-pcie-gen3 112f0000.pcie: PM: dpm_run_callback(): genpd_resume_noirq returns -110
> mtk-pcie-gen3 112f0000.pcie: PM: failed to resume noirq: error -110
>
> After investigation, looks pcie0 has the same problem as pcie1 as
> decribed in commit 3d7fdd8e38aa ("arm64: dts: mediatek: mt8195:
> Remove suspend-breaking reset from pcie1").
>
> Fixes: ecc0af6a3fe6 ("arm64: dts: mt8195: Add pcie and pcie phy nodes")
> Signed-off-by: Guoqing Jiang <guoqing.jiang@canonical.com>
Queued, thanks.
Matthias
> ---
> arch/arm64/boot/dts/mediatek/mt8195.dtsi | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> index dd065b1bf94a..50cf01452208 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
> @@ -1563,9 +1563,6 @@ pcie0: pcie@112f0000 {
>
> power-domains = <&spm MT8195_POWER_DOMAIN_PCIE_MAC_P0>;
>
> - resets = <&infracfg_ao MT8195_INFRA_RST2_PCIE_P0_SWRST>;
> - reset-names = "mac";
> -
> #interrupt-cells = <1>;
> interrupt-map-mask = <0 0 0 7>;
> interrupt-map = <0 0 0 1 &pcie_intc0 0>,
^ permalink raw reply [flat|nested] 6+ messages in thread