Felipe Balbi <balbi@xxxxxx> writes: > @@ -1136,8 +1136,7 @@ static const struct file_operations debug_fops = { > } while (0) > #define remove_debug_files(dev) \ > do { \ > - if (dev->debugfs_udc) \ > - debugfs_remove(dev->debugfs_udc); \ > + debugfs_remove(dev->debugfs_udc); \ > } while (0) Why this simpler form ? #define remove_debug_files(dev) debugfs_remove(dev->debugfs_udc) Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html