linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: Add USB host node for Exynos4
@ 2013-07-23 17:32 Dongjin Kim
  2013-07-24  3:38 ` Sachin Kamat
  2013-07-24 14:10 ` [PATCH v2] " Dongjin Kim
  0 siblings, 2 replies; 5+ messages in thread
From: Dongjin Kim @ 2013-07-23 17:32 UTC (permalink / raw)
  To: linux-arm-kernel

This patch adds EHCI and OHCI host device nodes for Exynos4.

Signed-off-by: Dongjin Kim <tobetter@gmail.com>
---
 arch/arm/boot/dts/exynos4.dtsi |   20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
index 3f94fe8..1cdbf89 100644
--- a/arch/arm/boot/dts/exynos4.dtsi
+++ b/arch/arm/boot/dts/exynos4.dtsi
@@ -155,6 +155,26 @@
 		status = "disabled";
 	};
 
+	ehci at 12580000 {
+		compatible = "samsung,exynos4210-ehci";
+		reg = <0x12580000 0x100>;
+		interrupts = <0 70 0>;
+		status = "disabled";
+
+		clocks = <&clock 304>;
+		clock-names = "usbhost";
+	};
+
+	ohci at 12590000 {
+		compatible = "samsung,exynos4210-ohci";
+		reg = <0x12580000 0x100>;
+		interrupts = <0 70 0>;
+		status = "disabled";
+
+		clocks = <&clock 304>;
+		clock-names = "usbhost";
+	};
+
 	mfc: codec at 13400000 {
 		compatible = "samsung,mfc-v5";
 		reg = <0x13400000 0x10000>;
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH] ARM: dts: Add USB host node for Exynos4
  2013-07-23 17:32 [PATCH] ARM: dts: Add USB host node for Exynos4 Dongjin Kim
@ 2013-07-24  3:38 ` Sachin Kamat
  2013-07-24  6:29   ` Dongjin Kim
  2013-07-24 14:10 ` [PATCH v2] " Dongjin Kim
  1 sibling, 1 reply; 5+ messages in thread
From: Sachin Kamat @ 2013-07-24  3:38 UTC (permalink / raw)
  To: linux-arm-kernel

On 23 July 2013 23:02, Dongjin Kim <tobetter@gmail.com> wrote:
> This patch adds EHCI and OHCI host device nodes for Exynos4.
>
> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
> ---
>  arch/arm/boot/dts/exynos4.dtsi |   20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> index 3f94fe8..1cdbf89 100644
> --- a/arch/arm/boot/dts/exynos4.dtsi
> +++ b/arch/arm/boot/dts/exynos4.dtsi
> @@ -155,6 +155,26 @@
>                 status = "disabled";
>         };
>
> +       ehci at 12580000 {
> +               compatible = "samsung,exynos4210-ehci";
> +               reg = <0x12580000 0x100>;
> +               interrupts = <0 70 0>;
> +               status = "disabled";
> +
> +               clocks = <&clock 304>;
> +               clock-names = "usbhost";
> +       };
> +
> +       ohci at 12590000 {
> +               compatible = "samsung,exynos4210-ohci";
> +               reg = <0x12580000 0x100>;

Register value and node name do not match. Typo?


> +               interrupts = <0 70 0>;
> +               status = "disabled";
> +
> +               clocks = <&clock 304>;
> +               clock-names = "usbhost";
> +       };
> +
>         mfc: codec at 13400000 {
>                 compatible = "samsung,mfc-v5";
>                 reg = <0x13400000 0x10000>;
> --
> 1.7.9.5
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
With warm regards,
Sachin

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ARM: dts: Add USB host node for Exynos4
  2013-07-24  3:38 ` Sachin Kamat
@ 2013-07-24  6:29   ` Dongjin Kim
  2013-07-24  6:49     ` Jingoo Han
  0 siblings, 1 reply; 5+ messages in thread
From: Dongjin Kim @ 2013-07-24  6:29 UTC (permalink / raw)
  To: linux-arm-kernel

+Jingoo Han

Sorry, its my bad. Typo problem.

I found another patch same with mine,
https://patchwork.kernel.org/patch/2628451, submitted by Jingoo Han
and tested on Exynos4210. Mine is tested on Exynos4412. Both SoC have
same USB host address space, so I think USB host device nodes can be
placed in exynos4.dtsi.

Regards,
Dongjin.

On Wed, Jul 24, 2013 at 12:38 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> On 23 July 2013 23:02, Dongjin Kim <tobetter@gmail.com> wrote:
>> This patch adds EHCI and OHCI host device nodes for Exynos4.
>>
>> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
>> ---
>>  arch/arm/boot/dts/exynos4.dtsi |   20 ++++++++++++++++++++
>>  1 file changed, 20 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
>> index 3f94fe8..1cdbf89 100644
>> --- a/arch/arm/boot/dts/exynos4.dtsi
>> +++ b/arch/arm/boot/dts/exynos4.dtsi
>> @@ -155,6 +155,26 @@
>>                 status = "disabled";
>>         };
>>
>> +       ehci at 12580000 {
>> +               compatible = "samsung,exynos4210-ehci";
>> +               reg = <0x12580000 0x100>;
>> +               interrupts = <0 70 0>;
>> +               status = "disabled";
>> +
>> +               clocks = <&clock 304>;
>> +               clock-names = "usbhost";
>> +       };
>> +
>> +       ohci at 12590000 {
>> +               compatible = "samsung,exynos4210-ohci";
>> +               reg = <0x12580000 0x100>;
>
> Register value and node name do not match. Typo?
>
>
>> +               interrupts = <0 70 0>;
>> +               status = "disabled";
>> +
>> +               clocks = <&clock 304>;
>> +               clock-names = "usbhost";
>> +       };
>> +
>>         mfc: codec at 13400000 {
>>                 compatible = "samsung,mfc-v5";
>>                 reg = <0x13400000 0x10000>;
>> --
>> 1.7.9.5
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
>> the body of a message to majordomo at vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
>
>
> --
> With warm regards,
> Sachin

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] ARM: dts: Add USB host node for Exynos4
  2013-07-24  6:29   ` Dongjin Kim
@ 2013-07-24  6:49     ` Jingoo Han
  0 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-07-24  6:49 UTC (permalink / raw)
  To: linux-arm-kernel

