diff mbox

[v2,4/4] ARM: dts: Add adc and thermistors for exynos5250-snow

Message ID 1363207202-10858-5-git-send-email-dianders@chromium.org (mailing list archive)
State New, archived
Headers show

Commit Message

Doug Anderson March 13, 2013, 8:40 p.m. UTC
Hook up the exynos5250-snow thermistors via the device tree now that
there's a driver available to use them.

Signed-off-by: Doug Anderson <dianders@chromium.org>
---
Changes in v2:
- Match 'uV' -> 'uv' change in Naveen's bindings.

 arch/arm/boot/dts/cros5250-common.dtsi |  4 ++++
 arch/arm/boot/dts/exynos5250-snow.dts  | 31 +++++++++++++++++++++++++++++++
 2 files changed, 35 insertions(+)

Comments

Naveen Krishna Ch March 15, 2013, 10:24 a.m. UTC | #1
Doug,

On 14 March 2013 02:10, Doug Anderson <dianders@chromium.org> wrote:
> Hook up the exynos5250-snow thermistors via the device tree now that
> there's a driver available to use them.
>
> Signed-off-by: Doug Anderson <dianders@chromium.org>
> ---
> Changes in v2:
> - Match 'uV' -> 'uv' change in Naveen's bindings.
>
>  arch/arm/boot/dts/cros5250-common.dtsi |  4 ++++
>  arch/arm/boot/dts/exynos5250-snow.dts  | 31 +++++++++++++++++++++++++++++++
>  2 files changed, 35 insertions(+)
>
> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
> index 46c0980..f7cc835 100644
> --- a/arch/arm/boot/dts/cros5250-common.dtsi
> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
> @@ -167,6 +167,10 @@
>                 status = "disabled";
>         };
>
> +       adc@12D10000 {
> +               vdd-supply = <&buck5_reg>;
> +       };
> +
>         hdmi {
>                 hpd-gpio = <&gpx3 7 0xf 1 3>;
>         };
> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
> index 17dd951..b78da7c 100644
> --- a/arch/arm/boot/dts/exynos5250-snow.dts
> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
> @@ -40,4 +40,35 @@
>                                 <&gpc4 5 2 3 0>, <&gpc4 6 2 3 0>;
>                 };
>         };
> +
> +       adc@12D10000 {
> +               ncp15wb473@3 {
> +                       compatible = "ntc,ncp15wb473";
> +                       pullup-uv = <1800000>;
> +                       pullup-ohm = <47000>;
> +                       pulldown-ohm = <0>;
> +                       io-channels = <&adc 3>;
> +               };
> +               ncp15wb473@4 {
> +                       compatible = "ntc,ncp15wb473";
> +                       pullup-uv = <1800000>;
> +                       pullup-ohm = <47000>;
> +                       pulldown-ohm = <0>;
> +                       io-channels = <&adc 4>;
> +               };
> +               ncp15wb473@5 {
> +                       compatible = "ntc,ncp15wb473";
> +                       pullup-uv = <1800000>;
> +                       pullup-ohm = <47000>;
> +                       pulldown-ohm = <0>;
> +                       io-channels = <&adc 5>;
> +               };
> +               ncp15wb473@6 {
> +                       compatible = "ntc,ncp15wb473";
> +                       pullup-uv = <1800000>;
> +                       pullup-ohm = <47000>;
> +                       pulldown-ohm = <0>;
> +                       io-channels = <&adc 6>;
> +               };
> +       };
>  };
Thanks for the upload, i've tested with a similar changes.
> --
> 1.8.1.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
Naveen Krishna Ch March 15, 2013, 3:42 p.m. UTC | #2
On 15 March 2013 15:54, Naveen Krishna Ch <naveenkrishna.ch@gmail.com> wrote:
> Doug,
>
> On 14 March 2013 02:10, Doug Anderson <dianders@chromium.org> wrote:
>> Hook up the exynos5250-snow thermistors via the device tree now that
>> there's a driver available to use them.
>>
>> Signed-off-by: Doug Anderson <dianders@chromium.org>
Tested-by: Naveen Krishna Chatradhi <ch.naveen@samsung.com>
used these changes to test on snow before submitting the ADC and NTC
driver changes.
>> ---
>> Changes in v2:
>> - Match 'uV' -> 'uv' change in Naveen's bindings.
>>
>>  arch/arm/boot/dts/cros5250-common.dtsi |  4 ++++
>>  arch/arm/boot/dts/exynos5250-snow.dts  | 31 +++++++++++++++++++++++++++++++
>>  2 files changed, 35 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
>> index 46c0980..f7cc835 100644
>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>> @@ -167,6 +167,10 @@
>>                 status = "disabled";
>>         };
>>
>> +       adc@12D10000 {
>> +               vdd-supply = <&buck5_reg>;
>> +       };
>> +
>>         hdmi {
>>                 hpd-gpio = <&gpx3 7 0xf 1 3>;
>>         };
>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
>> index 17dd951..b78da7c 100644
>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>> @@ -40,4 +40,35 @@
>>                                 <&gpc4 5 2 3 0>, <&gpc4 6 2 3 0>;
>>                 };
>>         };
>> +
>> +       adc@12D10000 {
>> +               ncp15wb473@3 {
>> +                       compatible = "ntc,ncp15wb473";
>> +                       pullup-uv = <1800000>;
>> +                       pullup-ohm = <47000>;
>> +                       pulldown-ohm = <0>;
>> +                       io-channels = <&adc 3>;
>> +               };
>> +               ncp15wb473@4 {
>> +                       compatible = "ntc,ncp15wb473";
>> +                       pullup-uv = <1800000>;
>> +                       pullup-ohm = <47000>;
>> +                       pulldown-ohm = <0>;
>> +                       io-channels = <&adc 4>;
>> +               };
>> +               ncp15wb473@5 {
>> +                       compatible = "ntc,ncp15wb473";
>> +                       pullup-uv = <1800000>;
>> +                       pullup-ohm = <47000>;
>> +                       pulldown-ohm = <0>;
>> +                       io-channels = <&adc 5>;
>> +               };
>> +               ncp15wb473@6 {
>> +                       compatible = "ntc,ncp15wb473";
>> +                       pullup-uv = <1800000>;
>> +                       pullup-ohm = <47000>;
>> +                       pulldown-ohm = <0>;
>> +                       io-channels = <&adc 6>;
>> +               };
>> +       };
>>  };
> Thanks for the upload, i've tested with a similar changes.
>> --
>> 1.8.1.3
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> Please read the FAQ at  http://www.tux.org/lkml/
>
>
>
> --
> Shine bright,
> (: Nav :)



