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

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

[Sean Young]

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

Signed-off-by: Sean Young <sean@mess.org>
---
 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
-- 
2.14.3

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

[Michael Kerrisk (man-opages)]

[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?

Cheers,

Michael

> Signed-off-by: Sean Young <sean@mess.org>
> ---
>   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
> 

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

[Sean Young]

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?

Mauro, as Alec is not responding, would you be able to sign this off?

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

Thanks,

Sean
> 
> Cheers,
> 
> Michael
> 
> > Signed-off-by: Sean Young <sean@mess.org>
> > ---
> >   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
> > 

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

[Sean Young]

Hi Alec,

On Sat, May 19, 2018 at 08:38:11AM +0200, Alec Leamas wrote:
> On 18/05/18 17:25, Sean Young wrote:
> > 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?
> 
> Yes, sorry, must have missed your message. These ioctl are used to
> exist, but are no more and should be removed.

Alec, you didn't reply to the list.

Can this be merged please?

Thanks,

Sean

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

[Mauro Carvalho Chehab]

Hi Michael/Alec,

Em Fri, 18 May 2018 16:25:29 +0100
Sean Young <sean@mess.org> 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@mess.org>
> > > ---
> > >   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

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

[Sean Young]

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@mess.org> 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.


Sean

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

[Mauro Carvalho Chehab]

Em Thu, 12 Jul 2018 14:21:18 +0100
Sean Young <sean@mess.org> escreveu:

> 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@mess.org> 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.

Ok, makes sense to me.

Reviewed-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

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



Thanks,
Mauro

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

[Sean Young]

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@mess.org> 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

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

[Alec Leamas]

Hi all,

On 29/10/18 18:30, Sean Young wrote:
> 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@mess.org> 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? 

Yes.

I have sent a reply on this (in the same spirit) "long time ago"

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


Cheers!
--a

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

[Michael Kerrisk (man-pages)]

Hello Sean,

On 4/23/18 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
>
> Signed-off-by: Sean Young <sean@mess.org>

Thanks for your persistence.

Mauro, Alec, thanks for your input.

Patch Applied.

Thanks!

Michael
> ---
>  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
> --
> 2.14.3
>


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/