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>

1 files changed, 0 insertions, 109 deletions

diff --git a/src/ipa/raspberrypi/cam_helper_ov5647.cpp b/src/ipa/raspberrypi/cam_helper_ov5647.cpp
deleted file mode 100644
index 5a99083d..00000000
--- a/src/ipa/raspberrypi/cam_helper_ov5647.cpp
+++ /dev/null
@@ -1,109 +0,0 @@
-/* SPDX-License-Identifier: BSD-2-Clause */
-/*
- * Copyright (C) 2019, Raspberry Pi Ltd
- *
- * cam_helper_ov5647.cpp - camera information for ov5647 sensor
- */
-
-#include <assert.h>
-
-#include "cam_helper.h"
-
-using namespace RPiController;
-
-class CamHelperOv5647 : public CamHelper
-{
-public:
-	CamHelperOv5647();
-	uint32_t gainCode(double gain) const override;
-	double gain(uint32_t gainCode) const override;
-	void getDelays(int &exposureDelay, int &gainDelay,
-		       int &vblankDelay, int &hblankDelay) const override;
-	unsigned int hideFramesStartup() const override;
-	unsigned int hideFramesModeSwitch() const override;
-	unsigned int mistrustFramesStartup() const override;
-	unsigned int mistrustFramesModeSwitch() const override;
-
-private:
-	/*
-	 * Smallest difference between the frame length and integration time,
-	 * in units of lines.
-	 */
-	static constexpr int frameIntegrationDiff = 4;
-};
-
-/*
- * OV5647 doesn't output metadata, so we have to use the "unicam parser" which
- * works by counting frames.
- */
-
-CamHelperOv5647::CamHelperOv5647()
-	: CamHelper({}, frameIntegrationDiff)
-{
-}
-
-uint32_t CamHelperOv5647::gainCode(double gain) const
-{
-	return static_cast<uint32_t>(gain * 16.0);
-}
-
-double CamHelperOv5647::gain(uint32_t gainCode) const
-{
-	return static_cast<double>(gainCode) / 16.0;
-}
-
-void CamHelperOv5647::getDelays(int &exposureDelay, int &gainDelay,
-				int &vblankDelay, int &hblankDelay) const
-{
-	/*
-	 * We run this sensor in a mode where the gain delay is bumped up to
-	 * 2. It seems to be the only way to make the delays "predictable".
-	 */
-	exposureDelay = 2;
-	gainDelay = 2;
-	vblankDelay = 2;
-	hblankDelay = 2;
-}
-
-unsigned int CamHelperOv5647::hideFramesStartup() const
-{
-	/*
-	 * On startup, we get a couple of under-exposed frames which
-	 * we don't want shown.
-	 */
-	return 2;
-}
-
-unsigned int CamHelperOv5647::hideFramesModeSwitch() const
-{
-	/*
-	 * After a mode switch, we get a couple of under-exposed frames which
-	 * we don't want shown.
-	 */
-	return 2;
-}
-
-unsigned int CamHelperOv5647::mistrustFramesStartup() const
-{
-	/*
-	 * First couple of frames are under-exposed and are no good for control
-	 * algos.
-	 */
-	return 2;
-}
-
-unsigned int CamHelperOv5647::mistrustFramesModeSwitch() const
-{
-	/*
-	 * First couple of frames are under-exposed even after a simple
-	 * mode switch, and are no good for control algos.
-	 */
-	return 2;
-}
-
-static CamHelper *create()
-{
-	return new CamHelperOv5647();
-}
-
-static RegisterCamHelper reg("ov5647", &create);