Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754585AbcJEH0l (ORCPT ); Wed, 5 Oct 2016 03:26:41 -0400 Received: from mail-yb0-f180.google.com ([209.85.213.180]:35592 "EHLO mail-yb0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754436AbcJEH0h (ORCPT ); Wed, 5 Oct 2016 03:26:37 -0400 MIME-Version: 1.0 In-Reply-To: References: <8760q9a8m6.fsf@notabene.neil.brown.name> <878tv297a0.fsf@notabene.neil.brown.name> <87y4326l44.fsf@notabene.neil.brown.name> <87twdo7ouz.fsf@notabene.neil.brown.name> From: Baolin Wang Date: Wed, 5 Oct 2016 15:26:35 +0800 Message-ID: Subject: Re: [PATCH v16 0/4] Introduce usb charger framework to deal with the usb gadget power negotation To: NeilBrown Cc: NeilBrown , Felipe Balbi , Greg KH , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse , robh@kernel.org, Jun Li , Marek Szyprowski , Ruslan Bilovol , Peter Chen , Alan Stern , r.baldyga@samsung.com, grygorii.strashko@ti.com, Yoshihiro Shimoda , Lee Jones , Mark Brown , Charles Keepax , patches@opensource.wolfsonmicro.com, Linux PM list , USB , device-mainlining@lists.linuxfoundation.org, LKML , "Bird, Timothy" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1324 Lines: 37 Hi Felipe, >> But you do! >> The mA number from the USB configuration is passed to usb_gadget_vbus_draw. >> Your patch passes that to usb_charger_set_cur_limit_by_type() >> which calls __usb_charger_set_cur_limit_by_type() which will set the >> cur_limit for whichever type uchger->type currently is. >> >> So when it is not relevant, your code *does* set some current limit. > > Suppose the charger type is DCP(it is not relevant to the mA number > from the USB configuration ), it will not do the USB enumeration, then > no USB configuration from host to set current. >From the talking, there are some issues (thanks for Neil's comments) need to be fixed as below: 1. Need to add the method getting charger type from extcon subsystem. 2. Need to remove the method getting charger type from power supply. 3. There are still some different views about reporting the maximum current or minimum current to power driver. Now the current v16 patchset can work well on my Spreadtrum platform and Jun's NXP platform, if you like to apply this patchset then I can send out new patches to fix above issues. If you don't like that, I can send out new version patchset to fix above issues. Could you give me some suggestions what should I do next step? Thanks. > > -- > Baolin.wang > Best Regards -- Baolin.wang Best Regards