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.

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

hwvdf: Allow setting max frequency when in auto adjustment mode #158

Merged
merged 2 commits into from
Nov 22, 2023

Conversation

cmmarslender
Copy link
Contributor

@cmmarslender cmmarslender commented Nov 22, 2023

Allows setting a max frequency when in the auto adjusting mode

With Option Set:

./hw_vdf_client --freq 1100 --auto-freq-period 10 --max-freq 1110 [...]
2023-11-22T01:33:03.998 Setting frequency to 1100.000000 MHz
2023-11-22T01:33:04.114 Frequency is 1100.000000 MHz
[...]
2023-11-22T01:33:14.117 Frequency is increased to 1110.0 MHz; freq_idx = 7635
2023-11-22T01:33:24.117 Can't increase frequency, already at maximum
2023-11-22T01:33:34.117 Can't increase frequency, already at maximum

Without Option Set:

./hw_vdf_client --freq 1100 --auto-freq-period 10 [...]
2023-11-22T01:35:11.408 Setting frequency to 1100.000000 MHz
2023-11-22T01:35:11.524 Frequency is 1100.000000 MHz
[...]
2023-11-22T01:35:21.527 Frequency is increased to 1110.0 MHz; freq_idx = 7635
2023-11-22T01:35:31.531 Frequency is increased to 1111.1 MHz; freq_idx = 7636
2023-11-22T01:35:41.534 Frequency is increased to 1112.5 MHz; freq_idx = 7637
2023-11-22T01:35:51.537 Frequency is increased to 1114.2 MHz; freq_idx = 7638

Copy link
Contributor

@wjblanke wjblanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

aok

@cmmarslender cmmarslender merged commit 11f9a0e into main Nov 22, 2023
41 checks passed
@cmmarslender cmmarslender deleted the max-freq branch November 22, 2023 04:58
@cmmarslender cmmarslender changed the title Max freq hwvdf: Allow setting max frequency when in auto adjustment mode Nov 22, 2023
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

Successfully merging this pull request may close these issues.

2 participants