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
Add support for Razer Blackwidow v3 Pro wireless #1901
Conversation
@z3ntu please have a look at this PR as the previous one was stuck |
@z3ntu any news for this PR? |
driver/razerkbd_driver.c
Outdated
case USB_DEVICE_ID_RAZER_BLACKWIDOW_V3_MINI_WIRELESS: | ||
report.transaction_id.id = 0x9F; | ||
break; | ||
default: | ||
report.transaction_id.id = 0x1f; | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this block added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed it for now, still keeping this "conversation" open, maybe it was super important for something
@@ -1533,7 +1549,11 @@ static ssize_t razer_attr_write_matrix_effect_reactive(struct device *dev, struc | |||
case USB_DEVICE_ID_RAZER_BLACKWIDOW_V3_PRO_WIRED: | |||
report = razer_chroma_extended_matrix_effect_reactive(VARSTORE, BACKLIGHT_LED, speed, (struct razer_rgb*)&buf[1]); | |||
break; | |||
|
|||
case USB_DEVICE_ID_RAZER_BLACKWIDOW_V3_PRO_WIRELESS: | |||
razer_set_device_mode(usb_dev, 0x00, 0x00); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the daemon likes to switch the device to driver mode (which I didn't check now), then maybe that should be removed if reactive effect doesn't work with it.
Overwriting the device mode here isn't great at all...
@@ -1699,7 +1718,7 @@ static ssize_t razer_attr_write_matrix_effect_static(struct device *dev, struct | |||
report.transaction_id.id = 0x1F; | |||
razer_send_payload(usb_dev, &report); | |||
break; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please revert all these unrelated whitespace changes in this file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done it myself
'set_custom_effect', 'set_key_row', 'get_game_mode', 'set_game_mode', 'get_macro_mode', 'set_macro_mode', | ||
'get_macro_effect', 'set_macro_effect', 'get_macros', 'delete_macro', 'add_macro', | ||
'set_starlight_random_effect', 'set_starlight_single_effect', 'set_starlight_dual_effect', | ||
# 'set_ripple_effect', 'set_ripple_effect_random_colour', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove the comment if it doesn't work?
Would be cool if it gets merged. |
Nice, I Happy Holidays |
@andriinuts @YoMamasNerd Does this PR work for you? See https://github.com/openrazer/openrazer/wiki/Building-a-package @nicfix There's still some open comments, could you resolve them? |
I'll try it this weekend probably. |
@z3ntu I may need some further help to get this working. Do you have a discord or something?
Battery reporting is working with the keyboard. And my mouse (Deathadder v3 Pro) is not getting recognized anymore. May be a result of being so far behind the official repo. |
@YoMamasNerd Telegram or Matrix (or IRC if necessary): https://github.com/openrazer/openrazer/wiki/Community-chat Regarding this error, looks like the device declares it supports |
c5c0a37
to
03ed6d0
Compare
What is not working: - Battery level in the notification - Ripple effect (all the others work correctly)
03ed6d0
to
4d241d9
Compare
Things that work
Things that does not work
|
Has there been any progress on the non-working items? |
Not yet. I've been too busy to fix the issues since they are not breaking anything for me. |
Closing in favor of #2056 |
Added partial support for BlackWidow V3 Pro wireless
What works:
What doesn't work:
What works meh:
This PR is heavily inspired by the one from @GeoDoX (#1623), I thought about forking from that branch but rebasing @GeoDoX's branch proved more difficult than starting from scratch.
Partially fixes #1279