Received-SPF: pass (google.com: domain of linux-kernel+bounces-156235-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <6a08e3ea-572d-442b-9b58-7da0b0ad212e@linux.intel.com>
Date: Wed, 24 Apr 2024 11:45:51 +0800
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Cc: baolu.lu@linux.intel.com, kernel-janitors@vger.kernel.org,
 linux-kernel@vger.kernel.org
Subject: Re: [PATCH][next] iommu/vt-d: remove redundant assignment to variable
 err
To: Colin Ian King <colin.i.king@gmail.com>,
 David Woodhouse <dwmw2@infradead.org>, Joerg Roedel <joro@8bytes.org>,
 Will Deacon <will@kernel.org>, Robin Murphy <robin.murphy@arm.com>,
 iommu@lists.linux.dev
References: <20240411090535.306326-1-colin.i.king@gmail.com>
Content-Language: en-US
From: Baolu Lu <baolu.lu@linux.intel.com>
In-Reply-To: <20240411090535.306326-1-colin.i.king@gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

On 4/11/24 5:05 PM, Colin Ian King wrote:
> Variable err is being assigned a value that is never read. It is
> either being re-assigned later on error exit paths, or never referenced
> on the non-error path.
> 
> Cleans up clang scan build warning:
> drivers/iommu/intel/dmar.c:1070:2: warning: Value stored to 'err' is
> never read [deadcode.DeadStores]`
> 
> Signed-off-by: Colin Ian King<colin.i.king@gmail.com>
> ---
>   drivers/iommu/intel/dmar.c | 1 -
>   1 file changed, 1 deletion(-)

Patch has been queued for iommu/vt-d.

Best regards,
baolu