From patchwork Wed Sep 7 21:35:08 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Boyd X-Patchwork-Id: 75710 Delivered-To: patch@linaro.org Received: by 10.140.106.11 with SMTP id d11csp527093qgf; Wed, 7 Sep 2016 14:39:33 -0700 (PDT) X-Received: by 10.66.183.206 with SMTP id eo14mr49149928pac.146.1473284373593; Wed, 07 Sep 2016 14:39:33 -0700 (PDT) Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x87si40673656pfa.79.2016.09.07.14.39.33; Wed, 07 Sep 2016 14:39:33 -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 S1757284AbcIGVjb (ORCPT + 8 others); Wed, 7 Sep 2016 17:39:31 -0400 Received: from mail-pf0-f177.google.com ([209.85.192.177]:35992 "EHLO mail-pf0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756621AbcIGVfg (ORCPT ); Wed, 7 Sep 2016 17:35:36 -0400 Received: by mail-pf0-f177.google.com with SMTP id 128so10401303pfb.3 for ; Wed, 07 Sep 2016 14:35:36 -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=KOf2bTXDZkIPGOscIa+5bdEzXNuOc8AwT7UgFtQ+F2d/LTxjssLZZx2f5oxGOuOGtz UacrCcMxfePW7SB5f0Bp9S6LEKE2QMBfMMSxzHgn/JhKyZm1LhNU6B7T9zLmISEmZCo0 7/Uvec950GbT8oEMLAJ4PGyJiSwDP5O0Pm+n4= 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=JM6JWO21oGIO+WkfT5q8WPhR9/xgVQsRI9AXW6FzzbliWKsEAArJ0vIBqFR32v22sc 5qU1MDcGIgaA2f6NuJNceZo19p0//dky849WwFvwnbbnR9JiWAhFNXqY3KGoa20e6/Jy Lm6i9DxhK4t2LDvB+N6Xxh5gXKzgW/VECMg3wE/xRb8YTCw6b8Jp2gaedGmxMMezNyWS 84xeRg2+f4Aotl9sFUJ9OB30cCFmZInbF80mvaMKHRXtRDYWEu31/UyhouAuA3ssMsij /G1qKsP5YeLXbwf1l/b2AeUNMbkYqn/0Wb3MQzcovTbO8L5llzBpsXZOntjQSFfA8jC+ AgTw== X-Gm-Message-State: AE9vXwNzG4c2BVzwfKm781pNk50YPAGmH4M+0JIBm3H0IDC6bGMSu5ra2UH0ehVqptWqQOSh X-Received: by 10.98.31.219 with SMTP id l88mr86204528pfj.155.1473284135684; Wed, 07 Sep 2016 14:35:35 -0700 (PDT) Received: from localhost.localdomain (ip68-101-172-78.sd.sd.cox.net. [68.101.172.78]) by smtp.gmail.com with ESMTPSA id 75sm51015417pfw.92.2016.09.07.14.35.34 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 07 Sep 2016 14:35:35 -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 v4 11/22] usb: chipidea: msm: Mark device as runtime pm active Date: Wed, 7 Sep 2016 14:35:08 -0700 Message-Id: <20160907213519.27340-12-stephen.boyd@linaro.org> X-Mailer: git-send-email 2.9.0.rc2.8.ga28705d In-Reply-To: <20160907213519.27340-1-stephen.boyd@linaro.org> References: <20160907213519.27340-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);