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: 9320035 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id D488A607D3 for ; Wed, 7 Sep 2016 21:39:33 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id E13A829433 for ; Wed, 7 Sep 2016 21:39:33 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id D5B8D29436; Wed, 7 Sep 2016 21:39:33 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8373329433 for ; Wed, 7 Sep 2016 21:39:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757284AbcIGVjb (ORCPT ); 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 X-Virus-Scanned: ClamAV using ClamSMTP 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(+) 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);