Skip to content

Check firmware version for temperature support? #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
caternuson opened this issue Jul 29, 2019 · 1 comment
Closed

Check firmware version for temperature support? #33

caternuson opened this issue Jul 29, 2019 · 1 comment

Comments

@caternuson
Copy link
Contributor

Not really an issue, just a potential option.

Currently the temperature property is marked as deprecated to deal with the feature being removed from the chip. See #30 and #32.

Another approach might be to query the version information registers:
image
and then raise exception if a newer rev:

if version > THE_LAST_VERSION_THAT_HAD_TEMPERATURE:
    raise RuntimeError("Feature not supported for this chip version")
@ladyada
Copy link
Member

ladyada commented Jul 29, 2019

v2 is what dropped it, but i dont think its wise to use the temp sensor at all...then that code wouldn't be usable by others

@caternuson
Copy link
Contributor Author

okie dokie. works for me.

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

No branches or pull requests

2 participants