linux-kernel - 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: <5c691c83-480b-8731-7e6a-7f28536c9a59@kernel.org>
Date:   Wed, 25 May 2022 08:44:38 +0200
From:   Jiri Slaby <jirislaby@...nel.org>
To:     Mauro Carvalho Chehab <mchehab@...nel.org>,
        Linus Torvalds <torvalds@...ux-foundation.org>
Cc:     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>,
        Sean Young <sean@...s.org>
Subject: lirc build broken [was: [GIT PULL for v5.18-rc1] media updates]

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.)

> thanks,-- 
js