On Thu, Feb 5, 2015 at 1:44 PM, Yani Ioannou <yani.ioannou@xxxxxxxxx> wrote: > Indeed you are correct, looks like one of the devices listed there is > the sensor hub - so I'm barking up the wrong tree. Not quite actually. If the i2c-hid driver works but the hid-sensor-hub does not pick up, there is probably an error in hid-sensor-hub. Could you paste the content of the rdesc file so we can check if hid-sensor-hub messed up or if your device has a wrong report descriptor? Cheers, Benjamin > > Will have to move on to the sensor drivers themselves I guess. Thanks > for your help! > > Yani > > On Thu, Feb 5, 2015 at 6:10 PM, Benjamin Tissoires > <benjamin.tissoires@xxxxxxxxx> wrote: >> On Thu, Feb 5, 2015 at 12:56 PM, Yani Ioannou <yani.ioannou@xxxxxxxxx> wrote: >>> Hi Benjamin, >>> >>> As far as I'm aware the sensor hub didn't show up under any kernel (I'll try >>> an earlier kernel to figure out if this is true) I didn't mean to say this >>> is a regression, sorry. >> >> Oh, my bad. >> >>> >>> The errors for the MSHW0030 device seem to be fatal - since the initial >>> report request fails, the driver doesn't try to work with the device. The >>> NTRG device however, is as you suggest, and it is fine - I shouldn't have >>> included the output for that device perhaps. >> >> Not being able to retrieve the reports should not be fatal in i2c_hid. >> >> There should be 2 devices showing up in: >> /sys/kernel/debug/hid/ >> (if you have debugfs mounted in /sys/kernel/debug) >> >> One of them is the Ntrig (0018:1B96:1B05.*), the other should be the sensor hub. >> >> Can you paste here the content of the rdesc file which is in the sensor_hub dir? >> >> And yes, if there is no sensor hub device showing under the hid >> debugfs dir (or under /sys/bus/hid/devices), that means that i2c-hid >> has removed the device. >> >> Cheers, >> Benjamin >> >>> >>> So in summary I'm trying to work out why the sensor hub driver is never >>> loaded, and as far as I can tell this is why. >>> >>> Thanks, >>> Yani >>> >>> >>> On Thu, Feb 5, 2015, 17:44 Benjamin Tissoires <benjamin.tissoires@xxxxxxxxx> >>> wrote: >>>> >>>> Hi, >>>> >>>> On Thu, Feb 5, 2015 at 11:29 AM, Yani Ioannou <yani.ioannou@xxxxxxxxx> >>>> wrote: >>>> > Hi, >>>> > >>>> > I've been trying to debug why the sensor hub on a Surface pro 3 isn't >>>> > detected in the latest mainline kernel (3.19-rc6). When I enable debug >>>> > I get the following output showing a bunch of errors trying to >>>> > communicate with the device MSHW0030, which seems to be the sensor >>>> > hub. >>>> >>>> These errors do not bother me that much. They appear when we try to >>>> read the reports from the device, and this has been known to not work >>>> on many devices. >>>> As long as the device emits by itself the reports, we are fine. >>>> >>>> There has been very few changes in the v3.19 series regarding i2c-hid, >>>> so if you get the same debug output on a v3.18 kernel (and the sensor >>>> hub works), I would think that the problem you are having is in the >>>> IIO subsystem. >>>> >>>> Cheers, >>>> Benjamin >>>> >>>> > >>>> > I've looked into the code/protocol a little and it seems the initial >>>> > feature reports that are returned are of incorrect size, but I don't >>>> > understand why without much prior knowledge of the domain. >>>> > >>>> > Any help and/or further debugging tips would be greatly appreciated. >>>> > >>>> > Thanks, >>>> > Yani >>>> > >>>> > [ 83.133219] i2c_hid i2c-MSHW0030:00: Fetching the HID descriptor >>>> > [ 83.133224] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=01 00 >>>> > [ 83.134826] i2c_hid i2c-MSHW0030:00: HID Descriptor: 1e 00 00 01 bf >>>> > 08 02 00 03 00 3f 00 04 00 3f 00 05 00 06 00 5e 04 c4 07 04 00 00 00 >>>> > 00 00 >>>> > [ 83.134928] i2c_hid i2c-MSHW0030:00: entering i2c_hid_parse >>>> > [ 83.134932] i2c_hid i2c-MSHW0030:00: i2c_hid_hwreset >>>> > [ 83.134935] i2c_hid i2c-MSHW0030:00: i2c_hid_set_power >>>> > [ 83.134938] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 00 08 >>>> > [ 83.135629] i2c_hid i2c-MSHW0030:00: resetting... >>>> > [ 83.135633] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 00 01 >>>> > [ 83.136305] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: waiting... >>>> > [ 83.138471] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: finished. >>>> > [ 83.138475] i2c_hid i2c-MSHW0030:00: asking HID report descriptor >>>> > [ 83.138477] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=02 00 >>>> > [ 83.191568] i2c_hid i2c-MSHW0030:00: Report Descriptor: 05 20 09 01 >>>> > a1 01 85 01 05 20 09 73 a1 00 0a 09 03 15 00 25 02 75 08 95 01 a1 02 >>>> > 0a 30 08 0a 31 08 0a 32 08 b1 00 c0 0a 0e 03 15 00 27 ff ff ff ff 75 >>>> > 20 95 01 55 00 b1 02 0a 52 14 15 00 26 ff >>>> > [ 83.193819] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.193822] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 31 02 >>>> > 06 00 >>>> > [ 83.194952] i2c_hid i2c-MSHW0030:00: report (len=14): 0e 00 01 01 60 >>>> > 00 00 00 14 00 01 06 02 10 >>>> > [ 83.194957] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.194959] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 32 02 >>>> > 06 00 >>>> > [ 83.196162] i2c_hid i2c-MSHW0030:00: report (len=14): 0e 00 02 01 10 >>>> > 00 00 00 00 00 01 06 02 10 >>>> > [ 83.196166] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.196167] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 33 02 >>>> > 06 00 >>>> > [ 83.196991] i2c_hid i2c-MSHW0030:00: report (len=14): 0e 00 03 01 c8 >>>> > 00 00 00 00 00 01 06 02 c8 >>>> > [ 83.196998] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.197000] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 34 02 >>>> > 06 00 >>>> > [ 83.198233] i2c_hid i2c-MSHW0030:00: report (len=30): 1e 00 04 01 64 >>>> > 00 00 00 3c 00 0a 00 64 00 0b 00 64 00 40 1f f4 01 41 1f 64 00 01 06 >>>> > 02 64 >>>> > [ 83.198238] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.198240] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 35 02 >>>> > 06 00 >>>> > [ 83.199093] i2c_hid i2c-MSHW0030:00: report (len=14): 0e 00 05 01 10 >>>> > 00 00 00 00 00 01 06 02 10 >>>> > [ 83.199096] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.199098] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 36 02 >>>> > 06 00 >>>> > [ 83.199699] i2c_hid i2c-MSHW0030:00: report (len=14): 0e 00 06 01 10 >>>> > 00 00 00 00 00 01 06 02 10 >>>> > [ 83.199703] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.199705] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 20 06 00 >>>> > [ 83.202008] i2c_hid i2c-MSHW0030:00: report (len=63): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.202011] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:63 / ret_size:0 >>>> > [ 83.202014] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.202015] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 21 06 00 >>>> > [ 83.204331] i2c_hid i2c-MSHW0030:00: report (len=63): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.204335] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:63 / ret_size:0 >>>> > [ 83.204337] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.204339] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 22 06 00 >>>> > [ 83.205447] i2c_hid i2c-MSHW0030:00: report (len=19): 13 00 22 cb a8 >>>> > 07 02 03 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.205451] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.205453] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 23 06 00 >>>> > [ 83.206745] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.206749] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.206752] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.206753] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 24 06 00 >>>> > [ 83.207780] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.207783] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.207786] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.207788] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 25 06 00 >>>> > [ 83.208803] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.208806] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.208809] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.208811] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 26 06 00 >>>> > [ 83.209824] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.209826] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.209828] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.209830] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 27 06 00 >>>> > [ 83.210588] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.210590] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.210592] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.210594] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 28 06 00 >>>> > [ 83.211659] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.211661] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.211663] i2c_hid i2c-MSHW0030:00: i2c_hid_get_report >>>> > [ 83.211665] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 3f >>>> > 02 29 06 00 >>>> > [ 83.212971] i2c_hid i2c-MSHW0030:00: report (len=19): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.212975] i2c_hid i2c-MSHW0030:00: error in i2c_hid_init_report >>>> > size:19 / ret_size:0 >>>> > [ 83.215406] i2c_hid i2c-MSHW0030:00: i2c_hid_set_power >>>> > [ 83.215410] i2c_hid i2c-MSHW0030:00: __i2c_hid_command: cmd=05 00 01 08 >>>> > [ 83.268607] i2c_hid i2c-NTRG0001:01: Fetching the HID descriptor >>>> > [ 83.268613] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=01 00 >>>> > [ 83.270130] i2c_hid i2c-NTRG0001:01: HID Descriptor: 1e 00 00 01 c7 >>>> > 01 02 00 03 00 1f 00 00 00 00 00 05 00 06 00 96 1b 05 1b 04 00 00 00 >>>> > 00 00 >>>> > [ 83.270211] i2c_hid i2c-NTRG0001:01: entering i2c_hid_parse >>>> > [ 83.270213] i2c_hid i2c-NTRG0001:01: i2c_hid_hwreset >>>> > [ 83.270215] i2c_hid i2c-NTRG0001:01: i2c_hid_set_power >>>> > [ 83.270216] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 00 08 >>>> > [ 83.270902] i2c_hid i2c-NTRG0001:01: resetting... >>>> > [ 83.270904] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 00 01 >>>> > [ 83.271685] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: waiting... >>>> > [ 83.461132] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: finished. >>>> > [ 83.461137] i2c_hid i2c-NTRG0001:01: asking HID report descriptor >>>> > [ 83.461140] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=02 00 >>>> > [ 83.471968] i2c_hid i2c-NTRG0001:01: Report Descriptor: 05 0d 09 02 >>>> > a1 01 85 01 09 20 35 00 a1 00 09 32 09 42 09 44 09 3c 09 45 15 00 25 >>>> > 01 75 01 95 05 81 02 95 03 81 03 05 01 09 30 75 10 95 01 a4 55 0e 65 >>>> > 11 46 ec 09 26 80 25 81 42 09 31 46 9d 06 >>>> > [ 83.472540] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.472542] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3c 02 >>>> > 06 00 >>>> > [ 83.473498] i2c_hid i2c-NTRG0001:01: report (len=9): 00 00 00 00 00 00 >>>> > 00 00 00 >>>> > [ 83.473507] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:9 / ret_size:0 >>>> > [ 83.473515] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.473520] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3b 02 >>>> > 06 00 >>>> > [ 83.473867] i2c_hid i2c-NTRG0001:01: report (len=5): 00 00 00 00 00 >>>> > [ 83.473872] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:5 / ret_size:0 >>>> > [ 83.473875] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.473878] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3f >>>> > 02 11 06 00 >>>> > [ 83.474493] i2c_hid i2c-NTRG0001:01: report (len=5): 03 01 1b 29 a9 >>>> > [ 83.474497] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:5 / ret_size:259 >>>> > [ 83.474500] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.474503] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3f >>>> > 02 15 06 00 >>>> > [ 83.475186] i2c_hid i2c-NTRG0001:01: report (len=8): 03 01 1b 29 a9 19 >>>> > 9f 9a >>>> > [ 83.475190] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:8 / ret_size:259 >>>> > [ 83.475192] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.475195] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3f >>>> > 02 18 06 00 >>>> > [ 83.476035] i2c_hid i2c-NTRG0001:01: report (len=15): 03 01 1b 29 a9 >>>> > 19 9f 9a 19 a4 bf dd c2 f3 29 >>>> > [ 83.476039] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:15 / ret_size:259 >>>> > [ 83.476041] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.476044] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 34 02 >>>> > 06 00 >>>> > [ 83.476354] i2c_hid i2c-NTRG0001:01: report (len=4): 04 00 04 0a >>>> > [ 83.476360] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.476363] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3a 02 >>>> > 06 00 >>>> > [ 83.476673] i2c_hid i2c-NTRG0001:01: report (len=4): 00 00 00 00 >>>> > [ 83.476676] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:4 / ret_size:0 >>>> > [ 83.476679] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.476682] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 3f >>>> > 02 1b 06 00 >>>> > [ 83.482978] i2c_hid i2c-NTRG0001:01: report (len=259): 03 01 1b 29 a9 >>>> > 19 9f 9a 19 a4 bf dd c2 f3 29 e1 34 44 89 23 3f 8c 56 79 f3 fc 57 d0 >>>> > ea 56 46 23 9a 15 d8 5a de 99 5e 2f fc 9f e2 f4 73 df d5 c5 36 ed 8d >>>> > 4e 28 db 8c 34 a3 b0 19 5c 4f ae bf 9c >>>> > [ 83.482986] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.482988] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 35 02 >>>> > 06 00 >>>> > [ 83.485124] i2c_hid i2c-NTRG0001:01: report (len=63): 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > 00 00 00 00 00 00 00 00 00 00 00 00 >>>> > [ 83.485127] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:63 / ret_size:0 >>>> > [ 83.485128] i2c_hid i2c-NTRG0001:01: i2c_hid_get_report >>>> > [ 83.485130] i2c_hid i2c-NTRG0001:01: __i2c_hid_command: cmd=05 00 37 02 >>>> > 06 00 >>>> > [ 83.486010] i2c_hid i2c-NTRG0001:01: report (len=7): 00 00 00 00 00 00 >>>> > 00 >>>> > [ 83.486012] i2c_hid i2c-NTRG0001:01: error in i2c_hid_init_report >>>> > size:7 / ret_size:0 >>>> > [ 83.486096] input: NTRG0001:01 1B96:1B05 Pen as >>>> > >>>> > /devices/pci0000:00/INT33C3:00/i2c-10/i2c-NTRG0001:01/0018:1B96:1B05.0008/input/input16 >>>> > [ 83.486306] input: NTRG0001:01 1B96:1B05 as >>>> > >>>> > /devices/pci0000:00/INT33C3:00/i2c-10/i2c-NTRG0001:01/0018:1B96:1B05.0008/input/input17 >>>> > [ 83.488543] hid-multitouch 0018:1B96:1B05.0008: input,hidraw4: >>>> > <UNKNOWN> HID v1.00 Mouse [NTRG0001:01 1B96:1B05] on >>>> > -- >>>> > To unsubscribe from this list: send the line "unsubscribe linux-input" >>>> > in >>>> > the body of a message to majordomo@xxxxxxxxxxxxxxx >>>> > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
