From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: kbuild test robot <lkp@intel.com>
Cc: Matt Sickler <Matt.Sickler@daktronics.com>,
kbuild-all@01.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: kpc2000: fix alloc_cast.cocci warnings
Date: Wed, 15 May 2019 15:16:13 +0200 [thread overview]
Message-ID: <20190515131613.GA23302@kroah.com> (raw)
In-Reply-To: <20190515121615.GA52823@lkp-kbuild04>
On Wed, May 15, 2019 at 08:16:15PM +0800, kbuild test robot wrote:
> From: kbuild test robot <lkp@intel.com>
>
> drivers/staging/kpc2000/kpc_dma/fileops.c:60:8-26: WARNING: casting value returned by memory allocation function to (struct aio_cb_data *) is useless.
>
> Remove casting the values returned by memory allocation functions
> like kmalloc, kzalloc, kmem_cache_alloc, kmem_cache_zalloc etc.
>
> Semantic patch information:
> This makes an effort to find cases of casting of values returned by
> kmalloc, kzalloc, kcalloc, kmem_cache_alloc, kmem_cache_zalloc,
> kmem_cache_alloc_node, kmalloc_node and kzalloc_node and removes
> the casting as it is not required. The result in the patch case may
> need some reformatting.
>
> Generated by: scripts/coccinelle/api/alloc/alloc_cast.cocci
>
> Fixes: 7df95299b94a ("staging: kpc2000: Add DMA driver")
> CC: Matt Sickler <Matt.Sickler@daktronics.com>
> Signed-off-by: kbuild test robot <lkp@intel.com>
> ---
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: 5ac94332248ee017964ba368cdda4ce647e3aba7
> commit: 7df95299b94a63ec67a6389fc02dc25019a80ee8 staging: kpc2000: Add DMA driver
>
> fileops.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- a/drivers/staging/kpc2000/kpc_dma/fileops.c
> +++ b/drivers/staging/kpc2000/kpc_dma/fileops.c
> @@ -57,7 +57,7 @@ int kpc_dma_transfer(struct dev_private
>
> dev_dbg(&priv->ldev->pldev->dev, "kpc_dma_transfer(priv = [%p], kcb = [%p], iov_base = [%p], iov_len = %ld) ldev = [%p]\n", priv, kcb, (void*)iov_base, iov_len, ldev);
>
> - acd = (struct aio_cb_data *) kzalloc(sizeof(struct aio_cb_data), GFP_KERNEL);
> + acd = kzalloc(sizeof(struct aio_cb_data), GFP_KERNEL);
> if (!acd){
> dev_err(&priv->ldev->pldev->dev, "Couldn't kmalloc space for for the aio data\n");
> return -ENOMEM;
Already fixed in my staging-testing branch, thanks.
greg k-h
prev parent reply other threads:[~2019-05-15 13:16 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-05-15 12:16 drivers/staging/kpc2000/kpc_dma/fileops.c:60:8-26: WARNING: casting value returned by memory allocation function to (struct aio_cb_data *) is useless kbuild test robot
2019-05-15 12:16 ` [PATCH] staging: kpc2000: fix alloc_cast.cocci warnings kbuild test robot
2019-05-15 13:16 ` Greg Kroah-Hartman [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190515131613.GA23302@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=Matt.Sickler@daktronics.com \
--cc=kbuild-all@01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkp@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.