Hi Linus, On mar., mars 21 2017, Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> wrote: > Hi, > > Obviously this cover letter is wrong. Actually I forgot to commit it in > git series. Here it is the correct one: > > After several months here it is the second version of the series adding > support for the pin and gpio controllers present on the Armada 37xx > SoCs. As the first version was sent in December here it is a link to it: http://lists.infradead.org/pipermail/linux-arm-kernel/2016-December/475216.html Also I am going to answer to your review to point what I have done in this new series. Gregory > > Each Armada 37xx SoC comes with 2 pin controllers: one on the south > bridge (managing 28 pins) and one on the north bridge (managing 36 pins). > > At the hardware level the controller configure the pins by group and not > pin by pin. > > The gpio controller is also capable to handle interrupt from gpio. > > In the second version several changes has been done: > > - Update binding documentation making clear that mfd and syscon must > be used (patch 1). > > - Split the fist patch adding pin controller support for Armada 37xx > in arm64 part (for kconfig) and pinctrl part (patch 2 and 3) > > - Add MFD_SYSCON dependency (patch 3) > > - Add kerneldoc for the armada_37xx_pin_group struct (patch 3) > > - Rename _add_function() to armada_37xx_add_function() (patch 3) > > - Use an inline function to update the reg offset (patch 4) > > - Rename gpiolib_register to gpiochip_register (patch 4) > > - Add a comment about the two registers limit (patch 4) > > - Add explicit gpio node in the device tree (patch 4) > > - Convert the driver to use GPIOLIB_IRQCHIP (patch 5) > > - Add a critical section when accessing the hardware registers (patch 5) > > - Use the gpio sub-node (patch 5) > > With these change most of the comment of the first revision have been addressed. > > Thanks, > > Gregory > > Gregory CLEMENT (7): > pinctrl: dt-bindings: Add documentation for Armada 37xx pin controllers > arm64: marvell: enable the Armada 37xx pinctrl driver > pinctrl: armada-37xx: Add pin controller support for Armada 37xx > pinctrl: armada-37xx: Add gpio support > pinctrl: aramda-37xx: Add irqchip support > ARM64: dts: marvell: Add pinctrl nodes for Armada 3700 > ARM64: dts: marvell: armada37xx: add pinctrl definition > > Documentation/devicetree/bindings/clock/armada3700-xtal-clock.txt | 7 +- > Documentation/devicetree/bindings/pinctrl/marvell,armada-37xx-pinctrl.txt | 143 +++++++++++- > arch/arm64/Kconfig.platforms | 2 +- > arch/arm64/boot/dts/marvell/armada-3720-db.dts | 8 +- > arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 71 +++++- > drivers/pinctrl/Makefile | 2 +- > drivers/pinctrl/mvebu/Kconfig | 7 +- > drivers/pinctrl/mvebu/Makefile | 3 +- > drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 918 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- > 9 files changed, 1153 insertions(+), 8 deletions(-) > create mode 100644 Documentation/devicetree/bindings/pinctrl/marvell,armada-37xx-pinctrl.txt > create mode 100644 drivers/pinctrl/mvebu/pinctrl-armada-37xx.c > > Sorry for the noise, > > Gregory > > > On mar., mars 21 2017, Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx> wrote: > >> this series add support for the pin and gpio controllers present on >> the Armada 37xx SoCs. >> >> Each Armada 37xx SoC comes with 2 pin controllers: one on the south >> bridge (managing 28 pins) and one on the north bridge (managing 36 pins). >> >> At the hardware level the controller configure the pins by group and not >> pin by pin. >> >> The gpio controller is also capable to handle interrupt from gpio. >> >> Gregory >> >> Gregory CLEMENT (7): >> pinctrl: dt-bindings: Add documentation for Armada 37xx pin controllers >> arm64: marvell: enable the Armada 37xx pinctrl driver >> pinctrl: armada-37xx: Add pin controller support for Armada 37xx >> pinctrl: armada-37xx: Add gpio support >> pinctrl: aramda-37xx: Add irqchip support >> ARM64: dts: marvell: Add pinctrl nodes for Armada 3700 >> ARM64: dts: marvell: armada37xx: add pinctrl definition >> >> Documentation/devicetree/bindings/clock/armada3700-xtal-clock.txt | 7 +- >> Documentation/devicetree/bindings/pinctrl/marvell,armada-37xx-pinctrl.txt | 143 +++++++++++- >> arch/arm64/Kconfig.platforms | 2 +- >> arch/arm64/boot/dts/marvell/armada-3720-db.dts | 8 +- >> arch/arm64/boot/dts/marvell/armada-37xx.dtsi | 71 +++++- >> drivers/pinctrl/Makefile | 2 +- >> drivers/pinctrl/mvebu/Kconfig | 7 +- >> drivers/pinctrl/mvebu/Makefile | 3 +- >> drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 918 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- >> 9 files changed, 1153 insertions(+), 8 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/pinctrl/marvell,armada-37xx-pinctrl.txt >> create mode 100644 drivers/pinctrl/mvebu/pinctrl-armada-37xx.c >> >> base-commit: c1ae3cfa0e89fa1a7ecc4c99031f5e9ae99d9201 >> -- >> git-series 0.9.1 > > -- > Gregory Clement, Free Electrons > Kernel, drivers, real-time and embedded Linux > development, consulting, training and support. > http://free-electrons.com > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html