linux-kernel - Re: [PATCH v11 02/11] media: staging: rkisp1: add document for rkisp1 meta buffer format

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 14 Nov 2019 07:58:56 -0300
From:   Helen Koike <helen.koike@...labora.com>
To:     Hans Verkuil <hverkuil@...all.nl>,
        linux-rockchip@...ts.infradead.org
Cc:     mark.rutland@....com, devicetree@...r.kernel.org,
        eddie.cai.linux@...il.com, mchehab@...nel.org, heiko@...ech.de,
        linux-arm-kernel@...ts.infradead.org, gregkh@...uxfoundation.org,
        jeffy.chen@...k-chips.com, zyc@...k-chips.com,
        linux-kernel@...r.kernel.org, tfiga@...omium.org,
        robh+dt@...nel.org, hans.verkuil@...co.com,
        laurent.pinchart@...asonboard.com, sakari.ailus@...ux.intel.com,
        kernel@...labora.com, ezequiel@...labora.com,
        linux-media@...r.kernel.org, jacob-chen@...wrt.com,
        zhengsq@...k-chips.com, Jacob Chen <jacob-chen@...k-chips.com>
Subject: Re: [PATCH v11 02/11] media: staging: rkisp1: add document for rkisp1
 meta buffer format



On 11/14/19 6:21 AM, Hans Verkuil wrote:
> On 11/14/19 6:12 AM, Helen Koike wrote:
>> From: Jacob Chen <jacob2.chen@...k-chips.com>
>>
>> This commit add document for rkisp1 meta buffer format
>>
>> Signed-off-by: Jacob Chen <jacob-chen@...k-chips.com>
>> [refactored for upstream]
>> Signed-off-by: Helen Koike <helen.koike@...labora.com>
> 
> checkpatch gives me:
> 
> WARNING: Missing Signed-off-by: line by nominal patch author 'Jacob Chen <jacob2.chen@...k-chips.com>'
> 
> Looking at this series I see duplicate Signed-off-by entries for Jacob Chen and a total
> of three different email addresses:
> 
> jacob2.chen@...k-chips.com
> jacob-chen@...k-chips.com
> cc@...k-chips.com

And I see a name in the Maintainers file as Jacob Chen <jacob-chen@...wrt.com>.
I was wondering if I could replace the email by jacob-chen@...wrt.com, or if I should
keep the original ones.

Helen

> 
> 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
>>
>