All of lore.kernel.org
 help / color / mirror / Atom feed
From: r.marek@sh.cvut.cz (Rudolf Marek)
To: lm-sensors@vger.kernel.org
Subject: [lm-sensors] [PATCH] New VID codes for Intel Core and Conroe
Date: Fri, 09 Jun 2006 19:00:12 +0000	[thread overview]
Message-ID: <4489C53C.2010702@sh.cvut.cz> (raw)
In-Reply-To: <447815E5.5060601@sh.cvut.cz>

Hi Jean,

> It includes an "Extended VRD 10.0" VID table. I seem to understand that
> the CPUs may alternatively use this table or the VRD 11.0 table you
> added support for. I don't really understand what would make a CPU use
> one rather than the other though. Can you please investigate? Then your
> patch might need some more work. At least it would be nice to point to
> the official location for the VRD 11.0 table.
> 

Ok I have investigated. It took me one hour to figure it all out.

the VID_SELECT pin is for motherboard that has LGA775 and wants to support old
and new CPUs. Intel introduced the bit already for older Celeron D
http://www.intel.com/design/celeron/datashts/304092.htm

Page 69 clarifies it all.

They already reserved enough bits for VID, using just 5 of them.

With the VID_SEL bit they will just do what we do in software to detect the VRM
version. So => we dont care about this bit.

As for the Extended VRD 10.0:

it just adds one more bit do be able to fine grain the 5bit VID. And because my
patch already rounds it we should not care about this too.

All in all we dont need to do anything special unless we switch to uV as return
value.

I will resend the patch in the moment with updated VRM11 stuff.

Regards
Rudolf


  parent reply	other threads:[~2006-06-09 19:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-27  9:03 [lm-sensors] [PATCH] New VID codes for Intel Core and Conroe Rudolf Marek
2006-05-27 11:22 ` Jean Delvare
2006-05-30  2:30 ` Yuan Mu
2006-05-30 21:36 ` Rudolf Marek
2006-05-31  1:50 ` Yuan Mu
2006-06-04 19:56 ` Rudolf Marek
2006-06-05  6:18 ` Jean Delvare
2006-06-08  9:05 ` Jean Delvare
2006-06-09 19:00 ` Rudolf Marek [this message]
2006-06-09 19:40 ` Rudolf Marek
2006-06-11 14:36 ` Jean Delvare

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=4489C53C.2010702@sh.cvut.cz \
    --to=r.marek@sh.cvut.cz \
    --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.