Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1433737

[PATCH 00/15] lirc_dev fixes and beautification

Show all headers | View raw

Hi,

because I wanted to add three ioctl commands in lirc, I ended up
with the patchset below.

This is a collection of fixes, added functionality, coding rework
and trivial coding style fixes.

The first patch is preparatory to the second, which allows the
user to create a lirc driver without receiver buffer, which is
obvious for transmitters. Besides, even though that buffer could
have been used also by transmitters, drivers might have the need
to handle it separately.

The rest of the patches is a series of coding style and code
rework, as I said, some of them are very trivial, but I sent them
anyway because I was on fire.

Patch 14 is a segfault fix, while the last patch adds the
possibility to send to ioctl the set frequency, get frequency and
set length command.

Thanks,
Andi

Andi Shyti (15):
  lirc_dev: place buffer allocation on separate function
  lirc_dev: allow bufferless driver registration
  lirc_dev: remove unnecessary debug prints
  lirc_dev: replace printk with pr_* or dev_*
  lirc_dev: simplify goto paths
  lirc_dev: do not use goto to create loops
  lirc_dev: simplify if statement in lirc_add_to_buf
  lirc_dev: remove double if ... else statement
  lirc_dev: merge three if statements in only one
  lirc_dev: remove CONFIG_COMPAT precompiler check
  lirc_dev: fix variable constant comparisons
  lirc_dev: fix error return value
  lirc_dev: extremely trivial comment style fix
  lirc_dev: fix potential segfault
  include: lirc: add set length and frequency ioctl options

 drivers/media/rc/lirc_dev.c | 297 +++++++++++++++++++++-----------------------
 include/media/lirc_dev.h    |  12 ++
 include/uapi/linux/lirc.h   |   4 +
 3 files changed, 156 insertions(+), 157 deletions(-)

-- 
2.8.1

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH 00/15] lirc_dev fixes and beautification Andi Shyti <andi.shyti@samsung.com> - 2016-06-29 15:30 +0200
  [PATCH 15/15] include: lirc: add set length and frequency ioctl options Andi Shyti <andi.shyti@samsung.com> - 2016-06-29 15:30 +0200
    Re: [PATCH 15/15] include: lirc: add set length and frequency ioctl  options Sean Young <sean@mess.org> - 2016-06-30 01:00 +0200
      Re: [PATCH 15/15] include: lirc: add set length and frequency ioctl  options Andi Shyti <andi@etezian.org> - 2016-06-30 01:40 +0200
  [PATCH 11/15] lirc_dev: fix variable constant comparisons Andi Shyti <andi.shyti@samsung.com> - 2016-06-29 15:30 +0200
  [PATCH 07/15] lirc_dev: simplify if statement in lirc_add_to_buf Andi Shyti <andi.shyti@samsung.com> - 2016-06-29 15:30 +0200
  [PATCH 09/15] lirc_dev: merge three if statements in only one Andi Shyti <andi.shyti@samsung.com> - 2016-06-29 15:30 +0200

csiph-web