| Message ID | 20210615105139.16171-3-david.plowman@raspberrypi.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series |
|
| Related | show |
Hi David, Thank you for your patch. On Tue, 15 Jun 2021 at 11:51, David Plowman <david.plowman@raspberrypi.com> wrote: > Warnings about the lack of AWB status results are demoted to being > just "Debug". With monochrome sensors becoming more common this would > otherwise overwhelm the console output, and in practice nothing is > really lost as it is normally very evident if AWB is failing to run. > > Signed-off-by: David Plowman <david.plowman@raspberrypi.com> > Looks good! Reviewed-by: Naushir Patuck <naush@raspberrypi.com> > --- > src/ipa/raspberrypi/controller/rpi/agc.cpp | 2 +- > src/ipa/raspberrypi/controller/rpi/alsc.cpp | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/ipa/raspberrypi/controller/rpi/agc.cpp > b/src/ipa/raspberrypi/controller/rpi/agc.cpp > index 55e80ac7..2c95f287 100644 > --- a/src/ipa/raspberrypi/controller/rpi/agc.cpp > +++ b/src/ipa/raspberrypi/controller/rpi/agc.cpp > @@ -481,7 +481,7 @@ void Agc::fetchAwbStatus(Metadata *image_metadata) > awb_.gain_g = 1.0; > awb_.gain_b = 1.0; > if (image_metadata->Get("awb.status", awb_) != 0) > - LOG(RPiAgc, Warning) << "Agc: no AWB status found"; > + LOG(RPiAgc, Debug) << "Agc: no AWB status found"; > } > > static double compute_initial_Y(bcm2835_isp_stats *stats, AwbStatus const > &awb, > diff --git a/src/ipa/raspberrypi/controller/rpi/alsc.cpp > b/src/ipa/raspberrypi/controller/rpi/alsc.cpp > index b0d24aa2..c58ef7d0 100644 > --- a/src/ipa/raspberrypi/controller/rpi/alsc.cpp > +++ b/src/ipa/raspberrypi/controller/rpi/alsc.cpp > @@ -267,7 +267,7 @@ double get_ct(Metadata *metadata, double default_ct) > AwbStatus awb_status; > awb_status.temperature_K = default_ct; // in case nothing found > if (metadata->Get("awb.status", awb_status) != 0) > - LOG(RPiAlsc, Warning) << "no AWB results found, using " > + LOG(RPiAlsc, Debug) << "no AWB results found, using " > << awb_status.temperature_K; > else > LOG(RPiAlsc, Debug) << "AWB results found, using " > -- > 2.20.1 > >
diff --git a/src/ipa/raspberrypi/controller/rpi/agc.cpp b/src/ipa/raspberrypi/controller/rpi/agc.cpp index 55e80ac7..2c95f287 100644 --- a/src/ipa/raspberrypi/controller/rpi/agc.cpp +++ b/src/ipa/raspberrypi/controller/rpi/agc.cpp @@ -481,7 +481,7 @@ void Agc::fetchAwbStatus(Metadata *image_metadata) awb_.gain_g = 1.0; awb_.gain_b = 1.0; if (image_metadata->Get("awb.status", awb_) != 0) - LOG(RPiAgc, Warning) << "Agc: no AWB status found"; + LOG(RPiAgc, Debug) << "Agc: no AWB status found"; } static double compute_initial_Y(bcm2835_isp_stats *stats, AwbStatus const &awb, diff --git a/src/ipa/raspberrypi/controller/rpi/alsc.cpp b/src/ipa/raspberrypi/controller/rpi/alsc.cpp index b0d24aa2..c58ef7d0 100644 --- a/src/ipa/raspberrypi/controller/rpi/alsc.cpp +++ b/src/ipa/raspberrypi/controller/rpi/alsc.cpp @@ -267,7 +267,7 @@ double get_ct(Metadata *metadata, double default_ct) AwbStatus awb_status; awb_status.temperature_K = default_ct; // in case nothing found if (metadata->Get("awb.status", awb_status) != 0) - LOG(RPiAlsc, Warning) << "no AWB results found, using " + LOG(RPiAlsc, Debug) << "no AWB results found, using " << awb_status.temperature_K; else LOG(RPiAlsc, Debug) << "AWB results found, using "
Warnings about the lack of AWB status results are demoted to being just "Debug". With monochrome sensors becoming more common this would otherwise overwhelm the console output, and in practice nothing is really lost as it is normally very evident if AWB is failing to run. Signed-off-by: David Plowman <david.plowman@raspberrypi.com> --- src/ipa/raspberrypi/controller/rpi/agc.cpp | 2 +- src/ipa/raspberrypi/controller/rpi/alsc.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)