On 16.02.2021 09:53, Du Cheng wrote: > align * in block comments on each line > > changes v3: > - add SUBSYSTEM in subject line > - add explanation to past version of this patch > > changes v2: > - move closing of comment to the same line > > changes v1: > - align * in block comments > > Signed-off-by: Du Cheng <ducheng2@xxxxxxxxx> > --- > drivers/staging/qlge/qlge_main.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/staging/qlge/qlge_main.c b/drivers/staging/qlge/qlge_main.c > index 5516be3af898..2682a0e474bd 100644 > --- a/drivers/staging/qlge/qlge_main.c > +++ b/drivers/staging/qlge/qlge_main.c > @@ -3815,8 +3815,7 @@ static int qlge_adapter_down(struct qlge_adapter *qdev) > > qlge_tx_ring_clean(qdev); > > - /* Call netif_napi_del() from common point. > - */ > + /* Call netif_napi_del() from common point. */ > for (i = 0; i < qdev->rss_ring_count; i++) > netif_napi_del(&qdev->rx_ring[i].napi); > > Typically such trivial patches aren't much appreciated for staging drivers. In the case here I have doubts that the comment as such provides any benefit. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel