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

Not all fans are updated #2

Closed
ykurlaev opened this issue Dec 22, 2020 · 1 comment
Closed

Not all fans are updated #2

ykurlaev opened this issue Dec 22, 2020 · 1 comment

Comments

@ykurlaev
Copy link

In nct6687_update_fans you have for (i = 0; i < ARRAY_SIZE(data->rpm); i++), but size of data->rpm is always 3 (u16 rpm[3][NCT6687_NUM_REG_FAN];).
Also, maybe it's good idea to add debug print that is already there for temperature and voltage update functions to the fan update function as well?
Fred78290 added a commit that referenced this issue Dec 23, 2020
@Fred78290
FIX: Issue #2
6035991 
Not all fans are updated
@Fred78290
Copy link
Owner

Thanks for your report.
I fixed the issue.

@Semmelrocks Semmelrocks mentioned this issue Oct 23, 2021
Open
@krishjainx krishjainx mentioned this issue Feb 24, 2022
Closed
@pipes80-collab pipes80-collab mentioned this issue Apr 24, 2022
Open
@Lithium64 Lithium64 mentioned this issue May 12, 2022
Open
@MOVZX MOVZX mentioned this issue May 24, 2022
Open
@Makhno77 Makhno77 mentioned this issue Sep 21, 2022
Open
@muaddibus muaddibus mentioned this issue Nov 21, 2022
Open
@RealChickenMan RealChickenMan mentioned this issue Jul 6, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Fred78290 @ykurlaev