From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752667AbdFMJXA (ORCPT ); Tue, 13 Jun 2017 05:23:00 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:54726 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752441AbdFMJWy (ORCPT ); Tue, 13 Jun 2017 05:22:54 -0400 Date: Tue, 13 Jun 2017 11:22:46 +0200 From: Greg KH To: Aviya Erenfeld Cc: goudapatilk@gmail.com, insafonov@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/3] staging: rtl8188eu: Remove redundant parenthesis Message-ID: <20170613092246.GC5677@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 13, 2017 at 08:50:38AM +0300, Aviya Erenfeld wrote: > Remove redundant parenthesis > > Signed-off-by: Aviya Erenfeld > --- > drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 22 +++++++++++----------- > 1 file changed, 11 insertions(+), 11 deletions(-) > > diff --git a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c b/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c > index 2ecfb11..c6a7df5 100644 > --- a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c > +++ b/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c > @@ -91,7 +91,7 @@ u32 _rtw_init_sta_priv(struct sta_priv *pstapriv) > for (i = 0; i < NUM_STA; i++) { > _rtw_init_stainfo(psta); > > - INIT_LIST_HEAD(&(pstapriv->sta_hash[i])); > + INIT_LIST_HEAD(&pstapriv->sta_hash[i]); What complaines about these? Does checkpatch? Or something else? It makes sense why they were added, don't you think so? It makes the code easier to understand in some places. thanks, greg k-h