From: Prashant Malani Date: Tue, 28 Sep 2021 10:19:30 +0000 (-0700) Subject: platform/x86: intel_scu_ipc: Fix busy loop expiry time X-Git-Tag: v6.1-rc5~2748^2~53 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f32c34d6cfbb3d1f1c26f223cb549cca9767cfbd;p=platform%2Fkernel%2Flinux-starfive.git platform/x86: intel_scu_ipc: Fix busy loop expiry time The macro IPC_TIMEOUT is already in jiffies (it is also used like that elsewhere in the file when calling wait_for_completion_timeout()). Don’t convert it using helper functions for the purposes of calculating the busy loop expiry time. Fixes: e7b7ab3847c9 (“platform/x86: intel_scu_ipc: Sleeping is fine when polling”) Signed-off-by: Prashant Malani Cc: Benson Leung Reviewed-by: Mika Westerberg Link: https://lore.kernel.org/r/20210928101932.2543937-2-pmalani@chromium.org Signed-off-by: Hans de Goede --- diff --git a/drivers/platform/x86/intel_scu_ipc.c b/drivers/platform/x86/intel_scu_ipc.c index bfa0cc2..cfb249d 100644 --- a/drivers/platform/x86/intel_scu_ipc.c +++ b/drivers/platform/x86/intel_scu_ipc.c @@ -232,7 +232,7 @@ static inline u32 ipc_data_readl(struct intel_scu_ipc_dev *scu, u32 offset) /* Wait till scu status is busy */ static inline int busy_loop(struct intel_scu_ipc_dev *scu) { - unsigned long end = jiffies + msecs_to_jiffies(IPC_TIMEOUT); + unsigned long end = jiffies + IPC_TIMEOUT; do { u32 status;