mailbox: mpfs: ditch a useless busy check
authorConor Dooley <conor.dooley@microchip.com>
Tue, 7 Mar 2023 20:22:53 +0000 (20:22 +0000)
committerConor Dooley <conor.dooley@microchip.com>
Mon, 3 Apr 2023 18:25:52 +0000 (19:25 +0100)
mpfs_mbox_rx_data() already checks if the system controller is busy
before attempting to do anything, so drop the second check before
reading any data.

No functional change intended.

Acked-by: Jassi Brar <jaswinder.singh@linaro.org>
Tested-by: Valentina Fernandez <valentina.fernandezalanis@microchip.com>
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
drivers/mailbox/mailbox-mpfs.c

index e0e825bdbad902990e77660e9d95dd7629a3cac5..0d176aba34624736f4a6d7f359aab66cc1a22ca7 100644 (file)
@@ -170,12 +170,10 @@ static void mpfs_mbox_rx_data(struct mbox_chan *chan)
        if (response->resp_status)
                return;
 
-       if (!mpfs_mbox_busy(mbox)) {
-               for (i = 0; i < num_words; i++) {
-                       response->resp_msg[i] =
-                               readl_relaxed(mbox->mbox_base
-                                             + mbox->resp_offset + i * 0x4);
-               }
+       for (i = 0; i < num_words; i++) {
+               response->resp_msg[i] =
+                       readl_relaxed(mbox->mbox_base
+                                     + mbox->resp_offset + i * 0x4);
        }
 
        mbox_chan_received_data(chan, response);