* Re: [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist
2020-04-06 19:42 [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist Alex Deucher
@ 2020-04-07 2:55 ` Huang Rui
2020-04-23 19:48 ` Alex Deucher
2020-04-23 19:55 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Huang Rui @ 2020-04-07 2:55 UTC (permalink / raw)
To: Alex Deucher; +Cc: amd-gfx, linux-pci, Alex Deucher, Christian König
On Mon, Apr 06, 2020 at 03:42:01PM -0400, Alex Deucher wrote:
> According to the hw architect, pre-ZEN parts support
> p2p writes and ZEN parts support both p2p reads and writes.
>
> Add entries for Zen parts Raven (0x15d0) and Renoir (0x1630).
>
> Cc: Christian König <christian.koenig@amd.com>
> Acked-by: Christian König <christian.koenig@amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Huang Rui <ray.huang@amd.com>
> ---
> drivers/pci/p2pdma.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
> index 9a8a38384121..91a4c987399d 100644
> --- a/drivers/pci/p2pdma.c
> +++ b/drivers/pci/p2pdma.c
> @@ -282,6 +282,8 @@ static const struct pci_p2pdma_whitelist_entry {
> } pci_p2pdma_whitelist[] = {
> /* AMD ZEN */
> {PCI_VENDOR_ID_AMD, 0x1450, 0},
> + {PCI_VENDOR_ID_AMD, 0x15d0, 0},
> + {PCI_VENDOR_ID_AMD, 0x1630, 0},
>
> /* Intel Xeon E5/Core i7 */
> {PCI_VENDOR_ID_INTEL, 0x3c00, REQ_SAME_HOST_BRIDGE},
> --
> 2.25.1
>
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=02%7C01%7Cray.huang%40amd.com%7C9ed9b4e22b2744af197e08d7da629a76%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637217989343230053&sdata=peIQSu9dCwpMRzKyCkU%2BgGFLzDlwcvpmdDyGKzeSFQ4%3D&reserved=0
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist
2020-04-06 19:42 [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist Alex Deucher
2020-04-07 2:55 ` Huang Rui
@ 2020-04-23 19:48 ` Alex Deucher
2020-04-23 19:55 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Alex Deucher @ 2020-04-23 19:48 UTC (permalink / raw)
To: amd-gfx list, Linux PCI, Bjorn Helgaas; +Cc: Alex Deucher, Christian König
+ Bjorn
Can chance I can get this picked up for -next?
Thanks,
Alex
On Mon, Apr 6, 2020 at 3:42 PM Alex Deucher <alexdeucher@gmail.com> wrote:
>
> According to the hw architect, pre-ZEN parts support
> p2p writes and ZEN parts support both p2p reads and writes.
>
> Add entries for Zen parts Raven (0x15d0) and Renoir (0x1630).
>
> Cc: Christian König <christian.koenig@amd.com>
> Acked-by: Christian König <christian.koenig@amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> ---
> drivers/pci/p2pdma.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
> index 9a8a38384121..91a4c987399d 100644
> --- a/drivers/pci/p2pdma.c
> +++ b/drivers/pci/p2pdma.c
> @@ -282,6 +282,8 @@ static const struct pci_p2pdma_whitelist_entry {
> } pci_p2pdma_whitelist[] = {
> /* AMD ZEN */
> {PCI_VENDOR_ID_AMD, 0x1450, 0},
> + {PCI_VENDOR_ID_AMD, 0x15d0, 0},
> + {PCI_VENDOR_ID_AMD, 0x1630, 0},
>
> /* Intel Xeon E5/Core i7 */
> {PCI_VENDOR_ID_INTEL, 0x3c00, REQ_SAME_HOST_BRIDGE},
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist
2020-04-06 19:42 [PATCH] PCI/P2PDMA: Add additional AMD ZEN root ports to the whitelist Alex Deucher
2020-04-07 2:55 ` Huang Rui
2020-04-23 19:48 ` Alex Deucher
@ 2020-04-23 19:55 ` Bjorn Helgaas
2 siblings, 0 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2020-04-23 19:55 UTC (permalink / raw)
To: Alex Deucher; +Cc: amd-gfx, linux-pci, Alex Deucher, Christian König
On Mon, Apr 06, 2020 at 03:42:01PM -0400, Alex Deucher wrote:
> According to the hw architect, pre-ZEN parts support
> p2p writes and ZEN parts support both p2p reads and writes.
>
> Add entries for Zen parts Raven (0x15d0) and Renoir (0x1630).
>
> Cc: Christian König <christian.koenig@amd.com>
> Acked-by: Christian König <christian.koenig@amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Applied with Huang's ack to pci/p2pdma for v5.8, thanks!
> ---
> drivers/pci/p2pdma.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/p2pdma.c b/drivers/pci/p2pdma.c
> index 9a8a38384121..91a4c987399d 100644
> --- a/drivers/pci/p2pdma.c
> +++ b/drivers/pci/p2pdma.c
> @@ -282,6 +282,8 @@ static const struct pci_p2pdma_whitelist_entry {
> } pci_p2pdma_whitelist[] = {
> /* AMD ZEN */
> {PCI_VENDOR_ID_AMD, 0x1450, 0},
> + {PCI_VENDOR_ID_AMD, 0x15d0, 0},
> + {PCI_VENDOR_ID_AMD, 0x1630, 0},
>
> /* Intel Xeon E5/Core i7 */
> {PCI_VENDOR_ID_INTEL, 0x3c00, REQ_SAME_HOST_BRIDGE},
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread