Re: [PATCH] arm64: dts: Change pinconf controller node name to pinctrl

Re: [PATCH] arm64: dts: Change pinconf controller node name to pinctrl

[Florian Fainelli]

[-- Attachment #1.1: Type: text/plain, Size: 738 bytes --]

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
-- 
Florian


[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4221 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: Change pinconf controller node name to pinctrl

[Wei Xu]

Hi Tony,

On 2023/5/10 18: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: Wei Xu <xuwei5@hisilicon.com> #hisilicon
Thanks!

Best Regards,
Wei

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] arm64: dts: Change pinconf controller node name to pinctrl

[Dinh Nguyen]

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>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel