[libcamera-devel,v3,0/8] Raspberry Pi: Enable imx477 long exposure modes
mbox series

Message ID 20210702150940.226941-1-naush@raspberrypi.com
Headers show
Series
  • Raspberry Pi: Enable imx477 long exposure modes
Related show

Message

Naushir Patuck July 2, 2021, 3:09 p.m. UTC 
Hi,

There has been some substantial rework on this patch series in version 3:

- Added patches 1/8 - 3/8 to C++ify our DeviceStatus structure.
- Added patch 4/8 to store the frame length in DeviceStatus.
- Updated patch 5/8 to use frame length instead of exposure time to determine
if we are in long exposure mode.  The previous way of using exposure time would
be incorrect in cases where we have a short exposure time, and a long frame
length. Also change the CamHelperImx477::Prepare() method to not touch the metadata
buffers so many times.
- Patches 6/8 - 8/8 remain unchanged.

Regards,
Naush

Naushir Patuck (8):
  ipa: raspberrypi: Make device_status.h C++ only header, and update
    comments
  ipa: raspberrypi: Add a constructor struct DeviceStatus
  ipa: raspberrypi: Add an operator<< to struct DeviceStatus
  ipa: raspberrypi: Add frame_length to DeviceStatus
  ipa: raspberrypi: Allow long exposure modes for imx477.
  pipeline: raspberrypi: Use priority write for vblank when writing
    sensor ctrls
  ipa: raspberrypi: Increase the default max frame duration to 250s
  ipa: raspberrypi: Make variable names consistent

 src/ipa/raspberrypi/cam_helper.cpp            |   5 +-
 src/ipa/raspberrypi/cam_helper_imx219.cpp     |   6 +-
 src/ipa/raspberrypi/cam_helper_imx477.cpp     | 101 +++++++++++++++++-
 .../raspberrypi/controller/device_status.h    |  48 ++++++---
 src/ipa/raspberrypi/controller/rpi/agc.cpp    |   2 +-
 src/ipa/raspberrypi/controller/rpi/lux.cpp    |   8 +-
 src/ipa/raspberrypi/raspberrypi.cpp           |  17 ++-
 .../pipeline/raspberrypi/raspberrypi.cpp      |  37 +++++--
 8 files changed, 176 insertions(+), 48 deletions(-)