diff options
author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-06-12 08:25:54 -0400 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2019-06-14 01:44:58 -0400 |
commit | c33394bd006c2efa742b204e740c098763972dc9 (patch) | |
tree | fff013566a29fabda7a96c5165b705b36630db60 | |
parent | 635d7302caca40841077804c194ae8e00339e2a0 (diff) |
dmaengine: coh901318: no need to cast away call to debugfs_create_file()
No need to check the return value of debugfs_create_file(), so no need
to provide a fake "cast away" of the return value either.
Cc: Linus Walleij <linus.walleij@linaro.org>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: linux-arm-kernel@lists.infradead.org
Cc: dmaengine@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
-rw-r--r-- | drivers/dma/coh901318.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c index b69d66e44052..6f5bf6ae08b1 100644 --- a/drivers/dma/coh901318.c +++ b/drivers/dma/coh901318.c | |||
@@ -1378,10 +1378,8 @@ static int __init init_coh901318_debugfs(void) | |||
1378 | 1378 | ||
1379 | dma_dentry = debugfs_create_dir("dma", NULL); | 1379 | dma_dentry = debugfs_create_dir("dma", NULL); |
1380 | 1380 | ||
1381 | (void) debugfs_create_file("status", | 1381 | debugfs_create_file("status", S_IFREG | S_IRUGO, dma_dentry, NULL, |
1382 | S_IFREG | S_IRUGO, | 1382 | &coh901318_debugfs_status_operations); |
1383 | dma_dentry, NULL, | ||
1384 | &coh901318_debugfs_status_operations); | ||
1385 | return 0; | 1383 | return 0; |
1386 | } | 1384 | } |
1387 | 1385 | ||