2022-09-12 22:15:55

by Jules Irenge

[permalink] [raw]
Subject: [PATCH] octeon_ep: Remove useless casting value returned by vzalloc to structure

coccinelle reports a warning

WARNING: casting value returned by memory allocation
function to (struct octep_rx_buffer *) is useless.

To fix this the useless cast is removed.

Signed-off-by: Jules Irenge <[email protected]>
---
drivers/net/ethernet/marvell/octeon_ep/octep_rx.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/marvell/octeon_ep/octep_rx.c b/drivers/net/ethernet/marvell/octeon_ep/octep_rx.c
index d9ae0937d17a..392d9b0da0d7 100644
--- a/drivers/net/ethernet/marvell/octeon_ep/octep_rx.c
+++ b/drivers/net/ethernet/marvell/octeon_ep/octep_rx.c
@@ -158,8 +158,7 @@ static int octep_setup_oq(struct octep_device *oct, int q_no)
goto desc_dma_alloc_err;
}

- oq->buff_info = (struct octep_rx_buffer *)
- vzalloc(oq->max_count * OCTEP_OQ_RECVBUF_SIZE);
+ oq->buff_info = vzalloc(oq->max_count * OCTEP_OQ_RECVBUF_SIZE);
if (unlikely(!oq->buff_info)) {
dev_err(&oct->pdev->dev,
"Failed to allocate buffer info for OQ-%d\n", q_no);
--
2.35.1


2022-09-20 02:42:12

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH] octeon_ep: Remove useless casting value returned by vzalloc to structure

Hello:

This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski <[email protected]>:

On Mon, 12 Sep 2022 23:03:27 +0100 you wrote:
> coccinelle reports a warning
>
> WARNING: casting value returned by memory allocation
> function to (struct octep_rx_buffer *) is useless.
>
> To fix this the useless cast is removed.
>
> [...]

Here is the summary with links:
- octeon_ep: Remove useless casting value returned by vzalloc to structure
https://git.kernel.org/netdev/net-next/c/ed48cfedf1e2

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html