staging: most: dim2: remove unnecessary NULL check in service_done_flag()

Remove the !hdm_ch check. hdm_ch is derived from dev->hch + ch_idx
(pointer arithmetic on a struct member), so it can never be NULL.
Keep only the !hdm_ch->is_initialized check.

Reported-by: Dan Carpenter <error27@gmail.com>
Closes: https://lore.kernel.org/all/acuoL4DRi0pmsQY1@stanley.mountain
Signed-off-by: Gabriel Rondon <grondon@gmail.com>
Link: https://patch.msgid.link/20260331164443.47682-1-grondon@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/most/dim2/dim2.c b/drivers/staging/most/dim2/dim2.c
index 7953d4626752fbcce0c50b9bdf82f7bfabb70110..f5bd7d44f1656a78ddd03b0cff7675ccb0f7d7f2 100644 (file)
--- a/drivers/staging/most/dim2/dim2.c
+++ b/drivers/staging/most/dim2/dim2.c
@@ -271,7 +271,7 @@ static void service_done_flag(struct dim2_hdm *dev, int ch_idx)
        unsigned long flags;
        u8 *data;
 
-       if (!hdm_ch || !hdm_ch->is_initialized)
+       if (!hdm_ch->is_initialized)
                return;
 
        spin_lock_irqsave(&dim_lock, flags);