--
Shine bright,
(: Nav :)
Naveen Krishna Ch March 27, 2013, 6:36 p.m. UTC | #3
On 15 March 2013 08:42, Naveen Krishna Ch <naveenkrishna.ch@gmail.com> wrote:
> On 15 March 2013 15:54, Naveen Krishna Ch <naveenkrishna.ch@gmail.com> wrote:
>> Doug,
>>
>> On 14 March 2013 02:10, Doug Anderson <dianders@chromium.org> wrote:
>>> Hook up the exynos5250-snow thermistors via the device tree now that
>>> there's a driver available to use them.
>>>
>>> Signed-off-by: Doug Anderson <dianders@chromium.org>
> Tested-by: Naveen Krishna Chatradhi <ch.naveen@samsung.com>
> used these changes to test on snow before submitting the ADC and NTC
> driver changes.
Kukjin, Any update on this patch. you want us to rebase it ?

>>> ---
>>> Changes in v2:
>>> - Match 'uV' -> 'uv' change in Naveen's bindings.
>>>
>>>  arch/arm/boot/dts/cros5250-common.dtsi |  4 ++++
>>>  arch/arm/boot/dts/exynos5250-snow.dts  | 31 +++++++++++++++++++++++++++++++
>>>  2 files changed, 35 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
>>> index 46c0980..f7cc835 100644
>>> --- a/arch/arm/boot/dts/cros5250-common.dtsi
>>> +++ b/arch/arm/boot/dts/cros5250-common.dtsi
>>> @@ -167,6 +167,10 @@
>>>                 status = "disabled";
>>>         };
>>>
>>> +       adc@12D10000 {
>>> +               vdd-supply = <&buck5_reg>;
>>> +       };
>>> +
>>>         hdmi {
>>>                 hpd-gpio = <&gpx3 7 0xf 1 3>;
>>>         };
>>> diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
>>> index 17dd951..b78da7c 100644
>>> --- a/arch/arm/boot/dts/exynos5250-snow.dts
>>> +++ b/arch/arm/boot/dts/exynos5250-snow.dts
>>> @@ -40,4 +40,35 @@
>>>                                 <&gpc4 5 2 3 0>, <&gpc4 6 2 3 0>;
>>>                 };
>>>         };
>>> +
>>> +       adc@12D10000 {
>>> +               ncp15wb473@3 {
>>> +                       compatible = "ntc,ncp15wb473";
>>> +                       pullup-uv = <1800000>;
>>> +                       pullup-ohm = <47000>;
>>> +                       pulldown-ohm = <0>;
>>> +                       io-channels = <&adc 3>;
>>> +               };
>>> +               ncp15wb473@4 {
>>> +                       compatible = "ntc,ncp15wb473";
>>> +                       pullup-uv = <1800000>;
>>> +                       pullup-ohm = <47000>;
>>> +                       pulldown-ohm = <0>;
>>> +                       io-channels = <&adc 4>;
>>> +               };
>>> +               ncp15wb473@5 {
>>> +                       compatible = "ntc,ncp15wb473";
>>> +                       pullup-uv = <1800000>;
>>> +                       pullup-ohm = <47000>;
>>> +                       pulldown-ohm = <0>;
>>> +                       io-channels = <&adc 5>;
>>> +               };
>>> +               ncp15wb473@6 {
>>> +                       compatible = "ntc,ncp15wb473";
>>> +                       pullup-uv = <1800000>;
>>> +                       pullup-ohm = <47000>;
>>> +                       pulldown-ohm = <0>;
>>> +                       io-channels = <&adc 6>;
>>> +               };
>>> +       };
>>>  };
>> Thanks for the upload, i've tested with a similar changes.
>>> --
>>> 1.8.1.3
>>>
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>> Please read the FAQ at  http://www.tux.org/lkml/
>>
>>
>>
>> --
>> Shine bright,
>> (: Nav :)
>
>
>
> --
> Shine bright,
> (: Nav :)



