Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 25 May 2022 09:57:27 +0100
From:   Sean Young <sean@mess.org>
To:     Jiri Slaby <jirislaby@kernel.org>
Cc:     Mauro Carvalho Chehab <mchehab@kernel.org>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Linux Media Mailing List <linux-media@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: lirc build broken [was: [GIT PULL for v5.18-rc1] media updates]
Message-ID: <Yo3vd4QRmwZsRhlH@gofer.mess.org>
References: <20220322101406.459e2950@coco.lan>
 <a0470450-ecfd-2918-e04a-7b57c1fd7694@kernel.org>
 <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

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