Re: [PATCH 00/17] usb: dwc2: improvements

2018-02-05 Thread John Youn 

On 01/19/2018 02:39 AM, Grigor Tovmasyan wrote:

This series contains patches which are already have been sent in
"usb: dwc2: fixes, enhancements and new features" series.

That patch series was too large, and based on community feedbacks decided to
split that series into small pieces. This is a second part.

In this series we included only improvements for dwc2 driver, such as moving
functions or deleting unused parts of code.

Patches 2-3 and 14-17 are logically linked.

All patches were tested on HAPS-DX7 platform.


Changes after first submission:

usb: dwc2: Set AHB burst size to INCR
 
 Because of containing two non-related modifications this patch divided into two patches:

 - "usb: dwc2: Set AHB burst size to INCR"
 - "usb: dwc2: Use AHB burst size parameter"

"usb: dwc2: pci: Fix error handling in dwc2_pci_probe"
 
 Divided into 4 patches:

 - "usb: dwc2: pci: Handle error cleanup in probe"
 - "usb: dwc2: pci: Move devm_kzalloc() before platform_device_add()"
 - "usb: dwc2: pci: Move usb_phy_generic_register()"
 - "usb: dwc2: pci: Replace kzalloc() with devm_kzalloc()"

"usb: dwc2: Update bit polling functionality":
 
 - This patch is a merging of "usb: dwc2: Move polling function to core.c."

   and "usb: dwc2: Use common polling function.".
 - Added dwc2_hsotg_wait_bit_clear() function, because in some cases,
   we should wait for bit would be cleared not set.
  


usb: dwc2: host: Fix transaction errors in host mode

 - Renamed patch name from "usb: dwc2: host: Setting TOUTCAL and USBTRDTIM 
fields in host mode".
 - Removed programming of USBTRDTIM bitfiled, because this field available
   only in Device mode.
 - Added comment with description of TOUTCAL bitfiled in comment.
 - In description added platforms for which this patch fixes transactions 
error.


usb: dwc2: Change TxFIFO and RxFIFO flushing flow

 Rewritten loop for waiting AHB master in IDLE state.





Grigor Tovmasyan (2):
   usb: dwc2: Delete unused functionality
   usb: dwc2: Make dwc2_force_mode() static

Minas Harutyunyan (3):
   usb: dwc2: hcd: Fix host channel halt flow
   usb: dwc2: host: Fix transaction errors in host mode
   usb: dwc2: Change TxFIFO and RxFIFO flushing flow

Razmik Karapetyan (7):
   usb: dwc2: Use AHB burst size parameter
   usb: dwc2: Set AHB burst size to INCR
   usb: dwc2: Update dwc2_handle_incomplete_isoc_in() function
   usb: dwc2: Update dwc2_handle_incomplete_isoc_out() function
   usb: dwc2: Update GINTSTS_GOUTNAKEFF interrupt handling
   usb: dwc2: Rename bit set/clear function names
   usb: dwc2: Remove unnecessary debug prints

Sevak Arakelyan (1):
   usb: dwc2: Update bit polling functionality

Vardan Mikayelyan (4):
   usb: dwc2: pci: Replace kzalloc() with devm_kzalloc()
   usb: dwc2: pci: Move usb_phy_generic_register()
   usb: dwc2: pci: Move devm_kzalloc() before platform_device_add()
   usb: dwc2: pci: Handle error cleanup in probe

  drivers/usb/dwc2/core.c   | 120 +---
  drivers/usb/dwc2/core.h   |  25 ++---
  drivers/usb/dwc2/gadget.c | 116 +++
  drivers/usb/dwc2/hcd.c| 136 +-
  drivers/usb/dwc2/hcd.h|  56 ---
  drivers/usb/dwc2/params.c |   2 +-
  drivers/usb/dwc2/pci.c|  27 +
  7 files changed, 189 insertions(+), 293 deletions(-)



