Re: [PATCH V5 4/5] iommu/dma-iommu: Use the dev->coherent_dma_mask

To: Tom Murphy <murphyt7@xxxxxx>
Subject: Re: [PATCH V5 4/5] iommu/dma-iommu: Use the dev->coherent_dma_mask
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Tue, 20 Aug 2019 02:43:24 -0700
Cc: iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx, Heiko Stuebner <heiko@xxxxxxxxx>, virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx, linux-tegra@xxxxxxxxxxxxxxx, Thierry Reding <thierry.reding@xxxxxxxxx>, Will Deacon <will@xxxxxxxxxx>, Jean-Philippe Brucker <jean-philippe@xxxxxxxxxx>, linux-samsung-soc@xxxxxxxxxxxxxxx, Krzysztof Kozlowski <krzk@xxxxxxxxxx>, Jonathan Hunter <jonathanh@xxxxxxxxxx>, linux-rockchip@xxxxxxxxxxxxxxxxxxx, Andy Gross <agross@xxxxxxxxxx>, Gerald Schaefer <gerald.schaefer@xxxxxxxxxx>, linux-s390@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, linux-mediatek@xxxxxxxxxxxxxxxxxxx, Matthias Brugger <matthias.bgg@xxxxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, David Woodhouse <dwmw2@xxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, Kukjin Kim <kgene@xxxxxxxxxx>, Robin Murphy <robin.murphy@xxxxxxx>
In-reply-to: <20190815110944.3579-5-murphyt7@tcd.ie>
References: <20190815110944.3579-1-murphyt7@tcd.ie> <20190815110944.3579-5-murphyt7@tcd.ie>
User-agent: Mutt/1.11.4 (2019-03-13)

Looks good, and should probably be queued up asap as a bug fix:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH V5 0/5] iommu/amd: Convert the AMD iommu driver to the dma-iommu api
  - From: Tom Murphy
- [PATCH V5 4/5] iommu/dma-iommu: Use the dev->coherent_dma_mask
  - From: Tom Murphy