linux-toradex.git/drivers/media/rc/streamzap.c, branch v4.4.156

[media] ir: Fix IR_MAX_DURATION enforcement

2015-06-05T14:26:42+00:00

Don't assume that IR_MAX_DURATION is a bitmask. It isn't.

Signed-off-by: Mauro Carvalho Chehab

[media] rc-core: use USB API functions rather than constants

2014-09-22T20:27:01+00:00

This patch introduces the use of !usb_endpoint_dir_in(epd) and
!usb_endpoint_xfer_int(epd).

The Coccinelle semantic patch that makes these changes is as follows:

- ((epd->bEndpointAddress & \(USB_ENDPOINT_DIR_MASK\|0x80\)) !=
-  \(USB_DIR_IN\|0x80\))
+ !usb_endpoint_dir_in(epd)

@@ struct usb_endpoint_descriptor *epd; @@

- ((epd->bmAttributes & \(USB_ENDPOINT_XFERTYPE_MASK\|3\)) !=
- \(USB_ENDPOINT_XFER_INT\|3\))
+ !usb_endpoint_xfer_int(epd)

Signed-off-by: Himangi Saraogi 
Acked-by: Julia Lawall 
Signed-off-by: Mauro Carvalho Chehab

[media] rc-core: merge rc5 and streamzap decoders

2014-07-25T22:10:44+00:00

Now that the protocol is part of the scancode, it is pretty easy to merge
the rc5 and streamzap decoders. An additional advantage is that the decoder
is now stricter as it waits for the trailing silence before determining that
a command is a valid rc5/streamzap command (which avoids collisions that I've
seen with e.g. Sony protocols).

Signed-off-by: David Härdeman 
Signed-off-by: Mauro Carvalho Chehab

[media] rc-core: remove protocol arrays

2014-07-25T22:10:43+00:00

The basic API of rc-core used to be:

	dev = rc_allocate_device();
	dev->x = a;
	dev->y = b;
	dev->z = c;
	rc_register_device();

which is a pretty common pattern in the kernel, after the introduction of
protocol arrays the API looks something like:

	dev = rc_allocate_device();
	dev->x = a;
	rc_set_allowed_protocols(dev, RC_BIT_X);
	dev->z = c;
	rc_register_device();

There's no real need for the protocols to be an array, so change it
back to be consistent (and in preparation for the following patches).

[m.chehab@samsung.com: added missing changes at some files]
Signed-off-by: David Härdeman 
Signed-off-by: Mauro Carvalho Chehab

USB: media: streamzap: remove CONFIG_USB_DEBUG

2014-05-29T18:22:47+00:00

