Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Backport k10temp changes from 6.0.0 #181

Closed
wants to merge 1 commit into from
Closed

Backport k10temp changes from 6.0.0 #181

wants to merge 1 commit into from

Conversation

jackpot51
Copy link
Member

Should enable Ryzen 7000 temperature monitoring

@jackpot51 jackpot51 requested review from a team October 13, 2022 14:39
@jackpot51 jackpot51 self-assigned this Oct 13, 2022
13r0ck
13r0ck previously approved these changes Oct 14, 2022
View reviewed changes
@leviport
Copy link
Member

leviport commented Oct 18, 2022 •
edited
Loading

I cherry-picked this commit into #183, so this can be closed if that PR gets merged.

This didn't work quite right in #183, so this PR is still valid, once it is updated from master.

@jackpot51
Copy link
Member Author

@leviport rebased on master

@leviport
Copy link
Member

I'm seeing tctl appear now, but the value looks wrong:
Screenshot from 2022-10-19 12-34-37

@jackpot51
Copy link
Member Author

Will use #182 for this instead, the backport is not complete.

@jackpot51 jackpot51 closed this Oct 20, 2022
@jackpot51 jackpot51 deleted the k10temp branch October 20, 2022 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@13r0ck 13r0ck 13r0ck left review comments

Assignees

@jackpot51 jackpot51

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jackpot51 @leviport @13r0ck