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

hwmon: Differentiate between Aquaero 5 and 6, add pwm_mode support #84

Open
wants to merge 3 commits into
base: hwmon-state
Choose a base branch
from

Conversation

aleksamagicka
Copy link
Owner

This series adds support for reading and writing hwmon_pwm_mode for Aquaero devices. While the Aquaero 6 supports both DC and PWM fans for all four physical slots, Aquaero 5 supports PWM only for the fourth fan slot.

Patch 1 reads the Aquaero hardware version, which the devices send as part of the sensor report, and exposes it through debugfs.

Patch 2 uses that raw version value and differentiates between Aquaero 5 and 6, which is necessary to know to be able to expose the right amount of entries.

Patch 3 implements the hwmon_pwm_mode functionality.


@leoratte, do I have your Co-developed-by: and Signed-off-by: for these patches, since we worked on them together in #70? (Assuming you're OK with them.)

Copy link
Contributor

@leoratte leoratte left a comment

Choose a reason for hiding this comment

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

lgtm. I've tested this on my Aquaero 5 and everything works as expected.

@leoratte
Copy link
Contributor

leoratte commented Aug 1, 2023

@leoratte, do I have your Co-developed-by: and Signed-off-by: for these patches, since we worked on them together in #70? (Assuming you're OK with them.)

Sure.

@aleksamagicka
Copy link
Owner Author

Thanks, added your tags to these commits.

@aleksamagicka
Copy link
Owner Author

This is the next thing that I'm sending to LKML soon.

@aleksamagicka
Copy link
Owner Author

TODO: update for #92.

@aleksamagicka
Copy link
Owner Author

Now updated after #92.

@aleksamagicka aleksamagicka mentioned this pull request Sep 24, 2023
13 tasks
Aquaero devices expose their hardware version in
the sensor report. Read it and show it through debugfs.

Co-developed-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
With the raw hardware version value now read from the sensor
report, determine whether it's an Aquaero 5 or 6.

Co-developed-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
Expose reading and writing hwmon_pwm_mode for Aquaero devices.
Aquaero 5 can physically accept only one PWM fan (in the fourth
slot), while Aquaero 6 can accept both DC and PWM fans in all
four slots.

Co-developed-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Leonard Anderweit <leonard.anderweit@gmail.com>
Signed-off-by: Aleksa Savic <savicaleksa83@gmail.com>
@aleksamagicka
Copy link
Owner Author

Updated 6c86ba7 and made the wait_for_completion call interruptible with timeout in the debugfs entry for hw_version.

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