On 11/14/19 6:12 AM, Helen Koike wrote: > From: Jacob Chen <jacob2.chen@xxxxxxxxxxxxxx> > > This commit add document for rkisp1 meta buffer format > > Signed-off-by: Jacob Chen <jacob-chen@xxxxxxxxxxxxxx> > [refactored for upstream] > Signed-off-by: Helen Koike <helen.koike@xxxxxxxxxxxxx> checkpatch gives me: WARNING: Missing Signed-off-by: line by nominal patch author 'Jacob Chen <jacob2.chen@xxxxxxxxxxxxxx>' Looking at this series I see duplicate Signed-off-by entries for Jacob Chen and a total of three different email addresses: jacob2.chen@xxxxxxxxxxxxxx jacob-chen@xxxxxxxxxxxxxx cc@xxxxxxxxxxxxxx It's confusing. Regards, Hans > > --- > > Changes in v11: None > Changes in v10: > - unsquash > > Changes in v9: > - squash > - migrate to staging > - remove meta-formats.rst update > > Changes in v8: > - Add SPDX in the header > - Remove emacs configs > - Fix doc style > > Changes in v7: > - s/correspond/corresponding > - s/use/uses > - s/docuemnt/document > > .../uapi/v4l/pixfmt-meta-rkisp1-params.rst | 23 +++++++++++++++++++ > .../uapi/v4l/pixfmt-meta-rkisp1-stat.rst | 22 ++++++++++++++++++ > 2 files changed, 45 insertions(+) > create mode 100644 drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst > create mode 100644 drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst > > diff --git a/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst b/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst > new file mode 100644 > index 000000000000..103b5cb79b7c > --- /dev/null > +++ b/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-params.rst > @@ -0,0 +1,23 @@ > +.. SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +.. _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. > + > +It uses c-struct :c:type:`rkisp1_isp_params_cfg`, which is defined in > +the ``linux/rkisp1-config.h`` header file. > + > +The parameters consist of multiple modules. > +The module won't be updated if the corresponding bit was not set in module_*_update. > + > +.. kernel-doc:: include/uapi/linux/rkisp1-config.h > + :functions: rkisp1_isp_params_cfg > diff --git a/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst b/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst > new file mode 100644 > index 000000000000..4ad303f96421 > --- /dev/null > +++ b/drivers/staging/media/rkisp1/Documentation/media/uapi/v4l/pixfmt-meta-rkisp1-stat.rst > @@ -0,0 +1,22 @@ > +.. SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +.. _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. > + > +It uses c-struct :c:type:`rkisp1_stat_buffer`, which is defined in > +the ``linux/rkisp1-config.h`` header file. > + > +.. kernel-doc:: include/uapi/linux/rkisp1-config.h > + :functions: rkisp1_stat_buffer >