https://bugzilla.rpmfusion.org/show_bug.cgi?id=6469

--- Comment #11 from Hans de Goede <h...@hansg.org> ---
Nicolas,

Thank you for your help with reviewing this.

Yes, I will take care of the userspace packages. It might be helpful for me to
explain what the entire stack looks like:

1. A set of out of tree open-source drivers (this review request)
2. ipu6-camera-bins-firmware, firmware for 1. (review bug 6474)
3. ivsc-firmware, more firmware for 1. (review bug 6491)
4. ipu6-camera-bins set of binary only x86_64 libraries talking to 1. (review
bug 6474)
5. ipu6-camera-hal opensource (apache-2.0) library layer over ipu6-camera-bins
(review bug 6508)
6. gstreamer1-plugins-icamerasrc gstreamer plugin building on top of 5. (review
bug 6534)

This is the minimum stack after this the camera can be tested with gst-launch.

But to actually get it to work we need another program on top called
v4l2-relayd which uses the v4l2-loopback kernel module. This will listen for
userspace opening the loopback /dev/video0 device and then it will start a
gstreamer pipeline for this
(review bug 6529).

We want users to be able to just do a "sudo dnf install akmod-intel-ipu6" and
then get a fully working stack.

Nicolas, so I guess this means that we need to add the following Requires to
ipu6-camera-bins since that provides intel-ipu6-kmod-common ?   :

Requires: ipu6-camera-bins-firmware
Requires: ivsc-firmware
Requires: gstreamer1-plugins-icamerasrc
Requires: v4l2-relayd

(note gstreamer1-plugins-icamerasrc will bring in ipu6-camera-hal).

Having these there feels a bit weird which is why I suggested a stub
intel-ipu6-kmod-common package to Kate, but if the usual way for kmods is to
just put these inside ipu6-camera-bins then that is fine with me too.

> So this package should have:
> Requires:        intel-ipu6-kmod >= %{?epoch}:%{version}

I agree on the "Requires: intel-ipu6-kmod" but I think it should be unversioned
there is no tight version coupling here and there actually is nothing
resembling proper versioning from upstream at all. So I think we want just:

Requires: intel-ipu6-kmod

added on top of the Requires I mentioned above.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
_______________________________________________
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Reply via email to