On Tue, Oct 30, 2012 at 11:09 AM, Benjamin Tissoires
[off-list ref] wrote:

On Mon, Oct 29, 2012 at 8:25 PM, Henrik Rydberg [off-list ref] wrote:

quoted

Hi Benjamin,

quoted

Currently, there is no way to know the index of the current field
in the .input_mapping and .event callbacks  when this field is inside
an array of HID fields.
This patch forwards this index to the input_mapping and event
callbacks.

I agree with the idea, but the function argument list is becoming
ridiculously long... Could we remove the usage pointer argument, at
least?

yeah, totally agree. Let me just check whether it will not introduce
more problems than it solves for my driver.

Well, after a deeper look, it's a really bad idea. Every drivers that
implements input_mapping, input_mapped, event use at least once the
struct usage. So this would require change nearly every hid driver.

To solve that, and to minimize the impact on the other drivers, I'm
going to add a field in struct usage with the appropriate index.

Cheers,
Benjamin

quoted

        int (*event)(struct hid_device *hdev, struct hid_field *field,
                        unsigned int usage_index, __s32 value);

quoted

@@ -1071,19 +1072,24 @@ static void hid_input_field(struct hid_device *hid, struct hid_field *field,
      for (n = 0; n < count; n++) {

              if (HID_MAIN_ITEM_VARIABLE & field->flags) {
-                     hid_process_event(hid, field, &field->usage[n], value[n], interrupt);
+                     hid_process_event(hid, field, &field->usage[n], n,
+                             value[n], interrupt);
                      continue;
              }

              if (field->value[n] >= min && field->value[n] <= max
                      && field->usage[field->value[n] - min].hid
                      && search(value, field->value[n], count))
-                             hid_process_event(hid, field, &field->usage[field->value[n] - min], 0, interrupt);
+                             hid_process_event(hid, field,
+                                     &field->usage[field->value[n] - min], n,
+                                     0, interrupt);

Wrong index?

oops, I'll have to check that.

Thanks,
Benjamin

quoted

quoted

              if (value[n] >= min && value[n] <= max
                      && field->usage[value[n] - min].hid
                      && search(field->value, value[n], count))
-                             hid_process_event(hid, field, &field->usage[value[n] - min], 1, interrupt);
+                             hid_process_event(hid, field,
+                                     &field->usage[value[n] - min], n,
+                                     1, interrupt);

Wrong index?

quoted

      }

      memcpy(field->value, value, count * sizeof(__s32));

Thanks,
Henrik