Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751978AbcC3K67 (ORCPT ); Wed, 30 Mar 2016 06:58:59 -0400 Received: from mail-db3on0094.outbound.protection.outlook.com ([157.55.234.94]:42656 "EHLO emea01-db3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751541AbcC3K65 (ORCPT ); Wed, 30 Mar 2016 06:58:57 -0400 From: Jun Li To: Baolin Wang CC: Peter Chen , Felipe Balbi , "Greg KH" , Sebastian Reichel , "Dmitry Eremin-Solenikov" , David Woodhouse , Peter Chen , Alan Stern , "r.baldyga@samsung.com" , Yoshihiro Shimoda , Lee Jones , Mark Brown , Charles Keepax , "patches@opensource.wolfsonmicro.com" , Linux PM list , USB , "device-mainlining@lists.linuxfoundation.org" , LKML Subject: RE: [PATCH v8 0/4] Introduce usb charger framework to deal with the usb gadget power negotation Thread-Topic: [PATCH v8 0/4] Introduce usb charger framework to deal with the usb gadget power negotation Thread-Index: AQHRhcncDPeno7bqik6en7wacMJBHJ9pvwGAgASx+gCAAa8+QIAAFJIAgAEXTJCAAD9RgIAAF0AwgAAfdQCAAAKJQA== Date: Wed, 30 Mar 2016 10:58:52 +0000 Message-ID: References: <20160325070937.GA22398@peterchendt> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: linaro.org; dkim=none (message not signed) header.d=none;linaro.org; dmarc=none action=none header.from=nxp.com; x-originating-ip: [199.59.226.141] x-ms-office365-filtering-correlation-id: fe937689-1954-45e5-8ad8-08d3588a47fb x-microsoft-exchange-diagnostics: 1;AM4PR04MB1667;5:oyksRREdxB0xEulleh1uAZbIGrodi70myOqqhZX3VjBxTdRbcc97ejAPLbGQIKi1axvwVTWoXI6XZGCr6sTUkIGY817FUqNuXgqZ+DBVKgL++LyUZGT9w8UwQplK7SuvvSBPCcnQEI5SqaiEKqJ4qg==;24:K5b4BXE5QPOPVyorS5kAzg02HwC/r5bH6pWqdTpBGpBDjROQYqogKY6ZJRwkbDx3Y36meeyoITUwFNfd+0yk69xyBh6Voi3SxkGd4Nx11jQ= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:AM4PR04MB1667; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:; x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:AM4PR04MB1667;BCL:0;PCL:0;RULEID:;SRVR:AM4PR04MB1667; x-forefront-prvs: 08978A8F5C x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(13464003)(377454003)(24454002)(76576001)(106116001)(33656002)(66066001)(76176999)(50986999)(54356999)(5004730100002)(11100500001)(5003600100002)(110136002)(189998001)(5002640100001)(5008740100001)(87936001)(5890100001)(92566002)(5250100002)(586003)(86362001)(19580395003)(19580405001)(1096002)(1220700001)(102836003)(81166005)(6116002)(3846002)(3660700001)(3280700002)(2906002)(4326007)(2950100001)(2900100001)(93886004)(74316001)(7059030);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR04MB1667;H:AM4PR04MB1665.eurprd04.prod.outlook.com;FPR:;SPF:None;MLV:sfv;LANG:en; spamdiagnosticoutput: 1:23 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Mar 2016 10:58:52.4364 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR04MB1667 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u2UAx34K021624 Content-Length: 4663 Lines: 117 > -----Original Message----- > From: Baolin Wang [mailto:baolin.wang@linaro.org] > Sent: Wednesday, March 30, 2016 5:31 PM > To: Jun Li > Cc: Peter Chen ; Felipe Balbi ; > Greg KH ; Sebastian Reichel ; > Dmitry Eremin-Solenikov ; David Woodhouse > ; Peter Chen ; Alan Stern > ; r.baldyga@samsung.com; Yoshihiro Shimoda > ; Lee Jones ; Mark > Brown ; Charles Keepax > ; patches@opensource.wolfsonmicro.com; > Linux PM list ; USB ; > device-mainlining@lists.linuxfoundation.org; LKML kernel@vger.kernel.org> > Subject: Re: [PATCH v8 0/4] Introduce usb charger framework to deal with > the usb gadget power negotation > > On 30 March 2016 at 16:07, Jun Li wrote: > > Hi > > >> On 30 March 2016 at 10:54, Jun Li wrote: > >> >> >> It is not for udc driver but for power users who want to > >> >> >> negotiate with USB subsystem. > >> >> >> > >> >> > > >> >> > Seems you don't want to guarantee charger type detection is done > >> >> > before gadget connection(pullup DP), right? > >> >> > I see you call usb_charger_detect_type() in each gadget usb > >> >> > state > >> >> changes. > >> >> > >> >> I am not sure I get your point correctly, please correct me if I > >> >> misunderstand you. > >> >> We need to check the charger type at every event comes from the > >> >> usb gadget state changes or the extcon device state changes, which > >> >> means a new charger plugin or pullup. > >> >> > >> > > >> > According to usb charger spec, my understanding is you can't do > >> > real charger detection procedure *after* gadget _connection_(pullup > >> > DP), also I don't > >> > >> Why can not? Charger detection is usually from PMIC. > > > > Charger detection process will impact DP/DM line state, see usb > > charger spec v1.2 for detail detection process, section 4.6.3 says: > > > > "A PD is allowed to *disconnect* and repeat the charger detection > > process multiple times while attached. The PD is required to wait for > > a time of at least TCP_VDM_EN max between disconnecting and restarting > > the charger detection process." > > > > As Peter mentioned, the charger detection should happen between VBUS > > detection and gadget pull up DP for first plug in case. So when&after > > gadget connect (pullup DP), you should already know the charger type. > > Make sense. In our company's solution, charger detection can be done by > hardware from PMIC at first, then it will not affect the DP/DM line when > gadget starts to enumeration. I see, charger type detection is done automatically by PMIC when VBUS is detected in your case, you just assume the process is complete before SW do gadget connect. To make the framework common, you may do one time charger type check when vbus is on, and save it to avoid repeat charger type check. > In the 'usb_charger_detect_type()', it > usually get the charger type from type registers has been done by hardware > from PMIC, which can not affect the DP/DM line. > > > > > Li Jun > > > >> > >> > think it's necessary to check charger type at every event from usb > >> gadget. > >> > >> My meaning is not every event from usb gadget. When the usb gadget > >> state changes or the extcon device (maybe GPIO detection) state > >> changes, which means charger plugin or pullup, we need to check the > >> charger type to set current. > > > > From your below code, you call usb_charger_notify_others() in every > > state change. > > I think it does not matter. In case the usb charger missed some gadget > state changes. Or I replace it with 'USB_STATE_CONFIGURED' state. > Thanks. > > > > > if (uchger->old_gadget_state != state) { > > uchger->old_gadget_state = state; > > > > if (state >= USB_STATE_ATTACHED) > > uchger_state = USB_CHARGER_PRESENT; > > else if (state == USB_STATE_NOTATTACHED) > > uchger_state = USB_CHARGER_REMOVE; > > else > > /* this else will never happen */ > > uchger_state = USB_CHARGER_DEFAULT; > > > > usb_charger_notify_others(uchger, uchger_state); } > > > >> > >> > Something in gadget driver you can utilize is only vbus detection, > >> > and report diff current by diff usb state if it's a SDP. > >> > >> -- > >> Baolin.wang > >> Best Regards > > > > -- > Baolin.wang > Best Regards