FYI, The HPET code in i82801ex, does absolutely nothing. It reads the reg and configs it but does not write it back to the reg.......
And ICH 0-4 does not support HPET and should be removed from code....
On Wed, Mar 17, 2010 at 12:23:16PM -0400, Joseph Smith wrote:
FYI, The HPET code in i82801ex, does absolutely nothing. It reads the reg and configs it but does not write it back to the reg.......
That should be fixed, yes.
And ICH 0-4 does not support HPET and should be removed from code....
Not quite, ICH3 and ICH4 do have HPET, just undocumented by Intel. There's been kernel support to force-enable it since 2007 which obviously works fine and has been tested by various people since then.
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdif...
So I think we should keep the ICH3/4 code around (ICH0-ICH2 can be safely dropped I guess).
Uwe.
On 3/17/10 6:43 PM, Uwe Hermann wrote:
And ICH 0-4 does not support HPET and should be removed from code....
Not quite, ICH3 and ICH4 do have HPET, just undocumented by Intel. There's been kernel support to force-enable it since 2007 which obviously works fine and has been tested by various people since then.
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdif...
So I think we should keep the ICH3/4 code around (ICH0-ICH2 can be safely dropped I guess).
Uh,.. you are right..! sorry for my quick shot.. I'll back out the dx change I made... CX never had enable_hpet code it seems.
On 03/17/2010 02:37 PM, Stefan Reinauer wrote:
On 3/17/10 6:43 PM, Uwe Hermann wrote:
And ICH 0-4 does not support HPET and should be removed from code....
Not quite, ICH3 and ICH4 do have HPET, just undocumented by Intel. There's been kernel support to force-enable it since 2007 which obviously works fine and has been tested by various people since then.
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdif...
So I think we should keep the ICH3/4 code around (ICH0-ICH2 can be safely dropped I guess).
Uh,.. you are right..! sorry for my quick shot.. I'll back out the dx change I made... CX never had enable_hpet code it seems.
Ah cool! Stefan, don't do anything, new patch coming.
-
Joseph Smith -
Stefan Reinauer -
Uwe Hermann