Re: [PATCH] iommu: Constify struct iommu_ops

Thierry Reding
2014-06-27T08:18:50Z

[-- Attachment #1: Type: text/plain, Size: 1042 bytes --]

On Thu, Jun 26, 2014 at 09:04:54PM +0200, Thierry Reding wrote:
> From: Thierry Reding <treding@nvidia.com>
>
> This structure is read-only data and should never be modified.
>
> Signed-off-by: Thierry Reding <treding@nvidia.com>
> ---
> drivers/iommu/amd_iommu.c | 4 ++--
> drivers/iommu/arm-smmu.c | 2 +-
> drivers/iommu/exynos-iommu.c | 2 +-
> drivers/iommu/fsl_pamu_domain.c | 2 +-
> drivers/iommu/intel-iommu.c | 4 ++--
> drivers/iommu/iommu.c | 19 ++++++++++++++-----
> drivers/iommu/ipmmu-vmsa.c | 2 +-
> drivers/iommu/msm_iommu.c | 2 +-
> drivers/iommu/omap-iommu.c | 2 +-
> drivers/iommu/shmobile-iommu.c | 2 +-
> drivers/iommu/tegra-gart.c | 2 +-
> drivers/iommu/tegra-smmu.c | 2 +-
> include/linux/iommu.h | 4 ++--
> 13 files changed, 29 insertions(+), 20 deletions(-)

I noticed that this patch is missing a hunk to include/linux/device.h
and I've sent a v2 to address this. Apologies for the noise.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 819 bytes --]