[PATCH 3/6] ARM: OMAP2+: Move plat/iovmm.h to include/linux/omap-iommu.h
Tony Lindgren
tony at atomide.com
Thu Oct 25 12:56:44 EDT 2012
* Laurent Pinchart <laurent.pinchart at ideasonboard.com> [121025 01:39]:
>
> I still think you should split this in two files, omap-iommu.h and omap-
> iovmm.h. The later would just be arch/arm/plat-omap/include/plat/iovmm.h moved
> to include/linux.h.
Can you please explain a bit more why you're thinking a separate
omap-iovmm.h is needed in addtion to omap-iommu.h?
My reasoning for not adding it is that neither intel nor amd needs
more than intel-iommu.h and amd-iommu.h. And hopefully the iommu
framework will eventually provide the API needed. And I'd rather
not be the person introducing this second new file into
include/linux :)
Joerg and Ohad, do you have any opinions on this?
Regards,
Tony
More information about the linux-arm-kernel
mailing list