From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Brian Magnuson" Subject: Re: [PATCH] new driver for wireless xbox receiver for review Date: Wed, 30 May 2007 10:51:05 -0400 Message-ID: References: <20070529043957.GB6157@rcn.com> <20070530002846.GA8595@rcn.com> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_5685_19582898.1180536665443" Return-path: In-Reply-To: Sender: owner-linux-input@atrey.karlin.mff.cuni.cz List-Help: List-Owner: List-Post: List-Unsubscribe: To: Jan Kratochvil Cc: Dmitry Torokhov , linux-input@atrey.karlin.mff.cuni.cz List-Id: linux-input@vger.kernel.org ------=_Part_5685_19582898.1180536665443 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline Hi, First off, are there list archives for linux-input? I wasn't able to find any and http://atrey.karlin.mff.cuni.cz/lab/listserv.html is a dead link. Remaining comments inline. On 5/30/07, Jan Kratochvil wrote: > > Hello, > > >> > Knew that was coming :). Should be able to manage it. Since the > >> wireless > >> > model returns a expanded data format it needs to be handled > specially. > >> > Another flag in the xpad_device struct? > >> > >> We have xtype field, I think it should be used. > > > > > > I don't see this field in xpad_device in your tree or Linus'. > > It is not yet in Linus' tree, but it definitely is in Dmitry input tree. > > http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=commit;h=bad058541d848500b5f2cc049c1082d0b2935b7d Indeed it is. I think this is just some basic git ignorance on my part. I clearly see it in the link you provided but if I descend from the tree link starting from top commit at http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=summary I get some older version of xpad.c. Am I just being dense (likely) or is there something wrong? I noticed in your patch that you worked on some old version of xpad. You'll > see that in dmitry tree there is support for rumble as well as other minor > changes. (And I submited support for controlling leds yesterday) > Btw you are stating that it process packet is based on information from > http://euc.jp/periphs/xbox-controller.en.html, which I doubt, because this > page > is actually about xbox (1) controller. Not xbox 360. But on the other hand > you > are interpreting the report correctly. :) Turns out I started developing from a version of xpad.c distributed with Ubuntu which 1) is old and 2) appears to contain some extra patches on top of that. That link you refer to is a a stale comment. I'll remove it. > > > In any case I'll take these changes and your earlier ones, merge it into > > xpad.c and > > resubmit. > > Good luck, > Jan Thanks :) -Brian ------=_Part_5685_19582898.1180536665443 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Hi,

First off,  are there list archives for linux-input?  I wasn't able to find any and
http://atrey.karlin.mff.cuni.cz/lab/listserv.html is a dead link.  Remaining
comments inline.

On 5/30/07, Jan Kratochvil <honza@jikos.cz> wrote:
Hello,

>> > Knew that was coming :).  Should be able to manage it.  Since the
>> wireless
>> > model returns a expanded data format it needs to be handled specially.
>> > Another flag in the xpad_device struct?
>>
>> We have xtype field, I think it should be used.
>
>
> I don't see this field in xpad_device in your tree or Linus'.

It is not yet in Linus' tree, but it definitely is in Dmitry input tree.
http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=commit;h=bad058541d848500b5f2cc049c1082d0b2935b7d

Indeed it is.  I think this is just some basic git ignorance on my part.  I clearly see it in
the link you provided but if I descend from the tree link starting from top commit at
http://git.kernel.org/?p=linux/kernel/git/dtor/input.git;a=summary I get some older
version of xpad.c.  Am I just being dense (likely) or is there something wrong?

I noticed in your patch that you worked on some old version of xpad. You'll
see that in dmitry tree there is support for rumble as well as other minor
changes. (And I submited support for controlling leds yesterday)
Btw you are stating that it process packet is based on information from
http://euc.jp/periphs/xbox-controller.en.html, which I doubt, because this page
is actually about xbox (1) controller. Not xbox 360. But on the other hand you
are interpreting the report correctly. :)

Turns out I started developing from a version of xpad.c  distributed with Ubuntu which
1) is old and 2) appears to contain some extra patches on top of that.  That link you
refer to is a a stale comment. I'll remove it.

>
> In any case I'll take these changes and your earlier ones, merge it into
> xpad.c and
> resubmit.

Good luck,
Jan

Thanks :)

-Brian
------=_Part_5685_19582898.1180536665443--