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: Dan Williams <dan.j.willi...@intel.com>
Cc: Vinod Koul <vk...@kernel.org>
Cc: dmaeng...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/dma/amba-pl08x.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c
index 464725dcad00..9adc7a2fa3d3 100644
--- a/drivers/dma/amba-pl08x.c
+++ b/drivers/dma/amba-pl08x.c
@@ -2508,9 +2508,8 @@ DEFINE_SHOW_ATTRIBUTE(pl08x_debugfs);
 static void init_pl08x_debugfs(struct pl08x_driver_data *pl08x)
 {
        /* Expose a simple debugfs interface to view all clocks */
-       (void) debugfs_create_file(dev_name(&pl08x->adev->dev),
-                       S_IFREG | S_IRUGO, NULL, pl08x,
-                       &pl08x_debugfs_fops);
+       debugfs_create_file(dev_name(&pl08x->adev->dev), S_IFREG | S_IRUGO,
+                           NULL, pl08x, &pl08x_debugfs_fops);
 }
 
 #else
-- 
2.22.0

Reply via email to