]> git.baikalelectronics.ru Git - kernel.git/commitdiff
hinic: Remove useless DMA-32 fallback configuration
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 9 Jan 2022 15:57:50 +0000 (16:57 +0100)
committerJakub Kicinski <kuba@kernel.org>
Mon, 10 Jan 2022 00:52:21 +0000 (16:52 -0800)
As stated in [1], dma_set_mask() with a 64-bit mask never fails if
dev->dma_mask is non-NULL.
So, if it fails, the 32 bits case will also fail for the same reason.

Simplify code and remove some dead code accordingly.

[1]: https://lkml.org/lkml/2021/6/7/398

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Link: https://lore.kernel.org/r/23541c28df8d0dcd3663b5dbe0f76af71e70e9cc.1641743855.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/huawei/hinic/hinic_main.c

index 1e1b1be86174d94fc8918fa6edf398186db5d415..05329292d940f8b2d83206750d1496239b1b604a 100644 (file)
@@ -1392,12 +1392,8 @@ static int hinic_probe(struct pci_dev *pdev,
 
        err = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
        if (err) {
-               dev_warn(&pdev->dev, "Couldn't set 64-bit DMA mask\n");
-               err = dma_set_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
-               if (err) {
-                       dev_err(&pdev->dev, "Failed to set DMA mask\n");
-                       goto err_dma_mask;
-               }
+               dev_err(&pdev->dev, "Failed to set DMA mask\n");
+               goto err_dma_mask;
        }
 
        err = nic_dev_init(pdev);