From patchwork Fri Apr 22 08:15:55 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 66418 Delivered-To: patches@linaro.org Received: by 10.140.93.198 with SMTP id d64csp584792qge; Fri, 22 Apr 2016 01:16:09 -0700 (PDT) X-Received: by 10.112.30.131 with SMTP id s3mr6925429lbh.124.1461312969339; Fri, 22 Apr 2016 01:16:09 -0700 (PDT) Return-Path: Received: from mail-lf0-x22c.google.com (mail-lf0-x22c.google.com. [2a00:1450:4010:c07::22c]) by mx.google.com with ESMTPS id 198si2017198lfh.105.2016.04.22.01.16.09 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 22 Apr 2016 01:16:09 -0700 (PDT) Received-SPF: pass (google.com: domain of ulf.hansson@linaro.org designates 2a00:1450:4010:c07::22c as permitted sender) client-ip=2a00:1450:4010:c07::22c; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: domain of ulf.hansson@linaro.org designates 2a00:1450:4010:c07::22c as permitted sender) smtp.mailfrom=ulf.hansson@linaro.org; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org Received: by mail-lf0-x22c.google.com with SMTP id e190so74919004lfe.0 for ; Fri, 22 Apr 2016 01:16:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=okDTD369Sx2SGH7WUKzNZXmpG6CgbhfWCL2NQdrM1kw=; b=IEYPaTNBNv+UpD+AkM2og6AJGSkulE52NYDww1yFqv3nNux5IYmqa0Rldcg/ouKzVV eBWYHkqOojw1qx8DRvNTaKNMSmGQohyB0wNBwBRaGAAaWg2JrggwIH8Kf4HekokARsZP CdlzUj4tAhQWedzuWqdfvVxx871Ba55ScU6f4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=okDTD369Sx2SGH7WUKzNZXmpG6CgbhfWCL2NQdrM1kw=; b=ARHteTj4WLI/IVzQezgtuRO2bXv+QXUnfxC45RLu1pcLgRiuj0oOeIFyXjYAzMNPQL ZVCzvmAwKp4QGGszwT6pQLKr8Y6NCY4qIEv8/xq2/mrpgk0pMDS39rCT6Clqz+Q0a5WQ c58J7DvtloRN4TcHybpGS7zGErhNowconHkjp9nD+2LT7XFLDcax8maZxSnxD9NYx18a XiOb/KKFMPcp3vG18vH67ACNwV6s57kCMp5lXOieVyfLPsU4yo/S/kWOOrP8fR3Rxn48 R7VWhWBllztPbznjJnot7KxQWB7Vvm1Xno7//PsftxidADV4rWUz/l7rFhNAYg0afhHd Ddww== X-Gm-Message-State: AOPr4FVXiO4hk4V5lw6DYfry+eXKTrEI5t5Yig1YArbA8eszofOxNSkaJWihFTiuoFHBXvYNWCM= X-Received: by 10.25.127.208 with SMTP id a199mr8251654lfd.46.1461312968996; Fri, 22 Apr 2016 01:16:08 -0700 (PDT) Return-Path: Received: from localhost.localdomain (h-155-4-128-67.na.cust.bahnhof.se. [155.4.128.67]) by smtp.gmail.com with ESMTPSA id v4sm1317109lbi.20.2016.04.22.01.16.07 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 22 Apr 2016 01:16:07 -0700 (PDT) From: Ulf Hansson To: "Rafael J. Wysocki" , Kevin Hilman , Ulf Hansson , linux-pm@vger.kernel.org Cc: Len Brown , Pavel Machek , Geert Uytterhoeven , Lina Iyer , Axel Haslam , Marek Szyprowski , Jon Hunter , Andy Gross , Laurent Pinchart Subject: [PATCH 2/2] PM / Domains: Drop unnecessary wakeup code from pm_genpd_prepare() Date: Fri, 22 Apr 2016 10:15:55 +0200 Message-Id: <1461312955-8547-3-git-send-email-ulf.hansson@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1461312955-8547-1-git-send-email-ulf.hansson@linaro.org> References: <1461312955-8547-1-git-send-email-ulf.hansson@linaro.org> As the PM core already have wakeup management during the system PM phase, it seems reasonable that genpd and its users should be able to rely on that. Therefore let's remove this from pm_genpd_prepare(). Signed-off-by: Ulf Hansson --- drivers/base/power/domain.c | 6 ------ 1 file changed, 6 deletions(-) -- 1.9.1 diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index 60a3573..de23b64 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -730,12 +730,6 @@ static int pm_genpd_prepare(struct device *dev) * at this point and a system wakeup event should be reported if it's * set up to wake up the system from sleep states. */ - if (pm_runtime_barrier(dev) && device_may_wakeup(dev)) - pm_wakeup_event(dev, 0); - - if (pm_wakeup_pending()) - return -EBUSY; - if (resume_needed(dev, genpd)) pm_runtime_resume(dev);