diff options
author | Serge Semin <[email protected]> | 2023-01-13 20:13:53 +0300 |
---|---|---|
committer | Lorenzo Pieralisi <[email protected]> | 2023-01-27 17:15:33 +0100 |
commit | 37d058aae75b965a1c597979a194c9dba79ac913 (patch) | |
tree | 5ab6f20055b87748f5456c3a0c4b9e104776267d /tools | |
parent | 7ca9f025a731b3391f9321a324d8eb7a5b008c5a (diff) |
dmaengine: dw-edma: Stop checking debugfs_create_*() return value
The debugfs_create_*() functions never return NULL, so checking their
return value for NULL is pointless. Secondly, the debugfs subsystem is
designed to be as simple as possible, so if one of the debugfs_create_*()
method in a hierarchy fails, the following methods should silently return
the passed erroneous parental dentry. Finally, the code should work no
matter whether anything debugfs-related fails.
To make code simpler and debugfs-independent, stop checking the
debugfs_create_*() return values.
If the debugfs file system is unavailable, skip the debugfs node
initialization altogether to preserve some memory space.
Link: https://lore.kernel.org/r/[email protected]
Tested-by: Manivannan Sadhasivam <[email protected]>
Signed-off-by: Serge Semin <[email protected]>
Signed-off-by: Lorenzo Pieralisi <[email protected]>
Signed-off-by: Bjorn Helgaas <[email protected]>
Reviewed-by: Manivannan Sadhasivam <[email protected]>
Acked-by: Vinod Koul <[email protected]>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions