All of lore.kernel.org
 help / color / mirror / Atom feed
From: Guenter Roeck <linux@roeck-us.net>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] [PATCH 1/5] hwmon: (coretemp) Drop dependency on PCI for TjMax detection on Atom CP
Date: Fri, 12 Oct 2012 16:48:24 +0000	[thread overview]
Message-ID: <20121012164824.GD6300@roeck-us.net> (raw)
In-Reply-To: <1349816943-17697-2-git-send-email-linux@roeck-us.net>

On Fri, Oct 12, 2012 at 11:01:32AM +0200, Jean Delvare wrote:
> On Fri, 12 Oct 2012 10:40:32 +0200, Jean Delvare wrote:
> > I think we simply never bothered special-casing the Atom 230/330.
> 
> Oh actually we did, I didn't remember that. So well no, I have no idea
> where the 100°C for Atom 230 comes from, probably we just got it wrong
> when introducing tjmax_table in commit 41e58a1f.
> 
Maybe. Part of the problem is that TjMax is really 90 C according to the
230/330 datasheet, and that at least the 330 uses not TjMax but 125 as
the base temperature for its temperature offset calculation. So we don't
really have an idea what the 230 does.

Would be great if you have a chance to test it with the 230 before we blindly
change it to some other value.

Thanks,
Guenter

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

      parent reply	other threads:[~2012-10-12 16:48 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-09 21:08 [lm-sensors] [PATCH 1/5] hwmon: (coretemp) Drop dependency on PCI for TjMax detection on Atom CPUs Guenter Roeck
2012-10-10 13:44 ` [lm-sensors] [PATCH 1/5] hwmon: (coretemp) Drop dependency on PCI for TjMax detection on Atom CP Jean Delvare
2012-10-10 13:52 ` Guenter Roeck
2012-10-10 14:00 ` Guenter Roeck
2012-10-10 18:13 ` Guenter Roeck
2012-10-12  8:40 ` Jean Delvare
2012-10-12  9:01 ` Jean Delvare
2012-10-12 16:48 ` Guenter Roeck [this message]

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=20121012164824.GD6300@roeck-us.net \
    --to=linux@roeck-us.net \
    --cc=lm-sensors@vger.kernel.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.