* [PATCH 0/2] crypto: ccp - Remove unnecessary includes
@ 2019-08-02 23:20 Bjorn Helgaas
2019-08-02 23:20 ` [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly Bjorn Helgaas
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Bjorn Helgaas @ 2019-08-02 23:20 UTC (permalink / raw)
To: Tom Lendacky, Gary Hook
Cc: Herbert Xu, David S . Miller, linux-crypto, linux-kernel, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
CCP includes <linux/pci.h> many times unnecessarily. Add a couple
DMA-related includes for dma_direction and dma_get_mask(), which were
previously included indirectly via <linux/pci.h>. Then remove the
unnecessary includes of <linux/pci.h>.
Bjorn Helgaas (2):
crypto: ccp - Include DMA declarations explicitly
crypto: ccp - Remove unnecessary linux/pci.h include
drivers/crypto/ccp/ccp-crypto.h | 1 -
drivers/crypto/ccp/ccp-dev-v3.c | 1 -
drivers/crypto/ccp/ccp-dev-v5.c | 1 -
drivers/crypto/ccp/ccp-dev.h | 2 +-
drivers/crypto/ccp/ccp-dmaengine.c | 1 +
drivers/crypto/ccp/ccp-ops.c | 1 -
drivers/crypto/ccp/psp-dev.h | 1 -
drivers/crypto/ccp/sp-dev.h | 1 -
8 files changed, 2 insertions(+), 7 deletions(-)
--
2.22.0.770.g0f2c4a37fd-goog
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly
2019-08-02 23:20 [PATCH 0/2] crypto: ccp - Remove unnecessary includes Bjorn Helgaas
@ 2019-08-02 23:20 ` Bjorn Helgaas
2019-08-06 16:44 ` Gary R Hook
2019-08-02 23:20 ` [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include Bjorn Helgaas
2019-08-09 6:20 ` [PATCH 0/2] crypto: ccp - Remove unnecessary includes Herbert Xu
2 siblings, 1 reply; 6+ messages in thread
From: Bjorn Helgaas @ 2019-08-02 23:20 UTC (permalink / raw)
To: Tom Lendacky, Gary Hook
Cc: Herbert Xu, David S . Miller, linux-crypto, linux-kernel, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
ccp-dev.h uses dma_direction, which is defined in linux/dma-direction.h.
Include that explicitly instead of relying on it being included via
linux/pci.h, since ccp-dev.h requires nothing else from linux/pci.h.
Similarly, ccp-dmaengine.c uses dma_get_mask(), which is defined in
linux/dma-mapping.h, so include that explicitly since it requires nothing
else from linux/pci.h.
A future patch will remove the includes of linux/pci.h where it is not
needed.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
drivers/crypto/ccp/ccp-dev.h | 1 +
drivers/crypto/ccp/ccp-dmaengine.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/drivers/crypto/ccp/ccp-dev.h b/drivers/crypto/ccp/ccp-dev.h
index 5e624920fd99..89aee0900a06 100644
--- a/drivers/crypto/ccp/ccp-dev.h
+++ b/drivers/crypto/ccp/ccp-dev.h
@@ -17,6 +17,7 @@
#include <linux/mutex.h>
#include <linux/list.h>
#include <linux/wait.h>
+#include <linux/dma-direction.h>
#include <linux/dmapool.h>
#include <linux/hw_random.h>
#include <linux/bitops.h>
diff --git a/drivers/crypto/ccp/ccp-dmaengine.c b/drivers/crypto/ccp/ccp-dmaengine.c
index 7f22a45bbc11..f69d495873f0 100644
--- a/drivers/crypto/ccp/ccp-dmaengine.c
+++ b/drivers/crypto/ccp/ccp-dmaengine.c
@@ -9,6 +9,7 @@
#include <linux/module.h>
#include <linux/kernel.h>
+#include <linux/dma-mapping.h>
#include <linux/dmaengine.h>
#include <linux/spinlock.h>
#include <linux/mutex.h>
--
2.22.0.770.g0f2c4a37fd-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include
2019-08-02 23:20 [PATCH 0/2] crypto: ccp - Remove unnecessary includes Bjorn Helgaas
2019-08-02 23:20 ` [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly Bjorn Helgaas
@ 2019-08-02 23:20 ` Bjorn Helgaas
2019-08-06 16:45 ` Gary R Hook
2019-08-09 6:20 ` [PATCH 0/2] crypto: ccp - Remove unnecessary includes Herbert Xu
2 siblings, 1 reply; 6+ messages in thread
From: Bjorn Helgaas @ 2019-08-02 23:20 UTC (permalink / raw)
To: Tom Lendacky, Gary Hook
Cc: Herbert Xu, David S . Miller, linux-crypto, linux-kernel, Bjorn Helgaas
From: Bjorn Helgaas <bhelgaas@google.com>
Remove unused includes of linux/pci.h.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
---
drivers/crypto/ccp/ccp-crypto.h | 1 -
drivers/crypto/ccp/ccp-dev-v3.c | 1 -
drivers/crypto/ccp/ccp-dev-v5.c | 1 -
drivers/crypto/ccp/ccp-dev.h | 1 -
drivers/crypto/ccp/ccp-ops.c | 1 -
drivers/crypto/ccp/psp-dev.h | 1 -
drivers/crypto/ccp/sp-dev.h | 1 -
7 files changed, 7 deletions(-)
diff --git a/drivers/crypto/ccp/ccp-crypto.h b/drivers/crypto/ccp/ccp-crypto.h
index 622b34c17643..903e74e7ad1b 100644
--- a/drivers/crypto/ccp/ccp-crypto.h
+++ b/drivers/crypto/ccp/ccp-crypto.h
@@ -12,7 +12,6 @@
#include <linux/list.h>
#include <linux/wait.h>
-#include <linux/pci.h>
#include <linux/ccp.h>
#include <crypto/algapi.h>
#include <crypto/aes.h>
diff --git a/drivers/crypto/ccp/ccp-dev-v3.c b/drivers/crypto/ccp/ccp-dev-v3.c
index 2b7d47ed5c74..09924f2c264f 100644
--- a/drivers/crypto/ccp/ccp-dev-v3.c
+++ b/drivers/crypto/ccp/ccp-dev-v3.c
@@ -10,7 +10,6 @@
#include <linux/module.h>
#include <linux/kernel.h>
-#include <linux/pci.h>
#include <linux/kthread.h>
#include <linux/interrupt.h>
#include <linux/ccp.h>
diff --git a/drivers/crypto/ccp/ccp-dev-v5.c b/drivers/crypto/ccp/ccp-dev-v5.c
index 217e41bbadaf..0b6ef334f9b7 100644
--- a/drivers/crypto/ccp/ccp-dev-v5.c
+++ b/drivers/crypto/ccp/ccp-dev-v5.c
@@ -9,7 +9,6 @@
#include <linux/module.h>
#include <linux/kernel.h>
-#include <linux/pci.h>
#include <linux/kthread.h>
#include <linux/debugfs.h>
#include <linux/dma-mapping.h>
diff --git a/drivers/crypto/ccp/ccp-dev.h b/drivers/crypto/ccp/ccp-dev.h
index 89aee0900a06..57749c5a5373 100644
--- a/drivers/crypto/ccp/ccp-dev.h
+++ b/drivers/crypto/ccp/ccp-dev.h
@@ -12,7 +12,6 @@
#define __CCP_DEV_H__
#include <linux/device.h>
-#include <linux/pci.h>
#include <linux/spinlock.h>
#include <linux/mutex.h>
#include <linux/list.h>
diff --git a/drivers/crypto/ccp/ccp-ops.c b/drivers/crypto/ccp/ccp-ops.c
index c69ed4bae2eb..b565c08bbe28 100644
--- a/drivers/crypto/ccp/ccp-ops.c
+++ b/drivers/crypto/ccp/ccp-ops.c
@@ -10,7 +10,6 @@
#include <linux/module.h>
#include <linux/kernel.h>
-#include <linux/pci.h>
#include <linux/interrupt.h>
#include <crypto/scatterwalk.h>
#include <crypto/des.h>
diff --git a/drivers/crypto/ccp/psp-dev.h b/drivers/crypto/ccp/psp-dev.h
index c5e06c92d40e..82a084f02990 100644
--- a/drivers/crypto/ccp/psp-dev.h
+++ b/drivers/crypto/ccp/psp-dev.h
@@ -11,7 +11,6 @@
#define __PSP_DEV_H__
#include <linux/device.h>
-#include <linux/pci.h>
#include <linux/spinlock.h>
#include <linux/mutex.h>
#include <linux/list.h>
diff --git a/drivers/crypto/ccp/sp-dev.h b/drivers/crypto/ccp/sp-dev.h
index 8abe9ea7e76f..53c12562d31e 100644
--- a/drivers/crypto/ccp/sp-dev.h
+++ b/drivers/crypto/ccp/sp-dev.h
@@ -13,7 +13,6 @@
#define __SP_DEV_H__
#include <linux/device.h>
-#include <linux/pci.h>
#include <linux/spinlock.h>
#include <linux/mutex.h>
#include <linux/list.h>
--
2.22.0.770.g0f2c4a37fd-goog
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly
2019-08-02 23:20 ` [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly Bjorn Helgaas
@ 2019-08-06 16:44 ` Gary R Hook
0 siblings, 0 replies; 6+ messages in thread
From: Gary R Hook @ 2019-08-06 16:44 UTC (permalink / raw)
To: Bjorn Helgaas, Lendacky, Thomas, Hook, Gary
Cc: Herbert Xu, David S . Miller, linux-crypto, linux-kernel, Bjorn Helgaas
On 8/2/19 6:20 PM, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> ccp-dev.h uses dma_direction, which is defined in linux/dma-direction.h.
> Include that explicitly instead of relying on it being included via
> linux/pci.h, since ccp-dev.h requires nothing else from linux/pci.h.
>
> Similarly, ccp-dmaengine.c uses dma_get_mask(), which is defined in
> linux/dma-mapping.h, so include that explicitly since it requires nothing
> else from linux/pci.h.
>
> A future patch will remove the includes of linux/pci.h where it is not
> needed.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Gary R Hook <gary.hook@amd.com>
> ---
> drivers/crypto/ccp/ccp-dev.h | 1 +
> drivers/crypto/ccp/ccp-dmaengine.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/drivers/crypto/ccp/ccp-dev.h b/drivers/crypto/ccp/ccp-dev.h
> index 5e624920fd99..89aee0900a06 100644
> --- a/drivers/crypto/ccp/ccp-dev.h
> +++ b/drivers/crypto/ccp/ccp-dev.h
> @@ -17,6 +17,7 @@
> #include <linux/mutex.h>
> #include <linux/list.h>
> #include <linux/wait.h>
> +#include <linux/dma-direction.h>
> #include <linux/dmapool.h>
> #include <linux/hw_random.h>
> #include <linux/bitops.h>
> diff --git a/drivers/crypto/ccp/ccp-dmaengine.c b/drivers/crypto/ccp/ccp-dmaengine.c
> index 7f22a45bbc11..f69d495873f0 100644
> --- a/drivers/crypto/ccp/ccp-dmaengine.c
> +++ b/drivers/crypto/ccp/ccp-dmaengine.c
> @@ -9,6 +9,7 @@
>
> #include <linux/module.h>
> #include <linux/kernel.h>
> +#include <linux/dma-mapping.h>
> #include <linux/dmaengine.h>
> #include <linux/spinlock.h>
> #include <linux/mutex.h>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include
2019-08-02 23:20 ` [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include Bjorn Helgaas
@ 2019-08-06 16:45 ` Gary R Hook
0 siblings, 0 replies; 6+ messages in thread
From: Gary R Hook @ 2019-08-06 16:45 UTC (permalink / raw)
To: Bjorn Helgaas, Lendacky, Thomas, Hook, Gary
Cc: Herbert Xu, David S . Miller, linux-crypto, linux-kernel, Bjorn Helgaas
On 8/2/19 6:20 PM, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> Remove unused includes of linux/pci.h.
>
> Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Gary R Hook <gary.hook@amd.com>
> ---
> drivers/crypto/ccp/ccp-crypto.h | 1 -
> drivers/crypto/ccp/ccp-dev-v3.c | 1 -
> drivers/crypto/ccp/ccp-dev-v5.c | 1 -
> drivers/crypto/ccp/ccp-dev.h | 1 -
> drivers/crypto/ccp/ccp-ops.c | 1 -
> drivers/crypto/ccp/psp-dev.h | 1 -
> drivers/crypto/ccp/sp-dev.h | 1 -
> 7 files changed, 7 deletions(-)
>
> diff --git a/drivers/crypto/ccp/ccp-crypto.h b/drivers/crypto/ccp/ccp-crypto.h
> index 622b34c17643..903e74e7ad1b 100644
> --- a/drivers/crypto/ccp/ccp-crypto.h
> +++ b/drivers/crypto/ccp/ccp-crypto.h
> @@ -12,7 +12,6 @@
>
> #include <linux/list.h>
> #include <linux/wait.h>
> -#include <linux/pci.h>
> #include <linux/ccp.h>
> #include <crypto/algapi.h>
> #include <crypto/aes.h>
> diff --git a/drivers/crypto/ccp/ccp-dev-v3.c b/drivers/crypto/ccp/ccp-dev-v3.c
> index 2b7d47ed5c74..09924f2c264f 100644
> --- a/drivers/crypto/ccp/ccp-dev-v3.c
> +++ b/drivers/crypto/ccp/ccp-dev-v3.c
> @@ -10,7 +10,6 @@
>
> #include <linux/module.h>
> #include <linux/kernel.h>
> -#include <linux/pci.h>
> #include <linux/kthread.h>
> #include <linux/interrupt.h>
> #include <linux/ccp.h>
> diff --git a/drivers/crypto/ccp/ccp-dev-v5.c b/drivers/crypto/ccp/ccp-dev-v5.c
> index 217e41bbadaf..0b6ef334f9b7 100644
> --- a/drivers/crypto/ccp/ccp-dev-v5.c
> +++ b/drivers/crypto/ccp/ccp-dev-v5.c
> @@ -9,7 +9,6 @@
>
> #include <linux/module.h>
> #include <linux/kernel.h>
> -#include <linux/pci.h>
> #include <linux/kthread.h>
> #include <linux/debugfs.h>
> #include <linux/dma-mapping.h>
> diff --git a/drivers/crypto/ccp/ccp-dev.h b/drivers/crypto/ccp/ccp-dev.h
> index 89aee0900a06..57749c5a5373 100644
> --- a/drivers/crypto/ccp/ccp-dev.h
> +++ b/drivers/crypto/ccp/ccp-dev.h
> @@ -12,7 +12,6 @@
> #define __CCP_DEV_H__
>
> #include <linux/device.h>
> -#include <linux/pci.h>
> #include <linux/spinlock.h>
> #include <linux/mutex.h>
> #include <linux/list.h>
> diff --git a/drivers/crypto/ccp/ccp-ops.c b/drivers/crypto/ccp/ccp-ops.c
> index c69ed4bae2eb..b565c08bbe28 100644
> --- a/drivers/crypto/ccp/ccp-ops.c
> +++ b/drivers/crypto/ccp/ccp-ops.c
> @@ -10,7 +10,6 @@
>
> #include <linux/module.h>
> #include <linux/kernel.h>
> -#include <linux/pci.h>
> #include <linux/interrupt.h>
> #include <crypto/scatterwalk.h>
> #include <crypto/des.h>
> diff --git a/drivers/crypto/ccp/psp-dev.h b/drivers/crypto/ccp/psp-dev.h
> index c5e06c92d40e..82a084f02990 100644
> --- a/drivers/crypto/ccp/psp-dev.h
> +++ b/drivers/crypto/ccp/psp-dev.h
> @@ -11,7 +11,6 @@
> #define __PSP_DEV_H__
>
> #include <linux/device.h>
> -#include <linux/pci.h>
> #include <linux/spinlock.h>
> #include <linux/mutex.h>
> #include <linux/list.h>
> diff --git a/drivers/crypto/ccp/sp-dev.h b/drivers/crypto/ccp/sp-dev.h
> index 8abe9ea7e76f..53c12562d31e 100644
> --- a/drivers/crypto/ccp/sp-dev.h
> +++ b/drivers/crypto/ccp/sp-dev.h
> @@ -13,7 +13,6 @@
> #define __SP_DEV_H__
>
> #include <linux/device.h>
> -#include <linux/pci.h>
> #include <linux/spinlock.h>
> #include <linux/mutex.h>
> #include <linux/list.h>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] crypto: ccp - Remove unnecessary includes
2019-08-02 23:20 [PATCH 0/2] crypto: ccp - Remove unnecessary includes Bjorn Helgaas
2019-08-02 23:20 ` [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly Bjorn Helgaas
2019-08-02 23:20 ` [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include Bjorn Helgaas
@ 2019-08-09 6:20 ` Herbert Xu
2 siblings, 0 replies; 6+ messages in thread
From: Herbert Xu @ 2019-08-09 6:20 UTC (permalink / raw)
To: Bjorn Helgaas
Cc: Tom Lendacky, Gary Hook, David S . Miller, linux-crypto,
linux-kernel, Bjorn Helgaas
On Fri, Aug 02, 2019 at 06:20:10PM -0500, Bjorn Helgaas wrote:
> From: Bjorn Helgaas <bhelgaas@google.com>
>
> CCP includes <linux/pci.h> many times unnecessarily. Add a couple
> DMA-related includes for dma_direction and dma_get_mask(), which were
> previously included indirectly via <linux/pci.h>. Then remove the
> unnecessary includes of <linux/pci.h>.
>
> Bjorn Helgaas (2):
> crypto: ccp - Include DMA declarations explicitly
> crypto: ccp - Remove unnecessary linux/pci.h include
>
> drivers/crypto/ccp/ccp-crypto.h | 1 -
> drivers/crypto/ccp/ccp-dev-v3.c | 1 -
> drivers/crypto/ccp/ccp-dev-v5.c | 1 -
> drivers/crypto/ccp/ccp-dev.h | 2 +-
> drivers/crypto/ccp/ccp-dmaengine.c | 1 +
> drivers/crypto/ccp/ccp-ops.c | 1 -
> drivers/crypto/ccp/psp-dev.h | 1 -
> drivers/crypto/ccp/sp-dev.h | 1 -
> 8 files changed, 2 insertions(+), 7 deletions(-)
All applied. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-08-09 6:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-02 23:20 [PATCH 0/2] crypto: ccp - Remove unnecessary includes Bjorn Helgaas
2019-08-02 23:20 ` [PATCH 1/2] crypto: ccp - Include DMA declarations explicitly Bjorn Helgaas
2019-08-06 16:44 ` Gary R Hook
2019-08-02 23:20 ` [PATCH 2/2] crypto: ccp - Remove unnecessary linux/pci.h include Bjorn Helgaas
2019-08-06 16:45 ` Gary R Hook
2019-08-09 6:20 ` [PATCH 0/2] crypto: ccp - Remove unnecessary includes Herbert Xu
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.