Re: [PATCH v5 02/16] media: doc: add document for rkisp1 meta buffer format
From: Hans Verkuil <hidden>
Date: 2018-02-06 14:28:14
Also in:
linux-arm-kernel, linux-media, linux-rockchip, lkml
On 12/29/17 08:52, Shunqian Zheng wrote:
quoted hunk ↗ jump to hunk
From: Jacob Chen <redacted> This commit add docuemnt for rkisp1 meta buffer format Signed-off-by: Jacob Chen <redacted> --- Documentation/media/uapi/v4l/meta-formats.rst | 2 ++ .../media/uapi/v4l/pixfmt-meta-rkisp1-params.rst | 17 +++++++++++++++++ .../media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst | 18 ++++++++++++++++++ 3 files changed, 37 insertions(+) create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst create mode 100644 Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rstdiff --git a/Documentation/media/uapi/v4l/meta-formats.rst b/Documentation/media/uapi/v4l/meta-formats.rst index 01e24e3..1b82814 100644 --- a/Documentation/media/uapi/v4l/meta-formats.rst +++ b/Documentation/media/uapi/v4l/meta-formats.rst@@ -14,3 +14,5 @@ These formats are used for the :ref:`metadata` interface only. pixfmt-meta-vsp1-hgo pixfmt-meta-vsp1-hgt + pixfmt-meta-rkisp1-params + pixfmt-meta-rkisp1-statdiff --git a/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst b/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst new file mode 100644 index 0000000..ed344d4 --- /dev/null +++ b/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst@@ -0,0 +1,17 @@ +.. -*- coding: utf-8; mode: rst -*- + +.. _v4l2-meta-fmt-rkisp1-params: + +******************************* +V4L2_META_FMT_RK_ISP1_PARAMS +******************************* + +Rockchip ISP1 Parameters Data + +Description +=========== + +This format describes input parameters for the Rockchip ISP1. + +The data use c-struct :c:type:`rkisp1_isp_params_cfg`, which is defined in +the ``linux/rkisp1-config.h`` header file, See it for details.
One more question: does the ISP produce a reasonable picture if it doesn't receive these params? If not (i.e. you always need to provide params), then I think you should provide a default rkisp1_isp_params_cfg struct that can be used as a template for application writers. Perhaps it can be part of the driver as the initial params config. I think even if the ISP does work without params it is still worthwhile doing this. The params are complex and having an example on how to initialize it would be helpful. Regards, Hans
quoted hunk ↗ jump to hunk
diff --git a/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst b/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst new file mode 100644 index 0000000..5ecc403 --- /dev/null +++ b/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst@@ -0,0 +1,18 @@ +.. -*- coding: utf-8; mode: rst -*- + +.. _v4l2-meta-fmt-rkisp1-stat: + +******************************* +V4L2_META_FMT_RK_ISP1_STAT_3A +******************************* + +Rockchip ISP1 Statistics Data + +Description +=========== + +This format describes image color statistics information generated by the Rockchip +ISP1. + +The data use c-struct :c:type:`rkisp1_stat_buffer`, which is defined in +the ``linux/cifisp_stat.h`` header file, See it for details.
-- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html