--
Shine bright,
(: Nav :)
diff mbox

Patch

diff --git a/arch/arm/boot/dts/cros5250-common.dtsi b/arch/arm/boot/dts/cros5250-common.dtsi
index 46c0980..f7cc835 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -167,6 +167,10 @@ 
 		status = "disabled";
 	};
 
+	adc@12D10000 {
+		vdd-supply = <&buck5_reg>;
+	};
+
 	hdmi {
 		hpd-gpio = <&gpx3 7 0xf 1 3>;
 	};
diff --git a/arch/arm/boot/dts/exynos5250-snow.dts b/arch/arm/boot/dts/exynos5250-snow.dts
index 17dd951..b78da7c 100644
--- a/arch/arm/boot/dts/exynos5250-snow.dts
+++ b/arch/arm/boot/dts/exynos5250-snow.dts
@@ -40,4 +40,35 @@ 
 				<&gpc4 5 2 3 0>, <&gpc4 6 2 3 0>;
 		};
 	};
+
+	adc@12D10000 {
+		ncp15wb473@3 {
+			compatible = "ntc,ncp15wb473";
+			pullup-uv = <1800000>;
+			pullup-ohm = <47000>;
+			pulldown-ohm = <0>;
+			io-channels = <&adc 3>;
+		};
+		ncp15wb473@4 {
+			compatible = "ntc,ncp15wb473";
+			pullup-uv = <1800000>;
+			pullup-ohm = <47000>;
+			pulldown-ohm = <0>;
+			io-channels = <&adc 4>;
+		};
+		ncp15wb473@5 {
+			compatible = "ntc,ncp15wb473";
+			pullup-uv = <1800000>;
+			pullup-ohm = <47000>;
+			pulldown-ohm = <0>;
+			io-channels = <&adc 5>;
+		};
+		ncp15wb473@6 {
+			compatible = "ntc,ncp15wb473";
+			pullup-uv = <1800000>;
+			pullup-ohm = <47000>;
+			pulldown-ohm = <0>;
+			io-channels = <&adc 6>;
+		};
+	};
 };