On Wed, Feb 14, 2024 at 5:24 PM Théo Lebrun <theo.lebrun@xxxxxxxxxxx> wrote: > > 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 reset available). > > 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. > I hope Linus can leave his Tested-by under this series then. > We have some dependencies, kept neatly to the end. Those are: > - The base platform support series from Grégory [1]. This relates to the > last four patches (20 thru 23), ie defconfig and devicetree. > - The OLB syscon support series [0]. It provides reset and pinctrl nodes > inside the devicetree. This relates to the last two patches (22 and > 23), ie resets and gpio-ranges DT props. GPIO works fine without it > if patches 22 and 23 are dropped. > > 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 > Are you targeting the GPIO branch with this or pinctrl? I guess GPIO so I'll need Linus' Acks under the pinctrl patches. > [0]: https://lore.kernel.org/lkml/20240212-mbly-clk-v6-0-c46fa1f93839@xxxxxxxxxxx/ > [1]: https://lore.kernel.org/lkml/20240205153503.574468-1-gregory.clement@xxxxxxxxxxx/ Please advise the relevant maintainers that they should provide an immutable branch for these series once they're queued in their respective trees. Bart [snip]