* [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM @ 2019-12-27 1:27 Bjorn Andersson 2019-12-27 8:51 ` Stanimir Varbanov 2020-02-26 10:22 ` Lorenzo Pieralisi 0 siblings, 2 replies; 10+ messages in thread From: Bjorn Andersson @ 2019-12-27 1:27 UTC (permalink / raw) To: Stanimir Varbanov, Lorenzo Pieralisi, Andrew Murray, Bjorn Helgaas Cc: linux-arm-msm, linux-pci, linux-kernel, Marc Gonzalez, stable There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit the fixup to only affect the relevant PCIe bridges. Cc: stable@vger.kernel.org Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> --- Stan, I picked up all the suggested device id's from the previous thread and added 0x1000 for QCS404. I looked at creating platform specific defines in pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would prefer that I do this anyway. drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c index 5ea527a6bd9f..138e1a2d21cc 100644 --- a/drivers/pci/controller/dwc/pcie-qcom.c +++ b/drivers/pci/controller/dwc/pcie-qcom.c @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) { dev->class = PCI_CLASS_BRIDGE_PCI << 8; } -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); static struct platform_driver qcom_pcie_driver = { .probe = qcom_pcie_probe, -- 2.24.0 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-27 1:27 [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM Bjorn Andersson @ 2019-12-27 8:51 ` Stanimir Varbanov 2019-12-28 15:41 ` Marc Gonzalez 2020-02-26 10:22 ` Lorenzo Pieralisi 1 sibling, 1 reply; 10+ messages in thread From: Stanimir Varbanov @ 2019-12-27 8:51 UTC (permalink / raw) To: Bjorn Andersson, Lorenzo Pieralisi, Andrew Murray, Bjorn Helgaas Cc: linux-arm-msm, linux-pci, linux-kernel, Marc Gonzalez, stable Hi Bjorn, On 12/27/19 3:27 AM, Bjorn Andersson wrote: > There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit > the fixup to only affect the relevant PCIe bridges. > > Cc: stable@vger.kernel.org > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > > Stan, I picked up all the suggested device id's from the previous thread and > added 0x1000 for QCS404. I looked at creating platform specific defines in > pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would > prefer that I do this anyway. Looks good, Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> > > drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > index 5ea527a6bd9f..138e1a2d21cc 100644 > --- a/drivers/pci/controller/dwc/pcie-qcom.c > +++ b/drivers/pci/controller/dwc/pcie-qcom.c > @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) > { > dev->class = PCI_CLASS_BRIDGE_PCI << 8; > } > -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); > > static struct platform_driver qcom_pcie_driver = { > .probe = qcom_pcie_probe, > -- regards, Stan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-27 8:51 ` Stanimir Varbanov @ 2019-12-28 15:41 ` Marc Gonzalez 2019-12-29 2:45 ` Bjorn Andersson 0 siblings, 1 reply; 10+ messages in thread From: Marc Gonzalez @ 2019-12-28 15:41 UTC (permalink / raw) To: Stanimir Varbanov, Bjorn Andersson, Lorenzo Pieralisi, Andrew Murray, Bjorn Helgaas Cc: linux-arm-msm, linux-pci, linux-kernel, stable On 27/12/2019 09:51, Stanimir Varbanov wrote: > On 12/27/19 3:27 AM, Bjorn Andersson wrote: > >> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit >> the fixup to only affect the relevant PCIe bridges. >> >> Cc: stable@vger.kernel.org >> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> >> --- >> >> Stan, I picked up all the suggested device id's from the previous thread and >> added 0x1000 for QCS404. I looked at creating platform specific defines in >> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would >> prefer that I do this anyway. > > Looks good, > > Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> > >> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c >> index 5ea527a6bd9f..138e1a2d21cc 100644 >> --- a/drivers/pci/controller/dwc/pcie-qcom.c >> +++ b/drivers/pci/controller/dwc/pcie-qcom.c >> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) >> { >> dev->class = PCI_CLASS_BRIDGE_PCI << 8; >> } >> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); Hrmmm... still not CCed on the patch, and still don't think the fixup is required(?) for 0x106 and 0x107. Regards. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-28 15:41 ` Marc Gonzalez @ 2019-12-29 2:45 ` Bjorn Andersson 2019-12-30 20:25 ` Marc Gonzalez 0 siblings, 1 reply; 10+ messages in thread From: Bjorn Andersson @ 2019-12-29 2:45 UTC (permalink / raw) To: Marc Gonzalez Cc: Stanimir Varbanov, Lorenzo Pieralisi, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, stable On Sat 28 Dec 07:41 PST 2019, Marc Gonzalez wrote: > On 27/12/2019 09:51, Stanimir Varbanov wrote: > > > On 12/27/19 3:27 AM, Bjorn Andersson wrote: > > > >> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit > >> the fixup to only affect the relevant PCIe bridges. > >> > >> Cc: stable@vger.kernel.org > >> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > >> --- > >> > >> Stan, I picked up all the suggested device id's from the previous thread and > >> added 0x1000 for QCS404. I looked at creating platform specific defines in > >> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would > >> prefer that I do this anyway. > > > > Looks good, > > > > Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> > > > >> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- > >> 1 file changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > >> index 5ea527a6bd9f..138e1a2d21cc 100644 > >> --- a/drivers/pci/controller/dwc/pcie-qcom.c > >> +++ b/drivers/pci/controller/dwc/pcie-qcom.c > >> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) > >> { > >> dev->class = PCI_CLASS_BRIDGE_PCI << 8; > >> } > >> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); > > Hrmmm... still not CCed on the patch, You are Cc'ed on the patch, but as usual your mail server responds "451 too many errors from your ip" and throw my emails away. > and still don't think the fixup is required(?) for 0x106 and 0x107. > I re-read your reply in my v1 thread. So we know that 0x104 doesn't need the fixup, so resumably only 0x101 needs the fixup? Regards, Bjorn ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-29 2:45 ` Bjorn Andersson @ 2019-12-30 20:25 ` Marc Gonzalez 2020-02-21 14:35 ` Lorenzo Pieralisi 0 siblings, 1 reply; 10+ messages in thread From: Marc Gonzalez @ 2019-12-30 20:25 UTC (permalink / raw) To: Bjorn Andersson Cc: Stanimir Varbanov, Lorenzo Pieralisi, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, stable On 29/12/2019 03:45, Bjorn Andersson wrote: > On Sat 28 Dec 07:41 PST 2019, Marc Gonzalez wrote: > >> On 27/12/2019 09:51, Stanimir Varbanov wrote: >> >>> On 12/27/19 3:27 AM, Bjorn Andersson wrote: >>> >>>> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit >>>> the fixup to only affect the relevant PCIe bridges. >>>> >>>> Cc: stable@vger.kernel.org >>>> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> >>>> --- >>>> >>>> Stan, I picked up all the suggested device id's from the previous thread and >>>> added 0x1000 for QCS404. I looked at creating platform specific defines in >>>> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would >>>> prefer that I do this anyway. >>> >>> Looks good, >>> >>> Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> >>> >>>> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- >>>> 1 file changed, 7 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c >>>> index 5ea527a6bd9f..138e1a2d21cc 100644 >>>> --- a/drivers/pci/controller/dwc/pcie-qcom.c >>>> +++ b/drivers/pci/controller/dwc/pcie-qcom.c >>>> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) >>>> { >>>> dev->class = PCI_CLASS_BRIDGE_PCI << 8; >>>> } >>>> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); >> >> Hrmmm... still not CCed on the patch, > > You are Cc'ed on the patch, but as usual your mail server responds "451 > too many errors from your ip" and throw my emails away. > >> and still don't think the fixup is required(?) for 0x106 and 0x107. >> > > I re-read your reply in my v1 thread. So we know that 0x104 doesn't need > the fixup, so presumably only 0x101 needs the fixup? I apologize for the tone of my reply. I did not mean to sound so snarky. All I can say is that, if I remember correctly, the fixup was not necessary on apq8098 (0x0105) and it was probably not required on msm8996 and sdm845. For older platforms, all bets are off. Regards. ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-30 20:25 ` Marc Gonzalez @ 2020-02-21 14:35 ` Lorenzo Pieralisi 2020-02-26 9:27 ` Stanimir Varbanov 0 siblings, 1 reply; 10+ messages in thread From: Lorenzo Pieralisi @ 2020-02-21 14:35 UTC (permalink / raw) To: Marc Gonzalez Cc: Bjorn Andersson, Stanimir Varbanov, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, stable On Mon, Dec 30, 2019 at 09:25:28PM +0100, Marc Gonzalez wrote: > On 29/12/2019 03:45, Bjorn Andersson wrote: > > > On Sat 28 Dec 07:41 PST 2019, Marc Gonzalez wrote: > > > >> On 27/12/2019 09:51, Stanimir Varbanov wrote: > >> > >>> On 12/27/19 3:27 AM, Bjorn Andersson wrote: > >>> > >>>> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit > >>>> the fixup to only affect the relevant PCIe bridges. > >>>> > >>>> Cc: stable@vger.kernel.org > >>>> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > >>>> --- > >>>> > >>>> Stan, I picked up all the suggested device id's from the previous thread and > >>>> added 0x1000 for QCS404. I looked at creating platform specific defines in > >>>> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would > >>>> prefer that I do this anyway. > >>> > >>> Looks good, > >>> > >>> Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> > >>> > >>>> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- > >>>> 1 file changed, 7 insertions(+), 1 deletion(-) > >>>> > >>>> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > >>>> index 5ea527a6bd9f..138e1a2d21cc 100644 > >>>> --- a/drivers/pci/controller/dwc/pcie-qcom.c > >>>> +++ b/drivers/pci/controller/dwc/pcie-qcom.c > >>>> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) > >>>> { > >>>> dev->class = PCI_CLASS_BRIDGE_PCI << 8; > >>>> } > >>>> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); > >>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); > >> > >> Hrmmm... still not CCed on the patch, > > > > You are Cc'ed on the patch, but as usual your mail server responds "451 > > too many errors from your ip" and throw my emails away. > > > >> and still don't think the fixup is required(?) for 0x106 and 0x107. > >> > > > > I re-read your reply in my v1 thread. So we know that 0x104 doesn't need > > the fixup, so presumably only 0x101 needs the fixup? > > I apologize for the tone of my reply. I did not mean to sound > so snarky. > > All I can say is that, if I remember correctly, the fixup was > not necessary on apq8098 (0x0105) and it was probably not > required on msm8996 and sdm845. For older platforms, all bets > are off. How are we proceeding with this patch then ? Thanks, Lorenzo ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2020-02-21 14:35 ` Lorenzo Pieralisi @ 2020-02-26 9:27 ` Stanimir Varbanov 0 siblings, 0 replies; 10+ messages in thread From: Stanimir Varbanov @ 2020-02-26 9:27 UTC (permalink / raw) To: Lorenzo Pieralisi, Marc Gonzalez Cc: Bjorn Andersson, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, stable Hi Lorenzo, On 2/21/20 4:35 PM, Lorenzo Pieralisi wrote: > On Mon, Dec 30, 2019 at 09:25:28PM +0100, Marc Gonzalez wrote: >> On 29/12/2019 03:45, Bjorn Andersson wrote: >> >>> On Sat 28 Dec 07:41 PST 2019, Marc Gonzalez wrote: >>> >>>> On 27/12/2019 09:51, Stanimir Varbanov wrote: >>>> >>>>> On 12/27/19 3:27 AM, Bjorn Andersson wrote: >>>>> >>>>>> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit >>>>>> the fixup to only affect the relevant PCIe bridges. >>>>>> >>>>>> Cc: stable@vger.kernel.org >>>>>> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> >>>>>> --- >>>>>> >>>>>> Stan, I picked up all the suggested device id's from the previous thread and >>>>>> added 0x1000 for QCS404. I looked at creating platform specific defines in >>>>>> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would >>>>>> prefer that I do this anyway. >>>>> >>>>> Looks good, >>>>> >>>>> Acked-by: Stanimir Varbanov <svarbanov@mm-sol.com> >>>>> >>>>>> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- >>>>>> 1 file changed, 7 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c >>>>>> index 5ea527a6bd9f..138e1a2d21cc 100644 >>>>>> --- a/drivers/pci/controller/dwc/pcie-qcom.c >>>>>> +++ b/drivers/pci/controller/dwc/pcie-qcom.c >>>>>> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) >>>>>> { >>>>>> dev->class = PCI_CLASS_BRIDGE_PCI << 8; >>>>>> } >>>>>> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); >>>>>> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); >>>> >>>> Hrmmm... still not CCed on the patch, >>> >>> You are Cc'ed on the patch, but as usual your mail server responds "451 >>> too many errors from your ip" and throw my emails away. >>> >>>> and still don't think the fixup is required(?) for 0x106 and 0x107. >>>> >>> >>> I re-read your reply in my v1 thread. So we know that 0x104 doesn't need >>> the fixup, so presumably only 0x101 needs the fixup? >> >> I apologize for the tone of my reply. I did not mean to sound >> so snarky. >> >> All I can say is that, if I remember correctly, the fixup was >> not necessary on apq8098 (0x0105) and it was probably not >> required on msm8996 and sdm845. For older platforms, all bets >> are off. > > How are we proceeding with this patch then ? It took too much time, please take it as-is in v2 with my Ack. We can drop the not-affected SoCs with follow-up patches once we are sure that we do not break the supported SoCs. -- regards, Stan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2019-12-27 1:27 [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM Bjorn Andersson 2019-12-27 8:51 ` Stanimir Varbanov @ 2020-02-26 10:22 ` Lorenzo Pieralisi 2020-02-26 10:56 ` Stanimir Varbanov 1 sibling, 1 reply; 10+ messages in thread From: Lorenzo Pieralisi @ 2020-02-26 10:22 UTC (permalink / raw) To: Bjorn Andersson Cc: Stanimir Varbanov, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, Marc Gonzalez, stable On Thu, Dec 26, 2019 at 05:27:17PM -0800, Bjorn Andersson wrote: > There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit > the fixup to only affect the relevant PCIe bridges. > > Cc: stable@vger.kernel.org Hi Bjorn, to simplify stable's merging, would you mind helping me with the stable releases you want this patch to apply to please ? I will apply it then. Thanks, Lorenzo > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > --- > > Stan, I picked up all the suggested device id's from the previous thread and > added 0x1000 for QCS404. I looked at creating platform specific defines in > pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would > prefer that I do this anyway. > > drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > index 5ea527a6bd9f..138e1a2d21cc 100644 > --- a/drivers/pci/controller/dwc/pcie-qcom.c > +++ b/drivers/pci/controller/dwc/pcie-qcom.c > @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) > { > dev->class = PCI_CLASS_BRIDGE_PCI << 8; > } > -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); > +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); > > static struct platform_driver qcom_pcie_driver = { > .probe = qcom_pcie_probe, > -- > 2.24.0 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2020-02-26 10:22 ` Lorenzo Pieralisi @ 2020-02-26 10:56 ` Stanimir Varbanov 2020-02-26 11:05 ` Lorenzo Pieralisi 0 siblings, 1 reply; 10+ messages in thread From: Stanimir Varbanov @ 2020-02-26 10:56 UTC (permalink / raw) To: Lorenzo Pieralisi, Bjorn Andersson Cc: Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, Marc Gonzalez, stable Hi Lorenzo, On 2/26/20 12:22 PM, Lorenzo Pieralisi wrote: > On Thu, Dec 26, 2019 at 05:27:17PM -0800, Bjorn Andersson wrote: >> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit >> the fixup to only affect the relevant PCIe bridges. >> >> Cc: stable@vger.kernel.org > > Hi Bjorn, > > to simplify stable's merging, would you mind helping me with > the stable releases you want this patch to apply to please ? > We've to have this in the patch: Cc: stable@vger.kernel.org # v5.2+ Fixes: 322f03436692 ("PCI: qcom: Use default config space read function") > I will apply it then. > > Thanks, > Lorenzo > >> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> >> --- >> >> Stan, I picked up all the suggested device id's from the previous thread and >> added 0x1000 for QCS404. I looked at creating platform specific defines in >> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would >> prefer that I do this anyway. >> >> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c >> index 5ea527a6bd9f..138e1a2d21cc 100644 >> --- a/drivers/pci/controller/dwc/pcie-qcom.c >> +++ b/drivers/pci/controller/dwc/pcie-qcom.c >> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) >> { >> dev->class = PCI_CLASS_BRIDGE_PCI << 8; >> } >> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); >> >> static struct platform_driver qcom_pcie_driver = { >> .probe = qcom_pcie_probe, >> -- >> 2.24.0 >> -- regards, Stan ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM 2020-02-26 10:56 ` Stanimir Varbanov @ 2020-02-26 11:05 ` Lorenzo Pieralisi 0 siblings, 0 replies; 10+ messages in thread From: Lorenzo Pieralisi @ 2020-02-26 11:05 UTC (permalink / raw) To: Stanimir Varbanov Cc: Bjorn Andersson, Andrew Murray, Bjorn Helgaas, linux-arm-msm, linux-pci, linux-kernel, Marc Gonzalez, stable On Wed, Feb 26, 2020 at 12:56:23PM +0200, Stanimir Varbanov wrote: > Hi Lorenzo, > > On 2/26/20 12:22 PM, Lorenzo Pieralisi wrote: > > On Thu, Dec 26, 2019 at 05:27:17PM -0800, Bjorn Andersson wrote: > >> There exists non-bridge PCIe devices with PCI_VENDOR_ID_QCOM, so limit > >> the fixup to only affect the relevant PCIe bridges. > >> > >> Cc: stable@vger.kernel.org > > > > Hi Bjorn, > > > > to simplify stable's merging, would you mind helping me with > > the stable releases you want this patch to apply to please ? > > > > We've to have this in the patch: > > Cc: stable@vger.kernel.org # v5.2+ > Fixes: 322f03436692 ("PCI: qcom: Use default config space read function") Done, applied to pci/qcom for v5.7. Thanks, Lorenzo > > I will apply it then. > > > > Thanks, > > Lorenzo > > > >> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org> > >> --- > >> > >> Stan, I picked up all the suggested device id's from the previous thread and > >> added 0x1000 for QCS404. I looked at creating platform specific defines in > >> pci_ids.h, but SDM845 has both 106 and 107... Please let me know if you would > >> prefer that I do this anyway. > >> > >> drivers/pci/controller/dwc/pcie-qcom.c | 8 +++++++- > >> 1 file changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/drivers/pci/controller/dwc/pcie-qcom.c b/drivers/pci/controller/dwc/pcie-qcom.c > >> index 5ea527a6bd9f..138e1a2d21cc 100644 > >> --- a/drivers/pci/controller/dwc/pcie-qcom.c > >> +++ b/drivers/pci/controller/dwc/pcie-qcom.c > >> @@ -1439,7 +1439,13 @@ static void qcom_fixup_class(struct pci_dev *dev) > >> { > >> dev->class = PCI_CLASS_BRIDGE_PCI << 8; > >> } > >> -DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, PCI_ANY_ID, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0101, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0104, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0106, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0107, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x0302, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1000, qcom_fixup_class); > >> +DECLARE_PCI_FIXUP_EARLY(PCI_VENDOR_ID_QCOM, 0x1001, qcom_fixup_class); > >> > >> static struct platform_driver qcom_pcie_driver = { > >> .probe = qcom_pcie_probe, > >> -- > >> 2.24.0 > >> > > -- > regards, > Stan ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-02-26 11:05 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-12-27 1:27 [PATCH v2] PCI: qcom: Fix the fixup of PCI_VENDOR_ID_QCOM Bjorn Andersson 2019-12-27 8:51 ` Stanimir Varbanov 2019-12-28 15:41 ` Marc Gonzalez 2019-12-29 2:45 ` Bjorn Andersson 2019-12-30 20:25 ` Marc Gonzalez 2020-02-21 14:35 ` Lorenzo Pieralisi 2020-02-26 9:27 ` Stanimir Varbanov 2020-02-26 10:22 ` Lorenzo Pieralisi 2020-02-26 10:56 ` Stanimir Varbanov 2020-02-26 11:05 ` Lorenzo Pieralisi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).