Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Michael Ellerman <patch-notifications@ellerman.id.au>
To:     Nicholas Piggin <npiggin@gmail.com>,
        Christophe Leroy <christophe.leroy@csgroup.eu>,
        Rob Herring <robh@kernel.org>
Cc:     linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org
In-Reply-To: <20230609183244.1767325-1-robh@kernel.org>
References: <20230609183244.1767325-1-robh@kernel.org>
Subject: Re: [PATCH] powerpc: fsl_rio: Use of_range_to_resource() for "ranges" parsing
Message-Id: <168836201902.50010.6190167965224825737.b4-ty@ellerman.id.au>
Date:   Mon, 03 Jul 2023 15:26:59 +1000
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Fri, 09 Jun 2023 12:32:44 -0600, Rob Herring wrote:
> "ranges" is a standard property with common parsing functions. Users
> shouldn't be implementing their own parsing of it. Refactor the FSL RapidIO
> "ranges" parsing to use of_range_to_resource() instead.
> 
> One change is the original code would look for "#size-cells" and
> "#address-cells" in the parent node if not found in the port child
> nodes. That is non-standard behavior and not necessary AFAICT. In 2011
> in commit 54986964c13c ("powerpc/85xx: Update SRIO device tree nodes")
> there was an ABI break. The upstream .dts files have been correct since
> at least that point.
> 
> [...]

Applied to powerpc/next.

[1/1] powerpc: fsl_rio: Use of_range_to_resource() for "ranges" parsing
      https://git.kernel.org/powerpc/c/c4ae1799a5a358388acb610512c68666f8758364

cheers