[RFC 1/5] ARM: dev_archdata: add private iommu extension
From: Ohad Ben-Cohen <hidden>
Date: 2011-10-05 09:26:56
Also in:
linux-omap, lkml
Hi Russell, On Sun, Sep 25, 2011 at 1:58 PM, Ohad Ben-Cohen [off-list ref] wrote:
Add a private iommu pointer to the ARM-specific arch data in the device struct, which will be used to attach iommu-specific data to devices which require iommu support.
Can you please take a look and ack/nack ? With this change in hand, generic code will now be able to control OMAP's iommu devices too, without having to use any OMAP-specific API (this is demonstrated in this patch set). This will allow us to further clean the existing mainline OMAP iommu users, and focus on generic IOMMU code for future users (rpmsg/remoteproc and Marek's upcoming generic DMA API). The change is only effective when an iommu driver, which supports the generic IOMMU API, is built (all of them must select CONFIG_IOMMU_API), and otherwise is a nop. Obviously the change itself is not OMAP-specific, and it will be useful for other platforms too. Thanks! Ohad.
quoted hunk
Different iommu implementations (on different platforms) will attach different types of data to this pointer, so 'void *' is currently used (the downside is reduced typesafety). Note: ia64, x86 and sparc have this exact iommu extension as well, and if others are likely to adopt it too, we might want to consider adding this to the device struct itself directly. Signed-off-by: Ohad Ben-Cohen <redacted> Cc: Russell King <redacted> Cc: Arnd Bergmann <arnd@arndb.de> Cc: Grant Likely <redacted> --- ?arch/arm/include/asm/device.h | ? ?3 +++ ?1 files changed, 3 insertions(+), 0 deletions(-)diff --git a/arch/arm/include/asm/device.h b/arch/arm/include/asm/device.h index 9f390ce..6615f03 100644 --- a/arch/arm/include/asm/device.h +++ b/arch/arm/include/asm/device.h@@ -10,6 +10,9 @@ struct dev_archdata {?#ifdef CONFIG_DMABOUNCE ? ? ? ?struct dmabounce_device_info *dmabounce; ?#endif +#ifdef CONFIG_IOMMU_API + ? ? ? void *iommu; /* private IOMMU data */ +#endif ?}; ?struct pdev_archdata { -- 1.7.4.1