diff options
| author | Joerg Roedel <jroedel@suse.de> | 2021-04-15 17:44:42 +0300 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2021-04-15 18:05:23 +0300 |
| commit | 84b6269c3c28b920088d4d5ae3723dc740221ae9 (patch) | |
| tree | 5682b4d0d1a1edf30c10549753b5786f992ce239 /drivers/iommu | |
| parent | 7876a83ffe8c23c7049a63c747a7b96cafaf10a4 (diff) | |
| download | linux-84b6269c3c28b920088d4d5ae3723dc740221ae9.tar.xz | |
iommu/fsl-pamu: Fix uninitialized variable warning
The variable 'i' in the function update_liodn_stash() is not
initialized and only used in a debug printk(). So it has no
meaning at all, remove it.
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Link: https://lore.kernel.org/r/20210415144442.25103-1-joro@8bytes.org
Diffstat (limited to 'drivers/iommu')
| -rw-r--r-- | drivers/iommu/fsl_pamu_domain.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/iommu/fsl_pamu_domain.c b/drivers/iommu/fsl_pamu_domain.c index 32944d67baa4..0ac781186dbd 100644 --- a/drivers/iommu/fsl_pamu_domain.c +++ b/drivers/iommu/fsl_pamu_domain.c @@ -57,14 +57,13 @@ static int __init iommu_init_mempool(void) static int update_liodn_stash(int liodn, struct fsl_dma_domain *dma_domain, u32 val) { - int ret = 0, i; + int ret = 0; unsigned long flags; spin_lock_irqsave(&iommu_lock, flags); ret = pamu_update_paace_stash(liodn, val); if (ret) { - pr_debug("Failed to update SPAACE %d field for liodn %d\n ", - i, liodn); + pr_debug("Failed to update SPAACE for liodn %d\n ", liodn); spin_unlock_irqrestore(&iommu_lock, flags); return ret; } |