Linux Media Controller development
 help / color / mirror / Atom feed
From: Karl Mehltretter <kmehltretter@gmail.com>
To: Keke Li <keke.li@amlogic.com>
Cc: Karl Mehltretter <kmehltretter@gmail.com>,
	Jacopo Mondi <jacopo.mondi@ideasonboard.com>,
	Hans Verkuil <hverkuil@kernel.org>,
	linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: [PATCH] media: uapi: Fix C3 ISP type references in kernel-doc
Date: Sat, 12 Sep 2026 01:30:11 +0200	[thread overview]
Message-ID: <20260911233011.30951-1-kmehltretter@gmail.com> (raw)

The C3 ISP UAPI comments use singular names for the AWB, AE, and AF tap
point enums, and misspell the parameter block header struct. The
corresponding definitions already used the correct names when these
comments were introduced.

Refer to c3_isp_params_awb_tap_points, c3_isp_params_ae_tap_points,
c3_isp_params_af_tap_points, and c3_isp_params_block_header.

Fixes: 6d406187ebc0 ("media: uapi: Add stats info and parameters buffer for C3 ISP")
Assisted-by: LLM
Signed-off-by: Karl Mehltretter <kmehltretter@gmail.com>
---
 include/uapi/linux/media/amlogic/c3-isp-config.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/uapi/linux/media/amlogic/c3-isp-config.h b/include/uapi/linux/media/amlogic/c3-isp-config.h
index 92db5dcdda181cb31665e230cc56b443fa37a0be..fc70b3124ccacff635a7d9c0135d4e3821ba1a33 100644
--- a/include/uapi/linux/media/amlogic/c3-isp-config.h
+++ b/include/uapi/linux/media/amlogic/c3-isp-config.h
@@ -247,7 +247,7 @@ enum c3_isp_params_awb_tap_points {
  * from :c:type:`c3_isp_params_block_type`
  *
  * @header: the C3 ISP parameters block header
- * @tap_point: the tap point from enum c3_isp_params_awb_tap_point
+ * @tap_point: the tap point from enum c3_isp_params_awb_tap_points
  * @satur_vald: AWB statistic over saturation control
  *		value: 0: disable, 1: enable
  * @horiz_zones_num: active number of hotizontal zones [0..32]
@@ -305,7 +305,7 @@ enum c3_isp_params_ae_tap_points {
  * @header: the C3 ISP parameters block header
  * @horiz_zones_num: active number of horizontal zones [0..17]
  * @vert_zones_num: active number of vertical zones [0..15]
- * @tap_point: the tap point from enum c3_isp_params_ae_tap_point
+ * @tap_point: the tap point from enum c3_isp_params_ae_tap_points
  * @zone_weight: array of weights for AE statistics zones [0..15]
  * @horiz_coord: the horizontal coordinate of points on the diagonal [0..2888]
  * @vert_coord: the vertical coordinate of points on the diagonal [0..2240]
@@ -342,7 +342,7 @@ enum c3_isp_params_af_tap_points {
  * from :c:type:`c3_isp_params_block_type`
  *
  * @header: the C3 ISP parameters block header
- * @tap_point: the tap point from enum c3_isp_params_af_tap_point
+ * @tap_point: the tap point from enum c3_isp_params_af_tap_points
  * @horiz_zones_num: active number of hotizontal zones [0..17]
  * @vert_zones_num: active number of vertical zones [0..15]
  * @reserved: applications must zero this array
@@ -487,7 +487,7 @@ struct c3_isp_params_blc {
  *	| | | b_gain = ...;                                               | | |
  *	| | | gb_gain = ...;                                              | | |
  *	| | +------------------ struct c3_isp_params_awb_config ----------+ | |
- *	| | | +---------- struct c3_isp_param_block_header header ------+ | | |
+ *	| | | +---------- struct c3_isp_params_block_header header -----+ | | |
  *	| | | | type = C3_ISP_PARAMS_BLOCK_AWB_CONFIG;                  | | | |
  *	| | | | flags = C3_ISP_PARAMS_BLOCK_FL_NONE;                    | | | |
  *	| | | | size = sizeof(struct c3_isp_params_awb_config)          | | | |
-- 
2.39.5 (Apple Git-154)


                 reply	other threads:[~2026-09-11 23:31 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260911233011.30951-1-kmehltretter@gmail.com \
    --to=kmehltretter@gmail.com \
    --cc=hverkuil@kernel.org \
    --cc=jacopo.mondi@ideasonboard.com \
    --cc=keke.li@amlogic.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox