[PATCH 0/1] AMDGPU SI support almost fixed

[michel@xxxxxxxxxxx (Michel Dänzer)]

On 19/08/16 06:24 PM, Marek Olšák wrote:
> I've added this. The dmesg message is gone, but the cursor is still missing.
> 
> diff --git a/src/amdgpu_kms.c b/src/amdgpu_kms.c
> index 9ee48df..d9f15a4 100644
> --- a/src/amdgpu_kms.c
> +++ b/src/amdgpu_kms.c
> @@ -929,8 +929,14 @@ Bool AMDGPUPreInit_KMS(ScrnInfoPtr pScrn, int flags)
>         else
>                 pAMDGPUEnt->HasCRTC2 = TRUE;
> 
> -       info->cursor_w = CURSOR_WIDTH_CIK;
> -       info->cursor_h = CURSOR_HEIGHT_CIK;
> +       if (info->ChipFamily >= CHIP_FAMILY_TAHITI &&
> +           info->ChipFamily <= CHIP_FAMILY_HAINAN) {
> +               info->cursor_w = CURSOR_WIDTH;
> +               info->cursor_h = CURSOR_HEIGHT;
> +       } else {
> +               info->cursor_w = CURSOR_WIDTH_CIK;
> +               info->cursor_h = CURSOR_HEIGHT_CIK;
> +       }
> 
>         amdgpu_query_heap_size(pAMDGPUEnt->pDev, AMDGPU_GEM_DOMAIN_GTT,
>                                 &heap_size, &max_allocation);

Looks good; does the HW cursor work with the modesetting driver?


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer