Re: ?==?utf-8?q? [RFC i-g-t 8/9] lib: Replace intel specific header includes with intel_drm_stubs.h.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Friday, May 20, 2016 23:59 BST, robert.foss@xxxxxxxxxxxxx wrote: 
> From: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> 
> Replace intel specific header includes with intel_drm_stubs.h.
> 
> The stubbed functions will all call igt_require(false) and cause a skip.
> 
> Signed-off-by: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> ---
>  lib/drmtest.c           | 2 +-
>  lib/gpgpu_fill.c        | 7 +++----
>  lib/igt_aux.c           | 2 +-
>  lib/igt_aux.h           | 3 ++-
>  lib/igt_debugfs.c       | 4 ++--
>  lib/igt_draw.h          | 3 +--
>  lib/igt_fb.h            | 3 ++-
>  lib/igt_kms.c           | 3 +--
>  lib/intel_batchbuffer.c | 4 ----
>  lib/intel_batchbuffer.h | 3 +--
>  lib/intel_chipset.c     | 2 +-
>  lib/ioctl_wrappers.c    | 1 -
>  lib/ioctl_wrappers.h    | 4 ++--
>  lib/media_fill_gen7.c   | 3 +--
>  lib/media_fill_gen8.c   | 4 +---
>  lib/media_fill_gen8lp.c | 6 ++----
>  lib/media_fill_gen9.c   | 4 +---
>  lib/media_spin.c        | 2 --
>  lib/rendercopy_gen6.c   | 5 ++---
>  lib/rendercopy_gen7.c   | 4 +---
>  lib/rendercopy_gen8.c   | 4 +---
>  lib/rendercopy_gen9.c   | 5 +----
>  lib/rendercopy_i830.c   | 5 +----
>  lib/rendercopy_i915.c   | 9 +++------
>  24 files changed, 31 insertions(+), 61 deletions(-)
> 
This and next patch should be a simple sed job. Untangling the header inclusion(s) is a worthy goal imho, although it's not something that should be mixed in here.

The usual disclaimer... I've got little-to-no say when it comes to IGT so take all the above with a grain (truckload) of salt. That said, I'm fairly confident I haven't gone off the rocker this time and the reasoning is sound ;-)

Thanks
Emil

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux