[libcamera-devel] [PATCH] ipa: ipu3: Fix minor Doxygen issues in IPAFrameContext

Kieran Bingham kieran.bingham at ideasonboard.com
Tue May 17 18:44:47 CEST 2022


Quoting Laurent Pinchart via libcamera-devel (2022-05-17 15:18:40)
> The IPAFrameContext documentation has a spurious Doxygen \struct
> statement that is not needed, and uses a \struct instead of a \var in
> another location. Fix both issues. This doesn't cause any change in the
> generated documentation.
> 

I'd be tempted to reference the known issue and reported bug on doxygen
here, but it doesn't matter so much.


Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
>  src/ipa/ipu3/ipa_context.cpp | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/ipa/ipu3/ipa_context.cpp b/src/ipa/ipu3/ipa_context.cpp
> index b1570dde7fe4..26e74c747877 100644
> --- a/src/ipa/ipu3/ipa_context.cpp
> +++ b/src/ipa/ipu3/ipa_context.cpp
> @@ -81,7 +81,6 @@ namespace libcamera::ipa::ipu3 {
>   * \var IPAFrameContext::af
>   * \brief Context for the Automatic Focus algorithm
>   *
> - * \struct  IPAFrameContext::af
>   * \var IPAFrameContext::af.focus
>   * \brief Current position of the lens
>   *
> @@ -140,7 +139,7 @@ namespace libcamera::ipa::ipu3 {
>   * \var IPAFrameContext::awb
>   * \brief Context for the Automatic White Balance algorithm
>   *
> - * \struct IPAFrameContext::awb.gains
> + * \var IPAFrameContext::awb.gains
>   * \brief White balance gains
>   *
>   * \var IPAFrameContext::awb.gains.red
> 
> base-commit: 5efb6c8e2acdc2dc9edf1f22f6c2a5818062b69e
> -- 
> Regards,
> 
> Laurent Pinchart
>


More information about the libcamera-devel mailing list