Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754917AbcDFMR7 (ORCPT ); Wed, 6 Apr 2016 08:17:59 -0400 Received: from mail-db3on0066.outbound.protection.outlook.com ([157.55.234.66]:23232 "EHLO emea01-db3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753312AbcDFMR5 (ORCPT ); Wed, 6 Apr 2016 08:17:57 -0400 From: Jun Li To: Baolin Wang , Peter Chen CC: 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 v9 2/4] gadget: Support for the usb charger framework Thread-Topic: [PATCH v9 2/4] gadget: Support for the usb charger framework Thread-Index: AQHRi+evqEo8RiU/f0GkOiqzWpmAkZ98kaEAgAA5vgCAABNbsA== Date: Wed, 6 Apr 2016 12:03:20 +0000 Message-ID: References: <13c2f4fb71958bf9a5527acbed8b8b60dc569656.1459494744.git.baolin.wang@linaro.org> <20160406071956.GA21101@shlinux2.ap.freescale.net> 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.225.131] x-ms-office365-filtering-correlation-id: f3c499a0-f174-4801-9810-08d35e13729c x-microsoft-exchange-diagnostics: 1;AM4PR04MB2129;5:q2WEuLNvLFw0S7G33cdqsHP06dxoaM4blcv1rP2ggdOp5t/Vn+KUg1h+tZb406h9etQs55Dz48lsLsPQMvA/e0shB/hQSPA3rGfge3rSoE4iP4voWeQ6Q/26C1mPGE1MZs3Z6eE3hQQEvOCrhyleRA==;24:sUA1jpywbhwv6+twPZ5J67OQ+4up7QWZdXyIIXSdW7lz0AjCHSB+3hwjRUz3nJsnYYeFnPPaYLkHr16IKbeSxP2v7YGYZ34/1O4jYO/1HqE= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:AM4PR04MB2129; 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)(3002001)(10201501046)(6055026);SRVR:AM4PR04MB2129;BCL:0;PCL:0;RULEID:;SRVR:AM4PR04MB2129; x-forefront-prvs: 0904004ECB x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(24454002)(13464003)(377454003)(2900100001)(2950100001)(189998001)(19580395003)(1096002)(5001770100001)(1220700001)(19580405001)(10400500002)(3280700002)(122556002)(3660700001)(81166005)(586003)(86362001)(4326007)(2906002)(6116002)(5008740100001)(3846002)(102836003)(92566002)(66066001)(77096005)(76576001)(76176999)(50986999)(54356999)(74316001)(93886004)(5003600100002)(87936001)(33656002)(5004730100002)(106116001)(5002640100001)(7059030);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR04MB2129;H:AM4PR04MB2130.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: 06 Apr 2016 12:03:20.8064 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR04MB2129 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 u36CI6Oc002334 Content-Length: 1924 Lines: 42 Hi > -----Original Message----- > From: linux-usb-owner@vger.kernel.org [mailto:linux-usb- > owner@vger.kernel.org] On Behalf Of Baolin Wang > Sent: Wednesday, April 06, 2016 6:47 PM > To: Peter Chen > Cc: 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 v9 2/4] gadget: Support for the usb charger framework > > On 6 April 2016 at 15:19, Peter Chen wrote: > > On Fri, Apr 01, 2016 at 03:21:50PM +0800, Baolin Wang wrote: > >> > >> @@ -563,6 +564,8 @@ struct usb_gadget_ops { > >> struct usb_ep *(*match_ep)(struct usb_gadget *, > >> struct usb_endpoint_descriptor *, > >> struct usb_ss_ep_comp_descriptor *); > >> + /* get the charger type */ > >> + enum usb_charger_type (*get_charger_type)(struct usb_gadget *); > >> }; > > > > Since we already have get_charger_type callback at usb_charger > > structure, why we still need this API at usb_gadget_ops? > > In case some users want to get charger type at gadget level. > Why gadget needs to know charger type? I also don't catch the intent of This api, as my understanding, gadget only need report gadget state changes. All information required for usb charger is charger type and gadget state. Li Jun