* [PATCH v2 0/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO @ 2018-04-13 8:08 Phil Edworthy 2018-04-13 8:08 ` [PATCH v2 1/2] " Phil Edworthy 2018-04-13 8:08 ` [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts Phil Edworthy 0 siblings, 2 replies; 7+ messages in thread From: Phil Edworthy @ 2018-04-13 8:08 UTC (permalink / raw) To: Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones Cc: Andy Shevchenko, Michel Pollet, linux-gpio, devicetree, linux-renesas-soc, linux-kernel, Phil Edworthy All details about the change is in patch 0001. patch 0002 fixes up other users of 'struct dwapb_port_property'. One point to mention is that I have made it possible for users to have unconncted interrupts by specifying holes in the list of interrupts. This is done by supporting the interrupts-extended DT prop. However, I have no use for this and had to hack some test case for this. Perhaps the driver should support 1 interrupt or all GPIOa as interrupts? Phil Edworthy (2): gpio: dwapb: Add support for 1 interrupt per port A GPIO mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts .../devicetree/bindings/gpio/snps-dwapb-gpio.txt | 9 ++++- drivers/gpio/gpio-dwapb.c | 43 +++++++++++++++++----- drivers/mfd/intel_quark_i2c_gpio.c | 3 +- include/linux/platform_data/gpio-dwapb.h | 3 +- 4 files changed, 45 insertions(+), 13 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 1/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO 2018-04-13 8:08 [PATCH v2 0/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO Phil Edworthy @ 2018-04-13 8:08 ` Phil Edworthy 2018-04-13 22:16 ` kbuild test robot 2018-04-14 0:46 ` kbuild test robot 2018-04-13 8:08 ` [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts Phil Edworthy 1 sibling, 2 replies; 7+ messages in thread From: Phil Edworthy @ 2018-04-13 8:08 UTC (permalink / raw) To: Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones Cc: Andy Shevchenko, Michel Pollet, linux-gpio, devicetree, linux-renesas-soc, linux-kernel, Phil Edworthy The DesignWare GPIO IP can be configured for either 1 interrupt or 1 per GPIO in port A, but the driver currently only supports 1 interrupt. See the DesignWare DW_apb_gpio Databook description of the 'GPIO_INTR_IO' parameter. This change allows the driver to work with up to 32 interrupts, it will get as many interrupts as specified in the DT 'interrupts' property. It doesn't do anything clever with the different interrupts, it just calls the same handler used for single interrupt hardware. Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> --- v2: - Replaced interrupt-mask DT prop with support for the interrupts-extended prop. This means replacing the call to irq_of_parse_and_map() with calls to of_irq_parse_one() and irq_create_of_mapping(). Note: There are a few *code* lines over 80 chars, but this is just guidance, right? Especially as there are already some lines over 80 chars. --- .../devicetree/bindings/gpio/snps-dwapb-gpio.txt | 9 ++++- drivers/gpio/gpio-dwapb.c | 43 +++++++++++++++++----- include/linux/platform_data/gpio-dwapb.h | 3 +- 3 files changed, 43 insertions(+), 12 deletions(-) diff --git a/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt b/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt index 4a75da7..3c1118b 100644 --- a/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt +++ b/Documentation/devicetree/bindings/gpio/snps-dwapb-gpio.txt @@ -26,8 +26,13 @@ controller. the second encodes the triger flags encoded as described in Documentation/devicetree/bindings/interrupt-controller/interrupts.txt - interrupt-parent : The parent interrupt controller. -- interrupts : The interrupt to the parent controller raised when GPIOs - generate the interrupts. +- interrupts : The interrupts to the parent controller raised when GPIOs + generate the interrupts. If the controller provides one combined interrupt + for all GPIOs, specify a single interrupt. If the controller provides one + interrupt for each GPIO, provide a list of interrupts that correspond to each + of the GPIO pins. When specifying multiple interrupts, if any are unconnected, + use the interrupts-extended property to specify the interrupts and set the + interrupt controller handle for unused interrupts to 0. - snps,nr-gpios : The number of pins in the port, a single cell. - resets : Reset line for the controller. diff --git a/drivers/gpio/gpio-dwapb.c b/drivers/gpio/gpio-dwapb.c index 226977f..3273504 100644 --- a/drivers/gpio/gpio-dwapb.c +++ b/drivers/gpio/gpio-dwapb.c @@ -441,14 +441,19 @@ static void dwapb_configure_irqs(struct dwapb_gpio *gpio, irq_gc->chip_types[1].handler = handle_edge_irq; if (!pp->irq_shared) { - irq_set_chained_handler_and_data(pp->irq, dwapb_irq_handler, - gpio); + int i; + + for (i = 0; i < pp->ngpio; i++) { + if (pp->irq[i]) + irq_set_chained_handler_and_data(pp->irq[i], + dwapb_irq_handler, gpio); + } } else { /* * Request a shared IRQ since where MFD would have devices * using the same irq pin */ - err = devm_request_irq(gpio->dev, pp->irq, + err = devm_request_irq(gpio->dev, pp->irq[0], dwapb_irq_handler_mfd, IRQF_SHARED, "gpio-dwapb-mfd", gpio); if (err) { @@ -524,7 +529,7 @@ static int dwapb_gpio_add_port(struct dwapb_gpio *gpio, if (pp->idx == 0) port->gc.set_config = dwapb_gpio_set_config; - if (pp->irq) + if (pp->has_irq) dwapb_configure_irqs(gpio, port, pp); err = gpiochip_add_data(&port->gc, port); @@ -535,7 +540,7 @@ static int dwapb_gpio_add_port(struct dwapb_gpio *gpio, port->is_registered = true; /* Add GPIO-signaled ACPI event support */ - if (pp->irq) + if (pp->has_irq) acpi_gpiochip_request_interrupts(&port->gc); return err; @@ -601,13 +606,33 @@ dwapb_gpio_get_pdata(struct device *dev) if (dev->of_node && pp->idx == 0 && fwnode_property_read_bool(fwnode, "interrupt-controller")) { - pp->irq = irq_of_parse_and_map(to_of_node(fwnode), 0); - if (!pp->irq) + struct device_node *np = to_of_node(fwnode); + struct of_phandle_args oirq; + unsigned int j; + + /* + * The IP has configuration options to allow a single + * combined interrupt or one per gpio. If one per gpio, + * some might not be used. + */ + for (j = 0; j < pp->ngpio; j++) { + if (of_irq_parse_one(np, j, &oirq)) + continue; + + pp->irq[j] = irq_create_of_mapping(&oirq); + if (pp->irq[j]) + pp->has_irq = true; + } + + if (!pp->has_irq) dev_warn(dev, "no irq for port%d\n", pp->idx); } - if (has_acpi_companion(dev) && pp->idx == 0) - pp->irq = platform_get_irq(to_platform_device(dev), 0); + if (has_acpi_companion(dev) && pp->idx == 0) { + pp->irq[0] = platform_get_irq(to_platform_device(dev), 0); + if (pp->irq[0]) + pp->has_irq = true; + } pp->irq_shared = false; pp->gpio_base = -1; diff --git a/include/linux/platform_data/gpio-dwapb.h b/include/linux/platform_data/gpio-dwapb.h index 2dc7f4a..5a52d69 100644 --- a/include/linux/platform_data/gpio-dwapb.h +++ b/include/linux/platform_data/gpio-dwapb.h @@ -19,7 +19,8 @@ struct dwapb_port_property { unsigned int idx; unsigned int ngpio; unsigned int gpio_base; - unsigned int irq; + unsigned int irq[32]; + bool has_irq; bool irq_shared; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO 2018-04-13 8:08 ` [PATCH v2 1/2] " Phil Edworthy @ 2018-04-13 22:16 ` kbuild test robot 2018-04-14 0:46 ` kbuild test robot 1 sibling, 0 replies; 7+ messages in thread From: kbuild test robot @ 2018-04-13 22:16 UTC (permalink / raw) To: Phil Edworthy Cc: kbuild-all, Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones, Andy Shevchenko, Michel Pollet, linux-gpio, devicetree, linux-renesas-soc, linux-kernel, Phil Edworthy [-- Attachment #1: Type: text/plain, Size: 1723 bytes --] Hi Phil, Thank you for the patch! Yet something to improve: [auto build test ERROR on gpio/for-next] [also build test ERROR on v4.16 next-20180413] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Phil-Edworthy/gpio-dwapb-Add-support-for-1-interrupt-per-port-A-GPIO/20180414-042450 base: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-gpio.git for-next config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=sparc64 All errors (new ones prefixed by >>): drivers/gpio/gpio-dwapb.o: In function `dwapb_gpio_probe': >> gpio-dwapb.c:(.text+0x1444): undefined reference to `of_irq_parse_one' `.exit.data' referenced in section `.exit.text' of drivers/tty/n_hdlc.o: defined in discarded section `.exit.data' of drivers/tty/n_hdlc.o `.exit.data' referenced in section `.exit.text' of drivers/tty/n_hdlc.o: defined in discarded section `.exit.data' of drivers/tty/n_hdlc.o `.exit.data' referenced in section `.exit.text' of drivers/tty/n_hdlc.o: defined in discarded section `.exit.data' of drivers/tty/n_hdlc.o `.exit.data' referenced in section `.exit.text' of drivers/tty/n_hdlc.o: defined in discarded section `.exit.data' of drivers/tty/n_hdlc.o --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 53478 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 1/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO 2018-04-13 8:08 ` [PATCH v2 1/2] " Phil Edworthy 2018-04-13 22:16 ` kbuild test robot @ 2018-04-14 0:46 ` kbuild test robot 1 sibling, 0 replies; 7+ messages in thread From: kbuild test robot @ 2018-04-14 0:46 UTC (permalink / raw) To: Phil Edworthy Cc: kbuild-all, Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones, Andy Shevchenko, Michel Pollet, linux-gpio, devicetree, linux-renesas-soc, linux-kernel, Phil Edworthy [-- Attachment #1: Type: text/plain, Size: 943 bytes --] Hi Phil, Thank you for the patch! Yet something to improve: [auto build test ERROR on gpio/for-next] [also build test ERROR on v4.16 next-20180413] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Phil-Edworthy/gpio-dwapb-Add-support-for-1-interrupt-per-port-A-GPIO/20180414-042450 base: https://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-gpio.git for-next config: x86_64-randconfig-ne0-04140637 (attached as .config) compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 All errors (new ones prefixed by >>): >> ERROR: "of_irq_parse_one" [drivers/gpio/gpio-dwapb.ko] undefined! --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 30075 bytes --] ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts 2018-04-13 8:08 [PATCH v2 0/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO Phil Edworthy 2018-04-13 8:08 ` [PATCH v2 1/2] " Phil Edworthy @ 2018-04-13 8:08 ` Phil Edworthy 2018-04-13 8:20 ` Geert Uytterhoeven 1 sibling, 1 reply; 7+ messages in thread From: Phil Edworthy @ 2018-04-13 8:08 UTC (permalink / raw) To: Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones Cc: Andy Shevchenko, Michel Pollet, linux-gpio, devicetree, linux-renesas-soc, linux-kernel, Phil Edworthy Since the way the Synopsys GPIO interrupts are stored has changed, this driver needs to be updated in line with the changes. Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> --- v2: - New patch in v2 to fix the only other user of struct dwapb_port_property. --- drivers/mfd/intel_quark_i2c_gpio.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/intel_quark_i2c_gpio.c b/drivers/mfd/intel_quark_i2c_gpio.c index 90e35de..5bddb84 100644 --- a/drivers/mfd/intel_quark_i2c_gpio.c +++ b/drivers/mfd/intel_quark_i2c_gpio.c @@ -233,7 +233,8 @@ static int intel_quark_gpio_setup(struct pci_dev *pdev, struct mfd_cell *cell) pdata->properties->idx = 0; pdata->properties->ngpio = INTEL_QUARK_MFD_NGPIO; pdata->properties->gpio_base = INTEL_QUARK_MFD_GPIO_BASE; - pdata->properties->irq = pdev->irq; + pdata->properties->irq[0] = pdev->irq; + pdata->properties->has_irq = true; pdata->properties->irq_shared = true; cell->platform_data = pdata; -- 2.7.4 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts 2018-04-13 8:08 ` [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts Phil Edworthy @ 2018-04-13 8:20 ` Geert Uytterhoeven 2018-04-13 8:24 ` Phil Edworthy 0 siblings, 1 reply; 7+ messages in thread From: Geert Uytterhoeven @ 2018-04-13 8:20 UTC (permalink / raw) To: Phil Edworthy Cc: Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones, Andy Shevchenko, Michel Pollet, open list:GPIO SUBSYSTEM, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux-Renesas, Linux Kernel Mailing List Hi Phil, On Fri, Apr 13, 2018 at 10:08 AM, Phil Edworthy <phil.edworthy@renesas.com> wrote: > Since the way the Synopsys GPIO interrupts are stored has changed, this > driver needs to be updated in line with the changes. > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > --- > v2: > - New patch in v2 to fix the only other user of struct dwapb_port_property. Thanks for your patch! To avoid bisection compile failures due to the changed type of dwapb_port_property.irq, I think this should be folded into the first patch, > --- > drivers/mfd/intel_quark_i2c_gpio.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/mfd/intel_quark_i2c_gpio.c b/drivers/mfd/intel_quark_i2c_gpio.c > index 90e35de..5bddb84 100644 > --- a/drivers/mfd/intel_quark_i2c_gpio.c > +++ b/drivers/mfd/intel_quark_i2c_gpio.c > @@ -233,7 +233,8 @@ static int intel_quark_gpio_setup(struct pci_dev *pdev, struct mfd_cell *cell) > pdata->properties->idx = 0; > pdata->properties->ngpio = INTEL_QUARK_MFD_NGPIO; > pdata->properties->gpio_base = INTEL_QUARK_MFD_GPIO_BASE; > - pdata->properties->irq = pdev->irq; > + pdata->properties->irq[0] = pdev->irq; > + pdata->properties->has_irq = true; > pdata->properties->irq_shared = true; > > cell->platform_data = pdata; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts 2018-04-13 8:20 ` Geert Uytterhoeven @ 2018-04-13 8:24 ` Phil Edworthy 0 siblings, 0 replies; 7+ messages in thread From: Phil Edworthy @ 2018-04-13 8:24 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Hoan Tran, Linus Walleij, Rob Herring, Mark Rutland, Lee Jones, Andy Shevchenko, Michel Pollet, open list:GPIO SUBSYSTEM, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Linux-Renesas, Linux Kernel Mailing List Hi Geert, On 13 April 2018 09:20 Geert Uytterhoeven wrote: > On Fri, Apr 13, 2018 at 10:08 AM, Phil Edworthy wrote: > > Since the way the Synopsys GPIO interrupts are stored has changed, > > this driver needs to be updated in line with the changes. > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > --- > > v2: > > - New patch in v2 to fix the only other user of struct > dwapb_port_property. > > Thanks for your patch! > > To avoid bisection compile failures due to the changed type of > dwapb_port_property.irq, I think this should be folded into the first patch, Right, thanks for pointing this out! Phil > > --- > > drivers/mfd/intel_quark_i2c_gpio.c | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/mfd/intel_quark_i2c_gpio.c > > b/drivers/mfd/intel_quark_i2c_gpio.c > > index 90e35de..5bddb84 100644 > > --- a/drivers/mfd/intel_quark_i2c_gpio.c > > +++ b/drivers/mfd/intel_quark_i2c_gpio.c > > @@ -233,7 +233,8 @@ static int intel_quark_gpio_setup(struct pci_dev > *pdev, struct mfd_cell *cell) > > pdata->properties->idx = 0; > > pdata->properties->ngpio = INTEL_QUARK_MFD_NGPIO; > > pdata->properties->gpio_base = INTEL_QUARK_MFD_GPIO_BASE; > > - pdata->properties->irq = pdev->irq; > > + pdata->properties->irq[0] = pdev->irq; > > + pdata->properties->has_irq = true; > > pdata->properties->irq_shared = true; > > > > cell->platform_data = pdata; > > Gr{oetje,eeting}s, > > Geert > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux- > m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-04-14 0:47 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-04-13 8:08 [PATCH v2 0/2] gpio: dwapb: Add support for 1 interrupt per port A GPIO Phil Edworthy 2018-04-13 8:08 ` [PATCH v2 1/2] " Phil Edworthy 2018-04-13 22:16 ` kbuild test robot 2018-04-14 0:46 ` kbuild test robot 2018-04-13 8:08 ` [PATCH v2 2/2] mfd: intel_quark_i2c_gpio: Update Synopsys GPIO interrupts Phil Edworthy 2018-04-13 8:20 ` Geert Uytterhoeven 2018-04-13 8:24 ` Phil Edworthy
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).