From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 97449C433EF for ; Tue, 5 Jul 2022 07:25:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230347AbiGEHZx (ORCPT ); Tue, 5 Jul 2022 03:25:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53458 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231182AbiGEHZv (ORCPT ); Tue, 5 Jul 2022 03:25:51 -0400 Received: from gofer.mess.org (gofer.mess.org [IPv6:2a02:8011:d000:212::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7400A2615 for ; Tue, 5 Jul 2022 00:25:50 -0700 (PDT) Received: by gofer.mess.org (Postfix, from userid 1000) id 8E4661002A0; Tue, 5 Jul 2022 08:25:48 +0100 (BST) Date: Tue, 5 Jul 2022 08:25:48 +0100 From: Sean Young To: Marko =?iso-8859-1?B?TeRrZWzk?= Cc: linux-media@vger.kernel.org Subject: Re: [PATCH 2/2] media: rtl28xxu: improve IR receiver Message-ID: References: <704b3d7e5a7a95cbd5e4dfc25a41454e919aed95.1644683294.git.sean@mess.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hi Marko, 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. https://www.kernel.org/doc/html/latest/process/submitting-patches.html#no-mime-no-links-no-compression-no-attachments-just-plain-text > > 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? Sean