Re: [PATCH v3 0/5] Add support for Fujitsu USB host controller

2015-02-11 Thread Mathias Nyman
On 10.02.2015 17:43, Sneeker Yeh wrote:
 Hi
 
 2015-01-31 0:38 GMT+08:00 Felipe Balbi ba...@ti.com:
 Hi,

 On Thu, Jan 29, 2015 at 10:23:12AM -0600, Felipe Balbi wrote:
 On Tue, Jan 27, 2015 at 09:22:50AM -0600, Felipe Balbi wrote:
 Hi,

 On Sun, Jan 25, 2015 at 04:13:23PM +0800, Sneeker Yeh wrote:
 These patches add support for XHCI compliant Host controller found
 on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
 The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 
 driver
 and last four patch is about quirk implementation of errata in Synopsis
 DesignWare USB3 IP.

 Patch 1 introduces a quirk with device disconnection management necessary
 Synopsys Designware USB3 IP with versions  3.00a and hardware 
 configuration
 DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without the
 quirk, that host controller will die after a usb device is disconnected 
 from
 port of root hub.

 Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci 
 platform
 data.

 Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis DesignWare 
 USB3
 IP core driver.

 Patch 4 introduces using a quirk based on a errata of Synopsis
 DesignWare USB3 IP which is versions  3.00a and has hardware 
 configuration
 DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. 
 As a
 result this quirk has to be enabled via platform data or device tree.

 Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare 
 USB3 IP
 driver.


 Mathias, let me know how you want to handle this. Either I take them
 all, or you take them all. What do you prefer ?

 Mathias ?

 Mathias, a reminder on this series.
 
 Would any problem is still in my patchset?
 e.g. I might still not arrange these patch in a appropriate order so
 that Mathias cannot review and accept these?
 

Sorry about the delay.
Let me take a better look at the first patch and reasons behind the
race. I want to check if a quirk is enough or if we need to dig deeper into 
xhci.

I'll try to do it today still

-Mathias

--
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 v3 0/5] Add support for Fujitsu USB host controller

2015-02-10 Thread Sneeker Yeh
Hi

2015-01-31 0:38 GMT+08:00 Felipe Balbi ba...@ti.com:
 Hi,

 On Thu, Jan 29, 2015 at 10:23:12AM -0600, Felipe Balbi wrote:
 On Tue, Jan 27, 2015 at 09:22:50AM -0600, Felipe Balbi wrote:
  Hi,
 
  On Sun, Jan 25, 2015 at 04:13:23PM +0800, Sneeker Yeh wrote:
   These patches add support for XHCI compliant Host controller found
   on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
   The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 
   driver
   and last four patch is about quirk implementation of errata in Synopsis
   DesignWare USB3 IP.
  
   Patch 1 introduces a quirk with device disconnection management necessary
   Synopsys Designware USB3 IP with versions  3.00a and hardware 
   configuration
   DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without 
   the
   quirk, that host controller will die after a usb device is disconnected 
   from
   port of root hub.
  
   Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci 
   platform
   data.
  
   Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis 
   DesignWare USB3
   IP core driver.
  
   Patch 4 introduces using a quirk based on a errata of Synopsis
   DesignWare USB3 IP which is versions  3.00a and has hardware 
   configuration
   DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. 
   As a
   result this quirk has to be enabled via platform data or device tree.
  
   Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare 
   USB3 IP
   driver.
  
 
  Mathias, let me know how you want to handle this. Either I take them
  all, or you take them all. What do you prefer ?

 Mathias ?

 Mathias, a reminder on this series.

Would any problem is still in my patchset?
e.g. I might still not arrange these patch in a appropriate order so
that Mathias cannot review and accept these?

BR,
Sneeker, sincerely


 --
 balbi
--
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 v3 0/5] Add support for Fujitsu USB host controller

2015-01-30 Thread Felipe Balbi
Hi,

On Thu, Jan 29, 2015 at 10:23:12AM -0600, Felipe Balbi wrote:
 On Tue, Jan 27, 2015 at 09:22:50AM -0600, Felipe Balbi wrote:
  Hi,
  
  On Sun, Jan 25, 2015 at 04:13:23PM +0800, Sneeker Yeh wrote:
   These patches add support for XHCI compliant Host controller found
   on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
   The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 
   driver
   and last four patch is about quirk implementation of errata in Synopsis
   DesignWare USB3 IP.
   
   Patch 1 introduces a quirk with device disconnection management necessary
   Synopsys Designware USB3 IP with versions  3.00a and hardware 
   configuration
   DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without the
   quirk, that host controller will die after a usb device is disconnected 
   from
   port of root hub.
   
   Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci 
   platform
   data.
   
   Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis DesignWare 
   USB3
   IP core driver.
   
   Patch 4 introduces using a quirk based on a errata of Synopsis
   DesignWare USB3 IP which is versions  3.00a and has hardware 
   configuration
   DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. 
   As a
   result this quirk has to be enabled via platform data or device tree.
   
   Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare 
   USB3 IP
   driver. 
   
  
  Mathias, let me know how you want to handle this. Either I take them
  all, or you take them all. What do you prefer ?
 
 Mathias ?

Mathias, a reminder on this series.

-- 
balbi


signature.asc
Description: Digital signature


Re: [PATCH v3 0/5] Add support for Fujitsu USB host controller

