From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj2-f5.google.com (mail-pj2-f5.google.com [74.125.227.133]) (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 5E5B5448380 for ; Tue, 25 Aug 2026 12:54:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.227.133 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787662463; cv=none; b=jynjdSZpDluasz0I7HYw0zCfciwboubdsSZwTtT3BN3G+pXQxGCDl5g7rMqUim1kHwpxfgUdOur+1qe+Y9SxNYFFSOEwB+tVejgXeevneZFsYJEv2wrIPMhyXQcvPe/Ax+b4czrJXY+ldNWIWHVzX9BUUvUi2eIqDBb9PajXfn0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1787662463; c=relaxed/simple; bh=JgE4s7TnCtsNJp/p+NQGbyLEXgV+DXhJYnOP5DiTIoI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=qnayAuq699SDLIPq/gGdjs0dETtYtNRQF7Ejzktz4r314zL8KAVCusPtcLrPSgkhSiZx4iFA77/POrv/NH54pda/7SZTwURaeEvoThalEshhiHFqtcngdkhWp+RUBQr85ooFUkgmKMHSfLyyv2SItMxeR6zZHZmNhATh9lva6z0= 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=my4Xt78z; arc=none smtp.client-ip=74.125.227.133 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="my4Xt78z" Received: by mail-pj2-f5.google.com with SMTP id 98e67ed59e1d1-39662b5cde7so112131a91.1 for ; Tue, 25 Aug 2026 05:54:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1787662462; x=1788267262; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=OuVTycujd5+HwneLyLqVtIfZIL5wSc+qs01S1uCZrMU=; b=my4Xt78zI6gl/qXY6NAgfNRilrxCsaOD9E3eG4iLGUrabcsHhFP5gWxNCEYs6LW9Ko IIf6eubiaUR9XEKJF0o40bxnXoXwzI859WBHZdm+vTNEtQrIlQlHi8jfWIJqeAO105in e6ZRWc4PSjxmHfIiBr1d5xlzmZ2vVk+ksX3r0CT+1m/bbFP+H2jvgC2+V92TFboZiKtr 3Ih1l3MyMaqbnLPUn1Q4uLTaMm+sMay6WjH0A+HbddwCyxn/ChBuSULZhj8lJWC+FThr oN++CfZPVDecfreOcyfw+EG56i/DPKofUkT2S/8CkIoZHcz4JAb5txj0s4BKzCav8UVN ukVA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1787662462; x=1788267262; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=OuVTycujd5+HwneLyLqVtIfZIL5wSc+qs01S1uCZrMU=; b=KrRCNVQtFIN/QheEXHLgsXtFdGN8I3CkbJermFdZwpbTQ2cSbJheaYplcENzUb4Xae XC4hYD0HZcv7Ox99WVGpPzV4MT+z0a0jdMAd7ccLkFwXWpc8d4wjNlFLW00eT/NLx9ef liLrPS0YsjRja6lZ15Uby1Vpdnw6QUUk8xLE/yL90xYxfd/0t5/zYQHAroadBDSFuS8f 5x6eIWO3TzR6So5WVtqSZl/Kx08OJSUec0xUl449hxonTyv4xrynE5iw/QiJyUODKXB2 51nIz6evnjRqjTpzYXx9qVuTpP0/OpenUEHQVgT4cx9tRuxyWXddPtg+QL8hc0Y6vJXO 9DiA== X-Forwarded-Encrypted: i=1; AHgh+RqeWUGKNK3HgriKOfeSG5g/JCTLZaKPz2VPLwXwr7CsLzkuAxR5dsoyYAlNAkXcpQMgj9SQbsG3VVLjtlA=@vger.kernel.org X-Gm-Message-State: AFuF++l7QTwD/U5CWqdBdyXQHWO6sIi0zbd2ShTeMMLqzDzR8Yw2WRWZ fX6Sgx5au7FWaWVNEJfQ5uypD3JpUnJJjLE4sAv3auhsLfE2npD1iZHrGCE62wTo X-Gm-Gg: AR+sD10vSWr2Cu2DIz/1M+4GLkWJDKuukk8HHweUi71uqmbm4SNIDbqyQYugahDpoR4 65BbuqY7ZtHkHfa/mV1vwiOXqA3BqXzzaXGkrPJIqTDYFxI/3VFhDv0zD2YYKuhoZRpvpj9sapq 7/SuX1nxbyu/xlzj6/eagIk9OyLxRMgv1D2076gpc792C1egI2lsBAwpeEX9hllA/yTpKDe1yBD d5ftowNjnWVyOx3tIs24Wo3pio6nNbZw3uMAPKIZSyUOp56/fLXUZPooqes8z4eOANU0kmVKkDq XsRrpM6KTQ0sd3U8/7wbxHw8Vde5tRg0PmR58QxjTOq9JvW1V9yDw3g+HwppGDPbx8ZYm0YxewO ctusw531cU7paoVpFvBUOksoVweIF0vB02iw98Vk9TGwVBA0dMwzpeuSJ95DPV4E/31fj8mF+nb YdIq3NMGwagPnWq931DXAEK9xsWM+JZt/qehUulIxgGMMaAB7sHdI1nHIlAKMoBF2YAR94vyO3D AgZduaolOTSd8kyaAWnJeS0E3g= X-Received: by 2002:a17:90a:d605:b0:38e:5964:97a8 with SMTP id 98e67ed59e1d1-395df5e8fefmr50038307a91.16.1787662461737; Tue, 25 Aug 2026 05:54:21 -0700 (PDT) Received: from localhost.localdomain ([98.148.34.5]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-327f92359c9sm37098329eec.27.2026.08.25.05.54.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Aug 2026 05:54:21 -0700 (PDT) From: Hungyu Lin To: Greg Kroah-Hartman Cc: Dan Carpenter , linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Hungyu Lin Subject: [PATCH 1/2] staging: rtl8723bs: make rtw_free_xmitbuf_ext() return void Date: Tue, 25 Aug 2026 12:54:05 +0000 Message-ID: <20260825125406.1710-2-dennylin0707@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260825125406.1710-1-dennylin0707@gmail.com> References: <20260825125406.1710-1-dennylin0707@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit The return value of rtw_free_xmitbuf_ext() is never checked. Change its return type to void and remove the redundant _SUCCESS and _FAIL return values. Also remove the unnecessary extern keyword from its declaration. Signed-off-by: Hungyu Lin --- drivers/staging/rtl8723bs/core/rtw_xmit.c | 6 ++---- drivers/staging/rtl8723bs/include/rtw_xmit.h | 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_xmit.c b/drivers/staging/rtl8723bs/core/rtw_xmit.c index 81d9e713fc12..a1a0d75d823e 100644 --- a/drivers/staging/rtl8723bs/core/rtw_xmit.c +++ b/drivers/staging/rtl8723bs/core/rtw_xmit.c @@ -1534,13 +1534,13 @@ struct xmit_buf *rtw_alloc_xmitbuf_ext(struct xmit_priv *pxmitpriv) return pxmitbuf; } -s32 rtw_free_xmitbuf_ext(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf) +void rtw_free_xmitbuf_ext(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf) { unsigned long irqL; struct __queue *pfree_queue = &pxmitpriv->free_xmit_extbuf_queue; if (!pxmitbuf) - return _FAIL; + return; spin_lock_irqsave(&pfree_queue->lock, irqL); @@ -1550,8 +1550,6 @@ s32 rtw_free_xmitbuf_ext(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf) pxmitpriv->free_xmit_extbuf_cnt++; spin_unlock_irqrestore(&pfree_queue->lock, irqL); - - return _SUCCESS; } struct xmit_buf *rtw_alloc_xmitbuf(struct xmit_priv *pxmitpriv) diff --git a/drivers/staging/rtl8723bs/include/rtw_xmit.h b/drivers/staging/rtl8723bs/include/rtw_xmit.h index 917e846e9ce2..0dad1349e52b 100644 --- a/drivers/staging/rtl8723bs/include/rtw_xmit.h +++ b/drivers/staging/rtl8723bs/include/rtw_xmit.h @@ -403,7 +403,7 @@ extern struct xmit_frame *__rtw_alloc_cmdxmitframe(struct xmit_priv *pxmitpriv, #define rtw_alloc_bcnxmitframe(p) __rtw_alloc_cmdxmitframe(p, CMDBUF_BEACON) extern struct xmit_buf *rtw_alloc_xmitbuf_ext(struct xmit_priv *pxmitpriv); -extern s32 rtw_free_xmitbuf_ext(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf); +void rtw_free_xmitbuf_ext(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf); extern struct xmit_buf *rtw_alloc_xmitbuf(struct xmit_priv *pxmitpriv); extern s32 rtw_free_xmitbuf(struct xmit_priv *pxmitpriv, struct xmit_buf *pxmitbuf); -- 2.43.0