[Freedreno] [PATCH] drm: Print unadorned pointers

Felix Kuehling felix.kuehling at amd.com
Wed Apr 18 15:53:31 UTC 2018


On 2018-04-18 05:24 AM, Alexey Brodkin wrote:
> After commit ad67b74 ("printk: hash addresses printed with %p")
> pointers are being hashed when printed. However, this makes
> debug output completely useless. Switch to %px in order to see the
> unadorned kernel pointers.
My understanding of the printk pointer hashing change was to force
people to think more carefully when they really need to print a kernel
pointer. When it is only used to identify an object, then a hash works
just fine. Most of the changes I see in amdgpu/amdkfd fall into this
category.

As I see it, changing all %p to %px by a script takes the thought
process out of it and subverts the intention of the original pointer
hashing change.

Regards,
  Felix


>
> This was done with the following one-liner:
>  find drivers/gpu/drm -type f -name "*.c" -exec sed -r -i '/DRM_DEBUG|KERN_DEBUG|pr_debug/ s/%p\b/%px/g' {} +
>
> Signed-off-by: Alexey Brodkin <abrodkin at synopsys.com>
> Cc: Borislav Petkov <bp at suse.de>
> Cc: Tobin C. Harding <me at tobin.cc>
> Cc: Alex Deucher <alexander.deucher at amd.com>
> Cc: Andrey Grodzovsky <andrey.grodzovsky at amd.com>
> Cc: Arnd Bergmann <arnd at arndb.de>
> Cc: Benjamin Gaignard <benjamin.gaignard at linaro.org>
> Cc: Chen-Yu Tsai <wens at csie.org>
> Cc: Christian Gmeiner <christian.gmeiner at gmail.com>
> Cc: "Christian König" <christian.koenig at amd.com>
> Cc: Cihangir Akturk <cakturk at gmail.com>
> Cc: CK Hu <ck.hu at mediatek.com>
> Cc: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: Dave Airlie <airlied at redhat.com>
> Cc: David Airlie <airlied at linux.ie>
> Cc: "David (ChunMing) Zhou" <David1.Zhou at amd.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> Cc: Gustavo Padovan <gustavo at padovan.org>
> Cc: Harry Wentland <harry.wentland at amd.com>
> Cc: "Heiko Stübner" <heiko at sntech.de>
> Cc: Ingo Molnar <mingo at kernel.org>
> Cc: Jani Nikula <jani.nikula at linux.intel.com>
> Cc: "Jerry (Fangzhi) Zuo" <Jerry.Zuo at amd.com>
> Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> Cc: Krzysztof Kozlowski <krzk at kernel.org>
> Cc: "Leo (Sunpeng) Li" <sunpeng.li at amd.com>
> Cc: Lucas Stach <l.stach at pengutronix.de>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Cc: Matthias Brugger <matthias.bgg at gmail.com>
> Cc: Maxime Ripard <maxime.ripard at bootlin.com>
> Cc: "Michel Dänzer" <michel.daenzer at amd.com>
> Cc: Oded Gabbay <oded.gabbay at gmail.com>
> Cc: Philipp Zabel <p.zabel at pengutronix.de>
> Cc: Rob Clark <robdclark at gmail.com>
> Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Cc: Roger He <Hongbo.He at amd.com>
> Cc: Roman Li <Roman.Li at amd.com>
> Cc: Russell King <linux at armlinux.org.uk>
> Cc: Samuel Li <Samuel.Li at amd.com>
> Cc: Sandy Huang <hjc at rock-chips.com>
> Cc: Sean Paul <seanpaul at chromium.org>
> Cc: Shirish S <shirish.s at amd.com>
> Cc: Sinclair Yeh <syeh at vmware.com>
> Cc: Thomas Hellstrom <thellstrom at vmware.com>
> Cc: Tom Lendacky <thomas.lendacky at amd.com>
> Cc: Tony Cheng <Tony.Cheng at amd.com>
> Cc: Vincent Abriou <vincent.abriou at st.com>
> Cc: VMware Graphics <linux-graphics-maintainer at vmware.com>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-arm-msm at vger.kernel.org
> Cc: linux-kernel at vger.kernel.org
> Cc: linux-mediatek at lists.infradead.org
> Cc: linux-rockchip at lists.infradead.org
> Cc: etnaviv at lists.freedesktop.org
> Cc: freedreno at lists.freedesktop.org
> Cc: amd-gfx at lists.freedesktop.org
> Cc: intel-gfx at lists.freedesktop.org
> Cc: virtualization at lists.linux-foundation.org
> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c   | 14 +++----
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c        |  4 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_chardev.c           |  4 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c            |  2 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_device.c            | 10 ++---
>  drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c          |  4 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_events.c            |  4 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c      |  2 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_process.c           |  4 +-
>  drivers/gpu/drm/amd/amdkfd/kfd_queue.c             | 18 ++++-----
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  | 14 +++----
>  .../amd/display/amdgpu_dm/amdgpu_dm_mst_types.c    |  2 +-
>  drivers/gpu/drm/armada/armada_gem.c                | 12 +++---
>  drivers/gpu/drm/drm_atomic.c                       | 44 +++++++++++-----------
>  drivers/gpu/drm/drm_bufs.c                         |  8 ++--
>  drivers/gpu/drm/drm_dp_mst_topology.c              |  4 +-
>  drivers/gpu/drm/drm_lease.c                        |  6 +--
>  drivers/gpu/drm/drm_lock.c                         |  2 +-
>  drivers/gpu/drm/drm_scatter.c                      |  4 +-
>  drivers/gpu/drm/etnaviv/etnaviv_drv.c              |  6 +--
>  drivers/gpu/drm/i810/i810_dma.c                    |  2 +-
>  drivers/gpu/drm/i915/i915_perf.c                   |  2 +-
>  drivers/gpu/drm/i915/intel_display.c               |  2 +-
>  drivers/gpu/drm/i915/intel_guc_ct.c                |  4 +-
>  drivers/gpu/drm/i915/intel_guc_submission.c        |  2 +-
>  drivers/gpu/drm/i915/intel_uc_fw.c                 |  2 +-
>  drivers/gpu/drm/mediatek/mtk_drm_gem.c             |  2 +-
>  drivers/gpu/drm/mga/mga_warp.c                     |  2 +-
>  drivers/gpu/drm/msm/msm_drv.c                      |  4 +-
>  drivers/gpu/drm/qxl/qxl_cmd.c                      |  4 +-
>  drivers/gpu/drm/qxl/qxl_fb.c                       |  2 +-
>  drivers/gpu/drm/qxl/qxl_ttm.c                      |  2 +-
>  drivers/gpu/drm/radeon/radeon_display.c            |  2 +-
>  drivers/gpu/drm/radeon/radeon_dp_mst.c             | 12 +++---
>  drivers/gpu/drm/radeon/radeon_object.c             |  2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c      |  2 +-
>  drivers/gpu/drm/savage/savage_bci.c                |  2 +-
>  drivers/gpu/drm/sti/sti_gdp.c                      |  4 +-
>  drivers/gpu/drm/sti/sti_mixer.c                    |  2 +-
>  drivers/gpu/drm/sun4i/sun4i_crtc.c                 |  4 +-
>  drivers/gpu/drm/ttm/ttm_page_alloc.c               |  2 +-
>  drivers/gpu/drm/udl/udl_fb.c                       |  2 +-
>  drivers/gpu/drm/via/via_dma.c                      |  4 +-
>  drivers/gpu/drm/via/via_irq.c                      |  2 +-
>  drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c            |  2 +-
>  45 files changed, 120 insertions(+), 120 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> index 1d6e1479da38..32e85fe83152 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c
> @@ -438,7 +438,7 @@ static int add_bo_to_vm(struct amdgpu_device *adev, struct kgd_mem *mem,
>  	if (!bo_va_entry)
>  		return -ENOMEM;
>  
> -	pr_debug("\t add VA 0x%llx - 0x%llx to vm %p\n", va,
> +	pr_debug("\t add VA 0x%llx - 0x%llx to vm %px\n", va,
>  			va + bo_size, vm);
>  
>  	/* Add BO to VM internal data structures*/
> @@ -497,7 +497,7 @@ static int add_bo_to_vm(struct amdgpu_device *adev, struct kgd_mem *mem,
>  static void remove_bo_from_vm(struct amdgpu_device *adev,
>  		struct kfd_bo_va_list *entry, unsigned long size)
>  {
> -	pr_debug("\t remove VA 0x%llx - 0x%llx in entry %p\n",
> +	pr_debug("\t remove VA 0x%llx - 0x%llx in entry %px\n",
>  			entry->va,
>  			entry->va + size, entry);
>  	amdgpu_vm_bo_rmv(adev, entry->bo_va);
> @@ -982,7 +982,7 @@ void amdgpu_amdkfd_gpuvm_destroy_process_vm(struct kgd_dev *kgd, void *vm)
>  	if (WARN_ON(!kgd || !vm))
>  		return;
>  
> -	pr_debug("Destroying process vm %p\n", vm);
> +	pr_debug("Destroying process vm %px\n", vm);
>  
>  	/* Release the VM context */
>  	amdgpu_vm_fini(adev, avm);
> @@ -1191,7 +1191,7 @@ int amdgpu_amdkfd_gpuvm_map_memory_to_gpu(
>  	domain = mem->domain;
>  	bo_size = bo->tbo.mem.size;
>  
> -	pr_debug("Map VA 0x%llx - 0x%llx to vm %p domain %s\n",
> +	pr_debug("Map VA 0x%llx - 0x%llx to vm %px domain %s\n",
>  			mem->va,
>  			mem->va + bo_size * (1 + mem->aql_queue),
>  			vm, domain_string(domain));
> @@ -1231,7 +1231,7 @@ int amdgpu_amdkfd_gpuvm_map_memory_to_gpu(
>  
>  	list_for_each_entry(entry, &mem->bo_va_list, bo_list) {
>  		if (entry->bo_va->base.vm == vm && !entry->is_mapped) {
> -			pr_debug("\t map VA 0x%llx - 0x%llx in entry %p\n",
> +			pr_debug("\t map VA 0x%llx - 0x%llx in entry %px\n",
>  					entry->va, entry->va + bo_size,
>  					entry);
>  
> @@ -1302,14 +1302,14 @@ int amdgpu_amdkfd_gpuvm_unmap_memory_from_gpu(
>  	if (unlikely(ret))
>  		goto unreserve_out;
>  
> -	pr_debug("Unmap VA 0x%llx - 0x%llx from vm %p\n",
> +	pr_debug("Unmap VA 0x%llx - 0x%llx from vm %px\n",
>  		mem->va,
>  		mem->va + bo_size * (1 + mem->aql_queue),
>  		vm);
>  
>  	list_for_each_entry(entry, &mem->bo_va_list, bo_list) {
>  		if (entry->bo_va->base.vm == vm && entry->is_mapped) {
> -			pr_debug("\t unmap VA 0x%llx - 0x%llx from entry %p\n",
> +			pr_debug("\t unmap VA 0x%llx - 0x%llx from entry %px\n",
>  					entry->va,
>  					entry->va + bo_size,
>  					entry);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> index 93f700ab1bfb..44a19d7cc941 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> @@ -112,7 +112,7 @@ static void amdgpu_display_flip_work_func(struct work_struct *__work)
>  	spin_unlock_irqrestore(&crtc->dev->event_lock, flags);
>  
>  
> -	DRM_DEBUG_DRIVER("crtc:%d[%p], pflip_stat:AMDGPU_FLIP_SUBMITTED, work: %p,\n",
> +	DRM_DEBUG_DRIVER("crtc:%d[%px], pflip_stat:AMDGPU_FLIP_SUBMITTED, work: %px,\n",
>  					 amdgpu_crtc->crtc_id, amdgpu_crtc, work);
>  
>  }
> @@ -226,7 +226,7 @@ int amdgpu_display_crtc_page_flip_target(struct drm_crtc *crtc,
>  	amdgpu_crtc->pflip_works = work;
>  
>  
> -	DRM_DEBUG_DRIVER("crtc:%d[%p], pflip_stat:AMDGPU_FLIP_PENDING, work: %p,\n",
> +	DRM_DEBUG_DRIVER("crtc:%d[%px], pflip_stat:AMDGPU_FLIP_PENDING, work: %px,\n",
>  					 amdgpu_crtc->crtc_id, amdgpu_crtc, work);
>  	/* update crtc fb */
>  	crtc->primary->fb = fb;
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c b/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> index cd679cf1fd30..fc91de6a7f02 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_chardev.c
> @@ -233,7 +233,7 @@ static int set_queue_properties_from_user(struct queue_properties *q_properties,
>  	pr_debug("Queue Size: 0x%llX, %u\n",
>  			q_properties->queue_size, args->ring_size);
>  
> -	pr_debug("Queue r/w Pointers: %p, %p\n",
> +	pr_debug("Queue r/w Pointers: %px, %px\n",
>  			q_properties->read_ptr,
>  			q_properties->write_ptr);
>  
> @@ -725,7 +725,7 @@ static int kfd_ioctl_dbg_wave_control(struct file *filep,
>  
>  	mutex_lock(kfd_get_dbgmgr_mutex());
>  
> -	pr_debug("Calling dbg manager process %p, operand %u, mode %u, trapId %u, message %u\n",
> +	pr_debug("Calling dbg manager process %px, operand %u, mode %u, trapId %u, message %u\n",
>  			wac_info.process, wac_info.operand,
>  			wac_info.mode, wac_info.trapId,
>  			wac_info.dbgWave_msg.DbgWaveMsg.WaveMsgInfoGen2.Value);
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c b/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
> index afb26f205d29..c3350d0270b2 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_dbgdev.c
> @@ -401,7 +401,7 @@ static int dbgdev_address_watch_diq(struct kfd_dbgdev *dbgdev,
>  		pr_debug("\t\t%30s\n", "* * * * * * * * * * * * * * * * * *");
>  		pr_debug("\t\t%20s %08x\n", "register index :", i);
>  		pr_debug("\t\t%20s %08x\n", "vmid is :", vmid);
> -		pr_debug("\t\t%20s %p\n", "Add ptr is :",
> +		pr_debug("\t\t%20s %px\n", "Add ptr is :",
>  				adw_info->watch_address);
>  		pr_debug("\t\t%20s %08llx\n", "Add     is :",
>  				adw_info->watch_address[i]);
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device.c b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> index 3346699960dd..57c26155e42e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device.c
> @@ -612,7 +612,7 @@ static int kfd_gtt_sa_init(struct kfd_dev *kfd, unsigned int buf_size,
>  	if (!kfd->gtt_sa_bitmap)
>  		return -ENOMEM;
>  
> -	pr_debug("gtt_sa_num_of_chunks = %d, gtt_sa_bitmap = %p\n",
> +	pr_debug("gtt_sa_num_of_chunks = %d, gtt_sa_bitmap = %px\n",
>  			kfd->gtt_sa_num_of_chunks, kfd->gtt_sa_bitmap);
>  
>  	mutex_init(&kfd->gtt_sa_lock);
> @@ -656,7 +656,7 @@ int kfd_gtt_sa_allocate(struct kfd_dev *kfd, unsigned int size,
>  	if ((*mem_obj) == NULL)
>  		return -ENOMEM;
>  
> -	pr_debug("Allocated mem_obj = %p for size = %d\n", *mem_obj, size);
> +	pr_debug("Allocated mem_obj = %px for size = %d\n", *mem_obj, size);
>  
>  	start_search = 0;
>  
> @@ -686,7 +686,7 @@ int kfd_gtt_sa_allocate(struct kfd_dev *kfd, unsigned int size,
>  					found,
>  					kfd->gtt_sa_chunk_size);
>  
> -	pr_debug("gpu_addr = %p, cpu_addr = %p\n",
> +	pr_debug("gpu_addr = %px, cpu_addr = %px\n",
>  			(uint64_t *) (*mem_obj)->gpu_addr, (*mem_obj)->cpu_ptr);
>  
>  	/* If we need only one chunk, mark it as allocated and get out */
> @@ -740,7 +740,7 @@ int kfd_gtt_sa_allocate(struct kfd_dev *kfd, unsigned int size,
>  	return 0;
>  
>  kfd_gtt_no_free_chunk:
> -	pr_debug("Allocation failed with mem_obj = %p\n", mem_obj);
> +	pr_debug("Allocation failed with mem_obj = %px\n", mem_obj);
>  	mutex_unlock(&kfd->gtt_sa_lock);
>  	kfree(mem_obj);
>  	return -ENOMEM;
> @@ -754,7 +754,7 @@ int kfd_gtt_sa_free(struct kfd_dev *kfd, struct kfd_mem_obj *mem_obj)
>  	if (!mem_obj)
>  		return 0;
>  
> -	pr_debug("Free mem_obj = %p, range_start = %d, range_end = %d\n",
> +	pr_debug("Free mem_obj = %px, range_start = %d, range_end = %d\n",
>  			mem_obj, mem_obj->range_start, mem_obj->range_end);
>  
>  	mutex_lock(&kfd->gtt_sa_lock);
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
> index ebb4da14e3df..7b5ad0c577c8 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c
> @@ -116,7 +116,7 @@ int kfd_doorbell_init(struct kfd_dev *kfd)
>  	pr_debug("doorbell aperture size  == 0x%08lX\n",
>  			kfd->shared_resources.doorbell_aperture_size);
>  
> -	pr_debug("doorbell kernel address == %p\n", kfd->doorbell_kernel_ptr);
> +	pr_debug("doorbell kernel address == %px\n", kfd->doorbell_kernel_ptr);
>  
>  	return 0;
>  }
> @@ -214,7 +214,7 @@ inline void write_kernel_doorbell(u32 __iomem *db, u32 value)
>  {
>  	if (db) {
>  		writel(value, db);
> -		pr_debug("Writing %d to doorbell address %p\n", value, db);
> +		pr_debug("Writing %d to doorbell address %px\n", value, db);
>  	}
>  }
>  
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_events.c b/drivers/gpu/drm/amd/amdkfd/kfd_events.c
> index 4890a90f1e44..1b03308faa0e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_events.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_events.c
> @@ -81,7 +81,7 @@ static struct kfd_signal_page *allocate_signal_page(struct kfd_process *p)
>  
>  	page->kernel_address = backing_store;
>  	page->need_to_free_pages = true;
> -	pr_debug("Allocated new event signal page at %p, for process %p\n",
> +	pr_debug("Allocated new event signal page at %px, for process %px\n",
>  			page, p);
>  
>  	return page;
> @@ -202,7 +202,7 @@ static int create_signal_event(struct file *devkfd,
>  	p->signal_event_count++;
>  
>  	ev->user_signal_address = &p->signal_page->user_address[ev->event_id];
> -	pr_debug("Signal event number %zu created with id %d, address %p\n",
> +	pr_debug("Signal event number %zu created with id %d, address %px\n",
>  			p->signal_event_count, ev->event_id,
>  			ev->user_signal_address);
>  
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c b/drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c
> index 69f496485331..b39b5b813dab 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_kernel_queue.c
> @@ -222,7 +222,7 @@ static int acquire_packet_buffer(struct kernel_queue *kq,
>  
>  	pr_debug("rptr: %d\n", rptr);
>  	pr_debug("wptr: %d\n", wptr);
> -	pr_debug("queue_address 0x%p\n", queue_address);
> +	pr_debug("queue_address 0x%px\n", queue_address);
>  
>  	available_size = (rptr + queue_size_dwords - 1 - wptr) %
>  							queue_size_dwords;
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> index 1711ad0642f7..c5c1b2158910 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> @@ -468,7 +468,7 @@ static int kfd_process_init_cwsr_apu(struct kfd_process *p, struct file *filep)
>  		memcpy(qpd->cwsr_kaddr, dev->cwsr_isa, dev->cwsr_isa_size);
>  
>  		qpd->tma_addr = qpd->tba_addr + KFD_CWSR_TMA_OFFSET;
> -		pr_debug("set tba :0x%llx, tma:0x%llx, cwsr_kaddr:%p for pqm.\n",
> +		pr_debug("set tba :0x%llx, tma:0x%llx, cwsr_kaddr:%px for pqm.\n",
>  			qpd->tba_addr, qpd->tma_addr, qpd->cwsr_kaddr);
>  	}
>  
> @@ -499,7 +499,7 @@ static int kfd_process_device_init_cwsr_dgpu(struct kfd_process_device *pdd)
>  	memcpy(qpd->cwsr_kaddr, dev->cwsr_isa, dev->cwsr_isa_size);
>  
>  	qpd->tma_addr = qpd->tba_addr + KFD_CWSR_TMA_OFFSET;
> -	pr_debug("set tba :0x%llx, tma:0x%llx, cwsr_kaddr:%p for pqm.\n",
> +	pr_debug("set tba :0x%llx, tma:0x%llx, cwsr_kaddr:%px for pqm.\n",
>  		 qpd->tba_addr, qpd->tma_addr, qpd->cwsr_kaddr);
>  
>  	return 0;
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_queue.c b/drivers/gpu/drm/amd/amdkfd/kfd_queue.c
> index a5315d4f1c95..56d3e05b53ba 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_queue.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_queue.c
> @@ -36,9 +36,9 @@ void print_queue_properties(struct queue_properties *q)
>  	pr_debug("Queue Address: 0x%llX\n", q->queue_address);
>  	pr_debug("Queue Id: %u\n", q->queue_id);
>  	pr_debug("Queue Process Vmid: %u\n", q->vmid);
> -	pr_debug("Queue Read Pointer: 0x%p\n", q->read_ptr);
> -	pr_debug("Queue Write Pointer: 0x%p\n", q->write_ptr);
> -	pr_debug("Queue Doorbell Pointer: 0x%p\n", q->doorbell_ptr);
> +	pr_debug("Queue Read Pointer: 0x%px\n", q->read_ptr);
> +	pr_debug("Queue Write Pointer: 0x%px\n", q->write_ptr);
> +	pr_debug("Queue Doorbell Pointer: 0x%px\n", q->doorbell_ptr);
>  	pr_debug("Queue Doorbell Offset: %u\n", q->doorbell_off);
>  }
>  
> @@ -53,14 +53,14 @@ void print_queue(struct queue *q)
>  	pr_debug("Queue Address: 0x%llX\n", q->properties.queue_address);
>  	pr_debug("Queue Id: %u\n", q->properties.queue_id);
>  	pr_debug("Queue Process Vmid: %u\n", q->properties.vmid);
> -	pr_debug("Queue Read Pointer: 0x%p\n", q->properties.read_ptr);
> -	pr_debug("Queue Write Pointer: 0x%p\n", q->properties.write_ptr);
> -	pr_debug("Queue Doorbell Pointer: 0x%p\n", q->properties.doorbell_ptr);
> +	pr_debug("Queue Read Pointer: 0x%px\n", q->properties.read_ptr);
> +	pr_debug("Queue Write Pointer: 0x%px\n", q->properties.write_ptr);
> +	pr_debug("Queue Doorbell Pointer: 0x%px\n", q->properties.doorbell_ptr);
>  	pr_debug("Queue Doorbell Offset: %u\n", q->properties.doorbell_off);
> -	pr_debug("Queue MQD Address: 0x%p\n", q->mqd);
> +	pr_debug("Queue MQD Address: 0x%px\n", q->mqd);
>  	pr_debug("Queue MQD Gart: 0x%llX\n", q->gart_mqd_addr);
> -	pr_debug("Queue Process Address: 0x%p\n", q->process);
> -	pr_debug("Queue Device Address: 0x%p\n", q->device);
> +	pr_debug("Queue Process Address: 0x%px\n", q->process);
> +	pr_debug("Queue Device Address: 0x%px\n", q->device);
>  }
>  
>  int init_queue(struct queue **q, const struct queue_properties *properties)
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 4e2f379ce217..c50779aa5daa 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -275,7 +275,7 @@ static void dm_pflip_high_irq(void *interrupt_params)
>  	spin_lock_irqsave(&adev->ddev->event_lock, flags);
>  
>  	if (amdgpu_crtc->pflip_status != AMDGPU_FLIP_SUBMITTED){
> -		DRM_DEBUG_DRIVER("amdgpu_crtc->pflip_status = %d !=AMDGPU_FLIP_SUBMITTED(%d) on crtc:%d[%p] \n",
> +		DRM_DEBUG_DRIVER("amdgpu_crtc->pflip_status = %d !=AMDGPU_FLIP_SUBMITTED(%d) on crtc:%d[%px] \n",
>  						 amdgpu_crtc->pflip_status,
>  						 AMDGPU_FLIP_SUBMITTED,
>  						 amdgpu_crtc->crtc_id,
> @@ -301,7 +301,7 @@ static void dm_pflip_high_irq(void *interrupt_params)
>  	amdgpu_crtc->pflip_status = AMDGPU_FLIP_NONE;
>  	spin_unlock_irqrestore(&adev->ddev->event_lock, flags);
>  
> -	DRM_DEBUG_DRIVER("%s - crtc :%d[%p], pflip_stat:AMDGPU_FLIP_NONE\n",
> +	DRM_DEBUG_DRIVER("%s - crtc :%d[%px], pflip_stat:AMDGPU_FLIP_NONE\n",
>  					__func__, amdgpu_crtc->crtc_id, amdgpu_crtc);
>  
>  	drm_crtc_vblank_put(&amdgpu_crtc->base);
> @@ -463,7 +463,7 @@ static int amdgpu_dm_init(struct amdgpu_device *adev)
>  		DRM_ERROR(
>  		"amdgpu: failed to initialize freesync_module.\n");
>  	} else
> -		DRM_DEBUG_DRIVER("amdgpu: freesync_module init done %p.\n",
> +		DRM_DEBUG_DRIVER("amdgpu: freesync_module init done %px.\n",
>  				adev->dm.freesync_module);
>  
>  	amdgpu_dm_init_color_mod();
> @@ -543,7 +543,7 @@ static int detect_mst_link_for_all_connectors(struct drm_device *dev)
>  		aconnector = to_amdgpu_dm_connector(connector);
>  		if (aconnector->dc_link->type == dc_connection_mst_branch &&
>  		    aconnector->mst_mgr.aux) {
> -			DRM_DEBUG_DRIVER("DM_MST: starting TM on aconnector: %p [id: %d]\n",
> +			DRM_DEBUG_DRIVER("DM_MST: starting TM on aconnector: %px [id: %d]\n",
>  					aconnector, aconnector->base.base.id);
>  
>  			ret = drm_dp_mst_topology_mgr_set_mst(&aconnector->mst_mgr, true);
> @@ -895,7 +895,7 @@ amdgpu_dm_update_connector_after_detect(struct amdgpu_dm_connector *aconnector)
>  		return;
>  	}
>  
> -	DRM_DEBUG_DRIVER("DCHPD: connector_id=%d: Old sink=%p New sink=%p\n",
> +	DRM_DEBUG_DRIVER("DCHPD: connector_id=%d: Old sink=%px New sink=%px\n",
>  		aconnector->connector_id, aconnector->dc_sink, sink);
>  
>  	mutex_lock(&dev->mode_config.mutex);
> @@ -4185,7 +4185,7 @@ static void amdgpu_dm_atomic_commit_tail(struct drm_atomic_state *state)
>  
>  		if (modeset_required(new_crtc_state, dm_new_crtc_state->stream, dm_old_crtc_state->stream)) {
>  
> -			DRM_DEBUG_DRIVER("Atomic commit: SET crtc id %d: [%p]\n", acrtc->crtc_id, acrtc);
> +			DRM_DEBUG_DRIVER("Atomic commit: SET crtc id %d: [%px]\n", acrtc->crtc_id, acrtc);
>  
>  			if (!dm_new_crtc_state->stream) {
>  				/*
> @@ -4215,7 +4215,7 @@ static void amdgpu_dm_atomic_commit_tail(struct drm_atomic_state *state)
>  			acrtc->hw_mode = new_crtc_state->mode;
>  			crtc->hwmode = new_crtc_state->mode;
>  		} else if (modereset_required(new_crtc_state)) {
> -			DRM_DEBUG_DRIVER("Atomic commit: RESET. crtc id %d:[%p]\n", acrtc->crtc_id, acrtc);
> +			DRM_DEBUG_DRIVER("Atomic commit: RESET. crtc id %d:[%px]\n", acrtc->crtc_id, acrtc);
>  
>  			/* i.e. reset mode */
>  			if (dm_old_crtc_state->stream)
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> index 8291d74f26bc..4fc246cd6ec4 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> @@ -302,7 +302,7 @@ dm_dp_create_fake_mst_encoder(struct amdgpu_dm_connector *connector)
>  	struct drm_encoder *enc_master =
>  		connector_funcs->best_encoder(&connector->base);
>  
> -	DRM_DEBUG_KMS("enc master is %p\n", enc_master);
> +	DRM_DEBUG_KMS("enc master is %px\n", enc_master);
>  	amdgpu_encoder = kzalloc(sizeof(*amdgpu_encoder), GFP_KERNEL);
>  	if (!amdgpu_encoder)
>  		return NULL;
> diff --git a/drivers/gpu/drm/armada/armada_gem.c b/drivers/gpu/drm/armada/armada_gem.c
> index a97f509743a5..075840a3735f 100644
> --- a/drivers/gpu/drm/armada/armada_gem.c
> +++ b/drivers/gpu/drm/armada/armada_gem.c
> @@ -50,7 +50,7 @@ void armada_gem_free_object(struct drm_gem_object *obj)
>  	struct armada_gem_object *dobj = drm_to_armada_gem(obj);
>  	struct armada_private *priv = obj->dev->dev_private;
>  
> -	DRM_DEBUG_DRIVER("release obj %p\n", dobj);
> +	DRM_DEBUG_DRIVER("release obj %px\n", dobj);
>  
>  	drm_gem_free_mmap_offset(&dobj->obj);
>  
> @@ -177,7 +177,7 @@ armada_gem_linear_back(struct drm_device *dev, struct armada_gem_object *obj)
>  		obj->mapped = true;
>  	}
>  
> -	DRM_DEBUG_DRIVER("obj %p phys %#llx dev %#llx\n", obj,
> +	DRM_DEBUG_DRIVER("obj %px phys %#llx dev %#llx\n", obj,
>  			 (unsigned long long)obj->phys_addr,
>  			 (unsigned long long)obj->dev_addr);
>  
> @@ -206,7 +206,7 @@ armada_gem_alloc_private_object(struct drm_device *dev, size_t size)
>  
>  	drm_gem_private_object_init(dev, &obj->obj, size);
>  
> -	DRM_DEBUG_DRIVER("alloc private obj %p size %zu\n", obj, size);
> +	DRM_DEBUG_DRIVER("alloc private obj %px size %zu\n", obj, size);
>  
>  	return obj;
>  }
> @@ -231,7 +231,7 @@ static struct armada_gem_object *armada_gem_alloc_object(struct drm_device *dev,
>  	mapping = obj->obj.filp->f_mapping;
>  	mapping_set_gfp_mask(mapping, GFP_HIGHUSER | __GFP_RECLAIMABLE);
>  
> -	DRM_DEBUG_DRIVER("alloc obj %p size %zu\n", obj, size);
> +	DRM_DEBUG_DRIVER("alloc obj %px size %zu\n", obj, size);
>  
>  	return obj;
>  }
> @@ -263,7 +263,7 @@ int armada_gem_dumb_create(struct drm_file *file, struct drm_device *dev,
>  	args->handle = handle;
>  
>  	/* drop reference from allocate - handle holds it now */
> -	DRM_DEBUG_DRIVER("obj %p size %zu handle %#x\n", dobj, size, handle);
> +	DRM_DEBUG_DRIVER("obj %px size %zu handle %#x\n", dobj, size, handle);
>   err:
>  	drm_gem_object_put_unlocked(&dobj->obj);
>  	return ret;
> @@ -295,7 +295,7 @@ int armada_gem_create_ioctl(struct drm_device *dev, void *data,
>  	args->handle = handle;
>  
>  	/* drop reference from allocate - handle holds it now */
> -	DRM_DEBUG_DRIVER("obj %p size %zu handle %#x\n", dobj, size, handle);
> +	DRM_DEBUG_DRIVER("obj %px size %zu handle %#x\n", dobj, size, handle);
>   err:
>  	drm_gem_object_put_unlocked(&dobj->obj);
>  	return ret;
> diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
> index 7d25c42f22db..239b010fc010 100644
> --- a/drivers/gpu/drm/drm_atomic.c
> +++ b/drivers/gpu/drm/drm_atomic.c
> @@ -92,7 +92,7 @@ drm_atomic_state_init(struct drm_device *dev, struct drm_atomic_state *state)
>  
>  	state->dev = dev;
>  
> -	DRM_DEBUG_ATOMIC("Allocated atomic state %p\n", state);
> +	DRM_DEBUG_ATOMIC("Allocated atomic state %px\n", state);
>  
>  	return 0;
>  fail:
> @@ -143,7 +143,7 @@ void drm_atomic_state_default_clear(struct drm_atomic_state *state)
>  	struct drm_mode_config *config = &dev->mode_config;
>  	int i;
>  
> -	DRM_DEBUG_ATOMIC("Clearing atomic state %p\n", state);
> +	DRM_DEBUG_ATOMIC("Clearing atomic state %px\n", state);
>  
>  	for (i = 0; i < state->num_connector; i++) {
>  		struct drm_connector *connector = state->connectors[i].ptr;
> @@ -240,7 +240,7 @@ void __drm_atomic_state_free(struct kref *ref)
>  
>  	drm_atomic_state_clear(state);
>  
> -	DRM_DEBUG_ATOMIC("Freeing atomic state %p\n", state);
> +	DRM_DEBUG_ATOMIC("Freeing atomic state %px\n", state);
>  
>  	if (config->funcs->atomic_state_free) {
>  		config->funcs->atomic_state_free(state);
> @@ -293,7 +293,7 @@ drm_atomic_get_crtc_state(struct drm_atomic_state *state,
>  	state->crtcs[index].ptr = crtc;
>  	crtc_state->state = state;
>  
> -	DRM_DEBUG_ATOMIC("Added [CRTC:%d:%s] %p state to %p\n",
> +	DRM_DEBUG_ATOMIC("Added [CRTC:%d:%s] %px state to %px\n",
>  			 crtc->base.id, crtc->name, crtc_state, state);
>  
>  	return crtc_state;
> @@ -351,12 +351,12 @@ int drm_atomic_set_mode_for_crtc(struct drm_crtc_state *state,
>  
>  		drm_mode_copy(&state->mode, mode);
>  		state->enable = true;
> -		DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %px\n",
>  				 mode->name, state);
>  	} else {
>  		memset(&state->mode, 0, sizeof(state->mode));
>  		state->enable = false;
> -		DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %px\n",
>  				 state);
>  	}
>  
> @@ -396,11 +396,11 @@ int drm_atomic_set_mode_prop_for_crtc(struct drm_crtc_state *state,
>  
>  		state->mode_blob = drm_property_blob_get(blob);
>  		state->enable = true;
> -		DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [MODE:%s] for CRTC state %px\n",
>  				 state->mode.name, state);
>  	} else {
>  		state->enable = false;
> -		DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [NOMODE] for CRTC state %px\n",
>  				 state);
>  	}
>  
> @@ -710,7 +710,7 @@ drm_atomic_get_plane_state(struct drm_atomic_state *state,
>  	state->planes[index].new_state = plane_state;
>  	plane_state->state = state;
>  
> -	DRM_DEBUG_ATOMIC("Added [PLANE:%d:%s] %p state to %p\n",
> +	DRM_DEBUG_ATOMIC("Added [PLANE:%d:%s] %px state to %px\n",
>  			 plane->base.id, plane->name, plane_state, state);
>  
>  	if (plane_state->crtc) {
> @@ -1111,7 +1111,7 @@ drm_atomic_get_private_obj_state(struct drm_atomic_state *state,
>  
>  	state->num_private_objs = num_objs;
>  
> -	DRM_DEBUG_ATOMIC("Added new private object %p state %p to %p\n",
> +	DRM_DEBUG_ATOMIC("Added new private object %px state %px to %px\n",
>  			 obj, obj_state, state);
>  
>  	return obj_state;
> @@ -1178,7 +1178,7 @@ drm_atomic_get_connector_state(struct drm_atomic_state *state,
>  	state->connectors[index].ptr = connector;
>  	connector_state->state = state;
>  
> -	DRM_DEBUG_ATOMIC("Added [CONNECTOR:%d:%s] %p state to %p\n",
> +	DRM_DEBUG_ATOMIC("Added [CONNECTOR:%d:%s] %px state to %px\n",
>  			 connector->base.id, connector->name,
>  			 connector_state, state);
>  
> @@ -1442,10 +1442,10 @@ drm_atomic_set_crtc_for_plane(struct drm_plane_state *plane_state,
>  	}
>  
>  	if (crtc)
> -		DRM_DEBUG_ATOMIC("Link plane state %p to [CRTC:%d:%s]\n",
> +		DRM_DEBUG_ATOMIC("Link plane state %px to [CRTC:%d:%s]\n",
>  				 plane_state, crtc->base.id, crtc->name);
>  	else
> -		DRM_DEBUG_ATOMIC("Link plane state %p to [NOCRTC]\n",
> +		DRM_DEBUG_ATOMIC("Link plane state %px to [NOCRTC]\n",
>  				 plane_state);
>  
>  	return 0;
> @@ -1467,10 +1467,10 @@ drm_atomic_set_fb_for_plane(struct drm_plane_state *plane_state,
>  			    struct drm_framebuffer *fb)
>  {
>  	if (fb)
> -		DRM_DEBUG_ATOMIC("Set [FB:%d] for plane state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [FB:%d] for plane state %px\n",
>  				 fb->base.id, plane_state);
>  	else
> -		DRM_DEBUG_ATOMIC("Set [NOFB] for plane state %p\n",
> +		DRM_DEBUG_ATOMIC("Set [NOFB] for plane state %px\n",
>  				 plane_state);
>  
>  	drm_framebuffer_assign(&plane_state->fb, fb);
> @@ -1551,10 +1551,10 @@ drm_atomic_set_crtc_for_connector(struct drm_connector_state *conn_state,
>  		drm_connector_get(conn_state->connector);
>  		conn_state->crtc = crtc;
>  
> -		DRM_DEBUG_ATOMIC("Link connector state %p to [CRTC:%d:%s]\n",
> +		DRM_DEBUG_ATOMIC("Link connector state %px to [CRTC:%d:%s]\n",
>  				 conn_state, crtc->base.id, crtc->name);
>  	} else {
> -		DRM_DEBUG_ATOMIC("Link connector state %p to [NOCRTC]\n",
> +		DRM_DEBUG_ATOMIC("Link connector state %px to [NOCRTC]\n",
>  				 conn_state);
>  	}
>  
> @@ -1598,7 +1598,7 @@ drm_atomic_add_affected_connectors(struct drm_atomic_state *state,
>  	if (ret)
>  		return ret;
>  
> -	DRM_DEBUG_ATOMIC("Adding all current connectors for [CRTC:%d:%s] to %p\n",
> +	DRM_DEBUG_ATOMIC("Adding all current connectors for [CRTC:%d:%s] to %px\n",
>  			 crtc->base.id, crtc->name, state);
>  
>  	/*
> @@ -1682,7 +1682,7 @@ int drm_atomic_check_only(struct drm_atomic_state *state)
>  	struct drm_crtc_state *crtc_state;
>  	int i, ret = 0;
>  
> -	DRM_DEBUG_ATOMIC("checking %p\n", state);
> +	DRM_DEBUG_ATOMIC("checking %px\n", state);
>  
>  	for_each_new_plane_in_state(state, plane, plane_state, i) {
>  		ret = drm_atomic_plane_check(plane, plane_state);
> @@ -1745,7 +1745,7 @@ int drm_atomic_commit(struct drm_atomic_state *state)
>  	if (ret)
>  		return ret;
>  
> -	DRM_DEBUG_ATOMIC("committing %p\n", state);
> +	DRM_DEBUG_ATOMIC("committing %px\n", state);
>  
>  	return config->funcs->atomic_commit(state->dev, state, false);
>  }
> @@ -1774,7 +1774,7 @@ int drm_atomic_nonblocking_commit(struct drm_atomic_state *state)
>  	if (ret)
>  		return ret;
>  
> -	DRM_DEBUG_ATOMIC("committing %p nonblocking\n", state);
> +	DRM_DEBUG_ATOMIC("committing %px nonblocking\n", state);
>  
>  	return config->funcs->atomic_commit(state->dev, state, true);
>  }
> @@ -1791,7 +1791,7 @@ static void drm_atomic_print_state(const struct drm_atomic_state *state)
>  	struct drm_connector_state *connector_state;
>  	int i;
>  
> -	DRM_DEBUG_ATOMIC("checking %p\n", state);
> +	DRM_DEBUG_ATOMIC("checking %px\n", state);
>  
>  	for_each_new_plane_in_state(state, plane, plane_state, i)
>  		drm_atomic_plane_print_state(&p, plane_state);
> diff --git a/drivers/gpu/drm/drm_bufs.c b/drivers/gpu/drm/drm_bufs.c
> index ba8cfe65c65b..f56a0893ef1b 100644
> --- a/drivers/gpu/drm/drm_bufs.c
> +++ b/drivers/gpu/drm/drm_bufs.c
> @@ -236,7 +236,7 @@ static int drm_addmap_core(struct drm_device *dev, resource_size_t offset,
>  			return 0;
>  		}
>  		map->handle = vmalloc_user(map->size);
> -		DRM_DEBUG("%lu %d %p\n",
> +		DRM_DEBUG("%lu %d %px\n",
>  			  map->size, order_base_2(map->size), map->handle);
>  		if (!map->handle) {
>  			kfree(map);
> @@ -793,7 +793,7 @@ int drm_legacy_addbufs_agp(struct drm_device *dev,
>  			return -ENOMEM;
>  		}
>  
> -		DRM_DEBUG("buffer %d @ %p\n", entry->buf_count, buf->address);
> +		DRM_DEBUG("buffer %d @ %px\n", entry->buf_count, buf->address);
>  
>  		offset += alignment;
>  		entry->buf_count++;
> @@ -995,7 +995,7 @@ int drm_legacy_addbufs_pci(struct drm_device *dev,
>  				return -ENOMEM;
>  			}
>  
> -			DRM_DEBUG("buffer %d @ %p\n",
> +			DRM_DEBUG("buffer %d @ %px\n",
>  				  entry->buf_count, buf->address);
>  		}
>  		byte_count += PAGE_SIZE << page_order;
> @@ -1156,7 +1156,7 @@ static int drm_legacy_addbufs_sg(struct drm_device *dev,
>  			return -ENOMEM;
>  		}
>  
> -		DRM_DEBUG("buffer %d @ %p\n", entry->buf_count, buf->address);
> +		DRM_DEBUG("buffer %d @ %px\n", entry->buf_count, buf->address);
>  
>  		offset += alignment;
>  		entry->buf_count++;
> diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c
> index 6fac4129e6a2..8ca8d418ca37 100644
> --- a/drivers/gpu/drm/drm_dp_mst_topology.c
> +++ b/drivers/gpu/drm/drm_dp_mst_topology.c
> @@ -814,7 +814,7 @@ static int drm_dp_mst_wait_tx_reply(struct drm_dp_mst_branch *mstb,
>  			goto out;
>  		}
>  	} else {
> -		DRM_DEBUG_KMS("timedout msg send %p %d %d\n", txmsg, txmsg->state, txmsg->seqno);
> +		DRM_DEBUG_KMS("timedout msg send %px %d %d\n", txmsg, txmsg->state, txmsg->seqno);
>  
>  		/* dump some state */
>  		ret = -EIO;
> @@ -2335,7 +2335,7 @@ static int drm_dp_mst_handle_down_rep(struct drm_dp_mst_topology_mgr *mgr)
>  		mutex_unlock(&mgr->qlock);
>  
>  		if (!txmsg) {
> -			DRM_DEBUG_KMS("Got MST reply with no msg %p %d %d %02x %02x\n",
> +			DRM_DEBUG_KMS("Got MST reply with no msg %px %d %d %02x %02x\n",
>  			       mstb,
>  			       mgr->down_rep_recv.initial_hdr.seqno,
>  			       mgr->down_rep_recv.initial_hdr.lct,
> diff --git a/drivers/gpu/drm/drm_lease.c b/drivers/gpu/drm/drm_lease.c
> index d345563fdff3..47baec81a817 100644
> --- a/drivers/gpu/drm/drm_lease.c
> +++ b/drivers/gpu/drm/drm_lease.c
> @@ -248,7 +248,7 @@ static struct drm_master *drm_lease_create(struct drm_master *lessor, struct idr
>  
>  	/* Move the leases over */
>  	lessee->leases = *leases;
> -	DRM_DEBUG_LEASE("new lessee %d %p, lessor %d %p\n", lessee->lessee_id, lessee, lessor->lessee_id, lessor);
> +	DRM_DEBUG_LEASE("new lessee %d %px, lessor %d %px\n", lessee->lessee_id, lessee, lessor->lessee_id, lessor);
>  
>  	mutex_unlock(&dev->mode_config.idr_mutex);
>  	return lessee;
> @@ -320,7 +320,7 @@ static void _drm_lease_revoke(struct drm_master *top)
>  	 * the tree is fully connected, we can do this without recursing
>  	 */
>  	for (;;) {
> -		DRM_DEBUG_LEASE("revoke leases for %p %d\n", master, master->lessee_id);
> +		DRM_DEBUG_LEASE("revoke leases for %px %d\n", master, master->lessee_id);
>  
>  		/* Evacuate the lease */
>  		idr_for_each_entry(&master->leases, entry, object)
> @@ -564,7 +564,7 @@ int drm_mode_create_lease_ioctl(struct drm_device *dev,
>  	}
>  
>  	/* Initialize the new file for DRM */
> -	DRM_DEBUG_LEASE("Initializing the file with %p\n", lessee_file->f_op->open);
> +	DRM_DEBUG_LEASE("Initializing the file with %px\n", lessee_file->f_op->open);
>  	ret = lessee_file->f_op->open(lessee_file->f_inode, lessee_file);
>  	if (ret)
>  		goto out_lessee_file;
> diff --git a/drivers/gpu/drm/drm_lock.c b/drivers/gpu/drm/drm_lock.c
> index 96bb6badb818..550b462c9526 100644
> --- a/drivers/gpu/drm/drm_lock.c
> +++ b/drivers/gpu/drm/drm_lock.c
> @@ -341,7 +341,7 @@ void drm_legacy_lock_release(struct drm_device *dev, struct file *filp)
>  		return;
>  
>  	if (drm_legacy_i_have_hw_lock(dev, file_priv)) {
> -		DRM_DEBUG("File %p released, freeing lock for context %d\n",
> +		DRM_DEBUG("File %px released, freeing lock for context %d\n",
>  			  filp, _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
>  		drm_legacy_lock_free(&file_priv->master->lock,
>  				     _DRM_LOCKING_CONTEXT(file_priv->master->lock.hw_lock->lock));
> diff --git a/drivers/gpu/drm/drm_scatter.c b/drivers/gpu/drm/drm_scatter.c
> index 275bca44f38c..06340c07ec12 100644
> --- a/drivers/gpu/drm/drm_scatter.c
> +++ b/drivers/gpu/drm/drm_scatter.c
> @@ -134,7 +134,7 @@ int drm_legacy_sg_alloc(struct drm_device *dev, void *data,
>  	entry->handle = ScatterHandle((unsigned long)entry->virtual);
>  
>  	DRM_DEBUG("handle  = %08lx\n", entry->handle);
> -	DRM_DEBUG("virtual = %p\n", entry->virtual);
> +	DRM_DEBUG("virtual = %px\n", entry->virtual);
>  
>  	for (i = (unsigned long)entry->virtual, j = 0; j < pages;
>  	     i += PAGE_SIZE, j++) {
> @@ -213,7 +213,7 @@ int drm_legacy_sg_free(struct drm_device *dev, void *data,
>  	if (!entry || entry->handle != request->handle)
>  		return -EINVAL;
>  
> -	DRM_DEBUG("virtual  = %p\n", entry->virtual);
> +	DRM_DEBUG("virtual  = %px\n", entry->virtual);
>  
>  	drm_sg_cleanup(entry);
>  
> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_drv.c b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> index ab50090d066c..09065596463b 100644
> --- a/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> +++ b/drivers/gpu/drm/etnaviv/etnaviv_drv.c
> @@ -52,7 +52,7 @@ void __iomem *etnaviv_ioremap(struct platform_device *pdev, const char *name,
>  	}
>  
>  	if (reglog)
> -		dev_printk(KERN_DEBUG, &pdev->dev, "IO:region %s 0x%p %08zx\n",
> +		dev_printk(KERN_DEBUG, &pdev->dev, "IO:region %s 0x%px %08zx\n",
>  			   dbgname, ptr, (size_t)resource_size(res));
>  
>  	return ptr;
> @@ -61,7 +61,7 @@ void __iomem *etnaviv_ioremap(struct platform_device *pdev, const char *name,
>  void etnaviv_writel(u32 data, void __iomem *addr)
>  {
>  	if (reglog)
> -		printk(KERN_DEBUG "IO:W %p %08x\n", addr, data);
> +		printk(KERN_DEBUG "IO:W %px %08x\n", addr, data);
>  
>  	writel(data, addr);
>  }
> @@ -71,7 +71,7 @@ u32 etnaviv_readl(const void __iomem *addr)
>  	u32 val = readl(addr);
>  
>  	if (reglog)
> -		printk(KERN_DEBUG "IO:R %p %08x\n", addr, val);
> +		printk(KERN_DEBUG "IO:R %px %08x\n", addr, val);
>  
>  	return val;
>  }
> diff --git a/drivers/gpu/drm/i810/i810_dma.c b/drivers/gpu/drm/i810/i810_dma.c
> index 576a417690d4..00b872fd27b0 100644
> --- a/drivers/gpu/drm/i810/i810_dma.c
> +++ b/drivers/gpu/drm/i810/i810_dma.c
> @@ -399,7 +399,7 @@ static int i810_dma_initialize(struct drm_device *dev,
>  		DRM_ERROR("Can not allocate hardware status page\n");
>  		return -ENOMEM;
>  	}
> -	DRM_DEBUG("hw status page @ %p\n", dev_priv->hw_status_page);
> +	DRM_DEBUG("hw status page @ %px\n", dev_priv->hw_status_page);
>  
>  	I810_WRITE(0x02080, dev_priv->dma_status_page);
>  	DRM_DEBUG("Enabled hardware status page\n");
> diff --git a/drivers/gpu/drm/i915/i915_perf.c b/drivers/gpu/drm/i915/i915_perf.c
> index abaca6edeb71..a90ca24b03ea 100644
> --- a/drivers/gpu/drm/i915/i915_perf.c
> +++ b/drivers/gpu/drm/i915/i915_perf.c
> @@ -1473,7 +1473,7 @@ static int alloc_oa_buffer(struct drm_i915_private *dev_priv)
>  
>  	dev_priv->perf.oa.ops.init_oa_buffer(dev_priv);
>  
> -	DRM_DEBUG_DRIVER("OA Buffer initialized, gtt offset = 0x%x, vaddr = %p\n",
> +	DRM_DEBUG_DRIVER("OA Buffer initialized, gtt offset = 0x%x, vaddr = %px\n",
>  			 i915_ggtt_offset(dev_priv->perf.oa.oa_buffer.vma),
>  			 dev_priv->perf.oa.oa_buffer.vaddr);
>  
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 3b48fd2561fe..9759257ac614 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -2732,7 +2732,7 @@ intel_alloc_initial_plane_obj(struct intel_crtc *crtc,
>  	}
>  
>  
> -	DRM_DEBUG_KMS("initial plane fb obj %p\n", obj);
> +	DRM_DEBUG_KMS("initial plane fb obj %px\n", obj);
>  	return true;
>  
>  out_unref_obj:
> diff --git a/drivers/gpu/drm/i915/intel_guc_ct.c b/drivers/gpu/drm/i915/intel_guc_ct.c
> index 24ad55752396..3c44c1946e66 100644
> --- a/drivers/gpu/drm/i915/intel_guc_ct.c
> +++ b/drivers/gpu/drm/i915/intel_guc_ct.c
> @@ -49,7 +49,7 @@ static inline const char *guc_ct_buffer_type_to_str(u32 type)
>  static void guc_ct_buffer_desc_init(struct guc_ct_buffer_desc *desc,
>  				    u32 cmds_addr, u32 size, u32 owner)
>  {
> -	DRM_DEBUG_DRIVER("CT: desc %p init addr=%#x size=%u owner=%u\n",
> +	DRM_DEBUG_DRIVER("CT: desc %px init addr=%#x size=%u owner=%u\n",
>  			 desc, cmds_addr, size, owner);
>  	memset(desc, 0, sizeof(*desc));
>  	desc->addr = cmds_addr;
> @@ -59,7 +59,7 @@ static void guc_ct_buffer_desc_init(struct guc_ct_buffer_desc *desc,
>  
>  static void guc_ct_buffer_desc_reset(struct guc_ct_buffer_desc *desc)
>  {
> -	DRM_DEBUG_DRIVER("CT: desc %p reset head=%u tail=%u\n",
> +	DRM_DEBUG_DRIVER("CT: desc %px reset head=%u tail=%u\n",
>  			 desc, desc->head, desc->tail);
>  	desc->head = 0;
>  	desc->tail = 0;
> diff --git a/drivers/gpu/drm/i915/intel_guc_submission.c b/drivers/gpu/drm/i915/intel_guc_submission.c
> index 8a8ad2fe158d..10af95e91ca1 100644
> --- a/drivers/gpu/drm/i915/intel_guc_submission.c
> +++ b/drivers/gpu/drm/i915/intel_guc_submission.c
> @@ -928,7 +928,7 @@ guc_client_alloc(struct drm_i915_private *dev_priv,
>  	if (ret)
>  		goto err_vaddr;
>  
> -	DRM_DEBUG_DRIVER("new priority %u client %p for engine(s) 0x%x: stage_id %u\n",
> +	DRM_DEBUG_DRIVER("new priority %u client %px for engine(s) 0x%x: stage_id %u\n",
>  			 priority, client, client->engines, client->stage_id);
>  	DRM_DEBUG_DRIVER("doorbell id %u, cacheline offset 0x%lx\n",
>  			 client->doorbell_id, client->doorbell_offset);
> diff --git a/drivers/gpu/drm/i915/intel_uc_fw.c b/drivers/gpu/drm/i915/intel_uc_fw.c
> index 3ec0ce505b76..4c968078c61c 100644
> --- a/drivers/gpu/drm/i915/intel_uc_fw.c
> +++ b/drivers/gpu/drm/i915/intel_uc_fw.c
> @@ -64,7 +64,7 @@ void intel_uc_fw_fetch(struct drm_i915_private *dev_priv,
>  		goto fail;
>  	}
>  
> -	DRM_DEBUG_DRIVER("%s fw size %zu ptr %p\n",
> +	DRM_DEBUG_DRIVER("%s fw size %zu ptr %px\n",
>  			 intel_uc_fw_type_repr(uc_fw->type), fw->size, fw);
>  
>  	/* Check the size of the blob before examining buffer contents */
> diff --git a/drivers/gpu/drm/mediatek/mtk_drm_gem.c b/drivers/gpu/drm/mediatek/mtk_drm_gem.c
> index f595ac816b55..ccdec81de9e3 100644
> --- a/drivers/gpu/drm/mediatek/mtk_drm_gem.c
> +++ b/drivers/gpu/drm/mediatek/mtk_drm_gem.c
> @@ -71,7 +71,7 @@ struct mtk_drm_gem_obj *mtk_drm_gem_create(struct drm_device *dev,
>  	if (alloc_kmap)
>  		mtk_gem->kvaddr = mtk_gem->cookie;
>  
> -	DRM_DEBUG_DRIVER("cookie = %p dma_addr = %pad size = %zu\n",
> +	DRM_DEBUG_DRIVER("cookie = %px dma_addr = %pad size = %zu\n",
>  			 mtk_gem->cookie, &mtk_gem->dma_addr,
>  			 size);
>  
> diff --git a/drivers/gpu/drm/mga/mga_warp.c b/drivers/gpu/drm/mga/mga_warp.c
> index 0b76352260a9..5b286f04f805 100644
> --- a/drivers/gpu/drm/mga/mga_warp.c
> +++ b/drivers/gpu/drm/mga/mga_warp.c
> @@ -116,7 +116,7 @@ int mga_warp_install_microcode(drm_mga_private_t *dev_priv)
>  	     rec = ihex_next_binrec(rec)) {
>  		unsigned int src_size, dst_size;
>  
> -		DRM_DEBUG(" pcbase = 0x%08lx  vcbase = %p\n", pcbase, vcbase);
> +		DRM_DEBUG(" pcbase = 0x%08lx  vcbase = %px\n", pcbase, vcbase);
>  		dev_priv->warp_pipe_phys[where] = pcbase;
>  		src_size = be16_to_cpu(rec->len);
>  		dst_size = WARP_UCODE_SIZE(src_size);
> diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c
> index 30cd514d8f7c..078e9c3183e8 100644
> --- a/drivers/gpu/drm/msm/msm_drv.c
> +++ b/drivers/gpu/drm/msm/msm_drv.c
> @@ -122,7 +122,7 @@ void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>  	}
>  
>  	if (reglog)
> -		printk(KERN_DEBUG "IO:region %s %p %08lx\n", dbgname, ptr, size);
> +		printk(KERN_DEBUG "IO:region %s %px %08lx\n", dbgname, ptr, size);
>  
>  	return ptr;
>  }
> @@ -130,7 +130,7 @@ void __iomem *msm_ioremap(struct platform_device *pdev, const char *name,
>  void msm_writel(u32 data, void __iomem *addr)
>  {
>  	if (reglog)
> -		printk(KERN_DEBUG "IO:W %p %08x\n", addr, data);
> +		printk(KERN_DEBUG "IO:W %px %08x\n", addr, data);
>  	writel(data, addr);
>  }
>  
> diff --git a/drivers/gpu/drm/qxl/qxl_cmd.c b/drivers/gpu/drm/qxl/qxl_cmd.c
> index c0fb52c6d4ca..f6c64c5cb99f 100644
> --- a/drivers/gpu/drm/qxl/qxl_cmd.c
> +++ b/drivers/gpu/drm/qxl/qxl_cmd.c
> @@ -381,7 +381,7 @@ void qxl_io_create_primary(struct qxl_device *qdev,
>  {
>  	struct qxl_surface_create *create;
>  
> -	DRM_DEBUG_DRIVER("qdev %p, ram_header %p\n", qdev, qdev->ram_header);
> +	DRM_DEBUG_DRIVER("qdev %px, ram_header %px\n", qdev, qdev->ram_header);
>  	create = &qdev->ram_header->create_surface;
>  	create->format = bo->surf.format;
>  	create->width = bo->surf.width;
> @@ -393,7 +393,7 @@ void qxl_io_create_primary(struct qxl_device *qdev,
>  		create->mem = qxl_bo_physical_address(qdev, bo, offset);
>  	}
>  
> -	DRM_DEBUG_DRIVER("mem = %llx, from %p\n", create->mem, bo->kptr);
> +	DRM_DEBUG_DRIVER("mem = %llx, from %px\n", create->mem, bo->kptr);
>  
>  	create->flags = QXL_SURF_FLAG_KEEP_DATA;
>  	create->type = QXL_SURF_TYPE_PRIMARY;
> diff --git a/drivers/gpu/drm/qxl/qxl_fb.c b/drivers/gpu/drm/qxl/qxl_fb.c
> index 338891401f35..ce506e702aed 100644
> --- a/drivers/gpu/drm/qxl/qxl_fb.c
> +++ b/drivers/gpu/drm/qxl/qxl_fb.c
> @@ -246,7 +246,7 @@ static int qxlfb_create(struct qxl_fbdev *qfbdev,
>  	shadow = vmalloc(mode_cmd.pitches[0] * mode_cmd.height);
>  	/* TODO: what's the usual response to memory allocation errors? */
>  	BUG_ON(!shadow);
> -	DRM_DEBUG_DRIVER("surface0 at gpu offset %lld, mmap_offset %lld (virt %p, shadow %p)\n",
> +	DRM_DEBUG_DRIVER("surface0 at gpu offset %lld, mmap_offset %lld (virt %px, shadow %px)\n",
>  			 qxl_bo_gpu_offset(qbo), qxl_bo_mmap_offset(qbo),
>  			 qbo->kptr, shadow);
>  	size = mode_cmd.pitches[0] * mode_cmd.height;
> diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c
> index ee2340e31f06..bbbf9433d32a 100644
> --- a/drivers/gpu/drm/qxl/qxl_ttm.c
> +++ b/drivers/gpu/drm/qxl/qxl_ttm.c
> @@ -133,7 +133,7 @@ int qxl_mmap(struct file *filp, struct vm_area_struct *vma)
>  		 "filp->private_data->minor->dev->dev_private == NULL\n");
>  		return -EINVAL;
>  	}
> -	DRM_DEBUG_DRIVER("filp->private_data = 0x%p, vma->vm_pgoff = %lx\n",
> +	DRM_DEBUG_DRIVER("filp->private_data = 0x%px, vma->vm_pgoff = %lx\n",
>  		  filp->private_data, vma->vm_pgoff);
>  
>  	r = ttm_bo_mmap(filp, vma, &qdev->mman.bdev);
> diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c
> index 26129b2b082d..9f38090e89ef 100644
> --- a/drivers/gpu/drm/radeon/radeon_display.c
> +++ b/drivers/gpu/drm/radeon/radeon_display.c
> @@ -513,7 +513,7 @@ static int radeon_crtc_page_flip_target(struct drm_crtc *crtc,
>  	new_rbo = gem_to_radeon_bo(obj);
>  
>  	/* pin the new buffer */
> -	DRM_DEBUG_DRIVER("flip-ioctl() cur_rbo = %p, new_rbo = %p\n",
> +	DRM_DEBUG_DRIVER("flip-ioctl() cur_rbo = %px, new_rbo = %px\n",
>  			 work->old_rbo, new_rbo);
>  
>  	r = radeon_bo_reserve(new_rbo, false);
> diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> index cd8a3ee16649..44016392115e 100644
> --- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
> +++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
> @@ -193,7 +193,7 @@ static int radeon_dp_mst_get_ddc_modes(struct drm_connector *connector)
>  
>  	edid = drm_dp_mst_get_edid(connector, &master->mst_mgr, radeon_connector->port);
>  	radeon_connector->edid = edid;
> -	DRM_DEBUG_KMS("edid retrieved %p\n", edid);
> +	DRM_DEBUG_KMS("edid retrieved %px\n", edid);
>  	if (radeon_connector->edid) {
>  		drm_mode_connector_update_edid_property(&radeon_connector->base, radeon_connector->edid);
>  		ret = drm_add_edid_modes(&radeon_connector->base, radeon_connector->edid);
> @@ -348,7 +348,7 @@ radeon_connector *radeon_mst_find_connector(struct drm_encoder *encoder)
>  		if (!radeon_connector->is_mst_connector)
>  			continue;
>  
> -		DRM_DEBUG_KMS("checking %p vs %p\n", connector->encoder, encoder);
> +		DRM_DEBUG_KMS("checking %px vs %px\n", connector->encoder, encoder);
>  		if (connector->encoder == encoder)
>  			return radeon_connector;
>  	}
> @@ -374,7 +374,7 @@ void radeon_dp_mst_prepare_pll(struct drm_crtc *crtc, struct drm_display_mode *m
>  			radeon_crtc->bpc = 8;
>  	}
>  
> -	DRM_DEBUG_KMS("dp_clock %p %d\n", dig_connector, dig_connector->dp_clock);
> +	DRM_DEBUG_KMS("dp_clock %px %d\n", dig_connector, dig_connector->dp_clock);
>  	dp_clock = dig_connector->dp_clock;
>  	radeon_crtc->ss_enabled =
>  		radeon_atombios_get_asic_ss_info(rdev, &radeon_crtc->ss,
> @@ -533,7 +533,7 @@ static bool radeon_mst_mode_fixup(struct drm_encoder *encoder,
>  	dig_connector = mst_enc->connector->con_priv;
>  	dig_connector->dp_lane_count = drm_dp_max_lane_count(dig_connector->dpcd);
>  	dig_connector->dp_clock = drm_dp_max_link_rate(dig_connector->dpcd);
> -	DRM_DEBUG_KMS("dig clock %p %d %d\n", dig_connector,
> +	DRM_DEBUG_KMS("dig clock %px %d %d\n", dig_connector,
>  		      dig_connector->dp_lane_count, dig_connector->dp_clock);
>  	return true;
>  }
> @@ -547,7 +547,7 @@ static void radeon_mst_encoder_prepare(struct drm_encoder *encoder)
>  
>  	radeon_connector = radeon_mst_find_connector(encoder);
>  	if (!radeon_connector) {
> -		DRM_DEBUG_KMS("failed to find connector %p\n", encoder);
> +		DRM_DEBUG_KMS("failed to find connector %px\n", encoder);
>  		return;
>  	}
>  	radeon_encoder = to_radeon_encoder(encoder);
> @@ -615,7 +615,7 @@ radeon_dp_create_fake_mst_encoder(struct radeon_connector *connector)
>  	const struct drm_connector_helper_funcs *connector_funcs = connector->base.helper_private;
>  	struct drm_encoder *enc_master = connector_funcs->best_encoder(&connector->base);
>  
> -	DRM_DEBUG_KMS("enc master is %p\n", enc_master);
> +	DRM_DEBUG_KMS("enc master is %px\n", enc_master);
>  	radeon_encoder = kzalloc(sizeof(*radeon_encoder), GFP_KERNEL);
>  	if (!radeon_encoder)
>  		return NULL;
> diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
> index edbb4cd519fd..24debb2760c3 100644
> --- a/drivers/gpu/drm/radeon/radeon_object.c
> +++ b/drivers/gpu/drm/radeon/radeon_object.c
> @@ -639,7 +639,7 @@ int radeon_bo_get_surface_reg(struct radeon_bo *bo)
>  		reg = &rdev->surface_regs[steal];
>  		old_object = reg->bo;
>  		/* blow away the mapping */
> -		DRM_DEBUG("stealing surface reg %d from %p\n", steal, old_object);
> +		DRM_DEBUG("stealing surface reg %d from %px\n", steal, old_object);
>  		ttm_bo_unmap_virtual(&old_object->tbo);
>  		old_object->surface_reg = -1;
>  		i = steal;
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> index e6650553f5d6..d90a372a6413 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c
> @@ -106,7 +106,7 @@ static int rockchip_drm_fbdev_create(struct drm_fb_helper *helper,
>  	fbi->screen_size = rk_obj->base.size;
>  	fbi->fix.smem_len = rk_obj->base.size;
>  
> -	DRM_DEBUG_KMS("FB [%dx%d]-%d kvaddr=%p offset=%ld size=%zu\n",
> +	DRM_DEBUG_KMS("FB [%dx%d]-%d kvaddr=%px offset=%ld size=%zu\n",
>  		      fb->width, fb->height, fb->format->depth,
>  		      rk_obj->kvaddr,
>  		      offset, size);
> diff --git a/drivers/gpu/drm/savage/savage_bci.c b/drivers/gpu/drm/savage/savage_bci.c
> index 2a5b8466d806..68ac86b83d1d 100644
> --- a/drivers/gpu/drm/savage/savage_bci.c
> +++ b/drivers/gpu/drm/savage/savage_bci.c
> @@ -265,7 +265,7 @@ static struct drm_buf *savage_freelist_get(struct drm_device * dev)
>  		return tail->buf;
>  	}
>  
> -	DRM_DEBUG("returning NULL, tail->buf=%p!\n", tail->buf);
> +	DRM_DEBUG("returning NULL, tail->buf=%px!\n", tail->buf);
>  	return NULL;
>  }
>  
> diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c
> index 9b2c47051b51..385841c46838 100644
> --- a/drivers/gpu/drm/sti/sti_gdp.c
> +++ b/drivers/gpu/drm/sti/sti_gdp.c
> @@ -529,7 +529,7 @@ static void sti_gdp_init(struct sti_gdp *gdp)
>  		gdp->node_list[i].top_field = base;
>  		gdp->node_list[i].top_field_paddr = dma_addr;
>  
> -		DRM_DEBUG_DRIVER("node[%d].top_field=%p\n", i, base);
> +		DRM_DEBUG_DRIVER("node[%d].top_field=%px\n", i, base);
>  		base += sizeof(struct sti_gdp_node);
>  		dma_addr += sizeof(struct sti_gdp_node);
>  
> @@ -539,7 +539,7 @@ static void sti_gdp_init(struct sti_gdp *gdp)
>  		}
>  		gdp->node_list[i].btm_field = base;
>  		gdp->node_list[i].btm_field_paddr = dma_addr;
> -		DRM_DEBUG_DRIVER("node[%d].btm_field=%p\n", i, base);
> +		DRM_DEBUG_DRIVER("node[%d].btm_field=%px\n", i, base);
>  		base += sizeof(struct sti_gdp_node);
>  		dma_addr += sizeof(struct sti_gdp_node);
>  	}
> diff --git a/drivers/gpu/drm/sti/sti_mixer.c b/drivers/gpu/drm/sti/sti_mixer.c
> index a4f45c74d678..6ea5f81dd9db 100644
> --- a/drivers/gpu/drm/sti/sti_mixer.c
> +++ b/drivers/gpu/drm/sti/sti_mixer.c
> @@ -367,7 +367,7 @@ struct sti_mixer *sti_mixer_create(struct device *dev,
>  	mixer->dev = dev;
>  	mixer->id = id;
>  
> -	DRM_DEBUG_DRIVER("%s created. Regs=%p\n",
> +	DRM_DEBUG_DRIVER("%s created. Regs=%px\n",
>  			 sti_mixer_to_str(mixer), mixer->regs);
>  
>  	return mixer;
> diff --git a/drivers/gpu/drm/sun4i/sun4i_crtc.c b/drivers/gpu/drm/sun4i/sun4i_crtc.c
> index 2d7c57406715..1d1994409809 100644
> --- a/drivers/gpu/drm/sun4i/sun4i_crtc.c
> +++ b/drivers/gpu/drm/sun4i/sun4i_crtc.c
> @@ -159,7 +159,7 @@ static int sun4i_crtc_enable_vblank(struct drm_crtc *crtc)
>  {
>  	struct sun4i_crtc *scrtc = drm_crtc_to_sun4i_crtc(crtc);
>  
> -	DRM_DEBUG_DRIVER("Enabling VBLANK on crtc %p\n", crtc);
> +	DRM_DEBUG_DRIVER("Enabling VBLANK on crtc %px\n", crtc);
>  
>  	sun4i_tcon_enable_vblank(scrtc->tcon, true);
>  
> @@ -170,7 +170,7 @@ static void sun4i_crtc_disable_vblank(struct drm_crtc *crtc)
>  {
>  	struct sun4i_crtc *scrtc = drm_crtc_to_sun4i_crtc(crtc);
>  
> -	DRM_DEBUG_DRIVER("Disabling VBLANK on crtc %p\n", crtc);
> +	DRM_DEBUG_DRIVER("Disabling VBLANK on crtc %px\n", crtc);
>  
>  	sun4i_tcon_enable_vblank(scrtc->tcon, false);
>  }
> diff --git a/drivers/gpu/drm/ttm/ttm_page_alloc.c b/drivers/gpu/drm/ttm/ttm_page_alloc.c
> index f0481b7b60c5..32b3c1b42115 100644
> --- a/drivers/gpu/drm/ttm/ttm_page_alloc.c
> +++ b/drivers/gpu/drm/ttm/ttm_page_alloc.c
> @@ -658,7 +658,7 @@ static void ttm_page_pool_fill_locked(struct ttm_page_pool *pool, int ttm_flags,
>  			++pool->nrefills;
>  			pool->npages += alloc_size;
>  		} else {
> -			pr_debug("Failed to fill pool (%p)\n", pool);
> +			pr_debug("Failed to fill pool (%px)\n", pool);
>  			/* If we have any pages left put them to the pool. */
>  			list_for_each_entry(p, &new_pages, lru) {
>  				++cpages;
> diff --git a/drivers/gpu/drm/udl/udl_fb.c b/drivers/gpu/drm/udl/udl_fb.c
> index 2ebdc6d5a76e..7032206c103f 100644
> --- a/drivers/gpu/drm/udl/udl_fb.c
> +++ b/drivers/gpu/drm/udl/udl_fb.c
> @@ -406,7 +406,7 @@ static int udlfb_create(struct drm_fb_helper *helper,
>  	drm_fb_helper_fill_fix(info, fb->pitches[0], fb->format->depth);
>  	drm_fb_helper_fill_var(info, &ufbdev->helper, sizes->fb_width, sizes->fb_height);
>  
> -	DRM_DEBUG_KMS("allocated %dx%d vmal %p\n",
> +	DRM_DEBUG_KMS("allocated %dx%d vmal %px\n",
>  		      fb->width, fb->height,
>  		      ufbdev->ufb.obj->vmapping);
>  
> diff --git a/drivers/gpu/drm/via/via_dma.c b/drivers/gpu/drm/via/via_dma.c
> index d17d8f245c1a..5d5e322b7c48 100644
> --- a/drivers/gpu/drm/via/via_dma.c
> +++ b/drivers/gpu/drm/via/via_dma.c
> @@ -332,7 +332,7 @@ static int via_cmdbuffer(struct drm_device *dev, void *data, struct drm_file *fi
>  
>  	LOCK_TEST_WITH_RETURN(dev, file_priv);
>  
> -	DRM_DEBUG("buf %p size %lu\n", cmdbuf->buf, cmdbuf->size);
> +	DRM_DEBUG("buf %px size %lu\n", cmdbuf->buf, cmdbuf->size);
>  
>  	ret = via_dispatch_cmdbuffer(dev, cmdbuf);
>  	return ret;
> @@ -368,7 +368,7 @@ static int via_pci_cmdbuffer(struct drm_device *dev, void *data, struct drm_file
>  
>  	LOCK_TEST_WITH_RETURN(dev, file_priv);
>  
> -	DRM_DEBUG("buf %p size %lu\n", cmdbuf->buf, cmdbuf->size);
> +	DRM_DEBUG("buf %px size %lu\n", cmdbuf->buf, cmdbuf->size);
>  
>  	ret = via_dispatch_pci_cmdbuffer(dev, cmdbuf);
>  	return ret;
> diff --git a/drivers/gpu/drm/via/via_irq.c b/drivers/gpu/drm/via/via_irq.c
> index c96830ccc0ec..1c36ebfe2c45 100644
> --- a/drivers/gpu/drm/via/via_irq.c
> +++ b/drivers/gpu/drm/via/via_irq.c
> @@ -259,7 +259,7 @@ void via_driver_irq_preinstall(struct drm_device *dev)
>  	drm_via_irq_t *cur_irq;
>  	int i;
>  
> -	DRM_DEBUG("dev_priv: %p\n", dev_priv);
> +	DRM_DEBUG("dev_priv: %px\n", dev_priv);
>  	if (dev_priv) {
>  		cur_irq = dev_priv->via_irqs;
>  
> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c b/drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c
> index 222c9c2123a1..edfe750b300b 100644
> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c
> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_overlay.c
> @@ -308,7 +308,7 @@ static int vmw_overlay_update_stream(struct vmw_private *dev_priv,
>  	if (!buf)
>  		return -EINVAL;
>  
> -	DRM_DEBUG("   %s: old %p, new %p, %spaused\n", __func__,
> +	DRM_DEBUG("   %s: old %px, new %px, %spaused\n", __func__,
>  		  stream->buf, buf, stream->paused ? "" : "not ");
>  
>  	if (stream->buf != buf) {



More information about the Freedreno mailing list