On Tue, 2016-03-08 at 16:02 +0800, Lisheng011 wrote:
> 
> 在 2016/3/8 15:18, Andy Shevchenko 写道:
> > 
> > On Tue, 2016-03-08 at 11:52 +0800, Lisheng wrote:
> > > 
> > > From: Qianqian Xie <xieqianq...@huawei.com>
> > > 
> > > The variable head in hns_nic_tx_fini_pro has read a value,
> > > but the value is not used. The patch will solve it.


> -     /* for hardware bug fixed */
> > This is not good. Explanation is needed why you removed this
> > comment.

> This bug-fixed comment is for the function pointer 
> ring_data->fini_process,that is called in function
> hns_nic_common_poll.


> The comment should not be here, so remove it.

Please, add above paragraph to the commit message.

-- 
Andy Shevchenko <andriy.shevche...@linux.intel.com>
Intel Finland Oy

Reply via email to