On Mon, Nov 24, 2014 at 10:45:03PM +0100, Robert Jarzmik wrote:
> Felipe Balbi <ba...@ti.com> 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

Reply via email to