Re: [PATCH v6 2/2] mmc: sdhci-msm: Use pm ops instead of macro to restore crypto keys
Dmitry Baryshkov <[email protected]> Mon, 3 Aug 2026 21:10:03 +0300
| Newsgroups | org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.linux-mmc |
|---|---|
| Message-ID | <stu224dgiay6mau2enqyf27m54laucdzqaudya6wxcnm7ldu4o@omftbvyuvmqa> |
On Mon, Aug 03, 2026 at 10:01:23PM +0530, Neeraj Soni wrote: > From: Ram Prakash Gupta <[email protected]> > > Inline Crypto Engine (ICE) keys are lost after hibernation entry and this > needs to be restored when hibernation exits. ICE keys are re-programmed > during sdhci_msm_ice_init() but it may not cover cases where the > hibernation image is already restored. > > Unwrap the pm ops and use directly in driver to add the call to restore > Inline Crypto Engine (ICE) keys. This ensures that ICE is brought into > same state as before hibernation. > > Also set MMC_CAP2_CRYPTO_NO_REPROG to indicate that re-programming of ICE > keys is not needed during MMC runtime suspend/resume. Hmm, if there is a flag which controls reprogramming of the keys, why can't we use a default and make the core reprogram those? > > Signed-off-by: Ram Prakash Gupta <[email protected]> > Signed-off-by: Seshu Madhavi Puppala <[email protected]> > Co-developed-by: Ram Prakash Gupta <[email protected]> > Signed-off-by: Ram Prakash Gupta <[email protected]> > Co-developed-by: Sarthak Garg <[email protected]> > Signed-off-by: Sarthak Garg <[email protected]> > Signed-off-by: Debraj Mukhopadhyay <[email protected]> > Signed-off-by: Neeraj Soni <[email protected]> > --- > drivers/mmc/host/sdhci-msm.c | 23 ++++++++++++++++++++++- > 1 file changed, 22 insertions(+), 1 deletion(-) > -- With best wishes Dmitry