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.wall...@linaro.org>
Cc: Vinod Koul <vk...@kernel.org>
Cc: Dan Williams <dan.j.willi...@intel.com>
Cc: linux-arm-ker...@lists.infradead.org
Cc: dmaeng...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/dma/coh901318.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c
index 547786ac342b..e51d836afcc7 100644
--- a/drivers/dma/coh901318.c
+++ b/drivers/dma/coh901318.c
@@ -1378,10 +1378,8 @@ static int __init init_coh901318_debugfs(void)
 
        dma_dentry = debugfs_create_dir("dma", NULL);
 
-       (void) debugfs_create_file("status",
-                                  S_IFREG | S_IRUGO,
-                                  dma_dentry, NULL,
-                                  &coh901318_debugfs_status_operations);
+       debugfs_create_file("status", S_IFREG | S_IRUGO, dma_dentry, NULL,
+                           &coh901318_debugfs_status_operations);
        return 0;
 }
 
-- 
2.22.0

Reply via email to