From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756599AbaJ2Rqi (ORCPT ); Wed, 29 Oct 2014 13:46:38 -0400 Received: from mail-pd0-f170.google.com ([209.85.192.170]:33411 "EHLO mail-pd0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756579AbaJ2Rqd (ORCPT ); Wed, 29 Oct 2014 13:46:33 -0400 From: Kevin Hilman To: Krzysztof Kozlowski Cc: "Rafael J. Wysocki" , Pavel Machek , Len Brown , Greg Kroah-Hartman , Inki Dae , Joonyoung Shim , Seung-Woo Kim , Kyungmin Park , David Airlie , Kukjin Kim , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Andrzej Hajda , Marek Szyprowski , Bartlomiej Zolnierkiewicz Subject: Re: [RFC 1/2] PM / Domains: Power on domain early during system resume References: <1414072090-11293-1-git-send-email-k.kozlowski@samsung.com> <1414072090-11293-2-git-send-email-k.kozlowski@samsung.com> Date: Wed, 29 Oct 2014 10:46:29 -0700 In-Reply-To: <1414072090-11293-2-git-send-email-k.kozlowski@samsung.com> (Krzysztof Kozlowski's message of "Thu, 23 Oct 2014 15:48:09 +0200") Message-ID: <7hk33iahvu.fsf@deeprootsystems.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Krzysztof Kozlowski writes: > When resuming the system the power domain has to be powered on early so > any runtime PM aware devices could resume. > > This fixes following scenario reproduced on Exynos DRM: > 1. Power domain is off before suspending the system. > 2. System is suspended to RAM. > 3. Resuming starts. The Exynos DRM driver resume callback is called. > 4. The Exynos DRM driver calls drm_helper_resume_force_mode which turns > the screen on by calling exynos_dsi_dpms with DRM_MODE_DPMS_ON. Dumb Q: if the device (and power domain) were off before (and during) suspend, why are they being resumed? Shouldn't the resume path restore things to the same state they were before suspend? Kevin