[PATCH 0/1] AMDGPU SI support almost fixed

[Michel Dänzer]

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.


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