linux-stable.git/drivers/media/dvb-core/dvbdev.c, branch linux-4.15.y

Simplify major/minor non-dynamic logic

2017-10-11T19:32:11+00:00

changeset 6bbf7a855d20 ("media: dvbdev: convert DVB device types into an enum")
added a new warning on gcc 6:

>> drivers/media/dvb-core/dvbdev.c:86:1: warning: control reaches end of non-void function [-Wreturn-type]

That's because gcc is not smart enough to see that all types are
present at the switch. Also, the current code is not too optimized.

So, replace it to a more optimized one, based on a static table.

Reported-by: kbuild test robot 
Fixes: 6bbf7a855d20 ("media: dvbdev: convert DVB device types into an enum")
Signed-off-by: Mauro Carvalho Chehab

media: dvbdev: convert DVB device types into an enum

2017-10-11T17:00:44+00:00

Enums can be documented via kernel-doc. So, convert the
DVB_DEVICE_* macros to an enum.

Signed-off-by: Mauro Carvalho Chehab

[media] media: Drop FSF's postal address from the source code files

2017-01-27T13:38:09+00:00

Drop the FSF's postal address from the source code files that typically
contain mostly the license text. Of the 628 removed instances, 578 are
outdated.

The patch has been created with the following command without manual edits:

git grep -l "675 Mass Ave\|59 Temple Place\|51 Franklin St" -- \
	drivers/media/ include/media|while read i; do i=$i perl -e '
open(F,"< $ENV{i}");
$a=join("", );
$a =~ s/[ \t]*\*\n.*You should.*\n.*along with.*\n.*(\n.*USA.*$)?\n//m
	&& $a =~ s/(^.*)Or, (point your browser to) /$1To obtain the license, $2\n$1/m;
close(F);
open(F, "> $ENV{i}");
print F $a;
close(F);'; done

Signed-off-by: Sakari Ailus

[media] dvbdev: split dvb_unregister_device()

2016-11-18T17:02:39+00:00

dvb_unregister_device() has a major problem: it combines unregistering
with memory disposal.  Sometimes, it is necessary to unregister a
device, but no memory can be freed yet, because a process still has a
(stale) file handle.  Therefore, we need to split
dvb_unregister_device().  This will allow sanitizing a few callers.

With my new design, dvb_unregister_device() appears misnamed, but to
reduce patch noise, I'm not renaming it just yet.

Signed-off-by: Max Kellermann 
Signed-off-by: Mauro Carvalho Chehab

[media] dvb-core: use pr_foo() instead of printk()

2016-10-21T10:41:19+00:00

The dvb-core directly calls printk() without using the modern
printk macros, or using the proper printk levels. Change it
to use pr_foo().

Signed-off-by: Mauro Carvalho Chehab

[media] media/dvb-core: forward media_create_pad_links() return value

2016-05-07T13:56:55+00:00

Instead of always return -ENOMEM, return the real error that should
come from media_create_pad_link().

Signed-off-by: Max Kellermann 
Signed-off-by: Mauro Carvalho Chehab

[media] dvb-core: fix return code checking for devices with CA

2016-03-03T17:33:33+00:00

The test for the return code was mistakenly inverted. This caused DVB
devices with CA module to fail on modprobe.

Tested with TechnoTrend CT2-4650 CI USB tuner.

Signed-off-by: Olli Salonen 
Signed-off-by: Mauro Carvalho Chehab

[media] dvbdev: the space is required after ','

2016-02-01T12:25:59+00:00

The space is missing after ',', and this will be introduce much
noise when checking new patch around them.

Signed-off-by: Xiubo Li 
Signed-off-by: Mauro Carvalho Chehab

[media] dvbdev: replace kcalloc with kzalloc

2016-02-01T12:25:16+00:00

Since the number of elements equals to 1, so just use kzalloc to
simplify the code and make it more readable.

Signed-off-by: Xiubo Li 
Signed-off-by: Mauro Carvalho Chehab

[media] dvbdev: remove useless parentheses after return

2016-02-01T12:24:44+00:00

The parentheses are not required after return, and just remove it.

Signed-off-by: Xiubo Li 
Signed-off-by: Mauro Carvalho Chehab