[PATCH 0/1] AMDGPU SI support almost fixed

Marek Olšák maraeo at gmail.com
Mon Aug 22 13:22:27 UTC 2016


On Mon, Aug 22, 2016 at 3:18 AM, Dave Airlie <airlied at gmail.com> wrote:
> On 19 August 2016 at 19:01, Marek Olšák <maraeo at gmail.com> wrote:
>> Hi,
>>
>> This amdgpu patch fixes DCE support. Now only the cursor is broken:
>>
>> [   37.936022] [drm:dce_v6_0_crtc_cursor_set2 [amdgpu]] *ERROR* bad cursor width or height 128 x 128
>>
>> Everything else seems to work. I've not tested UVD and VCE.
>>
>> Now I'm gonna send out all userspace patches that I intend to merge before the kernel support lands. I think that shouldn't be a problem.
>
> Please don't land anything in userspace until kernel side is in
> drm-next. At least if it's kernel API related.
>
> Been burned too many times with people shipping libdrm before
> something got set in stone in the kernel.

There is just a new definition in amdgpu_drm.h: AMDGPU_FAMILY_SI

However, userspace doesn't need it, which means there is practically
no kernel API change from the userspace point of view.

I can delay pushing the AMDGPU_FAMILY_SI definition, but I don't think
anything else needs to be delayed.

Marek


More information about the amd-gfx mailing list