Skip to content

Commit da82f95

Browse files
committed
mailbox: mpfs: ditch a useless busy check
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>
1 parent b5984a9 commit da82f95

1 file changed

Lines changed: 4 additions & 6 deletions

File tree

drivers/mailbox/mailbox-mpfs.c

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -170,12 +170,10 @@ static void mpfs_mbox_rx_data(struct mbox_chan *chan)
170170
if (response->resp_status)
171171
return;
172172

173-
if (!mpfs_mbox_busy(mbox)) {
174-
for (i = 0; i < num_words; i++) {
175-
response->resp_msg[i] =
176-
readl_relaxed(mbox->mbox_base
177-
+ mbox->resp_offset + i * 0x4);
178-
}
173+
for (i = 0; i < num_words; i++) {
174+
response->resp_msg[i] =
175+
readl_relaxed(mbox->mbox_base
176+
+ mbox->resp_offset + i * 0x4);
179177
}
180178

181179
mbox_chan_received_data(chan, response);

0 commit comments

Comments
 (0)