From: "Marko Mäkelä" <marko.makela@iki.fi>
To: Sean Young <sean@mess.org>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH 2/2] media: rtl28xxu: improve IR receiver
Date: Tue, 5 Jul 2022 11:48:50 +0300 [thread overview]
Message-ID: <YsP68l7IuktIiMLD@jyty> (raw)
In-Reply-To: <YsPnfBYnW2GJk4Hn@gofer.mess.org>
Hi Sean,
Tue, Jul 05, 2022 at 08:25:48AM +0100, Sean Young wrote:
>On Mon, Jul 04, 2022 at 10:04:38PM +0300, Marko Mäkelä wrote:
>> Mon, Jul 04, 2022 at 11:00:38AM +0100, Sean Young wrote:
>> > On Mon, Jul 04, 2022 at 12:20:01PM +0300, Marko Mäkelä wrote:
>> > > Mon, Jul 04, 2022 at 08:21:39AM +0100, Sean Young wrote:
>> > > > On Sun, Jul 03, 2022 at 08:02:14PM +0300, Marko Mäkelä wrote:
>> > > I tested the attached patch (which was created on 5.19-rc5, which
>> > > failed to boot on my system for unrelated reasons) on Linux 5.17, on
>> > > top of your fixes to rtl28xxu and rc-core.
>> >
>> > You'll need to fix this.
>>
>> The 5.19-rc5 boot failure could have been related to LUKS setup on that
>> machine, because a kernel panic message was displayed before I was being
>> prompted for an encryption key. The modules would not have been loaded at
>> that point, so I do not think that it is related to my modifications.
>>
>> When compiled for the v5.17 kernel release tag on another computer, the
>> patch that implements rc_keydown_or_repeat() worked for me.
>>
>> It does not look like there are many changes in drivers/media/rc between
>> 5.17 and 5.19.
>
>Your patch needs a `Signed-off-by` and it should not be attached, it should
>be inline in your email.
Thank you for your patience. I hope that I got it right. It would be my
very first patch submission to the Linux kernel. I did not see it appear
on this list archive yet. You are Cc'd.
>> > See https://github.com/seanyoung/cir/
>>
>> This could open up many possibilities. Would the decoded events also be
>> available via some low-level interface to user-space programs, in addition
>> to the input event driver?
>
>The plan was for it to run once, generate an eBPF program, attach that an
>exit. The eBPF program sends the decoded stuff to the lirc chardev in
>this struct:
>
>https://www.kernel.org/doc/html/latest/userspace-api/media/rc/lirc-dev-intro.html#data-types-used-by-lirc-mode-scancode
>
>This is the struct you're amending with LIRC_SCANCODE_FLAG_REPEAT.
>
>Will that be sufficient for your needs?
I think that it should cover the most common types of remote control
units.
I can name an example of a complex IR remote control, which I think
would be challenging to repurpose for controlling anything else than the
original type of device. But, I would think that something Bluetooth or
WLAN based on a touchscreen device will replace IR in such applications.
The remote control of my air conditioner presents all settings on a
local LCD. On every change, maybe after a short timeout of inactivity,
it will send a long IR message with all the settings. The 32 bits of
keycode or 64 bits of scancode would not be sufficient for that.
Marko
next prev parent reply other threads:[~2022-07-05 8:48 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-12 16:32 [PATCH 0/2] Fix rtl28xxu nec/rc5 receiver Sean Young
2022-02-12 16:32 ` [PATCH 1/2] media: rc-core: split IR timeout into rawir timeout and keyup delay Sean Young
2022-02-12 16:32 ` [PATCH 2/2] media: rtl28xxu: improve IR receiver Sean Young
2022-06-26 12:33 ` Marko Mäkelä
2022-06-27 5:00 ` Marko Mäkelä
2022-07-02 8:17 ` Sean Young
2022-06-27 10:53 ` Sean Young
2022-06-28 6:27 ` Marko Mäkelä
2022-07-02 8:14 ` Sean Young
2022-07-03 17:02 ` Marko Mäkelä
2022-07-04 7:21 ` Sean Young
2022-07-04 9:20 ` Marko Mäkelä
2022-07-04 10:00 ` Sean Young
2022-07-04 19:04 ` Marko Mäkelä
2022-07-05 7:25 ` Sean Young
2022-07-05 8:48 ` Marko Mäkelä [this message]
2022-07-05 9:26 ` Sean Young
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=YsP68l7IuktIiMLD@jyty \
--to=marko.makela@iki.fi \
--cc=linux-media@vger.kernel.org \
--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 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.