From patchwork Mon Jan 6 01:34:00 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Dmitry Osipenko X-Patchwork-Id: 1217925 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-tegra-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="gEXPB5wZ"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 47rdQ43lgkz9sRs for ; Mon, 6 Jan 2020 12:36:12 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727423AbgAFBgF (ORCPT ); Sun, 5 Jan 2020 20:36:05 -0500 Received: from mail-lf1-f68.google.com ([209.85.167.68]:39687 "EHLO mail-lf1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727296AbgAFBeq (ORCPT ); Sun, 5 Jan 2020 20:34:46 -0500 Received: by mail-lf1-f68.google.com with SMTP id y1so35320857lfb.6; Sun, 05 Jan 2020 17:34:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=CkOxMb8MLB/z4Q/y2dHyNNeQ96KLRUp7GW5wicnZ/Co=; b=gEXPB5wZoCjRBYNpi5yemWO85whmF6IFyO7yOxC3A00Kv8UpN0HX9SMsLg3ksCFpV1 mNCi73WMakKkwcerlaoHZRVx5gnT8TbdXs4QQtDHDWP2jMaQKq1NSzl51IpovaOu8Yfu pQcI6d6a95zMGeRdYJy5DEuJTzMD69/1b0dl3U253RneD/egE3D4wR+BQumgLWkap3O8 PBPpKFyKv2lPUcglmGxjLQNSbdUHixRpuk4wscsIUDBGMTasax2gnRrDUbr9XiIj/p7B Ka7/shjJtiW1cVzYFgINJol9k+ulCceoaX/k9ngICvmSuImjj8L2pUqEf1oUVF2arO9p FBUA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=CkOxMb8MLB/z4Q/y2dHyNNeQ96KLRUp7GW5wicnZ/Co=; b=aMepnaVkNlIpqVUoGc/P3xSA1cgExfVAmSuvwimAmQQCqNRFqZNEhDXQt3KlkglkYa 8DHeKgTHN2MTgBnloaFaDuTCrwvqJ3veEY4ZblN0WOAGF7KAABjO4a/BJTY7tNHUDaL/ S+HlWJRXbHdlhGMkEjh1e3S4zyukeKx4xPfin+q1EDw9en88D66UExJLeyOd8+0xQCP0 aee3Y+QwmVgmFboRsuLBkZBbq3Ee73g+VBOX9YM75Ghc7oQB5cmxQ8Arb76MBa+ExrET x4UfYirWegreuRaTOfm4u+2imQgdfY8ANrot8KqVAWgPxNoiS2nIhK3K8N4T9uaAF3Et cRNQ== X-Gm-Message-State: APjAAAVVAUGFmaz0zYDU+PFMRRkIU5nZxNFv0NRD1SwTrxCm9SnMx5gY zrVOa6qC0MKM4tIB5/EcyuW8Rg69 X-Google-Smtp-Source: APXvYqznR+oyJpYX509o8r8Sw5SiTXqU+JN9HQTo/g0PhBrrGBJWw0KgiqOzDzxer3ZO1aw0SQXIiA== X-Received: by 2002:a05:6512:244:: with SMTP id b4mr54406085lfo.85.1578274484369; Sun, 05 Jan 2020 17:34:44 -0800 (PST) Received: from localhost.localdomain (79-139-233-37.dynamic.spd-mgts.ru. [79.139.233.37]) by smtp.gmail.com with ESMTPSA id h10sm28235739ljc.39.2020.01.05.17.34.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 05 Jan 2020 17:34:43 -0800 (PST) From: Dmitry Osipenko To: Greg Kroah-Hartman , Peter Chen , Thierry Reding , Jonathan Hunter , Felipe Balbi , =?utf-8?q?Micha=C5=82_Miros?= =?utf-8?q?=C5=82aw?= Cc: linux-usb@vger.kernel.org, linux-tegra@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH v4 04/20] usb: phy: tegra: Keep track of power on-off state Date: Mon, 6 Jan 2020 04:34:00 +0300 Message-Id: <20200106013416.9604-5-digetx@gmail.com> X-Mailer: git-send-email 2.24.0 In-Reply-To: <20200106013416.9604-1-digetx@gmail.com> References: <20200106013416.9604-1-digetx@gmail.com> MIME-Version: 1.0 Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org The PHY driver should keep track of the enable state, otherwise enable refcount is screwed if USB driver tries to enable PHY when it is already enabled. This will be the case for ChipIdea and Tegra EHCI drivers once PHY driver will gain support for the init/shutdown callbacks. Signed-off-by: Dmitry Osipenko --- drivers/usb/phy/phy-tegra-usb.c | 30 ++++++++++++++++++++++++++---- include/linux/usb/tegra_usb_phy.h | 1 + 2 files changed, 27 insertions(+), 4 deletions(-) diff --git a/drivers/usb/phy/phy-tegra-usb.c b/drivers/usb/phy/phy-tegra-usb.c index 99acfde4ab8d..88466c7f13e6 100644 --- a/drivers/usb/phy/phy-tegra-usb.c +++ b/drivers/usb/phy/phy-tegra-usb.c @@ -785,18 +785,40 @@ static void tegra_usb_phy_close(struct tegra_usb_phy *phy) static int tegra_usb_phy_power_on(struct tegra_usb_phy *phy) { + int err; + + if (phy->powered_on) + return 0; + if (phy->is_ulpi_phy) - return ulpi_phy_power_on(phy); + err = ulpi_phy_power_on(phy); else - return utmi_phy_power_on(phy); + err = utmi_phy_power_on(phy); + if (err) + return err; + + phy->powered_on = true; + + return 0; } static int tegra_usb_phy_power_off(struct tegra_usb_phy *phy) { + int err; + + if (!phy->powered_on) + return 0; + if (phy->is_ulpi_phy) - return ulpi_phy_power_off(phy); + err = ulpi_phy_power_off(phy); else - return utmi_phy_power_off(phy); + err = utmi_phy_power_off(phy); + if (err) + return err; + + phy->powered_on = false; + + return 0; } static int tegra_usb_phy_suspend(struct usb_phy *x, int suspend) diff --git a/include/linux/usb/tegra_usb_phy.h b/include/linux/usb/tegra_usb_phy.h index 0c5c3ea8b2d7..3ae73bdc6245 100644 --- a/include/linux/usb/tegra_usb_phy.h +++ b/include/linux/usb/tegra_usb_phy.h @@ -78,6 +78,7 @@ struct tegra_usb_phy { bool is_ulpi_phy; int reset_gpio; struct reset_control *pad_rst; + bool powered_on; }; void tegra_usb_phy_preresume(struct usb_phy *phy);