From nobody Sun Jun 21 04:19:28 2026 Received: from mail-dy1-f181.google.com (mail-dy1-f181.google.com [74.125.82.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 E83543859EC for ; Tue, 7 Apr 2026 13:34:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.82.181 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775568868; cv=none; b=n8+8d6gnJRwTeBXIn05y7In33NJ1ntUOCeD6N4VZ6MJe/n3hVcw1GHLePGwalE63vx/u7dv/EBgY+oozL1eX1Bm3zOsFg1NgosBsNqVQYyaRvy1akJ0shwJLF+UzoPta/PSjoKdLpdtN5AMf6yivkLQ7eGEOrarlLrR1HgQmRKg= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775568868; c=relaxed/simple; bh=HlU87euuBGUAn5caArfz/j03nMokMW9Lsh5piLGHEOE=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=PBCeVmhRZU3tZ/yBSAJX4ouvig5BsZhXnNOAZGKzD6ylM7zyKu3+gFo+yYIqca1CS8zjZcHPaHgnX1B/tNsY3teVuwXpTVHShsyE48tpKD1OpTRerHM6JhXQEf84Rhv6jHx8bGMFu7I9FG6HCyxAPRsiKVkhXhtyEgfuRqkAFjU= 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=IppmudQ8; arc=none smtp.client-ip=74.125.82.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="IppmudQ8" Received: by mail-dy1-f181.google.com with SMTP id 5a478bee46e88-2ba9c484e5eso4815624eec.1 for ; Tue, 07 Apr 2026 06:34:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1775568865; x=1776173665; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=FJ5UGvzlNyGAgIzX2gahc4byA8+M/F6HMi4wZ7/WiGI=; b=IppmudQ8RTgLyw2440g6Bq1bhL9lvUbbucCoeaOIhfhmCEM9t7b1r2wA7u6eiAnoLg //zQZNpQarNccvMNF9eNq5oEWQOEwukX5QCT7ekDXTT8i+yBfl/butKkjsqO9pHUeG0O d3PJ7iR0+SpAzUiM2cozadDBo1sn7T8xsjgj+1b2Tvglw7BjghnBRDH8v/2o6IQ/R9Fo kFeYIQaLzu3WND7x2zCrunAE3h1FGj8fcQwtxYBOVI3Xm9OcJJAlGRRycOibOc/oqXiO lbhpIotlN9MlvZqjjXjhYnvWWoah4HXiGfoDCpWawXnWGo31zRFv0ZT58abnLoE3HXiz ZiiA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1775568865; x=1776173665; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=FJ5UGvzlNyGAgIzX2gahc4byA8+M/F6HMi4wZ7/WiGI=; b=E79wpMG4S+KWSmuZQ1K9xBQOtZAtPSoQUxgPz6BwuB/xlcQCi4dgACT0LhCb9laA1i iaRRNeba/GOTBXSFggc/uoiEW6ktJxIapvhwHpsOdTEL9wZfK0siZRXYNDRf2zEJ7ZDq fHMNfw85bApu6xCI0VbIx2aw/2S1wKTf3j8cvjJR1x3xIypgIvFaMsr/+pzL9hoHWmpE t6IfYefk6yfCr3v7glwacqduibS8pLXoCjc8sxAk3brpnuAmYOtltC0UhOTUhI96Rhk9 kfHaYNwivBJicVuX61oQkYnxy6aI6b//DxVBDUdb8o+Lem0wLxuk5zW9TOdp9J6ranp8 qZSw== X-Forwarded-Encrypted: i=1; AJvYcCWlN7m7BW1DKrupEHkySXYLhpJUbK9HuxxAhSmeJ6mmu4rMDWnGg1FXPwCB2jeUGhAqUgnLAwUyX2qsiSI=@vger.kernel.org X-Gm-Message-State: AOJu0YyFPXH2TKRiEPgOm+QSxyNgR773+mdPkB5rm2MnRpdRayLt6ey7 GsPyDZGGaA6Akpgj3TPQrkBOKkSniVFO/3Lqy8A3W1hMXra8w734zZSU X-Gm-Gg: AeBDies7HeKKENS8G8chzE2qmQCREfW+1TnWePxdLr17yV5xKu/aQUZLUMhafHOMjMM pL464A7r+gL3MZelNZjQBdPhJCNkFkKMpXZRkHKkFrBY3oZlov28omBm+ca171P9+SQPXPcvnZt yo1tMp1V2RplHtziwoJ0hzoy/NlIzEsjI1e5motN2BLS8c/kkdowmG6RhedKCknzyI6OUmSXrs4 pHFOdXr2i36/j1727qR5x9fnEhdDYE/BLnx4j+fD6fz0fstEk3WZqbfG9q5tlMEvPDJ6eEKscLC sdlYAiTInafkNMyI8SKW7ZldFtVCR912Mz71wixHxnV6p1bY3+td36FIbMYnfHXDg5a0SypHwth qIsH692LiEXztvoL9ocKWH7SkuEhJ7r1e6fI1yxvIKeNia1qnys/zYxPPQ6mNU+yR9iNw4xfGtZ VCL2uxoDzKuKjyVMQRsVsRa3J9oa4togTrVbATEvvHJM+Tc60BZ/iYr0PZwrGQGPhzRur3jEMys k/mfA== X-Received: by 2002:a05:7300:8c04:b0:2c4:4363:3742 with SMTP id 5a478bee46e88-2cbf9fe1c97mr8797529eec.9.1775568864869; Tue, 07 Apr 2026 06:34:24 -0700 (PDT) Received: from localhost.localdomain ([2804:14d:4c64:82a2:a160:4978:2ff1:7409]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-2d1e4ad4cc7sm1906367eec.7.2026.04.07.06.34.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 07 Apr 2026 06:34:24 -0700 (PDT) From: Rodrigo Gobbi To: gregkh@linuxfoundation.org Cc: ~lkcamp/patches@lists.sr.ht, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, linux-kernel-mentees@lists.linux.dev, a.nasrolahi01@gmail.com Subject: [PATCH] staging: rtl8723bs: fix unnamed parameters warning detected at checkpatch Date: Tue, 7 Apr 2026 10:27:21 -0300 Message-ID: <20260407133416.13983-1-rodrigo.gobbi.7@gmail.com> X-Mailer: git-send-email 2.48.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Detected the following WARNING: function definition argument 'struct adapter *' should also have an identifier name. Add explicit names on those definitions. Signed-off-by: Rodrigo Gobbi --- Hi, all, Found a set of these warnings with checkpatch: drivers/staging/rtl8723bs/include/hal_btcoex.h:25: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/hal_btcoex.h:49: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/hal_btcoex.h:50: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/hal_btcoex.h:51: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/hal_intf.h:255: WARNING: function definit= ion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_btcoex.h:18: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_btcoex.h:19: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_btcoex.h:24: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_btcoex.h:25: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_btcoex.h:26: WARNING: function defini= tion argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_pwrctrl.h:218: WARNING: function defi= nition argument 'struct adapter *' should also have an identifier name drivers/staging/rtl8723bs/include/rtw_pwrctrl.h:219: WARNING: function defi= nition argument 'struct adapter *' should also have an identifier name ...the ones from hal_btcoex.h is already ongoing at [1]. Fixed the rest of them following the pattern of each file like: struct adapter *padapter or=20 struct adapter *adapter Tks and regards. [1] https://lore.kernel.org/all/20250720145524.9733-4-A.Nasrolahi01@gmail.c= om/ --- drivers/staging/rtl8723bs/include/hal_intf.h | 2 +- drivers/staging/rtl8723bs/include/rtw_btcoex.h | 10 +++++----- drivers/staging/rtl8723bs/include/rtw_pwrctrl.h | 4 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8723bs/include/hal_intf.h b/drivers/staging= /rtl8723bs/include/hal_intf.h index b193854bfe6e..10762ffbafb6 100644 --- a/drivers/staging/rtl8723bs/include/hal_intf.h +++ b/drivers/staging/rtl8723bs/include/hal_intf.h @@ -252,7 +252,7 @@ c2h_id_filter rtw_hal_c2h_id_filter_ccx(struct adapter = *adapter); s32 rtw_hal_macid_sleep(struct adapter *padapter, u32 macid); s32 rtw_hal_macid_wakeup(struct adapter *padapter, u32 macid); =20 -s32 rtw_hal_fill_h2c_cmd(struct adapter *, u8 ElementID, u32 CmdLen, u8 *p= CmdBuffer); +s32 rtw_hal_fill_h2c_cmd(struct adapter *padapter, u8 ElementID, u32 CmdLe= n, u8 *pCmdBuffer); =20 void SetHwReg8723BS(struct adapter *padapter, u8 variable, u8 *val); void GetHwReg8723BS(struct adapter *padapter, u8 variable, u8 *val); diff --git a/drivers/staging/rtl8723bs/include/rtw_btcoex.h b/drivers/stagi= ng/rtl8723bs/include/rtw_btcoex.h index 19764c80b8ba..cb566825e77f 100644 --- a/drivers/staging/rtl8723bs/include/rtw_btcoex.h +++ b/drivers/staging/rtl8723bs/include/rtw_btcoex.h @@ -15,14 +15,14 @@ #define PACKET_ARP 2 #define PACKET_EAPOL 3 =20 -void rtw_btcoex_MediaStatusNotify(struct adapter *, u8 mediaStatus); -void rtw_btcoex_HaltNotify(struct adapter *); +void rtw_btcoex_MediaStatusNotify(struct adapter *padapter, u8 mediaStatus= ); +void rtw_btcoex_HaltNotify(struct adapter *padapter); =20 /* =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D */ /* Below Functions are called by BT-Coex */ /* =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D */ -void rtw_btcoex_RejectApAggregatedPacket(struct adapter *, u8 enable); -void rtw_btcoex_LPS_Enter(struct adapter *); -void rtw_btcoex_LPS_Leave(struct adapter *); +void rtw_btcoex_RejectApAggregatedPacket(struct adapter *padapter, u8 enab= le); +void rtw_btcoex_LPS_Enter(struct adapter *padapter); +void rtw_btcoex_LPS_Leave(struct adapter *padapter); =20 #endif /* __RTW_BTCOEX_H__ */ diff --git a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h b/drivers/stag= ing/rtl8723bs/include/rtw_pwrctrl.h index c27d07861b8c..df5dbcceea06 100644 --- a/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h +++ b/drivers/staging/rtl8723bs/include/rtw_pwrctrl.h @@ -215,8 +215,8 @@ struct pwrctrl_priv { extern void rtw_init_pwrctrl_priv(struct adapter *adapter); extern void rtw_free_pwrctrl_priv(struct adapter *adapter); =20 -s32 rtw_register_task_alive(struct adapter *, u32 task); -void rtw_unregister_task_alive(struct adapter *, u32 task); +s32 rtw_register_task_alive(struct adapter *adapter, u32 task); +void rtw_unregister_task_alive(struct adapter *adapter, u32 task); extern s32 rtw_register_tx_alive(struct adapter *padapter); extern void rtw_unregister_tx_alive(struct adapter *padapter); extern s32 rtw_register_cmd_alive(struct adapter *padapter); --=20 2.48.1