diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2020-04-24 18:11:45 +0200 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2020-04-27 18:17:52 +0200 |
| commit | 35c5fc028511c743db5313f0ef8cac35141706e9 (patch) | |
| tree | 6d9a00e092ca35265efc2981c3fa31f325d3a58c /drivers/dma/dmatest.c | |
| parent | Revert "dmaengine: dmatest: timeout value of -1 should specify infinite wait" (diff) | |
| download | linux-35c5fc028511c743db5313f0ef8cac35141706e9.tar.xz linux-35c5fc028511c743db5313f0ef8cac35141706e9.zip | |
dmaengine: dmatest: Allow negative timeout value to specify infinite wait
The dmatest module parameter 'timeout' is documented as accepting a -1 to mean
"infinite timeout". However, an infinite timeout is not advised, nor possible
since the module parameter is an unsigned int, which won't accept a negative
value. Change the parameter type to be signed integer.
Cc: Gary Hook <Gary.Hook@amd.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20200424161147.16895-4-andriy.shevchenko@linux.intel.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/dma/dmatest.c')
| -rw-r--r-- | drivers/dma/dmatest.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/dma/dmatest.c b/drivers/dma/dmatest.c index d02089850ec1..f86903661428 100644 --- a/drivers/dma/dmatest.c +++ b/drivers/dma/dmatest.c @@ -60,7 +60,7 @@ MODULE_PARM_DESC(pq_sources, "Number of p+q source buffers (default: 3)"); static int timeout = 3000; -module_param(timeout, uint, S_IRUGO | S_IWUSR); +module_param(timeout, int, S_IRUGO | S_IWUSR); MODULE_PARM_DESC(timeout, "Transfer Timeout in msec (default: 3000), " "Pass -1 for infinite timeout"); |