From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH] dmaengine: rcar-dmac: Make DMAC reinit during system resume explicit Date: Wed, 17 Jan 2018 15:49:17 +0530 Message-ID: <20180117101917.GN18649@localhost> References: <1516181908-16930-1-git-send-email-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1516181908-16930-1-git-send-email-geert+renesas@glider.be> Sender: linux-renesas-soc-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Dan Williams , Ulf Hansson , "Rafael J . Wysocki" , dmaengine@vger.kernel.org, linux-pm@vger.kernel.org, linux-renesas-soc@vger.kernel.org List-Id: linux-pm@vger.kernel.org On Wed, Jan 17, 2018 at 10:38:28AM +0100, Geert Uytterhoeven wrote: > The current (empty) system sleep callbacks rely on the PM core to force > a runtime resume to reinitialize the DMAC registers during system > resume. Without a reinitialization, e.g. SCIF DMA will hang silently > after a system resume on R-Car Gen3. > > Make this explicit by using pm_runtime_force_{suspend,resume}() as the > system sleep callbacks instead. Use SET_LATE_SYSTEM_SLEEP_PM_OPS() as > DMA engines must be initialized before all DMA slave devices. > > Suggested-by: Ulf Hansson > Signed-off-by: Geert Uytterhoeven > --- > This is a dependency for "[PATCH 1/2] PM / genpd: Stop/start devices > without pm_runtime_force_suspend/resume()" > (https://www.spinics.net/lists/kernel/msg2696802.html), so perhaps it > makes most sense if Rafael takes it through the PM tree? Sounds okay to me. Acked-By: Vinod Koul -- ~Vinod