firmware: ti_sci: Fix TISCI mailbox receive timeout handling
authorAndreas Dannenberg <dannenberg@ti.com>
Wed, 24 Apr 2019 19:20:08 +0000 (14:20 -0500)
committerTom Rini <trini@konsulko.com>
Fri, 3 May 2019 11:23:17 +0000 (07:23 -0400)
An earlier commit converted the TISCI receive timeouts to be specified
in ms rather than us however it failed to take this change into account
when passing the actual timeout to be used when invoking the mailbox
receive API. This leads to the actual timeout to be 1,000 times shorter
than expected and as a result certain TISCI operations would fail.

Fix the issue by converting the timeout declared in ms to us on the fly
as expected by the respective API.

Fixes: fd6b40b1ba20 ("firmware: ti_sci: Add support for NAVSS resource management")
Signed-off-by: Andreas Dannenberg <dannenberg@ti.com>
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
drivers/firmware/ti_sci.c

index 1196ce07123d7189c78c189df86602509fa5f2a1..66d7dd48aceef5e7f0a2f5da69dc714f72e5d3ce 100644 (file)
@@ -158,7 +158,7 @@ static inline int ti_sci_get_response(struct ti_sci_info *info,
        int ret;
 
        /* Receive the response */
-       ret = mbox_recv(chan, msg, info->desc->max_rx_timeout_ms);
+       ret = mbox_recv(chan, msg, info->desc->max_rx_timeout_ms * 1000);
        if (ret) {
                dev_err(info->dev, "%s: Message receive failed. ret = %d\n",
                        __func__, ret);