-
Notifications
You must be signed in to change notification settings - Fork 8
Support for Razer Blade 15" - 2019 Advanced #9
Comments
As jank as it is, my temporary workaround is basically commenting out all references to battery care in the tray app, and it seems to run perfectly fine. With all functions working. I might have a go tomorrow (later today) at making the battery care function a soft requirement or something. |
Hi @rainyskye, thanks for opening the issue, and I'm glad to see you managed to make it work for you. I merged today #2 which adds some infrastructure to support more devices. Different laptops have different sets of features, e.g. battery health, boost, and overclock might not be present on some models. Folks here came up with compatibility spec and I'll need to come up with something similar. Let me think over the weekend about how to integrate such functionality. It touches both |
@rainyskye , could you please share a bit more about manually specifying the PID in code? I was able to find some lines related to this, but due to a lack of experience with rust, I'm not sure how to get things to work as you did. |
Manually specifying the PID in code is essentially the same as
So if all you want to do is attempt to make your machine work with no other changes, you can use |
Sounds good, I think specifying a feature set in the PID list is the easiest way, obviously implementation isn't as simple, but let me know if I could perhaps help at all. :) Would make future support as easy as potentially an auto detect for which are supported, then they can be strictly defined, that is assuming Razer haven't used the same PID across multiple models with unique feature sets. |
@rainyskye , oooh, got it. Though, the cli-only implementation is a bit clunky. Initially, I thought that you recompiled the whole -tray app with the necessary PIDs adjusted to yours. Nevertheless, tried using only the CLI and it worked just fine:
Thanks! Now I need to figure out how to get the tray app to work with my PIDs. |
tray app relies on autodetect and uses hardcoded PIDs from this list razer-ctl/librazer/src/device.rs Lines 12 to 23 in 69817b1
However, it still might not work, because it queries all information that is hardcoded now. As discussed here #12 (comment), PID is not sufficient to differentiate between laptop models and I plan to move to Razer Model IDs.
If the aforementioned changes made |
Device/Specs:
Found this project after considering having a go at my own, Synapse is sh*t so kind of need an alternative.
I've been able to get
razer-cli enumerate
to show theRazerDevice
instances, and with manually specifying, or compiling librazer with the PID for my laptop, I can get every function working as far as I know (Fan Control, Performance Modes, Logo Control, Keyboard Backlight).However trying to run
razer-cli info
ends up spitting out aError: Command not supported
orError: Command failed with unknown status: 01
, which are the same errors returned by attempting to use functions that don't exist on my model (Battery Care) (There may be more but I'm not sure).razer-cli
itself works (exclinfo
), and I expect razer-tray may be failing due to not getting a valid response from an info check?Any help on getting this SKU supported would be appreciated, thanks for all your work so far :)
The text was updated successfully, but these errors were encountered: