@@ -16,7 +16,6 @@
namespace libcamera {
struct DebayerParams {
- static constexpr unsigned int kGain10 = 256;
static constexpr unsigned int kRGBLookupSize = 256;
using ColorLookupTable = std::array<uint8_t, kRGBLookupSize>;
@@ -290,8 +290,7 @@ void IPASoftSimple::processStats(const ControlList &sensorControls)
for (unsigned int i = 0; i < DebayerParams::kRGBLookupSize; i++) {
constexpr unsigned int div =
- DebayerParams::kRGBLookupSize * DebayerParams::kGain10 /
- kGammaLookupSize;
+ DebayerParams::kRGBLookupSize * 256 / kGammaLookupSize;
unsigned int idx;
/* Apply gamma after gain! */
@@ -18,11 +18,6 @@ namespace libcamera {
* \brief Struct to hold the debayer parameters.
*/
-/**
- * \var DebayerParams::kGain10
- * \brief const value for 1.0 gain
- */
-
/**
* \var DebayerParams::kRGBLookupSize
* \brief Size of a color lookup table
The constant is used in a single place internally and doesn't belong to DebayerParams anymore. Let's use 256 directly. Signed-off-by: Milan Zamazal <mzamazal@redhat.com> --- include/libcamera/internal/software_isp/debayer_params.h | 1 - src/ipa/simple/soft_simple.cpp | 3 +-- src/libcamera/software_isp/debayer.cpp | 5 ----- 3 files changed, 1 insertion(+), 8 deletions(-)