From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751312AbdARHht (ORCPT ); Wed, 18 Jan 2017 02:37:49 -0500 Received: from mail-pg0-f66.google.com ([74.125.83.66]:33339 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751231AbdARHhs (ORCPT ); Wed, 18 Jan 2017 02:37:48 -0500 Date: Wed, 18 Jan 2017 15:34:32 +0800 From: Peter Chen To: Stephen Boyd Cc: linux-usb@vger.kernel.org, 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 , Kishon Vijay Abraham I Subject: Re: [PATCH v6 23/25] usb: chipidea: Pullup D+ in device mode via phy APIs Message-ID: <20170118073432.GA26659@b29397-desktop> References: <20161228225711.698-1-stephen.boyd@linaro.org> <20161228225711.698-24-stephen.boyd@linaro.org> <20170103065318.GB14542@b29397-desktop> <148418039309.32258.7247753739063801774@sboyd-linaro> <20170112095040.GA15726@b29397-desktop> <148426139182.20672.2422337338087784924@sboyd-linaro> <20170113033536.GA20723@b29397-desktop> <148433778067.32719.6179204976256776867@sboyd-linaro> <20170116034551.GA27364@b29397-desktop> <148467591341.23457.3638889856786813589@sboyd-linaro> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <148467591341.23457.3638889856786813589@sboyd-linaro> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 17, 2017 at 09:58:33AM -0800, Stephen Boyd wrote: > Quoting Peter Chen (2017-01-15 19:45:51) > > > > So, you need to call phy_set_mode when switching between host and device. > > Besides, you also need to toggle VBUSVLDEXT when the external vbus > > is on or off at device mode (doesn't need for host mode), is it correct? > > Correct. > > > > > At include/linux/usb/phy.h, we have .set_vbus interface, maybe you need > > to port it to generic phy framework. > > > > Ok. I'll look into that. Can the other patches in this series be picked > up? Otherwise I can resend them all again once I fix the phy_set_mode() > call location and introduce a new phy op. I can pick up chipidea patches after you test the patch I supplied at: [PATCH v6 11/25] usb: chipidea: vbus event may exist before starting gadget You may ping other maintainers to pick up other patches. -- Best Regards, Peter Chen