Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

net: ipa: Remove redundant pm_runtime_mark_last_busy() calls

pm_runtime_put_autosuspend(), pm_runtime_put_sync_autosuspend(),
pm_runtime_autosuspend() and pm_request_autosuspend() now include a call
to pm_runtime_mark_last_busy(). Remove the now-reduntant explicit call to
pm_runtime_mark_last_busy().

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Link: https://patch.msgid.link/20251027115022.390997-2-sakari.ailus@linux.intel.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

authored by

Sakari Ailus and committed by
Jakub Kicinski
a5d937dd 9f2674e1

-10
-1
drivers/net/ipa/ipa_interrupt.c
··· 149 149 iowrite32(pending, ipa->reg_virt + reg_offset(reg)); 150 150 } 151 151 out_power_put: 152 - pm_runtime_mark_last_busy(dev); 153 152 (void)pm_runtime_put_autosuspend(dev); 154 153 155 154 return IRQ_HANDLED;
-1
drivers/net/ipa/ipa_main.c
··· 903 903 if (ret) 904 904 goto err_deconfig; 905 905 done: 906 - pm_runtime_mark_last_busy(dev); 907 906 (void)pm_runtime_put_autosuspend(dev); 908 907 909 908 return 0;
-4
drivers/net/ipa/ipa_modem.c
··· 71 71 72 72 netif_start_queue(netdev); 73 73 74 - pm_runtime_mark_last_busy(dev); 75 74 (void)pm_runtime_put_autosuspend(dev); 76 75 77 76 return 0; ··· 101 102 ipa_endpoint_disable_one(priv->rx); 102 103 ipa_endpoint_disable_one(priv->tx); 103 104 out_power_put: 104 - pm_runtime_mark_last_busy(dev); 105 105 (void)pm_runtime_put_autosuspend(dev); 106 106 107 107 return 0; ··· 173 175 174 176 ret = ipa_endpoint_skb_tx(endpoint, skb); 175 177 176 - pm_runtime_mark_last_busy(dev); 177 178 (void)pm_runtime_put_autosuspend(dev); 178 179 179 180 if (ret) { ··· 429 432 dev_err(dev, "error %d zeroing modem memory regions\n", ret); 430 433 431 434 out_power_put: 432 - pm_runtime_mark_last_busy(dev); 433 435 (void)pm_runtime_put_autosuspend(dev); 434 436 } 435 437
-2
drivers/net/ipa/ipa_smp2p.c
··· 171 171 WARN(ret != 0, "error %d from ipa_setup()\n", ret); 172 172 173 173 out_power_put: 174 - pm_runtime_mark_last_busy(dev); 175 174 (void)pm_runtime_put_autosuspend(dev); 176 175 177 176 return IRQ_HANDLED; ··· 212 213 if (!ipa->smp2p->power_on) 213 214 return; 214 215 215 - pm_runtime_mark_last_busy(dev); 216 216 (void)pm_runtime_put_autosuspend(dev); 217 217 ipa->smp2p->power_on = false; 218 218 }
-2
drivers/net/ipa/ipa_uc.c
··· 158 158 if (ipa->uc_powered) { 159 159 ipa->uc_loaded = true; 160 160 ipa_power_retention(ipa, true); 161 - pm_runtime_mark_last_busy(dev); 162 161 (void)pm_runtime_put_autosuspend(dev); 163 162 ipa->uc_powered = false; 164 163 } else { ··· 202 203 if (!ipa->uc_powered) 203 204 return; 204 205 205 - pm_runtime_mark_last_busy(dev); 206 206 (void)pm_runtime_put_autosuspend(dev); 207 207 } 208 208