Re: [PATCH v2 00/32] usb: dwc2: various bug fixes

2015-09-24 Thread John Youn 
On 9/22/2015 6:15 AM, Mian Yousaf Kaukab wrote:
> Hi,
> This series consists of various bug fixes for both host and gadget
> sides. All patches are verified on dwc2 v3.0a with dedicated fifos.
> It would be good to get some Tested-bys for other platforms.
> 
> It is based on testing/next branch in Felipe's git.
> 
> Thank you,
> 
> Best regards,
> Yousaf
> 
> History:
> v2:
>  - Rebase to latest testing/next
>  - Fix lock issue found by John Youn when calling
>dwc2_hsotg_core_init_disconnected() from dwc2_conn_id_status_change()
>  - Fix comments from John Youn
>  - Fix comments from Doug Anderson
> 
> v1:
>  - Fix following comments from David Cohen (received on an internal
>list):
> - Fix build when "usb: dwc2: host: create a function to handle-
>   port_resume" is applied.
> - Take spin locks within if statements in "usb: dwc2: host: enter-
>   hibernation during bus suspend"
> - Remove extra 100us delay in "usb: dwc2: host: enter-
>   hibernation during bus suspend"
> - Fix spelling mistakes in "usb: dwc2: host: update hcd and-
>   lx_state during start/stop callbacks"
> - Change dev_warn to dev_dbg in "usb: dwc2: host: reset frame-
>number after suspend"
> - Change mdelay to usleep_range in "usb: dwc2: host: wait 3ms for-
>   controller stabilization"
>  - Fix comments from Sergei Shtylyov
> 
> Gregory Herrero (22):
>   usb: dwc2: host: don't clear hprt0 status bits when exiting
> hibernation
>   usb: dwc2: host: create a function to handle port_resume
>   usb: dwc2: host: add flag to reflect bus state
>   usb: dwc2: host: enter hibernation during bus suspend
>   usb: dwc2: host: update hcd and lx_state during start/stop callbacks
>   usb: dwc2: host: avoid resetting lx_state to L3 during disconnect
>   usb: dwc2: host: ignore wakeup interrupt if hibernation supported
>   usb: dwc2: host: resume only if bus is suspended
>   usb: dwc2: host: reset frame number after suspend
>   usb: dwc2: host: disconnect hcd prior stopping it
>   usb: dwc2: host: disable interrupt during stop
>   usb: dwc2: host: clear pending interrupts prior hibernation
>   usb: dwc2: host: wait 3ms for controller stabilization
>   usb: dwc2: host: correctly dump urb isochronous descriptors
>   usb: dwc2: host: use correct frame number during qh init
>   usb: dwc2: host: kill remaining urbs using -ECONNRESET status
>   usb: dwc2: gadget: ensure lx_state corresponds to current state
>   usb: dwc2: gadget: don't modify pullup state in host mode
>   usb: dwc2: gadget: set op_state in vbus_session call
>   usb: dwc2: gadget: abort core init if core_reset fails
>   usb: dwc2: gadget: unmask idstschng interrupt only if controller
> supports it
>   usb: dwc2: gadget: exit hibernation before power down
> 
> Mian Yousaf Kaukab (10):
>   usb: dwc2: host: add disconnect interrupt to host only interrupts
>   usb: dwc2: gadget: initialize op_state for peripheral only
> configuration
>   usb: dwc2: force dr_mode in case of configuration mismatch
>   usb: dwc2: gadget: ignore stall check for ep0
>   usb: dwc2: gadget: print complete setup packet
>   usb: dwc2: gadget: stop current transfer on dequeue
>   usb: dwc2: gadget: kill ep0 requests before reinitializing core
>   usb: dwc2: gadget: only reset core after addressed state
>   usb: dwc2: gadget: handle reset interrupt before endpoint interrupts
>   usb: dwc2: exit hibernation on session request
> 
>  drivers/usb/dwc2/core.c  |  14 ++-
>  drivers/usb/dwc2/core.h  |   3 +-
>  drivers/usb/dwc2/core_intr.c |  29 --
>  drivers/usb/dwc2/gadget.c| 210 
> ++-
>  drivers/usb/dwc2/hcd.c   | 209 --
>  drivers/usb/dwc2/hcd_queue.c |  11 +++
>  drivers/usb/dwc2/platform.c  |  11 +++
>  7 files changed, 386 insertions(+), 101 deletions(-)
> 


For this series:

Tested-by: John Youn 
Acked-by: John Youn 

Regards,
John


--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 00/32] usb: dwc2: various bug fixes

