linux-kernel - Re: [PATCH 4/5] ARM: dts: qcom: use GPIO flags for tlmm

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <e53cee9d-8eb8-6f8c-4b88-5dedb394a758@somainline.org>
Date:   Wed, 3 Aug 2022 17:35:39 +0200
From:   Konrad Dybcio <konrad.dybcio@...ainline.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Loic Poulain <loic.poulain@...aro.org>,
        Robert Foss <robert.foss@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        linux-i2c@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org
Cc:     corbet@....net
Subject: Re: [PATCH 4/5] ARM: dts: qcom: use GPIO flags for tlmm



On 2.08.2022 17:39, Krzysztof Kozlowski wrote:
> Use respective GPIO_ACTIVE_LOW/HIGH flags for tlmm GPIOs.  Include
> gpio.h header if this is first usage of that flag.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@...ainline.org>

Konrad
>  arch/arm/boot/dts/qcom-apq8074-dragonboard.dts           | 3 ++-
>  arch/arm/boot/dts/qcom-ipq4019-ap.dk01.1.dtsi            | 3 ++-
>  arch/arm/boot/dts/qcom-ipq4019-ap.dk04.1.dtsi            | 4 ++--
>  arch/arm/boot/dts/qcom-ipq4019-ap.dk07.1-c1.dts          | 5 +++--
>  arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts | 2 +-
>  5 files changed, 10 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> index 3051a861ff0c..91716298ec5e 100644
> --- a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> +++ b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts
> @@ -1,4 +1,5 @@
>  // SPDX-License-Identifier: GPL-2.0
> +#include <dt-bindings/gpio/gpio.h>
>  #include "qcom-msm8974.dtsi"
>  #include "qcom-pm8841.dtsi"
>  #include "qcom-pm8941.dtsi"
> @@ -261,7 +262,7 @@ &sdhc_1 {
>  &sdhc_2 {
>  	status = "okay";
>  
> -	cd-gpios = <&tlmm 62 0x1>;
> +	cd-gpios = <&tlmm 62 GPIO_ACTIVE_LOW>;
>  	vmmc-supply = <&pm8941_l21>;
>  	vqmmc-supply = <&pm8941_l13>;
>  
> diff --git a/arch/arm/boot/dts/qcom-ipq4019-ap.dk01.1.dtsi b/arch/arm/boot/dts/qcom-ipq4019-ap.dk01.1.dtsi
> index 03bb9e1768c4..0505270cf508 100644
> --- a/arch/arm/boot/dts/qcom-ipq4019-ap.dk01.1.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq4019-ap.dk01.1.dtsi
> @@ -14,6 +14,7 @@
>   *
>   */
>  
> +#include <dt-bindings/gpio/gpio.h>
>  #include "qcom-ipq4019.dtsi"
>  
>  / {
> @@ -72,7 +73,7 @@ spi@...5000 {
>  			pinctrl-0 = <&spi_0_pins>;
>  			pinctrl-names = "default";
>  			status = "okay";
> -			cs-gpios = <&tlmm 54 0>;
> +			cs-gpios = <&tlmm 54 GPIO_ACTIVE_HIGH>;
>  
>  			mx25l25635e@0 {
>  				#address-cells = <1>;
> diff --git a/arch/arm/boot/dts/qcom-ipq4019-ap.dk04.1.dtsi b/arch/arm/boot/dts/qcom-ipq4019-ap.dk04.1.dtsi
> index 44a9597d8bfd..c2f5222e72de 100644
> --- a/arch/arm/boot/dts/qcom-ipq4019-ap.dk04.1.dtsi
> +++ b/arch/arm/boot/dts/qcom-ipq4019-ap.dk04.1.dtsi
> @@ -87,7 +87,7 @@ spi@...5000 { /* BLSP1 QUP1 */
>  			pinctrl-0 = <&spi_0_pins>;
>  			pinctrl-names = "default";
>  			status = "okay";
> -			cs-gpios = <&tlmm 12 0>;
> +			cs-gpios = <&tlmm 12 GPIO_ACTIVE_HIGH>;
>  
>  			flash@0 {
>  				#address-cells = <1>;
> @@ -100,7 +100,7 @@ flash@0 {
>  
>  		pci@...00000 {
>  			status = "okay";
> -			perst-gpio = <&tlmm 38 0x1>;
> +			perst-gpio = <&tlmm 38 GPIO_ACTIVE_LOW>;
>  		};
>  
>  		qpic-nand@...0000 {
> diff --git a/arch/arm/boot/dts/qcom-ipq4019-ap.dk07.1-c1.dts b/arch/arm/boot/dts/qcom-ipq4019-ap.dk07.1-c1.dts
> index c7a6e77da272..7fc33149060c 100644
> --- a/arch/arm/boot/dts/qcom-ipq4019-ap.dk07.1-c1.dts
> +++ b/arch/arm/boot/dts/qcom-ipq4019-ap.dk07.1-c1.dts
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0
>  // Copyright (c) 2018, The Linux Foundation. All rights reserved.
>  
> +#include <dt-bindings/gpio/gpio.h>
>  #include "qcom-ipq4019-ap.dk07.1.dtsi"
>  
>  / {
> @@ -10,7 +11,7 @@ / {
>  	soc {
>  		pci@...00000 {
>  			status = "okay";
> -			perst-gpio = <&tlmm 38 0x1>;
> +			perst-gpio = <&tlmm 38 GPIO_ACTIVE_LOW>;
>  		};
>  
>  		spi@...6000 {
> @@ -50,7 +51,7 @@ spi@...5000 {
>  			pinctrl-0 = <&spi_0_pins>;
>  			pinctrl-names = "default";
>  			status = "okay";
> -			cs-gpios = <&tlmm 12 0>;
> +			cs-gpios = <&tlmm 12 GPIO_ACTIVE_HIGH>;
>  
>  			flash@0 {
>  				#address-cells = <1>;
> diff --git a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts
> index ec5d340562b6..6daceaa87802 100644
> --- a/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts
> +++ b/arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dts
> @@ -175,7 +175,7 @@ i2c-gate {
>  			ak8963@f {
>  				compatible = "asahi-kasei,ak8963";
>  				reg = <0x0f>;
> -				gpios = <&tlmm 67 0>;
> +				gpios = <&tlmm 67 GPIO_ACTIVE_HIGH>;
>  				vid-supply = <&pm8941_lvs1>;
>  				vdd-supply = <&pm8941_l17>;
>  			};