bchretien / AlienFxLite

:alien: A keyboard color management tool for Alienware laptops (written in Java with a C++ native library).
BSD 3-Clause "New" or "Revised" License
71 stars 14 forks source link

M17xR4 issue #23

Open shriyashchougule opened 6 years ago

shriyashchougule commented 6 years ago

Hi, I was testing it on my alienware 15 laptop which has M17xR4 device. When I try to apply the profile, I get a popup error saying "AlienFX error. Error occured while trying to communicate with the AlienFX device: Data length should be 9 but was -9".

qwarlock commented 6 years ago

Yea I am getting this too. I corrected this about a year ago but did not upload the code and lost my fixed version. I am looking for a copy somewhere. If I remember right it is 2 changes in the LEDController. I will do a pull request if I remember what I did.

--

Got it. The isBusy Routine never comes back. I commented out everything that is in the insides of the method wairForReady instide AlienFXController. That fixes the -9 return wrong length error. But the colors are wrong. It lets you set RED and GREEN. The GREEN is actually set when you select BLUE. So one last part to fix. Will update this when I have that fixed and then send the pull request. I will fork this over as well just in case no one is watching this one. Will get back to you in a bit.

qwarlock commented 6 years ago

I forked the repo and made the corrections to fix the error and to fix the broken color controls. I made a pull request to the author for him to review and make the changes of use my code to begin his changes in order to fix. You can wait for him to do the fix or you can pull my fork of the project and use that to control your Alienware until he is able to adjust.

axsaucedo commented 6 years ago

Thanks for this contribution @qwarlock - I have tried it for 15 r4 but unfortunately it fails with the error: "Device unknown: no controller available"