2015-01-29 Thread Felipe Balbi
On Tue, Jan 27, 2015 at 09:22:50AM -0600, Felipe Balbi wrote:
 Hi,
 
 On Sun, Jan 25, 2015 at 04:13:23PM +0800, Sneeker Yeh wrote:
  These patches add support for XHCI compliant Host controller found
  on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
  The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 
  driver
  and last four patch is about quirk implementation of errata in Synopsis
  DesignWare USB3 IP.
  
  Patch 1 introduces a quirk with device disconnection management necessary
  Synopsys Designware USB3 IP with versions  3.00a and hardware configuration
  DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without the
  quirk, that host controller will die after a usb device is disconnected from
  port of root hub.
  
  Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci 
  platform
  data.
  
  Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis DesignWare 
  USB3
  IP core driver.
  
  Patch 4 introduces using a quirk based on a errata of Synopsis
  DesignWare USB3 IP which is versions  3.00a and has hardware configuration
  DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. As 
  a
  result this quirk has to be enabled via platform data or device tree.
  
  Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare USB3 
  IP
  driver. 
  
 
 Mathias, let me know how you want to handle this. Either I take them
 all, or you take them all. What do you prefer ?

Mathias ?

-- 
balbi


signature.asc
Description: Digital signature


Re: [PATCH v3 0/5] Add support for Fujitsu USB host controller

2015-01-27 Thread Felipe Balbi
Hi,

On Sun, Jan 25, 2015 at 04:13:23PM +0800, Sneeker Yeh wrote:
 These patches add support for XHCI compliant Host controller found
 on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
 The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 
 driver
 and last four patch is about quirk implementation of errata in Synopsis
 DesignWare USB3 IP.
 
 Patch 1 introduces a quirk with device disconnection management necessary
 Synopsys Designware USB3 IP with versions  3.00a and hardware configuration
 DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without the
 quirk, that host controller will die after a usb device is disconnected from
 port of root hub.
 
 Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci 
 platform
 data.
 
 Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis DesignWare USB3
 IP core driver.
 
 Patch 4 introduces using a quirk based on a errata of Synopsis
 DesignWare USB3 IP which is versions  3.00a and has hardware configuration
 DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. As a
 result this quirk has to be enabled via platform data or device tree.
 
 Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare USB3 IP
 driver. 
 

Mathias, let me know how you want to handle this. Either I take them
all, or you take them all. What do you prefer ?

-- 
balbi


signature.asc
Description: Digital signature


[PATCH v3 0/5] Add support for Fujitsu USB host controller

2015-01-25 Thread Sneeker Yeh
These patches add support for XHCI compliant Host controller found
on Fujitsu Socs, and are based on http://lwn.net/Articles/629162/
The first patch is to add Fujitsu glue layer of Synopsis DesignWare USB3 driver
and last four patch is about quirk implementation of errata in Synopsis
DesignWare USB3 IP.

Patch 1 introduces a quirk with device disconnection management necessary
Synopsys Designware USB3 IP with versions  3.00a and hardware configuration
DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1. It solves a problem where without the
quirk, that host controller will die after a usb device is disconnected from
port of root hub.

Patch 2 is to set Synopsis quirk in xhci platform driver based on xhci platform
data.

Patch 3 is to add a revison number 2.90a and 3.00a of Synopsis DesignWare USB3
IP core driver.

Patch 4 introduces using a quirk based on a errata of Synopsis
DesignWare USB3 IP which is versions  3.00a and has hardware configuration
DWC_USB3_SUSPEND_ON_DISCONNECT_EN=1, which cannot be read from software. As a
result this quirk has to be enabled via platform data or device tree.

Patch 5 introduces Fujitsu Specific Glue layer in Synopsis DesignWare USB3 IP
driver. 

Successfully tested on Fujitsu mb86s7x board.

Changes since v2 (RFC):
[https://lkml.org/lkml/2014/12/15/929]
 - based on Felipe's comment, re-order patches to avoid breaking bisectability,
 - based on Felipe's comment, add comment to structure's member, and sort it
   alphabetically,
 - based on Felipe's comment, add another v2.90 revision number in dwc3 IP.

Changes since v1 (RFC):
[https://lkml.org/lkml/2014/12/15/929]
 - based on Arnd's comment, remove entire unnecessary Fujitsu EHCI/OHCI glue,
 - based on Felipe's comment, fix mis-using of runtime-pm API and setting dma
   mask, remove unnecessary comment, and refactor suspend/resume handler in
   Fujitsu Specific Glue layer in dwc3,
 - based on Felipe's comment, add more commit log and comments in Synopsis
   quirk implementation, and separate it into four patches.

Sneeker Yeh (5):
  xhci: add a quirk for device disconnection errata for Synopsis
Designware USB3 core
  xhci: Platform: Set Synopsis device disconnection quirk based on
platform data
  usb: dwc3: add revision number DWC3_REVISION_290A and
DWC3_REVISION_300A
  usb: dwc3: Add quirk for Synopsis device disconnection errata
  usb: dwc3: add Fujitsu Specific Glue layer

 Documentation/devicetree/bindings/usb/dwc3.txt |   17 ++
 .../devicetree/bindings/usb/fujitsu-dwc3.txt   |   33 
 drivers/usb/dwc3/Kconfig   |   11 ++
 drivers/usb/dwc3/Makefile  |1 +
 drivers/usb/dwc3/core.c|6 +
 drivers/usb/dwc3/core.h|6 +
 drivers/usb/dwc3/dwc3-mb86s70.c|  206 
 drivers/usb/dwc3/host.c|4 +
 drivers/usb/dwc3/platform_data.h   |8 +
 drivers/usb/host/xhci-hub.c|4 +
 drivers/usb/host/xhci-plat.c   |3 +
 drivers/usb/host/xhci.c|   29 +++
 drivers/usb/host/xhci.h|   24 +++
 include/linux/usb/xhci_pdriver.h   |4 +
 14 files changed, 356 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/usb/fujitsu-dwc3.txt
 create mode 100644 drivers/usb/dwc3/dwc3-mb86s70.c

-- 
1.7.9.5

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