media: vidtv: remove unneeded variable make code cleaner
authorMinghao Chi <chi.minghao@zte.com.cn>
Fri, 10 Dec 2021 02:47:21 +0000 (03:47 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 14 Dec 2021 15:19:04 +0000 (16:19 +0100)
return value form directly instead of
taking this in another redundant variable.

Link: https://lore.kernel.org/linux-media/20211210024721.425145-1-chi.minghao@zte.com.cn
Reported-by: Zeal Robot <zealci@zte.com.cm>
Signed-off-by: Minghao Chi <chi.minghao@zte.com.cn>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/test-drivers/vidtv/vidtv_psi.c

index c11ac8d..a587538 100644 (file)
@@ -94,34 +94,28 @@ static void vidtv_psi_update_version_num(struct vidtv_psi_table_header *h)
 static u16 vidtv_psi_get_sec_len(struct vidtv_psi_table_header *h)
 {
        u16 mask;
-       u16 ret;
 
        mask = GENMASK(11, 0);
 
-       ret = be16_to_cpu(h->bitfield) & mask;
-       return ret;
+       return be16_to_cpu(h->bitfield) & mask;
 }
 
 u16 vidtv_psi_get_pat_program_pid(struct vidtv_psi_table_pat_program *p)
 {
        u16 mask;
-       u16 ret;
 
        mask = GENMASK(12, 0);
 
-       ret = be16_to_cpu(p->bitfield) & mask;
-       return ret;
+       return be16_to_cpu(p->bitfield) & mask;
 }
 
 u16 vidtv_psi_pmt_stream_get_elem_pid(struct vidtv_psi_table_pmt_stream *s)
 {
        u16 mask;
-       u16 ret;
 
        mask = GENMASK(12, 0);
 
-       ret = be16_to_cpu(s->bitfield) & mask;
-       return ret;
+       return be16_to_cpu(s->bitfield) & mask;
 }
 
 static void vidtv_psi_set_desc_loop_len(__be16 *bitfield, u16 new_len,