linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12
@ 2013-07-23 17:31 Dongjin Kim
  2013-07-24  5:12 ` Sachin Kamat
  0 siblings, 1 reply; 4+ messages in thread
From: Dongjin Kim @ 2013-07-23 17:31 UTC (permalink / raw)
  Cc: Dongjin Kim, Russell King, Kukjin Kim, Sachin Kamat, Tomasz Figa,
	Thomas Abraham, Kyungmin Park, linux-arm-kernel,
	linux-samsung-soc, linux-kernel

This patch adds device nodes for USBPHY to Exynos4x12.

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

diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
index 01da194..9c3335b 100644
--- a/arch/arm/boot/dts/exynos4x12.dtsi
+++ b/arch/arm/boot/dts/exynos4x12.dtsi
@@ -73,4 +73,22 @@
 		clock-names = "sclk_fimg2d", "fimg2d";
 		status = "disabled";
 	};
+
+	usbphy@125B00000 {
+		#address-cells = <1>;
+		#size-cells = <1>;
+		compatible = "samsung,exynos4x12-usb2phy";
+		reg = <0x125B0000 0x100>;
+		ranges;
+
+		clocks = <&clock 2>, <&clock 305>;
+		clock-names = "xusbxti", "otg";
+		status = "disabled";
+
+		usbphy-sys {
+			/* USB device and host PHY_CONTROL registers */
+			reg = <0x10020704 0xc>,
+			      <0x1001021c 0x4>;
+		};
+	};
 };
-- 
1.7.9.5


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

* Re: [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12
  2013-07-23 17:31 [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12 Dongjin Kim
@ 2013-07-24  5:12 ` Sachin Kamat
  2013-07-24  5:59   ` Dongjin Kim
  0 siblings, 1 reply; 4+ messages in thread
From: Sachin Kamat @ 2013-07-24  5:12 UTC (permalink / raw)
  To: Dongjin Kim
  Cc: Russell King, Kukjin Kim, Tomasz Figa, Thomas Abraham,
	Kyungmin Park, linux-arm-kernel, linux-samsung-soc, linux-kernel

Hi Dongjin,

On 23 July 2013 23:01, Dongjin Kim <tobetter@gmail.com> wrote:
> This patch adds device nodes for USBPHY to Exynos4x12.
>
> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
> ---
>  arch/arm/boot/dts/exynos4x12.dtsi |   18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
> index 01da194..9c3335b 100644
> --- a/arch/arm/boot/dts/exynos4x12.dtsi
> +++ b/arch/arm/boot/dts/exynos4x12.dtsi
> @@ -73,4 +73,22 @@
>                 clock-names = "sclk_fimg2d", "fimg2d";
>                 status = "disabled";
>         };
> +
> +       usbphy@125B00000 {

Extra 0 above.

> +               #address-cells = <1>;
> +               #size-cells = <1>;
> +               compatible = "samsung,exynos4x12-usb2phy";
> +               reg = <0x125B0000 0x100>;
> +               ranges;
> +
> +               clocks = <&clock 2>, <&clock 305>;
> +               clock-names = "xusbxti", "otg";
> +               status = "disabled";
> +
> +               usbphy-sys {
> +                       /* USB device and host PHY_CONTROL registers */
> +                       reg = <0x10020704 0xc>,
> +                             <0x1001021c 0x4>;
> +               };
> +       };
>  };

Please add this node after tmu node (satisfies alphabetical order as
well as increasing address value)


-- 
With warm regards,
Sachin

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

* Re: [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12
  2013-07-24  5:12 ` Sachin Kamat
@ 2013-07-24  5:59   ` Dongjin Kim
  2013-07-24  6:16     ` Sachin Kamat
  0 siblings, 1 reply; 4+ messages in thread
From: Dongjin Kim @ 2013-07-24  5:59 UTC (permalink / raw)
  To: Sachin Kamat
  Cc: Russell King, Kukjin Kim, Tomasz Figa, Thomas Abraham,
	Kyungmin Park, linux-arm-kernel, linux-samsung-soc, linux-kernel

Thanks Sachin,

