diff options
author | Naushir Patuck <naush@raspberrypi.com> | 2023-05-03 13:20:27 +0100 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2023-05-04 20:47:40 +0300 |
commit | 726e9274ea95fa46352556d340c5793a8da51fcd (patch) | |
tree | 80f6adcdbf744f9317e09eff3e80c602b384a753 /src/ipa/raspberrypi/cam_helper_imx219.cpp | |
parent | 46aefed208fef4bc8d6f6e8882b92b9af710a60b (diff) |
pipeline: ipa: raspberrypi: Refactor and move the Raspberry Pi code
Split the Raspberry Pi pipeline handler and IPA source code into common
and VC4/BCM2835 specific file structures.
For the pipeline handler, the common code files now live in
src/libcamera/pipeline/rpi/common/
and the VC4-specific files in src/libcamera/pipeline/rpi/vc4/.
For the IPA, the common code files now live in
src/ipa/rpi/{cam_helper,controller}/
and the vc4 specific files in src/ipa/rpi/vc4/. With this change, the
camera tuning files are now installed under share/libcamera/ipa/rpi/vc4/.
To build the pipeline and IPA, the meson configuration options have now
changed from "raspberrypi" to "rpi/vc4":
meson setup build -Dipas=rpi/vc4 -Dpipelines=rpi/vc4
Signed-off-by: Naushir Patuck <naush@raspberrypi.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'src/ipa/raspberrypi/cam_helper_imx219.cpp')
-rw-r--r-- | src/ipa/raspberrypi/cam_helper_imx219.cpp | 115 |
1 files changed, 0 insertions, 115 deletions
diff --git a/src/ipa/raspberrypi/cam_helper_imx219.cpp b/src/ipa/raspberrypi/cam_helper_imx219.cpp deleted file mode 100644 index c3337ed0..00000000 --- a/src/ipa/raspberrypi/cam_helper_imx219.cpp +++ /dev/null @@ -1,115 +0,0 @@ -/* SPDX-License-Identifier: BSD-2-Clause */ -/* - * Copyright (C) 2019, Raspberry Pi Ltd - * - * cam_helper_imx219.cpp - camera helper for imx219 sensor - */ - -#include <assert.h> -#include <stddef.h> -#include <stdio.h> -#include <stdlib.h> - -/* - * We have observed that the imx219 embedded data stream randomly returns junk - * register values. Do not rely on embedded data until this has been resolved. - */ -#define ENABLE_EMBEDDED_DATA 0 - -#include "cam_helper.h" -#if ENABLE_EMBEDDED_DATA -#include "md_parser.h" -#endif - -using namespace RPiController; - -/* - * We care about one gain register and a pair of exposure registers. Their I2C - * addresses from the Sony IMX219 datasheet: - */ -constexpr uint32_t gainReg = 0x157; -constexpr uint32_t expHiReg = 0x15a; -constexpr uint32_t expLoReg = 0x15b; -constexpr uint32_t frameLengthHiReg = 0x160; -constexpr uint32_t frameLengthLoReg = 0x161; -constexpr uint32_t lineLengthHiReg = 0x162; -constexpr uint32_t lineLengthLoReg = 0x163; -constexpr std::initializer_list<uint32_t> registerList [[maybe_unused]] - = { expHiReg, expLoReg, gainReg, frameLengthHiReg, frameLengthLoReg, - lineLengthHiReg, lineLengthLoReg }; - -class CamHelperImx219 : public CamHelper -{ -public: - CamHelperImx219(); - uint32_t gainCode(double gain) const override; - double gain(uint32_t gainCode) const override; - unsigned int mistrustFramesModeSwitch() const override; - bool sensorEmbeddedDataPresent() const override; - -private: - /* - * Smallest difference between the frame length and integration time, - * in units of lines. - */ - static constexpr int frameIntegrationDiff = 4; - - void populateMetadata(const MdParser::RegisterMap ®isters, - Metadata &metadata) const override; -}; - -CamHelperImx219::CamHelperImx219() -#if ENABLE_EMBEDDED_DATA - : CamHelper(std::make_unique<MdParserSmia>(registerList), frameIntegrationDiff) -#else - : CamHelper({}, frameIntegrationDiff) -#endif -{ -} - -uint32_t CamHelperImx219::gainCode(double gain) const -{ - return (uint32_t)(256 - 256 / gain); -} - -double CamHelperImx219::gain(uint32_t gainCode) const -{ - return 256.0 / (256 - gainCode); -} - -unsigned int CamHelperImx219::mistrustFramesModeSwitch() const -{ - /* - * For reasons unknown, we do occasionally get a bogus metadata frame - * at a mode switch (though not at start-up). Possibly warrants some - * investigation, though not a big deal. - */ - return 1; -} - -bool CamHelperImx219::sensorEmbeddedDataPresent() const -{ - return ENABLE_EMBEDDED_DATA; -} - -void CamHelperImx219::populateMetadata(const MdParser::RegisterMap ®isters, - Metadata &metadata) const -{ - DeviceStatus deviceStatus; - - deviceStatus.lineLength = lineLengthPckToDuration(registers.at(lineLengthHiReg) * 256 + - registers.at(lineLengthLoReg)); - deviceStatus.shutterSpeed = exposure(registers.at(expHiReg) * 256 + registers.at(expLoReg), - deviceStatus.lineLength); - deviceStatus.analogueGain = gain(registers.at(gainReg)); - deviceStatus.frameLength = registers.at(frameLengthHiReg) * 256 + registers.at(frameLengthLoReg); - - metadata.set("device.status", deviceStatus); -} - -static CamHelper *create() -{ - return new CamHelperImx219(); -} - -static RegisterCamHelper reg("imx219", &create); |