[v2,1/3] ipa: rkisp1: Fix awb algorithm brief
diff mbox series

Message ID 20260204-kbingham-fixups-v2-1-c341cced1fad@ideasonboard.com
State Accepted
Headers show
Series
  • ipa: awb: Fix and cleanup RKISP1 and Simple Awb
Related show

Commit Message

Kieran Bingham Feb. 4, 2026, 10:42 p.m. UTC
The Awb brief was not updated from when the Bayes AWB was added.

Extend it to account for this.

Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/ipa/rkisp1/algorithms/awb.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jacopo Mondi Feb. 5, 2026, 7:51 a.m. UTC | #1
Hi Kieran

On Wed, Feb 04, 2026 at 10:42:56PM +0000, Kieran Bingham wrote:
> The Awb brief was not updated from when the Bayes AWB was added.
>
> Extend it to account for this.
>
> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>

Thanks
  j

> ---
>  src/ipa/rkisp1/algorithms/awb.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/ipa/rkisp1/algorithms/awb.cpp b/src/ipa/rkisp1/algorithms/awb.cpp
> index e8da7974a1d6ad9ec2e22c6b013aead0a88ee1c3..f83da545be856c08cd758dc20a5ace91344101cf 100644
> --- a/src/ipa/rkisp1/algorithms/awb.cpp
> +++ b/src/ipa/rkisp1/algorithms/awb.cpp
> @@ -30,7 +30,7 @@ namespace ipa::rkisp1::algorithms {
>
>  /**
>   * \class Awb
> - * \brief A Grey world white balance correction algorithm
> + * \brief Manage the white balance with automatic and manual controls
>   */
>
>  LOG_DEFINE_CATEGORY(RkISP1Awb)
>
> --
> 2.52.0
>
Barnabás Pőcze Feb. 5, 2026, 10:27 a.m. UTC | #2
2026. 02. 04. 23:42 keltezéssel, Kieran Bingham írta:
> The Awb brief was not updated from when the Bayes AWB was added.
> 
> Extend it to account for this.
> 
> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> ---

Looks ok to me.

Reviewed-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>


>   src/ipa/rkisp1/algorithms/awb.cpp | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/rkisp1/algorithms/awb.cpp b/src/ipa/rkisp1/algorithms/awb.cpp
> index e8da7974a1d6ad9ec2e22c6b013aead0a88ee1c3..f83da545be856c08cd758dc20a5ace91344101cf 100644
> --- a/src/ipa/rkisp1/algorithms/awb.cpp
> +++ b/src/ipa/rkisp1/algorithms/awb.cpp
> @@ -30,7 +30,7 @@ namespace ipa::rkisp1::algorithms {
>   
>   /**
>    * \class Awb
> - * \brief A Grey world white balance correction algorithm
> + * \brief Manage the white balance with automatic and manual controls
>    */
>   
>   LOG_DEFINE_CATEGORY(RkISP1Awb)
>
Milan Zamazal Feb. 9, 2026, 3:59 p.m. UTC | #3
Kieran Bingham <kieran.bingham@ideasonboard.com> writes:

> The Awb brief was not updated from when the Bayes AWB was added.
>
> Extend it to account for this.
>
> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>

Reviewed-by: Milan Zamazal <mzamazal@redhat.com>

> ---
>  src/ipa/rkisp1/algorithms/awb.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/ipa/rkisp1/algorithms/awb.cpp b/src/ipa/rkisp1/algorithms/awb.cpp
> index e8da7974a1d6ad9ec2e22c6b013aead0a88ee1c3..f83da545be856c08cd758dc20a5ace91344101cf 100644
> --- a/src/ipa/rkisp1/algorithms/awb.cpp
> +++ b/src/ipa/rkisp1/algorithms/awb.cpp
> @@ -30,7 +30,7 @@ namespace ipa::rkisp1::algorithms {
>  
>  /**
>   * \class Awb
> - * \brief A Grey world white balance correction algorithm
> + * \brief Manage the white balance with automatic and manual controls
>   */
>  
>  LOG_DEFINE_CATEGORY(RkISP1Awb)

Patch
diff mbox series

diff --git a/src/ipa/rkisp1/algorithms/awb.cpp b/src/ipa/rkisp1/algorithms/awb.cpp
index e8da7974a1d6ad9ec2e22c6b013aead0a88ee1c3..f83da545be856c08cd758dc20a5ace91344101cf 100644
--- a/src/ipa/rkisp1/algorithms/awb.cpp
+++ b/src/ipa/rkisp1/algorithms/awb.cpp
@@ -30,7 +30,7 @@  namespace ipa::rkisp1::algorithms {
 
 /**
  * \class Awb
- * \brief A Grey world white balance correction algorithm
+ * \brief Manage the white balance with automatic and manual controls
  */
 
 LOG_DEFINE_CATEGORY(RkISP1Awb)