diff mbox series

arm64: dts: Change pinconf controller node name to pinctrl

Message ID 20230510103816.39015-1-tony@atomide.com
State New
Headers show
Series arm64: dts: Change pinconf controller node name to pinctrl | expand

Commit Message

Tony Lindgren May 10, 2023, 10:38 a.m. UTC
According to the pinctrl binding pinmux and pinctrl are valid controller
names. Let's replace pinconf with pinctrl so we don't get new warnings
when pinctrl-singl yaml binding gets merged.

Cc: Conor Dooley <conor+dt@kernel.org>
Cc: Dinh Nguyen <dinguyen@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Niravkumar L Rabara <niravkumar.l.rabara@intel.com>
Cc: Ray Jui <rjui@broadcom.com>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Wei Xu <xuwei5@hisilicon.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
 arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi | 2 +-
 arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi              | 2 +-
 arch/arm64/boot/dts/intel/socfpga_agilex.dtsi               | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Comments

Florian Fainelli May 18, 2023, 10:56 p.m. UTC | #1
On 5/10/23 03:38, Tony Lindgren wrote:
> According to the pinctrl binding pinmux and pinctrl are valid controller
> names. Let's replace pinconf with pinctrl so we don't get new warnings
> when pinctrl-singl yaml binding gets merged.
> 
> Cc: Conor Dooley <conor+dt@kernel.org>
> Cc: Dinh Nguyen <dinguyen@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: Niravkumar L Rabara <niravkumar.l.rabara@intel.com>
> Cc: Ray Jui <rjui@broadcom.com>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Scott Branden <sbranden@broadcom.com>
> Cc: Wei Xu <xuwei5@hisilicon.com>
> Signed-off-by: Tony Lindgren <tony@atomide.com>

Acked-by: Florian Fainelli <florian.fainelli@broadcom.com> 
#stingray-pinctrl.dtsi
Dinh Nguyen June 13, 2023, 10:40 p.m. UTC | #2
On 5/10/23 07:10, Krzysztof Kozlowski wrote:
> On 10/05/2023 12:53, Tony Lindgren wrote:
>> * Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [230510 10:51]:
>>> On 10/05/2023 12:38, Tony Lindgren wrote:
>>>> According to the pinctrl binding pinmux and pinctrl are valid controller
>>>> names. Let's replace pinconf with pinctrl so we don't get new warnings
>>>> when pinctrl-singl yaml binding gets merged.
>>>>
>>>> Cc: Conor Dooley <conor+dt@kernel.org>
>>>> Cc: Dinh Nguyen <dinguyen@kernel.org>
>>>> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
>>>> Cc: Niravkumar L Rabara <niravkumar.l.rabara@intel.com>
>>>> Cc: Ray Jui <rjui@broadcom.com>
>>>> Cc: Rob Herring <robh+dt@kernel.org>
>>>> Cc: Scott Branden <sbranden@broadcom.com>
>>>> Cc: Wei Xu <xuwei5@hisilicon.com>
>>>> Signed-off-by: Tony Lindgren <tony@atomide.com>
>>>> ---
>>>>   arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi | 2 +-
>>>>   arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi              | 2 +-
>>>>   arch/arm64/boot/dts/intel/socfpga_agilex.dtsi               | 2 +-
>>>>   3 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> These should be either split by subsystem or you will have to push it
>>> through soc.
>>
> 
> FWIW:
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> Best regards,
> Krzysztof
> 

For socfpga_agilex:

Acked-by: Dinh Nguyen <dinguyen@kernel.org>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi b/arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi
--- a/arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi
+++ b/arch/arm64/boot/dts/broadcom/stingray/stingray-pinctrl.dtsi
@@ -32,7 +32,7 @@ 
 
 #include <dt-bindings/pinctrl/brcm,pinctrl-stingray.h>
 
-		pinconf: pinconf@140000 {
+		pinconf: pinctrl@140000 {
 			compatible = "pinconf-single";
 			reg = <0x00140000 0x250>;
 			pinctrl-single,register-width = <32>;
diff --git a/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi b/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
--- a/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
@@ -172,7 +172,7 @@  combphy1: phy@858 {
 			};
 		};
 
-		pmx0: pinconf@8a21000 {
+		pmx0: pinctrl@8a21000 {
 			compatible = "pinconf-single";
 			reg = <0x8a21000 0x180>;
 			pinctrl-single,register-width = <32>;
diff --git a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
--- a/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
+++ b/arch/arm64/boot/dts/intel/socfpga_agilex.dtsi
@@ -365,7 +365,7 @@  pinctrl0: pinctrl@ffd13000 {
 			pinctrl-single,function-mask = <0x0000000f>;
 		};
 
-		pinctrl1: pinconf@ffd13100 {
+		pinctrl1: pinctrl@ffd13100 {
 			compatible = "pinctrl-single";
 			#pinctrl-cells = <1>;
 			reg = <0xffd13100 0x20>;