Skip to content

Commit

Permalink
xlat: workaround V4L2_CID_USER_IMX_BASE Linux kernel ABI breakage
Browse files Browse the repository at this point in the history
Linux kernel commit v4.18-rc2-106-g421860b9d47053badce4b247576fa48df9ab4c48
has changed the value of V4L2_CID_USER_IMX_BASE constant introduced
by commit v4.13-rc1~141^2~121 because the old value was already used
by V4L2_CID_USER_MAX217X_BASE.

This is of course an ABI breakage that affects Linux kernels starting
with 4.13 and up to 4.18, as well as their LTS derivatives.

Since the imx driver didn't provide any public control ID definitions,
it looks like the best way to handle this situation is to pretend that
the old value of V4L2_CID_USER_IMX_BASE didn't exist.

* xlat/v4l2_control_id_bases.in (V4L2_CID_USER_IMX_BASE): Redefine.

Co-Authored-by: Eugene Syromyatnikov <evgsyr@gmail.com>
  • Loading branch information
ldv-alt and esyr committed Aug 19, 2018
1 parent a264eb3 commit b89a69d
Showing 1 changed file with 16 additions and 1 deletion.
17 changes: 16 additions & 1 deletion xlat/v4l2_control_id_bases.in
Expand Up @@ -2,7 +2,22 @@ V4L2_CID_BASE (V4L2_CTRL_CLASS_USER | 0x900)
V4L2_CID_USER_MEYE_BASE (V4L2_CID_BASE + 0x1000)
V4L2_CID_USER_BTTV_BASE (V4L2_CID_BASE + 0x1010)
V4L2_CID_USER_TI_VPE_BASE (V4L2_CID_BASE + 0x1050)
V4L2_CID_USER_IMX_BASE (V4L2_CID_BASE + 0x1090)
#ifndef STRACE_WORKAROUND_FOR_V4L2_CID_USER_IMX_BASE
# define STRACE_WORKAROUND_FOR_V4L2_CID_USER_IMX_BASE
/*
* Linux kernel commit v4.18-rc2-106-g421860b9d47053badce4b247576fa48df9ab4c48
* has changed the value of V4L2_CID_USER_IMX_BASE constant introduced
* by commit v4.13-rc1~141^2~121 because the old value was already used
* by V4L2_CID_USER_MAX217X_BASE.
* This is of course an ABI breakage that affects Linux kernels starting
* with 4.13 and up to 4.18, as well as their LTS derivatives.
* Since the imx driver didn't provide any public control ID definitions,
* it looks like the best way to handle this situation is to pretend that
* the old value of V4L2_CID_USER_IMX_BASE didn't exist.
*/
# undef V4L2_CID_USER_IMX_BASE
#endif
V4L2_CID_USER_IMX_BASE (V4L2_CID_BASE + 0x10b0)
V4L2_CID_MPEG_BASE (V4L2_CTRL_CLASS_MPEG | 0x900)
V4L2_CID_MPEG_CX2341X_BASE (V4L2_CTRL_CLASS_MPEG | 0x1000)
V4L2_CID_MPEG_MFC51_BASE (V4L2_CTRL_CLASS_MPEG | 0x1100)
Expand Down

0 comments on commit b89a69d

Please sign in to comment.