Commit d1bae819 authored by Tomer Tayar's avatar Tomer Tayar Committed by Oded Gabbay
Browse files

accel/habanalabs: remove unneeded irq_handler variable 



'irq_handler' in gaudi2_enable_msix(), is just assigned with a function
name and then used when calling request_threaded_irq().
Remove the variable and use the function name directly as an argument.

Signed-off-by: default avatarTomer Tayar <ttayar@habana.ai>
Reviewed-by: default avatarOded Gabbay <ogabbay@kernel.org>
Signed-off-by: default avatarOded Gabbay <ogabbay@kernel.org>
Reviewed-by: default avatarStanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
parent 5e09ae92

drivers/accel/habanalabs/gaudi2/gaudi2.c

+3 −5
Original line number Diff line number Diff line
@@ -3974,7 +3974,6 @@ static int gaudi2_enable_msix(struct hl_device *hdev) 
	struct asic_fixed_properties *prop = &hdev->asic_prop;

	struct gaudi2_device *gaudi2 = hdev->asic_specific;

	int rc, irq, i, j, user_irq_init_cnt;

	irq_handler_t irq_handler;

	struct hl_cq *cq;



	if (gaudi2->hw_cap_initialized & HW_CAP_MSIX)
@@ -4024,10 +4023,9 @@ static int gaudi2_enable_msix(struct hl_device *hdev) 
			i++, j++, user_irq_init_cnt++) {



		irq = pci_irq_vector(hdev->pdev, i);

		irq_handler = hl_irq_handler_user_interrupt;



		rc = request_threaded_irq(irq, irq_handler, hl_irq_user_interrupt_thread_handler,

				IRQF_ONESHOT, gaudi2_irq_name(i), &hdev->user_interrupt[j]);

		rc = request_threaded_irq(irq, hl_irq_handler_user_interrupt,

						hl_irq_user_interrupt_thread_handler, IRQF_ONESHOT,

						gaudi2_irq_name(i), &hdev->user_interrupt[j]);



		if (rc) {

			dev_err(hdev->dev, "Failed to request IRQ %d", irq);