From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f49.google.com (mail-ed1-f49.google.com [209.85.208.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BA1D5356770 for ; Mon, 1 Jun 2026 11:29:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780313348; cv=none; b=EnEgqSjEBJ53mTvbQaDmZAANKZjOUysWkoHkOccErGeEEa7bOJppWOIAgw6x2eZ7d30Y8HeEgnSJc5UMihiO+P4kAEgLdL9rCu5/0dDPsptzk6FIJo/SmAgvz8vm/B9t7tXtZcbn35GUdEbM1hsjl1ZbDQlY90R1Y2gWaDRLOUQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780313348; c=relaxed/simple; bh=0f6NuXc5QBHbcrP089lHyilyIEpGfEMRuODqvbMR3tU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=TrTwxQoKRizowvWMaORlgQhycm31r5sc37cLsYXl7KH59Kb3TqPlBTuo8swWbRT6cDEBrwGlqwBfgXQo5+hPQTvjoPaYpB9/J/uM2Q6ya/QcwPzTVPrqBUE9XqEygUefSzHGPfdSOfhlEkO23BteFPWbsRD6MZrcPIcm+3RwyNM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=JgtDP71q; arc=none smtp.client-ip=209.85.208.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="JgtDP71q" Received: by mail-ed1-f49.google.com with SMTP id 4fb4d7f45d1cf-6886be3d39bso15460781a12.3 for ; Mon, 01 Jun 2026 04:29:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1780313345; x=1780918145; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=dnas+wiD4/81j9F8DaAEG6ad+WURnj/jBxPPwOi/tWA=; b=JgtDP71qA36shY1Ay4fD2mNLhC+6l6A4Mdqy2+0Wq65nXNexwyscDaTuKbYG21OxiE g7uP0NiDBO9LfxIeGWzKx9fYTizGQAC1QvDlHfojx1uOlp8OjfTksYlJn/YHJQqbYJh6 H6A/JQOBT2hm0Q8nqLxkgQQ0FLuaO295YCSutMbTlxIX8MUQXuvmTKnosM089DH2Xau2 a55Uavy4IND1Xua2GUW/TpmwctYJDO9j0u9Kyt0ZPBbcZXQGRljeO36dPTJr7kI/8BWa +PbWIhrnR/zPOaarhuIJ1SFMwhK3w4935DjBLVjUN7tJbEpmFZeUvMITs81apdiaR3tj JifQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1780313345; x=1780918145; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=dnas+wiD4/81j9F8DaAEG6ad+WURnj/jBxPPwOi/tWA=; b=MdDFdhFnmcLM3oKjiEv+oyZfyuk1dzoVGpUlDCXiqW0X1lb0Va/QDLOM9UESW74P7H caYhlcy0BV0It/rZHaF3x5LkLRzc+qQZsbN+YUwpECOixvdvdXiLJuy2jImT8GvKXhnU GKvj73WL7DPlZyskM4j9gJ2IroSX9ZenUju+iFgXqHdRTcVHb5FoHydShZc04G3t7O7b ju/zyF5IUwFE8siQU2Lsc+aOAz0uip6h0PmwgObo+YuyNyBQnGv5nwvQzPknGjZhH+jF WEOr4KszyYRgGVUZ5shf+NHVi/sRwmDGZzeil7kQFlDSWz1G0jhxssi0WiX98WTXEaVk ZgnQ== X-Forwarded-Encrypted: i=1; AFNElJ/EbOD9MUM07YiN7GBh/hr3rHXT+KkLSVAWbW/C4b+crZki7I2PweCSCrw78NSkh/Ufxxs9XfsWAbUXCoY=@vger.kernel.org X-Gm-Message-State: AOJu0Yy0+X5Fj8Qp9Qx2TxQoXF/j0cpEWuUFwLn44j9WKjcvPnhTvoXX u6B+Bb7b4Roik5tRptfWhrqew7pqkcp4NEmSlqtAElpiGuSF5VzIdATb X-Gm-Gg: Acq92OG3nh9qUXX9Z8LT2b9BTrV+rFY9skXiugmXaW1Ky/HDBZTJ4PFLGtvdlnxRkWU uht6VvmoF8yk5p+8U1YZT1Zp1k8vs8n5K+uwvkZlbf3ECO3agCmeBb2GGAcNnyXYTHQuWp34j3z wzVHqtUlEDd2j6V4uDKXMztIhuFtwoquTAywXGXBzPola1iQvxzfD1EhX+vTviKzWkwaflACRn1 uzKAXk6WOZgeJqaxLdA3kF5l/HHAdmAVQnKD2k6tAZKNcYMTctOYWzMW06sOoEuK1OehvVcb3D5 G59K3bl0WFJgGjht9NhqSstJQ8nh8vEOpZKig8DmrJohjAGpvKcVF2hzFEBqnnO5izYK2+lj/Zl GwrJkDh/UBubyoJzkEaNBL+V3kYg1Q7kHVZ5b12VwiZZ2n7x+UE88hCHSp2+uc03Aufsumm94Hu HjkJDJ2UDpCjJDC2WCGI6mIa0wQbftUPoz8GFG+kVrR3gv X-Received: by 2002:a05:6402:2112:b0:68b:583:fb37 with SMTP id 4fb4d7f45d1cf-68c8ae13bc3mr5363368a12.10.1780313345032; Mon, 01 Jun 2026 04:29:05 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-68ceb7a7dc1sm1882521a12.9.2026.06.01.04.29.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Jun 2026 04:29:04 -0700 (PDT) Date: Mon, 1 Jun 2026 14:29:00 +0300 From: Dan Carpenter To: Eugene Mavick Cc: gregkh@linuxfoundation.org, khomenkov@mailbox.org, tomasz.unger@yahoo.pl, arthur.stupa@gmail.com, jannik@jrehkemper.de, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] staging: rtl8723bs: remove unnecessary parentheses Message-ID: References: <20260530031621.443015-1-mavick4022@gmail.com> <20260531061022.573089-1-mavick4022@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260531061022.573089-1-mavick4022@gmail.com> On Sun, May 31, 2026 at 02:10:22PM +0800, Eugene Mavick wrote: > Remove unnecessary parentheses to clear checkpatch.pl warnings > > Example of fixed warnings: > CHECK: Unnecessary parentheses around dvobj->cam_cache[id] > CHECK: Unnecessary parentheses around 'val != 0xfe' > > Checkpatch warnings regarding line length above 100 columns on modified > lines were also fixed > > Signed-off-by: Eugene Mavick > --- > V1->V2: > Fixed following checkpatch warnings on modified lines: > CHECK: Alignment should match open parenthesis(line 1114, 1249) > CHECK: Logical continuations should be on the previous line(line 1251) > > v1: https://lore.kernel.org/all/20260530031621.443015-1-mavick4022@gmail.com/ > > .../staging/rtl8723bs/core/rtw_wlan_util.c | 93 ++++++++++--------- > 1 file changed, 49 insertions(+), 44 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c > index 1d37c2d5b10d..84e9772a39c2 100644 > --- a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c > +++ b/drivers/staging/rtl8723bs/core/rtw_wlan_util.c > @@ -118,7 +118,7 @@ int is_basicrate(struct adapter *padapter, unsigned char rate) > for (i = 0; i < NumRates; i++) { > val = pmlmeext->basicrate[i]; > > - if ((val != 0xff) && (val != 0xfe)) > + if (val != 0xff && val != 0xfe) Just leave this one as-is. (staging rules). We deliberately silenced this checkpatch warning so it's weird for checkpatch to complain. Same for the other conditions. > if (rate == ratetbl_val_2wifirate(val)) > return true; > } > @@ -351,7 +351,7 @@ int is_client_associated_to_ap(struct adapter *padapter) > return _FAIL; > > pmlmeext = &padapter->mlmeextpriv; > - pmlmeinfo = &(pmlmeext->mlmext_info); > + pmlmeinfo = &pmlmeext->mlmext_info; These changes are fine though. Can you resend without the condtion changes? regards, dan carpenter