Re: [PATCH 03/15] drm/msm/mdp5: Remove direct access of dev->iommu_fwspec

From: Jean-Philippe Brucker
Date: Mon Mar 16 2020 - 11:42:20 EST


On Tue, Mar 10, 2020 at 10:12:17AM +0100, Joerg Roedel wrote:
> From: Joerg Roedel <jroedel@xxxxxxx>
>
> Use the accessor functions instead of directly dereferencing
> dev->iommu_fwspec.
>
> Signed-off-by: Joerg Roedel <jroedel@xxxxxxx>

Reviewed-by: Jean-Philippe Brucker <jean-philippe@xxxxxxxxxx>

> ---
> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> index e43ecd4be10a..1252e1d76340 100644
> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c
> @@ -725,7 +725,7 @@ struct msm_kms *mdp5_kms_init(struct drm_device *dev)
>
> if (config->platform.iommu) {
> iommu_dev = &pdev->dev;
> - if (!iommu_dev->iommu_fwspec)
> + if (!dev_iommu_fwspec_get(iommu_dev))
> iommu_dev = iommu_dev->parent;
>
> aspace = msm_gem_address_space_create(iommu_dev,
> --
> 2.17.1
>