On 06/11/2025 14:46, Boris Brezillon wrote:
> It's only used in a couple places and everyone else is just using
> sched_queue_delayed_work(sched, tick, 0) directly, so let's make
> this consistent.
> 
> Signed-off-by: Boris Brezillon <[email protected]>

Reviewed-by: Steven Price <[email protected]>

> ---
>  drivers/gpu/drm/panthor/panthor_sched.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/drivers/gpu/drm/panthor/panthor_sched.c 
> b/drivers/gpu/drm/panthor/panthor_sched.c
> index 923816397751..6b8428ca8145 100644
> --- a/drivers/gpu/drm/panthor/panthor_sched.c
> +++ b/drivers/gpu/drm/panthor/panthor_sched.c
> @@ -2668,13 +2668,6 @@ static void panthor_group_start(struct panthor_group 
> *group)
>       group_put(group);
>  }
>  
> -static void panthor_sched_immediate_tick(struct panthor_device *ptdev)
> -{
> -     struct panthor_scheduler *sched = ptdev->scheduler;
> -
> -     sched_queue_delayed_work(sched, tick, 0);
> -}
> -
>  /**
>   * panthor_sched_report_mmu_fault() - Report MMU faults to the scheduler.
>   */
> @@ -2682,13 +2675,13 @@ void panthor_sched_report_mmu_fault(struct 
> panthor_device *ptdev)
>  {
>       /* Force a tick to immediately kill faulty groups. */
>       if (ptdev->scheduler)
> -             panthor_sched_immediate_tick(ptdev);
> +             sched_queue_delayed_work(ptdev->scheduler, tick, 0);
>  }
>  
>  void panthor_sched_resume(struct panthor_device *ptdev)
>  {
>       /* Force a tick to re-evaluate after a resume. */
> -     panthor_sched_immediate_tick(ptdev);
> +     sched_queue_delayed_work(ptdev->scheduler, tick, 0);
>  }
>  
>  void panthor_sched_suspend(struct panthor_device *ptdev)

Reply via email to