[PATCH ipsec-next v2] sky2: Remove redundant assignment and parentheses

[PATCH ipsec-next v2] sky2: Remove redundant assignment and parentheses

[luo penghao]

The variable err will be reassigned on subsequent branches, and this
assignment does not perform related value operations. This will cause
the double parentheses to be redundant, so the inner parentheses should
be deleted.

clang_analyzer complains as follows:

drivers/net/ethernet/marvell/sky2.c:4988: warning:

Although the value stored to 'err' is used in the enclosing expression,
the value is never actually read from 'err'.

Changes in v2:

modify title category:octeontx2-af to sky2.
delete the inner parentheses.

Reported-by: Zeal Robot <zealci@zte.com.cn>
Signed-off-by: luo penghao <luo.penghao@zte.com.cn>
---
 drivers/net/ethernet/marvell/sky2.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/marvell/sky2.c b/drivers/net/ethernet/marvell/sky2.c
index 8b8bff5..33558aa 100644
--- a/drivers/net/ethernet/marvell/sky2.c
+++ b/drivers/net/ethernet/marvell/sky2.c
@@ -4985,7 +4985,7 @@ static int sky2_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 	pci_set_master(pdev);
 
 	if (sizeof(dma_addr_t) > sizeof(u32) &&
-	    !(err = dma_set_mask(&pdev->dev, DMA_BIT_MASK(64)))) {
+	    !dma_set_mask(&pdev->dev, DMA_BIT_MASK(64))) {
 		using_dac = 1;
 		err = dma_set_coherent_mask(&pdev->dev, DMA_BIT_MASK(64));
 		if (err < 0) {
-- 
2.15.2

Re: [PATCH ipsec-next v2] sky2: Remove redundant assignment and parentheses

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Thu, 28 Oct 2021 03:15:51 +0000 you wrote:
> The variable err will be reassigned on subsequent branches, and this
> assignment does not perform related value operations. This will cause
> the double parentheses to be redundant, so the inner parentheses should
> be deleted.
> 
> clang_analyzer complains as follows:
> 
> [...]

Here is the summary with links:
  - [ipsec-next,v2] sky2: Remove redundant assignment and parentheses
    https://git.kernel.org/netdev/net-next/c/6a03bfbd5ead

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

Re: [PATCH ipsec-next v2] sky2: Remove redundant assignment and parentheses

[Jakub Kicinski]

On Thu, 28 Oct 2021 03:15:51 +0000 luo penghao wrote:
> The variable err will be reassigned on subsequent branches, and this
> assignment does not perform related value operations. This will cause
> the double parentheses to be redundant, so the inner parentheses should
> be deleted.

Why the "ipsec-next" in the subject? :)