Re: [PATCH] hid: put the case in the right switch statement

Adding Jiri to the recipient list of the patch, otherwise, the thread
may fall in the depth of his mailbox :)

Cheers,
Benjamin

On Thu, Oct 25, 2012 at 7:08 PM, Benjamin Tissoires
[off-list ref] wrote:

Hi Alan,

Yes, I saw that too.

Acked-by: Benjamin Tissoires <redacted>

On Thu, Oct 25, 2012 at 4:35 PM, Alan Cox [off-list ref] wrote:

quoted

From: Alan Cox <redacted>

Signed-off-by: Alan Cox <redacted>
---

 drivers/hid/hid-multitouch.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c
index 3eb02b9..c97011c 100644
--- a/drivers/hid/hid-multitouch.c
+++ b/drivers/hid/hid-multitouch.c

@@ -421,11 +421,11 @@ static int mt_input_mapping(struct hid_device *hdev, struct hid_input *hi,
                         * contact max are global to the report */
                        td->last_field_index = field->index;
                        return -1;
-               }
                case HID_DG_TOUCH:
                        /* Legacy devices use TIPSWITCH and not TOUCH.
                         * Let's just ignore this field. */
                        return -1;
+               }
                /* let hid-input decide for the others */
                return 0;


--

To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html