linux-media.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "David Härdeman" <david@hardeman.nu>
To: Jarod Wilson <jarod@redhat.com>
Cc: linux-media@vger.kernel.org, mchehab@redhat.com, sean@mess.org
Subject: Re: [PATCH 0/8] rc-core: patches for 3.7
Date: Mon, 3 Sep 2012 23:55:21 +0200	[thread overview]
Message-ID: <20120903215521.GA6675@hardeman.nu> (raw)
In-Reply-To: <20120830195612.GA13026@redhat.com>

On Thu, Aug 30, 2012 at 03:56:12PM -0400, Jarod Wilson wrote:
>On Sat, Aug 25, 2012 at 11:46:47PM +0200, David Härdeman wrote:
>> This is two minor winbond-cir fixes as well as the first six patches
>> from my previous patchbomb.
>> 
>> The latter have been modified so that backwards compatibility is retained
>> as much as possible (the format of the sysfs files do not change for
>> example).
>
>I've read through the set, and it all seems to make sense to me, but I
>haven't actually tried it out with any of the hardware I've got. I assume
>its been tested on various other hardware though.

I've tested the patches on mceusb hardware (RX only) and using some
scripted TX/RX testing with rc-loopback. I haven't tested (this latest
version) on winbond-cir hardware yet as I'm travelling.

>Side note: my life has been turned a wee bit upside down, been busy
>dealing with some fairly big changes, and that's still ongoing, thus the
>relative lack of repsonsiveness on, well, anything, lately.

If you have limited time and bandwidth I'd suggest that you focus on the
API changes - in this particular patchset that would be the addition of
the "struct rc_keymap_entry" to the EVIOC[GS]KEYCODE_V2 ioctl (maybe we
should involve the input maintainer as well?).

Getting the API right would benefit from you cooperation, getting bugs
worked out is hopefully something which can be done with or without you.

Thanks,
David


      reply	other threads:[~2012-09-03 21:55 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-25 21:46 [PATCH 0/8] rc-core: patches for 3.7 David Härdeman
2012-08-25 21:46 ` [PATCH 1/8] winbond-cir: correctness fix David Härdeman
2012-08-25 21:46 ` [PATCH 2/8] winbond-cir: asynchronous tx David Härdeman
2012-08-25 21:47 ` [PATCH 3/8] rc-core: add separate defines for protocol bitmaps and numbers David Härdeman
2012-09-25 20:19   ` Mauro Carvalho Chehab
2012-08-25 21:47 ` [PATCH 4/8] rc-core: don't throw away protocol information David Härdeman
2012-09-25 18:40   ` Mauro Carvalho Chehab
2012-08-25 21:47 ` [PATCH 5/8] rc-core: use the full 32 bits for NEC scancodes David Härdeman
2012-08-25 21:47 ` [PATCH 6/8] rc-core: merge rc5 and streamzap decoders David Härdeman
2012-08-25 21:47 ` [PATCH 7/8] rc-core: rename ir_input_class to rc_class David Härdeman
2012-08-25 21:47 ` [PATCH 8/8] rc-core: initialize rc-core earlier if built-in David Härdeman
2012-08-30 19:56 ` [PATCH 0/8] rc-core: patches for 3.7 Jarod Wilson
2012-09-03 21:55   ` David Härdeman [this message]

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=20120903215521.GA6675@hardeman.nu \
    --to=david@hardeman.nu \
    --cc=jarod@redhat.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@redhat.com \
    --cc=sean@mess.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).