From: Jaswinder Singh Rajput <jaswinder@kernel.org>
To: lm-sensors@vger.kernel.org
Subject: Re: [lm-sensors] Test of k8temp
Date: Mon, 06 Jul 2009 15:39:51 +0000 [thread overview]
Message-ID: <1246894071.3361.3.camel@ht.satnam> (raw)
In-Reply-To: <4A520751.3070803@assembler.cz>
Hello Rudolf,
On Mon, 2009-07-06 at 16:16 +0200, Rudolf Marek wrote:
> Hi again,
>
>
> >Anyway, In the patch, replace :
>
> >+ /* Some AMD 10H cpus reports Inaccurate Temperature Measurement */
> >+ if ((temp < 21000) || (temp > 90000))
>
> >+ /* Some AMD 10H cpus reports Inaccurate Temperature Measurement */
> >+ if ((temp < 10000) || (temp > 90000))
>
> So, now you see that workaround on some arbitrary temperature simply does not
> make sense.
>
> Please don't add support for fam10h CPUs. This is not a point of HW monitoring,
> to rule out arbitrary temperatures.
>
I totally agree with you that's why I removed the support for 10h as per
your suggestion.
> I'm perfectly fine if you come up with a patch for fam 11h because the Errata
> seems to have vanished...
>
x86: hwmon/k8temp.c Add support for AMD 11H
http://git.kernel.org/?p=linux/kernel/git/x86/linux-2.6-tip.git;a=commit;h18e6a08d340d853459a39b4104eab0334568427
is still waiting for your approval.
Thanks,
--
JSR
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
next prev parent reply other threads:[~2009-07-06 15:39 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-06 14:16 [lm-sensors] Test of k8temp Rudolf Marek
2009-07-06 15:39 ` Jaswinder Singh Rajput [this message]
2009-07-06 16:44 ` 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=1246894071.3361.3.camel@ht.satnam \
--to=jaswinder@kernel.org \
--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.