From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5BF10C3A5A2 for ; Tue, 3 Sep 2019 17:39:38 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 31C8421883 for ; Tue, 3 Sep 2019 17:39:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="V1JLMs9x"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="mSaHHI4H"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="FcbIZ/0q" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 31C8421883 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ArD1vIKCqmPNqeE00c0UhMA42L+oxyPMY4HHe9F8XwY=; b=V1JLMs9xxkFyfm 2t7BtIEDHI7+hSe1tAhNcW4tRc89D7q/8jcTUpw04uE2Fece+1ee3wAiYg92IGJ2OsDyiLWkPYiSp ZuBqBGRyL5T0vdn59jyb8scQXV/pNtjDI+P2kaIZ81cfa2R/Z+zHXJ/b8jal24hic1Q+Go/pJaJnJ QbIaohngXxOpiBv7b72d+HoWFFa2MvO3gx/Lj2DDOdXwsD//L4nwTzIzt6CFFt9nmqOrznKKczTWb r9Y2w3dn6rxoLb7ejUZzxCYx0j8J2t9053SuysX16JAz1s90qk64kr4Ank7WzxdXXLyr9cngXgU6f iQc5dVN5Nl/p3HOTF1Mg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92 #3 (Red Hat Linux)) id 1i5CmD-0004Cg-OA; Tue, 03 Sep 2019 17:39:33 +0000 Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.92 #3 (Red Hat Linux)) id 1i5Cm9-0004Ad-P6 for linux-arm-kernel@lists.infradead.org; Tue, 03 Sep 2019 17:39:31 +0000 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id A509E6076A; Tue, 3 Sep 2019 17:39:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1567532368; bh=xKD/ENinILDUC661caazKHdhGNze9nJEsqCHmhYq/gI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mSaHHI4HlqZuevjNtneF5Fa2yJjpP7bfH9j9pabBFRGe2jeFBT8mW9GzJPts/jfhS X3LvycC7PVgj6ebntpN4mHQyNl6fk+wPTvVN62n/ytmfCZdEFBnZHrbYmABrjNTKw7 bliWM9PTZ5qt6qS7ysbpzLR53CqNUkYOuLQlcQfg= Received: from jackp-linux.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: jackp@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id EF13C602DC; Tue, 3 Sep 2019 17:39:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1567532366; bh=xKD/ENinILDUC661caazKHdhGNze9nJEsqCHmhYq/gI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=FcbIZ/0q69EnCWiymdG7m0I+ACVb6+Yw9NnKnzl+XyDOR89aqQpX2ebgNrgB1GS7Z LJ7N5D2R1hF4yrXVwlMyOHwD00Q1EMANIY0r0jA/4pBPA3BUDQj7RwDk2/j/BQ7/n/ IB6yq6j6JC7XeMKSOSo3R0uwCk6DUCRQEiVdWhqg= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org EF13C602DC Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=jackp@codeaurora.org Date: Tue, 3 Sep 2019 10:39:24 -0700 From: Jack Pham To: Jorge Ramirez Subject: Re: [PATCH v4 3/4] dt-bindings: Add Qualcomm USB SuperSpeed PHY bindings Message-ID: <20190903173924.GB9754@jackp-linux.qualcomm.com> References: <20190207111734.24171-1-jorge.ramirez-ortiz@linaro.org> <20190207111734.24171-4-jorge.ramirez-ortiz@linaro.org> <20190223165218.GB572@tuxbook-pro> <6dc0957d-5806-7643-4454-966015865d38@linaro.org> <5d694878.1c69fb81.5f13b.ec4f@mx.google.com> <20190830164520.GK26807@tuxbook-pro> <5d696ad2.1c69fb81.977ea.39e5@mx.google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190903_103929_858254_75AB9D49 X-CRM114-Status: GOOD ( 26.93 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, robh@kernel.org, kishon@ti.com, gregkh@linuxfoundation.org, Stephen Boyd , linux-usb@vger.kernel.org, khasim.mohammed@linaro.org, linux-kernel@vger.kernel.org, Bjorn Andersson , devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, andy.gross@linaro.org, shawn.guo@linaro.org, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Sep 02, 2019 at 08:23:04AM +0200, Jorge Ramirez wrote: > On 8/30/19 20:28, Stephen Boyd wrote: > > Quoting Bjorn Andersson (2019-08-30 09:45:20) > >> On Fri 30 Aug 09:01 PDT 2019, Stephen Boyd wrote: > >> > >>> Quoting Jorge Ramirez (2019-08-29 00:03:48) > >>>> On 2/23/19 17:52, Bjorn Andersson wrote: > >>>>> On Thu 07 Feb 03:17 PST 2019, Jorge Ramirez-Ortiz wrote: > >>>>>> + > >>>>>> +Required child nodes: > >>>>>> + > >>>>>> +- usb connector node as defined in bindings/connector/usb-connector.txt > >>>>>> + containing the property vbus-supply. > >>>>>> + > >>>>>> +Example: > >>>>>> + > >>>>>> +usb3_phy: usb3-phy@78000 { > >>>>>> + compatible = "qcom,snps-usb-ssphy"; > >>>>>> + reg = <0x78000 0x400>; > >>>>>> + #phy-cells = <0>; > >>>>>> + clocks = <&rpmcc RPM_SMD_LN_BB_CLK>, > >>>>>> + <&gcc GCC_USB_HS_PHY_CFG_AHB_CLK>, > >>>>>> + <&gcc GCC_USB3_PHY_PIPE_CLK>; > >>>>>> + clock-names = "ref", "phy", "pipe"; > >>>>>> + resets = <&gcc GCC_USB3_PHY_BCR>, > >>>>>> + <&gcc GCC_USB3PHY_PHY_BCR>; > >>>>>> + reset-names = "com", "phy"; > >>>>>> + vdd-supply = <&vreg_l3_1p05>; > >>>>>> + vdda1p8-supply = <&vreg_l5_1p8>; > >>>>>> + usb3_c_connector: usb3-c-connector { > >>> > >>> Node name should be 'connector', not usb3-c-connector. > >>> > >> > >> It probably has to be usb-c-connector, because we have a > >> micro-usb-connector on the same board. > > > > Ok. Or connector@1 and connector@2? Our toplevel node container story is > > sort of sad because we have to play tricks with node names. But in the > > example, just connector I presume? > > > >> > >>>>> > >>>>> The USB-C connector is attached both to the HS and SS PHYs, so I think > >>>>> you should represent this external to this node and use of_graph to > >>>>> query it. > >>>> > >>>> but AFAICS we wont be able to retrieve the vbux-supply from an external > >>>> node (that interface does not exist). > >>>> > >>>> rob, do you have a suggestion? > >>> > >>> Shouldn't the vbus supply be in the phy? Or is this a situation where > >>> the phy itself doesn't have the vbus supply going to it because the PMIC > >>> gets in the way and handles the vbus for the connector by having the SoC > >>> communicate with the PMIC about when to turn the vbus on and off, etc? > >>> > >> > >> That's correct, the VBUS comes out of the PMIC and goes directly to the > >> connector. > >> > >> The additional complicating factor here is that the connector is wired > >> to a USB2 phy as well, so we need to wire up detection and vbus control > >> to both of them - but I think this will be fine, if we can only figure > >> out a sane way of getting hold of the vbus-supply. > >> > > > > Does it really matter to describe this situation though? Maybe it's > > simpler to throw the vbus supply into the phy and control it from the > > phy driver, even if it never really goes there. Or put it into the > > toplevel usb controller? > > > that would work for me - the connector definition seemed a better way to > explain the connectivity but since we cant retrieve the supply from the > external node is not of much functional use. > > but please let me know how to proceed. shall I add the supply back to > the phy? Putting it in the toplevel usb node makes sense to me, since that's usually the driver that knows when it's switching into host mode and needs to turn on VBUS. The dwc3-qcom driver & bindings currently don't do this but there's precedent in a couple of the other dwc3 "glues"--see Documentation/devicetree/bindings/usb/{amlogic\,dwc3,omap-usb}.txt One exception is if the PMIC is also USB-PD capable and can do power role swap, in which case the VBUS control needs to be done by the TCPM, so that'd be a case where having vbus-supply in the connector node might make more sense. Jack -- The Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel