All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frank Praznik <frank.praznik@gmail.com>
To: Pavel Machek <pavel@ucw.cz>, Frank Praznik <frank.praznik@oh.rr.com>
Cc: linux-input@vger.kernel.org, jkosina@suse.cz
Subject: Re: [PATCH 2/5] hid: sony: Add support for the Sony Motion Controller
Date: Tue, 05 May 2015 19:28:46 -0400	[thread overview]
Message-ID: <5549522E.9010107@gmail.com> (raw)
In-Reply-To: <20150505211640.GA10272@amd>

On 5/5/2015 17:16, Pavel Machek wrote:
> Hi!
>
>> Add a fixed-up HID descriptor for the Sony motion controller and enable
>> controls for the LED light as well as force-feedback.
>>
>> The LED is multi-colored (red, green, blue) and has a 'global' control to
>> allow it to be flashed or switched on an off atomically.
>>
>> The motion controller has a single rumble motor so the higher of the left and
>> right values is used to set the speed.
>>
>> Signed-off-by: Pavel Machek <pavel@ucw.cz>
>> Signed-off-by: Frank Praznik <frank.praznik@oh.rr.com>
>> ---
>>
>>   Pavel, can you please test this and make sure it works?
> I tried (on 4.1-rc2), and got this (full dmesg in attachment)
>
> Best regards,
> 										Pavel
>
<snip>

Ack, sorry about that mistake.  On the plus side, it uncovered a 
mishandled failure path in the initialization function.  I'll add a 
patch for that in v2 of the series.

  parent reply	other threads:[~2015-05-05 23:28 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-05-05  2:34 [PATCH 0/5] Support for the Sony Motion Controller and other cleanups Frank Praznik
2015-05-05  2:34 ` [PATCH 1/5] hid: sony: Add the product ID for the Sony Motion Controller Frank Praznik
2015-05-05  2:34 ` [PATCH 2/5] hid: sony: Add support " Frank Praznik
2015-05-05 21:16   ` Pavel Machek
2015-05-05 21:24     ` Pavel Machek
2015-05-05 21:36     ` Pavel Machek
2015-05-05 21:47       ` Pavel Machek
2015-05-05 21:55         ` Pavel Machek
2015-05-05 23:37         ` Frank Praznik
2015-05-06  9:31           ` Pavel Machek
2015-05-05 23:28     ` Frank Praznik [this message]
2015-05-05  2:34 ` [PATCH 3/5] hid: sony: Correct Sony device ordering Frank Praznik
2015-05-05  2:34 ` [PATCH 4/5] hid: sony: Simplify LED initialization and eliminate redundant copies when updating LED states Frank Praznik
2015-05-05  2:34 ` [PATCH 5/5] hid: sony: Correct a typo in a HID descriptor comment and explain the odd Sixaxis axis mapping Frank Praznik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5549522E.9010107@gmail.com \
    --to=frank.praznik@gmail.com \
    --cc=frank.praznik@oh.rr.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=pavel@ucw.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.