[PATCH 0/1] AMDGPU SI support almost fixed

[Marek Olšák]

On Fri, Aug 19, 2016 at 12:04 PM, Michel Dänzer <michel at daenzer.net> wrote:
> On 19/08/16 07:00 PM, Marek Olšák wrote:
>> On Fri, Aug 19, 2016 at 11:45 AM, Michel Dänzer <michel at daenzer.net> wrote:
>>> 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?
>>
>> No, the cursor doesn't work with modesetting either. There are no
>> errors in dmesg.
>
> Then it's probably a kernel driver issue.

OK. I'll leave that to our DCE experts. ;)

Marek