[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kernel-janitors
Subject:    Re: [PATCH] amdkfd: Remove unessary void pointer cast
From:       Oded Gabbay <oded.gabbay () amd ! com>
Date:       2015-04-26 9:20:26
Message-ID: 553CADDA.7050800 () amd ! com
[Download RAW message or body]



On 04/23/2015 12:58 PM, Firo Yang wrote:
> kmalloc() returns a void pointer - no need to cast it in
> drivers/gpu/drm/amd/amdkfd/kfd_process.c::kfd_process_destroy_delayed()
>
> Signed-off-by: Firo Yang <firogm@gmail.com>
> ---
>   drivers/gpu/drm/amd/amdkfd/kfd_process.c | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_process.c b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> index 945d622..4d7bc95 100644
> --- a/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_process.c
> @@ -203,8 +203,7 @@ static void kfd_process_destroy_delayed(struct rcu_head *rcu)
>
>   	mmdrop(p->mm);
>
> -	work = (struct kfd_process_release_work *)
> -		kmalloc(sizeof(struct kfd_process_release_work), GFP_ATOMIC);
> +	work = kmalloc(sizeof(struct kfd_process_release_work), GFP_ATOMIC);
>
>   	if (work) {
>   		INIT_WORK((struct work_struct *) work, kfd_process_wq_release);
>
Thanks!
Applied to my -next tree

	Oded
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]