From nobody Wed Sep 10 06:05:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 62BF7C7EE31 for ; Wed, 1 Mar 2023 12:58:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229943AbjCAM6v (ORCPT ); Wed, 1 Mar 2023 07:58:51 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57392 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229935AbjCAM6t (ORCPT ); Wed, 1 Mar 2023 07:58:49 -0500 Received: from mail-ot1-x331.google.com (mail-ot1-x331.google.com [IPv6:2607:f8b0:4864:20::331]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 58D301A647 for ; Wed, 1 Mar 2023 04:58:48 -0800 (PST) Received: by mail-ot1-x331.google.com with SMTP id q11-20020a056830440b00b00693c1a62101so7493460otv.0 for ; Wed, 01 Mar 2023 04:58:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1677675527; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=OyK1NRjw/4c0nP3WiY5DJv3QaTW6QxlnmL7lL+zTtyk=; b=bDQWML8wSwB8+GUWJ/35tvSFmVYqnPkC/OFQvZIdEh32mMSq0HNtBFMxDZ7RnPqofX H1ECS406rlQGoCL+5FDOwjO7JYec/+r+KUF1lfwW2Z6yh4q5lZDcCSkd13dLyatmAjoX 92+0l4f7TnUpLtpdVLsChIsWC/Aa4lke3jeIFRoxVkS7Ofqi9WfrXqpBAqUUjF72YRpF J7NfohRIiscqEMjhEoOsQLWhEjWwurkWxWAxy1Xnc2J9Ds90yvCqwz/tPKg+cqBjPxxN aDJMW+parsmOUvvmg+gu8KuBmJ2iFdS9KKe4m8IZ1GPU1uOtVw+ScjATIVX4NTqFha14 2Mkg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1677675527; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=OyK1NRjw/4c0nP3WiY5DJv3QaTW6QxlnmL7lL+zTtyk=; b=IcLlBbwECLuR9WUfysb9H+57EKZwWaSIaFNuZUMcC2V6cKGRZu9P12xSmc8bTVe6Xn lgjlFK/Dg0ordx0lIm7ksjc9uwmbiavzb2EOPMb9F3Cfu3ldSZ+aWYBzxoQ1Z+vzcoT0 HfPKrxBfAQY7A0jQj1yZzzSW8NxrUJ7I/712R/Tux8JrjVtaMUlvqpTDFDsazzXSx9jT x846fYMyMNAdj398uGMsdXc2GfgOslf8/PRg+HSwD1NwB7URgAXM6Zk85zNHbM62wuCG UcI/3M2FHhfYoNDtOkKYhDLe9E3EpoH+y+Lq+VaLC2q8T/6YDGgR+ju/BsthxYXbNwBt gVXw== X-Gm-Message-State: AO0yUKU41gzcVDzE6Gl/4WGY5Hxr5Xxg7jvKypb8b9/QqOilJvP4+IaT 4UQnVLt9mLgrT5DZZA6Ph5J9xw== X-Google-Smtp-Source: AK7set/dVBq5ekwOQCiM49scymI8WZzuOpGkFc5nXsYLFQ8q2ASWJRNLwosukPEax+QDm9VD2Sma2Q== X-Received: by 2002:a9d:71db:0:b0:68b:ba4d:1c94 with SMTP id z27-20020a9d71db000000b0068bba4d1c94mr1984390otj.4.1677675527667; Wed, 01 Mar 2023 04:58:47 -0800 (PST) Received: from fedora.attlocal.net (69-109-179-158.lightspeed.dybhfl.sbcglobal.net. [69.109.179.158]) by smtp.gmail.com with ESMTPSA id g21-20020a056830309500b0068bc48c61a5sm4865064ots.19.2023.03.01.04.58.46 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Mar 2023 04:58:47 -0800 (PST) From: William Breathitt Gray To: linus.walleij@linaro.org, brgl@bgdev.pl Cc: broonie@kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, quarium@gmail.com, jhentges@accesio.com, jay.dolan@accesio.com, William Breathitt Gray Subject: [PATCH v2 1/2] regmap: Pass regmap and irq_drv_data as parameters for set_type_config() Date: Mon, 27 Feb 2023 22:56:27 -0500 Message-Id: X-Mailer: git-send-email 2.39.2 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" Allow struct regmap_irq_chip set_type_config() callbacks to access the device regmap and irq_drv_data by passing them as parameters. Signed-off-by: William Breathitt Gray --- Changes in v2: none drivers/base/regmap/regmap-irq.c | 13 +++++++++---- include/linux/regmap.h | 12 ++++++++---- 2 files changed, 17 insertions(+), 8 deletions(-) diff --git a/drivers/base/regmap/regmap-irq.c b/drivers/base/regmap/regmap-= irq.c index a8f185430a07..eac55a3af6d9 100644 --- a/drivers/base/regmap/regmap-irq.c +++ b/drivers/base/regmap/regmap-irq.c @@ -333,8 +333,9 @@ static int regmap_irq_set_type(struct irq_data *data, u= nsigned int type) } =20 if (d->chip->set_type_config) { - ret =3D d->chip->set_type_config(d->config_buf, type, - irq_data, reg); + ret =3D d->chip->set_type_config(map, d->config_buf, type, + irq_data, reg, + d->chip->irq_drv_data); if (ret) return ret; } @@ -650,18 +651,22 @@ EXPORT_SYMBOL_GPL(regmap_irq_get_irq_reg_linear); =20 /** * regmap_irq_set_type_config_simple() - Simple IRQ type configuration cal= lback. + * @map: The regmap for the device. * @buf: Buffer containing configuration register values, this is a 2D arr= ay of * `num_config_bases` rows, each of `num_config_regs` elements. * @type: The requested IRQ type. * @irq_data: The IRQ being configured. * @idx: Index of the irq's config registers within each array `buf[i]` + * @irq_drv_data: Driver specific IRQ data * * This is a &struct regmap_irq_chip->set_type_config callback suitable for * chips with one config register. Register values are updated according to * the &struct regmap_irq_type data associated with an IRQ. */ -int regmap_irq_set_type_config_simple(unsigned int **buf, unsigned int typ= e, - const struct regmap_irq *irq_data, int idx) +int regmap_irq_set_type_config_simple(struct regmap *map, unsigned int **b= uf, + unsigned int type, + const struct regmap_irq *irq_data, + int idx, void *irq_drv_data) { const struct regmap_irq_type *t =3D &irq_data->type; =20 diff --git a/include/linux/regmap.h b/include/linux/regmap.h index a3bc695bcca0..49073f5ae87a 100644 --- a/include/linux/regmap.h +++ b/include/linux/regmap.h @@ -1648,8 +1648,10 @@ struct regmap_irq_chip { unsigned int mask_buf, void *irq_drv_data); int (*set_type_virt)(unsigned int **buf, unsigned int type, unsigned long hwirq, int reg); - int (*set_type_config)(unsigned int **buf, unsigned int type, - const struct regmap_irq *irq_data, int idx); + int (*set_type_config)(struct regmap *map, unsigned int **buf, + unsigned int type, + const struct regmap_irq *irq_data, int idx, + void *irq_drv_data); unsigned int (*get_irq_reg)(struct regmap_irq_chip_data *data, unsigned int base, int index); void *irq_drv_data; @@ -1657,8 +1659,10 @@ struct regmap_irq_chip { =20 unsigned int regmap_irq_get_irq_reg_linear(struct regmap_irq_chip_data *da= ta, unsigned int base, int index); -int regmap_irq_set_type_config_simple(unsigned int **buf, unsigned int typ= e, - const struct regmap_irq *irq_data, int idx); +int regmap_irq_set_type_config_simple(struct regmap *map, unsigned int **b= uf, + unsigned int type, + const struct regmap_irq *irq_data, + int idx, void *irq_drv_data); =20 int regmap_add_irq_chip(struct regmap *map, int irq, int irq_flags, int irq_base, const struct regmap_irq_chip *chip, --=20 2.39.2 From nobody Wed Sep 10 06:05:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DB18CC7EE2D for ; Wed, 1 Mar 2023 12:58:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229935AbjCAM65 (ORCPT ); Wed, 1 Mar 2023 07:58:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57534 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229738AbjCAM6w (ORCPT ); Wed, 1 Mar 2023 07:58:52 -0500 Received: from mail-ot1-x32e.google.com (mail-ot1-x32e.google.com [IPv6:2607:f8b0:4864:20::32e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C8D1727D79 for ; Wed, 1 Mar 2023 04:58:49 -0800 (PST) Received: by mail-ot1-x32e.google.com with SMTP id 32-20020a9d0323000000b0069426a71d79so1852676otv.10 for ; Wed, 01 Mar 2023 04:58:49 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1677675529; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=XtRuzTJywCTMo/LDLm00cy0ICFbUs5QPr0i4Eope2Us=; b=YRTZFShQrrKIFkKl+fhXpyMozaTIDMuQYDMafgsGz7jHS4zVhECaF17ubGbowDxXp2 tkcWFxOx5+sD4Wir5/XF74w0zMGpXUsQthlsE4vWh9nSKAxpYpdAKhJlQb7tXmX3sgSk 0p/rSEkYe5OGXLVC3iW3hwWrAWyDY2l9DF5x39SnGp2tb8xroWlGQ43lksCWi7b2XJzz OkfvAkD8WL8mg1qF5mymBIDZn4fxi8ZtgwOWiOVAZjNTZLJ5sLW1miHu1ZaOSkU/QYcO 7k/kC32tBZK6O/nX+c0RIM4YVNOldV9IiznT3VNsE7kvVFNduoB5tRuDUBBKJyH27lK8 VTug== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1677675529; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=XtRuzTJywCTMo/LDLm00cy0ICFbUs5QPr0i4Eope2Us=; b=JAL8ELa7S7H1tI9TGVDN0VlSB/EcAnKWH+C6RPdQfmYoTtM58aHl/m6GOiE6ahnx1G UH3hnBEtPJyTLYAbBerv1bWc2CeJkMzJDP3/Z+3fHIhKrlfs5aN8cjnq3zDwNPyM31QM 0J75PJaKLT2crRQo5LmY2hyiRpf9OOaHCocb7w7Fs2fYXoRV/I8O6XenQ6UM0sJRzUtr TPPmUAthCyJI2+6TlHxGOfWgC8DPIQ8isbpLp6AoBBfHxbUVlk/OYrQOwGfdG5vrbgbQ iddrV9tN39GB/1pqil8CQkEIchatol9MOGQkLX0PzgMR2fzGhYlmEa/WZ7+yyTHI4e9a m5gQ== X-Gm-Message-State: AO0yUKUd+3n3dfS2EgFpjHUV+DtB2o/qZ1TcCKaRSoJO84OwVxUK357L IQTfu379xYwBn6cXBvNLUURojg== X-Google-Smtp-Source: AK7set+o0swWhBXszNbikDeNTlAPC50cowBFcR7nLCEVzRQBOc9UcTRN0JEMJvboBvXBj2NAgCd13Q== X-Received: by 2002:a9d:178c:0:b0:693:c1a6:2266 with SMTP id j12-20020a9d178c000000b00693c1a62266mr3067248otj.19.1677675528971; Wed, 01 Mar 2023 04:58:48 -0800 (PST) Received: from fedora.attlocal.net (69-109-179-158.lightspeed.dybhfl.sbcglobal.net. [69.109.179.158]) by smtp.gmail.com with ESMTPSA id g21-20020a056830309500b0068bc48c61a5sm4865064ots.19.2023.03.01.04.58.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Mar 2023 04:58:48 -0800 (PST) From: William Breathitt Gray To: linus.walleij@linaro.org, brgl@bgdev.pl Cc: broonie@kernel.org, linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org, quarium@gmail.com, jhentges@accesio.com, jay.dolan@accesio.com, William Breathitt Gray Subject: [PATCH v2 2/2] gpio: pcie-idio-24: Migrate to the regmap API Date: Mon, 27 Feb 2023 22:56:28 -0500 Message-Id: <579419ad72fa259a93e4bcbcd9dd969b2a3295df.1677555956.git.william.gray@linaro.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" The regmap API supports IO port accessors so we can take advantage of regmap abstractions rather than handling access to the device registers directly in the driver. For the PCIe-IDIO-24 series of devices, the following BARs are available: BAR[0]: memory mapped PEX8311 BAR[1]: I/O mapped PEX8311 BAR[2]: I/O mapped card registers There are 24 FET Output lines, 24 Isolated Input lines, and 8 TTL/CMOS lines (which may be configured for either output or input). The GPIO lines are exposed by the following card registers: Base +0x0-0x2 (Read/Write): FET Outputs Base +0xB (Read/Write): TTL/CMOS Base +0x4-0x6 (Read): Isolated Inputs Base +0x7 (Read): TTL/CMOS In order for the device to support interrupts, the PLX PEX8311 internal PCI wire interrupt and local interrupt input must first be enabled. The following card registers for Change-Of-State may be used: Base +0x8-0xA (Read): COS Status Inputs Base +0x8-0xA (Write): COS Clear Inputs Base +0xB (Read): COS Status TTL/CMOS Base +0xB (Write): COS Clear TTL/CMOS Base +0xE (Read/Write): COS Enable The COS Enable register is used to enable/disable interrupts and configure the interrupt levels; each bit maps to a group of eight inputs as described below: Bit 0: IRQ EN Rising Edge IN0-7 Bit 1: IRQ EN Rising Edge IN8-15 Bit 2: IRQ EN Rising Edge IN16-23 Bit 3: IRQ EN Rising Edge TTL0-7 Bit 4: IRQ EN Falling Edge IN0-7 Bit 5: IRQ EN Falling Edge IN8-15 Bit 6: IRQ EN Falling Edge IN16-23 Bit 7: IRQ EN Falling Edge TTL0-7 An interrupt is asserted when a change-of-state matching the interrupt level configuration respective for a particular group of eight inputs with enabled COS is detected. The COS Status registers may be read to determine which inputs have changed; if interrupts were enabled, an IRQ will be generated for the set bits in these registers. Writing the value read from the COS Status register back to the respective COS Clear register will clear just those interrupts. Cc: Arnaud de Turckheim Cc: John Hentges Cc: Jay Dolan Signed-off-by: William Breathitt Gray --- Changes in v2: - Simplify PCIe-IDIO-24 register offset defines to remove superfluous arithmetic - Check for NULL pointer after chip->irq_drv_data allocation - Set gpio_regmap drvdata and use gpio_regmap_get_drvdata() to get the regmap in idio_24_reg_map_xlate() drivers/gpio/Kconfig | 3 + drivers/gpio/gpio-pcie-idio-24.c | 693 ++++++++++++------------------- 2 files changed, 262 insertions(+), 434 deletions(-) diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig index 406e8bda487f..06c7a96e6033 100644 --- a/drivers/gpio/Kconfig +++ b/drivers/gpio/Kconfig @@ -1578,7 +1578,10 @@ config GPIO_PCI_IDIO_16 =20 config GPIO_PCIE_IDIO_24 tristate "ACCES PCIe-IDIO-24 GPIO support" + select REGMAP_IRQ + select REGMAP_MMIO select GPIOLIB_IRQCHIP + select GPIO_REGMAP help Enables GPIO support for the ACCES PCIe-IDIO-24 family (PCIe-IDIO-24, PCIe-IDI-24, PCIe-IDO-24, PCIe-IDIO-12). An interrupt is generated diff --git a/drivers/gpio/gpio-pcie-idio-24.c b/drivers/gpio/gpio-pcie-idio= -24.c index 8a9b98fa418f..9878b81c23c8 100644 --- a/drivers/gpio/gpio-pcie-idio-24.c +++ b/drivers/gpio/gpio-pcie-idio-24.c @@ -15,17 +15,15 @@ * This driver supports the following ACCES devices: PCIe-IDIO-24, * PCIe-IDI-24, PCIe-IDO-24, and PCIe-IDIO-12. */ -#include -#include +#include #include -#include -#include -#include -#include +#include +#include +#include #include #include #include -#include +#include #include =20 /* @@ -68,416 +66,229 @@ #define PLX_PEX8311_PCI_LCS_INTCSR 0x68 #define INTCSR_INTERNAL_PCI_WIRE BIT(8) #define INTCSR_LOCAL_INPUT BIT(11) +#define IDIO_24_ENABLE_IRQ (INTCSR_INTERNAL_PCI_WIRE | INTCSR_LOC= AL_INPUT) =20 -/** - * struct idio_24_gpio_reg - GPIO device registers structure - * @out0_7: Read: FET Outputs 0-7 - * Write: FET Outputs 0-7 - * @out8_15: Read: FET Outputs 8-15 - * Write: FET Outputs 8-15 - * @out16_23: Read: FET Outputs 16-23 - * Write: FET Outputs 16-23 - * @ttl_out0_7: Read: TTL/CMOS Outputs 0-7 - * Write: TTL/CMOS Outputs 0-7 - * @in0_7: Read: Isolated Inputs 0-7 - * Write: Reserved - * @in8_15: Read: Isolated Inputs 8-15 - * Write: Reserved - * @in16_23: Read: Isolated Inputs 16-23 - * Write: Reserved - * @ttl_in0_7: Read: TTL/CMOS Inputs 0-7 - * Write: Reserved - * @cos0_7: Read: COS Status Inputs 0-7 - * Write: COS Clear Inputs 0-7 - * @cos8_15: Read: COS Status Inputs 8-15 - * Write: COS Clear Inputs 8-15 - * @cos16_23: Read: COS Status Inputs 16-23 - * Write: COS Clear Inputs 16-23 - * @cos_ttl0_7: Read: COS Status TTL/CMOS 0-7 - * Write: COS Clear TTL/CMOS 0-7 - * @ctl: Read: Control Register - * Write: Control Register - * @reserved: Read: Reserved - * Write: Reserved - * @cos_enable: Read: COS Enable - * Write: COS Enable - * @soft_reset: Read: IRQ Output Pin Status - * Write: Software Board Reset - */ -struct idio_24_gpio_reg { - u8 out0_7; - u8 out8_15; - u8 out16_23; - u8 ttl_out0_7; - u8 in0_7; - u8 in8_15; - u8 in16_23; - u8 ttl_in0_7; - u8 cos0_7; - u8 cos8_15; - u8 cos16_23; - u8 cos_ttl0_7; - u8 ctl; - u8 reserved; - u8 cos_enable; - u8 soft_reset; -}; - -/** - * struct idio_24_gpio - GPIO device private data structure - * @chip: instance of the gpio_chip - * @lock: synchronization lock to prevent I/O race conditions - * @reg: I/O address offset for the GPIO device registers - * @irq_mask: I/O bits affected by interrupts - */ -struct idio_24_gpio { - struct gpio_chip chip; - raw_spinlock_t lock; - __u8 __iomem *plx; - struct idio_24_gpio_reg __iomem *reg; - unsigned long irq_mask; -}; - -static int idio_24_gpio_get_direction(struct gpio_chip *chip, - unsigned int offset) -{ - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - const unsigned long out_mode_mask =3D BIT(1); +#define IDIO_24_OUT_BASE 0x0 +#define IDIO_24_TTLCMOS_OUT_REG 0x3 +#define IDIO_24_IN_BASE 0x4 +#define IDIO_24_TTLCMOS_IN_REG 0x7 +#define IDIO_24_COS_STATUS_BASE 0x8 +#define IDIO_24_CONTROL_REG 0xC +#define IDIO_24_COS_ENABLE 0xE +#define IDIO_24_SOFT_RESET 0xF =20 - /* FET Outputs */ - if (offset < 24) - return GPIO_LINE_DIRECTION_OUT; +#define CONTROL_REG_OUT_MODE BIT(1) =20 - /* Isolated Inputs */ - if (offset < 48) - return GPIO_LINE_DIRECTION_IN; +#define COS_ENABLE_RISING BIT(1) +#define COS_ENABLE_FALLING BIT(4) +#define COS_ENABLE_BOTH (COS_ENABLE_RISING | COS_ENABLE_FALLING) =20 - /* TTL/CMOS I/O */ - /* OUT MODE =3D 1 when TTL/CMOS Output Mode is set */ - if (ioread8(&idio24gpio->reg->ctl) & out_mode_mask) - return GPIO_LINE_DIRECTION_OUT; - - return GPIO_LINE_DIRECTION_IN; -} +static const struct regmap_range pex8311_intcsr_wr_ranges[] =3D { + regmap_reg_range(0x0, 0x0), +}; +static const struct regmap_range pex8311_intcsr_rd_ranges[] =3D { + regmap_reg_range(0x0, 0x0), +}; +static const struct regmap_range pex8311_intcsr_volatile_ranges[] =3D { + regmap_reg_range(0x0, 0x0), +}; +static const struct regmap_access_table pex8311_intcsr_wr_table =3D { + .yes_ranges =3D pex8311_intcsr_wr_ranges, + .n_yes_ranges =3D ARRAY_SIZE(pex8311_intcsr_wr_ranges), +}; +static const struct regmap_access_table pex8311_intcsr_rd_table =3D { + .yes_ranges =3D pex8311_intcsr_rd_ranges, + .n_yes_ranges =3D ARRAY_SIZE(pex8311_intcsr_rd_ranges), +}; +static const struct regmap_access_table pex8311_intcsr_volatile_table =3D { + .yes_ranges =3D pex8311_intcsr_volatile_ranges, + .n_yes_ranges =3D ARRAY_SIZE(pex8311_intcsr_volatile_ranges), +}; =20 -static int idio_24_gpio_direction_input(struct gpio_chip *chip, - unsigned int offset) -{ - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long flags; - unsigned int ctl_state; - const unsigned long out_mode_mask =3D BIT(1); +static const struct regmap_config pex8311_intcsr_regmap_config =3D { + .name =3D "pex8311_intcsr", + .reg_bits =3D 32, + .reg_stride =3D 1, + .val_bits =3D 32, + .io_port =3D true, + .max_register =3D 0x0, + .wr_table =3D &pex8311_intcsr_wr_table, + .rd_table =3D &pex8311_intcsr_rd_table, + .volatile_table =3D &pex8311_intcsr_volatile_table, +}; =20 - /* TTL/CMOS I/O */ - if (offset > 47) { - raw_spin_lock_irqsave(&idio24gpio->lock, flags); +static const struct regmap_range idio_24_wr_ranges[] =3D { + regmap_reg_range(0x0, 0x3), regmap_reg_range(0x8, 0xC), + regmap_reg_range(0xE, 0xF), +}; +static const struct regmap_range idio_24_rd_ranges[] =3D { + regmap_reg_range(0x0, 0xC), regmap_reg_range(0xE, 0xF), +}; +static const struct regmap_range idio_24_volatile_ranges[] =3D { + regmap_reg_range(0x4, 0xB), regmap_reg_range(0xF, 0xF), +}; +static const struct regmap_access_table idio_24_wr_table =3D { + .yes_ranges =3D idio_24_wr_ranges, + .n_yes_ranges =3D ARRAY_SIZE(idio_24_wr_ranges), +}; +static const struct regmap_access_table idio_24_rd_table =3D { + .yes_ranges =3D idio_24_rd_ranges, + .n_yes_ranges =3D ARRAY_SIZE(idio_24_rd_ranges), +}; +static const struct regmap_access_table idio_24_volatile_table =3D { + .yes_ranges =3D idio_24_volatile_ranges, + .n_yes_ranges =3D ARRAY_SIZE(idio_24_volatile_ranges), +}; =20 - /* Clear TTL/CMOS Output Mode */ - ctl_state =3D ioread8(&idio24gpio->reg->ctl) & ~out_mode_mask; - iowrite8(ctl_state, &idio24gpio->reg->ctl); +static const struct regmap_config idio_24_regmap_config =3D { + .reg_bits =3D 8, + .reg_stride =3D 1, + .val_bits =3D 8, + .io_port =3D true, + .max_register =3D 0xF, + .wr_table =3D &idio_24_wr_table, + .rd_table =3D &idio_24_rd_table, + .volatile_table =3D &idio_24_volatile_table, + .cache_type =3D REGCACHE_FLAT, +}; =20 - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); +#define IDIO_24_NGPIO_PER_REG 8 +#define IDIO_24_REGMAP_IRQ(_id) \ + [24 + _id] =3D { \ + .reg_offset =3D (_id) / IDIO_24_NGPIO_PER_REG, \ + .mask =3D BIT((_id) % IDIO_24_NGPIO_PER_REG), \ + .type =3D { .types_supported =3D IRQ_TYPE_EDGE_BOTH }, \ } +#define IDIO_24_IIN_IRQ(_id) IDIO_24_REGMAP_IRQ(_id) +#define IDIO_24_TTL_IRQ(_id) IDIO_24_REGMAP_IRQ(24 + _id) + +static const struct regmap_irq idio_24_regmap_irqs[] =3D { + IDIO_24_IIN_IRQ(0), IDIO_24_IIN_IRQ(1), IDIO_24_IIN_IRQ(2), /* IIN 0-2 */ + IDIO_24_IIN_IRQ(3), IDIO_24_IIN_IRQ(4), IDIO_24_IIN_IRQ(5), /* IIN 3-5 */ + IDIO_24_IIN_IRQ(6), IDIO_24_IIN_IRQ(7), IDIO_24_IIN_IRQ(8), /* IIN 6-8 */ + IDIO_24_IIN_IRQ(9), IDIO_24_IIN_IRQ(10), IDIO_24_IIN_IRQ(11), /* IIN 9-11= */ + IDIO_24_IIN_IRQ(12), IDIO_24_IIN_IRQ(13), IDIO_24_IIN_IRQ(14), /* IIN 12-= 14 */ + IDIO_24_IIN_IRQ(15), IDIO_24_IIN_IRQ(16), IDIO_24_IIN_IRQ(17), /* IIN 15-= 17 */ + IDIO_24_IIN_IRQ(18), IDIO_24_IIN_IRQ(19), IDIO_24_IIN_IRQ(20), /* IIN 18-= 20 */ + IDIO_24_IIN_IRQ(21), IDIO_24_IIN_IRQ(22), IDIO_24_IIN_IRQ(23), /* IIN 21-= 23 */ + IDIO_24_TTL_IRQ(0), IDIO_24_TTL_IRQ(1), IDIO_24_TTL_IRQ(2), /* TTL 0-2 */ + IDIO_24_TTL_IRQ(3), IDIO_24_TTL_IRQ(4), IDIO_24_TTL_IRQ(5), /* TTL 3-5 */ + IDIO_24_TTL_IRQ(6), IDIO_24_TTL_IRQ(7), /* TTL 6-7 */ +}; =20 - return 0; -} - -static int idio_24_gpio_direction_output(struct gpio_chip *chip, - unsigned int offset, int value) +static int idio_24_set_type_config(struct regmap *const map, + unsigned int **const buf, + const unsigned int type, + const struct regmap_irq *const irq_data, + const int idx, void *const irq_drv_data) { - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long flags; - unsigned int ctl_state; - const unsigned long out_mode_mask =3D BIT(1); - - /* TTL/CMOS I/O */ - if (offset > 47) { - raw_spin_lock_irqsave(&idio24gpio->lock, flags); - - /* Set TTL/CMOS Output Mode */ - ctl_state =3D ioread8(&idio24gpio->reg->ctl) | out_mode_mask; - iowrite8(ctl_state, &idio24gpio->reg->ctl); + const unsigned int offset =3D irq_data->reg_offset; + const unsigned int rising =3D COS_ENABLE_RISING << offset; + const unsigned int falling =3D COS_ENABLE_FALLING << offset; + const unsigned int mask =3D COS_ENABLE_BOTH << offset; + unsigned int *const irq_type =3D irq_drv_data; + unsigned int new; + unsigned int cos_enable; + int err; =20 - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); + switch (type) { + case IRQ_TYPE_EDGE_RISING: + new =3D rising; + break; + case IRQ_TYPE_EDGE_FALLING: + new =3D falling; + break; + case IRQ_TYPE_EDGE_BOTH: + new =3D mask; + break; + default: + return -EINVAL; } =20 - chip->set(chip, offset, value); - return 0; -} - -static int idio_24_gpio_get(struct gpio_chip *chip, unsigned int offset) -{ - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - const unsigned long offset_mask =3D BIT(offset % 8); - const unsigned long out_mode_mask =3D BIT(1); - - /* FET Outputs */ - if (offset < 8) - return !!(ioread8(&idio24gpio->reg->out0_7) & offset_mask); - - if (offset < 16) - return !!(ioread8(&idio24gpio->reg->out8_15) & offset_mask); - - if (offset < 24) - return !!(ioread8(&idio24gpio->reg->out16_23) & offset_mask); - - /* Isolated Inputs */ - if (offset < 32) - return !!(ioread8(&idio24gpio->reg->in0_7) & offset_mask); - - if (offset < 40) - return !!(ioread8(&idio24gpio->reg->in8_15) & offset_mask); - - if (offset < 48) - return !!(ioread8(&idio24gpio->reg->in16_23) & offset_mask); + /* replace old bitmap with new bitmap */ + *irq_type =3D (*irq_type & ~mask) | (new & mask); =20 - /* TTL/CMOS Outputs */ - if (ioread8(&idio24gpio->reg->ctl) & out_mode_mask) - return !!(ioread8(&idio24gpio->reg->ttl_out0_7) & offset_mask); - - /* TTL/CMOS Inputs */ - return !!(ioread8(&idio24gpio->reg->ttl_in0_7) & offset_mask); -} - -static int idio_24_gpio_get_multiple(struct gpio_chip *chip, - unsigned long *mask, unsigned long *bits) -{ - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long offset; - unsigned long gpio_mask; - void __iomem *ports[] =3D { - &idio24gpio->reg->out0_7, &idio24gpio->reg->out8_15, - &idio24gpio->reg->out16_23, &idio24gpio->reg->in0_7, - &idio24gpio->reg->in8_15, &idio24gpio->reg->in16_23, - }; - size_t index; - unsigned long port_state; - const unsigned long out_mode_mask =3D BIT(1); - - /* clear bits array to a clean slate */ - bitmap_zero(bits, chip->ngpio); - - for_each_set_clump8(offset, gpio_mask, mask, ARRAY_SIZE(ports) * 8) { - index =3D offset / 8; - - /* read bits from current gpio port (port 6 is TTL GPIO) */ - if (index < 6) - port_state =3D ioread8(ports[index]); - else if (ioread8(&idio24gpio->reg->ctl) & out_mode_mask) - port_state =3D ioread8(&idio24gpio->reg->ttl_out0_7); - else - port_state =3D ioread8(&idio24gpio->reg->ttl_in0_7); - - port_state &=3D gpio_mask; - - bitmap_set_value8(bits, port_state, offset); - } + err =3D regmap_read(map, IDIO_24_COS_ENABLE, &cos_enable); + if (err) + return err; =20 + /* if COS is currently enabled */ + if (cos_enable & mask) + return regmap_update_bits(map, IDIO_24_COS_ENABLE, mask, + *irq_type); return 0; } =20 -static void idio_24_gpio_set(struct gpio_chip *chip, unsigned int offset, - int value) +static int idio_24_handle_mask_sync(struct regmap *const map, const int in= dex, + const unsigned int mask_buf_def, + const unsigned int mask_buf, + void *const irq_drv_data) { - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - const unsigned long out_mode_mask =3D BIT(1); - void __iomem *base; - const unsigned int mask =3D BIT(offset % 8); - unsigned long flags; - unsigned int out_state; - - /* Isolated Inputs */ - if (offset > 23 && offset < 48) - return; - - /* TTL/CMOS Inputs */ - if (offset > 47 && !(ioread8(&idio24gpio->reg->ctl) & out_mode_mask)) - return; - - /* TTL/CMOS Outputs */ - if (offset > 47) - base =3D &idio24gpio->reg->ttl_out0_7; - /* FET Outputs */ - else if (offset > 15) - base =3D &idio24gpio->reg->out16_23; - else if (offset > 7) - base =3D &idio24gpio->reg->out8_15; - else - base =3D &idio24gpio->reg->out0_7; - - raw_spin_lock_irqsave(&idio24gpio->lock, flags); - - if (value) - out_state =3D ioread8(base) | mask; - else - out_state =3D ioread8(base) & ~mask; - - iowrite8(out_state, base); - - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); -} + const unsigned int irq_type_mask =3D COS_ENABLE_BOTH << index; + unsigned int *const irq_type =3D irq_drv_data; =20 -static void idio_24_gpio_set_multiple(struct gpio_chip *chip, - unsigned long *mask, unsigned long *bits) -{ - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long offset; - unsigned long gpio_mask; - void __iomem *ports[] =3D { - &idio24gpio->reg->out0_7, &idio24gpio->reg->out8_15, - &idio24gpio->reg->out16_23 - }; - size_t index; - unsigned long bitmask; - unsigned long flags; - unsigned long out_state; - const unsigned long out_mode_mask =3D BIT(1); - - for_each_set_clump8(offset, gpio_mask, mask, ARRAY_SIZE(ports) * 8) { - index =3D offset / 8; - - bitmask =3D bitmap_get_value8(bits, offset) & gpio_mask; - - raw_spin_lock_irqsave(&idio24gpio->lock, flags); - - /* read bits from current gpio port (port 6 is TTL GPIO) */ - if (index < 6) { - out_state =3D ioread8(ports[index]); - } else if (ioread8(&idio24gpio->reg->ctl) & out_mode_mask) { - out_state =3D ioread8(&idio24gpio->reg->ttl_out0_7); - } else { - /* skip TTL GPIO if set for input */ - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); - continue; - } - - /* set requested bit states */ - out_state &=3D ~gpio_mask; - out_state |=3D bitmask; + /* if all are masked, disable interrupts */ + if (mask_buf =3D=3D mask_buf_def) + return regmap_update_bits(map, IDIO_24_COS_ENABLE, + irq_type_mask, ~irq_type_mask); =20 - /* write bits for current gpio port (port 6 is TTL GPIO) */ - if (index < 6) - iowrite8(out_state, ports[index]); - else - iowrite8(out_state, &idio24gpio->reg->ttl_out0_7); - - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); - } + return regmap_update_bits(map, IDIO_24_COS_ENABLE, irq_type_mask, + *irq_type); } =20 -static void idio_24_irq_ack(struct irq_data *data) +static int idio_24_reg_mask_xlate(struct gpio_regmap *const gpio, + const unsigned int base, + const unsigned int offset, + unsigned int *const reg, + unsigned int *const mask) { -} - -static void idio_24_irq_mask(struct irq_data *data) -{ - struct gpio_chip *const chip =3D irq_data_get_irq_chip_data(data); - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long flags; - const unsigned long bit_offset =3D irqd_to_hwirq(data) - 24; - unsigned char new_irq_mask; - const unsigned long bank_offset =3D bit_offset / 8; - unsigned char cos_enable_state; - - raw_spin_lock_irqsave(&idio24gpio->lock, flags); - - idio24gpio->irq_mask &=3D ~BIT(bit_offset); - new_irq_mask =3D idio24gpio->irq_mask >> bank_offset * 8; - - if (!new_irq_mask) { - cos_enable_state =3D ioread8(&idio24gpio->reg->cos_enable); - - /* Disable Rising Edge detection */ - cos_enable_state &=3D ~BIT(bank_offset); - /* Disable Falling Edge detection */ - cos_enable_state &=3D ~BIT(bank_offset + 4); - - iowrite8(cos_enable_state, &idio24gpio->reg->cos_enable); - } - - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); -} - -static void idio_24_irq_unmask(struct irq_data *data) -{ - struct gpio_chip *const chip =3D irq_data_get_irq_chip_data(data); - struct idio_24_gpio *const idio24gpio =3D gpiochip_get_data(chip); - unsigned long flags; - unsigned char prev_irq_mask; - const unsigned long bit_offset =3D irqd_to_hwirq(data) - 24; - const unsigned long bank_offset =3D bit_offset / 8; - unsigned char cos_enable_state; - - raw_spin_lock_irqsave(&idio24gpio->lock, flags); + const unsigned int out_stride =3D offset / IDIO_24_NGPIO_PER_REG; + const unsigned int in_stride =3D (offset - 24) / IDIO_24_NGPIO_PER_REG; + struct regmap *const map =3D gpio_regmap_get_drvdata(gpio); + int err; + unsigned int ctrl_reg; =20 - prev_irq_mask =3D idio24gpio->irq_mask >> bank_offset * 8; - idio24gpio->irq_mask |=3D BIT(bit_offset); + switch (base) { + case IDIO_24_OUT_BASE: + *mask =3D BIT(offset % IDIO_24_NGPIO_PER_REG); =20 - if (!prev_irq_mask) { - cos_enable_state =3D ioread8(&idio24gpio->reg->cos_enable); + /* FET Outputs */ + if (offset < 24) { + *reg =3D IDIO_24_OUT_BASE + out_stride; + return 0; + } =20 - /* Enable Rising Edge detection */ - cos_enable_state |=3D BIT(bank_offset); - /* Enable Falling Edge detection */ - cos_enable_state |=3D BIT(bank_offset + 4); + /* Isolated Inputs */ + if (offset < 48) { + *reg =3D IDIO_24_IN_BASE + in_stride; + return 0; + } =20 - iowrite8(cos_enable_state, &idio24gpio->reg->cos_enable); - } + err =3D regmap_read(map, IDIO_24_CONTROL_REG, &ctrl_reg); + if (err) + return err; =20 - raw_spin_unlock_irqrestore(&idio24gpio->lock, flags); -} + /* TTL/CMOS Outputs */ + if (ctrl_reg & CONTROL_REG_OUT_MODE) { + *reg =3D IDIO_24_TTLCMOS_OUT_REG; + return 0; + } =20 -static int idio_24_irq_set_type(struct irq_data *data, unsigned int flow_t= ype) -{ - /* The only valid irq types are none and both-edges */ - if (flow_type !=3D IRQ_TYPE_NONE && - (flow_type & IRQ_TYPE_EDGE_BOTH) !=3D IRQ_TYPE_EDGE_BOTH) + /* TTL/CMOS Inputs */ + *reg =3D IDIO_24_TTLCMOS_IN_REG; + return 0; + case IDIO_24_CONTROL_REG: + /* We can only set direction for TTL/CMOS lines */ + if (offset < 48) + return -EOPNOTSUPP; + + *reg =3D IDIO_24_CONTROL_REG; + *mask =3D CONTROL_REG_OUT_MODE; + return 0; + default: + /* Should never reach this path */ return -EINVAL; - - return 0; -} - -static struct irq_chip idio_24_irqchip =3D { - .name =3D "pcie-idio-24", - .irq_ack =3D idio_24_irq_ack, - .irq_mask =3D idio_24_irq_mask, - .irq_unmask =3D idio_24_irq_unmask, - .irq_set_type =3D idio_24_irq_set_type -}; - -static irqreturn_t idio_24_irq_handler(int irq, void *dev_id) -{ - struct idio_24_gpio *const idio24gpio =3D dev_id; - unsigned long irq_status; - struct gpio_chip *const chip =3D &idio24gpio->chip; - unsigned long irq_mask; - int gpio; - - raw_spin_lock(&idio24gpio->lock); - - /* Read Change-Of-State status */ - irq_status =3D ioread32(&idio24gpio->reg->cos0_7); - - raw_spin_unlock(&idio24gpio->lock); - - /* Make sure our device generated IRQ */ - if (!irq_status) - return IRQ_NONE; - - /* Handle only unmasked IRQ */ - irq_mask =3D idio24gpio->irq_mask & irq_status; - - for_each_set_bit(gpio, &irq_mask, chip->ngpio - 24) - generic_handle_domain_irq(chip->irq.domain, gpio + 24); - - raw_spin_lock(&idio24gpio->lock); - - /* Clear Change-Of-State status */ - iowrite32(irq_status, &idio24gpio->reg->cos0_7); - - raw_spin_unlock(&idio24gpio->lock); - - return IRQ_HANDLED; + } } =20 #define IDIO_24_NGPIO 56 @@ -494,16 +305,18 @@ static const char *idio_24_names[IDIO_24_NGPIO] =3D { static int idio_24_probe(struct pci_dev *pdev, const struct pci_device_id = *id) { struct device *const dev =3D &pdev->dev; - struct idio_24_gpio *idio24gpio; int err; const size_t pci_plx_bar_index =3D 1; const size_t pci_bar_index =3D 2; const char *const name =3D pci_name(pdev); - struct gpio_irq_chip *girq; - - idio24gpio =3D devm_kzalloc(dev, sizeof(*idio24gpio), GFP_KERNEL); - if (!idio24gpio) - return -ENOMEM; + struct gpio_regmap_config gpio_config =3D {}; + void __iomem *pex8311_intcsr; + void __iomem *idio_24_regs; + struct regmap *pex8311_intcsr_map; + struct regmap *idio_24_map; + struct regmap_irq_chip *chip; + unsigned int irq_type; + struct regmap_irq_chip_data *chip_data; =20 err =3D pcim_enable_device(pdev); if (err) { @@ -517,57 +330,69 @@ static int idio_24_probe(struct pci_dev *pdev, const = struct pci_device_id *id) return err; } =20 - idio24gpio->plx =3D pcim_iomap_table(pdev)[pci_plx_bar_index]; - idio24gpio->reg =3D pcim_iomap_table(pdev)[pci_bar_index]; - - idio24gpio->chip.label =3D name; - idio24gpio->chip.parent =3D dev; - idio24gpio->chip.owner =3D THIS_MODULE; - idio24gpio->chip.base =3D -1; - idio24gpio->chip.ngpio =3D IDIO_24_NGPIO; - idio24gpio->chip.names =3D idio_24_names; - idio24gpio->chip.get_direction =3D idio_24_gpio_get_direction; - idio24gpio->chip.direction_input =3D idio_24_gpio_direction_input; - idio24gpio->chip.direction_output =3D idio_24_gpio_direction_output; - idio24gpio->chip.get =3D idio_24_gpio_get; - idio24gpio->chip.get_multiple =3D idio_24_gpio_get_multiple; - idio24gpio->chip.set =3D idio_24_gpio_set; - idio24gpio->chip.set_multiple =3D idio_24_gpio_set_multiple; - - girq =3D &idio24gpio->chip.irq; - girq->chip =3D &idio_24_irqchip; - /* This will let us handle the parent IRQ in the driver */ - girq->parent_handler =3D NULL; - girq->num_parents =3D 0; - girq->parents =3D NULL; - girq->default_type =3D IRQ_TYPE_NONE; - girq->handler =3D handle_edge_irq; - - raw_spin_lock_init(&idio24gpio->lock); + pex8311_intcsr =3D pcim_iomap_table(pdev)[pci_plx_bar_index] + PLX_PEX831= 1_PCI_LCS_INTCSR; + idio_24_regs =3D pcim_iomap_table(pdev)[pci_bar_index]; + + pex8311_intcsr_map =3D devm_regmap_init_mmio(dev, pex8311_intcsr, + &pex8311_intcsr_regmap_config); + if (IS_ERR(pex8311_intcsr_map)) + return dev_err_probe(dev, PTR_ERR(pex8311_intcsr_map), + "Unable to initialize PEX8311 register map\n"); + idio_24_map =3D devm_regmap_init_mmio(dev, idio_24_regs, + &idio_24_regmap_config); + if (IS_ERR(idio_24_map)) + return dev_err_probe(dev, PTR_ERR(idio_24_map), + "Unable to initialize register map\n"); + + chip =3D devm_kzalloc(dev, sizeof(*chip), GFP_KERNEL); + if (!chip) + return -ENOMEM; + + chip->irq_drv_data =3D devm_kzalloc(dev, sizeof(irq_type), GFP_KERNEL); + if (!chip->irq_drv_data) + return -ENOMEM; + + chip->name =3D name; + chip->status_base =3D IDIO_24_COS_STATUS_BASE; + chip->mask_base =3D IDIO_24_COS_ENABLE; + chip->ack_base =3D IDIO_24_COS_STATUS_BASE; + chip->num_regs =3D 4; + chip->irqs =3D idio_24_regmap_irqs; + chip->num_irqs =3D ARRAY_SIZE(idio_24_regmap_irqs); + chip->handle_mask_sync =3D idio_24_handle_mask_sync; + chip->set_type_config =3D idio_24_set_type_config; =20 /* Software board reset */ - iowrite8(0, &idio24gpio->reg->soft_reset); + err =3D regmap_write(idio_24_map, IDIO_24_SOFT_RESET, 0); + if (err) + return err; /* * enable PLX PEX8311 internal PCI wire interrupt and local interrupt * input */ - iowrite8((INTCSR_INTERNAL_PCI_WIRE | INTCSR_LOCAL_INPUT) >> 8, - idio24gpio->plx + PLX_PEX8311_PCI_LCS_INTCSR + 1); - - err =3D devm_gpiochip_add_data(dev, &idio24gpio->chip, idio24gpio); - if (err) { - dev_err(dev, "GPIO registering failed (%d)\n", err); + err =3D regmap_update_bits(pex8311_intcsr_map, 0x0, IDIO_24_ENABLE_IRQ, + IDIO_24_ENABLE_IRQ); + if (err) return err; - } - - err =3D devm_request_irq(dev, pdev->irq, idio_24_irq_handler, IRQF_SHARED, - name, idio24gpio); - if (err) { - dev_err(dev, "IRQ handler registering failed (%d)\n", err); - return err; - } =20 - return 0; + err =3D devm_regmap_add_irq_chip(dev, idio_24_map, pdev->irq, 0, 0, chip, + &chip_data); + if (err) + return dev_err_probe(dev, err, "IRQ registration failed\n"); + + gpio_config.parent =3D dev; + gpio_config.regmap =3D idio_24_map; + gpio_config.ngpio =3D IDIO_24_NGPIO; + gpio_config.names =3D idio_24_names; + gpio_config.reg_dat_base =3D GPIO_REGMAP_ADDR(IDIO_24_OUT_BASE); + gpio_config.reg_set_base =3D GPIO_REGMAP_ADDR(IDIO_24_OUT_BASE); + gpio_config.reg_dir_out_base =3D GPIO_REGMAP_ADDR(IDIO_24_CONTROL_REG); + gpio_config.ngpio_per_reg =3D IDIO_24_NGPIO_PER_REG; + gpio_config.irq_domain =3D regmap_irq_get_domain(chip_data); + gpio_config.reg_mask_xlate =3D idio_24_reg_mask_xlate; + gpio_config.drvdata =3D idio_24_map; + + return PTR_ERR_OR_ZERO(devm_gpio_regmap_register(dev, &gpio_config)); } =20 static const struct pci_device_id idio_24_pci_dev_id[] =3D { --=20 2.39.2