* [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. This series is v2, previous version patches: [RFC 1/9]: http://thread.gmane.org/gmane.linux.power-management.general/31056 [v1]: http://thread.gmane.org/gmane.linux.ports.tegra/11713/ [v2]: http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg465561.html Changes from v2: 1. remove the label "nct1008". Changes from v1: 1. add vendor strings "onnn" for nct1008 node. Changes from RFC: 1. Enable it for Tegra114 Dalmore. Wei Ni (2): ARM: dt: t30 cardhu: add dt entry for nct1008 ARM: dt: t114 dalmore: add dt entry for nct1008 arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 2 files changed, 14 insertions(+) -- 1.7.9.5 ^ permalink raw reply [flat|nested] 28+ messages in thread
* [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. This series is v2, previous version patches: [RFC 1/9]: http://thread.gmane.org/gmane.linux.power-management.general/31056 [v1]: http://thread.gmane.org/gmane.linux.ports.tegra/11713/ [v2]: http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg465561.html Changes from v2: 1. remove the label "nct1008". Changes from v1: 1. add vendor strings "onnn" for nct1008 node. Changes from RFC: 1. Enable it for Tegra114 Dalmore. Wei Ni (2): ARM: dt: t30 cardhu: add dt entry for nct1008 ARM: dt: t114 dalmore: add dt entry for nct1008 arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 2 files changed, 14 insertions(+) -- 1.7.9.5 _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. This series is v2, previous version patches: [RFC 1/9]: http://thread.gmane.org/gmane.linux.power-management.general/31056 [v1]: http://thread.gmane.org/gmane.linux.ports.tegra/11713/ [v2]: http://www.mail-archive.com/linux-kernel@vger.kernel.org/msg465561.html Changes from v2: 1. remove the label "nct1008". Changes from v1: 1. add vendor strings "onnn" for nct1008 node. Changes from RFC: 1. Enable it for Tegra114 Dalmore. Wei Ni (2): ARM: dt: t30 cardhu: add dt entry for nct1008 ARM: dt: t114 dalmore: add dt entry for nct1008 arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 2 files changed, 14 insertions(+) -- 1.7.9.5 ^ permalink raw reply [flat|nested] 28+ messages in thread
* [PATCH v3 1/2] ARM: dt: t30 cardhu: add dt entry for nct1008 2013-07-12 7:49 ` Wei Ni (?) @ 2013-07-12 7:49 ` Wei Ni -1 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t30 cardhu. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi index f65b53d..e5759ca 100644 --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi @@ -286,6 +286,13 @@ }; }; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(CC, 2) IRQ_TYPE_LEVEL_LOW>; + }; }; spi@7000da00 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 28+ messages in thread
* [lm-sensors] [PATCH v3 1/2] ARM: dt: t30 cardhu: add dt entry for nct1008 @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t30 cardhu. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi index f65b53d..e5759ca 100644 --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi @@ -286,6 +286,13 @@ }; }; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(CC, 2) IRQ_TYPE_LEVEL_LOW>; + }; }; spi@7000da00 { -- 1.7.9.5 _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH v3 1/2] ARM: dt: t30 cardhu: add dt entry for nct1008 @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t30 cardhu. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra30-cardhu.dtsi | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi index f65b53d..e5759ca 100644 --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi @@ -286,6 +286,13 @@ }; }; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(CC, 2) IRQ_TYPE_LEVEL_LOW>; + }; }; spi@7000da00 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH v3 2/2] ARM: dt: t114 dalmore: add dt entry for nct1008 2013-07-12 7:49 ` Wei Ni (?) @ 2013-07-12 7:49 ` Wei Ni -1 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t114 dalmore. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts index cb640eb..47ec7eb 100644 --- a/arch/arm/boot/dts/tegra114-dalmore.dts +++ b/arch/arm/boot/dts/tegra114-dalmore.dts @@ -738,6 +738,13 @@ realtek,ldo1-en-gpios = <&gpio TEGRA_GPIO(V, 3) GPIO_ACTIVE_HIGH>; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(O, 4) IRQ_TYPE_LEVEL_LOW>; + }; }; i2c@7000d000 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 28+ messages in thread
* [lm-sensors] [PATCH v3 2/2] ARM: dt: t114 dalmore: add dt entry for nct1008 @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t114 dalmore. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts index cb640eb..47ec7eb 100644 --- a/arch/arm/boot/dts/tegra114-dalmore.dts +++ b/arch/arm/boot/dts/tegra114-dalmore.dts @@ -738,6 +738,13 @@ realtek,ldo1-en-gpios <&gpio TEGRA_GPIO(V, 3) GPIO_ACTIVE_HIGH>; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(O, 4) IRQ_TYPE_LEVEL_LOW>; + }; }; i2c@7000d000 { -- 1.7.9.5 _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply related [flat|nested] 28+ messages in thread
* [PATCH v3 2/2] ARM: dt: t114 dalmore: add dt entry for nct1008 @ 2013-07-12 7:49 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-12 7:49 UTC (permalink / raw) To: khali, swarren, linux, thierry.reding Cc: lm-sensors, linux-kernel, linux-tegra, Wei Ni Enable thermal sensor nct1008 for t114 dalmore. Signed-off-by: Wei Ni <wni@nvidia.com> --- arch/arm/boot/dts/tegra114-dalmore.dts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts index cb640eb..47ec7eb 100644 --- a/arch/arm/boot/dts/tegra114-dalmore.dts +++ b/arch/arm/boot/dts/tegra114-dalmore.dts @@ -738,6 +738,13 @@ realtek,ldo1-en-gpios = <&gpio TEGRA_GPIO(V, 3) GPIO_ACTIVE_HIGH>; }; + + nct1008 { + compatible = "onnn,nct1008"; + reg = <0x4c>; + interrupt-parent = <&gpio>; + interrupts = <TEGRA_GPIO(O, 4) IRQ_TYPE_LEVEL_LOW>; + }; }; i2c@7000d000 { -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 28+ messages in thread
[parent not found: <1373615364-18559-1-git-send-email-wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-12 7:49 ` Wei Ni (?) @ 2013-07-15 16:48 ` Stephen Warren -1 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 16:48 UTC (permalink / raw) To: Wei Ni Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/12/2013 01:49 AM, Wei Ni wrote: > Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. I have applied the series to Tegra's for-3.12/dt branch. ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-15 16:48 ` Stephen Warren 0 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 16:48 UTC (permalink / raw) To: Wei Ni Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/12/2013 01:49 AM, Wei Ni wrote: > Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. I have applied the series to Tegra's for-3.12/dt branch. _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-15 16:48 ` Stephen Warren 0 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 16:48 UTC (permalink / raw) To: Wei Ni Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/12/2013 01:49 AM, Wei Ni wrote: > Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. I have applied the series to Tegra's for-3.12/dt branch. ^ permalink raw reply [flat|nested] 28+ messages in thread
[parent not found: <51E427F1.9010106-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-15 16:48 ` Stephen Warren (?) @ 2013-07-15 17:13 ` Stephen Warren -1 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 17:13 UTC (permalink / raw) To: Wei Ni Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/15/2013 10:48 AM, Stephen Warren wrote: > On 07/12/2013 01:49 AM, Wei Ni wrote: >> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. > > I have applied the series to Tegra's for-3.12/dt branch. Actually, I see the following on Dalmore (although Cardhu works): [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) [ 29.453930] lm90 0-004c: Failed to read convrate register! [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) [ 29.465408] lm90 0-004c: Initialization failed! So, there's something wrong here, and I've dropped the patch for Dalmore. Does this patch actually work for you? ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-15 17:13 ` Stephen Warren 0 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 17:13 UTC (permalink / raw) To: Wei Ni Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/15/2013 10:48 AM, Stephen Warren wrote: > On 07/12/2013 01:49 AM, Wei Ni wrote: >> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. > > I have applied the series to Tegra's for-3.12/dt branch. Actually, I see the following on Dalmore (although Cardhu works): [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) [ 29.453930] lm90 0-004c: Failed to read convrate register! [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) [ 29.465408] lm90 0-004c: Initialization failed! So, there's something wrong here, and I've dropped the patch for Dalmore. Does this patch actually work for you? _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-15 17:13 ` Stephen Warren 0 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-15 17:13 UTC (permalink / raw) To: Wei Ni Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/15/2013 10:48 AM, Stephen Warren wrote: > On 07/12/2013 01:49 AM, Wei Ni wrote: >> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. > > I have applied the series to Tegra's for-3.12/dt branch. Actually, I see the following on Dalmore (although Cardhu works): [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) [ 29.453930] lm90 0-004c: Failed to read convrate register! [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) [ 29.465408] lm90 0-004c: Initialization failed! So, there's something wrong here, and I've dropped the patch for Dalmore. Does this patch actually work for you? ^ permalink raw reply [flat|nested] 28+ messages in thread
[parent not found: <51E42DB0.8090006-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-15 17:13 ` Stephen Warren (?) @ 2013-07-16 11:14 ` Wei Ni -1 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-16 11:14 UTC (permalink / raw) To: Stephen Warren Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/16/2013 01:13 AM, Stephen Warren wrote: > On 07/15/2013 10:48 AM, Stephen Warren wrote: >> On 07/12/2013 01:49 AM, Wei Ni wrote: >>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >> >> I have applied the series to Tegra's for-3.12/dt branch. > > Actually, I see the following on Dalmore (although Cardhu works): > > [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) > [ 29.453930] lm90 0-004c: Failed to read convrate register! > [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) > [ 29.465408] lm90 0-004c: Initialization failed! > > So, there's something wrong here, and I've dropped the patch for > Dalmore. Does this patch actually work for you? > I test it with the Chrome u-boot, the lm90 works fine. But today, if I tried to use the uboot from git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the i2c can't work, the lm90 driver read register failed. I traced it, but can't find the root cause yet. ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-16 11:14 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-16 11:14 UTC (permalink / raw) To: Stephen Warren Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/16/2013 01:13 AM, Stephen Warren wrote: > On 07/15/2013 10:48 AM, Stephen Warren wrote: >> On 07/12/2013 01:49 AM, Wei Ni wrote: >>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >> >> I have applied the series to Tegra's for-3.12/dt branch. > > Actually, I see the following on Dalmore (although Cardhu works): > > [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) > [ 29.453930] lm90 0-004c: Failed to read convrate register! > [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) > [ 29.465408] lm90 0-004c: Initialization failed! > > So, there's something wrong here, and I've dropped the patch for > Dalmore. Does this patch actually work for you? > I test it with the Chrome u-boot, the lm90 works fine. But today, if I tried to use the uboot from git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the i2c can't work, the lm90 driver read register failed. I traced it, but can't find the root cause yet. _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-16 11:14 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-16 11:14 UTC (permalink / raw) To: Stephen Warren Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/16/2013 01:13 AM, Stephen Warren wrote: > On 07/15/2013 10:48 AM, Stephen Warren wrote: >> On 07/12/2013 01:49 AM, Wei Ni wrote: >>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >> >> I have applied the series to Tegra's for-3.12/dt branch. > > Actually, I see the following on Dalmore (although Cardhu works): > > [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) > [ 29.453930] lm90 0-004c: Failed to read convrate register! > [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) > [ 29.465408] lm90 0-004c: Initialization failed! > > So, there's something wrong here, and I've dropped the patch for > Dalmore. Does this patch actually work for you? > I test it with the Chrome u-boot, the lm90 works fine. But today, if I tried to use the uboot from git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the i2c can't work, the lm90 driver read register failed. I traced it, but can't find the root cause yet. ^ permalink raw reply [flat|nested] 28+ messages in thread
[parent not found: <51E52B28.9090708-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-16 11:14 ` Wei Ni (?) @ 2013-07-17 8:17 ` Wei Ni -1 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 8:17 UTC (permalink / raw) To: Stephen Warren Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/16/2013 07:14 PM, Wei Ni wrote: > On 07/16/2013 01:13 AM, Stephen Warren wrote: >> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>> >>> I have applied the series to Tegra's for-3.12/dt branch. >> >> Actually, I see the following on Dalmore (although Cardhu works): >> >> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >> [ 29.453930] lm90 0-004c: Failed to read convrate register! >> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >> [ 29.465408] lm90 0-004c: Initialization failed! >> >> So, there's something wrong here, and I've dropped the patch for >> Dalmore. Does this patch actually work for you? >> > I test it with the Chrome u-boot, the lm90 works fine. > But today, if I tried to use the uboot from > git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the > i2c can't work, the lm90 driver read register failed. > I traced it, but can't find the root cause yet. Hi, Stephen I found the root cause. In the Chrome u-boot, it will turn on the regulator for the lm90, but in the upstream u-boot, it didn't do it. I think this regulator should be handled in the kernel driver, but it seems the regulator palmas tps65913 didn't be supported on the dalmore yet. So may be we have to wait the palmas ready on dalmore. Do you have any suggestions? Thanks. Wei. > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 8:17 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 8:17 UTC (permalink / raw) To: Stephen Warren Cc: khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/16/2013 07:14 PM, Wei Ni wrote: > On 07/16/2013 01:13 AM, Stephen Warren wrote: >> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>> >>> I have applied the series to Tegra's for-3.12/dt branch. >> >> Actually, I see the following on Dalmore (although Cardhu works): >> >> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >> [ 29.453930] lm90 0-004c: Failed to read convrate register! >> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >> [ 29.465408] lm90 0-004c: Initialization failed! >> >> So, there's something wrong here, and I've dropped the patch for >> Dalmore. Does this patch actually work for you? >> > I test it with the Chrome u-boot, the lm90 works fine. > But today, if I tried to use the uboot from > git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the > i2c can't work, the lm90 driver read register failed. > I traced it, but can't find the root cause yet. Hi, Stephen I found the root cause. In the Chrome u-boot, it will turn on the regulator for the lm90, but in the upstream u-boot, it didn't do it. I think this regulator should be handled in the kernel driver, but it seems the regulator palmas tps65913 didn't be supported on the dalmore yet. So may be we have to wait the palmas ready on dalmore. Do you have any suggestions? Thanks. Wei. > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 8:17 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 8:17 UTC (permalink / raw) To: Stephen Warren Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/16/2013 07:14 PM, Wei Ni wrote: > On 07/16/2013 01:13 AM, Stephen Warren wrote: >> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>> >>> I have applied the series to Tegra's for-3.12/dt branch. >> >> Actually, I see the following on Dalmore (although Cardhu works): >> >> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >> [ 29.453930] lm90 0-004c: Failed to read convrate register! >> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >> [ 29.465408] lm90 0-004c: Initialization failed! >> >> So, there's something wrong here, and I've dropped the patch for >> Dalmore. Does this patch actually work for you? >> > I test it with the Chrome u-boot, the lm90 works fine. > But today, if I tried to use the uboot from > git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the > i2c can't work, the lm90 driver read register failed. > I traced it, but can't find the root cause yet. Hi, Stephen I found the root cause. In the Chrome u-boot, it will turn on the regulator for the lm90, but in the upstream u-boot, it didn't do it. I think this regulator should be handled in the kernel driver, but it seems the regulator palmas tps65913 didn't be supported on the dalmore yet. So may be we have to wait the palmas ready on dalmore. Do you have any suggestions? Thanks. Wei. > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-17 8:17 ` Wei Ni @ 2013-07-17 8:23 ` Mikko Perttunen -1 siblings, 0 replies; 28+ messages in thread From: Mikko Perttunen @ 2013-07-17 8:23 UTC (permalink / raw) To: Wei Ni Cc: Stephen Warren, khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/17/2013 11:17 AM, Wei Ni wrote: > On 07/16/2013 07:14 PM, Wei Ni wrote: >> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>> >>>> I have applied the series to Tegra's for-3.12/dt branch. >>> >>> Actually, I see the following on Dalmore (although Cardhu works): >>> >>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>> [ 29.465408] lm90 0-004c: Initialization failed! >>> >>> So, there's something wrong here, and I've dropped the patch for >>> Dalmore. Does this patch actually work for you? >>> >> I test it with the Chrome u-boot, the lm90 works fine. >> But today, if I tried to use the uboot from >> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >> i2c can't work, the lm90 driver read register failed. >> I traced it, but can't find the root cause yet. > > Hi, Stephen > I found the root cause. > In the Chrome u-boot, it will turn on the regulator for the lm90, but in > the upstream u-boot, it didn't do it. > I think this regulator should be handled in the kernel driver, but it > seems the regulator palmas tps65913 didn't be supported on the dalmore yet. > So may be we have to wait the palmas ready on dalmore. A patch to enable tps65913 on dalmore has already been posted a couple of days ago by Laxman Dewangan, so this shouldn't be too problematic. > > Do you have any suggestions? > > Thanks. > Wei. > >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 8:23 ` Mikko Perttunen 0 siblings, 0 replies; 28+ messages in thread From: Mikko Perttunen @ 2013-07-17 8:23 UTC (permalink / raw) To: Wei Ni Cc: Stephen Warren, khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/17/2013 11:17 AM, Wei Ni wrote: > On 07/16/2013 07:14 PM, Wei Ni wrote: >> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>> >>>> I have applied the series to Tegra's for-3.12/dt branch. >>> >>> Actually, I see the following on Dalmore (although Cardhu works): >>> >>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>> [ 29.465408] lm90 0-004c: Initialization failed! >>> >>> So, there's something wrong here, and I've dropped the patch for >>> Dalmore. Does this patch actually work for you? >>> >> I test it with the Chrome u-boot, the lm90 works fine. >> But today, if I tried to use the uboot from >> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >> i2c can't work, the lm90 driver read register failed. >> I traced it, but can't find the root cause yet. > > Hi, Stephen > I found the root cause. > In the Chrome u-boot, it will turn on the regulator for the lm90, but in > the upstream u-boot, it didn't do it. > I think this regulator should be handled in the kernel driver, but it > seems the regulator palmas tps65913 didn't be supported on the dalmore yet. > So may be we have to wait the palmas ready on dalmore. A patch to enable tps65913 on dalmore has already been posted a couple of days ago by Laxman Dewangan, so this shouldn't be too problematic. > > Do you have any suggestions? > > Thanks. > Wei. > >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
[parent not found: <51E6549A.1010006-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-17 8:23 ` [lm-sensors] " Mikko Perttunen (?) @ 2013-07-17 9:08 ` Wei Ni -1 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 9:08 UTC (permalink / raw) To: Mikko Perttunen Cc: Stephen Warren, khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/17/2013 04:23 PM, Mikko Perttunen wrote: > On 07/17/2013 11:17 AM, Wei Ni wrote: >> On 07/16/2013 07:14 PM, Wei Ni wrote: >>> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>>> >>>>> I have applied the series to Tegra's for-3.12/dt branch. >>>> >>>> Actually, I see the following on Dalmore (although Cardhu works): >>>> >>>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>>> [ 29.465408] lm90 0-004c: Initialization failed! >>>> >>>> So, there's something wrong here, and I've dropped the patch for >>>> Dalmore. Does this patch actually work for you? >>>> >>> I test it with the Chrome u-boot, the lm90 works fine. >>> But today, if I tried to use the uboot from >>> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >>> i2c can't work, the lm90 driver read register failed. >>> I traced it, but can't find the root cause yet. >> >> Hi, Stephen >> I found the root cause. >> In the Chrome u-boot, it will turn on the regulator for the lm90, but in >> the upstream u-boot, it didn't do it. >> I think this regulator should be handled in the kernel driver, but it >> seems the regulator palmas tps65913 didn't be supported on the dalmore yet. >> So may be we have to wait the palmas ready on dalmore. > > A patch to enable tps65913 on dalmore has already been posted a couple > of days > ago by Laxman Dewangan, so this shouldn't be too problematic. Oh, great, I will look it. > >> >> Do you have any suggestions? >> >> Thanks. >> Wei. >> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >>> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 9:08 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 9:08 UTC (permalink / raw) To: Mikko Perttunen Cc: Stephen Warren, khali-PUYAD+kWke1g9hUCZPvPmw, linux-0h96xk9xTtrk1uMJSBkQmQ, thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, lm-sensors-GZX6beZjE8VD60Wz+7aTrA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA On 07/17/2013 04:23 PM, Mikko Perttunen wrote: > On 07/17/2013 11:17 AM, Wei Ni wrote: >> On 07/16/2013 07:14 PM, Wei Ni wrote: >>> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>>> >>>>> I have applied the series to Tegra's for-3.12/dt branch. >>>> >>>> Actually, I see the following on Dalmore (although Cardhu works): >>>> >>>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>>> [ 29.465408] lm90 0-004c: Initialization failed! >>>> >>>> So, there's something wrong here, and I've dropped the patch for >>>> Dalmore. Does this patch actually work for you? >>>> >>> I test it with the Chrome u-boot, the lm90 works fine. >>> But today, if I tried to use the uboot from >>> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >>> i2c can't work, the lm90 driver read register failed. >>> I traced it, but can't find the root cause yet. >> >> Hi, Stephen >> I found the root cause. >> In the Chrome u-boot, it will turn on the regulator for the lm90, but in >> the upstream u-boot, it didn't do it. >> I think this regulator should be handled in the kernel driver, but it >> seems the regulator palmas tps65913 didn't be supported on the dalmore yet. >> So may be we have to wait the palmas ready on dalmore. > > A patch to enable tps65913 on dalmore has already been posted a couple > of days > ago by Laxman Dewangan, so this shouldn't be too problematic. Oh, great, I will look it. > >> >> Do you have any suggestions? >> >> Thanks. >> Wei. >> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 9:08 ` Wei Ni 0 siblings, 0 replies; 28+ messages in thread From: Wei Ni @ 2013-07-17 9:08 UTC (permalink / raw) To: Mikko Perttunen Cc: Stephen Warren, khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/17/2013 04:23 PM, Mikko Perttunen wrote: > On 07/17/2013 11:17 AM, Wei Ni wrote: >> On 07/16/2013 07:14 PM, Wei Ni wrote: >>> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>>> >>>>> I have applied the series to Tegra's for-3.12/dt branch. >>>> >>>> Actually, I see the following on Dalmore (although Cardhu works): >>>> >>>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>>> [ 29.465408] lm90 0-004c: Initialization failed! >>>> >>>> So, there's something wrong here, and I've dropped the patch for >>>> Dalmore. Does this patch actually work for you? >>>> >>> I test it with the Chrome u-boot, the lm90 works fine. >>> But today, if I tried to use the uboot from >>> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >>> i2c can't work, the lm90 driver read register failed. >>> I traced it, but can't find the root cause yet. >> >> Hi, Stephen >> I found the root cause. >> In the Chrome u-boot, it will turn on the regulator for the lm90, but in >> the upstream u-boot, it didn't do it. >> I think this regulator should be handled in the kernel driver, but it >> seems the regulator palmas tps65913 didn't be supported on the dalmore yet. >> So may be we have to wait the palmas ready on dalmore. > > A patch to enable tps65913 on dalmore has already been posted a couple > of days > ago by Laxman Dewangan, so this shouldn't be too problematic. Oh, great, I will look it. > >> >> Do you have any suggestions? >> >> Thanks. >> Wei. >> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. 2013-07-17 8:17 ` Wei Ni @ 2013-07-17 17:05 ` Stephen Warren -1 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-17 17:05 UTC (permalink / raw) To: Wei Ni Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/17/2013 02:17 AM, Wei Ni wrote: > On 07/16/2013 07:14 PM, Wei Ni wrote: >> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>> >>>> I have applied the series to Tegra's for-3.12/dt branch. >>> >>> Actually, I see the following on Dalmore (although Cardhu works): >>> >>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>> [ 29.465408] lm90 0-004c: Initialization failed! >>> >>> So, there's something wrong here, and I've dropped the patch for >>> Dalmore. Does this patch actually work for you? >>> >> I test it with the Chrome u-boot, the lm90 works fine. >> But today, if I tried to use the uboot from >> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >> i2c can't work, the lm90 driver read register failed. >> I traced it, but can't find the root cause yet. > > Hi, Stephen > I found the root cause. > In the Chrome u-boot, it will turn on the regulator for the lm90, but in > the upstream u-boot, it didn't do it. > I think this regulator should be handled in the kernel driver, but it > seems the regulator palmas tps65913 didn't be supported on the dalmore yet. > So may be we have to wait the palmas ready on dalmore. Yes, I agree. (although Laxman's regulator patch caused the SD card to stop working, so that needs to be fixed up, but hopefully it's a simple issue). ^ permalink raw reply [flat|nested] 28+ messages in thread
* Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114. @ 2013-07-17 17:05 ` Stephen Warren 0 siblings, 0 replies; 28+ messages in thread From: Stephen Warren @ 2013-07-17 17:05 UTC (permalink / raw) To: Wei Ni Cc: khali, linux, thierry.reding, lm-sensors, linux-kernel, linux-tegra On 07/17/2013 02:17 AM, Wei Ni wrote: > On 07/16/2013 07:14 PM, Wei Ni wrote: >> On 07/16/2013 01:13 AM, Stephen Warren wrote: >>> On 07/15/2013 10:48 AM, Stephen Warren wrote: >>>> On 07/12/2013 01:49 AM, Wei Ni wrote: >>>>> Enable thermal sensor lm90 for Tegra30 Cardhu and Tegra114 Dalmore. >>>> >>>> I have applied the series to Tegra's for-3.12/dt branch. >>> >>> Actually, I see the following on Dalmore (although Cardhu works): >>> >>> [ 29.448539] lm90 0-004c: Register 0x4 read failed (-121) >>> [ 29.453930] lm90 0-004c: Failed to read convrate register! >>> [ 29.460034] lm90 0-004c: Register 0x3 read failed (-121) >>> [ 29.465408] lm90 0-004c: Initialization failed! >>> >>> So, there's something wrong here, and I've dropped the patch for >>> Dalmore. Does this patch actually work for you? >>> >> I test it with the Chrome u-boot, the lm90 works fine. >> But today, if I tried to use the uboot from >> git://git.denx.de/u-boot-tegra.git, I can repo your errors, it seems the >> i2c can't work, the lm90 driver read register failed. >> I traced it, but can't find the root cause yet. > > Hi, Stephen > I found the root cause. > In the Chrome u-boot, it will turn on the regulator for the lm90, but in > the upstream u-boot, it didn't do it. > I think this regulator should be handled in the kernel driver, but it > seems the regulator palmas tps65913 didn't be supported on the dalmore yet. > So may be we have to wait the palmas ready on dalmore. Yes, I agree. (although Laxman's regulator patch caused the SD card to stop working, so that needs to be fixed up, but hopefully it's a simple issue). _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 28+ messages in thread
end of thread, other threads:[~2013-07-17 17:05 UTC | newest] Thread overview: 28+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2013-07-12 7:49 [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114 Wei Ni 2013-07-12 7:49 ` [lm-sensors] " Wei Ni 2013-07-12 7:49 ` Wei Ni 2013-07-12 7:49 ` [PATCH v3 1/2] ARM: dt: t30 cardhu: add dt entry for nct1008 Wei Ni 2013-07-12 7:49 ` [lm-sensors] " Wei Ni 2013-07-12 7:49 ` Wei Ni 2013-07-12 7:49 ` [PATCH v3 2/2] ARM: dt: t114 dalmore: " Wei Ni 2013-07-12 7:49 ` [lm-sensors] " Wei Ni 2013-07-12 7:49 ` Wei Ni [not found] ` <1373615364-18559-1-git-send-email-wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2013-07-15 16:48 ` [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114 Stephen Warren 2013-07-15 16:48 ` [lm-sensors] " Stephen Warren 2013-07-15 16:48 ` Stephen Warren [not found] ` <51E427F1.9010106-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> 2013-07-15 17:13 ` Stephen Warren 2013-07-15 17:13 ` [lm-sensors] " Stephen Warren 2013-07-15 17:13 ` Stephen Warren [not found] ` <51E42DB0.8090006-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> 2013-07-16 11:14 ` Wei Ni 2013-07-16 11:14 ` [lm-sensors] " Wei Ni 2013-07-16 11:14 ` Wei Ni [not found] ` <51E52B28.9090708-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2013-07-17 8:17 ` Wei Ni 2013-07-17 8:17 ` [lm-sensors] " Wei Ni 2013-07-17 8:17 ` Wei Ni 2013-07-17 8:23 ` Mikko Perttunen 2013-07-17 8:23 ` [lm-sensors] " Mikko Perttunen [not found] ` <51E6549A.1010006-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2013-07-17 9:08 ` Wei Ni 2013-07-17 9:08 ` [lm-sensors] " Wei Ni 2013-07-17 9:08 ` Wei Ni 2013-07-17 17:05 ` Stephen Warren 2013-07-17 17:05 ` [lm-sensors] " Stephen Warren
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.