linux-kernel - Re: lirc build broken [was: [GIT PULL for v5.18-rc1] media updates]

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Yo3vd4QRmwZsRhlH@gofer.mess.org>
Date:   Wed, 25 May 2022 09:57:27 +0100
From:   Sean Young <sean@...s.org>
To:     Jiri Slaby <jirislaby@...nel.org>
Cc:     Mauro Carvalho Chehab <mchehab@...nel.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Linux Media Mailing List <linux-media@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: lirc build broken [was: [GIT PULL for v5.18-rc1] media updates]

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