diff mbox

[v2,8/8] ARM: dts: omap5: add dwc3 core dt data

Message ID 20130306091942.GJ28587@arwen.pp.htv.fi (mailing list archive)
State New, archived
Headers show

Commit Message

Felipe Balbi March 6, 2013, 9:19 a.m. UTC
Hi,

On Wed, Mar 06, 2013 at 02:25:48PM +0530, kishon wrote:
> Hi,
> 
> On Tuesday 05 March 2013 08:09 PM, Felipe Balbi wrote:
> >Hi,
> >
> >On Tue, Mar 05, 2013 at 08:07:03PM +0530, Kishon Vijay Abraham I wrote:
> >>Add dwc3 core dt data as a subnode to dwc3 omap glue data in omap5 dt
> >>data file. The information for the entered data node is available @
> >>Documentation/devicetree/bindings/usb/dwc3.txt
> >>
> >>Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
> >>---
> >>  arch/arm/boot/dts/omap5.dtsi |    7 +++++++
> >>  1 file changed, 7 insertions(+)
> >>
> >>diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
> >>index c4eb2ec..24dd69f 100644
> >>--- a/arch/arm/boot/dts/omap5.dtsi
> >>+++ b/arch/arm/boot/dts/omap5.dtsi
> >>@@ -514,6 +514,13 @@
> >>  			#size-cells = <1>;
> >>  			utmi-mode = <2>;
> >>  			ranges;
> >>+			dwc3@4a030000 {
> >>+				compatible = "synopsys,dwc3";
> >>+				reg = <0x4a030000 0x1000>;
> >>+				interrupts = <0 92 4>;
> >>+				usb-phy = <&usb2_phy>, <&usb3_phy>;
> >>+				tx-fifo-resize;
> >
> >I think this should be merged with previous patch. Also, we need to
> >check whether tx-fifo-resize is needed with ES2 or not ;-)
> 
> yeah. ES2 works without tx-fifo-resize.

Great, I guess we can deprecate that attribute. Can you check if ES2
still works with:



starting on 1.94a (IIRC) core was supposed to control PHYs directly and
since there's nobody in linux mainline using any version prior to 1.94a,
we could drop that too.

Comments

Kishon Vijay Abraham I March 6, 2013, 9:21 a.m. UTC | #1
On Wednesday 06 March 2013 02:49 PM, Felipe Balbi wrote:
> Hi,
>
> On Wed, Mar 06, 2013 at 02:25:48PM +0530, kishon wrote:
>> Hi,
>>
>> On Tuesday 05 March 2013 08:09 PM, Felipe Balbi wrote:
>>> Hi,
>>>
>>> On Tue, Mar 05, 2013 at 08:07:03PM +0530, Kishon Vijay Abraham I wrote:
>>>> Add dwc3 core dt data as a subnode to dwc3 omap glue data in omap5 dt
>>>> data file. The information for the entered data node is available @
>>>> Documentation/devicetree/bindings/usb/dwc3.txt
>>>>
>>>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
>>>> ---
>>>>   arch/arm/boot/dts/omap5.dtsi |    7 +++++++
>>>>   1 file changed, 7 insertions(+)
>>>>
>>>> diff --git a/arch/arm/boot/dts/omap5.dtsi b/arch/arm/boot/dts/omap5.dtsi
>>>> index c4eb2ec..24dd69f 100644
>>>> --- a/arch/arm/boot/dts/omap5.dtsi
>>>> +++ b/arch/arm/boot/dts/omap5.dtsi
>>>> @@ -514,6 +514,13 @@
>>>>   			#size-cells = <1>;
>>>>   			utmi-mode = <2>;
>>>>   			ranges;
>>>> +			dwc3@4a030000 {
>>>> +				compatible = "synopsys,dwc3";
>>>> +				reg = <0x4a030000 0x1000>;
>>>> +				interrupts = <0 92 4>;
>>>> +				usb-phy = <&usb2_phy>, <&usb3_phy>;
>>>> +				tx-fifo-resize;
>>>
>>> I think this should be merged with previous patch. Also, we need to
>>> check whether tx-fifo-resize is needed with ES2 or not ;-)
>>
>> yeah. ES2 works without tx-fifo-resize.
>
> Great, I guess we can deprecate that attribute. Can you check if ES2
> still works with:
>
> diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
> index 82e160e..cad0131 100644
> --- a/drivers/usb/dwc3/gadget.c
> +++ b/drivers/usb/dwc3/gadget.c
> @@ -2029,6 +2029,8 @@ static void dwc3_gadget_usb3_phy_suspend(struct dwc3 *dwc, int suspend)
>   {
>   	u32			reg;
>
> +	return;
> +
>   	reg = dwc3_readl(dwc->regs, DWC3_GUSB3PIPECTL(0));
>
>   	if (suspend)
> @@ -2043,6 +2045,8 @@ static void dwc3_gadget_usb2_phy_suspend(struct dwc3 *dwc, int suspend)
>   {
>   	u32			reg;
>
> +	return;
> +
>   	reg = dwc3_readl(dwc->regs, DWC3_GUSB2PHYCFG(0));
>
>   	if (suspend)
>
>
> starting on 1.94a (IIRC) core was supposed to control PHYs directly and
> since there's nobody in linux mainline using any version prior to 1.94a,
> we could drop that too.

Will check that in a while.

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

Patch

diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
index 82e160e..cad0131 100644
--- a/drivers/usb/dwc3/gadget.c
+++ b/drivers/usb/dwc3/gadget.c
@@ -2029,6 +2029,8 @@  static void dwc3_gadget_usb3_phy_suspend(struct dwc3 *dwc, int suspend)
 {
 	u32			reg;
 
+	return;
+
 	reg = dwc3_readl(dwc->regs, DWC3_GUSB3PIPECTL(0));
 
 	if (suspend)
@@ -2043,6 +2045,8 @@  static void dwc3_gadget_usb2_phy_suspend(struct dwc3 *dwc, int suspend)
 {
 	u32			reg;
 
+	return;
+
 	reg = dwc3_readl(dwc->regs, DWC3_GUSB2PHYCFG(0));
 
 	if (suspend)