Skip to content

Commit 683ec6d

Browse files
committed
ipc: ipc4: helper: drop redundant locking in ipc4_search_for_drv()
Drop the IRQ disable/enable in ipc4_search_for_drv(). The driver list is only modified at FW boot and when a new driver is registered at runtime via SOF_IPC4_GLB_LOAD_LIBRARY IPC. ipc4_search_for_drv() is only used when processing IPC messages. As IPC processing is serialized, it is not possible for the driver list to be modified concurrently with a call to ipc4_search_for_drv(). Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
1 parent e3246b8 commit 683ec6d

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

src/ipc/ipc4/helper.c

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1131,12 +1131,9 @@ __cold static const struct comp_driver *ipc4_search_for_drv(const void *uuid)
11311131
struct list_item *clist;
11321132
const struct comp_driver *drv = NULL;
11331133
struct comp_driver_info *info;
1134-
uint32_t flags;
11351134

11361135
assert_can_be_cold();
11371136

1138-
irq_local_disable(flags);
1139-
11401137
/* search driver list with UUID */
11411138
list_for_item(clist, &drivers->list) {
11421139
info = container_of(clist, struct comp_driver_info,
@@ -1151,7 +1148,6 @@ __cold static const struct comp_driver *ipc4_search_for_drv(const void *uuid)
11511148
}
11521149
}
11531150

1154-
irq_local_enable(flags);
11551151
return drv;
11561152
}
11571153

0 commit comments

Comments
 (0)