Skip to content
This repository has been archived by the owner on Oct 12, 2018. It is now read-only.

Using UInt64 overload of evaluateInteger in ACPISensors = KP #26

Closed
RehabMan opened this issue Dec 16, 2012 · 2 comments
Closed

Using UInt64 overload of evaluateInteger in ACPISensors = KP #26

RehabMan opened this issue Dec 16, 2012 · 2 comments

Comments

@RehabMan
Copy link
Collaborator

The code to evaluate ACPI methods in ACPISensors::getSensorValue is using the UInt64 version of IOACPIPlatform::evaluateInteger. This appears to cause a KP, where using the UInt32 version (as does slice's branch) does not.

There is also one more case in PTIDSensors where this method is used. My DSDT doesn't have the PTID, so I didn't test that, although changing it might be a good idea as well...

My branch: https://github.com/RehabMan/HWSensors

@RehabMan
Copy link
Collaborator Author

I guess the PTID is something in ACPI namespace added by the OS, in this case OS X (still learning this stuff). So I'm not sure of the right overload method call there!? Probably either works since the result fits into 32-bits.

@RehabMan
Copy link
Collaborator Author

Thanks for fixing this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant