Re: [PATCH] lirc.4: remove ioctls and feature bits which were never implemented

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael,

On Thu, Jul 12, 2018 at 02:21:18PM +0100, Sean Young wrote:
> On Thu, Jul 12, 2018 at 09:33:32AM -0300, Mauro Carvalho Chehab wrote:
> > Hi Michael/Alec,
> > 
> > Em Fri, 18 May 2018 16:25:29 +0100
> > Sean Young <sean@xxxxxxxx> escreveu:
> > 
> > > On Sun, May 06, 2018 at 12:34:53PM +0200, Michael Kerrisk (man-opages) wrote:
> > > > [CCing original author of this page]
> > > > 
> > > > 
> > > > On 04/23/2018 12:26 PM, Sean Young wrote:  
> > > > > The lirc header file included ioctls and feature bits which were never
> > > > > implemented by any driver. They were removed in commit:
> > > > > 
> > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=d55f09abe24b4dfadab246b6f217da547361cdb6  
> > > > 
> > > > Alec, does this patch look okay to you? 
> > 
> > Sean is the sub-maintainer responsible for the LIRC code at the
> > media subsystem. He knows more about the current implementation
> > than anyone else, as he's working hard to improve it, and got
> > rid of all legacy LIRC drivers from staging (either fixing them
> > or removing the few ones nobody uses anymore).
> > 
> > As part of his work, some ioctls got removed, in order to make
> > the LIRC interface to match the real implementation.
> >  
> > > Mauro, as Alec is not responding, would you be able to sign this off?
> > 
> > Most of the patch looks ok on my eyes. I noticed that some flags
> > still exists at include/uapi/linux/lirc.h:
> > 
> > 	LIRC_CAN_REC_RAW, LIRC_CAN_REC_PULSE, LIRC_CAN_SET_REC_FILTER
> > 	and LIRC_CAN_SEND_MODE2
> > 
> > Maybe instead of just removing, you would need to add some
> > explanation about them (or at the patch itself, explaining
> > why you're removing the descriptions for them).
> 
> Those flags do still exist in the header file, we decided to keep them
> so that code does not suddenly fail to build. These flags either never
> had implementations or only had out-of-tree implementations. So, I do
> not think they belong in the man page.
> 
> > > Alternatively, what can be done to progress this?
> > > 
> > > There is some new functionality in lirc which should be added to this man
> > > page too, so I have more to come (when I get round to writing it).
> > 
> > Yeah, making it reflect upstream sounds the right thing to do.
> 
> Absolutely, when kernel v4.18 is released there is more to add.

Ping, can this patch be merged please?

The lirc.4 man page is really out of date and misleading in parts. 


Sean



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux