[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <e7aba180-5064-4b1e-a011-8e48c5cf3083@ti.com>
Date: Thu, 23 Oct 2025 13:55:39 +0530
From: Beleswar Prasad Padhi <b-padhi@...com>
To: Hiago De Franco <hiagofranco@...il.com>, Andrew Davis <afd@...com>,
"Nagalla, Hari" <hnagalla@...com>
CC: <nm@...com>, <vigneshr@...com>, <kristo@...nel.org>, <robh@...nel.org>,
<krzk+dt@...nel.org>, <conor+dt@...nel.org>, <u-kumar1@...com>,
<jm@...com>, <d-gole@...com>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
Robert Nelson <robertcnelson@...il.com>,
João Paulo Gonçalves <joao.goncalves@...adex.com>,
Emanuele Ghidoli <emanuele.ghidoli@...adex.com>,
Francesco Dolcini
<francesco.dolcini@...adex.com>,
Matthias Schiffer
<matthias.schiffer@...tq-group.com>,
Logan Bristol
<logan.bristol@...xas.edu>,
Josua Mayer <josua@...id-run.com>, John Ma
<jma@...tec.com>,
Nathan Morrisson <nmorrisson@...tec.com>,
Garrett Giordano
<ggiordano@...tec.com>,
Matt McKee <mmckee@...tec.com>, Wadim Egorov
<w.egorov@...tec.de>,
Max Krummenacher <max.krummenacher@...adex.com>,
"Stefan Eichenberger" <stefan.eichenberger@...adex.com>,
Hiago De Franco
<hiago.franco@...adex.com>,
Diogo Ivo <diogo.ivo@...mens.com>,
Li Hua Qian
<huaqian.li@...mens.com>,
Jan Kiszka <jan.kiszka@...mens.com>,
Baocheng Su
<baocheng.su@...mens.com>,
Benedikt Niedermayr
<benedikt.niedermayr@...mens.com>,
<regressions@...ts.linux.dev>
Subject: Re: [REGRESSION] Suspend to RAM does not work anymore with
k3-am62-ti-ipc-firmware.dtsi
On 21/10/25 14:33, Beleswar Prasad Padhi wrote:
> Hi Hiago,
>
> On 20/10/25 19:47, Hiago De Franco wrote:
>> Hello all,
>>
>> After commit 1d6161617c10 (“arm64: dts: ti: k3-am62-ti-ipc-firmware:
>> Refactor IPC cfg into new dtsi”), suspend-to-RAM stopped working on
>> AM62x.
>
> The above commit is only refactoring changes and should not
> cause any trouble. I think the commit you are interested in
> should be: a49f991e740f ("arm64: dts: ti: k3-am62-verdin:
> Add missing cfg for TI IPC Firmware").
>
<snip>
>
>> Do you have suggestions on how best to fix this in the driver, or should
>> we consider reverting the DTS change until suspend-to-RAM works again?
>
> List of suggestions/solutions in order of preference:
> 1. If no intention to enable IPC on rprocs:
> Do _not_ include k3-am62-ti-ipc-firmware.dtsi
> 2. If intention is to enable IPC on rprocs:
> Make sure rproc firmware is available in rootfs.
> rproc would boot up and consume the mbox
> msg, suspend would be successful. Tested this
> on TI AM62x-sk with commit 1d6161617c, works
> 3. Add support in mbox driver to flush the pending
> queues.
Posted a RFC version for #3:
https://lore.kernel.org/all/20251022102015.1345696-1-b-padhi@ti.com/
It still has open questions regarding scenarios with
FIFO firewalling and supporting existing OMAP SoCs
which could restore context upon resume.
Cc: Andrew, Hari
Thanks,
Beleswar
Powered by blists - more mailing lists