diff options
| author | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-05-19 09:01:24 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-05-19 09:01:24 -0300 |
| commit | c087fe3ec71bad9bcba731c328b0eb6f306c0de6 (patch) | |
| tree | a7ad2a5b02fa907348e110fe854c4d372e12d979 /drivers/media/pci/ttpci | |
| parent | 4c9ef4f150589478ac0b26bc7db1216c0af207fb (diff) | |
| download | linux-c087fe3ec71bad9bcba731c328b0eb6f306c0de6.tar.gz linux-c087fe3ec71bad9bcba731c328b0eb6f306c0de6.tar.xz | |
[media] av7110: avoid switch fall through
On two switches, this driver have unannotated switch
fall through.
in the first case, it falls through a return. On the second
one, it prints undesired log messages on fall through.
Solve that by copying the commands that it should be
running. Gcc will very likely optimize it anyway, so this
sholdn't be causing any harm, and shuts up gcc warnings.
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media/pci/ttpci')
| -rw-r--r-- | drivers/media/pci/ttpci/av7110.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/media/pci/ttpci/av7110.c b/drivers/media/pci/ttpci/av7110.c index df9395c87178..f2905bd80366 100644 --- a/drivers/media/pci/ttpci/av7110.c +++ b/drivers/media/pci/ttpci/av7110.c @@ -336,6 +336,7 @@ static int DvbDmxFilterCallback(u8 *buffer1, size_t buffer1_len, av7110_p2t_write(buffer1, buffer1_len, dvbdmxfilter->feed->pid, &av7110->p2t_filter[dvbdmxfilter->index]); + return 0; default: return 0; } @@ -451,8 +452,12 @@ static void debiirq(unsigned long cookie) case DATA_CI_PUT: dprintk(4, "debi DATA_CI_PUT\n"); + xfer = TX_BUFF; + break; case DATA_MPEG_PLAY: dprintk(4, "debi DATA_MPEG_PLAY\n"); + xfer = TX_BUFF; + break; case DATA_BMP_LOAD: dprintk(4, "debi DATA_BMP_LOAD\n"); xfer = TX_BUFF; |