Re: [PATCH libdrm 0/6] drm-next header syncing

[Emil Velikov]

On 9 November 2017 at 08:28, Daniel Vetter  wrote:
> On Wed, Nov 08, 2017 at 11:34:41AM -0800, Eric Anholt wrote:
>> I wanted to update the vc4 header so that I could land some igt
>> testcases (though I'd much rather just be importing the header into
>> igt and ditching libdrm).  In the process, I cleaned up some other
>> header deltas to the kernel so that there's less diff for the next
>> person to look at.
>
> I'm mildly tempted to just pull a mesa in igt too and put the headers in
> directly. To avoid the need for way too many libdrm releases when hacking
> on tests we atm add all the uapi stuff with a local_ prefix, but that's
> kinda silly.
>
> Maybe start a thread with vc4 as example converted (I guess we need to
> make sure the include order is right) to get this started.

That shouldn't be an obstacle to merging Eric's patches, right?

I went through the series and it looks good. For the lot
Reviewed-by: Emil Velikov 

-Emil
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH libdrm 0/6] drm-next header syncing

[Daniel Vetter]

On Wed, Nov 08, 2017 at 11:34:41AM -0800, Eric Anholt wrote:
> I wanted to update the vc4 header so that I could land some igt
> testcases (though I'd much rather just be importing the header into
> igt and ditching libdrm).  In the process, I cleaned up some other
> header deltas to the kernel so that there's less diff for the next
> person to look at.

I'm mildly tempted to just pull a mesa in igt too and put the headers in
directly. To avoid the need for way too many libdrm releases when hacking
on tests we atm add all the uapi stuff with a local_ prefix, but that's
kinda silly.

Maybe start a thread with vc4 as example converted (I guess we need to
make sure the include order is right) to get this started.
-Daniel

> 
> I skipped the actual ABI changes for the other drivers, though.
> 
> Eric Anholt (6):
>   headers: Sync vc4 header from drm-next.
>   headers: Sync up some header guard changes from drm-next.
>   headers: Sync up some comment spelling and whitespace fixes from
> drm-next.
>   headers: Sync up mga_drm.h from drm-next.
>   headers: Sync up kernel changes to use kernel types instead of
> stdint.h.
>   headers: Drop outdated node about a delta in drm_mode.h.
> 
>  include/drm/README|  22 +---
>  include/drm/mga_drm.h |  12 -
>  include/drm/nouveau_drm.h |  94 +++---
>  include/drm/qxl_drm.h |  82 +++--
>  include/drm/r128_drm.h|  10 
>  include/drm/radeon_drm.h  | 128 
> +++---
>  include/drm/savage_drm.h  |  20 ++--
>  include/drm/sis_drm.h |  10 
>  include/drm/tegra_drm.h   |  14 +++--
>  include/drm/vc4_drm.h |  47 +++--
>  include/drm/via_drm.h |   8 +++
>  include/drm/vmwgfx_drm.h  |   9 
>  12 files changed, 280 insertions(+), 176 deletions(-)
> 
> -- 
> 2.15.0
> 
> ___
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH libdrm 0/6] drm-next header syncing

[Eric Anholt]

I wanted to update the vc4 header so that I could land some igt
testcases (though I'd much rather just be importing the header into
igt and ditching libdrm).  In the process, I cleaned up some other
header deltas to the kernel so that there's less diff for the next
person to look at.

I skipped the actual ABI changes for the other drivers, though.

Eric Anholt (6):
  headers: Sync vc4 header from drm-next.
  headers: Sync up some header guard changes from drm-next.
  headers: Sync up some comment spelling and whitespace fixes from
drm-next.
  headers: Sync up mga_drm.h from drm-next.
  headers: Sync up kernel changes to use kernel types instead of
stdint.h.
  headers: Drop outdated node about a delta in drm_mode.h.

 include/drm/README|  22 +---
 include/drm/mga_drm.h |  12 -
 include/drm/nouveau_drm.h |  94 +++---
 include/drm/qxl_drm.h |  82 +++--
 include/drm/r128_drm.h|  10 
 include/drm/radeon_drm.h  | 128 +++---
 include/drm/savage_drm.h  |  20 ++--
 include/drm/sis_drm.h |  10 
 include/drm/tegra_drm.h   |  14 +++--
 include/drm/vc4_drm.h |  47 +++--
 include/drm/via_drm.h |   8 +++
 include/drm/vmwgfx_drm.h  |   9 
 12 files changed, 280 insertions(+), 176 deletions(-)

-- 
2.15.0

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel