diff mbox series

[24/51] mailbox: mtk-cmdq-mailbox: Switch to __pm_runtime_put_autosuspend()

Message ID 20241004094123.113759-1-sakari.ailus@linux.intel.com (mailing list archive)
State New
Headers show
Series treewide: Switch to __pm_runtime_put_autosuspend() | expand

Commit Message

Sakari Ailus Oct. 4, 2024, 9:41 a.m. UTC
pm_runtime_put_autosuspend() will soon be changed to include a call to
pm_runtime_mark_last_busy(). This patch switches the current users to
__pm_runtime_put_autosuspend() which will continue to have the
functionality of old pm_runtime_put_autosuspend().

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
---
 drivers/mailbox/mtk-cmdq-mailbox.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Matthias Brugger Oct. 7, 2024, 2:27 p.m. UTC | #1
On 04/10/2024 11:41, Sakari Ailus wrote:
> pm_runtime_put_autosuspend() will soon be changed to include a call to
> pm_runtime_mark_last_busy(). This patch switches the current users to
> __pm_runtime_put_autosuspend() which will continue to have the
> functionality of old pm_runtime_put_autosuspend().
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>

Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

> ---
>   drivers/mailbox/mtk-cmdq-mailbox.c | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c
> index 4bff73532085..180906761eda 100644
> --- a/drivers/mailbox/mtk-cmdq-mailbox.c
> +++ b/drivers/mailbox/mtk-cmdq-mailbox.c
> @@ -397,7 +397,7 @@ static int cmdq_mbox_send_data(struct mbox_chan *chan, void *data)
>   
>   	task = kzalloc(sizeof(*task), GFP_ATOMIC);
>   	if (!task) {
> -		pm_runtime_put_autosuspend(cmdq->mbox.dev);
> +		__pm_runtime_put_autosuspend(cmdq->mbox.dev);
>   		return -ENOMEM;
>   	}
>   
> @@ -447,7 +447,7 @@ static int cmdq_mbox_send_data(struct mbox_chan *chan, void *data)
>   	list_move_tail(&task->list_entry, &thread->task_busy_list);
>   
>   	pm_runtime_mark_last_busy(cmdq->mbox.dev);
> -	pm_runtime_put_autosuspend(cmdq->mbox.dev);
> +	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
>   
>   	return 0;
>   }
> @@ -495,7 +495,7 @@ static void cmdq_mbox_shutdown(struct mbox_chan *chan)
>   	spin_unlock_irqrestore(&thread->chan->lock, flags);
>   
>   	pm_runtime_mark_last_busy(cmdq->mbox.dev);
> -	pm_runtime_put_autosuspend(cmdq->mbox.dev);
> +	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
>   }
>   
>   static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
> @@ -535,7 +535,7 @@ static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
>   out:
>   	spin_unlock_irqrestore(&thread->chan->lock, flags);
>   	pm_runtime_mark_last_busy(cmdq->mbox.dev);
> -	pm_runtime_put_autosuspend(cmdq->mbox.dev);
> +	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
>   
>   	return 0;
>   
> @@ -550,7 +550,7 @@ static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
>   		return -EFAULT;
>   	}
>   	pm_runtime_mark_last_busy(cmdq->mbox.dev);
> -	pm_runtime_put_autosuspend(cmdq->mbox.dev);
> +	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
>   	return 0;
>   }
>
diff mbox series

Patch

diff --git a/drivers/mailbox/mtk-cmdq-mailbox.c b/drivers/mailbox/mtk-cmdq-mailbox.c
index 4bff73532085..180906761eda 100644
--- a/drivers/mailbox/mtk-cmdq-mailbox.c
+++ b/drivers/mailbox/mtk-cmdq-mailbox.c
@@ -397,7 +397,7 @@  static int cmdq_mbox_send_data(struct mbox_chan *chan, void *data)
 
 	task = kzalloc(sizeof(*task), GFP_ATOMIC);
 	if (!task) {
-		pm_runtime_put_autosuspend(cmdq->mbox.dev);
+		__pm_runtime_put_autosuspend(cmdq->mbox.dev);
 		return -ENOMEM;
 	}
 
@@ -447,7 +447,7 @@  static int cmdq_mbox_send_data(struct mbox_chan *chan, void *data)
 	list_move_tail(&task->list_entry, &thread->task_busy_list);
 
 	pm_runtime_mark_last_busy(cmdq->mbox.dev);
-	pm_runtime_put_autosuspend(cmdq->mbox.dev);
+	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
 
 	return 0;
 }
@@ -495,7 +495,7 @@  static void cmdq_mbox_shutdown(struct mbox_chan *chan)
 	spin_unlock_irqrestore(&thread->chan->lock, flags);
 
 	pm_runtime_mark_last_busy(cmdq->mbox.dev);
-	pm_runtime_put_autosuspend(cmdq->mbox.dev);
+	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
 }
 
 static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
@@ -535,7 +535,7 @@  static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
 out:
 	spin_unlock_irqrestore(&thread->chan->lock, flags);
 	pm_runtime_mark_last_busy(cmdq->mbox.dev);
-	pm_runtime_put_autosuspend(cmdq->mbox.dev);
+	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
 
 	return 0;
 
@@ -550,7 +550,7 @@  static int cmdq_mbox_flush(struct mbox_chan *chan, unsigned long timeout)
 		return -EFAULT;
 	}
 	pm_runtime_mark_last_busy(cmdq->mbox.dev);
-	pm_runtime_put_autosuspend(cmdq->mbox.dev);
+	__pm_runtime_put_autosuspend(cmdq->mbox.dev);
 	return 0;
 }