From: Daniel W. S. Almeida <dwlsalme...@gmail.com>

[ Upstream commit 0a933a7f73d6c545dcbecb4f7a92d272aef4417b ]

The PMT write function was refactored and this broke the CRC computation.

Fix it.

Fixes: db9569f67e2e ("media: vidtv: cleanup PMT write table function")
Signed-off-by: Daniel W. S. Almeida <dwlsalme...@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+hua...@kernel.org>
Signed-off-by: Sasha Levin <sas...@kernel.org>
---
 drivers/media/test-drivers/vidtv/vidtv_psi.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/media/test-drivers/vidtv/vidtv_psi.c 
b/drivers/media/test-drivers/vidtv/vidtv_psi.c
index 4511a2a98405d..1724bb485e670 100644
--- a/drivers/media/test-drivers/vidtv/vidtv_psi.c
+++ b/drivers/media/test-drivers/vidtv/vidtv_psi.c
@@ -1164,6 +1164,8 @@ u32 vidtv_psi_pmt_write_into(struct 
vidtv_psi_pmt_write_args *args)
        struct vidtv_psi_desc *table_descriptor   = args->pmt->descriptor;
        struct vidtv_psi_table_pmt_stream *stream = args->pmt->stream;
        struct vidtv_psi_desc *stream_descriptor;
+       u32 crc = INITIAL_CRC;
+       u32 nbytes = 0;
        struct header_write_args h_args = {
                .dest_buf           = args->buf,
                .dest_offset        = args->offset,
@@ -1181,6 +1183,7 @@ u32 vidtv_psi_pmt_write_into(struct 
vidtv_psi_pmt_write_args *args)
                .new_psi_section    = false,
                .is_crc             = false,
                .dest_buf_sz        = args->buf_sz,
+               .crc                = &crc,
        };
        struct desc_write_args d_args   = {
                .dest_buf           = args->buf,
@@ -1193,8 +1196,6 @@ u32 vidtv_psi_pmt_write_into(struct 
vidtv_psi_pmt_write_args *args)
                .pid                = args->pid,
                .dest_buf_sz        = args->buf_sz,
        };
-       u32 crc = INITIAL_CRC;
-       u32 nbytes = 0;
 
        vidtv_psi_pmt_table_update_sec_len(args->pmt);
 
-- 
2.27.0



Reply via email to