Merge tag 'omap-for-v4.20/omap1-signed' of git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap into next/soc

A series of omap1 gpio changes for ams-delta

Janusz Krzysztofik has cleaned up ams-delta gpio usage along with
generic gpio framework improvments. This series contains the omap1
specific clean-up for ams-delta modem and unused gpios.

Note that this conflicts with the gpio-omap changes queued into
an immutable gpio branch ib-omap for the gpio-omap.h header file.
The merge resolution is to drop the IS_BUILTIN(CONFIG_GPIO_OMAP)
section and keep the #endif tagged for __ASSEMBLER__.

* tag 'omap-for-v4.20/omap1-signed' of git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap:
  ARM: OMAP1: ams-delta: Don't request unused GPIOs
  ARM: OMAP1: ams-delta-fiq: Use <linux/platform_data/gpio-omap.h>
  ARM: OMAP1: ams-delta: register MODEM device earlier
  ARM: OMAP1: ams-delta: initialize latch2 pins to safe values
  ARM: OMAP1: ams-delta: assign MODEM IRQ from GPIO descriptor

Signed-off-by: Arnd Bergmann <arnd@arndb.de>

4 files changed, 96 insertions, 88 deletions

diff --git a/arch/arm/mach-omap1/ams-delta-fiq-handler.S b/arch/arm/mach-omap1/ams-delta-fiq-handler.S
index ddc27638ba2a..e3faa0274b56 100644
--- a/arch/arm/mach-omap1/ams-delta-fiq-handler.S
+++ b/arch/arm/mach-omap1/ams-delta-fiq-handler.S
@@ -15,6 +15,7 @@
 
 #include <linux/linkage.h>
 #include <linux/platform_data/ams-delta-fiq.h>
+#include <linux/platform_data/gpio-omap.h>
 
 #include <asm/assembler.h>
 #include <mach/board-ams-delta.h>
@@ -24,17 +25,10 @@
 #include "soc.h"
 
 /*
- * GPIO related definitions, copied from arch/arm/plat-omap/gpio.c.
- * Unfortunately, those were not placed in a separate header file.
+ * OMAP1510 GPIO related symbol copied from arch/arm/mach-omap1/gpio15xx.c.
+ * Unfortunately, it was not placed in a separate header file.
  */
 #define OMAP1510_GPIO_BASE              0xFFFCE000
-#define OMAP1510_GPIO_DATA_INPUT        0x00
-#define OMAP1510_GPIO_DATA_OUTPUT       0x04
-#define OMAP1510_GPIO_DIR_CONTROL       0x08
-#define OMAP1510_GPIO_INT_CONTROL       0x0c
-#define OMAP1510_GPIO_INT_MASK          0x10
-#define OMAP1510_GPIO_INT_STATUS        0x14
-#define OMAP1510_GPIO_PIN_CONTROL       0x18
 
 /* GPIO register bitmasks */
 #define KEYBRD_DATA_MASK                (0x1 << AMS_DELTA_GPIO_PIN_KEYBRD_DATA)
