From patchwork Thu Sep 1 00:40:25 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Boyd X-Patchwork-Id: 75153 Delivered-To: patch@linaro.org Received: by 10.140.29.8 with SMTP id a8csp20705qga; Wed, 31 Aug 2016 17:44:01 -0700 (PDT) X-Received: by 10.98.46.199 with SMTP id u190mr5599675pfu.160.1472690641280; Wed, 31 Aug 2016 17:44:01 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e85si2395535pfb.236.2016.08.31.17.43.59; Wed, 31 Aug 2016 17:44:01 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-arm-msm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org; spf=pass (google.com: best guess record for domain of linux-arm-msm-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-arm-msm-owner@vger.kernel.org; dmarc=fail (p=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932297AbcIAAn6 (ORCPT + 8 others); Wed, 31 Aug 2016 20:43:58 -0400 Received: from mail-pf0-f175.google.com ([209.85.192.175]:32858 "EHLO mail-pf0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933863AbcIAAku (ORCPT ); Wed, 31 Aug 2016 20:40:50 -0400 Received: by mail-pf0-f175.google.com with SMTP id g202so1687501pfb.0 for ; Wed, 31 Aug 2016 17:40:50 -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=61EduWzqgU8457WIuqKkDMe19q/Dh0AD6eWIUZRtO7Y=; b=bv/8pKNhncbZGz1WOA9T9FrurkqyLyF6gXpclc7o/bHh/4hyYUbrA/SnWdlocsgrh/ Tl74nxTZs5kNSCKwuKCEXDIW0EXx3mLFVRWidRWy5Y7XFu45ERvqeX7hXxPHp6X1cMwl EcUXPO7e0ua2cxcdS2YDZBGigvonfLoO1G5Vg= 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=61EduWzqgU8457WIuqKkDMe19q/Dh0AD6eWIUZRtO7Y=; b=EilMWeiUdGoqJO1bfTwp3qztw6L4e1j0mc/PpzuZK/CC1noUSlSI/18w1UzqzIadGI 5CHPCe/XNQuScxX9YTUWI81E6sllxsof1iBMOZmlI7noQ4NdNan78b6WzFVO21aDGrVq +wZnF2iBdYHAqBAestnbdxITLGBhpKtD7mQiwF5kqwuP5OssZ6pPRUShHkUJI1+Vu5cd XKVQ+Dq4UyByfIjnzXlc9dS1rThauOr0R3DQtbYNVzOhGb4sR5HS3HgiG1vHNLRg+XBD 8ETY5yRFGMm/DRqBFsHpW/cBYgOnO2xTpQX37ftsHfeu+PrVpD7zdGimV8cR3dfiVHWq xOdg== X-Gm-Message-State: AE9vXwN4tn3ax4lZdeXaugrkGU7AFXuxO5AHsEDK7lr1DshNzMjGyM6z3ldjtu0/q1dil6zH X-Received: by 10.98.16.75 with SMTP id y72mr22024321pfi.50.1472690450018; Wed, 31 Aug 2016 17:40:50 -0700 (PDT) Received: from localhost.localdomain (i-global254.qualcomm.com. [199.106.103.254]) by smtp.gmail.com with ESMTPSA id m5sm2292303paw.40.2016.08.31.17.40.49 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 31 Aug 2016 17:40:49 -0700 (PDT) From: Stephen Boyd To: linux-usb@vger.kernel.org Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, Andy Gross , Bjorn Andersson , Neil Armstrong , Arnd Bergmann , Felipe Balbi , Peter Chen , Greg Kroah-Hartman Subject: [PATCH v3 11/22] usb: chipidea: msm: Mark device as runtime pm active Date: Wed, 31 Aug 2016 17:40:25 -0700 Message-Id: <20160901004036.23936-12-stephen.boyd@linaro.org> X-Mailer: git-send-email 2.9.0.rc2.8.ga28705d In-Reply-To: <20160901004036.23936-1-stephen.boyd@linaro.org> References: <20160901004036.23936-1-stephen.boyd@linaro.org> Sender: linux-arm-msm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-arm-msm@vger.kernel.org We're not properly marking the glue layer/wrapper device as runtime active, so runtime PM believes that the hardware state is inactive when we call pm_runtime_enable() in this driver. This causes a problem when the glue layer has a power domain associated with it, because runtime PM will go and disable the power domain to match the 'inactive' state of the device. Let's mark the device as active so that runtime PM doesn't improperly power down this device when it's actually active. Acked-by: Peter Chen Cc: Greg Kroah-Hartman Signed-off-by: Stephen Boyd --- drivers/usb/chipidea/ci_hdrc_msm.c | 1 + 1 file changed, 1 insertion(+) -- 2.9.0.rc2.8.ga28705d -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/drivers/usb/chipidea/ci_hdrc_msm.c b/drivers/usb/chipidea/ci_hdrc_msm.c index 3889809fd0c4..89c1a02d69b5 100644 --- a/drivers/usb/chipidea/ci_hdrc_msm.c +++ b/drivers/usb/chipidea/ci_hdrc_msm.c @@ -80,6 +80,7 @@ static int ci_hdrc_msm_probe(struct platform_device *pdev) platform_set_drvdata(pdev, plat_ci); + pm_runtime_set_active(&pdev->dev); pm_runtime_no_callbacks(&pdev->dev); pm_runtime_enable(&pdev->dev);