From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756105Ab3JKDVh (ORCPT ); Thu, 10 Oct 2013 23:21:37 -0400 Received: from mail-ie0-f179.google.com ([209.85.223.179]:55678 "EHLO mail-ie0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755983Ab3JKDVe (ORCPT ); Thu, 10 Oct 2013 23:21:34 -0400 Message-ID: <52576EBA.9000502@linaro.org> Date: Thu, 10 Oct 2013 23:21:30 -0400 From: Matt Porter User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130804 Thunderbird/17.0.8 MIME-Version: 1.0 To: Paul Zimmerman CC: "balbi@ti.com" , Greg Kroah-Hartman , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Christian Daudt , Linux USB List , Linux ARM Kernel List , Linux Kernel Mailing List , Devicetree List , Linaro Patches Subject: Re: [PATCH 2/5] usb: gadget: s3c-hsotg: support configurable UTMI PHY width References: <1381140752-312-1-git-send-email-matt.porter@linaro.org> <1381140752-312-3-git-send-email-matt.porter@linaro.org> <20131010152922.GF28375@radagast> <5256DBD0.8030008@linaro.org> <20131010174620.GC19802@radagast> <5256FAFA.1020509@linaro.org> In-Reply-To: <5256FAFA.1020509@linaro.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/10/2013 03:07 PM, Matt Porter wrote: > On 10/10/2013 01:57 PM, Paul Zimmerman wrote: >>> From: Felipe Balbi [mailto:balbi@ti.com] >>> Sent: Thursday, October 10, 2013 10:46 AM >>> >>> On Thu, Oct 10, 2013 at 12:54:40PM -0400, Matt Porter wrote: >>>> On 10/10/2013 11:29 AM, Felipe Balbi wrote: >>>>> On Mon, Oct 07, 2013 at 06:12:29AM -0400, Matt Porter wrote: >>>>>> Extend dwc2 binding with an optional utmi phy width property. >>>>>> Enable the s3c-hsotg.c driver to use standard dwc2 binding >>>>>> and enable configuration of the UTMI phy width based on the >>>>>> property. >>>>>> >>>>>> Signed-off-by: Matt Porter >>>>>> Reviewed-by: Markus Mayer >>>>>> Reviewed-by: Tim Kryger >>>>>> --- >>>>>> Documentation/devicetree/bindings/staging/dwc2.txt | 4 ++++ >>>>>> drivers/usb/gadget/s3c-hsotg.c | 18 >>>>>> +++++++++++++++++- >>>>>> drivers/usb/gadget/s3c-hsotg.h | 1 + >>>>>> 3 files changed, 22 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt >>> b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> index 1a1b7cf..fb6b8ee 100644 >>>>>> --- a/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> +++ b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> @@ -6,10 +6,14 @@ Required properties: >>>>>> - reg : Should contain 1 register range (address and length) >>>>>> - interrupts : Should contain 1 interrupt >>>>>> >>>>>> +Optional properties: >>>>>> +- snps,phy-utmi-width: Must contain the UTMI data width (either 8 >>>>>> or 16) >>>>> >>>>> isn't this available in any of the configuration registers ? >>>> >>>> Yes and no. HWCFG4 has a UTMI data width field. However, it has 3 >>>> valid states, "8", "16", or "8 or 16". The BCM281xx implementation is >>>> set to the latter and the attached phy is 8-bit. >>>> >>>> Looking at dwc2 prior to Matthijs Kooijman's patch [1] which starts >>>> validating the value of phy_utmi_width in that driver, the pci.c >>>> dwc2_module_params .phy_utmi_width field there even had the comment, >>>> "/* 16 bits - NOT DETECTABLE */". The autodetect code in >>>> dwc2_set_param_phy_utmi_width() will fail if HWCFG4 has the "8 or 16" >>>> option as it just decides to default to a phy width of 16 if nothing >>>> is configured by the platform glue. This property would also allow >>>> this issue to be addressed in that driver. >>> >>> fair enough, but I'd really like to hear from DT folks if your suggested >>> binding is acceptable. It seems like we can equally argue that it's a SW >>> configuration or HW description. >> >> It's definitely a HW description - the width of the UTMI data connection. >> >> But, which PHY is this? Does it have a register that could tell what >> the data width is? The dwc2 core has an (optional) PHY Vendor Control >> Register that allows reading the PHY registers. > > This is the integrated PHY on the bcm28155 part (the entire > bcm281xx/11351 family shares it). My register doc doesn't show the > optional PHY vendor control register as being present in this dwc2 > implementation. I'll track down with the design team if this is > accurate...but it appears we can't go that route. The design team confirmed that we do not have any registers that can be accessed via the PHY Vendor Control Register. All PHY control registers are implemented in the MMIO control block which is supported in the PHY control driver later in this series.They also confirmed there's no other way to detect that it's an 8-bit data path. -Matt From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Porter Subject: Re: [PATCH 2/5] usb: gadget: s3c-hsotg: support configurable UTMI PHY width Date: Thu, 10 Oct 2013 23:21:30 -0400 Message-ID: <52576EBA.9000502@linaro.org> References: <1381140752-312-1-git-send-email-matt.porter@linaro.org> <1381140752-312-3-git-send-email-matt.porter@linaro.org> <20131010152922.GF28375@radagast> <5256DBD0.8030008@linaro.org> <20131010174620.GC19802@radagast> <5256FAFA.1020509@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5256FAFA.1020509-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Paul Zimmerman Cc: "balbi-l0cyMroinI0@public.gmane.org" , Greg Kroah-Hartman , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Christian Daudt , Linux USB List , Linux ARM Kernel List , Linux Kernel Mailing List , Devicetree List , Linaro Patches List-Id: devicetree@vger.kernel.org On 10/10/2013 03:07 PM, Matt Porter wrote: > On 10/10/2013 01:57 PM, Paul Zimmerman wrote: >>> From: Felipe Balbi [mailto:balbi-l0cyMroinI0@public.gmane.org] >>> Sent: Thursday, October 10, 2013 10:46 AM >>> >>> On Thu, Oct 10, 2013 at 12:54:40PM -0400, Matt Porter wrote: >>>> On 10/10/2013 11:29 AM, Felipe Balbi wrote: >>>>> On Mon, Oct 07, 2013 at 06:12:29AM -0400, Matt Porter wrote: >>>>>> Extend dwc2 binding with an optional utmi phy width property. >>>>>> Enable the s3c-hsotg.c driver to use standard dwc2 binding >>>>>> and enable configuration of the UTMI phy width based on the >>>>>> property. >>>>>> >>>>>> Signed-off-by: Matt Porter >>>>>> Reviewed-by: Markus Mayer >>>>>> Reviewed-by: Tim Kryger >>>>>> --- >>>>>> Documentation/devicetree/bindings/staging/dwc2.txt | 4 ++++ >>>>>> drivers/usb/gadget/s3c-hsotg.c | 18 >>>>>> +++++++++++++++++- >>>>>> drivers/usb/gadget/s3c-hsotg.h | 1 + >>>>>> 3 files changed, 22 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt >>> b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> index 1a1b7cf..fb6b8ee 100644 >>>>>> --- a/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> +++ b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> @@ -6,10 +6,14 @@ Required properties: >>>>>> - reg : Should contain 1 register range (address and length) >>>>>> - interrupts : Should contain 1 interrupt >>>>>> >>>>>> +Optional properties: >>>>>> +- snps,phy-utmi-width: Must contain the UTMI data width (either 8 >>>>>> or 16) >>>>> >>>>> isn't this available in any of the configuration registers ? >>>> >>>> Yes and no. HWCFG4 has a UTMI data width field. However, it has 3 >>>> valid states, "8", "16", or "8 or 16". The BCM281xx implementation is >>>> set to the latter and the attached phy is 8-bit. >>>> >>>> Looking at dwc2 prior to Matthijs Kooijman's patch [1] which starts >>>> validating the value of phy_utmi_width in that driver, the pci.c >>>> dwc2_module_params .phy_utmi_width field there even had the comment, >>>> "/* 16 bits - NOT DETECTABLE */". The autodetect code in >>>> dwc2_set_param_phy_utmi_width() will fail if HWCFG4 has the "8 or 16" >>>> option as it just decides to default to a phy width of 16 if nothing >>>> is configured by the platform glue. This property would also allow >>>> this issue to be addressed in that driver. >>> >>> fair enough, but I'd really like to hear from DT folks if your suggested >>> binding is acceptable. It seems like we can equally argue that it's a SW >>> configuration or HW description. >> >> It's definitely a HW description - the width of the UTMI data connection. >> >> But, which PHY is this? Does it have a register that could tell what >> the data width is? The dwc2 core has an (optional) PHY Vendor Control >> Register that allows reading the PHY registers. > > This is the integrated PHY on the bcm28155 part (the entire > bcm281xx/11351 family shares it). My register doc doesn't show the > optional PHY vendor control register as being present in this dwc2 > implementation. I'll track down with the design team if this is > accurate...but it appears we can't go that route. The design team confirmed that we do not have any registers that can be accessed via the PHY Vendor Control Register. All PHY control registers are implemented in the MMIO control block which is supported in the PHY control driver later in this series.They also confirmed there's no other way to detect that it's an 8-bit data path. -Matt -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: matt.porter@linaro.org (Matt Porter) Date: Thu, 10 Oct 2013 23:21:30 -0400 Subject: [PATCH 2/5] usb: gadget: s3c-hsotg: support configurable UTMI PHY width In-Reply-To: <5256FAFA.1020509@linaro.org> References: <1381140752-312-1-git-send-email-matt.porter@linaro.org> <1381140752-312-3-git-send-email-matt.porter@linaro.org> <20131010152922.GF28375@radagast> <5256DBD0.8030008@linaro.org> <20131010174620.GC19802@radagast> <5256FAFA.1020509@linaro.org> Message-ID: <52576EBA.9000502@linaro.org> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 10/10/2013 03:07 PM, Matt Porter wrote: > On 10/10/2013 01:57 PM, Paul Zimmerman wrote: >>> From: Felipe Balbi [mailto:balbi at ti.com] >>> Sent: Thursday, October 10, 2013 10:46 AM >>> >>> On Thu, Oct 10, 2013 at 12:54:40PM -0400, Matt Porter wrote: >>>> On 10/10/2013 11:29 AM, Felipe Balbi wrote: >>>>> On Mon, Oct 07, 2013 at 06:12:29AM -0400, Matt Porter wrote: >>>>>> Extend dwc2 binding with an optional utmi phy width property. >>>>>> Enable the s3c-hsotg.c driver to use standard dwc2 binding >>>>>> and enable configuration of the UTMI phy width based on the >>>>>> property. >>>>>> >>>>>> Signed-off-by: Matt Porter >>>>>> Reviewed-by: Markus Mayer >>>>>> Reviewed-by: Tim Kryger >>>>>> --- >>>>>> Documentation/devicetree/bindings/staging/dwc2.txt | 4 ++++ >>>>>> drivers/usb/gadget/s3c-hsotg.c | 18 >>>>>> +++++++++++++++++- >>>>>> drivers/usb/gadget/s3c-hsotg.h | 1 + >>>>>> 3 files changed, 22 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/staging/dwc2.txt >>> b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> index 1a1b7cf..fb6b8ee 100644 >>>>>> --- a/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> +++ b/Documentation/devicetree/bindings/staging/dwc2.txt >>>>>> @@ -6,10 +6,14 @@ Required properties: >>>>>> - reg : Should contain 1 register range (address and length) >>>>>> - interrupts : Should contain 1 interrupt >>>>>> >>>>>> +Optional properties: >>>>>> +- snps,phy-utmi-width: Must contain the UTMI data width (either 8 >>>>>> or 16) >>>>> >>>>> isn't this available in any of the configuration registers ? >>>> >>>> Yes and no. HWCFG4 has a UTMI data width field. However, it has 3 >>>> valid states, "8", "16", or "8 or 16". The BCM281xx implementation is >>>> set to the latter and the attached phy is 8-bit. >>>> >>>> Looking at dwc2 prior to Matthijs Kooijman's patch [1] which starts >>>> validating the value of phy_utmi_width in that driver, the pci.c >>>> dwc2_module_params .phy_utmi_width field there even had the comment, >>>> "/* 16 bits - NOT DETECTABLE */". The autodetect code in >>>> dwc2_set_param_phy_utmi_width() will fail if HWCFG4 has the "8 or 16" >>>> option as it just decides to default to a phy width of 16 if nothing >>>> is configured by the platform glue. This property would also allow >>>> this issue to be addressed in that driver. >>> >>> fair enough, but I'd really like to hear from DT folks if your suggested >>> binding is acceptable. It seems like we can equally argue that it's a SW >>> configuration or HW description. >> >> It's definitely a HW description - the width of the UTMI data connection. >> >> But, which PHY is this? Does it have a register that could tell what >> the data width is? The dwc2 core has an (optional) PHY Vendor Control >> Register that allows reading the PHY registers. > > This is the integrated PHY on the bcm28155 part (the entire > bcm281xx/11351 family shares it). My register doc doesn't show the > optional PHY vendor control register as being present in this dwc2 > implementation. I'll track down with the design team if this is > accurate...but it appears we can't go that route. The design team confirmed that we do not have any registers that can be accessed via the PHY Vendor Control Register. All PHY control registers are implemented in the MMIO control block which is supported in the PHY control driver later in this series.They also confirmed there's no other way to detect that it's an 8-bit data path. -Matt