[PATCH] drm/amdkfd: Fix printing pointer cast

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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);
>>       }
>>  }
>>
>


[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux