* [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
@ 2019-10-05 17:58 Colin King
2019-10-05 18:10 ` Randy Dunlap
2019-10-07 16:08 ` Alex Deucher
0 siblings, 2 replies; 6+ messages in thread
From: Colin King @ 2019-10-05 17:58 UTC (permalink / raw)
To: Felix Kuehling, Alex Deucher, Christian König, David Zhou,
David Airlie, Daniel Vetter, amd-gfx, dri-devel
Cc: kernel-janitors, linux-kernel
From: Colin Ian King <colin.king@canonical.com>
Function kgd2kfd_init is missing a void argument, add it
to clean up the non-ANSI function declaration.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
drivers/gpu/drm/amd/amdkfd/kfd_module.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_module.c b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
index 986ff52d5750..f4b7f7e6c40e 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_module.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
@@ -82,7 +82,7 @@ static void kfd_exit(void)
kfd_chardev_exit();
}
-int kgd2kfd_init()
+int kgd2kfd_init(void)
{
return kfd_init();
}
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
2019-10-05 17:58 [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init Colin King
@ 2019-10-05 18:10 ` Randy Dunlap
2019-10-05 18:23 ` Colin Ian King
2019-10-07 16:08 ` Alex Deucher
1 sibling, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2019-10-05 18:10 UTC (permalink / raw)
To: Colin King, Felix Kuehling, Alex Deucher, Christian König,
David Zhou, David Airlie, Daniel Vetter, amd-gfx, dri-devel
Cc: kernel-janitors, linux-kernel
On 10/5/19 10:58 AM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> Function kgd2kfd_init is missing a void argument, add it
> to clean up the non-ANSI function declaration.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Randy Dunlap <rdunlap@infradead.org>
sparse reports 2 such warnings in amdgpu:
../drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_module.c:85:18: warning: non-ANSI function declaration of function 'kgd2kfd_init'
../drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:168:60: warning: non-ANSI function declaration of function 'amdgpu_amdkfd_gfx_10_0_get_functions'
Thanks.
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_module.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_module.c b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> index 986ff52d5750..f4b7f7e6c40e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> @@ -82,7 +82,7 @@ static void kfd_exit(void)
> kfd_chardev_exit();
> }
>
> -int kgd2kfd_init()
> +int kgd2kfd_init(void)
> {
> return kfd_init();
> }
>
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
2019-10-05 18:10 ` Randy Dunlap
@ 2019-10-05 18:23 ` Colin Ian King
2019-10-05 18:30 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Colin Ian King @ 2019-10-05 18:23 UTC (permalink / raw)
To: Randy Dunlap, Felix Kuehling, Alex Deucher, Christian König,
David Zhou, David Airlie, Daniel Vetter, amd-gfx, dri-devel
Cc: kernel-janitors, linux-kernel
On 05/10/2019 19:10, Randy Dunlap wrote:
> On 10/5/19 10:58 AM, Colin King wrote:
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> Function kgd2kfd_init is missing a void argument, add it
>> to clean up the non-ANSI function declaration.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>
> sparse reports 2 such warnings in amdgpu:
>
> ../drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_module.c:85:18: warning: non-ANSI function declaration of function 'kgd2kfd_init'
> ../drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:168:60: warning: non-ANSI function declaration of function 'amdgpu_amdkfd_gfx_10_0_get_functions'
Looking at linux-next, the amdgpu_amdkfd_gfx_10_0_get_functions() was
removed in commit:
commit e392c887df979112af94cfec08dd87f4dd55d085
Author: Yong Zhao <Yong.Zhao@amd.com>
Date: Fri Sep 27 22:03:42 2019 -0400
drm/amdkfd: Use array to probe kfd2kgd_calls
..so I didn't fix that one up
Coln
>
> Thanks.
>
>> ---
>> drivers/gpu/drm/amd/amdkfd/kfd_module.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_module.c b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> index 986ff52d5750..f4b7f7e6c40e 100644
>> --- a/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> @@ -82,7 +82,7 @@ static void kfd_exit(void)
>> kfd_chardev_exit();
>> }
>>
>> -int kgd2kfd_init()
>> +int kgd2kfd_init(void)
>> {
>> return kfd_init();
>> }
>>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
2019-10-05 18:23 ` Colin Ian King
@ 2019-10-05 18:30 ` Randy Dunlap
0 siblings, 0 replies; 6+ messages in thread
From: Randy Dunlap @ 2019-10-05 18:30 UTC (permalink / raw)
To: Colin Ian King, Felix Kuehling, Alex Deucher,
Christian König, David Zhou, David Airlie, Daniel Vetter,
amd-gfx, dri-devel
Cc: kernel-janitors, linux-kernel
On 10/5/19 11:23 AM, Colin Ian King wrote:
> On 05/10/2019 19:10, Randy Dunlap wrote:
>> On 10/5/19 10:58 AM, Colin King wrote:
>>> From: Colin Ian King <colin.king@canonical.com>
>>>
>>> Function kgd2kfd_init is missing a void argument, add it
>>> to clean up the non-ANSI function declaration.
>>>
>>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
>>
>> Acked-by: Randy Dunlap <rdunlap@infradead.org>
>>
>> sparse reports 2 such warnings in amdgpu:
>>
>> ../drivers/gpu/drm/amd/amdgpu/../amdkfd/kfd_module.c:85:18: warning: non-ANSI function declaration of function 'kgd2kfd_init'
>> ../drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:168:60: warning: non-ANSI function declaration of function 'amdgpu_amdkfd_gfx_10_0_get_functions'
>
> Looking at linux-next, the amdgpu_amdkfd_gfx_10_0_get_functions() was
> removed in commit:
>
> commit e392c887df979112af94cfec08dd87f4dd55d085
> Author: Yong Zhao <Yong.Zhao@amd.com>
> Date: Fri Sep 27 22:03:42 2019 -0400
>
> drm/amdkfd: Use array to probe kfd2kgd_calls
>
> ..so I didn't fix that one up
ah. Thanks.
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
2019-10-05 17:58 [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init Colin King
2019-10-05 18:10 ` Randy Dunlap
@ 2019-10-07 16:08 ` Alex Deucher
2019-10-07 18:39 ` Kuehling, Felix
1 sibling, 1 reply; 6+ messages in thread
From: Alex Deucher @ 2019-10-07 16:08 UTC (permalink / raw)
To: Colin King
Cc: Felix Kuehling, Alex Deucher, Christian König, David Zhou,
David Airlie, Daniel Vetter, amd-gfx list,
Maling list - DRI developers, kernel-janitors, LKML
On Sat, Oct 5, 2019 at 1:58 PM Colin King <colin.king@canonical.com> wrote:
>
> From: Colin Ian King <colin.king@canonical.com>
>
> Function kgd2kfd_init is missing a void argument, add it
> to clean up the non-ANSI function declaration.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
Applied. thanks!
Alex
> ---
> drivers/gpu/drm/amd/amdkfd/kfd_module.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_module.c b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> index 986ff52d5750..f4b7f7e6c40e 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
> @@ -82,7 +82,7 @@ static void kfd_exit(void)
> kfd_chardev_exit();
> }
>
> -int kgd2kfd_init()
> +int kgd2kfd_init(void)
> {
> return kfd_init();
> }
> --
> 2.20.1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init
2019-10-07 16:08 ` Alex Deucher
@ 2019-10-07 18:39 ` Kuehling, Felix
0 siblings, 0 replies; 6+ messages in thread
From: Kuehling, Felix @ 2019-10-07 18:39 UTC (permalink / raw)
To: Alex Deucher, Colin King
Cc: Deucher, Alexander, Koenig, Christian, Zhou, David(ChunMing),
David Airlie, Daniel Vetter, amd-gfx list,
Maling list - DRI developers, kernel-janitors, LKML
On 2019-10-07 12:08 p.m., Alex Deucher wrote:
> On Sat, Oct 5, 2019 at 1:58 PM Colin King <colin.king@canonical.com> wrote:
>> From: Colin Ian King <colin.king@canonical.com>
>>
>> Function kgd2kfd_init is missing a void argument, add it
>> to clean up the non-ANSI function declaration.
>>
>> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> Applied. thanks!
Thank you!
>
> Alex
>
>> ---
>> drivers/gpu/drm/amd/amdkfd/kfd_module.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_module.c b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> index 986ff52d5750..f4b7f7e6c40e 100644
>> --- a/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_module.c
>> @@ -82,7 +82,7 @@ static void kfd_exit(void)
>> kfd_chardev_exit();
>> }
>>
>> -int kgd2kfd_init()
>> +int kgd2kfd_init(void)
>> {
>> return kfd_init();
>> }
>> --
>> 2.20.1
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-10-07 18:39 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-05 17:58 [PATCH] drm/amdkfd: add missing void argument to function kgd2kfd_init Colin King
2019-10-05 18:10 ` Randy Dunlap
2019-10-05 18:23 ` Colin Ian King
2019-10-05 18:30 ` Randy Dunlap
2019-10-07 16:08 ` Alex Deucher
2019-10-07 18:39 ` Kuehling, Felix
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).