Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH 8/8] ARM: dts: socfgpa: Use generic "ngpios" rather than
 "snps,nr-gpios"
To:     Jisheng Zhang <Jisheng.Zhang@synaptics.com>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Khuong Dinh <khuong@os.amperecomputing.com>,
        Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>,
        Wei Xu <xuwei5@hisilicon.com>
Cc:     linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Saravana Kannan <saravanak@google.com>,
        Linus Walleij <linus.walleij@linaro.org>
References: <20201109170258.4a70c768@xhacker.debian>
 <20201109170712.6e8c1a90@xhacker.debian>
From:   Dinh Nguyen <dinguyen@kernel.org>
Message-ID: <f2cba2f2-338a-793f-a5bf-a95acc28e6f4@kernel.org>
Date:   Mon, 4 Jan 2021 15:19:29 -0600
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
MIME-Version: 1.0
In-Reply-To: <20201109170712.6e8c1a90@xhacker.debian>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 11/9/20 3:07 AM, Jisheng Zhang wrote:
> This is to remove similar errors as below:
> 
> OF: /.../gpio-port@0: could not find phandle
> 
> Commit 7569486d79ae ("gpio: dwapb: Add ngpios DT-property support")
> explained the reason of above errors well and added the generic
> "ngpios" property, let's use it.
> 
> Signed-off-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
> ---
>   arch/arm/boot/dts/socfpga.dtsi         | 6 +++---
>   arch/arm/boot/dts/socfpga_arria10.dtsi | 6 +++---
>   2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
> index 0b021eef0b53..32b7ad814ec6 100644
> --- a/arch/arm/boot/dts/socfpga.dtsi
> +++ b/arch/arm/boot/dts/socfpga.dtsi
> @@ -612,7 +612,7 @@ porta: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <29>;
> +				ngpios = <29>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> @@ -633,7 +633,7 @@ portb: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <29>;
> +				ngpios = <29>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> @@ -654,7 +654,7 @@ portc: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <27>;
> +				ngpios = <27>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> diff --git a/arch/arm/boot/dts/socfpga_arria10.dtsi b/arch/arm/boot/dts/socfpga_arria10.dtsi
> index 0013ec3463c4..34e23e0582fb 100644
> --- a/arch/arm/boot/dts/socfpga_arria10.dtsi
> +++ b/arch/arm/boot/dts/socfpga_arria10.dtsi
> @@ -491,7 +491,7 @@ porta: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <29>;
> +				ngpios = <29>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> @@ -511,7 +511,7 @@ portb: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <29>;
> +				ngpios = <29>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> @@ -531,7 +531,7 @@ portc: gpio-controller@0 {
>   				compatible = "snps,dw-apb-gpio-port";
>   				gpio-controller;
>   				#gpio-cells = <2>;
> -				snps,nr-gpios = <27>;
> +				ngpios = <27>;
>   				reg = <0>;
>   				interrupt-controller;
>   				#interrupt-cells = <2>;
> 

Applied to socfpga dts tree.

Thanks,
Dinh