On Wednesday, July 24, 2013 3:29 PM, Dongjin Kim wrote:
> 
> +Jingoo Han
> 
> Sorry, its my bad. Typo problem.
> 
> I found another patch same with mine,
> https://patchwork.kernel.org/patch/2628451, submitted by Jingoo Han
> and tested on Exynos4210. Mine is tested on Exynos4412. Both SoC have
> same USB host address space, so I think USB host device nodes can be
> placed in exynos4.dtsi.

OK, USB host device nodes can be placed in exynos4.dtsi.

However, next time, please leave comment on earlier patch.
The patch for Exynos4210 USB was already submitted 2 months ago.


> 
> Regards,
> Dongjin.
> 
> On Wed, Jul 24, 2013 at 12:38 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> > On 23 July 2013 23:02, Dongjin Kim <tobetter@gmail.com> wrote:
> >> This patch adds EHCI and OHCI host device nodes for Exynos4.
> >>
> >> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
> >> ---
> >>  arch/arm/boot/dts/exynos4.dtsi |   20 ++++++++++++++++++++
> >>  1 file changed, 20 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
> >> index 3f94fe8..1cdbf89 100644
> >> --- a/arch/arm/boot/dts/exynos4.dtsi
> >> +++ b/arch/arm/boot/dts/exynos4.dtsi
> >> @@ -155,6 +155,26 @@
> >>                 status = "disabled";
> >>         };
> >>
> >> +       ehci at 12580000 {
> >> +               compatible = "samsung,exynos4210-ehci";
> >> +               reg = <0x12580000 0x100>;
> >> +               interrupts = <0 70 0>;
> >> +               status = "disabled";
> >> +
> >> +               clocks = <&clock 304>;
> >> +               clock-names = "usbhost";
> >> +       };
> >> +
> >> +       ohci at 12590000 {
> >> +               compatible = "samsung,exynos4210-ohci";
> >> +               reg = <0x12580000 0x100>;
> >
> > Register value and node name do not match. Typo?
> >
> >
> >> +               interrupts = <0 70 0>;
> >> +               status = "disabled";
> >> +
> >> +               clocks = <&clock 304>;
> >> +               clock-names = "usbhost";
> >> +       };
> >> +
> >>         mfc: codec at 13400000 {
> >>                 compatible = "samsung,mfc-v5";
> >>                 reg = <0x13400000 0x10000>;
> >> --
> >> 1.7.9.5

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v2] ARM: dts: Add USB host node for Exynos4
  2013-07-23 17:32 [PATCH] ARM: dts: Add USB host node for Exynos4 Dongjin Kim
  2013-07-24  3:38 ` Sachin Kamat
@ 2013-07-24 14:10 ` Dongjin Kim
  1 sibling, 0 replies; 5+ messages in thread
From: Dongjin Kim @ 2013-07-24 14:10 UTC (permalink / raw)
  To: linux-arm-kernel

This patch adds EHCI and OHCI host device nodes for Exynos4.

Signed-off-by: Dongjin Kim <tobetter@gmail.com>
---
 arch/arm/boot/dts/exynos4.dtsi |   18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
index 3f94fe8..cbe5219 100644
--- a/arch/arm/boot/dts/exynos4.dtsi
+++ b/arch/arm/boot/dts/exynos4.dtsi
@@ -155,6 +155,24 @@
 		status = "disabled";
 	};
 
+	ehci at 12580000 {
+		compatible = "samsung,exynos4210-ehci";
+		reg = <0x12580000 0x100>;
+		interrupts = <0 70 0>;
+		clocks = <&clock 304>;
+		clock-names = "usbhost";
+		status = "disabled";
+	};
+
+	ohci at 12590000 {
+		compatible = "samsung,exynos4210-ohci";
+		reg = <0x12590000 0x100>;
+		interrupts = <0 70 0>;
+		clocks = <&clock 304>;
+		clock-names = "usbhost";
+		status = "disabled";
+	};
+
 	mfc: codec at 13400000 {
 		compatible = "samsung,mfc-v5";
 		reg = <0x13400000 0x10000>;
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-07-24 14:10 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-23 17:32 [PATCH] ARM: dts: Add USB host node for Exynos4 Dongjin Kim
2013-07-24  3:38 ` Sachin Kamat
2013-07-24  6:29   ` Dongjin Kim
2013-07-24  6:49     ` Jingoo Han
2013-07-24 14:10 ` [PATCH v2] " Dongjin Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).