dmaengine: coh901318: no need to cast away call to debugfs_create_file()
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 12 Jun 2019 12:25:54 +0000 (14:25 +0200)
committerVinod Koul <vkoul@kernel.org>
Fri, 14 Jun 2019 05:44:58 +0000 (11:14 +0530)
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>
drivers/dma/coh901318.c

index b69d66e440529ed2d436b849142dd6caf4dee398..6f5bf6ae08b1b18d045f1b3f9e8566757facde8f 100644 (file)
@@ -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;
 }