gpio: tegra186: remove unneeded loop in tegra186_gpio_init_route_mapping()

Reviewing the j loop over num_irqs_per_bank, in the code previous
to the fixes: commit, every j was used. now only when j == 0.
If only j == 0 is used, there is no need for the loop.

Fixes: 210386804745 ("gpio: tegra186: Support multiple interrupts per bank")
Signed-off-by: Tom Rix <trix@redhat.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

diff --git a/drivers/gpio/gpio-tegra186.c b/drivers/gpio/gpio-tegra186.c
index 9941f35af82359aeec02d216e87f4325438a5644..14c872b6ad054508ac8702cf0ebecdb7dc67c9bc 100644 (file)
--- a/drivers/gpio/gpio-tegra186.c
+++ b/drivers/gpio/gpio-tegra186.c
@@ -677,7 +677,7 @@ static const struct of_device_id tegra186_pmc_of_match[] = {
 static void tegra186_gpio_init_route_mapping(struct tegra_gpio *gpio)
 {
        struct device *dev = gpio->gpio.parent;
-       unsigned int i, j;
+       unsigned int i;
        u32 value;
 
        for (i = 0; i < gpio->soc->num_ports; i++) {
@@ -699,27 +699,23 @@ static void tegra186_gpio_init_route_mapping(struct tegra_gpio *gpio)
                         * On Tegra194 and later, each pin can be routed to one or more
                         * interrupts.
                         */
-                       for (j = 0; j < gpio->num_irqs_per_bank; j++) {
-                               dev_dbg(dev, "programming default interrupt routing for port %s\n",
-                                       port->name);
-
-                               offset = TEGRA186_GPIO_INT_ROUTE_MAPPING(p, j);
-
-                               /*
-                                * By default we only want to route GPIO pins to IRQ 0. This works
-                                * only under the assumption that we're running as the host kernel
-                                * and hence all GPIO pins are owned by Linux.
-                                *
-                                * For cases where Linux is the guest OS, the hypervisor will have
-                                * to configure the interrupt routing and pass only the valid
-                                * interrupts via device tree.
-                                */
-                               if (j == 0) {
-                                       value = readl(base + offset);
-                                       value = BIT(port->pins) - 1;
-                                       writel(value, base + offset);
-                               }
-                       }
+                       dev_dbg(dev, "programming default interrupt routing for port %s\n",
+                               port->name);
+
+                       offset = TEGRA186_GPIO_INT_ROUTE_MAPPING(p, 0);
+
+                       /*
+                        * By default we only want to route GPIO pins to IRQ 0. This works
+                        * only under the assumption that we're running as the host kernel
+                        * and hence all GPIO pins are owned by Linux.
+                        *
+                        * For cases where Linux is the guest OS, the hypervisor will have
+                        * to configure the interrupt routing and pass only the valid
+                        * interrupts via device tree.
+                        */
+                       value = readl(base + offset);
+                       value = BIT(port->pins) - 1;
+                       writel(value, base + offset);
                }
        }
 }