[PATCH] iommu/arm-smmu: remove redundant assignment to variable res
Robin Murphy
robin.murphy at arm.com
Fri Feb 25 06:32:19 PST 2022
On 2022-02-25 09:32, Colin Ian King wrote:
> The variable res is being assigned a value that isn't being read
> later. The assignment is redundant and can be removed.
>
> Cleans up clang scan warning:
> drivers/iommu/arm/arm-smmu/arm-smmu.c:2109:10: warning: Although the
> value stored to 'res' is used in the enclosing expression, the value
> is never actually read from 'res' [deadcode.DeadStores]
Thanks Colin, however in general we need to get rid of this
platform_get_resource() call ASAP anyway, so Will should hopefully be
picking up either [1] or [2] for 5.18.
Cheers,
Robin.
[1]
https://lore.kernel.org/linux-arm-kernel/b2a40caaf1622eb35c555074a0d72f4f0513cff9.1645106346.git.robin.murphy@arm.com/
[2]
https://lore.kernel.org/linux-arm-kernel/20211223130046.9365-2-prabhakar.mahadev-lad.rj@bp.renesas.com/
> Signed-off-by: Colin Ian King <colin.i.king at gmail.com>
> ---
> drivers/iommu/arm/arm-smmu/arm-smmu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iommu/arm/arm-smmu/arm-smmu.c b/drivers/iommu/arm/arm-smmu/arm-smmu.c
> index 4bc75c4ce402..f83d2c32b5a9 100644
> --- a/drivers/iommu/arm/arm-smmu/arm-smmu.c
> +++ b/drivers/iommu/arm/arm-smmu/arm-smmu.c
> @@ -2106,7 +2106,7 @@ static int arm_smmu_device_probe(struct platform_device *pdev)
> return PTR_ERR(smmu);
>
> num_irqs = 0;
> - while ((res = platform_get_resource(pdev, IORESOURCE_IRQ, num_irqs))) {
> + while (platform_get_resource(pdev, IORESOURCE_IRQ, num_irqs)) {
> num_irqs++;
> if (num_irqs > smmu->num_global_irqs)
> smmu->num_context_irqs++;
More information about the linux-arm-kernel
mailing list