It wasn't even being used for anything, so remove the test for the
config option (as the config option isn't present anymore), and remove
the unused module parameter "debug" so as to not confuse anyone.

Cc: Mauro Carvalho Chehab 
Cc: James Hogan 
Cc: "Antti Seppälä" 
Signed-off-by: Greg Kroah-Hartman

[media] rc: abstract access to allowed/enabled protocols

2014-03-11T16:24:39+00:00

The allowed and enabled protocol masks need to be expanded to be per
filter type in order to support wakeup filter protocol selection. To
ease that process abstract access to the rc_dev::allowed_protos and
rc_dev::enabled_protocols members with inline functions.

Signed-off-by: James Hogan 
Reviewed-by: Antti Seppälä 
Signed-off-by: Mauro Carvalho Chehab

Drivers: media: remove __dev* attributes.

2013-01-03T23:57:02+00:00

CONFIG_HOTPLUG is going away as an option.  As a result, the __dev*
markings need to be removed.

This change removes the use of __devinit, __devexit_p, __devinitdata,
__devinitconst, and __devexit from these drivers.

Based on patches originally written by Bill Pemberton, but redone by me
in order to handle some of the coding style issues better, by hand.

Cc: Bill Pemberton 
Cc: Mauro Carvalho Chehab 
Signed-off-by: Greg Kroah-Hartman

[media] rc-core: add separate defines for protocol bitmaps and numbers

2012-10-27T13:49:51+00:00

The RC_TYPE_* defines are currently used both where a single protocol is
expected and where a bitmap of protocols is expected.

Functions like rc_keydown() and functions which add/remove entries to the
keytable want a single protocol. Future userspace APIs would also
benefit from numeric protocols (rather than bitmap ones). Keytables are
smaller if they can use a small(ish) integer rather than a bitmap.

Other functions or struct members (e.g. allowed_protos,
enabled_protocols, etc) accept multiple protocols and need a bitmap.

Using different types reduces the risk of programmer error. Using a
protocol enum whereever possible also makes for a more future-proof
user-space API as we don't need to worry about a sufficient number of
bits being available (e.g. in structs used for ioctl() calls).

The use of both a number and a corresponding bit is dalso one in e.g.
the input subsystem as well (see all the references to set/clear bit when
changing keytables for example).

This patch separate the different usages in preparation for
upcoming patches.

Where a single protocol is expected, enum rc_type is used; where one or more
protocol(s) are expected, something like u64 is used.

The patch has been rewritten so that the format of the sysfs "protocols"
file is no longer altered (at the loss of some detail). The file itself
should probably be deprecated in the future though.

Signed-off-by: David Härdeman 
Cc: Andy Walls 
Cc: Maxim Levitsky 
Cc: Antti Palosaari 
Cc: Mike Isely 
Signed-off-by: Mauro Carvalho Chehab

module_param: make bool parameters really bool (drivers & misc)

2012-01-12T23:02:20+00:00

module_param(bool) used to counter-intuitively take an int.  In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.

It's time to remove the int/unsigned int option.  For this version
it'll simply give a warning, but it'll break next kernel version.

Acked-by: Mauro Carvalho Chehab 
Signed-off-by: Rusty Russell

USB: convert drivers/media/* to use module_usb_driver()

2011-11-18T17:46:12+00:00

This converts the drivers in drivers/media/* to use the
module_usb_driver() macro which makes the code smaller and a bit
simpler.

Added bonus is that it removes some unneeded kernel log messages about
drivers loading and/or unloading.

Cc: Mauro Carvalho Chehab 
Cc: Luca Risolia 
Cc: Jean-Francois Moine 
Cc: Frank Zago 
Cc: Olivier Lorin 
Cc: Erik Andren 
Cc: Hans de Goede 
Cc: Brian Johnson 
Cc: Leandro Costantino 
Cc: Antoine Jacquet 
Cc: Jarod Wilson 
Cc: Florian Mickler 
Cc: Antti Palosaari 
Cc: Michael Krufky 
Cc: "David Härdeman" 
Cc: Florent Audebert 
Cc: Sam Doshi 
Cc: Manu Abraham 
Cc: Olivier Grenie 
Cc: Patrick Boettcher 
Cc: "Igor M. Liplianin" 
Cc: Derek Kelly 
Cc: Malcolm Priestley 
Cc: Steven Toth 
Cc: "André Weidemann" 
Cc: Martin Wilks 
Cc: Tejun Heo 
Cc: Jose Alberto Reguero 
Cc: David Henningsson 
Cc: Paul Gortmaker 
Cc: Joe Perches 
Cc: Jesper Juhl 
Cc: Lucas De Marchi 
Cc: Hans Verkuil 
Cc: Alexey Khoroshilov 
Cc: Anssi Hannula 
Cc: Rafi Rubin 
Cc: Dan Carpenter 
Cc: Paul Bender 
Cc: Devin Heitmueller 
Cc: "Márcio A Alves" 
Cc: Julia Lawall 
Cc: Laurent Pinchart 
Cc: Chris Rankin 
Cc: Lee Jones 
Cc: Andy Walls 
Cc: Andrew Morton 
Cc: Mike Frysinger 
Cc: Dean Anderson 
Cc: Pete Eberlein 
Cc: Arvydas Sidorenko 
Cc: Andrea Anacleto 
Signed-off-by: Greg Kroah-Hartman