From: Alexandre Torgue <alexandre.torgue@st.com>
To: Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Linus Walleij <linus.walleij@linaro.org>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
arnd@arndb.de
Cc: devicetree@vger.kernel.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-gpio@vger.kernel.org
Subject: Re: [PATCH 5/5] ARM: dts: stm32: use dedicated files to manage stm32mp157 packages
Date: Tue, 21 May 2019 09:48:49 +0200 [thread overview]
Message-ID: <b236861c-2a25-d5a9-c892-3d2a33bacf30@st.com> (raw)
In-Reply-To: <1554895824-25709-6-git-send-email-alexandre.torgue@st.com>
Hi,
On 4/10/19 1:30 PM, Alexandre Torgue wrote:
> Four packages exist for stm32mp157 die. As ball-out is different between
> them, this patch covers those differences by creating dedicated pinctrl
> dtsi files. Each dtsi pinctrl package file describes the package ball-out
> through gpio-ranges.
>
> stm32mp157a-dk1 / dk2 boards embed a STM32MP_PKG_AC (TFBGA361 (12*12))
> package.
>
> stm32mp157c-ed1 / ev1 boards embed a STM32MP_PKG_AA (LFBGA448 (18*18))
> package.
>
> Signed-off-by: Alexandre Torgue <alexandre.torgue@st.com>
>
Applied on stm32-next.
Thanks.
Alex
> diff --git a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> index 9104896..3b4cc3c 100644
> --- a/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/stm32mp157-pinctrl.dtsi
> @@ -26,6 +26,7 @@
> st,bank-name = "GPIOA";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 0 16>;
> + status = "disabled";
> };
>
> gpiob: gpio@50003000 {
> @@ -38,6 +39,7 @@
> st,bank-name = "GPIOB";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 16 16>;
> + status = "disabled";
> };
>
> gpioc: gpio@50004000 {
> @@ -50,6 +52,7 @@
> st,bank-name = "GPIOC";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 32 16>;
> + status = "disabled";
> };
>
> gpiod: gpio@50005000 {
> @@ -62,6 +65,7 @@
> st,bank-name = "GPIOD";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 48 16>;
> + status = "disabled";
> };
>
> gpioe: gpio@50006000 {
> @@ -74,6 +78,7 @@
> st,bank-name = "GPIOE";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 64 16>;
> + status = "disabled";
> };
>
> gpiof: gpio@50007000 {
> @@ -86,6 +91,7 @@
> st,bank-name = "GPIOF";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 80 16>;
> + status = "disabled";
> };
>
> gpiog: gpio@50008000 {
> @@ -98,6 +104,7 @@
> st,bank-name = "GPIOG";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 96 16>;
> + status = "disabled";
> };
>
> gpioh: gpio@50009000 {
> @@ -110,6 +117,7 @@
> st,bank-name = "GPIOH";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 112 16>;
> + status = "disabled";
> };
>
> gpioi: gpio@5000a000 {
> @@ -122,6 +130,7 @@
> st,bank-name = "GPIOI";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 128 16>;
> + status = "disabled";
> };
>
> gpioj: gpio@5000b000 {
> @@ -134,6 +143,7 @@
> st,bank-name = "GPIOJ";
> ngpios = <16>;
> gpio-ranges = <&pinctrl 0 144 16>;
> + status = "disabled";
> };
>
> gpiok: gpio@5000c000 {
> @@ -146,6 +156,7 @@
> st,bank-name = "GPIOK";
> ngpios = <8>;
> gpio-ranges = <&pinctrl 0 160 8>;
> + status = "disabled";
> };
>
> cec_pins_a: cec-0 {
> @@ -427,6 +438,7 @@
> st,bank-ioport = <11>;
> ngpios = <8>;
> gpio-ranges = <&pinctrl_z 0 400 8>;
> + status = "disabled";
> };
>
> i2c4_pins_a: i2c4-0 {
> diff --git a/arch/arm/boot/dts/stm32mp157a-dk1.dts b/arch/arm/boot/dts/stm32mp157a-dk1.dts
> index 1b1886d..8d60641 100644
> --- a/arch/arm/boot/dts/stm32mp157a-dk1.dts
> +++ b/arch/arm/boot/dts/stm32mp157a-dk1.dts
> @@ -7,7 +7,7 @@
> /dts-v1/;
>
> #include "stm32mp157c.dtsi"
> -#include "stm32mp157-pinctrl.dtsi"
> +#include "stm32mp157xac-pinctrl.dtsi"
> #include <dt-bindings/gpio/gpio.h>
>
> / {
> diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> index 9fd7943..6ed9d49 100644
> --- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
> +++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
> @@ -6,7 +6,7 @@
> /dts-v1/;
>
> #include "stm32mp157c.dtsi"
> -#include "stm32mp157-pinctrl.dtsi"
> +#include "stm32mp157xaa-pinctrl.dtsi"
> #include <dt-bindings/gpio/gpio.h>
>
> / {
> diff --git a/arch/arm/boot/dts/stm32mp157xaa-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157xaa-pinctrl.dtsi
> new file mode 100644
> index 0000000..875adf5
> --- /dev/null
> +++ b/arch/arm/boot/dts/stm32mp157xaa-pinctrl.dtsi
> @@ -0,0 +1,90 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) STMicroelectronics 2019 - All Rights Reserved
> + * Author: Alexandre Torgue <alexandre.torgue@st.com>
> + */
> +
> +#include "stm32mp157-pinctrl.dtsi"
> +/ {
> + soc {
> + pinctrl: pin-controller@50002000 {
> + st,package = <STM32MP_PKG_AA>;
> +
> + gpioa: gpio@50002000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 0 16>;
> + };
> +
> + gpiob: gpio@50003000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 16 16>;
> + };
> +
> + gpioc: gpio@50004000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 32 16>;
> + };
> +
> + gpiod: gpio@50005000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 48 16>;
> + };
> +
> + gpioe: gpio@50006000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 64 16>;
> + };
> +
> + gpiof: gpio@50007000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 80 16>;
> + };
> +
> + gpiog: gpio@50008000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 96 16>;
> + };
> +
> + gpioh: gpio@50009000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 112 16>;
> + };
> +
> + gpioi: gpio@5000a000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 128 16>;
> + };
> +
> + gpioj: gpio@5000b000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 144 16>;
> + };
> +
> + gpiok: gpio@5000c000 {
> + status = "okay";
> + ngpios = <8>;
> + gpio-ranges = <&pinctrl 0 160 8>;
> + };
> + };
> +
> + pinctrl_z: pin-controller-z@54004000 {
> + st,package = <STM32MP_PKG_AA>;
> +
> + gpioz: gpio@54004000 {
> + status = "okay";
> + ngpios = <8>;
> + gpio-ranges = <&pinctrl_z 0 400 8>;
> + };
> + };
> + };
> +};
> diff --git a/arch/arm/boot/dts/stm32mp157xab-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157xab-pinctrl.dtsi
> new file mode 100644
> index 0000000..961fa12
> --- /dev/null
> +++ b/arch/arm/boot/dts/stm32mp157xab-pinctrl.dtsi
> @@ -0,0 +1,62 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) STMicroelectronics 2019 - All Rights Reserved
> + * Author: Alexandre Torgue <alexandre.torgue@st.com>
> + */
> +
> +#include "stm32mp157-pinctrl.dtsi"
> +/ {
> + soc {
> + pinctrl: pin-controller@50002000 {
> + st,package = <STM32MP_PKG_AB>;
> +
> + gpioa: gpio@50002000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 0 16>;
> + };
> +
> + gpiob: gpio@50003000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 16 16>;
> + };
> +
> + gpioc: gpio@50004000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 32 16>;
> + };
> +
> + gpiod: gpio@50005000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 48 16>;
> + };
> +
> + gpioe: gpio@50006000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 64 16>;
> + };
> +
> + gpiof: gpio@50007000 {
> + status = "okay";
> + ngpios = <6>;
> + gpio-ranges = <&pinctrl 6 86 6>;
> + };
> +
> + gpiog: gpio@50008000 {
> + status = "okay";
> + ngpios = <10>;
> + gpio-ranges = <&pinctrl 6 102 10>;
> + };
> +
> + gpioh: gpio@50009000 {
> + status = "okay";
> + ngpios = <2>;
> + gpio-ranges = <&pinctrl 0 112 2>;
> + };
> + };
> + };
> +};
> diff --git a/arch/arm/boot/dts/stm32mp157xac-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157xac-pinctrl.dtsi
> new file mode 100644
> index 0000000..26600f1
> --- /dev/null
> +++ b/arch/arm/boot/dts/stm32mp157xac-pinctrl.dtsi
> @@ -0,0 +1,78 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) STMicroelectronics 2019 - All Rights Reserved
> + * Author: Alexandre Torgue <alexandre.torgue@st.com>
> + */
> +
> +#include "stm32mp157-pinctrl.dtsi"
> +/ {
> + soc {
> + pinctrl: pin-controller@50002000 {
> + st,package = <STM32MP_PKG_AC>;
> +
> + gpioa: gpio@50002000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 0 16>;
> + };
> +
> + gpiob: gpio@50003000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 16 16>;
> + };
> +
> + gpioc: gpio@50004000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 32 16>;
> + };
> +
> + gpiod: gpio@50005000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 48 16>;
> + };
> +
> + gpioe: gpio@50006000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 64 16>;
> + };
> +
> + gpiof: gpio@50007000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 80 16>;
> + };
> +
> + gpiog: gpio@50008000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 96 16>;
> + };
> +
> + gpioh: gpio@50009000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 112 16>;
> + };
> +
> + gpioi: gpio@5000a000 {
> + status = "okay";
> + ngpios = <12>;
> + gpio-ranges = <&pinctrl 0 128 12>;
> + };
> + };
> +
> + pinctrl_z: pin-controller-z@54004000 {
> + st,package = <STM32MP_PKG_AC>;
> +
> + gpioz: gpio@54004000 {
> + status = "okay";
> + ngpios = <8>;
> + gpio-ranges = <&pinctrl_z 0 400 8>;
> + };
> + };
> + };
> +};
> diff --git a/arch/arm/boot/dts/stm32mp157xad-pinctrl.dtsi b/arch/arm/boot/dts/stm32mp157xad-pinctrl.dtsi
> new file mode 100644
> index 0000000..910113f
> --- /dev/null
> +++ b/arch/arm/boot/dts/stm32mp157xad-pinctrl.dtsi
> @@ -0,0 +1,62 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR BSD-3-Clause)
> +/*
> + * Copyright (C) STMicroelectronics 2019 - All Rights Reserved
> + * Author: Alexandre Torgue <alexandre.torgue@st.com>
> + */
> +
> +#include "stm32mp157-pinctrl.dtsi"
> +/ {
> + soc {
> + pinctrl: pin-controller@50002000 {
> + st,package = <STM32MP_PKG_AD>;
> +
> + gpioa: gpio@50002000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 0 16>;
> + };
> +
> + gpiob: gpio@50003000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 16 16>;
> + };
> +
> + gpioc: gpio@50004000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 32 16>;
> + };
> +
> + gpiod: gpio@50005000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 48 16>;
> + };
> +
> + gpioe: gpio@50006000 {
> + status = "okay";
> + ngpios = <16>;
> + gpio-ranges = <&pinctrl 0 64 16>;
> + };
> +
> + gpiof: gpio@50007000 {
> + status = "okay";
> + ngpios = <6>;
> + gpio-ranges = <&pinctrl 6 86 6>;
> + };
> +
> + gpiog: gpio@50008000 {
> + status = "okay";
> + ngpios = <10>;
> + gpio-ranges = <&pinctrl 6 102 10>;
> + };
> +
> + gpioh: gpio@50009000 {
> + status = "okay";
> + ngpios = <2>;
> + gpio-ranges = <&pinctrl 0 112 2>;
> + };
> + };
> + };
> +};
>
next prev parent reply other threads:[~2019-05-21 7:48 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-10 11:30 [PATCH 0/5] Add pin package management on STM32P157 Alexandre Torgue
2019-04-10 11:30 ` [PATCH 1/5] dt-bindings: pinctrl: stm32: add new entry for package information Alexandre Torgue
2019-04-10 11:30 ` [PATCH 2/5] pinctrl: stm32: introduce package support Alexandre Torgue
2019-04-10 11:30 ` [PATCH 3/5] pinctrl: stm32: add package information for stm32mp157c Alexandre Torgue
2019-04-10 11:30 ` [PATCH 4/5] pinctrl: stm32: align stm32mp157 pin names Alexandre Torgue
2019-04-10 11:30 ` [PATCH 5/5] ARM: dts: stm32: use dedicated files to manage stm32mp157 packages Alexandre Torgue
2019-04-23 8:48 ` Linus Walleij
2019-05-21 7:48 ` Alexandre Torgue [this message]
2019-04-23 8:47 ` [PATCH 0/5] Add pin package management on STM32P157 Linus Walleij
2019-04-23 9:02 ` Alexandre Torgue
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=b236861c-2a25-d5a9-c892-3d2a33bacf30@st.com \
--to=alexandre.torgue@st.com \
--cc=arnd@arndb.de \
--cc=devicetree@vger.kernel.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=mark.rutland@arm.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).