RE: [PATCH v2] drm/amd/display: add GTT domain support for dcn 3.1.5 and 3.1.6

2022-02-22 Thread Zhang, Yifan
[AMD Official Use Only]

Hi Alex,

I'm OK w/ this series. Acked.

BRs,
Yifan

-Original Message-
From: Alex Deucher  
Sent: Tuesday, February 22, 2022 10:17 PM
To: Wentland, Harry 
Cc: Zhang, Yifan ; amd-gfx list 
; Li, Sun peng (Leo) ; 
Deucher, Alexander ; Liang, Prike 
; Yin, Tianci (Rico) ; Koenig, 
Christian 
Subject: Re: [PATCH v2] drm/amd/display: add GTT domain support for dcn 3.1.5 
and 3.1.6

How about this series instead?
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwork.freedesktop.org%2Fseries%2F100519%2F&data=04%7C01%7Cyifan1.zhang%40amd.com%7C23aba82394f446c4c41b08d9f60e0687%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637811362370697462%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=oMn8bBcpNupXv3Kf%2FkMMbHLBtuQ45b46rEq4cAZYs2g%3D&reserved=0

Alex

On Tue, Feb 22, 2022 at 8:53 AM Harry Wentland  wrote:
>
> On 2022-02-22 01:03, Yifan Zhang wrote:
> > From: "Tianci.Yin" 
> >
> > this patch adds GTT domain support for dcn 3.1.5 and 3.1.6
> >
> > Signed-off-by: Tianci.Yin 
> > Signed-off-by: Yifan Zhang 
>
> Acked-by: Harry Wentland 
>
> Harry
>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c 
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > index 9709368b4915..37f4da219a5f 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > @@ -523,6 +523,8 @@ uint32_t amdgpu_display_supported_domains(struct 
> > amdgpu_device *adev,
> >   case IP_VERSION(3, 0, 1):
> >   case IP_VERSION(3, 1, 2):
> >   case IP_VERSION(3, 1, 3):
> > + case IP_VERSION(3, 1, 5):
> > + case IP_VERSION(3, 1, 6):
> >   domain |= AMDGPU_GEM_DOMAIN_GTT;
> >   break;
> >   default:
>


Re: [PATCH v2] drm/amd/display: add GTT domain support for dcn 3.1.5 and 3.1.6

2022-02-22 Thread Alex Deucher
How about this series instead?
https://patchwork.freedesktop.org/series/100519/

Alex

On Tue, Feb 22, 2022 at 8:53 AM Harry Wentland  wrote:
>
> On 2022-02-22 01:03, Yifan Zhang wrote:
> > From: "Tianci.Yin" 
> >
> > this patch adds GTT domain support for dcn 3.1.5 and 3.1.6
> >
> > Signed-off-by: Tianci.Yin 
> > Signed-off-by: Yifan Zhang 
>
> Acked-by: Harry Wentland 
>
> Harry
>
> > ---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c 
> > b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > index 9709368b4915..37f4da219a5f 100644
> > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> > @@ -523,6 +523,8 @@ uint32_t amdgpu_display_supported_domains(struct 
> > amdgpu_device *adev,
> >   case IP_VERSION(3, 0, 1):
> >   case IP_VERSION(3, 1, 2):
> >   case IP_VERSION(3, 1, 3):
> > + case IP_VERSION(3, 1, 5):
> > + case IP_VERSION(3, 1, 6):
> >   domain |= AMDGPU_GEM_DOMAIN_GTT;
> >   break;
> >   default:
>


Re: [PATCH v2] drm/amd/display: add GTT domain support for dcn 3.1.5 and 3.1.6

2022-02-22 Thread Harry Wentland
On 2022-02-22 01:03, Yifan Zhang wrote:
> From: "Tianci.Yin" 
> 
> this patch adds GTT domain support for dcn 3.1.5 and 3.1.6
> 
> Signed-off-by: Tianci.Yin 
> Signed-off-by: Yifan Zhang 

Acked-by: Harry Wentland 

Harry

> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c 
> b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> index 9709368b4915..37f4da219a5f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> @@ -523,6 +523,8 @@ uint32_t amdgpu_display_supported_domains(struct 
> amdgpu_device *adev,
>   case IP_VERSION(3, 0, 1):
>   case IP_VERSION(3, 1, 2):
>   case IP_VERSION(3, 1, 3):
> + case IP_VERSION(3, 1, 5):
> + case IP_VERSION(3, 1, 6):
>   domain |= AMDGPU_GEM_DOMAIN_GTT;
>   break;
>   default: