From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Jarod Wilson <jarod@redhat.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH v3] IR/mceusb: kill pinnacle-device-specific nonsense
Date: Sat, 03 Jul 2010 19:17:12 -0300 [thread overview]
Message-ID: <4C2FB6E8.5090001@redhat.com> (raw)
In-Reply-To: <20100703040227.GA31255@redhat.com>
Em 03-07-2010 01:02, Jarod Wilson escreveu:
> I have pinnacle hardware now. None of this pinnacle-specific crap is at
> all necessary (in fact, some of it needed to be removed to actually make
> it work). The only thing unique about this device is that it often
> transfers inbound data w/a header of 0x90, meaning 16 bytes of IR data
> following it, so I had to make adjustments for that, and now its working
> perfectly fine.
>
> v2: stillborn
>
> v3: remove completely unnecessary usb_reset_device() call that only served
> to piss off the pinnacle device regularly and unify/simplify some of the
> generation-specific device initialization code.
>
> post-mortem: it seems the pinnacle hardware actually still gets pissed off
> from time to time, but I can (try) to fix that later (if possible). The
> patch is still quite helpful from a code reduction standpoint.
>
> Signed-off-by: Jarod Wilson <jarod@redhat.com>
I've already applied a previous version of this patch:
http://git.linuxtv.org/v4l-dvb.git?a=commit;h=afd46362e573276e3fb0a44834ad320c97947233
Could you please rebase it against my tree?
> ---
> Makefile | 2 +-
> drivers/media/IR/mceusb.c | 110 +++++++++------------------------------------
> 2 files changed, 22 insertions(+), 90 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 6e39ec7..0417c74 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1,7 +1,7 @@
> VERSION = 2
> PATCHLEVEL = 6
> SUBLEVEL = 35
> -EXTRAVERSION = -rc1
> +EXTRAVERSION = -rc1-ir
Please, don't patch the upstream Makefile ;)
Cheers,
Mauro.
next prev parent reply other threads:[~2010-07-03 22:17 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-16 20:10 [PATCH] IR/mceusb: kill pinnacle-device-specific nonsense Jarod Wilson
2010-07-03 4:02 ` [PATCH v3] " Jarod Wilson
2010-07-03 22:17 ` Mauro Carvalho Chehab [this message]
2010-07-04 0:41 ` [PATCH] " Jarod Wilson
2010-07-04 1:42 ` [PATCH] IR/mceusb: unify and simplify different gen device init Jarod Wilson
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=4C2FB6E8.5090001@redhat.com \
--to=mchehab@redhat.com \
--cc=jarod@redhat.com \
--cc=linux-media@vger.kernel.org \
/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.