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

Support for Lian-Li Uni Fan SL-INFINITY fan hub #629

Open
CUexter opened this issue Sep 14, 2023 · 13 comments · May be fixed by #636
Open

Support for Lian-Li Uni Fan SL-INFINITY fan hub #629

CUexter opened this issue Sep 14, 2023 · 13 comments · May be fixed by #636
Assignees
Labels
new device Support for a new device

Comments

@CUexter
Copy link

CUexter commented Sep 14, 2023

Device type

Fan hub/controller

Product page

https://lian-li.com/product/uni-fan-sl-infinity/

First-party software

L-Connect 3

What monitoring functionality does the device support?

temperatures, fan/pump speeds or duty cycles

What configuration functionality does the device support?

fan/pump speeds or duty cycles, lighting of accessories like RGB fans or LED strips

Physical connection

No response

Connection protocol

USB

Additional information about the device

It seems like this can be used to help ?

Can you help with implementing or testing the changes needed for liquidctl to support this device?

No response

@CUexter CUexter added the new device Support for a new device label Sep 14, 2023
@LuNeder
Copy link

LuNeder commented Oct 7, 2023

Not sure if I should open a new issue for this, but please do the V2 too! (the linked program seems to also support it so I assume the method should be quite similar?)

Edit: this might also help

@ThimbleFire
Copy link

This would be great. The only software I'm familiar with that can run these operates only on Windows, and is Rem0o's FanControl. My fans will likely be stuck at 650 RPM until this gets added since they can't be controlled by the bios.

@LuNeder
Copy link

LuNeder commented Oct 14, 2023

This would be great. The only software I'm familiar with that can run these operates only on Windows, and is Rem0o's FanControl. My fans will likely be stuck at 650 RPM until this gets added since they can't be controlled by the bios.

@ThimbleFire the software op mentioned in their issue seem to have worked for me to set my fans to 100%, had to try it after my cpu was overheating when no software was controlling the fans and it seems to have done the job bc that didn’t happen anymore (and it’s a little bit more noisy too)

Tho yeah liquidctl support would be really really great since then we’d have better control and the ability to use stuff like coolero

@Serphentas
Copy link
Collaborator

The utility is licensed with MIT so it should not pose any problem with respect to code usage. I will work on an initial draft but I need someone who can test it, any volunteers ?

@LuNeder
Copy link

LuNeder commented Nov 8, 2023

The utility is licensed with MIT so it should not pose any problem with respect to code usage. I will work on an initial draft but I need someone who can test it, any volunteers ?

I could definitely test it!

@Serphentas
Copy link
Collaborator

OK I've got the initial port from Rust done, see Serphentas@3d018ab. I have to check a few things with Jonas and then I will ask you to kindly test it. Thank you for your patience.

@Serphentas Serphentas linked a pull request Nov 13, 2023 that will close this issue
15 tasks
@Serphentas
Copy link
Collaborator

I think the code is ready to be tested now, you may pull this branch. Let me know how things go.

@LuNeder
Copy link

LuNeder commented Nov 24, 2023

I think the code is ready to be tested now, you may pull this branch. Let me know how things go.

I'll try it this weekend and let you know, thanks!

@LuNeder
Copy link

LuNeder commented Nov 25, 2023

Hey, got to test it!

The device seems to be listed correctly

python3 -m liquidctl list                     
Device #0: LianLi-UNI SL

but it errors when trying to get the status:

sudo python3 -m liquidctl status        
ERROR: LianLi-UNI SL: unexpected error: NotImplementedError()

@Serphentas
Copy link
Collaborator

Thanks for your feedback. Forgot to mention only the set fanX speed N function is supported right now. The source repo I used did not showcase any kind of status probing, so that was left out.

Can you then test liquidctl set fan1 speed 100 ? You may add --sync-rgb to sync RGB (or not, leave it out) and --fan-mode {rpm,pwm} where pwm will not use the provided speed percentage.

@LuNeder
Copy link

LuNeder commented Nov 26, 2023

Oh, oke!!

Now I get:

sudo python3 -m liquidctl set fan1 speed 100
ERROR: LianLi-UNI SL: unexpected error: ValueError('fan channel must be between 1 and 6')

I tried other stuff instead of "fan1" (such as numbers between 1 and 6), but the same error happens

@Serphentas
Copy link
Collaborator

Yeah I've made a change to that part, I had misunderstood the expected value for channel. Would you mind pulling and trying again ?

@LuNeder
Copy link

LuNeder commented Dec 1, 2023

New error now:

sudo python3 -m liquidctl set fan1 speed 100
ERROR: LianLi-UNI SL: unexpected error: KeyError('fan_mode')

@Serphentas Serphentas self-assigned this Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new device Support for a new device
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants