diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-08-26 08:18:16 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-08-27 14:46:42 +0200 |
commit | 5af478341fa3b2c35a7062ad80581751bf388977 (patch) | |
tree | 20b17fca1433b4d25f1c562d869ce024e05ab047 /drivers/platform | |
parent | media: staging: omap4iss: make v4l2_file_operations const (diff) | |
download | linux-5af478341fa3b2c35a7062ad80581751bf388977.tar.xz linux-5af478341fa3b2c35a7062ad80581751bf388977.zip |
media: dib9000: delete some unused broken code
The dib9000_remove_slave_frontend() function isn't used.
I was reviewing it because my static checker claims it writes one
element beyond the end of the array. That's a false positive. What it
actually does is, if there are two or more front ends, then it prints a
debug message to say that it removed the first one, stored in
state->fe[1], and then it "removes" (scare quotes on purpose) the second
one, stored in state->fe[2]. Deleting a front end from the middle is
not really supported and breaks code like dib9000_release() which
assumes the first NULL front end marks the end of the list.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/platform')
0 files changed, 0 insertions, 0 deletions