Commit cf7d1bdf authored Nov 28, 2017 by Joe Perches Committed by Alex Deucher Dec 06, 2017

drm/ttm: Use a static string instead of an array of char *



Make the object a bit smaller by using a simple string instead
of a format string and array of char *.

$ size drivers/gpu/drm/ttm/ttm_page_alloc_dma.o*
   text	   data	    bss	    dec	    hex	filename
   8820	    216	   4136	  13172	   3374	drivers/gpu/drm/ttm/ttm_page_alloc_dma.o.defconfig.new
   8910	    216	   4136	  13262	   33ce	drivers/gpu/drm/ttm/ttm_page_alloc_dma.o.defconfig.old
  25383	   5044	   4384	  34811	   87fb	drivers/gpu/drm/ttm/ttm_page_alloc_dma.o.allyesconfig.new
  25797	   5428	   4384	  35609	   8b19	drivers/gpu/drm/ttm/ttm_page_alloc_dma.o.allyesconfig.old

Miscellanea:

o The h array had more entries than were emitted, all are now removed

Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

parent 1e88ad0a

drivers/gpu/drm/ttm/ttm_page_alloc_dma.c

+1 −4

Original line number	Diff line number	Diff line
		@@ -1244,15 +1244,12 @@ int ttm_dma_page_alloc_debugfs(struct seq_file m, void data)
		{
		struct device_pools *p;
		struct dma_pool *pool = NULL;
		char *h[] = {"pool", "refills", "pages freed", "inuse", "available",
		"name", "virt", "busaddr"};

		if (!_manager) {
		seq_printf(m, "No pool allocator running.\n");
		return 0;
		}
		seq_printf(m, "%13s %12s %13s %8s %8s %8s\n",
		h[0], h[1], h[2], h[3], h[4], h[5]);
		seq_printf(m, " pool refills pages freed inuse available name\n");
		mutex_lock(&_manager->lock);
		list_for_each_entry(p, &_manager->pools, pools) {
		struct device *dev = p->dev;