From: Stanislaw Gruszka <stanislaw.grus...@linux.intel.com>

ivpu_ipc_dispatch is always called with irqs disabled. Add lockdep
assertion and remove unneeded _irqsave/_irqrestore.

Signed-off-by: Stanislaw Gruszka <stanislaw.grus...@linux.intel.com>
Signed-off-by: Jacek Lawrynowicz <jacek.lawrynow...@linux.intel.com>
---
 drivers/accel/ivpu/ivpu_ipc.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/accel/ivpu/ivpu_ipc.c b/drivers/accel/ivpu/ivpu_ipc.c
index 31ae0e71a8a3..781c7e40505a 100644
--- a/drivers/accel/ivpu/ivpu_ipc.c
+++ b/drivers/accel/ivpu/ivpu_ipc.c
@@ -367,9 +367,9 @@ ivpu_ipc_dispatch(struct ivpu_device *vdev, struct 
ivpu_ipc_consumer *cons,
 {
        struct ivpu_ipc_info *ipc = vdev->ipc;
        struct ivpu_ipc_rx_msg *rx_msg;
-       unsigned long flags;
 
        lockdep_assert_held(&ipc->cons_list_lock);
+       lockdep_assert_irqs_disabled();
 
        rx_msg = kzalloc(sizeof(*rx_msg), GFP_ATOMIC);
        if (!rx_msg) {
@@ -382,9 +382,9 @@ ivpu_ipc_dispatch(struct ivpu_device *vdev, struct 
ivpu_ipc_consumer *cons,
        rx_msg->ipc_hdr = ipc_hdr;
        rx_msg->jsm_msg = jsm_msg;
 
-       spin_lock_irqsave(&cons->rx_lock, flags);
+       spin_lock(&cons->rx_lock);
        list_add_tail(&rx_msg->link, &cons->rx_msg_list);
-       spin_unlock_irqrestore(&cons->rx_lock, flags);
+       spin_unlock(&cons->rx_lock);
 
        wake_up(&cons->rx_msg_wq);
 }
-- 
2.42.0

Reply via email to