diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-04-06 21:38:56 +0300 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2016-04-07 15:51:47 +0300 |
| commit | 0b74ecdfbea893ae585bc694136a3f50f5856cfe (patch) | |
| tree | bf4be392163f8fa2ca69b5597536c523d78e5779 /drivers/iommu | |
| parent | fbedd9b9905c1643b9f7244d88999e39632bbd87 (diff) | |
| download | linux-0b74ecdfbea893ae585bc694136a3f50f5856cfe.tar.xz | |
iommu/vt-d: Silence an uninitialized variable warning
My static checker complains that "dma_alias" is uninitialized unless we
are dealing with a pci device. This is true but harmless. Anyway, we
can flip the condition around to silence the warning.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/intel-iommu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c index a2e1b7f14df2..e1852e845d21 100644 --- a/drivers/iommu/intel-iommu.c +++ b/drivers/iommu/intel-iommu.c @@ -2458,7 +2458,7 @@ static struct dmar_domain *get_domain_for_dev(struct device *dev, int gaw) } /* register PCI DMA alias device */ - if (req_id != dma_alias && dev_is_pci(dev)) { + if (dev_is_pci(dev) && req_id != dma_alias) { tmp = dmar_insert_one_dev_info(iommu, PCI_BUS_NUM(dma_alias), dma_alias & 0xff, NULL, domain); |