Skip to contentSkip to conversation

power: supply: ab8500: Remove unused variable

PATCH In review Souptick Joarder 2 participants 2 messages Mar 1, 2022 – Mar 4, 2022

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 power: supply: ab8500: Remove unused variable

From: "Souptick Joarder (HPE)" <[email protected]>

Kernel test robot reported below warning -> drivers/power/supply/ab8500_chargalg.c:790:13: warning:
variable 'delta_i_ua' set but not used [-Wunused-but-set-variable]

Remove unused variable delta_i_ua.

---
 drivers/power/supply/ab8500_chargalg.c | 3 ---
 1 file changed, 3 deletions(-)
drivers/power/supply/ab8500_chargalg.c +0 -3
index b5a3096e78a1..f73d29c09caf 100644
@@ -793,13 +793,10 @@ static void init_maxim_chg_curr(struct ab8500_chargalg *di)
  */
 static enum maxim_ret ab8500_chargalg_chg_curr_maxim(struct ab8500_chargalg *di)
 {
-	int delta_i_ua;
 
 	if (!di->bm->maxi->ena_maxi)
 		return MAXIM_RET_NOACTION;
 
-	delta_i_ua = di->ccm.original_iset_ua - di->batt_data.inst_curr_ua;
-
 	if (di->events.vbus_collapsed) {
 		dev_dbg(di->dev, "Charger voltage has collapsed %d\n",
 				di->ccm.wait_cnt);
Signature · 1 line
2.25.1
#1 lore raw

Hi,

Quoted 11 lines · Souptick Joarder

From: "Souptick Joarder (HPE)" <[email protected]>

Kernel test robot reported below warning -> drivers/power/supply/ab8500_chargalg.c:790:13: warning:
variable 'delta_i_ua' set but not used [-Wunused-but-set-variable]

Remove unused variable delta_i_ua.

---

Thanks, queued.

-- Sebastian

Quoted 24 lines
 drivers/power/supply/ab8500_chargalg.c | 3 ---
 1 file changed, 3 deletions(-)
drivers/power/supply/ab8500_chargalg.c +0 -3
index b5a3096e78a1..f73d29c09caf 100644
@@ -793,13 +793,10 @@ static void init_maxim_chg_curr(struct ab8500_chargalg *di)
  */
 static enum maxim_ret ab8500_chargalg_chg_curr_maxim(struct ab8500_chargalg *di)
 {
-	int delta_i_ua;
 
 	if (!di->bm->maxi->ena_maxi)
 		return MAXIM_RET_NOACTION;
 
-	delta_i_ua = di->ccm.original_iset_ua - di->batt_data.inst_curr_ua;
-
 	if (di->events.vbus_collapsed) {
 		dev_dbg(di->dev, "Charger voltage has collapsed %d\n",
 				di->ccm.wait_cnt);
Signature · 1 line
2.25.1