<div dir="auto">Reviewed-by: Marek Olšák <<a href="mailto:marek.olsak@amd.com">marek.olsak@amd.com</a>></div><div class="gmail_extra"><br><div class="gmail_quote">On Jun 22, 2017 3:52 PM, "Jan Vesely" <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">c6e1dc5247 switched to use gbm_bo_get_offset and gbm_bo_get_stride_for_plane.<br>
These symbols were added in mesa 17.1. Since there is no alternative to using them, bump the gbm requirement to 17.1.<br>
<br>
Fixes: c6e1dc5247 "piglit_drm_dma_buf: fix GPU offsets and strides"<br>
Signed-off-by: Jan Vesely <<a href="mailto:jan.vesely@rutgers.edu">jan.vesely@rutgers.edu</a>><br>
---<br>
CMakeLists.txt | 10 ++++------<br>
1 file changed, 4 insertions(+), 6 deletions(-)<br>
<br>
diff --git a/CMakeLists.txt b/CMakeLists.txt<br>
index b4dfe4f..90d056b 100644<br>
--- a/CMakeLists.txt<br>
+++ b/CMakeLists.txt<br>
@@ -137,16 +137,14 @@ IF(${CMAKE_SYSTEM_NAME} MATCHES "Linux")<br>
add_definitions(-DPIGLIT_HAS_<wbr>GLX)<br>
endif()<br>
<br>
- pkg_check_modules(GBM QUIET gbm)<br>
+ pkg_check_modules(GBM QUIET gbm>=17.1)<br>
if(GBM_FOUND)<br>
set(PIGLIT_HAS_GBM True)<br>
add_definitions(-DPIGLIT_HAS_<wbr>GBM)<br>
set(CMAKE_REQUIRED_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES} ${GBM_LIBRARIES})<br>
- if (GBM_VERSION VERSION_EQUAL "12.1" OR GBM_VERSION VERSION_GREATER "12.1")<br>
- CHECK_FUNCTION_EXISTS(gbm_bo_<wbr>map PIGLIT_HAS_GBM_BO_MAP)<br>
- if (PIGLIT_HAS_GBM_BO_MAP)<br>
- add_definitions(-DPIGLIT_HAS_<wbr>GBM_BO_MAP)<br>
- endif()<br>
+ CHECK_FUNCTION_EXISTS(gbm_bo_<wbr>map PIGLIT_HAS_GBM_BO_MAP)<br>
+ if (PIGLIT_HAS_GBM_BO_MAP)<br>
+ add_definitions(-DPIGLIT_HAS_<wbr>GBM_BO_MAP)<br>
endif()<br>
endif(GBM_FOUND)<br>
<br>
--<br>
2.9.4<br>
<br>
</blockquote></div></div>