Re: [PATCH v2 3/6] hid: sony: Add support for the Sony Motion Controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> Add a fixed-up HID descriptor for the Sony motion controller and enable
controls for the LED light as well as force-feedback.

Hi,
I think that a mistake has been made here with the report descriptor, it
looks like it is an exact duplicate of the one for the six-axis and it
does not match the documented layout from the psmove-api guys.

https://github.com/nitsch/moveonpc/wiki/Input-report

The document above shows that there are two copies of axis data (frame1
and frame2) for many of them, what is the best way of handling this in
kernel? Should we just ignore the 2nd?

I have attached the 'original' reports (ie. with hid-sony changing them)
for reference. Note the BT one has a trailing zero (like the DS4 has)
which prevents decoding... I just cropped it off to decode.
Simon.

Attachment: psmove_bt_report_orig.bin
Description: Binary data

Attachment: psmove_bt_report_orig.code
Description: Binary data

Attachment: psmove_usb_report_orig.bin
Description: Binary data

Attachment: psmove_usb_report_orig.code
Description: Binary data


[Index of Archives]     [Linux Media Devel]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Linux Wireless Networking]     [Linux Omap]

  Powered by Linux