leandrof/linux
1
0
Fork 0
forked from Minki/linux
Code Issues Pull Requests Packages Projects Releases Wiki Activity

iommu/ipmmu-vmsa: Clean up struct ipmmu_vmsa_iommu_priv

Browse Source 
Now that the IPMMU instance pointer is the only thing remaining in the
private data structure, we no longer need the extra level of indirection
and can simply stash that directlty in the fwspec.

Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
This commit is contained in:
Robin Murphy 2017-10-13 19:23:41 +01:00 committed by Alex Williamson
parent b354c73edc
commit e4efe4a9a2
1 changed files with 12 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
drivers/iommu/ipmmu-vmsa.c
View File

@ -58,16 +58,12 @@ struct ipmmu_vmsa_domain {
spinlock_t lock; /* Protects mappings */ spinlock_t lock; /* Protects mappings */
}; };
struct ipmmu_vmsa_iommu_priv {
struct ipmmu_vmsa_device *mmu;
};
static struct ipmmu_vmsa_domain *to_vmsa_domain(struct iommu_domain *dom) static struct ipmmu_vmsa_domain *to_vmsa_domain(struct iommu_domain *dom)
{ {
return container_of(dom, struct ipmmu_vmsa_domain, io_domain); return container_of(dom, struct ipmmu_vmsa_domain, io_domain);
} }
static struct ipmmu_vmsa_iommu_priv *to_priv(struct device *dev) static struct ipmmu_vmsa_device *to_ipmmu(struct device *dev)
{ {
return dev->iommu_fwspec ? dev->iommu_fwspec->iommu_priv : NULL; return dev->iommu_fwspec ? dev->iommu_fwspec->iommu_priv : NULL;
} }
@ -565,15 +561,14 @@ static void ipmmu_domain_free(struct iommu_domain *io_domain)
static int ipmmu_attach_device(struct iommu_domain *io_domain, static int ipmmu_attach_device(struct iommu_domain *io_domain,
struct device *dev) struct device *dev)
{ {
struct ipmmu_vmsa_iommu_priv *priv = to_priv(dev);
struct iommu_fwspec *fwspec = dev->iommu_fwspec; struct iommu_fwspec *fwspec = dev->iommu_fwspec;
struct ipmmu_vmsa_device *mmu = priv->mmu; struct ipmmu_vmsa_device *mmu = to_ipmmu(dev);
struct ipmmu_vmsa_domain *domain = to_vmsa_domain(io_domain); struct ipmmu_vmsa_domain *domain = to_vmsa_domain(io_domain);
unsigned long flags; unsigned long flags;
unsigned int i; unsigned int i;
int ret = 0; int ret = 0;
if (!priv || !priv->mmu) { if (!mmu) {
dev_err(dev, "Cannot attach to IPMMU\n"); dev_err(dev, "Cannot attach to IPMMU\n");
return -ENXIO; return -ENXIO;
} }
@ -662,18 +657,12 @@ static int ipmmu_init_platform_device(struct device *dev,
struct of_phandle_args *args) struct of_phandle_args *args)
{ {
struct platform_device *ipmmu_pdev; struct platform_device *ipmmu_pdev;
struct ipmmu_vmsa_iommu_priv *priv;
ipmmu_pdev = of_find_device_by_node(args->np); ipmmu_pdev = of_find_device_by_node(args->np);
if (!ipmmu_pdev) if (!ipmmu_pdev)
return -ENODEV; return -ENODEV;
priv = kzalloc(sizeof(*priv), GFP_KERNEL); dev->iommu_fwspec->iommu_priv = platform_get_drvdata(ipmmu_pdev);
if (!priv)
return -ENOMEM;
priv->mmu = platform_get_drvdata(ipmmu_pdev);
dev->iommu_fwspec->iommu_priv = priv;
return 0; return 0;
} }
@ -683,7 +672,7 @@ static int ipmmu_of_xlate(struct device *dev,
iommu_fwspec_add_ids(dev, spec->args, 1); iommu_fwspec_add_ids(dev, spec->args, 1);
/* Initialize once - xlate() will call multiple times */ /* Initialize once - xlate() will call multiple times */
if (to_priv(dev)) if (to_ipmmu(dev))
return 0; return 0;
return ipmmu_init_platform_device(dev, spec); return ipmmu_init_platform_device(dev, spec);
@ -693,14 +682,14 @@ static int ipmmu_of_xlate(struct device *dev,
static int ipmmu_add_device(struct device *dev) static int ipmmu_add_device(struct device *dev)
{ {
struct ipmmu_vmsa_device *mmu = NULL; struct ipmmu_vmsa_device *mmu = to_ipmmu(dev);
struct iommu_group *group; struct iommu_group *group;
int ret; int ret;
/* /*
* Only let through devices that have been verified in xlate() * Only let through devices that have been verified in xlate()
*/ */
if (!to_priv(dev)) if (!mmu)
return -ENODEV; return -ENODEV;
/* Create a device group and add the device to it. */ /* Create a device group and add the device to it. */
@ -729,7 +718,6 @@ static int ipmmu_add_device(struct device *dev)
* - Make the mapping size configurable ? We currently use a 2GB mapping * - Make the mapping size configurable ? We currently use a 2GB mapping
* at a 1GB offset to ensure that NULL VAs will fault. * at a 1GB offset to ensure that NULL VAs will fault.
*/ */
mmu = to_priv(dev)->mmu;
if (!mmu->mapping) { if (!mmu->mapping) {
struct dma_iommu_mapping *mapping; struct dma_iommu_mapping *mapping;
@ -795,7 +783,7 @@ static int ipmmu_add_device_dma(struct device *dev)
/* /*
* Only let through devices that have been verified in xlate() * Only let through devices that have been verified in xlate()
*/ */
if (!to_priv(dev)) if (!to_ipmmu(dev))
return -ENODEV; return -ENODEV;
group = iommu_group_get_for_dev(dev); group = iommu_group_get_for_dev(dev);
@ -812,15 +800,15 @@ static void ipmmu_remove_device_dma(struct device *dev)
static struct iommu_group *ipmmu_find_group(struct device *dev) static struct iommu_group *ipmmu_find_group(struct device *dev)
{ {
struct ipmmu_vmsa_iommu_priv *priv = to_priv(dev); struct ipmmu_vmsa_device *mmu = to_ipmmu(dev);
struct iommu_group *group; struct iommu_group *group;
if (priv->mmu->group) if (mmu->group)
return iommu_group_ref_get(priv->mmu->group); return iommu_group_ref_get(mmu->group);
group = iommu_group_alloc(); group = iommu_group_alloc();
if (!IS_ERR(group)) if (!IS_ERR(group))
priv->mmu->group = group; mmu->group = group;
return group; return group;
} }