From: "Bera Yüzlü" <b9788213@gmail.com>
To: gregkh@linuxfoundation.org
Cc: straube.linux@gmail.com, andriy.shevchenko@linux.intel.com,
ethantidmore06@gmail.com, hansg@kernel.org, Yeking@Red54.com,
rayfraytech@gmail.com, linux-staging@lists.linux.dev,
linux-kernel@vger.kernel.org, "Bera Yüzlü" <b9788213@gmail.com>
Subject: [PATCH] staging: rtl8723bs: remove copy function
Date: Fri, 20 Mar 2026 10:18:31 +0300 [thread overview]
Message-ID: <20260320071830.28713-2-b9788213@gmail.com> (raw)
GetU1ByteIntegerFromStringInDecimal() is a copy of kstrtou8().
Remove its usages to kstrtou8() and check the return value.
Signed-off-by: Bera Yüzlü <b9788213@gmail.com>
---
drivers/staging/rtl8723bs/hal/hal_com.c | 19 -------------------
.../staging/rtl8723bs/hal/hal_com_phycfg.c | 14 ++++++++++----
drivers/staging/rtl8723bs/include/hal_com.h | 2 --
3 files changed, 10 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/rtl8723bs/hal/hal_com.c b/drivers/staging/rtl8723bs/hal/hal_com.c
index 50370b14ce7c..4f4a430c9f87 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com.c
@@ -751,25 +751,6 @@ void SetHalODMVar(
}
}
-
-bool GetU1ByteIntegerFromStringInDecimal(char *Str, u8 *pInt)
-{
- u16 i = 0;
- *pInt = 0;
-
- while (Str[i] != '\0') {
- if (Str[i] >= '0' && Str[i] <= '9') {
- *pInt *= 10;
- *pInt += (Str[i] - '0');
- } else
- return false;
-
- ++i;
- }
-
- return true;
-}
-
void rtw_hal_check_rxfifo_full(struct adapter *adapter)
{
/* switch counter to RX fifo */
diff --git a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
index bdd595a99b98..447b72954804 100644
--- a/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
+++ b/drivers/staging/rtl8723bs/hal/hal_com_phycfg.c
@@ -816,11 +816,17 @@ void PHY_SetTxPowerLimit(
)
{
struct hal_com_data *pHalData = GET_HAL_DATA(Adapter);
- u8 regulation = 0, bandwidth = 0, rateSection = 0, channel;
- s8 powerLimit = 0, prevPowerLimit, channelIndex;
+ u8 regulation = 0, bandwidth = 0, rateSection = 0, channel, powerLimit;
+ s8 prevPowerLimit, channelIndex;
+ int ret;
- GetU1ByteIntegerFromStringInDecimal((s8 *)Channel, &channel);
- GetU1ByteIntegerFromStringInDecimal((s8 *)PowerLimit, &powerLimit);
+ ret = kstrtou8((const char *)Channel, 10, &channel);
+ if (ret)
+ return;
+
+ ret = kstrtou8((const char *)PowerLimit, 10, &powerLimit);
+ if (ret)
+ return;
powerLimit = powerLimit > MAX_POWER_INDEX ? MAX_POWER_INDEX : powerLimit;
diff --git a/drivers/staging/rtl8723bs/include/hal_com.h b/drivers/staging/rtl8723bs/include/hal_com.h
index 483f0390addc..7c67fee148fa 100644
--- a/drivers/staging/rtl8723bs/include/hal_com.h
+++ b/drivers/staging/rtl8723bs/include/hal_com.h
@@ -141,8 +141,6 @@ void rtw_hal_check_rxfifo_full(struct adapter *adapter);
u8 GetHalDefVar(struct adapter *adapter, enum hal_def_variable variable,
void *value);
-bool GetU1ByteIntegerFromStringInDecimal(char *str, u8 *in);
-
#define HWSET_MAX_SIZE 512
void rtw_bb_rf_gain_offset(struct adapter *padapter);
--
2.53.0
next reply other threads:[~2026-03-20 7:19 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-20 7:18 Bera Yüzlü [this message]
2026-03-20 8:25 ` [PATCH] staging: rtl8723bs: remove copy function Dan Carpenter
2026-03-20 8:27 ` Andy Shevchenko
2026-03-20 18:24 ` Bera Yüzlü
2026-03-20 19:19 ` Bera Yüzlü
2026-03-20 19:26 ` Andy Shevchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260320071830.28713-2-b9788213@gmail.com \
--to=b9788213@gmail.com \
--cc=Yeking@Red54.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=ethantidmore06@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=hansg@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=rayfraytech@gmail.com \
--cc=straube.linux@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.