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