From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757030AbcH2JC2 (ORCPT ); Mon, 29 Aug 2016 05:02:28 -0400 Received: from mail-yb0-f182.google.com ([209.85.213.182]:35611 "EHLO mail-yb0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756883AbcH2JCZ (ORCPT ); Mon, 29 Aug 2016 05:02:25 -0400 MIME-Version: 1.0 In-Reply-To: References: From: Baolin Wang Date: Mon, 29 Aug 2016 17:02:23 +0800 Message-ID: Subject: Re: [PATCH v16 0/4] Introduce usb charger framework to deal with the usb gadget power negotation To: Felipe Balbi , Greg KH , Sebastian Reichel , Dmitry Eremin-Solenikov , David Woodhouse Cc: 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, Baolin Wang , Linux PM list , USB , device-mainlining@lists.linuxfoundation.org, LKML Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Felipe, On 11 August 2016 at 11:14, Baolin Wang wrote: > Hi Felipe, > > On 1 August 2016 at 15:09, Baolin Wang wrote: >> Currently the Linux kernel does not provide any standard integration of this >> feature that integrates the USB subsystem with the system power regulation >> provided by PMICs meaning that either vendors must add this in their kernels >> or USB gadget devices based on Linux (such as mobile phones) may not behave >> as they should. Thus provide a standard framework for doing this in kernel. >> >> Now introduce one user with wm831x_power to support and test the usb charger, >> which is pending testing. Moreover there may be other potential users will use >> it in future. > > Could you please apply this patchset into your 'next' branch if you > have no comments about it? Thank you. Since there are no other comments about this patchset for a long time, could you please apply this patchset? Thanks. > >> >> CHanges since v15: >> - Add charger state checking to avoid sending out duplicate notifies to users. >> - Add one work to notify power users the current has been changed. >> >> Changes since v14: >> - Add kernel documentation for struct usb_cahrger. >> - Remove some redundant WARN() functions. >> >> Changes since v13: >> - Remove the charger checking in usb_gadget_vbus_draw() function. >> - Rename some functions in charger.c file. >> - Rebase on git://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git tags/usb-for-v4.8 >> >> Changes since v12: >> - Remove the class and device things. >> - Link usb charger to udc-core.ko. >> - Create one "charger" subdirectory which holds all charger-related attributes. >> >> Changes since v11: >> - Reviewed and tested by Li Jun. >> >> Changes since v10: >> - Introduce usb_charger_get_state() function to check charger state. >> - Remove the mutex lock in usb_charger_set_cur_limit_by_type() function >> in case will be issued in atomic context. >> >> Baolin Wang (4): >> usb: gadget: Introduce the usb charger framework >> usb: gadget: Support for the usb charger framework >> usb: gadget: Integrate with the usb gadget supporting for usb charger >> power: wm831x_power: Support USB charger current limit management >> >> drivers/power/wm831x_power.c | 69 ++++ >> drivers/usb/gadget/Kconfig | 8 + >> drivers/usb/gadget/udc/Makefile | 1 + >> drivers/usb/gadget/udc/charger.c | 780 ++++++++++++++++++++++++++++++++++++++ >> drivers/usb/gadget/udc/core.c | 17 + >> include/linux/mfd/wm831x/pdata.h | 3 + >> include/linux/usb/charger.h | 186 +++++++++ >> include/linux/usb/gadget.h | 3 + >> include/uapi/linux/usb/charger.h | 31 ++ >> 9 files changed, 1098 insertions(+) >> create mode 100644 drivers/usb/gadget/udc/charger.c >> create mode 100644 include/linux/usb/charger.h >> create mode 100644 include/uapi/linux/usb/charger.h >> >> -- >> 1.7.9.5 >> > > > > -- > Baolin.wang > Best Regards -- Baolin.wang Best Regards