diff mbox series

gpu: host1x: Remove unused host1x_debug_dump_syncpts

Message ID 20241215214750.448209-1-linux@treblig.org
State Accepted
Headers show
Series gpu: host1x: Remove unused host1x_debug_dump_syncpts | expand

Commit Message

Dr. David Alan Gilbert Dec. 15, 2024, 9:47 p.m. UTC 
From: "Dr. David Alan Gilbert" <linux@treblig.org>

host1x_debug_dump_syncpts() has been unused since
commit f0fb260a0cdb ("gpu: host1x: Implement syncpoint wait using DMA
fences")

Remove it.

Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
---
 drivers/gpu/host1x/debug.c | 9 ---------
 drivers/gpu/host1x/debug.h | 1 -
 2 files changed, 10 deletions(-)

Comments

Mikko Perttunen Dec. 17, 2024, 7:24 a.m. UTC | #1 
On 12/16/24 6:47 AM, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> host1x_debug_dump_syncpts() has been unused since
> commit f0fb260a0cdb ("gpu: host1x: Implement syncpoint wait using DMA
> fences")
> 
> Remove it.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>   drivers/gpu/host1x/debug.c | 9 ---------
>   drivers/gpu/host1x/debug.h | 1 -
>   2 files changed, 10 deletions(-)
> 
> diff --git a/drivers/gpu/host1x/debug.c b/drivers/gpu/host1x/debug.c
> index a18cc8d8caf5..6433c00d5d7e 100644
> --- a/drivers/gpu/host1x/debug.c
> +++ b/drivers/gpu/host1x/debug.c
> @@ -216,12 +216,3 @@ void host1x_debug_dump(struct host1x *host1x)
>   
>   	show_all(host1x, &o, true);
>   }
> -
> -void host1x_debug_dump_syncpts(struct host1x *host1x)
> -{
> -	struct output o = {
> -		.fn = write_to_printk
> -	};
> -
> -	show_syncpts(host1x, &o, false);
> -}
> diff --git a/drivers/gpu/host1x/debug.h b/drivers/gpu/host1x/debug.h
> index 62bd8a091fa7..c43c61d876a9 100644
> --- a/drivers/gpu/host1x/debug.h
> +++ b/drivers/gpu/host1x/debug.h
> @@ -41,6 +41,5 @@ extern unsigned int host1x_debug_trace_cmdbuf;
>   void host1x_debug_init(struct host1x *host1x);
>   void host1x_debug_deinit(struct host1x *host1x);
>   void host1x_debug_dump(struct host1x *host1x);
> -void host1x_debug_dump_syncpts(struct host1x *host1x);
>   
>   #endif

Acked-by: Mikko Perttunen <mperttunen@nvidia.com>
Dr. David Alan Gilbert Feb. 28, 2025, 3:48 p.m. UTC | #2 
Hi,

* Mikko Perttunen (cyndis@kapsi.fi) wrote:
> On 12/16/24 6:47 AM, linux@treblig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > 
> > host1x_debug_dump_syncpts() has been unused since
> > commit f0fb260a0cdb ("gpu: host1x: Implement syncpoint wait using DMA
> > fences")
> > 
> > Remove it.
> > 
> > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > ---
> >   drivers/gpu/host1x/debug.c | 9 ---------
> >   drivers/gpu/host1x/debug.h | 1 -
> >   2 files changed, 10 deletions(-)
> > 
> > diff --git a/drivers/gpu/host1x/debug.c b/drivers/gpu/host1x/debug.c
> > index a18cc8d8caf5..6433c00d5d7e 100644
> > --- a/drivers/gpu/host1x/debug.c
> > +++ b/drivers/gpu/host1x/debug.c
> > @@ -216,12 +216,3 @@ void host1x_debug_dump(struct host1x *host1x)
> >   	show_all(host1x, &o, true);
> >   }
> > -
> > -void host1x_debug_dump_syncpts(struct host1x *host1x)
> > -{
> > -	struct output o = {
> > -		.fn = write_to_printk
> > -	};
> > -
> > -	show_syncpts(host1x, &o, false);
> > -}
> > diff --git a/drivers/gpu/host1x/debug.h b/drivers/gpu/host1x/debug.h
> > index 62bd8a091fa7..c43c61d876a9 100644
> > --- a/drivers/gpu/host1x/debug.h
> > +++ b/drivers/gpu/host1x/debug.h
> > @@ -41,6 +41,5 @@ extern unsigned int host1x_debug_trace_cmdbuf;
> >   void host1x_debug_init(struct host1x *host1x);
> >   void host1x_debug_deinit(struct host1x *host1x);
> >   void host1x_debug_dump(struct host1x *host1x);
> > -void host1x_debug_dump_syncpts(struct host1x *host1x);
> >   #endif
> 
> Acked-by: Mikko Perttunen <mperttunen@nvidia.com>

Can this find it's way into drm please.

Thanks,

Dave
Thierry Reding Feb. 28, 2025, 4:46 p.m. UTC | #3 
On Sun, Dec 15, 2024 at 09:47:50PM +0000, linux@treblig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@treblig.org>
> 
> host1x_debug_dump_syncpts() has been unused since
> commit f0fb260a0cdb ("gpu: host1x: Implement syncpoint wait using DMA
> fences")
> 
> Remove it.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> ---
>  drivers/gpu/host1x/debug.c | 9 ---------
>  drivers/gpu/host1x/debug.h | 1 -
>  2 files changed, 10 deletions(-)

Applied to drm-misc-next, thanks.

Thierry
Dr. David Alan Gilbert Feb. 28, 2025, 6:15 p.m. UTC | #4 
* Thierry Reding (thierry.reding@gmail.com) wrote:
> On Sun, Dec 15, 2024 at 09:47:50PM +0000, linux@treblig.org wrote:
> > From: "Dr. David Alan Gilbert" <linux@treblig.org>
> > 
> > host1x_debug_dump_syncpts() has been unused since
> > commit f0fb260a0cdb ("gpu: host1x: Implement syncpoint wait using DMA
> > fences")
> > 
> > Remove it.
> > 
> > Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org>
> > ---
> >  drivers/gpu/host1x/debug.c | 9 ---------
> >  drivers/gpu/host1x/debug.h | 1 -
> >  2 files changed, 10 deletions(-)
> 
> Applied to drm-misc-next, thanks.

Thanks

> Thierry
diff mbox series

Patch

diff --git a/drivers/gpu/host1x/debug.c b/drivers/gpu/host1x/debug.c
index a18cc8d8caf5..6433c00d5d7e 100644
--- a/drivers/gpu/host1x/debug.c
+++ b/drivers/gpu/host1x/debug.c
@@ -216,12 +216,3 @@  void host1x_debug_dump(struct host1x *host1x)
 
 	show_all(host1x, &o, true);
 }
-
-void host1x_debug_dump_syncpts(struct host1x *host1x)
-{
-	struct output o = {
-		.fn = write_to_printk
-	};
-
-	show_syncpts(host1x, &o, false);
-}
diff --git a/drivers/gpu/host1x/debug.h b/drivers/gpu/host1x/debug.h
index 62bd8a091fa7..c43c61d876a9 100644
--- a/drivers/gpu/host1x/debug.h
+++ b/drivers/gpu/host1x/debug.h
@@ -41,6 +41,5 @@  extern unsigned int host1x_debug_trace_cmdbuf;
 void host1x_debug_init(struct host1x *host1x);
 void host1x_debug_deinit(struct host1x *host1x);
 void host1x_debug_dump(struct host1x *host1x);
-void host1x_debug_dump_syncpts(struct host1x *host1x);
 
 #endif