2015-09-22 Thread Dinh Nguyen 
On 09/22/2015 08:16 AM, Mian Yousaf Kaukab wrote:
> Hi,
> This series consists of various bug fixes for both host and gadget
> sides. All patches are verified on dwc2 v3.0a with dedicated fifos.
> It would be good to get some Tested-bys for other platforms.
> 
> It is based on testing/next branch in Felipe's git.
> 
> Thank you,
> 

I tested this series on 2.93a and 3.20a versions of the IP and did not
see any issues.

Tested-by: Dinh Nguyen 

Thanks,
Dinh
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH v2 00/32] usb: dwc2: various bug fixes

2015-09-22 Thread Mian Yousaf Kaukab 
Hi,
This series consists of various bug fixes for both host and gadget
sides. All patches are verified on dwc2 v3.0a with dedicated fifos.
It would be good to get some Tested-bys for other platforms.

It is based on testing/next branch in Felipe's git.

Thank you,

Best regards,
Yousaf

History:
v2:
 - Rebase to latest testing/next
 - Fix lock issue found by John Youn when calling
   dwc2_hsotg_core_init_disconnected() from dwc2_conn_id_status_change()
 - Fix comments from John Youn
 - Fix comments from Doug Anderson

v1:
 - Fix following comments from David Cohen (received on an internal
   list):
- Fix build when "usb: dwc2: host: create a function to handle-
  port_resume" is applied.
- Take spin locks within if statements in "usb: dwc2: host: enter-
  hibernation during bus suspend"
- Remove extra 100us delay in "usb: dwc2: host: enter-
  hibernation during bus suspend"
- Fix spelling mistakes in "usb: dwc2: host: update hcd and-
  lx_state during start/stop callbacks"
- Change dev_warn to dev_dbg in "usb: dwc2: host: reset frame-
   number after suspend"
- Change mdelay to usleep_range in "usb: dwc2: host: wait 3ms for-
  controller stabilization"
 - Fix comments from Sergei Shtylyov

Gregory Herrero (22):
  usb: dwc2: host: don't clear hprt0 status bits when exiting
hibernation
  usb: dwc2: host: create a function to handle port_resume
  usb: dwc2: host: add flag to reflect bus state
  usb: dwc2: host: enter hibernation during bus suspend
  usb: dwc2: host: update hcd and lx_state during start/stop callbacks
  usb: dwc2: host: avoid resetting lx_state to L3 during disconnect
  usb: dwc2: host: ignore wakeup interrupt if hibernation supported
  usb: dwc2: host: resume only if bus is suspended
  usb: dwc2: host: reset frame number after suspend
  usb: dwc2: host: disconnect hcd prior stopping it
  usb: dwc2: host: disable interrupt during stop
  usb: dwc2: host: clear pending interrupts prior hibernation
  usb: dwc2: host: wait 3ms for controller stabilization
  usb: dwc2: host: correctly dump urb isochronous descriptors
  usb: dwc2: host: use correct frame number during qh init
  usb: dwc2: host: kill remaining urbs using -ECONNRESET status
  usb: dwc2: gadget: ensure lx_state corresponds to current state
  usb: dwc2: gadget: don't modify pullup state in host mode
  usb: dwc2: gadget: set op_state in vbus_session call
  usb: dwc2: gadget: abort core init if core_reset fails
  usb: dwc2: gadget: unmask idstschng interrupt only if controller
supports it
  usb: dwc2: gadget: exit hibernation before power down

Mian Yousaf Kaukab (10):
  usb: dwc2: host: add disconnect interrupt to host only interrupts
  usb: dwc2: gadget: initialize op_state for peripheral only
configuration
  usb: dwc2: force dr_mode in case of configuration mismatch
  usb: dwc2: gadget: ignore stall check for ep0
  usb: dwc2: gadget: print complete setup packet
  usb: dwc2: gadget: stop current transfer on dequeue
  usb: dwc2: gadget: kill ep0 requests before reinitializing core
  usb: dwc2: gadget: only reset core after addressed state
  usb: dwc2: gadget: handle reset interrupt before endpoint interrupts
  usb: dwc2: exit hibernation on session request

 drivers/usb/dwc2/core.c  |  14 ++-
 drivers/usb/dwc2/core.h  |   3 +-
 drivers/usb/dwc2/core_intr.c |  29 --
 drivers/usb/dwc2/gadget.c| 210 ++-
 drivers/usb/dwc2/hcd.c   | 209 --
 drivers/usb/dwc2/hcd_queue.c |  11 +++
 drivers/usb/dwc2/platform.c  |  11 +++
 7 files changed, 386 insertions(+), 101 deletions(-)

-- 
2.3.3

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html