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;
MIME-Version: 1.0
References: <20201204075117.10430-1-kishon@ti.com> <20201204075117.10430-3-kishon@ti.com>
In-Reply-To: <20201204075117.10430-3-kishon@ti.com>
From:   Rob Herring <robh+dt@kernel.org>
Date:   Mon, 7 Dec 2020 08:30:39 -0600
Message-ID: <CAL_JsqKQwx2qKJb5eAsutdHH5DevC+XH33yXrCBWE+OCrrQFYg@mail.gmail.com>
Subject: Re: [PATCH v2 2/3] PCI: j721e: Get offset within "syscon" from
 "ti,syscon-pcie-ctrl" phandle arg
To:     Kishon Vijay Abraham I <kishon@ti.com>
Cc:     Bjorn Helgaas <bhelgaas@google.com>, Tero Kristo <t-kristo@ti.com>,
        Nishanth Menon <nm@ti.com>, Tom Joseph <tjoseph@cadence.com>,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
        PCI <linux-pci@vger.kernel.org>, devicetree@vger.kernel.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        linux-omap <linux-omap@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, Dec 4, 2020 at 1:52 AM Kishon Vijay Abraham I <kishon@ti.com> wrote:
>
> Get "syscon" pcie_ctrl offset from the argument of "ti,syscon-pcie-ctrl"
> phandle. Previously a subnode to "syscon" node was added which has the
> exact memory mapped address of pcie_ctrl but now the offset of pcie_ctrl
> within "syscon" is now being passed as argument to "ti,syscon-pcie-ctrl"
> phandle.
>
> If the offset is not provided in "ti,syscon-pcie-ctrl", the
> full memory mapped address of pcie_ctrl is used in order to maintain old
> DT compatibility.
>
> This change is as discussed in [1]
>
> [1] -> http://lore.kernel.org/r/CAL_JsqKiUcO76bo1GoepWM1TusJWoty_BRy2hFSgtEVMqtrvvQ@mail.gmail.com
>
> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
> ---
>  drivers/pci/controller/cadence/pci-j721e.c | 28 +++++++++++++++-------
>  1 file changed, 19 insertions(+), 9 deletions(-)

Reviewed-by: Rob Herring <robh@kernel.org>