On Wed, Jul 24, 2013 at 2:12 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
> Hi Dongjin,
>
> On 23 July 2013 23:01, Dongjin Kim <tobetter@gmail.com> wrote:
>> This patch adds device nodes for USBPHY to Exynos4x12.
>>
>> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
>> ---
>>  arch/arm/boot/dts/exynos4x12.dtsi |   18 ++++++++++++++++++
>>  1 file changed, 18 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
>> index 01da194..9c3335b 100644
>> --- a/arch/arm/boot/dts/exynos4x12.dtsi
>> +++ b/arch/arm/boot/dts/exynos4x12.dtsi
>> @@ -73,4 +73,22 @@
>>                 clock-names = "sclk_fimg2d", "fimg2d";
>>                 status = "disabled";
>>         };
>> +
>> +       usbphy@125B00000 {
>
> Extra 0 above.
>
This is my bad, I will fix.

>> +               #address-cells = <1>;
>> +               #size-cells = <1>;
>> +               compatible = "samsung,exynos4x12-usb2phy";
>> +               reg = <0x125B0000 0x100>;
>> +               ranges;
>> +
>> +               clocks = <&clock 2>, <&clock 305>;
>> +               clock-names = "xusbxti", "otg";
>> +               status = "disabled";
>> +
>> +               usbphy-sys {
>> +                       /* USB device and host PHY_CONTROL registers */
>> +                       reg = <0x10020704 0xc>,
>> +                             <0x1001021c 0x4>;
>> +               };
>> +       };
>>  };
>
> Please add this node after tmu node (satisfies alphabetical order as
> well as increasing address value)
No tmu node in exynos4x12.dtsi in mainline v3.11-rc2 yet, even in
Kukjin's branch.

>
>
> --
> With warm regards,
> Sachin

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

* Re: [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12
  2013-07-24  5:59   ` Dongjin Kim
@ 2013-07-24  6:16     ` Sachin Kamat
  0 siblings, 0 replies; 4+ messages in thread
From: Sachin Kamat @ 2013-07-24  6:16 UTC (permalink / raw)
  To: Dongjin Kim
  Cc: Russell King, Kukjin Kim, Tomasz Figa, Thomas Abraham,
	Kyungmin Park, linux-arm-kernel, linux-samsung-soc, linux-kernel

On 24 July 2013 11:29, Dongjin Kim <tobetter@gmail.com> wrote:
> Thanks Sachin,
>
> On Wed, Jul 24, 2013 at 2:12 PM, Sachin Kamat <sachin.kamat@linaro.org> wrote:
>> Hi Dongjin,
>>
>> On 23 July 2013 23:01, Dongjin Kim <tobetter@gmail.com> wrote:
>>> This patch adds device nodes for USBPHY to Exynos4x12.
>>>
>>> Signed-off-by: Dongjin Kim <tobetter@gmail.com>
>>> ---
>>>  arch/arm/boot/dts/exynos4x12.dtsi |   18 ++++++++++++++++++
>>>  1 file changed, 18 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/exynos4x12.dtsi b/arch/arm/boot/dts/exynos4x12.dtsi
>>> index 01da194..9c3335b 100644
>>> --- a/arch/arm/boot/dts/exynos4x12.dtsi
>>> +++ b/arch/arm/boot/dts/exynos4x12.dtsi
>>> @@ -73,4 +73,22 @@
>>>                 clock-names = "sclk_fimg2d", "fimg2d";
>>>                 status = "disabled";
>>>         };
>>> +
>>> +       usbphy@125B00000 {
>>
>> Extra 0 above.
>>
> This is my bad, I will fix.
>
>>> +               #address-cells = <1>;
>>> +               #size-cells = <1>;
>>> +               compatible = "samsung,exynos4x12-usb2phy";
>>> +               reg = <0x125B0000 0x100>;
>>> +               ranges;
>>> +
>>> +               clocks = <&clock 2>, <&clock 305>;
>>> +               clock-names = "xusbxti", "otg";
>>> +               status = "disabled";
>>> +
>>> +               usbphy-sys {
>>> +                       /* USB device and host PHY_CONTROL registers */
>>> +                       reg = <0x10020704 0xc>,
>>> +                             <0x1001021c 0x4>;
>>> +               };
>>> +       };
>>>  };
>>
>> Please add this node after tmu node (satisfies alphabetical order as
>> well as increasing address value)
> No tmu node in exynos4x12.dtsi in mainline v3.11-rc2 yet, even in
> Kukjin's branch.

Sorry, that patch is not yet applied to for-next. I will re-send
again. You may ignore this comment for now.

-- 
With warm regards,
Sachin

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

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

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-23 17:31 [PATCH] ARM: dts: Add USBPHY nodes to Exynos4x12 Dongjin Kim
2013-07-24  5:12 ` Sachin Kamat
2013-07-24  5:59   ` Dongjin Kim
2013-07-24  6:16     ` Sachin Kamat

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