On Mon, Aug 17, 2015 at 11:48 PM, Jiri Pirko <[email protected]> wrote: > Tue, Aug 18, 2015 at 08:14:48AM CEST, [email protected] wrote: >>On Mon, Aug 17, 2015 at 10:55 PM, Jiri Pirko <[email protected]> wrote: >>> Tue, Aug 18, 2015 at 12:36:17AM CEST, [email protected] wrote: >>>>From: Scott Feldman <[email protected]> >>> >>> <snip> >>> >>>>+ rocker->dbg_dir = debugfs_create_dir(dbg_dir_name, rocker_dbg_root); >>>>+ if (!rocker->dbg_dir) >>> >>> You still check the retval here and >> >>It's OK. See include/linux/debugfs.h when CONFIG_DEBUG_FS=n. >>debugfs_create_dir() returns ERR_PTR(-ENODEV), which is !NULL. > > Does not make sense to check it for NULL then. It is never NULL...
It can be NULL when CONFIG_DEBUG_FS=y. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
