On 10/24/22 23:17, Colin Ian King wrote:
> Variable num_processed is just being incremented and it's never used
> anywhere else. The variable and the increment are redundant so
> remove it.
>
> Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx>
Applied to for-6.2. Thanks !
> ---
> drivers/ata/sata_dwc_460ex.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/ata/sata_dwc_460ex.c b/drivers/ata/sata_dwc_460ex.c
> index e3263e961045..fd699c5bfc34 100644
> --- a/drivers/ata/sata_dwc_460ex.c
> +++ b/drivers/ata/sata_dwc_460ex.c
> @@ -472,7 +472,7 @@ static irqreturn_t sata_dwc_isr(int irq, void *dev_instance)
> struct ata_queued_cmd *qc;
> unsigned long flags;
> u8 status, tag;
> - int handled, num_processed, port = 0;
> + int handled, port = 0;
> uint intpr, sactive, sactive2, tag_mask;
> struct sata_dwc_device_port *hsdevp;
> hsdev->sactive_issued = 0;
> @@ -618,9 +618,7 @@ static irqreturn_t sata_dwc_isr(int irq, void *dev_instance)
> dev_dbg(ap->dev, "%s ATA status register=0x%x\n", __func__, status);
>
> tag = 0;
> - num_processed = 0;
> while (tag_mask) {
> - num_processed++;
> while (!(tag_mask & 0x00000001)) {
> tag++;
> tag_mask <<= 1;
--
Damien Le Moal
Western Digital Research
