RE: [RESEND PATCH v11 1/4] gadget: Introduce the usb charger framework

From: Jun Li
Date: Mon Jun 13 2016 - 11:28:25 EST


Hi

> -----Original Message-----
> From: linux-usb-owner@xxxxxxxxxxxxxxx [mailto:linux-usb-
> owner@xxxxxxxxxxxxxxx] On Behalf Of Baolin Wang
> Sent: Monday, June 13, 2016 4:47 PM
> To: balbi@xxxxxxxxxx; gregkh@xxxxxxxxxxxxxxxxxxx; sre@xxxxxxxxxx;
> dbaryshkov@xxxxxxxxx; dwmw2@xxxxxxxxxxxxx
> Cc: robh@xxxxxxxxxx; m.szyprowski@xxxxxxxxxxx; ruslan.bilovol@xxxxxxxxx;
> peter.chen@xxxxxxxxxxxxx; stern@xxxxxxxxxxxxxxxxxxx; r.baldyga@xxxxxxxxxxx;
> yoshihiro.shimoda.uh@xxxxxxxxxxx; lee.jones@xxxxxxxxxx; broonie@xxxxxxxxxx;
> ckeepax@xxxxxxxxxxxxxxxxxxxxxxxxxxx; patches@xxxxxxxxxxxxxxxxxxxxxxxxxxx;
> baolin.wang@xxxxxxxxxx; linux-pm@xxxxxxxxxxxxxxx; linux-
> usb@xxxxxxxxxxxxxxx; device-mainlining@xxxxxxxxxxxxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx
> Subject: [RESEND PATCH v11 1/4] gadget: Introduce the usb charger
> framework
>
> This patch introduces the usb charger driver based on usb gadget that
> makes an enhancement to a power driver. It works well in practice but that
> requires a system with suitable hardware.
>
> The basic conception of the usb charger is that, when one usb charger is
> added or removed by reporting from the usb gadget state change or the
> extcon device state change, the usb charger will report to power user to
> set the current limitation.
>
> The usb charger will register notifiees on the usb gadget or the extcon
> device to get notified the usb charger state. It also supplies the
> notification mechanism to userspace When the usb charger state is changed.
>
> Power user will register a notifiee on the usb charger to get notified by
> status changes from the usb charger. It will report to power user to set
> the current limitation when detecting the usb charger is added or removed
> from extcon device state or usb gadget state.
>
> This patch doesn't yet integrate with the gadget code, so some functions
> which rely on the 'gadget' are not completed, that will be implemented in
> the following patches.
>
> Signed-off-by: Baolin Wang <baolin.wang@xxxxxxxxxx>

Reviewed-by: Li Jun <jun.li@xxxxxxx>
Tested-by: Li Jun <jun.li@xxxxxxx>

> ---
> drivers/usb/gadget/Kconfig | 7 +
> drivers/usb/gadget/udc/Makefile | 1 +
> drivers/usb/gadget/udc/charger.c | 770
> ++++++++++++++++++++++++++++++++++++++
> include/linux/usb/charger.h | 191 ++++++++++
> include/uapi/linux/usb/charger.h | 31 ++
> 5 files changed, 1000 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