diff --git a/arch/arm/mach-omap1/board-ams-delta.c b/arch/arm/mach-omap1/board-ams-delta.c
index dd28d2614d7f..318925ae3ebe 100644
--- a/arch/arm/mach-omap1/board-ams-delta.c
+++ b/arch/arm/mach-omap1/board-ams-delta.c
@@ -250,39 +250,6 @@ static struct platform_device latch2_gpio_device = {
 #define LATCH2_PIN_HOOKFLASH1           14
 #define LATCH2_PIN_HOOKFLASH2           15
 
-static const struct gpio latch_gpios[] __initconst = {
-        {
-                .gpio   = LATCH1_GPIO_BASE + 6,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "dockit1",
-        },
-        {
-                .gpio   = LATCH1_GPIO_BASE + 7,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "dockit2",
-        },
-        {
-                .gpio   = AMS_DELTA_GPIO_PIN_SCARD_RSTIN,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "scard_rstin",
-        },
-        {
-                .gpio   = AMS_DELTA_GPIO_PIN_SCARD_CMDVCC,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "scard_cmdvcc",
-        },
-        {
-                .gpio   = AMS_DELTA_LATCH2_GPIO_BASE + 14,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "hookflash1",
-        },
-        {
-                .gpio   = AMS_DELTA_LATCH2_GPIO_BASE + 15,
-                .flags  = GPIOF_OUT_INIT_LOW,
-                .label  = "hookflash2",
-        },
-};
-
 static struct regulator_consumer_supply modem_nreset_consumers[] = {
         REGULATOR_SUPPLY("RESET#", "serial8250.1"),
         REGULATOR_SUPPLY("POR", "cx20442-codec"),
@@ -321,20 +288,6 @@ struct modem_private_data {
 
 static struct modem_private_data modem_priv;
 
-void ams_delta_latch_write(int base, int ngpio, u16 mask, u16 value)
-{
-        int bit = 0;
-        u16 bitpos = 1 << bit;
-
-        for (; bit < ngpio; bit++, bitpos = bitpos << 1) {
-                if (!(mask & bitpos))
-                        continue;
-                else
-                        gpio_set_value(base + bit, (value & bitpos) != 0);
-        }
-}
-EXPORT_SYMBOL(ams_delta_latch_write);
-
 static struct resource ams_delta_nand_resources[] = {
         [0] = {
                 .start  = OMAP1_MPUIO_BASE,
@@ -630,6 +583,28 @@ static struct gpiod_hog ams_delta_gpio_hogs[] = {
         {},
 };
 
+static struct plat_serial8250_port ams_delta_modem_ports[];
+
+/*
+ * Obtain MODEM IRQ GPIO descriptor using its hardware pin
+ * number and assign related IRQ number to the MODEM port.
+ * Keep the GPIO descriptor open so nobody steps in.
+ */
+static void __init modem_assign_irq(struct gpio_chip *chip)
+{
+        struct gpio_desc *gpiod;
+
+        gpiod = gpiochip_request_own_desc(chip, AMS_DELTA_GPIO_PIN_MODEM_IRQ,
+                                          "modem_irq");
+        if (IS_ERR(gpiod)) {
+                pr_err("%s: modem IRQ GPIO request failed (%ld)\n", __func__,
+                       PTR_ERR(gpiod));
+        } else {
+                gpiod_direction_input(gpiod);
+                ams_delta_modem_ports[0].irq = gpiod_to_irq(gpiod);
+        }
+}
+
 /*
  * The purpose of this function is to take care of proper initialization of
  * devices and data structures which depend on GPIO lines provided by OMAP GPIO
@@ -649,7 +624,47 @@ static void __init omap_gpio_deps_init(void)
                 return;
         }
 
+        /*
+         * Start with FIQ initialization as it may have to request
+         * and release successfully each OMAP GPIO pin in turn.
+         */
         ams_delta_init_fiq(chip, &ams_delta_serio_device);
+
+        modem_assign_irq(chip);
+}
+
+/*
+ * Initialize latch2 pins with values which are safe for dependent on-board
+ * devices or useful for their successull initialization even before GPIO
+ * driver takes control over the latch pins:
+ * - LATCH2_PIN_LCD_VBLEN       = 0
+ * - LATCH2_PIN_LCD_NDISP       = 0     Keep LCD device powered off before its
+ *                                      driver takes control over it.
+ * - LATCH2_PIN_NAND_NCE        = 0
+ * - LATCH2_PIN_NAND_NWP        = 0     Keep NAND device down and write-
+ *                                      protected before its driver takes
+ *                                      control over it.
+ * - LATCH2_PIN_KEYBRD_PWR      = 0     Keep keyboard powered off before serio
+ *                                      driver takes control over it.
+ * - LATCH2_PIN_KEYBRD_DATAOUT  = 0     Keep low to avoid corruption of first
+ *                                      byte of data received from attached
+ *                                      keyboard when serio device is probed;
+ *                                      the pin is also hogged low by the latch2
+ *                                      GPIO driver as soon as it is ready.
+ * - LATCH2_PIN_MODEM_NRESET    = 1     Enable voice MODEM device, allowing for
+ *                                      its successful probe even before a
+ *                                      regulator it depends on, which in turn
+ *                                      takes control over the pin, is set up.
+ * - LATCH2_PIN_MODEM_CODEC     = 1     Attach voice MODEM CODEC data port
+ *                                      to the MODEM so the CODEC is under
+ *                                      control even if audio driver doesn't
+ *                                      take it over.
+ */
+static void __init ams_delta_latch2_init(void)
+{
+        u16 latch2 = 1 << LATCH2_PIN_MODEM_NRESET | 1 << LATCH2_PIN_MODEM_CODEC;
+
+        __raw_writew(latch2, LATCH2_VIRT);
 }
 
 static void __init ams_delta_init(void)
@@ -673,6 +688,7 @@ static void __init ams_delta_init(void)
         omap_cfg_reg(J18_1610_CAM_D7);