Re: [PATCH 1/1] usb: xhci: fix possible wild pointer

2016-12-01 Thread Baolin Wang
On 2 December 2016 at 12:40, Lu Baolu <baolu...@linux.intel.com> wrote: > Hi, > > On 12/02/2016 12:18 PM, Baolin Wang wrote: >> On 2 December 2016 at 10:29, Lu Baolu <baolu...@linux.intel.com> wrote: >>> handle_cmd_completion() frees a command structure which m

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2017-01-02 Thread Baolin Wang
On 2 January 2017 at 22:57, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 27.12.2016 05:07, Baolin Wang wrote: >> >> Hi, >> >> On 21 December 2016 at 21:00, Mathias Nyman >> <mathias.ny...@linux.intel.com> wrote: >>> >>> On

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi John, On 6 January 2017 at 03:08, John Youn wrote: > On 12/28/2016 5:29 PM, John Youn wrote: >> >> >>> Janusz Dziedzic writes: >>> On some platfroms(like x86 platform), when one core is running the >>> USB gadget >>> irq thread

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-03 Thread Baolin Wang
On 28 December 2016 at 20:30, Janusz Dziedzic <janusz.dzied...@gmail.com> wrote: > 2016-12-27 13:16 GMT+01:00 Baolin Wang <baolin.w...@linaro.org>: >> Hi, >> >> On 27 December 2016 at 19:11, Felipe Balbi <ba...@kernel.org> wrote: >>> >>>

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-04 Thread Baolin Wang
On 3 January 2017 at 20:33, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >> On 28 December 2016 at 20:30, Janusz Dziedzic <janusz.dzied...@gmail.com> >> wrote: >>> 2016-12-27 13:16 GMT+

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi, On 5 January 2017 at 17:26, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >>>>>>>> On 27 December 2016 at 18:52, Janusz Dziedzic >>>>>>>> <janusz.dzied...@

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi, On 5 January 2017 at 19:19, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: > > [...] > >>>>> and you have triggered this with mailine? How? We don't write to GEVNT* >>>>> regi

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
Hi, On 20 December 2016 at 12:29, Lu Baolu <baolu...@linux.intel.com> wrote: > Hi Mathias, > > On 12/19/2016 08:13 PM, Mathias Nyman wrote: >> On 19.12.2016 13:34, Baolin Wang wrote: >>> Hi Mathias, >>> >>> On 19 December 2016 at 18:33, Mathias

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
On 20 December 2016 at 14:39, Lu Baolu <baolu...@linux.intel.com> wrote: > Hi, > > On 12/20/2016 02:06 PM, Baolin Wang wrote: >> Hi, >> >> On 20 December 2016 at 12:29, Lu Baolu <baolu...@linux.intel.com> wrote: >>> Hi Mathias, >

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2016-12-19 Thread Baolin Wang
Hi Neil, On 3 November 2016 at 09:25, NeilBrown <ne...@suse.com> wrote: > On Tue, Nov 01 2016, Baolin Wang wrote: > > >>> So I won't be responding on this topic any further until I see a genuine >>> attempt to understand and resolve the inconsistencies with >&

