Commit 1dd48e48 authored by Laurent Pinchart's avatar Laurent Pinchart

v4l: vsp1: Fill display list headers without holding dlm spinlock

The display list headers are filled using information from the display
list only. Lower the display list manager spinlock contention by filling
the headers without holding the lock.
Signed-off-by: default avatarLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: default avatarKieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Acked-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent e6742e10
...@@ -483,8 +483,6 @@ void vsp1_dl_list_commit(struct vsp1_dl_list *dl) ...@@ -483,8 +483,6 @@ void vsp1_dl_list_commit(struct vsp1_dl_list *dl)
unsigned long flags; unsigned long flags;
bool update; bool update;
spin_lock_irqsave(&dlm->lock, flags);
if (dl->dlm->mode == VSP1_DL_MODE_HEADER) { if (dl->dlm->mode == VSP1_DL_MODE_HEADER) {
struct vsp1_dl_list *dl_child; struct vsp1_dl_list *dl_child;
...@@ -501,7 +499,11 @@ void vsp1_dl_list_commit(struct vsp1_dl_list *dl) ...@@ -501,7 +499,11 @@ void vsp1_dl_list_commit(struct vsp1_dl_list *dl)
vsp1_dl_list_fill_header(dl_child, last); vsp1_dl_list_fill_header(dl_child, last);
} }
}
spin_lock_irqsave(&dlm->lock, flags);
if (dl->dlm->mode == VSP1_DL_MODE_HEADER) {
/* /*
* Commit the head display list to hardware. Chained headers * Commit the head display list to hardware. Chained headers
* will auto-start. * will auto-start.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment