From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lj1-f178.google.com (mail-lj1-f178.google.com [209.85.208.178]) (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 8915F3368B7 for ; Wed, 15 Jul 2026 12:49:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.178 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784119762; cv=none; b=fWra6jTJSGVXgy9A4obKRJ/fGO/gs8oNQmFbvc0hH3fzcmm0h2t3S0J84tKzOX/6FaHc3b3ull5BOQ0JrfztYaoBpJtJCparBKcJSfSBUxkpaTdUIr4KZln/xEN9Lu4jA+liehe1x/CE2WM41N217sCjuLIXNfCsq/lHCb1vYyA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784119762; c=relaxed/simple; bh=33Zd67ehO5q/W2BvD/Nk7gcgGWYx65taCDXDpvuvQYI=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=CrURmY07tUKjBK3yf4QOWQ5KP7qsq4NLZCybTynKvmJz7HM/4orYxM9V5u1OI77ufP0jXggEp24ejkuKOQFotl4xcp1Co6Lg6IL47gYaBg7//W6CXxaFUnEfHnntY/ShbIdNUOj7QqxJy62+maLNa422SEWY0Y6Rs/WtftzIqYM= 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=KQ6c8hJ3; arc=none smtp.client-ip=209.85.208.178 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="KQ6c8hJ3" Received: by mail-lj1-f178.google.com with SMTP id 38308e7fff4ca-39c86945164so44909621fa.1 for ; Wed, 15 Jul 2026 05:49:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1784119760; x=1784724560; 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=19DglOy9oVvwhSIg2qx/B5ubXzP/pIL5UVJQSdZ01P8=; b=KQ6c8hJ3i5stR/hpq8z2kIE9mFPZJHH+CSuuw5hqD0uvq0ALNx4HoxUTvKWJmyib9W uqOX1r9YxJgPquqZYYFGZvXpIdsQ8Of8nQtztN72KU5JNrx3lsQ+M9vxbS8evMhAfM2O 23VBhtxinTY+pWoufmBvxcKN5fiDCtKqTNrCuPWYCLhxHRKTK6Nv5Xbkl67IWVSNk11Y lopYIe8JSgMSkpTXUynJbdmTfE2DAsjWcbyvABCUe5WKxBvvHcIGFCieW+aMfV1ANHlh 0sVwf0NaDhyIr6KvdSTbjKk8uxEeYl4Eaf2hbPLVPnqRWPSV4XfVfQ3wPLZ3cKkdPpMT g6mw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784119760; x=1784724560; 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=19DglOy9oVvwhSIg2qx/B5ubXzP/pIL5UVJQSdZ01P8=; b=ItJ26nHIzGVMm7WJHg+J+vCa3GjxrSrPK9gxPSy6e5nCwHJ4QyE7lBkEw9QVYO0oAt ZDDrf1JViGn7tz69Aed+9rm9Kkfq3VWAOjaFV5Ee1KlAhO+CU0BhAHMAFlRIbU5gvgeI 3ddoovcmNHLXWzG5QFo4D6iW6fPGg7VLwgfE4vz3rehOECQVn6ZRodYYovSo6dh1xjgL YHAJ+okbZ1AorjmkIkRiDW1lGqTUrw4CUp7f3KhUct33rJWa1qp511z6dcU3XeYBAKan NkXi2pSPEGlKTCO7mRlOg0SRt0bJcRuYR9NSYq6qqMm9/VtqskBQY3i2+8Xo5rp+XSAa RXNw== X-Forwarded-Encrypted: i=1; AHgh+RqTpbL2ffP/n6xwSwayKTrz6gRRPn2qCmPpbLGM5lV6Xh3HBZjmn7zgAvbn4Rnwz/vEBXtJiVgb58I2lgU=@vger.kernel.org X-Gm-Message-State: AOJu0YxoO+NVkblr4lXKJ6zjfmchAHCKDiiLYWVewTMjMJby9vRoWKNU IgEhHDtRfo9jYhrmOo5VEDMQddWg3PiRqO8ORuvWiuQFnjhIXlMTMolJ X-Gm-Gg: AfdE7ckgTjz3eteL7kD7ByKwJ0gXfwgl9Oq7TeNGPqPNk+NOpv9ZwFcyD9h5T0dtY5/ XQLI7zGhNDOnXyaspeQhbG45aT43NH2890IrszZHoPckix5XIs95MOxZQ2vqlsNIcr9yjXzDyys 3LMf5QQp8p+u+p2I5I46mClYzJz6DfLp3Be8keNywJ77Jpzo1oy9tCZDqXSXWi+GgnLrUb+yQ/8 8+V1pmXPPtkQZXRMBaPujjpXeXdvpBAreGyyBtORHEpCENEZQvWs3BO6eZOAzrVq++aNVWzw0JD FpvxRFmXoOyFdeqYhB2w/9KcFSxrLz2Umpqd7HRs2PkdT2W8U/SFo0vQzgeo9FwB4F/+sVJmVcb KLEH5nKzN0WtUDx4KJAZ+PwfGZ5SZWtZIvPTKlm/Z0y+F4GZ5ximhV1U0LLU/QPKEwrzQonpfsr vX7Pvgxvv/bcfEB+sEbQgTER/uJ053VMxjyOF0iYhx/5X7fveOesRVh0pEhdVvx148eyOV7gpzF 8WYgQDPIieJTjrzjIM45Wof X-Received: by 2002:a05:6512:3d0d:b0:5b0:1516:8799 with SMTP id 2adb3069b0e04-5b15d7766d5mr683495e87.17.1784119759380; Wed, 15 Jul 2026 05:49:19 -0700 (PDT) Received: from archlinux ([94.180.44.28]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-5b022fac999sm3404042e87.55.2026.07.15.05.49.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 15 Jul 2026 05:49:19 -0700 (PDT) From: Arsenii Pashchenko To: Greg Kroah-Hartman Cc: linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, ulijg308@gmail.com Subject: [PATCH 01/61] staging: rtl8723bs: remove redundant rtw_ap_set_group_key wrapper Date: Wed, 15 Jul 2026 19:47:29 +0700 Message-ID: <20260715124829.90290-2-ulijg308@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260715124829.90290-1-ulijg308@gmail.com> References: <20260715124829.90290-1-ulijg308@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 Eliminate the redundant rtw_ap_set_group_key() wrapper function by calling rtw_ap_set_key() directly with the set_tx parameter set to 1. This reduces boilerplate code and cleans up the driver interface. Signed-off-by: Arsenii Pashchenko --- drivers/staging/rtl8723bs/core/rtw_ap.c | 7 +------ drivers/staging/rtl8723bs/include/rtw_ap.h | 2 +- drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 8 ++++++-- 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_ap.c b/drivers/staging/rtl8723bs/core/rtw_ap.c index 065850a9e..a6640cd7a 100644 --- a/drivers/staging/rtl8723bs/core/rtw_ap.c +++ b/drivers/staging/rtl8723bs/core/rtw_ap.c @@ -1195,7 +1195,7 @@ u8 rtw_ap_set_pairwise_key(struct adapter *padapter, struct sta_info *psta) return res; } -static int rtw_ap_set_key(struct adapter *padapter, +int rtw_ap_set_key(struct adapter *padapter, u8 *key, u8 alg, int keyid, @@ -1258,11 +1258,6 @@ static int rtw_ap_set_key(struct adapter *padapter, return res; } -int rtw_ap_set_group_key(struct adapter *padapter, u8 *key, u8 alg, int keyid) -{ - return rtw_ap_set_key(padapter, key, alg, keyid, 1); -} - int rtw_ap_set_wep_key(struct adapter *padapter, u8 *key, u8 keylen, diff --git a/drivers/staging/rtl8723bs/include/rtw_ap.h b/drivers/staging/rtl8723bs/include/rtw_ap.h index 83e835eb6..cdd9f291f 100644 --- a/drivers/staging/rtl8723bs/include/rtw_ap.h +++ b/drivers/staging/rtl8723bs/include/rtw_ap.h @@ -22,7 +22,7 @@ int rtw_acl_add_sta(struct adapter *padapter, u8 *addr); void rtw_acl_remove_sta(struct adapter *padapter, u8 *addr); u8 rtw_ap_set_pairwise_key(struct adapter *padapter, struct sta_info *psta); -int rtw_ap_set_group_key(struct adapter *padapter, u8 *key, u8 alg, int keyid); +int rtw_ap_set_key(struct adapter *padapter, u8 *key, u8 alg, int keyid, u8 set_tx); int rtw_ap_set_wep_key(struct adapter *padapter, u8 *key, u8 keylen, int keyid, u8 set_tx); void associated_clients_update(struct adapter *padapter, u8 updated); diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c index 1484336d7..ce76a8586 100644 --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c @@ -604,7 +604,9 @@ static int rtw_cfg80211_ap_set_encryption(struct net_device *dev, struct ieee_pa psecuritypriv->dot11PrivacyAlgrthm = psecuritypriv->dot118021XGrpPrivacy;/* */ - rtw_ap_set_group_key(padapter, param->u.crypt.key, psecuritypriv->dot118021XGrpPrivacy, param->u.crypt.idx); + rtw_ap_set_key(padapter, param->u.crypt.key, + psecuritypriv->dot118021XGrpPrivacy, + param->u.crypt.idx, 1); pbcmc_sta = rtw_get_bcmc_stainfo(padapter); if (pbcmc_sta) { @@ -678,7 +680,9 @@ static int rtw_cfg80211_ap_set_encryption(struct net_device *dev, struct ieee_pa psecuritypriv->dot11PrivacyAlgrthm = psecuritypriv->dot118021XGrpPrivacy;/* */ - rtw_ap_set_group_key(padapter, param->u.crypt.key, psecuritypriv->dot118021XGrpPrivacy, param->u.crypt.idx); + rtw_ap_set_key(padapter, param->u.crypt.key, + psecuritypriv->dot118021XGrpPrivacy, + param->u.crypt.idx, 1); pbcmc_sta = rtw_get_bcmc_stainfo(padapter); if (pbcmc_sta) { -- 2.55.0