]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
gpio: mxc: remove dead code after switch to DT-only
authorAhmad Fatoum <a.fatoum@pengutronix.de>
Mon, 13 Jan 2025 22:19:11 +0000 (23:19 +0100)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Thu, 23 Jan 2025 07:44:51 +0000 (08:44 +0100)
struct platform_device::id was only set by board code, but since i.MX
became a devicetree-only platform, this will always be -1
(PLATFORM_DEVID_NONE).

Note: of_alias_get_id() returns a negative number on error and base
treats all negative errors the same, so we need not add any additional
error handling.

Fixes: 0f2c7af45d7e ("gpio: mxc: Convert the driver to DT-only")
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Link: https://lore.kernel.org/r/20250113-b4-imx-gpio-base-warning-v1-3-0a28731a5cf6@pengutronix.de
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
drivers/gpio/gpio-mxc.c

index 4cb455b2bdee71ba4eb20c93567c3b8db100dbb2..619b6fb9d833a4bb94a93b4209f01b49ad1cbdb0 100644 (file)
@@ -490,8 +490,7 @@ static int mxc_gpio_probe(struct platform_device *pdev)
        port->gc.request = mxc_gpio_request;
        port->gc.free = mxc_gpio_free;
        port->gc.to_irq = mxc_gpio_to_irq;
-       port->gc.base = (pdev->id < 0) ? of_alias_get_id(np, "gpio") * 32 :
-                                            pdev->id * 32;
+       port->gc.base = of_alias_get_id(np, "gpio") * 32;
 
        err = devm_gpiochip_add_data(&pdev->dev, &port->gc, port);
        if (err)