All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Riepe <michael.riepe@googlemail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org, Rudolf Marek <r.marek@assembler.cz>,
	lm-sensors@lm-sensors.org
Subject: Re: [lm-sensors] [PATCH] Enable coretemp driver on Intel Atom
Date: Thu, 18 Jun 2009 21:05:05 +0000	[thread overview]
Message-ID: <4A3AAC01.7050805@googlemail.com> (raw)
In-Reply-To: <20090618121326.42ab9c2f.akpm@linux-foundation.org>

Hi!

Andrew Morton wrote:

> I'm not 100% sure what to do about this patch.  I'm inclined to merge
> it, even though you think it might be giving the wrong numbers,
> because then someone might fix it.

That was the basic idea :-)

> otoh, if giving the wrong numbers leads to people's machines needlessly
> shutting down or something like that, then that's not so good.

I don't think that that will happen. Core temperatures on my 330 are
lower than those reported by the other sensors (core 0/1: 27/35 °C on an
idle system - hardly alarming in my opinion).

-- 
Michael "Tired" Riepe <michael.riepe@googlemail.com>
X-Tired: Each morning I get up I die a little

_______________________________________________
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: Michael Riepe <michael.riepe@googlemail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org, Rudolf Marek <r.marek@assembler.cz>,
	lm-sensors@lm-sensors.org
Subject: Re: [PATCH] Enable coretemp driver on Intel Atom
Date: Thu, 18 Jun 2009 23:05:05 +0200	[thread overview]
Message-ID: <4A3AAC01.7050805@googlemail.com> (raw)
In-Reply-To: <20090618121326.42ab9c2f.akpm@linux-foundation.org>

Hi!

Andrew Morton wrote:

> I'm not 100% sure what to do about this patch.  I'm inclined to merge
> it, even though you think it might be giving the wrong numbers,
> because then someone might fix it.

That was the basic idea :-)

> otoh, if giving the wrong numbers leads to people's machines needlessly
> shutting down or something like that, then that's not so good.

I don't think that that will happen. Core temperatures on my 330 are
lower than those reported by the other sensors (core 0/1: 27/35 °C on an
idle system - hardly alarming in my opinion).

-- 
Michael "Tired" Riepe <michael.riepe@googlemail.com>
X-Tired: Each morning I get up I die a little

  reply	other threads:[~2009-06-18 21:05 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-16 23:46 [PATCH] Enable coretemp driver on Intel Atom Michael Riepe
2009-06-18 19:13 ` [lm-sensors] " Andrew Morton
2009-06-18 19:13   ` Andrew Morton
2009-06-18 21:05   ` Michael Riepe [this message]
2009-06-18 21:05     ` Michael Riepe
2009-06-21 20:31     ` [lm-sensors] " Rudolf Marek
2009-06-21 20:31       ` Rudolf Marek

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=4A3AAC01.7050805@googlemail.com \
    --to=michael.riepe@googlemail.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.org \
    --cc=r.marek@assembler.cz \
    /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.