hexagon: time: Remove redundant null check for resource
authorHardevsinh Palaniya <hardevsinh.palaniya@siliconsignals.io>
Mon, 11 Nov 2024 14:24:10 +0000 (19:54 +0530)
committerBrian Cain <brian.cain@oss.qualcomm.com>
Fri, 31 Jan 2025 02:45:32 +0000 (18:45 -0800)
Null check for 'resource' before assignment is unnecessary because the
variable 'resource' is initialized to NULL at the beginning of the function.

Signed-off-by: Hardevsinh Palaniya <hardevsinh.palaniya@siliconsignals.io>
Link: https://lore.kernel.org/r/20241111142458.67854-1-hardevsinh.palaniya@siliconsignals.io
Signed-off-by: Brian Cain <bcain@quicinc.com>
Signed-off-by: Brian Cain <brian.cain@oss.qualcomm.com>
arch/hexagon/kernel/time.c

index f0f207e2a6947711745ba6677b037fa835498c9b..6f851e1cd4ee09468738b8cfd36fee84e123f588 100644 (file)
@@ -170,8 +170,7 @@ static void __init time_init_deferred(void)
 
        ce_dev->cpumask = cpu_all_mask;
 
-       if (!resource)
-               resource = rtos_timer_device.resource;
+       resource = rtos_timer_device.resource;
 
        /*  ioremap here means this has to run later, after paging init  */
        rtos_timer = ioremap(resource->start, resource_size(resource));