Hi Kent, I just realized when looking at this again, that you could remove the 0x before %p. I think %p will print 0x<address> itself.
Regards, Felix On 2017-11-30 11:49 AM, Kent Russell wrote: > Just print a pointer instead of casting > > Signed-off-by: Kent Russell <[email protected]> > --- > drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c | 7 +++---- > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > b/drivers/gpu/drm/amd/amdkfd/kfd_doorbell.c > index feb76c2..8abaa08 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 == 0x%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 == 0x%p\n", > + *doorbell_off, (kfd->doorbell_kernel_ptr + inx)); > > return kfd->doorbell_kernel_ptr + inx; > } _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
