On Mon, Nov 24, 2014 at 10:45:03PM +0100, Robert Jarzmik wrote:
> 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)
sure, I'll do that. Do I get a Review-by ?
--
balbi
Attachment:
signature.asc
Description: Digital signature
