From: Wei Ni <wni-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: "khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org"
<khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>,
"linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org"
<linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>,
"thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org"
<lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114.
Date: Tue, 16 Jul 2013 19:14:48 +0800 [thread overview]
Message-ID: <51E52B28.9090708@nvidia.com> (raw)
In-Reply-To: <51E42DB0.8090006-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
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.
WARNING: multiple messages have this Message-ID (diff)
From: Wei Ni <wni@nvidia.com>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: "khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org"
<khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>,
"linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org"
<linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org>,
"thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org"
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
"lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org"
<lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [lm-sensors] [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114.
Date: Tue, 16 Jul 2013 11:14:48 +0000 [thread overview]
Message-ID: <51E52B28.9090708@nvidia.com> (raw)
In-Reply-To: <51E42DB0.8090006-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
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
WARNING: multiple messages have this Message-ID (diff)
From: Wei Ni <wni@nvidia.com>
To: Stephen Warren <swarren@wwwdotorg.org>
Cc: "khali@linux-fr.org" <khali@linux-fr.org>,
"linux@roeck-us.net" <linux@roeck-us.net>,
"thierry.reding@gmail.com" <thierry.reding@gmail.com>,
"lm-sensors@lm-sensors.org" <lm-sensors@lm-sensors.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>
Subject: Re: [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114.
Date: Tue, 16 Jul 2013 19:14:48 +0800 [thread overview]
Message-ID: <51E52B28.9090708@nvidia.com> (raw)
In-Reply-To: <51E42DB0.8090006@wwwdotorg.org>
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.
next prev parent reply other threads:[~2013-07-16 11:14 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-12 7:49 [PATCH v3 0/2] Enable lm90 in Tegra30 and Tegra114 Wei Ni
2013-07-12 7:49 ` Wei Ni
2013-07-12 7:49 ` [lm-sensors] " 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 ` Wei Ni
2013-07-12 7:49 ` [lm-sensors] " Wei Ni
2013-07-12 7:49 ` [PATCH v3 2/2] ARM: dt: t114 dalmore: " Wei Ni
2013-07-12 7:49 ` Wei Ni
2013-07-12 7:49 ` [lm-sensors] " 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 ` Stephen Warren
2013-07-15 16:48 ` [lm-sensors] " Stephen Warren
[not found] ` <51E427F1.9010106-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-07-15 17:13 ` Stephen Warren
2013-07-15 17:13 ` Stephen Warren
2013-07-15 17:13 ` [lm-sensors] " Stephen Warren
[not found] ` <51E42DB0.8090006-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-07-16 11:14 ` Wei Ni [this message]
2013-07-16 11:14 ` Wei Ni
2013-07-16 11:14 ` [lm-sensors] " Wei Ni
[not found] ` <51E52B28.9090708-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-07-17 8:17 ` Wei Ni
2013-07-17 8:17 ` Wei Ni
2013-07-17 8:17 ` [lm-sensors] " 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 ` Wei Ni
2013-07-17 9:08 ` [lm-sensors] " Wei Ni
2013-07-17 17:05 ` Stephen Warren
2013-07-17 17:05 ` [lm-sensors] " Stephen Warren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51E52B28.9090708@nvidia.com \
--to=wni-ddmlm1+adcrqt0dzr+alfa@public.gmane.org \
--cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
--cc=linux-0h96xk9xTtrk1uMJSBkQmQ@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=lm-sensors-GZX6beZjE8VD60Wz+7aTrA@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.