For this series:

Acked-by: John Youn 

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

[PATCH 00/17] usb: dwc2: improvements

2018-01-19 Thread Grigor Tovmasyan 
This series contains patches which are already have been sent in
"usb: dwc2: fixes, enhancements and new features" series.

That patch series was too large, and based on community feedbacks decided to 
split that series into small pieces. This is a second part.

In this series we included only improvements for dwc2 driver, such as moving
functions or deleting unused parts of code. 

Patches 2-3 and 14-17 are logically linked.  

All patches were tested on HAPS-DX7 platform.


Changes after first submission:

usb: dwc2: Set AHB burst size to INCR

Because of containing two non-related modifications this patch divided into 
two patches:
- "usb: dwc2: Set AHB burst size to INCR"
- "usb: dwc2: Use AHB burst size parameter"

"usb: dwc2: pci: Fix error handling in dwc2_pci_probe" 

Divided into 4 patches:
- "usb: dwc2: pci: Handle error cleanup in probe"
- "usb: dwc2: pci: Move devm_kzalloc() before platform_device_add()"
- "usb: dwc2: pci: Move usb_phy_generic_register()"
- "usb: dwc2: pci: Replace kzalloc() with devm_kzalloc()"

"usb: dwc2: Update bit polling functionality":

- This patch is a merging of "usb: dwc2: Move polling function to core.c."  
  and "usb: dwc2: Use common polling function.".
- Added dwc2_hsotg_wait_bit_clear() function, because in some cases,
  we should wait for bit would be cleared not set.
 

usb: dwc2: host: Fix transaction errors in host mode

- Renamed patch name from "usb: dwc2: host: Setting TOUTCAL and USBTRDTIM 
fields in host mode".
- Removed programming of USBTRDTIM bitfiled, because this field available
  only in Device mode.
- Added comment with description of TOUTCAL bitfiled in comment.
- In description added platforms for which this patch fixes transactions 
error.


usb: dwc2: Change TxFIFO and RxFIFO flushing flow 

Rewritten loop for waiting AHB master in IDLE state.





Grigor Tovmasyan (2):
  usb: dwc2: Delete unused functionality
  usb: dwc2: Make dwc2_force_mode() static

Minas Harutyunyan (3):
  usb: dwc2: hcd: Fix host channel halt flow
  usb: dwc2: host: Fix transaction errors in host mode
  usb: dwc2: Change TxFIFO and RxFIFO flushing flow

Razmik Karapetyan (7):
  usb: dwc2: Use AHB burst size parameter
  usb: dwc2: Set AHB burst size to INCR
  usb: dwc2: Update dwc2_handle_incomplete_isoc_in() function
  usb: dwc2: Update dwc2_handle_incomplete_isoc_out() function
  usb: dwc2: Update GINTSTS_GOUTNAKEFF interrupt handling
  usb: dwc2: Rename bit set/clear function names
  usb: dwc2: Remove unnecessary debug prints

Sevak Arakelyan (1):
  usb: dwc2: Update bit polling functionality

Vardan Mikayelyan (4):
  usb: dwc2: pci: Replace kzalloc() with devm_kzalloc()
  usb: dwc2: pci: Move usb_phy_generic_register()
  usb: dwc2: pci: Move devm_kzalloc() before platform_device_add()
  usb: dwc2: pci: Handle error cleanup in probe

 drivers/usb/dwc2/core.c   | 120 +---
 drivers/usb/dwc2/core.h   |  25 ++---
 drivers/usb/dwc2/gadget.c | 116 +++
 drivers/usb/dwc2/hcd.c| 136 +-
 drivers/usb/dwc2/hcd.h|  56 ---
 drivers/usb/dwc2/params.c |   2 +-
 drivers/usb/dwc2/pci.c|  27 +
 7 files changed, 189 insertions(+), 293 deletions(-)

-- 
2.11.0

--
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