[PATCH v2 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
mer is pending, which means current timeout command has been handled by host and host has fetched new command and re-added the command timer, then just return and wait for new current command. If not, it means current command is timeout and need to be handled. Signed-off-by: Baolin Wang <ba

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
On 19 December 2016 at 20:13, Mathias Nyman <mathias.ny...@intel.com> wrote: > On 19.12.2016 13:34, Baolin Wang wrote: >> >> Hi Mathias, >> >> On 19 December 2016 at 18:33, Mathias Nyman >> <mathias.ny...@linux.intel.com> wrote: >>> >>

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-20 Thread Baolin Wang
Hi Mathias, On 20 December 2016 at 23:13, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 20.12.2016 09:30, Baolin Wang wrote: > ... > > Alright, I gathered all current work related to xhci races and timeouts > and put them into a branch: > > git://git.kern

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2016-12-20 Thread Baolin Wang
Hi, On 21 December 2016 at 06:07, NeilBrown <ne...@suse.com> wrote: > On Tue, Dec 20 2016, Baolin Wang wrote: > >> Hi Neil, >> >> On 3 November 2016 at 09:25, NeilBrown <ne...@suse.com> wrote: >>> On Tue, Nov 01 2016, Baolin Wang wrote: >>>

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2016-12-21 Thread Baolin Wang
On 21 December 2016 at 11:48, NeilBrown <ne...@suse.com> wrote: > On Wed, Dec 21 2016, Baolin Wang wrote: > >> Hi, >> >> On 21 December 2016 at 06:07, NeilBrown <ne...@suse.com> wrote: >>> On Tue, Dec 20 2016, Baolin Wang wrote: >>> >>

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
Hi, On 20 December 2016 at 15:18, Lu Baolu <baolu...@linux.intel.com> wrote: > Hi, > > On 12/20/2016 02:46 PM, Baolin Wang wrote: >> On 20 December 2016 at 14:39, Lu Baolu <baolu...@linux.intel.com> wrote: >>> Hi, >>> >>> On 12/20/2016

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-26 Thread Baolin Wang
Hi, On 21 December 2016 at 21:00, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 21.12.2016 04:22, Baolin Wang wrote: >> >> Hi Mathias, >> >> On 20 December 2016 at 23:13, Mathias Nyman >> <mathias.ny...@linux.intel.com> wrote: &

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Baolin Wang
Hi, On 27 December 2016 at 10:39, Lu Baolu <baolu...@linux.intel.com> wrote: > Hi, > > On 12/26/2016 04:01 PM, Baolin Wang wrote: >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhil

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Baolin Wang
Hi, On 27 December 2016 at 19:11, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >> Hi, >> >> On 27 December 2016 at 18:52, Janusz Dziedzic <janusz.dzied...@gmail.com> >> wrote: >>>

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Baolin Wang
Hi, On 27 December 2016 at 18:52, Janusz Dziedzic <janusz.dzied...@gmail.com> wrote: > 2016-12-26 9:01 GMT+01:00 Baolin Wang <baolin.w...@linaro.org>: >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_th

[PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Baolin Wang
event count in irq thread handler to make the USB function abnormal. We should add spin_lock/unlock() in dwc3_check_event_buf() to avoid this race. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/dwc3/gadget.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

Re: [PATCH v18 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2016-12-21 Thread Baolin Wang
On 22 December 2016 at 07:47, NeilBrown <ne...@suse.com> wrote: > On Wed, Dec 21 2016, Baolin Wang wrote: > >> On 21 December 2016 at 11:48, NeilBrown <ne...@suse.com> wrote: >>> On Wed, Dec 21 2016, Baolin Wang wrote: >>> >>>> Hi, >>>

Re: [PATCH 2/2] usb: host: xhci: Handle the right timeout command

2016-12-19 Thread Baolin Wang
Hi Mathias, On 19 December 2016 at 18:33, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 13.12.2016 05:21, Baolin Wang wrote: >> >> Hi Mathias, >> >> On 12 December 2016 at 23:52, Mathias Nyman >> <mathias.ny...@linux.intel.com> wrote: &

[PATCH v2 2/2] usb: phy: phy-qcom-8x16-usb: Remove redundant extcon register/unregister

2017-03-23 Thread Baolin Wang
Since usb phy core has added common code to register or unregister extcon device, then phy-qcom-8x16-usb driver does not need its own code to register/unregister extcon device, then remove them. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- Changes since v1: - No updates. --- d

[PATCH] usb: phy: Remove unused config

2017-03-23 Thread Baolin Wang
Since the old common Samsung USB PHY code has been removed by commit ea2fdf8423 ("usb: phy: samsung: remove old common USB PHY code"), thus remove the unused config. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/Kconfig |7 --- drivers/usb/phy

[PATCH v2 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-23 Thread Baolin Wang
, and some other helper functions to register extcon. Suggested-by: NeilBrown <ne...@suse.com> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- Changes since v1: - Fix build errors with random config. --- drivers/usb/phy/Kconfig |6 +++--- drivers/usb/phy/ph

Re: [PATCH v5 1/2] usb: host: plat: Enable xhci plat runtime PM

2017-03-22 Thread Baolin Wang
Hi, On 22 March 2017 at 20:43, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 22.03.2017 12:40, Baolin Wang wrote: >> >> Hi, >> >> On 22 March 2017 at 17:00, Felipe Balbi <ba...@kernel.org> wrote: >>> >>> >>> Hi, >&g

Re: [PATCH v2 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-29 Thread Baolin Wang
Hi, On 28 March 2017 at 21:40, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >> Usually usb phy need register one extcon device to get the connection >> notifications. It will remove some duplicate code if the

Re: [PATCH v2 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-29 Thread Baolin Wang
Hi, On 29 March 2017 at 06:56, NeilBrown <ne...@suse.com> wrote: > On Thu, Mar 23 2017, Baolin Wang wrote: > >> Usually usb phy need register one extcon device to get the connection >> notifications. It will remove some duplicate code if the extcon device >> i

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-29 Thread Baolin Wang
Hi, On 29 March 2017 at 07:04, NeilBrown <ne...@suse.com> wrote: > On Tue, Mar 07 2017, Baolin Wang wrote: > >> On 3 March 2017 at 10:23, NeilBrown <ne...@suse.com> wrote: >> >>> >>> I understand your reluctance to change drivers that you

[PATCH] usb: host: plat: Enable xHCI plat runtime PM

2017-03-29 Thread Baolin Wang
Enable the xHCI plat runtime PM for parent device to suspend/resume xHCI. Also call pm_runtime_forbid() in probe() function to force users to explicitly enable runtime pm using power/control in sysfs, in case some parent devices didn't implement runtime PM callbacks. Signed-off-by: Baolin Wang

Re: [PATCH v2 2/2] usb: phy: phy-qcom-8x16-usb: Remove redundant extcon register/unregister

2017-03-29 Thread Baolin Wang
Hi, On 28 March 2017 at 21:42, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >> Since usb phy core has added common code to register or unregister >> extcon device, then phy-qcom-8x16-usb driver does not need it

Re: [PATCH v5 1/2] usb: host: plat: Enable xhci plat runtime PM

2017-03-21 Thread Baolin Wang
Hi Mathias, On 21 February 2017 at 10:09, Baolin Wang <baolin.w...@linaro.org> wrote: >>>>>> Do you have any comments about this patch? Thanks. >>>>>> >>>>>>>drivers/usb/host/xhci-plat.c | 41 >>>>>>>

Re: [PATCH 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-20 Thread Baolin Wang
rove the system] > > url: > https://github.com/0day-ci/linux/commits/Baolin-Wang/usb-phy-Introduce-one-extcon-device-into-usb-phy/20170319-175509 > base: https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git next > config: i386-randconfig-i1-201712 (attached as .conf

[PATCH 2/2] usb: phy: phy-qcom-8x16-usb: Remove redundant extcon register/unregister

2017-03-17 Thread Baolin Wang
Since usb phy core has added common code to register or unregister extcon device, then phy-qcom-8x16-usb driver does not need its own code to register/unregister extcon device, then remove them. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy-qcom-8x16-usb.c

[PATCH 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-17 Thread Baolin Wang
, and some other helper functions to register extcon. Suggested-by: NeilBrown <ne...@suse.com> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/Kconfig | 17 ++--- drivers/usb/phy/phy.c | 44 inclu

Re: [PATCH v5 1/2] usb: host: plat: Enable xhci plat runtime PM

2017-03-21 Thread Baolin Wang
Hi, On 21 March 2017 at 16:07, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >>>> I don't yet understand why we can't just keep runtime pm disabled as a >>>> default for xhci platform devic

Re: [kbuild-all] [PATCH 1/2] usb: phy: Introduce one extcon device into usb phy

2017-03-22 Thread Baolin Wang
Hi, On 22 March 2017 at 14:32, Ye Xiaolong <xiaolong...@intel.com> wrote: > On 03/20, Baolin Wang wrote: >>Hi, >> >> >>On 19 March 2017 at 19:42, kbuild test robot <l...@intel.com> wrote: >>> Hi Baolin, >>> >>> [auto build test ERR

Re: [PATCH v5 1/2] usb: host: plat: Enable xhci plat runtime PM

2017-03-22 Thread Baolin Wang
Hi, On 22 March 2017 at 17:00, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >>>>>> I don't yet understand why we can't just keep runtime pm disabled as a >>>>>> default for xhci plat

Re: [PATCH] usb: host: plat: Enable xHCI plat runtime PM

2017-04-10 Thread Baolin Wang
Hi Mathias, On 30 March 2017 at 11:26, Baolin Wang <baolin.w...@linaro.org> wrote: > Enable the xHCI plat runtime PM for parent device to suspend/resume > xHCI. Also call pm_runtime_forbid() in probe() function to force users > to explicitly enable runtime pm using power/control in

Re: [PATCH] usb: host: plat: Enable xHCI plat runtime PM

2017-04-10 Thread Baolin Wang
Hi Mathias, On 10 April 2017 at 18:09, Mathias Nyman <mathias.ny...@linux.intel.com> wrote: > On 10.04.2017 10:57, Baolin Wang wrote: >> >> Hi Mathias, >> >> On 30 March 2017 at 11:26, Baolin Wang <baolin.w...@linaro.org> wrote: >>> >>> Ena

[PATCH v2] usb: host: plat: Enable xHCI plat runtime PM

2017-04-13 Thread Baolin Wang
Enable the xHCI plat runtime PM for parent device to suspend/resume xHCI. Also call pm_runtime_forbid() in probe() function to force users to explicitly enable runtime pm using power/control in sysfs, in case some parent devices didn't implement runtime PM callbacks. Signed-off-by: Baolin Wang

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-09 Thread Baolin Wang
On 10 March 2017 at 14:30, Jun Li wrote: >> >> > >> >> > Will generic phy need add extcon as well? >> >> >> >> Yes, will add a 'struct extcon_dev*' in 'struct usb_phy', which will >> >> be common code. >> >> >> > >> > I mean the common code need add 'struct extcon_dev' into both

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-10 Thread Baolin Wang
On 10 March 2017 at 16:27, Jun Li <jun...@nxp.com> wrote: > Hi > >> -Original Message----- >> From: Baolin Wang [mailto:baolin.w...@linaro.org] >> Sent: Friday, March 10, 2017 3:15 PM >> To: Jun Li <jun...@nxp.com> >> Cc: NeilBrown <ne...@

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-08 Thread Baolin Wang
Hi, On 9 March 2017 at 09:50, Jun Li <jun...@nxp.com> wrote: > Hi, > >> -Original Message----- >> From: Baolin Wang [mailto:baolin.w...@linaro.org] >> Sent: Tuesday, March 07, 2017 5:39 PM >> To: NeilBrown <ne...@suse.com> >> C

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-09 Thread Baolin Wang
On 9 March 2017 at 18:34, Jun Li <jun...@nxp.com> wrote: > > >> -Original Message----- >> From: Baolin Wang [mailto:baolin.w...@linaro.org] >> Sent: Thursday, March 09, 2017 2:11 PM >> To: Jun Li <jun...@nxp.com> >> Cc: NeilBrown <ne...@

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-14 Thread Baolin Wang
Hi, On 14 March 2017 at 17:57, Lee Jones <lee.jo...@linaro.org> wrote: > On Mon, 20 Feb 2017, Baolin Wang wrote: > > [...] > >> drivers/power/supply/wm831x_power.c | 63 +++ >> drivers/usb/gadget/Kconfig |8 + >> drivers/usb/gadget/udc/Makefile

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-13 Thread Baolin Wang
On 13 March 2017 at 09:09, Jun Li <jun...@nxp.com> wrote: > Hi, > >> -Original Message----- >> From: Baolin Wang [mailto:baolin.w...@linaro.org] >> Sent: Friday, March 10, 2017 6:52 PM >> To: Jun Li <jun...@nxp.com> >> Cc: NeilBrown <ne...@

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-13 Thread Baolin Wang
On 14 March 2017 at 09:10, Jun Li <jun...@nxp.com> wrote: > Hi, > >> -Original Message----- >> From: Baolin Wang [mailto:baolin.w...@linaro.org] >> Sent: Monday, March 13, 2017 4:44 PM >> To: Jun Li <jun...@nxp.com> >> Cc: NeilBrown <ne...@

Re: [PATCH v19 0/4] Introduce usb charger framework to deal with the usb gadget power negotation

2017-03-07 Thread Baolin Wang
On 3 March 2017 at 10:23, NeilBrown <ne...@suse.com> wrote: > On Mon, Feb 20 2017, 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 >

Re: [PATCH v4 3/3] power: wm831x_power: Support USB charger current limit management

2017-08-03 Thread Baolin Wang
On 4 August 2017 at 03:16, Rob Herring <r...@kernel.org> wrote: > On Thu, Jul 27, 2017 at 01:14:38PM +0800, Baolin Wang wrote: >> Integrate with the newly added USB charger interface to limit the current >> we draw from the USB input based on the input device configuration >

[PATCH v5 0/3] Introduce USB charger support in USB phy

2017-08-15 Thread Baolin Wang
DT binding documentation for wm831x_power driver. - Change 'usb-phy' as one optional property for wm831x_power driver. Changes since v1: - Fix building errors. Baolin Wang (3): include: uapi: usb: Introduce USB charger type and state definition usb: phy: Add USB charger support power

[PATCH v5 2/3] usb: phy: Add USB charger support

2017-08-15 Thread Baolin Wang
-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy.c | 272 +++ include/linux/usb/phy.h | 49 + 2 files changed, 321 insertions(+) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index 032f5af..2dc48bb 100644 --- a/d

Re: [PATCH v4 0/3] Introduce USB charger support in USB phy

2017-08-15 Thread Baolin Wang
Hi Felipe, On 15 August 2017 at 17:53, Felipe Balbi <ba...@kernel.org> wrote: > > Hi, > > Baolin Wang <baolin.w...@linaro.org> writes: >>> Currently the Linux kernel does not provide any standard integration of this >>> feature that integrates the USB

[PATCH v5 3/3] power: wm831x_power: Support USB charger current limit management

2017-08-15 Thread Baolin Wang
-by: Mark Brown <broo...@kernel.org> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> Acked-by: Lee Jones <lee.jo...@linaro.org> Acked-by: Charles Keepax <ckee...@opensource.wolfsonmicro.com> Acked-by: Sebastian Reichel <sebastian.reic...@collabora.co.uk> --- Doc

[PATCH v5 1/3] include: uapi: usb: Introduce USB charger type and state definition

2017-08-15 Thread Baolin Wang
Introducing USB charger type and state definition can help to support USB charging which will be added in USB phy core. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- include/uapi/linux/usb/charger.h | 31 +++ 1 file changed, 31 insertions(+) creat

Re: [PATCH] usb: dwc3: Fix the USB 3.0 hub detection bug after warm boot

2017-07-12 Thread Baolin Wang
Hi, On 12 July 2017 at 11:52, gustavo panizzo wrote: > The dwc3 could not release resources when the module is built-in > because this module does not have shutdown method. This causes the USB > 3.0 hub is not able to detect after warm boot. > > Original patch by Brian Kim,

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-12 Thread Baolin Wang
Hi, On 12 July 2017 at 15:25, Manu Gautam <mgau...@codeaurora.org> wrote: > > > On 7/12/2017 12:19 PM, Baolin Wang wrote: >> >> Hi, >> >> On 3 July 2017 at 19:25, Manu Gautam <mgau...@codeaurora.org> wrote: >>> >>> Driver powers-off P

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-12 Thread Baolin Wang
Hi, On 3 July 2017 at 19:25, Manu Gautam wrote: > Driver powers-off PHYs and reinitializes DWC3 core and gadget on > resume. While this works fine for gadget mode but in host > mode there is not re-initialization of host stack. Also, resetting > bus as part of

Re: [PATCH] usb: dwc3: Fix the USB 3.0 hub detection bug after warm boot

2017-07-12 Thread Baolin Wang
Hi, On 13 July 2017 at 07:20, gustavo panizzo <g...@zumbi.com.ar> wrote: > Hello Wang > > thanks for your response > > > On Wed, Jul 12, 2017 at 02:08:04PM +0800, Baolin Wang wrote: >> >> Hi, >> >> On 12 July 2017 at 11:52, gustavo panizzo &

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-13 Thread Baolin Wang
Hi, On 13 July 2017 at 15:26, Manu Gautam <mgau...@codeaurora.org> wrote: > > > On 7/13/2017 11:33 AM, Baolin Wang wrote: >> On 12 July 2017 at 16:58, Manu Gautam <mgau...@codeaurora.org> wrote: >>> >>> On 7/12/2017 2:06 PM, Baolin Wang wrote: >

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-13 Thread Baolin Wang
On 12 July 2017 at 16:58, Manu Gautam <mgau...@codeaurora.org> wrote: > > > On 7/12/2017 2:06 PM, Baolin Wang wrote: >> >> Hi, >> >> On 12 July 2017 at 15:25, Manu Gautam <mgau...@codeaurora.org> wrote: >>> >>> >>> On 7/12/

Re: [PATCH v2 0/3] Introduce USB charger support in USB phy

2017-07-18 Thread Baolin Wang
Hi, On 3 July 2017 at 14:07, Baolin Wang <baolin.w...@linaro.org> 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 v

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-20 Thread Baolin Wang
Hi Manu, On 20 July 2017 at 19:12, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi, > > > On 7/13/2017 1:13 PM, Baolin Wang wrote: >> Hi, >> >> On 13 July 2017 at 15:26, Manu Gautam <mgau...@codeaurora.org> wrote: >>> >>> On 7/13/201

[PATCH] usb: dwc3: Support the dwc3 host suspend/resume

2017-07-21 Thread Baolin Wang
the child device (xHCI device) in runtime resume callback (dwc3_host_resume()) of dwc3 device, now the dwc3 device's runtime state is not RPM_ACTIVE, which will block to resume its child device (xHCI device). Add ignore children flag can avoid this situation. Signed-off-by: Baolin Wang <baoli

Re: [PATCH 1/2] usb: dwc3: Don't reinitialize core during host bus-suspend/resume

2017-07-21 Thread Baolin Wang
Hi Manu, On 21 July 2017 at 10:32, Baolin Wang <baolin.w...@linaro.org> wrote: > Hi Manu, > > On 20 July 2017 at 19:12, Manu Gautam <mgau...@codeaurora.org> wrote: >> Hi, >> >> >> On 7/13/2017 1:13 PM, Baolin Wang wrote: >>> H

Re: [PATCH] usb: dwc3: Support the dwc3 host suspend/resume

2017-07-21 Thread Baolin Wang
On 21 July 2017 at 16:45, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi, > > > On 7/21/2017 12:28 PM, Baolin Wang wrote: >> For some mobile devices with strict power management, we also want to >> suspend the host when the slave was detached for power saving. T

[PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-25 Thread Baolin Wang
-by: Mark Brown <broo...@kernel.org> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- Documentation/devicetree/bindings/mfd/wm831x.txt |1 + drivers/power/supply/wm831x_power.c | 58 ++ 2 files changed, 59 insertions(+) diff --git a/D

[PATCH v3 2/3] usb: phy: Add USB charger support

2017-07-25 Thread Baolin Wang
-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy.c | 272 +++ include/linux/usb/phy.h | 49 + 2 files changed, 321 insertions(+) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index 032f5af..2dc48bb 100644 --- a/d

[PATCH v3 1/3] include: uapi: usb: Introduce USB charger type and state definition

2017-07-25 Thread Baolin Wang
Introducing USB charger type and state definition can help to support USB charging which will be added in USB phy core. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- include/uapi/linux/usb/charger.h | 31 +++ 1 file changed, 31 insertions(+) creat

[PATCH v3 0/3] Introduce USB charger support in USB phy

2017-07-25 Thread Baolin Wang
/bq24190_charger.c drivers/power/supply/charger-manager.c drivers/power/supply/qcom_smbb.c Changes since v1: - Fix building errors. Changes since v2: - Add DT binding documentation for wm831x_power driver. - Change 'usb-phy' as one optional property for wm831x_power driver. Baolin Wang (3

Re: [PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-25 Thread Baolin Wang
Hi, On 25 July 2017 at 17:59, Sebastian Reichel <sebastian.reic...@collabora.co.uk> wrote: > Hi, > > On Tue, Jul 25, 2017 at 04:00:01PM +0800, Baolin Wang wrote: >> Integrate with the newly added USB charger interface to limit the current >> we draw from the USB inpu

Re: [PATCH v3 2/3] usb: phy: Add USB charger support

2017-07-25 Thread Baolin Wang
Hi Manu, On 25 July 2017 at 19:19, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi, > > > On 7/25/2017 1:30 PM, Baolin Wang wrote: >> This patch introduces the usb charger support based on usb phy that >> makes an enhancement to a power driver. The basic co

Re: [PATCH] usb: dwc3: Support the dwc3 host suspend/resume

2017-07-23 Thread Baolin Wang
Hi, On 21 July 2017 at 17:13, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi, > > On 7/21/2017 2:31 PM, Baolin Wang wrote: >> On 21 July 2017 at 16:45, Manu Gautam <mgau...@codeaurora.org> wrote: >>> Hi, >>> >>> >>> On 7/21

[PATCH v4 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-26 Thread Baolin Wang
-by: Mark Brown <broo...@kernel.org> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> Acked-by: Lee Jones <lee.jo...@linaro.org> Acked-by: Charles Keepax <ckee...@opensource.wolfsonmicro.com> --- Documentation/devicetree/bindings/mfd/wm831x.txt |1 + drivers/pow

[PATCH v4 2/3] usb: phy: Add USB charger support

2017-07-26 Thread Baolin Wang
-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy.c | 272 +++ include/linux/usb/phy.h | 49 + 2 files changed, 321 insertions(+) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index 032f5af..2dc48bb 100644 --- a/d

[PATCH v4 0/3] Introduce USB charger support in USB phy

2017-07-26 Thread Baolin Wang
for wm831x_power driver. Changes since v1: - Fix building errors. Baolin Wang (3): include: uapi: usb: Introduce USB charger type and state definition usb: phy: Add USB charger support power: wm831x_power: Support USB charger current limit management Documentation/devicetree/bindings/mfd

[PATCH v4 1/3] include: uapi: usb: Introduce USB charger type and state definition

2017-07-26 Thread Baolin Wang
Introducing USB charger type and state definition can help to support USB charging which will be added in USB phy core. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- include/uapi/linux/usb/charger.h | 31 +++ 1 file changed, 31 insertions(+) creat

Re: [PATCH v3 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-26 Thread Baolin Wang
Hi, On 26 July 2017 at 20:08, Sebastian Reichel <sebastian.reic...@collabora.co.uk> wrote: > Hi, > > On Wed, Jul 26, 2017 at 11:05:25AM +0800, Baolin Wang wrote: >> On 25 July 2017 at 17:59, Sebastian Reichel >> <sebastian.reic...@collabora.co.uk> wrote: >&g

Re: [PATCH] usb: dwc3: Support the dwc3 host suspend/resume

2017-07-24 Thread Baolin Wang
Hi Manu, On 24 July 2017 at 16:42, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi Baolin, > > > On 7/24/2017 11:26 AM, Baolin Wang wrote: > >>>>> Other than what I pushed in my patch - >>>>> ("usb: dwc3: Don't reinitialize cor

Re: [PATCH] usb: dwc3: Support the dwc3 host suspend/resume

2017-07-24 Thread Baolin Wang
Hi Manu, On 24 July 2017 at 17:38, Manu Gautam <mgau...@codeaurora.org> wrote: > Hi, > > > On 7/24/2017 2:45 PM, Baolin Wang wrote: >> Hi Manu, >> >> On 24 July 2017 at 16:42, Manu Gautam <mgau...@codeaurora.org> wrote: >>> Hi Baolin, &

Re: [PATCH v2 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-04 Thread Baolin Wang
Hi, On 3 July 2017 at 16:53, Charles Keepax <ckee...@opensource.cirrus.com> wrote: > On Mon, Jul 03, 2017 at 02:07:15PM +0800, Baolin Wang wrote: >> Integrate with the newly added USB charger interface to limit the current >> we draw from the USB input based on the input

Re: [PATCH v2 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-04 Thread Baolin Wang
Hi, On 3 July 2017 at 23:50, Sebastian Reichel <sebastian.reic...@collabora.co.uk> wrote: > Hi, > > On Mon, Jul 03, 2017 at 02:07:15PM +0800, Baolin Wang wrote: >> Integrate with the newly added USB charger interface to limit the current >> we draw from the USB inpu

[PATCH v2 3/3] power: wm831x_power: Support USB charger current limit management

2017-07-03 Thread Baolin Wang
-by: Mark Brown <broo...@kernel.org> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/power/supply/wm831x_power.c | 61 +++ 1 file changed, 61 insertions(+) diff --git a/drivers/power/supply/wm831x_power.c b/drivers/power/supply/wm

[PATCH v2 1/3] include: uapi: usb: Introduce USB charger type and state definition

2017-07-03 Thread Baolin Wang
Introducing USB charger type and state definition can help to support USB charging which will be added in USB phy core. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- include/uapi/linux/usb/charger.h | 31 +++ 1 file changed, 31 insertions(+) creat

[PATCH v2 0/3] Introduce USB charger support in USB phy

2017-07-03 Thread Baolin Wang
not behave as they should. Thus provide a standard USB charger support in USB phy core for doing this in kernel. Now introduce one user with wm831x_power to support and test the usb charger. Changes since v1: - Fix building errors. Baolin Wang (3): include: uapi: usb: Introduce USB charger type

[PATCH v2 2/3] usb: phy: Add USB charger support

2017-07-03 Thread Baolin Wang
-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy.c | 265 +++ include/linux/usb/phy.h | 49 + 2 files changed, 314 insertions(+) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index 032f5af..db72802 100644 --- a/d

Re: Crash in gs_write_room() in kernel 4.4, and your gs_start_tx() patch

2017-06-29 Thread Baolin Wang
On 29 June 2017 at 07:01, Stephen Warren wrote: > > > Baolin, > > I have a question about the following commit: > > 511a36d2f357 "usb: gadget: Add the gserial port checking in gs_start_tx()" > >> When usb gadget is set gadget serial function, it will be crash in below >>

Re: [PATCH] usb: gadget: gserial: check if console kthread exists

2017-04-28 Thread Baolin Wang
Hi, On 28 April 2017 at 21:26, Bogdan Mirea wrote: > In the current implementation we get a Kernel Oops when calling rmmod > for g_serial driver. This happens because the function that creates the > kthread (gs_console_setup) which should be called at driver

Re: [PATCH] usb: gadget: gserial: check if console kthread exists

2017-04-29 Thread Baolin Wang
Hi Mirea, On 29 April 2017 at 17:58, Mirea, Bogdan-Stefan <bogdan-stefan_mi...@mentor.com> wrote: > Hi Baolin, > > On April 29, 2017 at 5:14, Baolin Wang <baolin.w...@linaro.org> wrote: >> Hi, >> >> On 28 April 2017 at 21:26, Bogdan Mirea <

Re: [PATCH v4 0/3] Introduce USB charger support in USB phy

2017-08-08 Thread Baolin Wang
Hi Felipe, On 27 July 2017 at 13:14, Baolin Wang <baolin.w...@linaro.org> 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 v

[PATCH 3/3] power: wm831x_power: Support USB charger current limit management

2017-06-27 Thread Baolin Wang
-by: Mark Brown <broo...@kernel.org> Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/power/supply/wm831x_power.c | 61 +++ 1 file changed, 61 insertions(+) diff --git a/drivers/power/supply/wm831x_power.c b/drivers/power/supply/wm

[PATCH 2/3] usb: phy: Add USB charger support

2017-06-27 Thread Baolin Wang
-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy.c | 265 +++ include/linux/usb/phy.h | 48 + 2 files changed, 313 insertions(+) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index 032f5af..db72802 100644 --- a/d

[PATCH 1/3] include: uapi: usb: Introduce USB charger type and state definition

2017-06-27 Thread Baolin Wang
Introducing USB charger type and state definition can help to support USB charging which will be added in USB phy core. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- include/uapi/linux/usb/charger.h | 31 +++ 1 file changed, 31 insertions(+) creat

[PATCH v3 1/3] usb: phy: Introduce one extcon device into usb phy

2017-05-05 Thread Baolin Wang
, and some other helper functions to register extcon. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/Kconfig |6 ++--- drivers/usb/phy/phy.c | 57 +++ include/linux/usb/phy.h |7 ++ 3 files changed, 67 inse

[PATCH v3 3/3] usb: phy: phy-msm-usb: Remove redundant extcon register/unregister

2017-05-05 Thread Baolin Wang
Since usb phy core has added common code to register or unregister extcon device, then phy-msm-usb driver does not need its own code to register/unregister extcon device, then remove them. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy-msm-usb.c

[PATCH v3 0/3] Introduce one extcon device into usb phy

2017-05-05 Thread Baolin Wang
: - Fix build errors with random config. Baolin Wang (3): usb: phy: Introduce one extcon device into usb phy usb: phy: phy-qcom-8x16-usb: Remove redundant extcon register/unregister usb: phy: phy-msm-usb: Remove redundant extcon register/unregister drivers/usb/phy/Kconfig

[PATCH v3 2/3] usb: phy: phy-qcom-8x16-usb: Remove redundant extcon register/unregister

2017-05-05 Thread Baolin Wang
Since usb phy core has added common code to register or unregister extcon device, then phy-qcom-8x16-usb driver does not need its own code to register/unregister extcon device, then remove them. Signed-off-by: Baolin Wang <baolin.w...@linaro.org> --- drivers/usb/phy/phy-qcom-8x16-usb.c

Re: [PATCH 2/3] usb: phy: Add USB charger support

2017-06-27 Thread Baolin Wang
note to > help improve the system] > > url: > https://github.com/0day-ci/linux/commits/Baolin-Wang/include-uapi-usb-Introduce-USB-charger-type-and-state-definition/20170628-093530 > base: https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git next > config: i386-ran

Re: [bug report] usb: phy: Add USB charger support

2017-08-27 Thread Baolin Wang
Hi Dan, On 25 August 2017 at 20:20, Dan Carpenter <dan.carpen...@oracle.com> wrote: > Hello Baolin Wang, > > This is a semi-automatic email about new static checker warnings. > > The patch a9081a008f84: "usb: phy: Add USB charger support" from Aug > 15, 2

<    1   2   3   4   5   6   >