| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <87efxpfsji.fsf@free-electrons.com>
Date: Wed, 22 Mar 2017 12:40:49 +0100
From: Gregory CLEMENT <gregory.clement@...e-electrons.com>
To: Linus Walleij <linus.walleij@...aro.org>
Cc: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>,
Andrew Lunn <andrew@...n.ch>,
Jason Cooper <jason@...edaemon.net>,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Nadav Haklai <nadavh@...vell.com>, linux-gpio@...r.kernel.org,
Rob Herring <robh+dt@...nel.org>,
Neta Zur Hershkovits <neta@...vell.com>,
Victor Gu <xigu@...vell.com>, Hua Jing <jinghua@...vell.com>,
Marcin Wojtas <mw@...ihalf.com>,
Wilson Ding <dingwei@...vell.com>,
linux-arm-kernel@...ts.infradead.org,
Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>
Subject: Re: [PATCH v2 0/7] Add support for pinctrl/gpio on Armada 37xx Was Re: [PATCH v2 0/7] Hi,
Hi Linus,
On mar., mars 21 2017, Gregory CLEMENT <gregory.clement@...e-electrons.com> 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@...e-electrons.com> 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@...ts.infradead.org
> 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
Powered by blists - more mailing lists