Re: [PATCH 2/2] iommu: Include linux/notifier.h

2012-08-03 Thread Joerg Roedel
On Wed, Jul 25, 2012 at 04:24:50PM +0200, Thierry Reding wrote:
 The linux/iommu.h header uses types defined in linux/notifier.h but
 doesn't include it.
 
 Signed-off-by: Thierry Reding thierry.red...@avionic-design.de

Better than adding the header-file is to add a forward declaration for
struct notifier_block. I added a patch to iommu/fixes for this.

Thanks,

Joerg

-- 
AMD Operating System Research Center

Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach
General Managers: Alberto Bozzo
Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632

___
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu


[PATCH 2/2] iommu: Include linux/notifier.h

2012-07-25 Thread Thierry Reding
The linux/iommu.h header uses types defined in linux/notifier.h but
doesn't include it.

Signed-off-by: Thierry Reding thierry.red...@avionic-design.de
---
 include/linux/iommu.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/iommu.h b/include/linux/iommu.h
index 9cbcc6a..4f64020 100644
--- a/include/linux/iommu.h
+++ b/include/linux/iommu.h
@@ -20,6 +20,7 @@
 #define __LINUX_IOMMU_H
 
 #include linux/errno.h
+#include linux/notifier.h
 #include linux/types.h
 
 #define IOMMU_READ (1)
-- 
1.7.11.2

___
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu