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 9D485C433FE for ; Wed, 25 May 2022 08:57:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234634AbiEYI5d (ORCPT ); Wed, 25 May 2022 04:57:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43040 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236620AbiEYI5a (ORCPT ); Wed, 25 May 2022 04:57:30 -0400 Received: from gofer.mess.org (gofer.mess.org [IPv6:2a02:8011:d000:212::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E27526FA2B; Wed, 25 May 2022 01:57:28 -0700 (PDT) Received: by gofer.mess.org (Postfix, from userid 1000) id 134C2101CCD; Wed, 25 May 2022 09:57:27 +0100 (BST) Date: Wed, 25 May 2022 09:57:27 +0100 From: Sean Young To: Jiri Slaby Cc: Mauro Carvalho Chehab , Linus Torvalds , Greg Kroah-Hartman , Andrew Morton , Linux Media Mailing List , Linux Kernel Mailing List Subject: Re: lirc build broken [was: [GIT PULL for v5.18-rc1] media updates] Message-ID: References: <20220322101406.459e2950@coco.lan> <5c691c83-480b-8731-7e6a-7f28536c9a59@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <5c691c83-480b-8731-7e6a-7f28536c9a59@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 25, 2022 at 08:44:38AM +0200, Jiri Slaby wrote: > On 25. 05. 22, 8:42, Jiri Slaby wrote: > > On 22. 03. 22, 10:14, Mauro Carvalho Chehab wrote: > > > Hi Linus, > > > > > > Please pull from: > > >    git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media > > > tags/media/v5.18-1 > > ... > > > Sean Young (10): > > ... > > >        media: lirc: remove unused lirc features > > > > Hi, > > > > this breaks lirc build: > > > [   59s] lircd.cpp:489:49: error: 'LIRC_CAN_SET_REC_FILTER' was not > > > declared in this scope; did you mean 'LIRC_CAN_SET_REC_CARRIER'? > > > [   59s]   489 |                     || (curr_driver->features & > > > LIRC_CAN_SET_REC_FILTER)) { > > > [   59s]       | > > > ^~~~~~~~~~~~~~~~~~~~~~~ > > > [   59s]       | > > > LIRC_CAN_SET_REC_CARRIER > > > [   59s] lircd.cpp: In function 'void loop()': > > > [   59s] lircd.cpp:2069:82: error: 'LIRC_CAN_NOTIFY_DECODE' was not > > > declared in this scope; did you mean 'DRVCTL_NOTIFY_DECODE'? > > > [   59s]  2069 |                         if > > > (curr_driver->drvctl_func && (curr_driver->features & > > > LIRC_CAN_NOTIFY_DECODE)) > > > [   59s] | > > > ^~~~~~~~~~~~~~~~~~~~~~ > > > [   59s] | > > > DRVCTL_NOTIFY_DECODE > > > > So the uapi header defines should be brought back, IMO. > > (And lirc fixed at the same time.) What is broken? Sean