Re: [PATCH v4 2/4] ARM: Exynos5250: Enabling ohci-exynos driver
Hi Kukjin, On Fri, Feb 1, 2013 at 3:56 AM, Kukjin Kim kgene@samsung.com wrote: Tomasz Figa wrote: Hi Vivek, [...] + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; For Samsung platforms we decided per board enabling of nodes and so this node should also contain: status = disabled; while in dts file of board using ohci there would be an overriding entry: usb@1212 { status = okay; }; I know that Exynos5250 has not been yet converted into this convention, but using it when adding new devices will simplify the process. Well, I have another opinion on this... Let's see, yeah, we are using following method in EXYNOS4 DT... - disabling IPs in SoC DT source and enabling IPs in board DT source But as I said, in EXYNOS5440 DT, I want to use following, because most of them should be enabled/used. - enabling IPs in SoC DT source and no need enabling in board DT source. I mean, it depends on each SoC's situation. So, since USB 2.0 (EHCI/OHCI) shall be always enabled on almost all the boards, so we shall keep it enabled in SOC DT source. In that case following patches can be picked for a merge: [PATCH v5 1/4] ARM: Exynos5250: Enabling ehci-s5p driver http://www.mail-archive.com/linux-usb@vger.kernel.org/msg12330.html [PATCH v4 2/4] ARM: Exynos5250: Enabling ohci-exynos driver http://www.mail-archive.com/linux-usb@vger.kernel.org/msg12266.html These will enable EHCI and OHCI support on exynos5250. Please let me know if you want me to resend them. ;-) -- Thanks Regards Vivek -- 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 v4 2/4] ARM: Exynos5250: Enabling ohci-exynos driver
Tomasz Figa wrote: Hi Vivek, [...] + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; For Samsung platforms we decided per board enabling of nodes and so this node should also contain: status = disabled; while in dts file of board using ohci there would be an overriding entry: usb@1212 { status = okay; }; I know that Exynos5250 has not been yet converted into this convention, but using it when adding new devices will simplify the process. Well, I have another opinion on this... Let's see, yeah, we are using following method in EXYNOS4 DT... - disabling IPs in SoC DT source and enabling IPs in board DT source But as I said, in EXYNOS5440 DT, I want to use following, because most of them should be enabled/used. - enabling IPs in SoC DT source and no need enabling in board DT source. I mean, it depends on each SoC's situation. Thanks. - Kukjin -- 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 v4 2/4] ARM: Exynos5250: Enabling ohci-exynos driver
Hi Tomasz, On Wed, Jan 16, 2013 at 1:14 PM, Tomasz Figa tomasz.f...@gmail.com wrote: Hi Vivek, On Tuesday 15 of January 2013 19:08:30 Vivek Gautam wrote: Adding OHCI device tree node for Exynos5250 along with the device base address. Signed-off-by: Vivek Gautam gautam.vi...@samsung.com Acked-by: Jingoo Han jg1@samsung.com Acked-by: Grant Likely grant.lik...@secretlab.ca --- .../devicetree/bindings/usb/exynos-usb.txt | 15 +++ arch/arm/boot/dts/exynos5250.dtsi | 6 ++ 2 files changed, 21 insertions(+), 0 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/exynos-usb.txt b/Documentation/devicetree/bindings/usb/exynos-usb.txt index e8bbb47..f66fcdd 100644 --- a/Documentation/devicetree/bindings/usb/exynos-usb.txt +++ b/Documentation/devicetree/bindings/usb/exynos-usb.txt @@ -23,3 +23,18 @@ Example: interrupts = 0 71 0; samsung,vbus-gpio = gpx2 6 1 3 3; }; + +OHCI +Required properties: + - compatible: should be samsung,exynos4210-ohci for USB 2.0 + OHCI companion controller in host mode. + - reg: physical base address of the controller and length of memory mapped + region. + - interrupts: interrupt number to the cpu. + +Example: + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; + }; diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi index 2cbe53e..ebb0907 100644 --- a/arch/arm/boot/dts/exynos5250.dtsi +++ b/arch/arm/boot/dts/exynos5250.dtsi @@ -281,6 +281,12 @@ interrupts = 0 71 0; }; + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; For Samsung platforms we decided per board enabling of nodes and so this node should also contain: status = disabled; while in dts file of board using ohci there would be an overriding entry: usb@1212 { status = okay; }; Yes, i could see that for exynos4x dt. I know that Exynos5250 has not been yet converted into this convention, but using it when adding new devices will simplify the process. Sure, will change this and other patches for usb as suggested. Thanks for the review. :-) -- Thanks Regards Vivek -- 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 v4 2/4] ARM: Exynos5250: Enabling ohci-exynos driver
Hi Vivek, On Tuesday 15 of January 2013 19:08:30 Vivek Gautam wrote: Adding OHCI device tree node for Exynos5250 along with the device base address. Signed-off-by: Vivek Gautam gautam.vi...@samsung.com Acked-by: Jingoo Han jg1@samsung.com Acked-by: Grant Likely grant.lik...@secretlab.ca --- .../devicetree/bindings/usb/exynos-usb.txt | 15 +++ arch/arm/boot/dts/exynos5250.dtsi | 6 ++ 2 files changed, 21 insertions(+), 0 deletions(-) diff --git a/Documentation/devicetree/bindings/usb/exynos-usb.txt b/Documentation/devicetree/bindings/usb/exynos-usb.txt index e8bbb47..f66fcdd 100644 --- a/Documentation/devicetree/bindings/usb/exynos-usb.txt +++ b/Documentation/devicetree/bindings/usb/exynos-usb.txt @@ -23,3 +23,18 @@ Example: interrupts = 0 71 0; samsung,vbus-gpio = gpx2 6 1 3 3; }; + +OHCI +Required properties: + - compatible: should be samsung,exynos4210-ohci for USB 2.0 + OHCI companion controller in host mode. + - reg: physical base address of the controller and length of memory mapped + region. + - interrupts: interrupt number to the cpu. + +Example: + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; + }; diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi index 2cbe53e..ebb0907 100644 --- a/arch/arm/boot/dts/exynos5250.dtsi +++ b/arch/arm/boot/dts/exynos5250.dtsi @@ -281,6 +281,12 @@ interrupts = 0 71 0; }; + usb@1212 { + compatible = samsung,exynos4210-ohci; + reg = 0x1212 0x100; + interrupts = 0 71 0; For Samsung platforms we decided per board enabling of nodes and so this node should also contain: status = disabled; while in dts file of board using ohci there would be an overriding entry: usb@1212 { status = okay; }; I know that Exynos5250 has not been yet converted into this convention, but using it when adding new devices will simplify the process. Best regards, Tomasz -- 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