Hi, This patch series reworks the Nomadik GPIO driver to bring it up to date to current kernel standards. We then add Mobileye EyeQ5 support that uses the same IP block but with limited functionality. We also add features required by our newly supported platform: - Dynamic GPIO ID allocation; - Make clock optional; - Shared IRQ (usecase: EyeQ5 has two banks using the same IRQ); - Handle variadic GPIO counts (usecase: EyeQ5 has <32 GPIOs per bank); - Grab optional reset at probe (usecase: EyeQ5 has a shared GPIO reset). This GPIO platform driver was previously declared & registered inside drivers/pinctrl/nomadik/pinctrl-nomadik.c, side-by-side with the pinctrl driver. Both are tightly integrated, mostly for muxing reasons. Now that gpio-nomadik is used for another platform, we loosen the relationship. The behavior should not change on already supported hardware but I do not have Nomadik hardware to test for that. We have some dependencies. Those are: - The base platform support series from Grégory [0], present in mips-next. It in turns depends on [1], also in mips-next. This relates to the last four patches (27 thru 30), ie defconfig and devicetree. - The OLB syscon support series [2]. It provides reset and pinctrl nodes inside the devicetree. This relates to the last two patches (29 and 30), ie resets and gpio-ranges DT props. GPIO works fine if patches 29 and 30 are dropped and bootloader deasserts the reset (it does). This has been tested on the EyeQ5 hardware, with the two parent series applied. It also works fine without the OLB syscon series when our last two patches are removed. It has been built on both Arm defconfigs that rely on pinctrl-nomadik: nhk8815_defconfig and u8500_defconfig. I don't have any Nomadik hardware to test though. Have a nice day, Théo [0]: https://lore.kernel.org/lkml/20240216174227.409400-1-gregory.clement@xxxxxxxxxxx/ [1]: https://lore.kernel.org/linux-mips/20240209-regname-v1-0-2125efa016ef@xxxxxxxxxxx/ [2]: https://lore.kernel.org/lkml/20240227-mbly-clk-v8-0-c57fbda7664a@xxxxxxxxxxx/ To: Linus Walleij <linus.walleij@xxxxxxxxxx> To: Bartosz Golaszewski <brgl@xxxxxxxx> To: Rob Herring <robh+dt@xxxxxxxxxx> To: Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx> To: Conor Dooley <conor+dt@xxxxxxxxxx> To: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> To: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx> Cc: linux-gpio@xxxxxxxxxxxxxxx Cc: devicetree@xxxxxxxxxxxxxxx Cc: linux-kernel@xxxxxxxxxxxxxxx Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx Cc: linux-mips@xxxxxxxxxxxxxxx Cc: Gregory CLEMENT <gregory.clement@xxxxxxxxxxx> Cc: Vladimir Kondratiev <vladimir.kondratiev@xxxxxxxxxxxx> Cc: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx> Cc: Tawfik Bayouk <tawfik.bayouk@xxxxxxxxxxxx> Signed-off-by: Théo Lebrun <theo.lebrun@xxxxxxxxxxx> Changes in v2: - dt-bindings: add description to gpio-bank. - dt-bindings: fix disabling of st,supports-sleepmode prop with st,supports-sleepmode compatible. - gpio: move "fix offset bug in nmk_pmx_set()" to the start of the series and add a Fixes trailer. The patch changed because of that. - pinctrl: allow building PINCTRL_NOMADIK and co with COMPILE_TEST. - gpio: in "support shared GPIO IRQs" remove fake raw lock and use generic_handle_domain_irq_safe() helper. - gpio: remove OF API calls. of_property_read_*() => device_property_read_*() of_find_device_by_node() => bus_find_device_by_of_node() - gpio: use device_is_compatible() rather than match data to detect for Mobileye SoC. If GPIO device is populated by pinctrl match data is unavailable. - gpio: rename quirk_mbly field to is_mobileye_soc. - gpio: add comment about unbalanced reset_control_deassert() and disable bind sysfs attributes. - gpio: use devm_platform_ioremap_resource() helper. - gpio: use devres version of clk_get*() in case of probe failure. - gpio: add missing <linux/slab.h> include. - Documentation: gpio: replace outdated comment of using a fake spin lock and mention the generic_handle_irq_safe() helper. - gpio: in "follow whitespace kernel coding conventions", add missing newline before headers. - Take 6 Reviewed-By Linus, 1 Reviewed-by and 1 Acked-by Krzysztof. - Link to v1: https://lore.kernel.org/r/20240214-mbly-gpio-v1-0-f88c0ccf372b@xxxxxxxxxxx --- Théo Lebrun (30): dt-bindings: gpio: nomadik: convert into yaml format dt-bindings: gpio: nomadik: add optional ngpios property dt-bindings: gpio: nomadik: add mobileye,eyeq5-gpio compatible dt-bindings: gpio: nomadik: add optional reset property gpio: nomadik: fix offset bug in nmk_pmx_set() gpio: nomadik: extract GPIO platform driver from drivers/pinctrl/nomadik/ pinctrl: nomadik: Kconfig: allow building with COMPILE_TEST pinctrl: nomadik: fix build warning (-Wformat) pinctrl: nomadik: fix build warning (-Wpointer-to-int-cast) pinctrl: nomadik: minimise indentation in probe pinctrl: nomadik: follow type-system kernel coding conventions pinctrl: nomadik: follow whitespace kernel coding conventions pinctrl: nomadik: follow conditional kernel coding conventions gpio: nomadik: add #include <linux/slab.h> gpio: nomadik: replace of_find_*() by bus_find_device_by_of_node() gpio: nomadik: replace of_property_read_*() by device_property_read_*() gpio: nomadik: use devm_platform_ioremap_resource() helper gpio: nomadik: use devres version of clk_get*() gpio: nomadik: request dynamic ID allocation gpio: nomadik: make clock optional gpio: nomadik: change driver name from gpio to gpio-nomadik gpio: nomadik: support shared GPIO IRQs gpio: nomadik: handle variadic GPIO count gpio: nomadik: support mobileye,eyeq5-gpio gpio: nomadik: grab optional reset control and deassert it at probe Documentation: gpio: mention generic_handle_irq_safe() MIPS: eyeq5_defconfig: enable GPIO by default MIPS: mobileye: eyeq5: add two GPIO bank nodes MIPS: mobileye: eyeq5: add resets to GPIO banks MIPS: mobileye: eyeq5: map GPIOs to pins using gpio-ranges .../devicetree/bindings/gpio/gpio-nmk.txt | 31 - .../devicetree/bindings/gpio/st,nomadik-gpio.yaml | 95 +++ Documentation/driver-api/gpio/driver.rst | 11 +- MAINTAINERS | 2 + arch/mips/boot/dts/mobileye/eyeq5.dtsi | 30 + arch/mips/configs/eyeq5_defconfig | 2 + drivers/gpio/Kconfig | 13 + drivers/gpio/Makefile | 1 + drivers/gpio/gpio-nomadik.c | 722 ++++++++++++++++ drivers/pinctrl/nomadik/Kconfig | 7 +- drivers/pinctrl/nomadik/pinctrl-nomadik-db8500.c | 3 +- drivers/pinctrl/nomadik/pinctrl-nomadik-stn8815.c | 3 +- drivers/pinctrl/nomadik/pinctrl-nomadik.c | 939 +++------------------ .../linux/gpio/gpio-nomadik.h | 123 ++- 14 files changed, 1117 insertions(+), 865 deletions(-) --- base-commit: 3ebad8a9638d4c667e8925968806a82e1b9025ea change-id: 20231023-mbly-gpio-a30571ec3283 Best regards, -- Théo Lebrun <theo.lebrun@xxxxxxxxxxx>
