quoted

 +               */
 +               if (report->id == FEATURE_KBD_LED_REPORT_ID1 ||
 +                               report->id == 
FEATURE_KBD_LED_REPORT_ID2) {

quoted

 +                       return -1;

is -1 a good return code? (this Q for all cases)

quoted

 +               /* 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");
 +                               }

quoted

 +                               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.