Thanks. This change is Reviewed-by: Felix Kuehling <[email protected]>
On 2017-12-04 06:50 AM, Kent Russell wrote: > Just print a pointer instead of casting > > v2: Remove the 0x prefix, since %p prints that automatically, and remove > it from one other spot as well > > Change-Id: I0564230d2985e703d59201832f50358da3b67afc > Signed-off-by: Kent Russell <[email protected]> > --- > drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > index feb76c2..ebb4da14 100644 > --- a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > @@ -116,8 +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 == 0x%08lX\n", > - (uintptr_t)kfd->doorbell_kernel_ptr); > + pr_debug("doorbell kernel address == %p\n", kfd->doorbell_kernel_ptr); > > return 0; > } > @@ -194,8 +193,8 @@ u32 __iomem *kfd_get_kernel_doorbell(struct kfd_dev *kfd, > > pr_debug("Get kernel queue doorbell\n" > " doorbell offset == 0x%08X\n" > - " kernel address == 0x%08lX\n", > - *doorbell_off, (uintptr_t)(kfd->doorbell_kernel_ptr + inx)); > + " kernel address == %p\n", > + *doorbell_off, (kfd->doorbell_kernel_ptr + inx)); > > return kfd->doorbell_kernel_ptr + inx; > } > @@ -215,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 0x%p\n", value, db); > + pr_debug("Writing %d to doorbell address %p\n", value, db); > } > } > _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
