<div dir="ltr">If the required definitions are in UAPI headers, maybe you can just import the most recent header into weston source for the time being. IIRC, libevdev (used by libinput) does something similar in order to keep the build from breaking on older kernels. Maybe Peter can explain better.<div><br></div><div>(Sent from web interface, so no signature available)</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 8, 2017 at 3:05 AM, Derek Foreman <span dir="ltr"><<a href="mailto:derekf@osg.samsung.com" target="_blank">derekf@osg.samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 07/02/17 07:17 PM, Bryce Harrington wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Tue, Feb 07, 2017 at 11:48:47AM +0200, Pekka Paalanen wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, 6 Feb 2017 12:57:41 -0500<br>
Micah Fedke <<a href="mailto:micah.fedke@collabora.co.uk" target="_blank">micah.fedke@collabora.co.uk</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The v4l2 API can be queried to detect if the input video image is<br>
horizontally or vertically flipped. If the image is y-flipped, we can<br>
set the ZWP_LINUX_BUFFER_PARAMS_V1_FLA<wbr>GS_Y_INVERT flag to notify the<br>
compositor. If the image is h-flipped, we can only print a warning<br>
since linux_buffer_params_v1 does not support horizontal flipping.<br>
<br>
Signed-off-by: Micah Fedke <<a href="mailto:micah.fedke@collabora.co.uk" target="_blank">micah.fedke@collabora.co.uk</a>><br>
---<br>
This version addresses Pekka's comments by renaming check_v4l2_control to<br>
check_v4l2_control_bool, removing the len parameter, separating out the logic<br>
tests in check_v4l2_control_bool and fixing spacing issues.<br>
<br>
clients/simple-dmabuf-v4l.c | 62 ++++++++++++++++++++++++++++++<wbr>+++++++++------<br>
1 file changed, 54 insertions(+), 8 deletions(-)<br>
</blockquote>
<br>
Hi,<br>
<br>
looks fine, pushed with my R-b:<br>
9a200d7..0fee977 master -> master<br>
</blockquote>
<br>
Breaks the build on older linux kernels.<br>
<br>
CC clients/weston_simple_dmabuf_<wbr>v4l-simple-dmabuf-v4l.o<br>
clients/simple-dmabuf-v4l.c:35<wbr>8:25: warning: ‘struct v4l2_query_ext_ctrl’ declared inside parameter list [enabled by default]<br>
const int expected_value)<br>
^<br>
clients/simple-dmabuf-v4l.c:35<wbr>8:25: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default]<br>
clients/simple-dmabuf-v4l.c: In function ‘check_v4l2_control_bool’:<br>
clients/simple-dmabuf-v4l.c:36<wbr>3:18: error: dereferencing pointer to incomplete type<br>
<a href="http://ctrl.id" rel="noreferrer" target="_blank">ctrl.id</a> = qectrl->id;<br>
^<br>
et al.<br>
<br>
Builds ok with the patch reverted, or with --disable-simple-dmabuf-v4l-cl<wbr>ient set.<br>
</blockquote>
<br></div></div>
Ouch, bad timing for a build break.<br>
<br>
IMHO this patch should land before the final release, to get all the y-inversion fix-ups in at the same time, but I don't like the idea of a beta release that fails to build after it succeeds at configure for some folk...<br>
<br>
If a quick fix isn't coming, maybe revert it before the beta release?<br>
<br>
It's not a huge stretch to consider properly dealing with y-flip a bug fix, and it is just a test app, so I think it'd be ok to land it during the beta...<div class="HOEnZb"><div class="h5"><br>
______________________________<wbr>_________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.freedeskto<wbr>p.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/wayland-devel" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/wayland-devel</a><br>
</div></div></blockquote></div><br></div></div>