diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c
index 8588e8562220..e25df9990f82 100644
--- a/drivers/spi/spi.c
+++ b/drivers/spi/spi.c
@@ -2857,6 +2857,8 @@ static acpi_status acpi_register_spi_device(struct spi_controller *ctlr,
 	 * here too, because this call sets the GPIO direction and/or bias.
 	 * Setting these needs to be done even if there is no driver, in which
 	 * case spi_probe() will never get called.
+	 * TODO: ideally the setup of the GPIO should be handled in a generic
+	 * manner in the ACPI/gpiolib core code.
 	 */
 	if (spi->irq < 0)
 		spi->irq = acpi_dev_gpio_irq_get(adev, 0);