From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753920AbdJID0H (ORCPT ); Sun, 8 Oct 2017 23:26:07 -0400 Received: from mailgw02.mediatek.com ([218.249.47.111]:42051 "EHLO mailgw02.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1753455AbdJID0F (ORCPT ); Sun, 8 Oct 2017 23:26:05 -0400 X-UUID: 4565e01bba6e4117ac323811a1ce1cd0-20171009 Message-ID: <1507519554.17567.65.camel@mhfsdcap03> Subject: Re: [PATCH 12/12] dt-bindings: usb: mtu3: remove optional pinctrls From: Chunfeng Yun To: Rob Herring CC: Greg Kroah-Hartman , Felipe Balbi , Mathias Nyman , Matthias Brugger , Mark Rutland , Ian Campbell , , , , , Date: Mon, 9 Oct 2017 11:25:54 +0800 In-Reply-To: <20171005223325.ryvu6ifjgcj5scf4@rob-hp-laptop> References: <20171005223325.ryvu6ifjgcj5scf4@rob-hp-laptop> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2017-10-05 at 17:33 -0500, Rob Herring wrote: > On Thu, Sep 28, 2017 at 08:17:20AM +0800, Chunfeng Yun wrote: > > Remove optional pinctrls due to using FORCE/RG_IDDIG to implement > > manual switch function. > > Another not backwards compatible change. Please explain why that is > okay. The old way of manual switch function uses the IDDIG pin (IDPIN) to switch mode between device and host indirectly by pulling up or pulling down it by driver. But there is a limitation, when the IDDIG pin is connected to the ground, the driver can't pull it up anymore, so it will fail when the receptacle is micro type. And there is no such limitation through FORCE/RG_IDDIG, because when the FORCE_IDDIG is set, the staus of RG_IDDIG is given higher priority than IDDIG pin. So it is okay no matter whether id_float and id_ground pins are provided or not. > > > > > Signed-off-by: Chunfeng Yun > > --- > > .../devicetree/bindings/usb/mediatek,mtu3.txt | 7 ++++--- > > 1 file changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/usb/mediatek,mtu3.txt b/Documentation/devicetree/bindings/usb/mediatek,mtu3.txt > > index 49c982b..b2271d8 100644 > > --- a/Documentation/devicetree/bindings/usb/mediatek,mtu3.txt > > +++ b/Documentation/devicetree/bindings/usb/mediatek,mtu3.txt > > @@ -30,9 +30,10 @@ Optional properties: > > when supports dual-role mode. > > - vbus-supply : reference to the VBUS regulator, needed when supports > > dual-role mode. > > - - pinctl-names : a pinctrl state named "default" must be defined, > > - "id_float" and "id_ground" are optinal which depends on > > - "mediatek,enable-manual-drd" > > + - pinctrl-names : a pinctrl state named "default" is optional, and need be > > + defined if auto drd switch is enabled, that means the property dr_mode > > + is set as "otg", and meanwhile the property "mediatek,enable-manual-drd" > > + is not set. > > - pinctrl-0 : pin control group > > See: Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt > > > > -- > > 1.7.9.5 > >