The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: Alexander Stein <alexander.stein@systec-electronic.com>
Cc: guenter.roeck@ericsson.com, "R,
	Durgadoss" <durgadoss.r@intel.com>,
	"Yu, Fenghua" <fenghua.yu@intel.com>,
	lm-sensors@lm-sensors.org, linux-kernel@vger.kernel.org
Subject: Re: [lm-sensors] coretemp: Support for Intel Atom E6XX CPU (TunnelCreek)?
Date: Fri, 16 Sep 2011 18:21:51 +0200	[thread overview]
Message-ID: <20110916182151.463d6003@endymion.delvare> (raw)
In-Reply-To: <201108031640.31196.alexander.stein@systec-electronic.com>

Hi Alexander,

On Wed, 3 Aug 2011 16:40:30 +0200, Alexander Stein wrote:
> On Wednesday 03 August 2011 15:26:53 R, Durgadoss wrote:
> > Yes I checked with some Atom Guys here.
> > Apparently there is no way to know it from the OS Level.
> > 
> > > On Monday 27 June 2011 18:54:44 R, Durgadoss wrote:
> > > > > Unfortunately, looking at Alexander's /proc/cpuinfo output, it looks
> > > > > like the E6xx chips have no mention of E6xx in the CPU ID string,
> > > > > which unfortunately kills the idea.
> > > > > 
> > > > > Do you know if there is _any_ means to distinguish E6xx from E6xxT ?
> > > > 
> > > > Sorry I do not know. But I can ask some folks here and get back
> > > > to you guys. Should not take more time, once I identify the right
> > > > person to ask :-)
> 
> So, is there a possibility to add a module parameter to force a either 
> specific tjmax or the T model for E6xx?

A parameter to force the T model for E6xx would be very very specific,
while the problem at hand is much broader: the heuristics in
adjust_tjmax() are best effort but largely insufficient in many (most?)
cases. So, as we have been enable to provide a relative temperature
scale interface in the past 2 years, a module parameter letting the
user set TjMax as needed seems the best approach to me. I'll submit a
patch doing this in a moment.

-- 
Jean Delvare

      reply	other threads:[~2011-09-16 16:22 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-27 10:20 coretemp: Support for Intel Atom E6XX CPU (TunnelCreek)? Alexander Stein
2011-06-27 13:42 ` Guenter Roeck
2011-06-27 14:41 ` Jean Delvare
2011-06-27 16:05   ` Alexander Stein
2011-06-27 16:10   ` Guenter Roeck
2011-06-27 16:40     ` [lm-sensors] " R, Durgadoss
2011-06-27 16:50       ` Guenter Roeck
2011-06-27 16:54         ` R, Durgadoss
2011-08-03 12:58           ` Alexander Stein
2011-08-03 13:26             ` R, Durgadoss
2011-08-03 14:40               ` Alexander Stein
2011-09-16 16:21                 ` Jean Delvare [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=20110916182151.463d6003@endymion.delvare \
    --to=khali@linux-fr.org \
    --cc=alexander.stein@systec-electronic.com \
    --cc=durgadoss.r@intel.com \
    --cc=fenghua.yu@intel.com \
    --cc=guenter.roeck@ericsson.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox