[lkml]   [2010]   [Sep]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 1/7 v3] HID: magicmouse: don't allow hidinput to initialize the device
    On Thu, Sep 2, 2010 at 5:28 AM, Jiri Slaby wrote:
    > On 09/02/2010 01:57 AM, Michael Poole wrote:
    >> Jiri Slaby wrote:
    >>> This looks weird. Is there any past discussion about why you cannot use
    >>> hidinput and you have to do all the input bits yourself while cheating
    >>> this very weird way?
    >> The Magic Mouse and Magic Trackpad do not publish HID descriptors for
    >> the multitouch reports.  Given the variable-length report packets --
    >> and especially the Magic Trackpad's new, mutant DOUBLE_REPORT_ID
    >> packets -- it would be non-trivial to write accurate descriptors that
    >> the HID core can use.  (Someone wrote a patch to try that a few months
    >> ago.  It ended up adding significantly more lines to hid-magicmouse.c
    >> than it removed, and it was not obvious to me that it got the Report
    >> Count fields correct.)
    > Ok, makes sense. The proper solution is to call a driver hook in
    > hidinput_connect to do the mapping instead of report parsing done there,
    > right? (And not setting [gs]etkeycode in that case.)

    I do not know how this would work with the current HID core; could you
    elaborate? For the Magic Mouse, there is a valid descriptor for the
    0x10 (traditional mouse motion and clicks) report, and a pretty
    trivial descriptor in the vendor-defined usage region (which I assume
    Apple drivers use to identify the ensemble of multi-touch,
    mouse-up/down, laser status and battery level reports). How would the
    driver's input_mapping() hook, or any other hook, map that single
    descriptor into fields for each of the parts of a multi-touch report,
    or support the other reports? How would hid-magicmouse describe the
    report so that hid-core handles the variable-length multi-touch
    reports properly? As far as I can tell, hid_report_raw_event() and
    its callees assume each report has fixed length.

    > Then the part of magicmouse_setup_input starting at the first __set_bit
    > will be exactly the hook.
    > Otherwise this hack looks like a hard cross-layer breakage.
    > /me wonders how HID issuers are creative in breaking standards.

    In Apple's case, apparently fairly creative. Devices from other
    sources seem to generate one report per touch, rather than cram as
    many touches as possible into a single report.

    Michael Poole
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2010-09-02 14:09    [W:0.032 / U:2.040 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site