Skip to content

Commit 85f78ce

Browse files
andy-shevvinodkoul
authored andcommitted
Revert "dmaengine: dmatest: timeout value of -1 should specify infinite wait"
This reverts commit ed04b7c. While it gives a good description what happens, the approach seems too confusing. Let's fix it in the following patch. Cc: Gary Hook <[email protected]> Signed-off-by: Andy Shevchenko <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Vinod Koul <[email protected]>
1 parent 955b176 commit 85f78ce

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

drivers/dma/dmatest.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ MODULE_PARM_DESC(pq_sources,
6262
static int timeout = 3000;
6363
module_param(timeout, uint, S_IRUGO | S_IWUSR);
6464
MODULE_PARM_DESC(timeout, "Transfer Timeout in msec (default: 3000), "
65-
"Pass 0xFFFFFFFF (4294967295) for maximum timeout");
65+
"Pass -1 for infinite timeout");
6666

6767
static bool noverify;
6868
module_param(noverify, bool, S_IRUGO | S_IWUSR);
@@ -98,7 +98,7 @@ MODULE_PARM_DESC(transfer_size, "Optional custom transfer size in bytes (default
9898
* @iterations: iterations before stopping test
9999
* @xor_sources: number of xor source buffers
100100
* @pq_sources: number of p+q source buffers
101-
* @timeout: transfer timeout in msec, 0 - 0xFFFFFFFF (4294967295)
101+
* @timeout: transfer timeout in msec, -1 for infinite timeout
102102
*/
103103
struct dmatest_params {
104104
unsigned int buf_size;
@@ -109,7 +109,7 @@ struct dmatest_params {
109109
unsigned int iterations;
110110
unsigned int xor_sources;
111111
unsigned int pq_sources;
112-
unsigned int timeout;
112+
int timeout;
113113
bool noverify;
114114
bool norandom;
115115
int alignment;

0 commit comments

Comments
 (0)