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;
Date:   Thu, 20 Jan 2022 11:30:52 -0600
From:   Rob Herring <robh@kernel.org>
To:     Michael Walle <michael@walle.cc>
Cc:     linux-kernel@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
        Frank Rowand <frowand.list@gmail.com>,
        devicetree@vger.kernel.org
Subject: Re: [PATCH v3 1/3] of: base: make small of_parse_phandle() variants
 static inline
Message-ID: <YemcTJ640SlBkT0V@robh.at.kernel.org>
References: <20220118173504.2867523-1-michael@walle.cc>
 <20220118173504.2867523-2-michael@walle.cc>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220118173504.2867523-2-michael@walle.cc>
Precedence: bulk

On Tue, 18 Jan 2022 18:35:02 +0100, Michael Walle wrote:
> Make all the smaller variants of the of_parse_phandle() static inline.
> This also let us remove the empty function stubs if CONFIG_OF is not
> defined.
> 
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Michael Walle <michael@walle.cc>
> ---
> changes since v3:
>  - rebased due to now signed index parameter
> 
> changes since v2:
>  - new patch
> 
>  drivers/of/base.c  | 128 ++----------------------------------
>  include/linux/of.h | 157 +++++++++++++++++++++++++++++++++++++--------
>  2 files changed, 135 insertions(+), 150 deletions(-)
> 

I moved the index < 0 checks into __of_parse_phandle_with_args 
and applied, thanks!