2022-03-11 16:21:21

by Fabien Dessenne

Subject: [PATCH] ARM: dts: stm32: fix pinctrl node name warnings (MPU soc)

The recent addition pinctrl.yaml in commit c09acbc499e8 ("dt-bindings:
pinctrl: use pinctrl.yaml") resulted in some node name warnings.
Fix the node names to the preferred 'pinctrl'.

Signed-off-by: Fabien Dessenne <[email protected]>
---
arch/arm/boot/dts/stm32mp131.dtsi | 2 +-
arch/arm/boot/dts/stm32mp151.dtsi | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/stm32mp131.dtsi b/arch/arm/boot/dts/stm32mp131.dtsi
index 86126dc0d898..57a911cddebe 100644
--- a/arch/arm/boot/dts/stm32mp131.dtsi
+++ b/arch/arm/boot/dts/stm32mp131.dtsi
@@ -164,7 +164,7 @@ ts_cal2: calib@5e {
* Break node order to solve dependency probe issue between
* pinctrl and exti.
*/
- pinctrl: pin-controller@50002000 {
+ pinctrl: pinctrl@50002000 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "st,stm32mp135-pinctrl";
diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
index 1cfc2f011e70..489fb9979085 100644
--- a/arch/arm/boot/dts/stm32mp151.dtsi
+++ b/arch/arm/boot/dts/stm32mp151.dtsi
@@ -1602,7 +1602,7 @@ tamp: tamp@5c00a000 {
* Break node order to solve dependency probe issue between
* pinctrl and exti.
*/
- pinctrl: pin-controller@50002000 {
+ pinctrl: pinctrl@50002000 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "st,stm32mp157-pinctrl";
@@ -1733,7 +1733,7 @@ gpiok: gpio@5000c000 {
};
};

- pinctrl_z: pin-controller-z@54004000 {
+ pinctrl_z: pinctrl@54004000 {
#address-cells = <1>;
#size-cells = <1>;
compatible = "st,stm32mp157-z-pinctrl";
--
2.25.1

2022-04-22 13:33:48

by Alexandre TORGUE

Subject: Re: [PATCH] ARM: dts: stm32: fix pinctrl node name warnings (MPU soc)

On 3/11/22 13:13, Fabien Dessenne wrote:
> The recent addition pinctrl.yaml in commit c09acbc499e8 ("dt-bindings:
> pinctrl: use pinctrl.yaml") resulted in some node name warnings.
> Fix the node names to the preferred 'pinctrl'.
>
> Signed-off-by: Fabien Dessenne <[email protected]>
> ---
> arch/arm/boot/dts/stm32mp131.dtsi | 2 +-
> arch/arm/boot/dts/stm32mp151.dtsi | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/boot/dts/stm32mp131.dtsi b/arch/arm/boot/dts/stm32mp131.dtsi
> index 86126dc0d898..57a911cddebe 100644
> --- a/arch/arm/boot/dts/stm32mp131.dtsi
> +++ b/arch/arm/boot/dts/stm32mp131.dtsi
> @@ -164,7 +164,7 @@ ts_cal2: calib@5e {
> * Break node order to solve dependency probe issue between
> * pinctrl and exti.
> */
> - pinctrl: pin-controller@50002000 {
> + pinctrl: pinctrl@50002000 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "st,stm32mp135-pinctrl";
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi b/arch/arm/boot/dts/stm32mp151.dtsi
> index 1cfc2f011e70..489fb9979085 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -1602,7 +1602,7 @@ tamp: tamp@5c00a000 {
> * Break node order to solve dependency probe issue between
> * pinctrl and exti.
> */
> - pinctrl: pin-controller@50002000 {
> + pinctrl: pinctrl@50002000 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "st,stm32mp157-pinctrl";
> @@ -1733,7 +1733,7 @@ gpiok: gpio@5000c000 {
> };
> };
>
> - pinctrl_z: pin-controller-z@54004000 {
> + pinctrl_z: pinctrl@54004000 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "st,stm32mp157-z-pinctrl";

Applied on stm32-next.

Thanks.
Alex