Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New versions of upstream dependencies break the driver. #79

Closed
danielksato opened this issue Apr 13, 2024 · 8 comments
Closed

New versions of upstream dependencies break the driver. #79

danielksato opened this issue Apr 13, 2024 · 8 comments

Comments

@danielksato
Copy link
Contributor

I just recently tried to update to some of the latest upstream releases that purport to include support for kernels 6.7 and 6.8. All I succeeded in doing was breaking everything!

I was able to get my webcam working again by following the steps in #77 to use icamerasrc-git-fix.old. I added the prepare section to its PKGBUILD, as well as intel-ipu6ep-hal-git.

In addition, I had to pin the version of intel-ipu6ep-hal-git to commit da2e2821244f21b95bcb37a1271bf73360c4669e, and the version of intel-ipu6-dkms-git to commit cfb7af1e50a9f04e06d6bb77ea2340eb78350859.

This got me up and running again too. As a bonus, Cheese works for me now, where it didn't before.

@elhossary
Copy link

Kindly, Can you make a pull request for it?

@danielksato
Copy link
Contributor Author

I'm not sure if codifying this workaround is the way to go, since there's been a lot of activity upstream and I'm hoping working kernel 6.7+ support is right around the corner. But that's a question for the maintainers, I suppose; I'll try to make a PR tonight. I'll need to figure out how to use a custom PKGBUILD for intel-ipu6-dkms-git since right now we're using the one from the AUR.

@elhossary
Copy link

In principle, this repo itself a wokaround for IPU6 cameras, as these cams are not working out of the box. It is crazy how fragile the drivers and the setup are on each update.
I suggested that to have at least something known to work, may be the maintainers make another branch out of your pull request.
I will personally clone the PR to have my cam working.
Thanks in advance BTW

danielksato added a commit to danielksato/archlinux-ipu6-webcam that referenced this issue Apr 14, 2024
As described in stefanpartheym#77 and stefanpartheym#79, FORTIFY_SOURCE is needed for Cmake to work
on icamerasrc and intel-ipu36-camera-hal. Additionally, recent upstream
releases have broken several dependencies. While I'm hoping the recent
flurry of activity upstream means support for kernel 6.7+ is on the way,
these workarounds should keep us running for now.
@danielksato
Copy link
Contributor Author

@elhossary I submitted a PR if you wanna take a peek. I copied the edited PKGBUILD from my AUR helper and edited it to reflect the conflict it creates between my intel-ipu6-dkms-git package and the AUR version. However, I don't really want test it locally since it might mean breaking a working install :D

@elhossary
Copy link

@danielksato Thank you for the PR. I will test that

@derkrasseleo
Copy link

I also ran into this build issue

<command-line>: error: "_FORTIFY_SOURCE" redefined [-Werror]
<command-line>: note: this is the location of the previous definition
cc1: all warnings being treated as errors
make[2]: *** [CMakeFiles/camhal_static.dir/build.make:76: CMakeFiles/camhal_static.dir/modules/ia_css/ipu6ep/src/ia_css_terminal.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:356: CMakeFiles/camhal_static.dir/all] Error 2
make: *** [Makefile:156: all] Error 2
==> ERROR: A failure occurred in build().
    Aborting...
ERROR: Failed to build/install: intel-ipu6ep-camera-hal-git

@marmistrz
Copy link

I also needed to add PKG_CONFIG_PATH=/usr/lib/ipu_adl/pkgconfig/, otherwise ia_imaging was not found by CMake.

@wille
Copy link

wille commented May 7, 2024

Running danielksato/fixes-for-recent-releases-77-79 on 6.6.29-1-lts on my Thinkpad X1 Carbon gen 11

Same issues as before, you need to restart v4l2-relayd.service a few times until it works, and sometimes if you boot and the camera is not responding, you need a couple reboots for it to wake up

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

No branches or pull requests

5 participants