From: James Hogan <james@albanarts.com>
To: Mauro Carvalho Chehab <m.chehab@samsung.com>
Cc: "Antti Seppälä" <a.seppala@gmail.com>,
linux-media@vger.kernel.org, "David Härdeman" <david@hardeman.nu>,
"Jarod Wilson" <jarod@redhat.com>,
"Wei Yongjun" <yongjun_wei@trendmicro.com.cn>,
"Hans Verkuil" <hans.verkuil@cisco.com>
Subject: Re: [PATCH v2 0/9] rc: Add IR encode based wakeup filtering
Date: Fri, 25 Jul 2014 21:46:58 +0100 [thread overview]
Message-ID: <1572764.UGr4uaq7NC@radagast> (raw)
In-Reply-To: <20140723163936.164aa577.m.chehab@samsung.com>
Hi Mauro,
On Wednesday 23 July 2014 16:39:36 Mauro Carvalho Chehab wrote:
> Em Fri, 14 Mar 2014 23:04:10 +0000
>
> James Hogan <james@albanarts.com> escreveu:
> > A recent discussion about proposed interfaces for setting up the
> > hardware wakeup filter lead to the conclusion that it could help to have
> > the generic capability to encode and modulate scancodes into raw IR
> > events so that drivers for hardware with a low level wake filter (on the
> > level of pulse/space durations) can still easily implement the higher
> > level scancode interface that is proposed.
> >
> > I posted an RFC patchset showing how this could work, and Antti Seppälä
> > posted additional patches to support rc5-sz and nuvoton-cir. This
> > patchset improves the original RFC patches and combines & updates
> > Antti's patches.
> >
> > I'm happy these patches are a good start at tackling the problem, as
> > long as Antti is happy with them and they work for him of course.
> >
> > Future work could include:
> > - Encoders for more protocols.
> > - Carrier signal events (no use unless a driver makes use of it).
> >
> > Patch 1 adds the new encode API.
> > Patches 2-3 adds some modulation helpers.
> > Patches 4-6 adds some raw encode implementations.
> > Patch 7 adds some rc-core support for encode based wakeup filtering.
> > Patch 8 adds debug loopback of encoded scancode when filter set.
> > Patch 9 (untested) adds encode based wakeup filtering to nuvoton-cir.
>
> > Changes in v2:
> Any news about this patch series? There are some comments about them,
> so I'll be tagging it as "changes requested" at patchwork, waiting
> for a v3 (or is it already there in the middle of the 49 patches from
> David?).
This patch series seems to have been forgotten. I do have a few changes on top
of v2 to address the review comments, so as you say I should probably rebase
and do a v3 at some point.
Cheers
James
prev parent reply other threads:[~2014-07-25 20:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-14 23:04 [PATCH v2 0/9] rc: Add IR encode based wakeup filtering James Hogan
2014-03-14 23:04 ` [PATCH v2 1/9] rc: ir-raw: Add scancode encoder callback James Hogan
2014-03-14 23:04 ` [PATCH v2 2/9] rc: ir-raw: Add pulse-distance modulation helper James Hogan
2014-03-14 23:04 ` [PATCH v2 3/9] rc: ir-raw: Add Manchester encoder (phase encoder) helper James Hogan
2014-03-14 23:04 ` [PATCH v2 4/9] rc: ir-nec-decoder: Add encode capability James Hogan
2014-03-14 23:04 ` [PATCH v2 5/9] rc: ir-rc5-decoder: " James Hogan
2014-03-14 23:04 ` [PATCH v2 6/9] rc: ir-rc5-sz-decoder: Add ir encoding support James Hogan
2014-03-16 8:34 ` Antti Seppälä
2014-03-16 11:50 ` James Hogan
2014-03-16 12:14 ` Antti Seppälä
2014-03-16 21:18 ` James Hogan
2014-03-17 16:34 ` Antti Seppälä
2014-03-14 23:04 ` [PATCH v2 7/9] rc: rc-core: Add support for encode_wakeup drivers James Hogan
2014-03-14 23:04 ` [PATCH v2 8/9] rc: rc-loopback: Add loopback of filter scancodes James Hogan
2014-03-14 23:04 ` [PATCH v2 9/9] rc: nuvoton-cir: Add support for writing wakeup samples via sysfs filter callback James Hogan
2014-03-16 8:39 ` Antti Seppälä
2014-03-16 11:52 ` James Hogan
2014-03-16 8:22 ` [PATCH v2 0/9] rc: Add IR encode based wakeup filtering Antti Seppälä
2014-03-16 22:41 ` James Hogan
2014-03-17 17:01 ` Antti Seppälä
2014-03-17 22:34 ` James Hogan
2014-03-25 0:15 ` David Härdeman
2014-07-23 19:39 ` Mauro Carvalho Chehab
2014-07-25 20:46 ` James Hogan [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=1572764.UGr4uaq7NC@radagast \
--to=james@albanarts.com \
--cc=a.seppala@gmail.com \
--cc=david@hardeman.nu \
--cc=hans.verkuil@cisco.com \
--cc=jarod@redhat.com \
--cc=linux-media@vger.kernel.org \
--cc=m.chehab@samsung.com \
--cc=yongjun_wei@trendmicro.com.cn \
/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.