List of configuration items to pass to the IPA about libcamera HOT 8 CLOSED

A relevant commit for this is ec37b2a

from libcamera.

A quick question on the here listed requirements

Do you foresee a usage on the IPA side of the full list of camera modes? In the current implementation it is required in order to find the currently used mode, but apart from that I don't see any usage in any algorithm. Is it fair to assume it is enough to provide to your IPA the currently used mode only ?

from libcamera.

gentle ping @naushir

from libcamera.

Sorry @jmondi, somehow I did not get any notification on this issue update.

Do you foresee a usage on the IPA side of the full list of camera modes? In the current implementation it is required in order to find the currently used mode, but apart from that I don't see any usage in any algorithm. Is it fair to assume it is enough to provide to your IPA the currently used mode only ?

No, I think the IPA only ever needs to know the currently used mode. Hopefully that makes the implementation simpler as well.

from libcamera.

Maybe I need to mention you directly to have the notification delivered?

from libcamera.

@naushir thanks for the info, I will update the list of requirements to drop the full list of modes and take this into account in the proposed implementation.

from libcamera.

Probably, but I wonder if there is a setting that always sends notifications without being mentioned.

from libcamera.

Also, the latest CamHelper class code we are using (to interpret modes, etc.) can be found here:
https://github.com/raspberrypi/libcamera/blob/master/src/ipa/raspberrypi/cam_helper.hpp

from libcamera.