From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f181.google.com (mail-lj1-f181.google.com [209.85.208.181]) (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 0DBD13FB07C for ; Thu, 7 May 2026 13:40:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778161230; cv=none; b=ue1cpfxkvGf8E2u9ygar4JIwEOL2nyze5278y32iSzldukMKJZZHANF0LqB1sPkK2dT3WBRiVg2x8JCCxrNhdMOiCTfYI6U1RQ5x5cs0Re1H4JanQ7fzAZGxQi/p5FZFDLox3YWpBqLhpL0UqBMYtiZH53kjNgoM2XnTDFDKVH0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778161230; c=relaxed/simple; bh=+5Y/fGdCgomxhu/5J4RBAqx4J6pEF+x+uObVxTIeefQ=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=WlofWaswaIDReGjMeTtbFOgLSrh5X3YErs7C8gGafxTGZagkRESYFSLWyET4+u9LLQsjFNyknWq9iTt0oUoIE+a91SvuZem8Co5OzY1bx17jYfkS2acP9a5qpGtoSAapDppRRjb8k9FOttyzye96ajAymm6NnddXo8gJAiHPDkA= 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=p8wzf/O5; arc=none smtp.client-ip=209.85.208.181 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="p8wzf/O5" Received: by mail-lj1-f181.google.com with SMTP id 38308e7fff4ca-38e7d983f79so7243661fa.0 for ; Thu, 07 May 2026 06:40:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778161227; x=1778766027; 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=ZYWpxniz0+OvEuP0nr4/rQG/zCJ5t00/Xi1x/KmCsIw=; b=p8wzf/O5hr2s9XmXVL26ymtUyJenzrDQ/L5WL+ESGZK+x90OdFaqq89Q/3MccvWnFo VszB64Lzz3gbjRQfMOr+V+SWJ+aSe4u1XmM3jxQtW06oWIgmBBwuo+SnSKJ1NndMgwAn IKTmKveYIq+Rq6CvwWsbL2dboUSS3kCI+0Sa1cb7HreXDOlEqVc5A5thwBM1Qjcv62RX GNoNBkgiLQMXI1sbKFHwydpB7ZmGt9QdEprL7YCGHG/Z5RfZEDmN+P48m/4YJ+bgDd7j 0UvoOITXM3vCn0n1tR35i3KFca7GeCRlG3dsBjH5AwYr512XYJGD/ZQcq6NHs2Rpfh8E vs4A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778161227; x=1778766027; 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=ZYWpxniz0+OvEuP0nr4/rQG/zCJ5t00/Xi1x/KmCsIw=; b=DoJqwHuH9iY1vfWpLwnyKwWhv2TjkFd33rLY+teKcVo2vuab/KGDng9EGj8PIKxOyl JCm+S0T424WkNGQabwK/wkm1DxDPhuyQUeAi7SXo1ACDwij8fpHsBw7aq4mXhQt2shk+ vkSRwNTK7qmhvojWDTr7I7JikEue8UJw5IPNe+sZP4iXhTvjk3+oYLob5MFHqGUFgDTX UDj6ISprF/476gmGrEus5MOgYttcfXR3x/uFnh66aGlDhvAGAy715Z1KkydYlFtGnxJl MppK91odEC2RleVCy8OS3/y4Dj7WIAUmtkYFXSXEPKFBaTMcZ+Qdlb3aQtEWHHI71Y+0 f2fg== X-Forwarded-Encrypted: i=1; AFNElJ8osxTxcruJZGJ4fyfQnOBeMvagiaIyXj1mYWd3pS89YDIlj0sHFB+/0QvuZ0GgM2wwWppMenUnbsyohfY=@vger.kernel.org X-Gm-Message-State: AOJu0YwpEftgh9UO12l9du+IcIUGkWmkil5wFXNrN+ps6HN/SKqluHc7 l5v+dDZXbyZW1RfDXTpv4WdD6juhEtoy7Pze2ocuS+ZqtAic9pCC6TNs4EOZE7XC X-Gm-Gg: AeBDiev7124DKWVKjoayu5VxIjUvIlww91GE2EJvklWtULJM93U4U6qXj+3c9ERmUna K47YWbNEyUgShZxI2lr4bNdBffzRqC8RUGFA8Sznnz9bOFfYgHHKZTwLQgpLAFKtzA+a+wnPf6/ SMyBEBfEm2O11z49rx2gu6uD0MyBD5QyVxhl1wFu3E+EhMdua1JX/XO8zb9UG3xu77iAM+l+9YK /F4NWoyDtuYPFaexyUFz7AEZW3kLF4pstuqxG9Hvop44+5cfbVT/s1k0FjoqHnHOMVCjhwUgYzf eztJ5uSJSus0PejjTYScbmVB/zuYfwJPXHALXKrMrY9pxd76ZOV0M1eRuhpIrR7YtM6KZ1uJJ7s jmXjtTztGTAQBCkqTDNsfIMP81iv35iCL2EnGKyiGkS4fDA2H0eJYy2zd1IU9raorOqr8vpk5St 3qbq0of+mEdGN8APtPoNNy3Y0uzY76RJHQfrY= X-Received: by 2002:ac2:4c48:0:b0:5a8:6e82:6840 with SMTP id 2adb3069b0e04-5a887cde953mr3927186e87.20.1778161226832; Thu, 07 May 2026 06:40:26 -0700 (PDT) Received: from archlinux ([185.140.161.214]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5a864c6f16esm4890146e87.15.2026.05.07.06.40.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 07 May 2026 06:40:26 -0700 (PDT) Date: Thu, 7 May 2026 16:40:23 +0300 From: Nikolay Kulikov To: Mohammed Sohail Cc: gregkh@linuxfoundation.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] staging: rtl8723bs: simplify NULL pointer checks Message-ID: References: <20260506190201.75571-1-mohd.sohail.biz@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: <20260506190201.75571-1-mohd.sohail.biz@gmail.com> On Thu, May 07, 2026 at 12:32:01AM +0530, Mohammed Sohail wrote: > Replace ptr == NULL with !ptr in rtw_recv.h and ioctl_cfg80211.c. > > Improves readability and follows kernel coding style. > No functional changes. > > Signed-off-by: Mohammed Sohail > --- > v2: > - Use real name in Signed-off-by as requested by reviewer > --- > drivers/staging/rtl8723bs/include/rtw_recv.h | 8 ++++---- > drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 2 +- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/include/rtw_recv.h b/drivers/staging/rtl8723bs/include/rtw_recv.h > index 8e45871f07f0..9c32d480a20a 100644 > --- a/drivers/staging/rtl8723bs/include/rtw_recv.h > +++ b/drivers/staging/rtl8723bs/include/rtw_recv.h > @@ -349,7 +349,7 @@ s32 rtw_recv_entry(union recv_frame *precv_frame); > static inline u8 *get_rxmem(union recv_frame *precvframe) > { > /* always return rx_head... */ > - if (precvframe == NULL) > + if (!precvframe) > return NULL; > > return precvframe->u.hdr.rx_head; > @@ -362,7 +362,7 @@ static inline u8 *recvframe_pull(union recv_frame *precvframe, signed int sz) > /* used for extract sz bytes from rx_data, update rx_data and return the updated rx_data to the caller */ > > > - if (precvframe == NULL) > + if (!precvframe) > return NULL; > > > @@ -387,7 +387,7 @@ static inline u8 *recvframe_put(union recv_frame *precvframe, signed int sz) > /* after putting, rx_tail must be still larger than rx_end. */ > unsigned char *prev_rx_tail; > > - if (precvframe == NULL) > + if (!precvframe) > return NULL; > > prev_rx_tail = precvframe->u.hdr.rx_tail; > @@ -414,7 +414,7 @@ static inline u8 *recvframe_pull_tail(union recv_frame *precvframe, signed int s > /* used for extract sz bytes from rx_end, update rx_end and return the updated rx_end to the caller */ > /* after pulling, rx_end must be still larger than rx_data. */ > > - if (precvframe == NULL) > + if (!precvframe) > return NULL; > > precvframe->u.hdr.rx_tail -= sz; > diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > index fd3bae31b0ed..8ae3adecafd9 100644 > --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > @@ -2429,7 +2429,7 @@ static int cfg80211_rtw_dump_station(struct wiphy *wiphy, > spin_lock_bh(&pstapriv->asoc_list_lock); > psta = rtw_sta_info_get_by_idx(idx, pstapriv); > spin_unlock_bh(&pstapriv->asoc_list_lock); > - if (psta == NULL) { > + if (!psta) { > ret = -ENOENT; > goto exit; > } This does not apply to staging-next. Thanks, Nikolay