Skip to content

Conversation

@tewarid
Copy link

@tewarid tewarid commented Apr 21, 2022

Fixes clang error: field within 'v4l2_ext_control' is less aligned
than 'v4l2_ext_control::(anonymous union

Signed-off-by: Devendra Tewari [email protected]

Fixes clang error: field within 'v4l2_ext_control' is less aligned
than 'v4l2_ext_control::(anonymous union

Signed-off-by: Devendra Tewari <[email protected]>
@kbingham
Copy link
Owner

Interesting, this file is actually from the linux kernel, so if it needs to be fixed - it needs to be fixed there, then imported here.
Can you check if this is still an issue in the main linux kernel ? or if it has already been fixed there?

@tewarid
Copy link
Author

tewarid commented Apr 22, 2022

Raspberry Pi kernel has the same issue. My own kernel build via yocto is using gcc whereas libcamera is built using clang version 14.0.1. 🤔

@kbingham
Copy link
Owner

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/uapi/linux/videodev2.h#n1742 is the upstream kernel link - and it seems relevant there.

Would you like to submit a patch to the linux kernel / linux-media mailing list?

@tewarid
Copy link
Author

tewarid commented Apr 22, 2022

Thanks - I'll submit the patch upstream.

@tewarid
Copy link
Author

tewarid commented Oct 3, 2022

Patch has been submitted upstream but doesn't look like any one has looked at it in a while https://patchwork.kernel.org/project/linux-media/patch/[email protected]/.

agherzan pushed a commit to agherzan/meta-raspberrypi that referenced this pull request Oct 10, 2022
Add Raspberry Pi pipeline handler and Image
Processing Algorithms.

Work around clang 14/15 build error:
field within 'v4l2_ext_control' is less aligned.
See
kbingham/libcamera#40.

Signed-off-by: Devendra Tewari <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants