On Tue, 30 Apr 2024 14:00:58 +0200, Wolfram Sang wrote: > There is a confusing pattern in the kernel to use a variable named 'timeout' to > store the result of wait_for_*() functions causing patterns like: > > timeout = wait_for_completion_timeout(...) > if (!timeout) return -ETIMEDOUT; > > with all kinds of permutations. Use 'time_left' as a variable to make the code > obvious and self explaining. > > [...] Applied, thanks! [1/3] slimbus: messaging: use 'time_left' variable with wait_for_completion_timeout() commit: 0eb9dda9d1db40acbabe923fe22c002e13890d39 [2/3] slimbus: qcom-ctrl: use 'time_left' variable with wait_for_completion_timeout() commit: 7d317b95d0334371481ec00ca31f5bf76bae8a82 [3/3] slimbus: qcom-ngd-ctrl: use 'time_left' variable with wait_for_completion_timeout() commit: 9f5fd5e2aebf06c37355cacc7f4b4410bc0ea233 Best regards, -- Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>