linux-toradex.git/drivers/media/firewire, branch v4.1.15

[media] firewire: Deletion of an unnecessary check before the function call "dvb_unregister_device"

2014-11-25T15:30:31+00:00

The dvb_unregister_device() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring 
Reviewed-by: Stefan Richter 
Signed-off-by: Mauro Carvalho Chehab

[media] fix a warning on avr32 arch

2014-11-03T17:53:15+00:00

X-Patchwork-Delegate: m.chehab@samsung.com
on avr32 arch, those warnings happen:
	drivers/media/firewire/firedtv-fw.c: In function 'node_update':
	drivers/media/firewire/firedtv-fw.c:329: warning: comparison is always true due to limited range of data type

In this particular case, the signal is desired, as the isochannel
var can be initalized with -1 inside the driver.

So, change the type to s8, to avoid issues on archs where char
is unsigned.

Signed-off-by: Mauro Carvalho Chehab 
Reviewed-by: Stefan Richter 
Signed-off-by: Mauro Carvalho Chehab

[media] firewire: firedtv-avc: fix more potential buffer overflow

2014-09-23T19:13:39+00:00

"program_info_length" is user controlled and can go up to 4095.  The
operand[] array has 509 bytes so we need to add a limit here to prevent
buffer overflows.

The " - 4" in the limit check is because we have 4 bytes more data to
add after the memcpy().

[mchehab@osg.samsung.com: as I merged the version 1 of the patch, I needed
 to rebase to apply just the differences between v1 and v2]
Signed-off-by: Dan Carpenter 

Signed-off-by: Mauro Carvalho Chehab

[media] firewire: firedtv-avc: potential buffer overflow

2014-09-23T19:13:37+00:00

"program_info_length" is user controlled and can go up to 4095.  The
operand[] array has 509 bytes so we need to add a limit here to prevent
buffer overflows.

Signed-off-by: Dan Carpenter 
Reviewed-by: Stefan Richter 
Signed-off-by: Mauro Carvalho Chehab

firewire: introduce fw_driver.probe and .remove methods

2013-06-09T16:15:00+00:00

FireWire upper layer drivers are converted from generic
    struct driver.probe() and .remove()
to bus-specific
    struct fw_driver.probe() and .remove().

The new .probe() adds a const struct ieee1394_device_id *id argument,
indicating the entry in the driver's device identifiers table which
matched the fw_unit to be probed.  This new argument is used by the
snd-firewire-speakers driver to look up device-specific parameters and
methods.  There is at least one other FireWire audio driver currently in
development in which this will be useful too.

The new .remove() drops the unused error return code.

Although all in-tree drivers are being converted to the new methods,
support for the old methods is left in place in this commit.  This
allows public developer trees to merge this commit and then move to the
new fw_driver methods.

Signed-off-by: Stefan Richter 
Acked-by: Clemens Ladisch  (for sound/firewire/)
Cc: Peter Hurley  (for drivers/staging/fwserial/)

[media] demux.h: Remove duplicated enum

2013-04-08T09:53:15+00:00

"enum dmx_ts_pes" and "typedef enum dmx_pes_type_t" are just the
same enum declared twice, since Kernel (2.6.12). There's no reason
to duplicate it there, and sparse complains about that:
	drivers/media/dvb-core/dmxdev.c:600:55: warning: mixing different enum types
So, remove the internal define, keeping just the external one.
Internally, use only "enum dmx_ts_pes", as it is too late to drop
dmx_pes_type_t from the userspace API.

Signed-off-by: Mauro Carvalho Chehab

[media] remove include/linux/dvb/dmx.h

2012-10-19T10:29:17+00:00

The only reason for this header is to make sure that include
linux/time.h were added before uapi/*/dmx.h. Just push down the
time.h header on the few places where this is used, and drop
this new header.

Signed-off-by: Mauro Carvalho Chehab

[media] Fix some Makefile rules

2012-08-16T22:55:03+00:00

On a few places, := were using instead of +=, causing drivers to
not compile.

While here, standardize the usage of += on all cases where multiple
lines are needed, and for obj-y/obj-m targets, and := when just one
line is needed, on -obj rules.

Reported-by: Hans Verkuil 
Identified-by: Antti Polosaari 
Tested-by: Antti Palosaari 
Signed-off-by: Mauro Carvalho Chehab

[media] firewire: move it one level up

2012-08-14T02:20:36+00:00

Move firewire to one level up, as the dvb subdirectory will be
removed.

Signed-off-by: Mauro Carvalho Chehab