* [PATCH] media: rkisp1: uapi: document which flags/structs relate to statistics config
@ 2021-03-23 9:22 Dafna Hirschfeld
2021-03-23 9:31 ` Sebastian Fricke
0 siblings, 1 reply; 2+ messages in thread
From: Dafna Hirschfeld @ 2021-03-23 9:22 UTC (permalink / raw)
To: linux-media
Cc: laurent.pinchart, dafna.hirschfeld, helen.koike, hverkuil,
kernel, dafna3, tfiga
Add a documentation to tell which flags and structs are
used to configure the statistics.
Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
---
include/uapi/linux/rkisp1-config.h | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/include/uapi/linux/rkisp1-config.h b/include/uapi/linux/rkisp1-config.h
index 36e3efb81b01..8d1f8bd192fc 100644
--- a/include/uapi/linux/rkisp1-config.h
+++ b/include/uapi/linux/rkisp1-config.h
@@ -15,7 +15,7 @@
#define RKISP1_CIF_ISP_MODULE_BLS (1U << 1)
/* Sensor De-gamma */
#define RKISP1_CIF_ISP_MODULE_SDG (1U << 2)
-/* Histogram */
+/* Histogram statistics configuration */
#define RKISP1_CIF_ISP_MODULE_HST (1U << 3)
/* Lens Shade Control */
#define RKISP1_CIF_ISP_MODULE_LSC (1U << 4)
@@ -31,13 +31,13 @@
#define RKISP1_CIF_ISP_MODULE_GOC (1U << 9)
/* Color Processing */
#define RKISP1_CIF_ISP_MODULE_CPROC (1U << 10)
-/* Auto Focus Control */
+/* Auto Focus Control statistics configuration */
#define RKISP1_CIF_ISP_MODULE_AFC (1U << 11)
-/* Auto White Balancing */
+/* Auto White Balancing statistics configuration */
#define RKISP1_CIF_ISP_MODULE_AWB (1U << 12)
/* Image Effect */
#define RKISP1_CIF_ISP_MODULE_IE (1U << 13)
-/* Auto Exposure Control */
+/* Auto Exposure Control statistics Configuration */
#define RKISP1_CIF_ISP_MODULE_AEC (1U << 14)
/* Wide Dynamic Range */
#define RKISP1_CIF_ISP_MODULE_WDR (1U << 15)
@@ -411,7 +411,7 @@ struct rkisp1_cif_isp_cproc_config {
};
/**
- * struct rkisp1_cif_isp_awb_meas_config - Configuration used by auto white balance
+ * struct rkisp1_cif_isp_awb_meas_config - Configuration for the AWB statistics
*
* @awb_mode: the awb meas mode. From enum rkisp1_cif_isp_awb_mode_type.
* @awb_wnd: white balance measurement window (in pixels)
@@ -550,7 +550,7 @@ struct rkisp1_cif_isp_goc_config {
};
/**
- * struct rkisp1_cif_isp_hst_config - Configuration used by Histogram
+ * struct rkisp1_cif_isp_hst_config - Configuration for Histogram statistics
*
* @mode: histogram mode (from enum rkisp1_cif_isp_histogram_mode)
* @histogram_predivider: process every stepsize pixel, all other pixels are
@@ -575,7 +575,7 @@ struct rkisp1_cif_isp_hst_config {
};
/**
- * struct rkisp1_cif_isp_aec_config - Configuration used by Auto Exposure Control
+ * struct rkisp1_cif_isp_aec_config - Configuration for Auto Exposure statistics
*
* @mode: Exposure measure mode (from enum rkisp1_cif_isp_exp_meas_mode)
* @autostop: stop mode (from enum rkisp1_cif_isp_exp_ctrl_autostop)
@@ -588,7 +588,7 @@ struct rkisp1_cif_isp_aec_config {
};
/**
- * struct rkisp1_cif_isp_afc_config - Configuration used by Auto Focus Control
+ * struct rkisp1_cif_isp_afc_config - Configuration for the Auto Focus statistics
*
* @num_afm_win: max RKISP1_CIF_ISP_AFM_MAX_WINDOWS
* @afm_win: coordinates of the meas window
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] media: rkisp1: uapi: document which flags/structs relate to statistics config
2021-03-23 9:22 [PATCH] media: rkisp1: uapi: document which flags/structs relate to statistics config Dafna Hirschfeld
@ 2021-03-23 9:31 ` Sebastian Fricke
0 siblings, 0 replies; 2+ messages in thread
From: Sebastian Fricke @ 2021-03-23 9:31 UTC (permalink / raw)
To: Dafna Hirschfeld
Cc: linux-media, laurent.pinchart, helen.koike, hverkuil, kernel,
dafna3, tfiga
Hey Dafna,
Thanks a lot for this change.
This will certainly help for the task of writing the IPAs in libcamera.
If you like you can add:
Reviewed-by: Sebastian Fricke <sebastian.fricke@posteo.net>
On 23.03.2021 10:22, Dafna Hirschfeld wrote:
>Add a documentation to tell which flags and structs are
I would rather write:
s/Add a documentation to tell/Modify the documentation to point out/
>used to configure the statistics.
>
>Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com>
>---
> include/uapi/linux/rkisp1-config.h | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
>diff --git a/include/uapi/linux/rkisp1-config.h b/include/uapi/linux/rkisp1-config.h
>index 36e3efb81b01..8d1f8bd192fc 100644
>--- a/include/uapi/linux/rkisp1-config.h
>+++ b/include/uapi/linux/rkisp1-config.h
>@@ -15,7 +15,7 @@
> #define RKISP1_CIF_ISP_MODULE_BLS (1U << 1)
> /* Sensor De-gamma */
> #define RKISP1_CIF_ISP_MODULE_SDG (1U << 2)
>-/* Histogram */
>+/* Histogram statistics configuration */
> #define RKISP1_CIF_ISP_MODULE_HST (1U << 3)
> /* Lens Shade Control */
> #define RKISP1_CIF_ISP_MODULE_LSC (1U << 4)
>@@ -31,13 +31,13 @@
> #define RKISP1_CIF_ISP_MODULE_GOC (1U << 9)
> /* Color Processing */
> #define RKISP1_CIF_ISP_MODULE_CPROC (1U << 10)
>-/* Auto Focus Control */
>+/* Auto Focus Control statistics configuration */
> #define RKISP1_CIF_ISP_MODULE_AFC (1U << 11)
>-/* Auto White Balancing */
>+/* Auto White Balancing statistics configuration */
> #define RKISP1_CIF_ISP_MODULE_AWB (1U << 12)
> /* Image Effect */
> #define RKISP1_CIF_ISP_MODULE_IE (1U << 13)
>-/* Auto Exposure Control */
>+/* Auto Exposure Control statistics Configuration */
In order to align with the other comments:
s/Configuration/configuration/
> #define RKISP1_CIF_ISP_MODULE_AEC (1U << 14)
> /* Wide Dynamic Range */
> #define RKISP1_CIF_ISP_MODULE_WDR (1U << 15)
>@@ -411,7 +411,7 @@ struct rkisp1_cif_isp_cproc_config {
> };
>
> /**
>- * struct rkisp1_cif_isp_awb_meas_config - Configuration used by auto white balance
>+ * struct rkisp1_cif_isp_awb_meas_config - Configuration for the AWB statistics
> *
> * @awb_mode: the awb meas mode. From enum rkisp1_cif_isp_awb_mode_type.
> * @awb_wnd: white balance measurement window (in pixels)
>@@ -550,7 +550,7 @@ struct rkisp1_cif_isp_goc_config {
> };
>
> /**
>- * struct rkisp1_cif_isp_hst_config - Configuration used by Histogram
>+ * struct rkisp1_cif_isp_hst_config - Configuration for Histogram statistics
> *
> * @mode: histogram mode (from enum rkisp1_cif_isp_histogram_mode)
> * @histogram_predivider: process every stepsize pixel, all other pixels are
>@@ -575,7 +575,7 @@ struct rkisp1_cif_isp_hst_config {
> };
>
> /**
>- * struct rkisp1_cif_isp_aec_config - Configuration used by Auto Exposure Control
>+ * struct rkisp1_cif_isp_aec_config - Configuration for Auto Exposure statistics
> *
> * @mode: Exposure measure mode (from enum rkisp1_cif_isp_exp_meas_mode)
> * @autostop: stop mode (from enum rkisp1_cif_isp_exp_ctrl_autostop)
>@@ -588,7 +588,7 @@ struct rkisp1_cif_isp_aec_config {
> };
>
> /**
>- * struct rkisp1_cif_isp_afc_config - Configuration used by Auto Focus Control
>+ * struct rkisp1_cif_isp_afc_config - Configuration for the Auto Focus statistics
> *
> * @num_afm_win: max RKISP1_CIF_ISP_AFM_MAX_WINDOWS
> * @afm_win: coordinates of the meas window
>--
>2.17.1
Greetings,
Sebastian
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-03-23 9:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-23 9:22 [PATCH] media: rkisp1: uapi: document which flags/structs relate to statistics config Dafna Hirschfeld
2021-03-23 9:31 ` Sebastian Fricke
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.