+ */
+ if (report->id == FEATURE_KBD_LED_REPORT_ID1 ||
+ report->id ==
FEATURE_KBD_LED_REPORT_ID2) {
+ return -1;
is -1 a good return code? (this Q for all cases)
+ /* Additional report filtering */
+ } else if (report->id == FEATURE_KBD_REPORT_ID) {
+ /* Fn+F5 "fan" symbol, trigger WMI event to
toggle next mode */
+ if (data[1] == 0xae) {
+ ret = asus_wmi_send_event(drvdata,
0xae);
+ if (ret < 0) {
+ hid_warn(hdev, "Asus failed
to trigger fan control event");
+ }
+ return -1;
In the case of this block I really don't have any idea how
to handle it. I want to stop these particular keycodes from
being evaluated elsewhere. Returning -1 seemed to be the only
way to do this, unless my understanding is very incorrect.
Any help or guidance on how to handle this is definitely
appreciated.
Cheers,
Luke.
