public inbox for linux-pci@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur
@ 2026-01-09  8:23 Krishna Chaitanya Chundru
  2026-01-09 15:58 ` Bjorn Helgaas
  2026-01-21 14:23 ` Manivannan Sadhasivam
  0 siblings, 2 replies; 3+ messages in thread
From: Krishna Chaitanya Chundru @ 2026-01-09  8:23 UTC (permalink / raw)
  To: Bjorn Helgaas; +Cc: linux-pci, linux-kernel, mani, Krishna Chaitanya Chundru

The Qualcomm Hamoa & Glymur root ports don't advertise an ACS capability,
but they do provide ACS-like features to disable peer transactions and
validate bus numbers in requests.

So add an ACS quirk for Hamoa & Glymur.

Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
---
 drivers/pci/quirks.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index b9c252aa6fe08a864cebe245f5dd7bf41fcc5116..75dee46f474a643cc79d112df1f5a57a9f6b95b1 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -5107,6 +5107,10 @@ static const struct pci_dev_acs_enabled {
 	{ PCI_VENDOR_ID_QCOM, 0x0401, pci_quirk_qcom_rp_acs },
 	/* QCOM SA8775P root port */
 	{ PCI_VENDOR_ID_QCOM, 0x0115, pci_quirk_qcom_rp_acs },
+	/* QCOM Hamoa root port */
+	{ PCI_VENDOR_ID_QCOM, 0x0111, pci_quirk_qcom_rp_acs },
+	/* QCOM Glymur root port */
+	{ PCI_VENDOR_ID_QCOM, 0x0120, pci_quirk_qcom_rp_acs },
 	/* HXT SD4800 root ports. The ACS design is same as QCOM QDF2xxx */
 	{ PCI_VENDOR_ID_HXT, 0x0401, pci_quirk_qcom_rp_acs },
 	/* Intel PCH root ports */

---
base-commit: 623fb9912f6af600cda3b6bd166ac738c1115ef4
change-id: 20260109-acs_quirk-0f8e83dc945e

Best regards,
-- 
Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur
  2026-01-09  8:23 [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur Krishna Chaitanya Chundru
@ 2026-01-09 15:58 ` Bjorn Helgaas
  2026-01-21 14:23 ` Manivannan Sadhasivam
  1 sibling, 0 replies; 3+ messages in thread
From: Bjorn Helgaas @ 2026-01-09 15:58 UTC (permalink / raw)
  To: Krishna Chaitanya Chundru; +Cc: Bjorn Helgaas, linux-pci, linux-kernel, mani

On Fri, Jan 09, 2026 at 01:53:32PM +0530, Krishna Chaitanya Chundru wrote:
> The Qualcomm Hamoa & Glymur root ports don't advertise an ACS capability,
> but they do provide ACS-like features to disable peer transactions and
> validate bus numbers in requests.
> 
> So add an ACS quirk for Hamoa & Glymur.
> 
> Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>

Applied to pci/virtualization for v6.20, thanks!

> ---
>  drivers/pci/quirks.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index b9c252aa6fe08a864cebe245f5dd7bf41fcc5116..75dee46f474a643cc79d112df1f5a57a9f6b95b1 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -5107,6 +5107,10 @@ static const struct pci_dev_acs_enabled {
>  	{ PCI_VENDOR_ID_QCOM, 0x0401, pci_quirk_qcom_rp_acs },
>  	/* QCOM SA8775P root port */
>  	{ PCI_VENDOR_ID_QCOM, 0x0115, pci_quirk_qcom_rp_acs },
> +	/* QCOM Hamoa root port */
> +	{ PCI_VENDOR_ID_QCOM, 0x0111, pci_quirk_qcom_rp_acs },
> +	/* QCOM Glymur root port */
> +	{ PCI_VENDOR_ID_QCOM, 0x0120, pci_quirk_qcom_rp_acs },
>  	/* HXT SD4800 root ports. The ACS design is same as QCOM QDF2xxx */
>  	{ PCI_VENDOR_ID_HXT, 0x0401, pci_quirk_qcom_rp_acs },
>  	/* Intel PCH root ports */
> 
> ---
> base-commit: 623fb9912f6af600cda3b6bd166ac738c1115ef4
> change-id: 20260109-acs_quirk-0f8e83dc945e
> 
> Best regards,
> -- 
> Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur
  2026-01-09  8:23 [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur Krishna Chaitanya Chundru
  2026-01-09 15:58 ` Bjorn Helgaas
@ 2026-01-21 14:23 ` Manivannan Sadhasivam
  1 sibling, 0 replies; 3+ messages in thread
From: Manivannan Sadhasivam @ 2026-01-21 14:23 UTC (permalink / raw)
  To: Krishna Chaitanya Chundru; +Cc: Bjorn Helgaas, linux-pci, linux-kernel

On Fri, Jan 09, 2026 at 01:53:32PM +0530, Krishna Chaitanya Chundru wrote:
> The Qualcomm Hamoa & Glymur root ports don't advertise an ACS capability,
> but they do provide ACS-like features to disable peer transactions and
> validate bus numbers in requests.
> 
> So add an ACS quirk for Hamoa & Glymur.
> 
> Signed-off-by: Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>

Reviewed-by: Manivannan Sadhasivam <mani@kernel.org>

- Mani

> ---
>  drivers/pci/quirks.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index b9c252aa6fe08a864cebe245f5dd7bf41fcc5116..75dee46f474a643cc79d112df1f5a57a9f6b95b1 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -5107,6 +5107,10 @@ static const struct pci_dev_acs_enabled {
>  	{ PCI_VENDOR_ID_QCOM, 0x0401, pci_quirk_qcom_rp_acs },
>  	/* QCOM SA8775P root port */
>  	{ PCI_VENDOR_ID_QCOM, 0x0115, pci_quirk_qcom_rp_acs },
> +	/* QCOM Hamoa root port */
> +	{ PCI_VENDOR_ID_QCOM, 0x0111, pci_quirk_qcom_rp_acs },
> +	/* QCOM Glymur root port */
> +	{ PCI_VENDOR_ID_QCOM, 0x0120, pci_quirk_qcom_rp_acs },
>  	/* HXT SD4800 root ports. The ACS design is same as QCOM QDF2xxx */
>  	{ PCI_VENDOR_ID_HXT, 0x0401, pci_quirk_qcom_rp_acs },
>  	/* Intel PCH root ports */
> 
> ---
> base-commit: 623fb9912f6af600cda3b6bd166ac738c1115ef4
> change-id: 20260109-acs_quirk-0f8e83dc945e
> 
> Best regards,
> -- 
> Krishna Chaitanya Chundru <krishna.chundru@oss.qualcomm.com>
> 

-- 
மணிவண்ணன் சதாசிவம்

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-01-21 14:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-09  8:23 [PATCH] PCI: Add ACS quirk for Qualcomm Hamoa & Glymur Krishna Chaitanya Chundru
2026-01-09 15:58 ` Bjorn Helgaas
2026-01-21 14:23 ` Manivannan Sadhasivam

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox