The error log used the loop index 'i' outside the loop when reporting a
config IRQ request failure, which is misleading. Replace it with 'queues'
which represents the config vector and rephrase the log message.

This is a non-functional change that improves readability and clarifies
the code intent.

Signed-off-by: Alok Tiwari <[email protected]>
---
v1 -> v2
target for next and print queues instead of ret
---
 drivers/vdpa/alibaba/eni_vdpa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/vdpa/alibaba/eni_vdpa.c b/drivers/vdpa/alibaba/eni_vdpa.c
index e476504db0c8..d8f8406ade98 100644
--- a/drivers/vdpa/alibaba/eni_vdpa.c
+++ b/drivers/vdpa/alibaba/eni_vdpa.c
@@ -196,7 +196,7 @@ static int eni_vdpa_request_irq(struct eni_vdpa *eni_vdpa)
        ret = devm_request_irq(&pdev->dev, irq, eni_vdpa_config_handler, 0,
                               eni_vdpa->msix_name, eni_vdpa);
        if (ret) {
-               ENI_ERR(pdev, "failed to request irq for config vq %d\n", i);
+               ENI_ERR(pdev, "failed to request irq %d for config\n", queues);
                goto err;
        }
        vp_legacy_config_vector(ldev, queues);
-- 
2.50.1


Reply via email to