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

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

Regards,
Mauro

> 
> Thanks,
> 
> Sean
> > 
> > Cheers,
> > 
> > Michael
> >   
> > > Signed-off-by: Sean Young <sean@xxxxxxxx>
> > > ---
> > >   man4/lirc.4 | 92 ++-----------------------------------------------------------
> > >   1 file changed, 2 insertions(+), 90 deletions(-)
> > > 
> > > diff --git a/man4/lirc.4 b/man4/lirc.4
> > > index 1e94a7163..3adff55f1 100644
> > > --- a/man4/lirc.4
> > > +++ b/man4/lirc.4
> > > @@ -78,9 +78,7 @@ The package reflects a timeout; see the
> > >   .B LIRC_SET_REC_TIMEOUT_REPORTS
> > >   ioctl.
> > >   .\"
> > > -.SS Reading input with the
> > > -.B LIRC_MODE_LIRCCODE
> > > -drivers
> > > +.SS Reading input with the LIRC_MODE_LIRCCODE drivers
> > >   .PP
> > >   In the \fBLIRC_MODE_LIRCCODE\fR
> > >   mode, the data returned by
> > > @@ -204,17 +202,11 @@ Currently serves no purpose since only
> > >   .BR LIRC_MODE_PULSE
> > >   is supported.
> > >   .TP
> > > -.BR LIRC_GET_SEND_CARRIER " (\fIvoid\fP)"
> > > -Get the modulation frequency (Hz).
> > > -.TP
> > >   .BR LIRC_SET_SEND_CARRIER " (\fIint\fP)"
> > >   Set the modulation frequency.
> > >   The argument is the frequency (Hz).
> > >   .TP
> > > -.BR LIRC_GET_SEND_CARRIER " (\fIvoid\fP)"
> > > -Get the modulation frequency used when decoding (Hz).
> > > -.TP
> > > -.BR SET_SEND_DUTY_CYCLE " (\fIint\fP)"
> > > +.BR LIRC_SET_SEND_DUTY_CYCLE " (\fIint\fP)"
> > >   Set the carrier duty cycle.
> > >   .I val
> > >   is a number in the range [0,100] which
> > > @@ -284,36 +276,6 @@ By default this should be turned off.
> > >   .BR LIRC_GET_REC_RESOLUTION " (\fIvoid\fP)"
> > >   Return the driver resolution (microseconds).
> > >   .TP
> > > -.BR LIRC_GET_MIN_FILTER_PULSE " (\fIvoid\fP)", " " \
> > > -LIRC_GET_MAX_FILTER_PULSE " (\fIvoid\fP)"
> > > -Some devices are able to filter out spikes in the incoming signal
> > > -using given filter rules.
> > > -These ioctls return the hardware capabilities that describe the bounds
> > > -of the possible filters.
> > > -Filter settings depend on the IR protocols that are expected.
> > > -.BR lircd (8)
> > > -derives the settings from all protocols definitions found in its
> > > -.BR lircd.conf (5)
> > > -config file.
> > > -.TP
> > > -.BR LIRC_GET_MIN_FILTER_SPACE " (\fIvoid\fP)", " " \
> > > -LIRC_GET_MAX_FILTER_SPACE " (\fIvoid\fP)"
> > > -See
> > > -.BR LIRC_GET_MIN_FILTER_PULSE .
> > > -.TP
> > > -.BR LIRC_SET_REC_FILTER " (\fIint\fP)"
> > > -Pulses/spaces shorter than this (microseconds) are filtered out by
> > > -hardware.
> > > -.TP
> > > -.BR LIRC_SET_REC_FILTER_PULSE " (\fIint\fP)", " " \
> > > -LIRC_SET_REC_FILTER_SPACE " (\fIint\fP)"
> > > -Pulses/spaces shorter than this (microseconds) are filtered out by
> > > -hardware.
> > > -If filters cannot be set independently for pulse/space, the
> > > -corresponding ioctls must return an error and
> > > -.BR LIRC_SET_REC_FILTER
> > > -should be used instead.
> > > -.TP
> > >   .BR LIRC_SET_TRANSMITTER_MASK
> > >   Enable the set of transmitters specified in
> > >   .IR val ,
> > > @@ -343,32 +305,6 @@ carrier reports.
> > >   In that case, it will be disabled as soon as you disable carrier reports.
> > >   Trying to disable a wide band receiver while carrier reports are active
> > >   will do nothing.
> > > -.TP
> > > -.BR LIRC_SETUP_START " (\fIvoid\fP), " LIRC_SETUP_END " (\fIvoid\fP)"
> > > -Setting of several driver parameters can be optimized by bracketing
> > > -the actual ioctl calls
> > > -.BR LIRC_SETUP_START
> > > -and
> > > -.BR LIRC_SETUP_END .
> > > -When a driver receives a
> > > -.BR LIRC_SETUP_START
> > > -ioctl, it can choose to not commit further setting changes to the
> > > -hardware until a
> > > -.BR LIRC_SETUP_END
> > > -is received.
> > > -But this is open to the driver implementation and every driver
> > > -must also handle parameter changes which are not encapsulated by
> > > -.BR LIRC_SETUP_START
> > > -and
> > > -.BR LIRC_SETUP_END .
> > > -Drivers can also choose to ignore these ioctls.
> > > -.TP
> > > -.BR LIRC_NOTIFY_DECODE " (\fIvoid\fP)"
> > > -This ioctl is called by
> > > -.BR lircd (8)
> > > -whenever a successful decoding of an incoming IR signal is possible.
> > > -This can be used by supporting hardware to give visual user
> > > -feedback, for example by flashing an LED.
> > >   .\"
> > >   .SH FEATURES
> > >   .PP
> > > @@ -378,14 +314,6 @@ The
> > >   ioctl returns a bit mask describing features of the driver.
> > >   The following bits may be returned in the mask:
> > >   .TP
> > > -.BR LIRC_CAN_REC_RAW
> > > -The driver is capable of receiving using
> > > -.BR LIRC_MODE_RAW .
> > > -.TP
> > > -.BR LIRC_CAN_REC_PULSE
> > > -The driver is capable of receiving using
> > > -.BR LIRC_MODE_PULSE .
> > > -.TP
> > >   .BR LIRC_CAN_REC_MODE2
> > >   The driver is capable of receiving using
> > >   .BR LIRC_MODE_MODE2 .
> > > @@ -426,10 +354,6 @@ The driver supports
> > >   The driver supports
> > >   .BR LIRC_SET_REC_TIMEOUT .
> > >   .TP
> > > -.BR LIRC_CAN_SET_REC_FILTER
> > > -The driver supports
> > > -.BR LIRC_SET_REC_FILTER .
> > > -.TP
> > >   .BR LIRC_CAN_MEASURE_CARRIER
> > >   The driver supports measuring of the modulation frequency using
> > >   .BR LIRC_SET_MEASURE_CARRIER_MODE .
> > > @@ -438,22 +362,10 @@ The driver supports measuring of the modulation frequency using
> > >   The driver supports learning mode using
> > >   .BR LIRC_SET_WIDEBAND_RECEIVER .
> > >   .TP
> > > -.BR LIRC_CAN_NOTIFY_DECODE
> > > -The driver supports
> > > -.BR LIRC_NOTIFY_DECODE .
> > > -.TP
> > > -.BR LIRC_CAN_SEND_RAW
> > > -The driver supports sending using
> > > -.BR LIRC_MODE_RAW .
> > > -.TP
> > >   .BR LIRC_CAN_SEND_PULSE
> > >   The driver supports sending using
> > >   .BR LIRC_MODE_PULSE .
> > >   .TP
> > > -.BR LIRC_CAN_SEND_MODE2
> > > -The driver supports sending using
> > > -.BR LIRC_MODE_MODE2 .
> > > -.TP
> > >   .BR LIRC_CAN_SEND_LIRCCODE
> > >   The driver supports sending.
> > >   (This is uncommon, since
> > >   



Thanks,
Mauro



[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