NekoCWD/libcamera
NekoCWD/libcamera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 7 Wiki Activity Actions

libcamera: ipa: raspberrypi: agc: Rename method to divideUpExposure

Browse source 
The method formerly known as divvyupExposure is given a more
understandable name.

Signed-off-by: David Plowman <david.plowman@raspberrypi.com>
Reviewed-by: Naushir Patuck <naush@raspberrypi.com>
Acked-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
David Plowman 2020-11-23 07:37:57 +00:00 • committed by Kieran Bingham
parent 42f4e313af
commit 6af665992d
2 changed files with 5 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/ipa/raspberrypi/controller/rpi/agc.cpp
View file

@ -225,7 +225,7 @@ void Agc::SwitchMode([[maybe_unused]] CameraMode const &camera_mode,
// write the results into the metadata we've been given. // write the results into the metadata we've been given.
if (status_.total_exposure_value) { if (status_.total_exposure_value) {
housekeepConfig(); housekeepConfig();
divvyupExposure(); divideUpExposure();
writeAndFinish(metadata, false); writeAndFinish(metadata, false);
} }
} }
@ -303,9 +303,9 @@ void Agc::Process(StatisticsPtr &stats, Metadata *image_metadata)
bool desaturate = applyDigitalGain(image_metadata, gain, target_Y); bool desaturate = applyDigitalGain(image_metadata, gain, target_Y);
// The results have to be filtered so as not to change too rapidly. // The results have to be filtered so as not to change too rapidly.
filterExposure(desaturate); filterExposure(desaturate);
// The last thing is to divvy up the exposure value into a shutter time // The last thing is to divide up the exposure value into a shutter time
// and analogue_gain, according to the current exposure mode. // and analogue_gain, according to the current exposure mode.
divvyupExposure(); divideUpExposure();
// Finally advertise what we've done. // Finally advertise what we've done.
writeAndFinish(image_metadata, desaturate); writeAndFinish(image_metadata, desaturate);
} }
@ -544,7 +544,7 @@ void Agc::filterExposure(bool desaturate)
<< " no dg " << filtered_.total_exposure_no_dg; << " no dg " << filtered_.total_exposure_no_dg;
} }
void Agc::divvyupExposure() void Agc::divideUpExposure()
{ {
// Sending the fixed shutter/gain cases through the same code may seem // Sending the fixed shutter/gain cases through the same code may seem
// unnecessary, but it will make more sense when extend this to cover // unnecessary, but it will make more sense when extend this to cover

2
src/ipa/raspberrypi/controller/rpi/agc.hpp
View file

@ -89,7 +89,7 @@ private:
bool applyDigitalGain(Metadata *image_metadata, double gain, bool applyDigitalGain(Metadata *image_metadata, double gain,
double target_Y); double target_Y);
void filterExposure(bool desaturate); void filterExposure(bool desaturate);
void divvyupExposure(); void divideUpExposure();
void writeAndFinish(Metadata *image_metadata, bool desaturate); void writeAndFinish(Metadata *image_metadata, bool desaturate);
AgcMeteringMode *metering_mode_; AgcMeteringMode *metering_mode_;
AgcExposureMode *exposure_mode_; AgcExposureMode *exposure_mode_;