Patrik Jakobsson 0c7d4e
From b13b308aa1c46d996561da45ba9baea82e45f94a Mon Sep 17 00:00:00 2001
Patrik Jakobsson 0c7d4e
From: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Patrik Jakobsson 0c7d4e
Date: Sat, 18 Sep 2021 07:36:42 +0200
Patrik Jakobsson 0c7d4e
Subject: mei: Remove usage of the deprecated "pci-dma-compat.h" API
Patrik Jakobsson 0c7d4e
Git-commit: 54fa156bb33ad06077ec709dc2331a4d5ee446f3
Patrik Jakobsson 0c7d4e
Patch-mainline: v5.16-rc1
Patrik Jakobsson 0c7d4e
References: jsc#PED-1218
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
In [1], Christoph Hellwig has proposed to remove the wrappers in
Patrik Jakobsson 0c7d4e
include/linux/pci-dma-compat.h.
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
Some reasons why this API should be removed have been given by Julia
Patrik Jakobsson 0c7d4e
Lawall in [2].
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
Finally, Arnd Bergmann reminded that the documentation was updated 11 years
Patrik Jakobsson 0c7d4e
ago to only describe the modern linux/dma-mapping.h interfaces and mark the
Patrik Jakobsson 0c7d4e
old bus-specific ones as no longer recommended, see commit 216bf58f4092
Patrik Jakobsson 0c7d4e
("Documentation: convert PCI-DMA-mapping.txt to use the generic DMA API").
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
A coccinelle script has been used to perform the needed transformation
Patrik Jakobsson 0c7d4e
Only relevant parts are given below.
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
@@
Patrik Jakobsson 0c7d4e
expression e1, e2;
Patrik Jakobsson 0c7d4e
@@
Patrik Jakobsson 0c7d4e
-    pci_set_dma_mask(e1, e2)
Patrik Jakobsson 0c7d4e
+    dma_set_mask(&e1->dev, e2)
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
[1]: https://lore.kernel.org/kernel-janitors/20200421081257.GA131897@infradead.org/
Patrik Jakobsson 0c7d4e
[2]: https://lore.kernel.org/kernel-janitors/alpine.DEB.2.22.394.2007120902170.2424@hadrien/
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
Patrik Jakobsson 0c7d4e
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Patrik Jakobsson 0c7d4e
Link: https://lore.kernel.org/r/c6f280927835c5677cc0367fccdc0ef54b307bd8.1631943364.git.christophe.jaillet@wanadoo.fr
Patrik Jakobsson 0c7d4e
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Patrik Jakobsson 0c7d4e
Acked-by: Patrik Jakobsson <pjakobsson@suse.de>
Patrik Jakobsson 0c7d4e
---
Patrik Jakobsson 0c7d4e
 drivers/misc/mei/pci-txe.c | 4 ++--
Patrik Jakobsson 0c7d4e
 1 file changed, 2 insertions(+), 2 deletions(-)
Patrik Jakobsson 0c7d4e
Patrik Jakobsson 0c7d4e
diff --git a/drivers/misc/mei/pci-txe.c b/drivers/misc/mei/pci-txe.c
Patrik Jakobsson 0c7d4e
index aec0483b8e72..fa20d9a27813 100644
Patrik Jakobsson 0c7d4e
--- a/drivers/misc/mei/pci-txe.c
Patrik Jakobsson 0c7d4e
+++ b/drivers/misc/mei/pci-txe.c
Patrik Jakobsson 0c7d4e
@@ -69,9 +69,9 @@ static int mei_txe_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
Patrik Jakobsson 0c7d4e
 		goto end;
Patrik Jakobsson 0c7d4e
 	}
Patrik Jakobsson 0c7d4e
 
Patrik Jakobsson 0c7d4e
-	err = pci_set_dma_mask(pdev, DMA_BIT_MASK(36));
Patrik Jakobsson 0c7d4e
+	err = dma_set_mask(&pdev->dev, DMA_BIT_MASK(36));
Patrik Jakobsson 0c7d4e
 	if (err) {
Patrik Jakobsson 0c7d4e
-		err = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));
Patrik Jakobsson 0c7d4e
+		err = dma_set_mask(&pdev->dev, DMA_BIT_MASK(32));
Patrik Jakobsson 0c7d4e
 		if (err) {
Patrik Jakobsson 0c7d4e
 			dev_err(&pdev->dev, "No suitable DMA available.\n");
Patrik Jakobsson 0c7d4e
 			goto end;
Patrik Jakobsson 0c7d4e
-- 
Patrik Jakobsson 0c7d4e
2.39.0
Patrik Jakobsson 0c7d4e