Re: [PATCH v2 07/27] thunderbolt: Allow passing NULL to tb_ctl_free()

From: Andy Shevchenko
Date: Mon May 29 2017 - 08:16:08 EST


On Sun, 2017-05-28 at 11:13 +0300, Mika Westerberg wrote:
> On Sat, May 27, 2017 at 06:41:28PM +0300, Andy Shevchenko wrote:
> > On Fri, May 26, 2017 at 7:09 PM, Mika Westerberg
> > <mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> > > Following the usual pattern used in many places, we allow passing
> > > NULL
> > > pointer to tb_ctl_free(). Then the user can call the function
> > > regardless
> > > if it has allocated control channel or not making the code bit
> > > simpler.
> > >
> >
> > +Suggested-by as you mentioned in cover letter?
>
> OK, I'll add it.
>
> BTW, should I do it for the other patch that was suggested by Andreas
> as
> well?

Sure


--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy