* [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO [not found] <20260113161152.3688309-1-shorne@gmail.com> @ 2026-01-13 16:11 ` Stafford Horne 2026-01-13 16:20 ` Bartosz Golaszewski 2026-01-13 16:11 ` [PATCH v4 2/6] gpio: mmio: Add compatible for " Stafford Horne 1 sibling, 1 reply; 7+ messages in thread From: Stafford Horne @ 2026-01-13 16:11 UTC (permalink / raw) To: LKML Cc: Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Stafford Horne, Linus Walleij, Geert Uytterhoeven, Bartosz Golaszewski, linux-gpio Add a device tree binding for the opencores GPIO controller. On FPGA Development boards with GPIOs the OpenRISC architecture uses the opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. Link: https://opencores.org/projects/gpio Signed-off-by: Stafford Horne <shorne@gmail.com> Reviewed-by: Linus Walleij <linusw@kernel.org> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> --- Since v3: - Removed example. - Re-order this patch to be before adding compatible string to driver as per device tree binding patch rules. - Add Reviewed-by's. Since v2: - Fixup (replace) patch to simply add opencores,gpio and add an example. (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) Since v1: - Fix schema to actually match the example. Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml index ee5d5d25ae82..a8823ca65e78 100644 --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml @@ -23,6 +23,7 @@ properties: - ni,169445-nand-gpio - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller - intel,ixp4xx-expansion-bus-mmio-gpio + - opencores,gpio big-endian: true -- 2.51.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO 2026-01-13 16:11 ` [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO Stafford Horne @ 2026-01-13 16:20 ` Bartosz Golaszewski 2026-01-13 16:31 ` Stafford Horne 0 siblings, 1 reply; 7+ messages in thread From: Bartosz Golaszewski @ 2026-01-13 16:20 UTC (permalink / raw) To: Stafford Horne Cc: LKML, Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Walleij, Geert Uytterhoeven, linux-gpio On Tue, Jan 13, 2026 at 5:15 PM Stafford Horne <shorne@gmail.com> wrote: > > Add a device tree binding for the opencores GPIO controller. > > On FPGA Development boards with GPIOs the OpenRISC architecture uses the > opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. > > Link: https://opencores.org/projects/gpio > Signed-off-by: Stafford Horne <shorne@gmail.com> > Reviewed-by: Linus Walleij <linusw@kernel.org> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- > Since v3: > - Removed example. > - Re-order this patch to be before adding compatible string to driver as per > device tree binding patch rules. > - Add Reviewed-by's. > Since v2: > - Fixup (replace) patch to simply add opencores,gpio and add an example. > (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio > as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) > Since v1: > - Fix schema to actually match the example. > > Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > index ee5d5d25ae82..a8823ca65e78 100644 > --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > @@ -23,6 +23,7 @@ properties: > - ni,169445-nand-gpio > - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller > - intel,ixp4xx-expansion-bus-mmio-gpio > + - opencores,gpio > > big-endian: true > > -- > 2.51.0 > This is not a follow-up patch. Please rebase your fix on top of linux-next. I already have the previous patch in my tree and will not be rebasing the entire for-next branch. Bartosz ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO 2026-01-13 16:20 ` Bartosz Golaszewski @ 2026-01-13 16:31 ` Stafford Horne 2026-01-14 8:31 ` Krzysztof Kozlowski 0 siblings, 1 reply; 7+ messages in thread From: Stafford Horne @ 2026-01-13 16:31 UTC (permalink / raw) To: Bartosz Golaszewski Cc: LKML, Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Walleij, Geert Uytterhoeven, linux-gpio On Tue, Jan 13, 2026 at 05:20:28PM +0100, Bartosz Golaszewski wrote: > On Tue, Jan 13, 2026 at 5:15 PM Stafford Horne <shorne@gmail.com> wrote: > > > > Add a device tree binding for the opencores GPIO controller. > > > > On FPGA Development boards with GPIOs the OpenRISC architecture uses the > > opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. > > > > Link: https://opencores.org/projects/gpio > > Signed-off-by: Stafford Horne <shorne@gmail.com> > > Reviewed-by: Linus Walleij <linusw@kernel.org> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > --- > > Since v3: > > - Removed example. > > - Re-order this patch to be before adding compatible string to driver as per > > device tree binding patch rules. > > - Add Reviewed-by's. > > Since v2: > > - Fixup (replace) patch to simply add opencores,gpio and add an example. > > (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio > > as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) > > Since v1: > > - Fix schema to actually match the example. > > > > Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > index ee5d5d25ae82..a8823ca65e78 100644 > > --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > @@ -23,6 +23,7 @@ properties: > > - ni,169445-nand-gpio > > - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller > > - intel,ixp4xx-expansion-bus-mmio-gpio > > + - opencores,gpio > > > > big-endian: true > > > > -- > > 2.51.0 > > > > This is not a follow-up patch. Please rebase your fix on top of > linux-next. I already have the previous patch in my tree and will not > be rebasing the entire for-next branch. OK, understood, I wasn't aware you would not rebase. I will rework this rebasing on linux-next reberting my previous dt-binding: patch first. Thanks for the clarification. -Stafford ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO 2026-01-13 16:31 ` Stafford Horne @ 2026-01-14 8:31 ` Krzysztof Kozlowski 2026-01-14 8:36 ` Bartosz Golaszewski 0 siblings, 1 reply; 7+ messages in thread From: Krzysztof Kozlowski @ 2026-01-14 8:31 UTC (permalink / raw) To: Stafford Horne Cc: Bartosz Golaszewski, LKML, Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Walleij, Geert Uytterhoeven, linux-gpio On Tue, Jan 13, 2026 at 04:31:33PM +0000, Stafford Horne wrote: > On Tue, Jan 13, 2026 at 05:20:28PM +0100, Bartosz Golaszewski wrote: > > On Tue, Jan 13, 2026 at 5:15 PM Stafford Horne <shorne@gmail.com> wrote: > > > > > > Add a device tree binding for the opencores GPIO controller. > > > > > > On FPGA Development boards with GPIOs the OpenRISC architecture uses the > > > opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. > > > > > > Link: https://opencores.org/projects/gpio > > > Signed-off-by: Stafford Horne <shorne@gmail.com> > > > Reviewed-by: Linus Walleij <linusw@kernel.org> > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > --- > > > Since v3: > > > - Removed example. > > > - Re-order this patch to be before adding compatible string to driver as per > > > device tree binding patch rules. > > > - Add Reviewed-by's. > > > Since v2: > > > - Fixup (replace) patch to simply add opencores,gpio and add an example. > > > (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio > > > as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) > > > Since v1: > > > - Fix schema to actually match the example. > > > > > > Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > index ee5d5d25ae82..a8823ca65e78 100644 > > > --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > @@ -23,6 +23,7 @@ properties: > > > - ni,169445-nand-gpio > > > - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller > > > - intel,ixp4xx-expansion-bus-mmio-gpio > > > + - opencores,gpio > > > > > > big-endian: true > > > > > > -- > > > 2.51.0 > > > > > > > This is not a follow-up patch. Please rebase your fix on top of > > linux-next. I already have the previous patch in my tree and will not > > be rebasing the entire for-next branch. > > OK, understood, I wasn't aware you would not rebase. I will rework this rebasing > on linux-next reberting my previous dt-binding: patch first. No, you need to rebase, not revert. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO 2026-01-14 8:31 ` Krzysztof Kozlowski @ 2026-01-14 8:36 ` Bartosz Golaszewski 2026-01-14 14:24 ` Stafford Horne 0 siblings, 1 reply; 7+ messages in thread From: Bartosz Golaszewski @ 2026-01-14 8:36 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Stafford Horne, LKML, Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Walleij, Geert Uytterhoeven, linux-gpio On Wed, Jan 14, 2026 at 9:31 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On Tue, Jan 13, 2026 at 04:31:33PM +0000, Stafford Horne wrote: > > On Tue, Jan 13, 2026 at 05:20:28PM +0100, Bartosz Golaszewski wrote: > > > On Tue, Jan 13, 2026 at 5:15 PM Stafford Horne <shorne@gmail.com> wrote: > > > > > > > > Add a device tree binding for the opencores GPIO controller. > > > > > > > > On FPGA Development boards with GPIOs the OpenRISC architecture uses the > > > > opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. > > > > > > > > Link: https://opencores.org/projects/gpio > > > > Signed-off-by: Stafford Horne <shorne@gmail.com> > > > > Reviewed-by: Linus Walleij <linusw@kernel.org> > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > --- > > > > Since v3: > > > > - Removed example. > > > > - Re-order this patch to be before adding compatible string to driver as per > > > > device tree binding patch rules. > > > > - Add Reviewed-by's. > > > > Since v2: > > > > - Fixup (replace) patch to simply add opencores,gpio and add an example. > > > > (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio > > > > as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) > > > > Since v1: > > > > - Fix schema to actually match the example. > > > > > > > > Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + > > > > 1 file changed, 1 insertion(+) > > > > > > > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > index ee5d5d25ae82..a8823ca65e78 100644 > > > > --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > @@ -23,6 +23,7 @@ properties: > > > > - ni,169445-nand-gpio > > > > - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller > > > > - intel,ixp4xx-expansion-bus-mmio-gpio > > > > + - opencores,gpio > > > > > > > > big-endian: true > > > > > > > > -- > > > > 2.51.0 > > > > > > > > > > This is not a follow-up patch. Please rebase your fix on top of > > > linux-next. I already have the previous patch in my tree and will not > > > be rebasing the entire for-next branch. > > > > OK, understood, I wasn't aware you would not rebase. I will rework this rebasing > > on linux-next reberting my previous dt-binding: patch first. > > No, you need to rebase, not revert. > Just to be extra-clear: take current linux-next (next-20260114) and - on top of it - create a patch that fixes the issues in an incremental way. No reverts. Also: add the Fixes: tag to your commit before your SoB. Bartosz ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO 2026-01-14 8:36 ` Bartosz Golaszewski @ 2026-01-14 14:24 ` Stafford Horne 0 siblings, 0 replies; 7+ messages in thread From: Stafford Horne @ 2026-01-14 14:24 UTC (permalink / raw) To: Bartosz Golaszewski Cc: Krzysztof Kozlowski, LKML, Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Linus Walleij, Geert Uytterhoeven, linux-gpio On Wed, Jan 14, 2026 at 09:36:55AM +0100, Bartosz Golaszewski wrote: > On Wed, Jan 14, 2026 at 9:31 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > > > On Tue, Jan 13, 2026 at 04:31:33PM +0000, Stafford Horne wrote: > > > On Tue, Jan 13, 2026 at 05:20:28PM +0100, Bartosz Golaszewski wrote: > > > > On Tue, Jan 13, 2026 at 5:15 PM Stafford Horne <shorne@gmail.com> wrote: > > > > > > > > > > Add a device tree binding for the opencores GPIO controller. > > > > > > > > > > On FPGA Development boards with GPIOs the OpenRISC architecture uses the > > > > > opencores gpio verilog rtl which is compatible with the MMIO GPIO driver. > > > > > > > > > > Link: https://opencores.org/projects/gpio > > > > > Signed-off-by: Stafford Horne <shorne@gmail.com> > > > > > Reviewed-by: Linus Walleij <linusw@kernel.org> > > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > > > --- > > > > > Since v3: > > > > > - Removed example. > > > > > - Re-order this patch to be before adding compatible string to driver as per > > > > > device tree binding patch rules. > > > > > - Add Reviewed-by's. > > > > > Since v2: > > > > > - Fixup (replace) patch to simply add opencores,gpio and add an example. > > > > > (It was incorrect to specifying opencores,gpio with brcm,bcm6345-gpio > > > > > as opencores,gpio is not the same hardware, its 8-bit vs 32-bit) > > > > > Since v1: > > > > > - Fix schema to actually match the example. > > > > > > > > > > Documentation/devicetree/bindings/gpio/gpio-mmio.yaml | 1 + > > > > > 1 file changed, 1 insertion(+) > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > > index ee5d5d25ae82..a8823ca65e78 100644 > > > > > --- a/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > > +++ b/Documentation/devicetree/bindings/gpio/gpio-mmio.yaml > > > > > @@ -23,6 +23,7 @@ properties: > > > > > - ni,169445-nand-gpio > > > > > - wd,mbl-gpio # Western Digital MyBook Live memory-mapped GPIO controller > > > > > - intel,ixp4xx-expansion-bus-mmio-gpio > > > > > + - opencores,gpio > > > > > > > > > > big-endian: true > > > > > > > > > > -- > > > > > 2.51.0 > > > > > > > > > > > > > This is not a follow-up patch. Please rebase your fix on top of > > > > linux-next. I already have the previous patch in my tree and will not > > > > be rebasing the entire for-next branch. > > > > > > OK, understood, I wasn't aware you would not rebase. I will rework this rebasing > > > on linux-next reberting my previous dt-binding: patch first. > > > > No, you need to rebase, not revert. > > > > Just to be extra-clear: take current linux-next (next-20260114) and - > on top of it - create a patch that fixes the issues in an incremental > way. No reverts. Also: add the Fixes: tag to your commit before your > SoB. Got it, I was doing this but preparing the series with a revert. Good thing I waited a bit before sending. -Stafford ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4 2/6] gpio: mmio: Add compatible for opencores GPIO [not found] <20260113161152.3688309-1-shorne@gmail.com> 2026-01-13 16:11 ` [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO Stafford Horne @ 2026-01-13 16:11 ` Stafford Horne 1 sibling, 0 replies; 7+ messages in thread From: Stafford Horne @ 2026-01-13 16:11 UTC (permalink / raw) To: LKML Cc: Linux OpenRISC, devicetree, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Stafford Horne, Linus Walleij, Geert Uytterhoeven, Bartosz Golaszewski, linux-gpio On FPGA Development boards with GPIOs the OpenRISC architecture uses the opencores gpio verilog rtl. This is compatible with the gpio-mmio. Add the compatible string to allow probing this driver from the devicetree. Link: https://opencores.org/projects/gpio Signed-off-by: Stafford Horne <shorne@gmail.com> Reviewed-by: Linus Walleij <linusw@kernel.org> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> --- Since v3: - Order this patch after the binding patch. - Add Reviewed-by's. Since v2: - New patch drivers/gpio/gpio-mmio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpio/gpio-mmio.c b/drivers/gpio/gpio-mmio.c index 5daf962b0323..edbcaad57d00 100644 --- a/drivers/gpio/gpio-mmio.c +++ b/drivers/gpio/gpio-mmio.c @@ -724,6 +724,7 @@ static const struct of_device_id gpio_mmio_of_match[] = { { .compatible = "wd,mbl-gpio" }, { .compatible = "ni,169445-nand-gpio" }, { .compatible = "intel,ixp4xx-expansion-bus-mmio-gpio" }, + { .compatible = "opencores,gpio" }, { } }; MODULE_DEVICE_TABLE(of, gpio_mmio_of_match); -- 2.51.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2026-01-14 14:24 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20260113161152.3688309-1-shorne@gmail.com>
2026-01-13 16:11 ` [PATCH v4 1/6] dt-bindings: gpio-mmio: Add opencores GPIO Stafford Horne
2026-01-13 16:20 ` Bartosz Golaszewski
2026-01-13 16:31 ` Stafford Horne
2026-01-14 8:31 ` Krzysztof Kozlowski
2026-01-14 8:36 ` Bartosz Golaszewski
2026-01-14 14:24 ` Stafford Horne
2026-01-13 16:11 ` [PATCH v4 2/6] gpio: mmio: Add compatible for " Stafford Horne
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox