On Tue, 22 Jan 2019 at 16:23, Greg Kroah-Hartman <gre...@linuxfoundation.org> wrote: > > When calling debugfs functions, there is no need to ever check the > return value. The function can work or not, but the code logic should > never do something different based on this.
Doesn't this boils done to whether we want to care to check if memory allocation failed? Somewhere down the call chain from debugfs_create_dir(), we end up in alloc_inode() and it looks like that can fail, no? Kind regards Uffe > > Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> > Cc: Kevin Hilman <khil...@kernel.org> > Cc: Ulf Hansson <ulf.hans...@linaro.org> > Cc: Len Brown <len.br...@intel.com> > Cc: linux...@vger.kernel.org > Signed-off-by: Greg Kroah-Hartman <gre...@linuxfoundation.org> > --- > drivers/base/power/domain.c | 11 ++--------- > 1 file changed, 2 insertions(+), 9 deletions(-) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index 500de1dee967..45eafe8cf7dd 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -2948,18 +2948,11 @@ static int __init genpd_debug_init(void) > > genpd_debugfs_dir = debugfs_create_dir("pm_genpd", NULL); > > - if (!genpd_debugfs_dir) > - return -ENOMEM; > - > - d = debugfs_create_file("pm_genpd_summary", S_IRUGO, > - genpd_debugfs_dir, NULL, &summary_fops); > - if (!d) > - return -ENOMEM; > + debugfs_create_file("pm_genpd_summary", S_IRUGO, genpd_debugfs_dir, > + NULL, &summary_fops); > > list_for_each_entry(genpd, &gpd_list, gpd_list_node) { > d = debugfs_create_dir(genpd->name, genpd_debugfs_dir); > - if (!d) > - return -ENOMEM; > > debugfs_create_file("current_state", 0444, > d, genpd, &status_fops); > -- > 2.20.1 >