Applied to -next
Thanks,
Oded
On Mon, Dec 4, 2017 at 4:05 PM, Felix Kuehling <felix.kuehling at amd.com> wrote:
> Thanks. This change is Reviewed-by: Felix Kuehling <Felix.Kuehling at amd.com>
>
>
> 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 <kent.russell at amd.com>
>> ---
>> 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);
>> }
>> }
>>
>
