On Fri, Nov 25, 2022 at 11:11:13AM +0530, Appana Durga Kedareswara rao wrote: > +static int xtmr_inject_remove(struct platform_device *pdev) > +{ > + debugfs_remove_recursive(dbgfs_root); > + dbgfs_root = NULL; Nit, no need to set this to NULL. Not an issue, but something you can clean up later... thanks, greg k-h