Re: [PATCH v1 1/3] xen/arm: smmu: Rename arm_smmu_xen_device with, device_iommu_info

[Julien Grall <julien.grall@linaro.org>]

Hi Manish,

On 27/03/15 07:20, Manish Jaggi wrote:
> arm_smmu_xen_device is not an intuitive name for a datastructure which
> represents
> device->archdata.iommu. Rename arm_smmu_xen_device with device_iommu_info

device_iommu_info is not more intuitive... At least arm_smmu_xen_device
shows that it's a specific Xen structure and not coming from the Linux
drivers.

Regards,

> Signed-off-by: Manish Jaggi <manish.jaggi@caviumnetworks.com>
> ---
>  xen/drivers/passthrough/arm/smmu.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/xen/drivers/passthrough/arm/smmu.c
> b/xen/drivers/passthrough/arm/smmu.c
> index a7a7da9..ab4f7a4 100644
> --- a/xen/drivers/passthrough/arm/smmu.c
> +++ b/xen/drivers/passthrough/arm/smmu.c
> @@ -247,12 +247,12 @@ struct arm_smmu_xen_domain {
>   * that would require to move some hackery (dummy iommu_group) in a
> more generic
>   * place.
>   * */
> -struct arm_smmu_xen_device {
> +struct device_iommu_info {
>      struct iommu_domain *domain;
>      struct iommu_group *group;
>  };
> 
> -#define dev_archdata(dev) ((struct arm_smmu_xen_device
> *)dev->archdata.iommu)
> +#define dev_archdata(dev) ((struct device_iommu_info
> *)dev->archdata.iommu)
>  #define dev_iommu_domain(dev) (dev_archdata(dev)->domain)
>  #define dev_iommu_group(dev) (dev_archdata(dev)->group)
> 
> @@ -2574,7 +2574,7 @@ static int arm_smmu_assign_dev(struct domain *d,
> u8 devfn,
>      xen_domain = domain_hvm_iommu(d)->arch.priv;
> 
>      if (!dev->archdata.iommu) {
> -        dev->archdata.iommu = xzalloc(struct arm_smmu_xen_device);
> +        dev->archdata.iommu = xzalloc(struct device_iommu_info);
>          if (!dev->archdata.iommu)
>              return -ENOMEM;
>      }


-- 
Julien Grall