Skip to content

Add support for HX1200i ATX3.1 variant#763

Merged
jonasmalacofilho merged 6 commits intoliquidctl:mainfrom
ToxicBakery:main
Mar 3, 2025
Merged

Add support for HX1200i ATX3.1 variant#763
jonasmalacofilho merged 6 commits intoliquidctl:mainfrom
ToxicBakery:main

Conversation

@ToxicBakery
Copy link
Contributor

@ToxicBakery ToxicBakery commented Jan 29, 2025 •
edited
Loading

Added and tested addition of HX1200i ATX 3.0 variant PSU.

Fixes: #762

I did not add a new device entry in documentation as the described name in lsusb and marketing material has not changed. If I was incorrect in this decision just point me to how it should be documented and I'm happy to do that work too.

$ python -m liquidctl list
Device #0: Corsair HX1200i ATX 3.0
$ python -m liquidctl initialize all
$ python -m liquidctl status
Corsair HX1200i ATX 3.0
├── Current uptime                 10:22:56  
├── Total uptime             1 day, 4:12:56  
├── VRM temperature                    36.8  °C
├── Case temperature                   29.0  °C
├── Fan control mode               Hardware  
├── Fan speed                             0  rpm
├── Input voltage                    115.00  V
├── +12V OCP mode                Multi rail  
├── +12V output voltage               12.05  V
├── +12V output current                6.75  A
├── +12V output power                 80.00  W
├── +5V output voltage                 5.08  V
├── +5V output current                 1.94  A
├── +5V output power                   9.50  W
├── +3.3V output voltage               3.33  V
├── +3.3V output current               3.00  A
├── +3.3V output power                 9.50  W
├── Total power output               110.00  W
├── Estimated input power            129.00  W
└── Estimated efficiency                 85  %

$ python -m liquidctl --match corsair set fan speed 50
$ python -m liquidctl status
Corsair HX1200i ATX 3.0
├── Current uptime                 10:25:09  
├── Total uptime             1 day, 4:15:09  
├── VRM temperature                    37.2  °C
├── Case temperature                   32.2  °C
├── Fan control mode               Software  
├── Fan speed                          1136  rpm
├── Input voltage                    115.00  V
├── +12V OCP mode                Multi rail  
├── +12V output voltage               12.06  V
├── +12V output current               11.75  A
├── +12V output power                138.00  W
├── +5V output voltage                 5.09  V
├── +5V output current                 1.94  A
├── +5V output power                   9.50  W
├── +3.3V output voltage               3.33  V
├── +3.3V output current               2.88  A
├── +3.3V output power                 9.50  W
├── Total power output               152.00  W
├── Estimated input power            172.00  W
└── Estimated efficiency                 88  %

Checklist:

New CLI flag?

  • Adjust the completion scripts in extra/completions/

New device?

  • Regenerate extra/linux/71-liquidctl.rules (instructions in the file header)
  • Add entry to the README's supported device list with applicable notes and git MRLV

New driver?

  • Document the protocol in docs/developer/protocol/

jonasmalacofilho
jonasmalacofilho reviewed Feb 10, 2025
View reviewed changes
Copy link
Member

@jonasmalacofilho jonasmalacofilho left a comment

Choose a reason for hiding this comment

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

Hey, thanks for the PR.

The input power curves are unlikely to have remained the same. Please see my other comment.

Regarding the docs, I think having new entries in the README and man page for ATX 3.1 variants can be useful for users, since it's more than just an annual refresh. In the case of the README table, I think a new row just for them should be added.

By the way, do you know whether the 0x1c1e and 0x1c1f variants are ATX 3.1 too?

Finally, the corresponding device guide should receive a "New in ..." note (please see the development process document for details). But a separate device guide is not necessary for the ATX 3.1 variants.

@ToxicBakery
Changed name of HX1200i ATX 3.1 to ATX 3.0
fb95256 
- Learned how to read documentation
- Updated Corsair documentation for ATX 3.0 CP-9020281-NA model
@ToxicBakery
Update corsair_hid_psu.py
78f992f 
Updated efficiency values for HX1200i ATX 3.0
jonasmalacofilho
jonasmalacofilho reviewed Feb 12, 2025
View reviewed changes
@jonasmalacofilho
Copy link
Member

jonasmalacofilho commented Mar 3, 2025

LGTM, and thanks for looking into this in this much detail/with this much attention!

@jonasmalacofilho jonasmalacofilho merged commit a3795fb into liquidctl:main Mar 3, 2025
13 checks passed
@mbway mbway mentioned this pull request May 24, 2025
Merged
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jonasmalacofilho jonasmalacofilho jonasmalacofilho left review comments

@aleksamagicka aleksamagicka aleksamagicka left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

HX1200i ATX 3.1

3 participants

@ToxicBakery @jonasmalacofilho @aleksamagicka