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;
MIME-Version: 1.0
References: <20221116214841.1116735-7-robimarko@gmail.com> <20221117192837.GA1203269@bhelgaas>
 <Y8FdxtCitnBd1c6k@lpieralisi>
In-Reply-To: <Y8FdxtCitnBd1c6k@lpieralisi>
From:   Robert Marko <robimarko@gmail.com>
Date:   Fri, 13 Jan 2023 17:41:31 +0100
Message-ID: <CAOX2RU5HznDP9KcjyO=6MjhrY-cU8dhYiskCiTzLD6jm-pNeRw@mail.gmail.com>
Subject: Re: [PATCH 7/9] PCI: qcom: add support for IPQ8074 Gen3 port
To:     Lorenzo Pieralisi <lpieralisi@kernel.org>
Cc:     Bjorn Helgaas <helgaas@kernel.org>, agross@kernel.org,
        andersson@kernel.org, konrad.dybcio@linaro.org,
        bhelgaas@google.com, robh+dt@kernel.org,
        krzysztof.kozlowski+dt@linaro.org, mani@kernel.org, kw@linux.com,
        svarbanov@mm-sol.com, shawn.guo@linaro.org,
        linux-arm-msm@vger.kernel.org, linux-pci@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Fri, 13 Jan 2023 at 14:34, Lorenzo Pieralisi <lpieralisi@kernel.org> wrote:
>
> On Thu, Nov 17, 2022 at 01:28:37PM -0600, Bjorn Helgaas wrote:
> > Hi Robert,
> >
> > If you post a v2 for any reason, capitalize the subject line to match
> > previous history:
> >
> >   PCI: qcom: Rename host-init error label
> >   PCI: qcom: Drop unused post_deinit callback
> >   PCI: qcom: Sort device-id table
> >   PCI: qcom: Clean up IP configurations
> >   ...
>
> If he could do that while rebasing on top of my current pci/qcom branch
> that would be ideal, I tried to cherry-pick patches 5,6,7 but 5 does not
> apply.

Sure, I will send v2 with that now as after cherry-picking the SM8350 commits
from pci/qcom they all still apply, will fixup the subject capitalization.

Regards,
Robert
>
> Thanks,
> Lorenzo
>
> >
> > On Wed, Nov 16, 2022 at 10:48:39PM +0100, Robert Marko wrote:
> > > IPQ8074 has one Gen2 and one Gen3 port, with Gen2 port already supported.
> > > Add compatible for Gen3 port which uses the same controller as IPQ6018.
> > >
> > > Signed-off-by: Robert Marko <robimarko@gmail.com>
> > > ---
> > >  drivers/pci/controller/dwc/pcie-qcom.c | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c
> > > index 77e5dc7b88ad..97e8510c53fb 100644
> > > --- a/drivers/pci/controller/dwc/pcie-qcom.c
> > > +++ b/drivers/pci/controller/dwc/pcie-qcom.c
> > > @@ -1817,6 +1817,7 @@ static const struct of_device_id qcom_pcie_match[] = {
> > >     { .compatible = "qcom,pcie-ipq8064", .data = &cfg_2_1_0 },
> > >     { .compatible = "qcom,pcie-ipq8064-v2", .data = &cfg_2_1_0 },
> > >     { .compatible = "qcom,pcie-ipq8074", .data = &cfg_2_3_3 },
> > > +   { .compatible = "qcom,pcie-ipq8074-gen3", .data = &cfg_2_9_0 },
> > >     { .compatible = "qcom,pcie-msm8996", .data = &cfg_2_3_2 },
> > >     { .compatible = "qcom,pcie-qcs404", .data = &cfg_2_4_0 },
> > >     { .compatible = "qcom,pcie-sa8540p", .data = &cfg_1_9_0 },
> > > --
> > > 2.38.1
> > >