lkml.org 
[lkml]   [2020]   [Jul]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v2 2/2] media: atomisp: Fix coding style issue - correct multiline comments
Em Wed, 22 Jul 2020 17:00:52 +0530
Rahul Gottipati <rahul.blr97@gmail.com> escreveu:

> This fixes some coding style issues of multiline comments to
> correct a few checkpatch.pl warnings.
>
> Signed-off-by: Rahul Gottipati <rahul.blr97@gmail.com>
> ---
> Changes in v2:
> Distributed changes across 2 patches instead of the previous 1.
> drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> index 9cdcbe774229..5bf3a86f98f8 100644
> --- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> +++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c
> @@ -1281,7 +1281,8 @@ static int atomisp_qbuf(struct file *file, void *fh, struct v4l2_buffer *buf)
> * But the capture number cannot be determined by HAL.
> * So HAL only sets the capture number to be 1 and queue multiple
> * buffers. Atomisp driver needs to check this case and re-trigger
> - * CSS to do capture when new buffer is queued. */
> + * CSS to do capture when new buffer is queued.
> + */
> if (asd->continuous_mode->val &&
> atomisp_subdev_source_pad(vdev)
> == ATOMISP_SUBDEV_PAD_SOURCE_CAPTURE &&
> @@ -1806,7 +1807,7 @@ static int atomisp_streamon(struct file *file, void *fh,
> /*
> * set freq to max when streaming count > 1 which indicate
> * dual camera would run
> - */
> + */
> if (atomisp_streaming_count(isp) > 1) {
> if (atomisp_freq_scaling(isp,
> ATOMISP_DFS_MODE_MAX, false) < 0)
> @@ -2438,7 +2439,8 @@ static int atomisp_g_ext_ctrls(struct file *file, void *fh,
> int i, ret = 0;
>
> /* input_lock is not need for the Camera related IOCTLs
> - * The input_lock downgrade the FPS of 3A*/
> + * The input_lock downgrade the FPS of 3A
> + */


On media (and on several subsystems), we keep the first line in blank,
on multi-line comments:

/*
* input_lock is not need for the Camera related IOCTLs
* The input_lock downgrade the FPS of 3A
*/

> ret = atomisp_camera_g_ext_ctrls(file, fh, c);
> if (ret != -EINVAL)
> return ret;
> @@ -2521,7 +2523,8 @@ static int atomisp_camera_s_ext_ctrls(struct file *file, void *fh,
> v4l2_s_ctrl(NULL, isp->flash->ctrl_handler,
> &ctrl);
> /* When flash mode is changed we need to reset
> - * flash state */
> + * flash state
> + */
> if (ctrl.id == V4L2_CID_FLASH_MODE) {
> asd->params.flash_state =
> ATOMISP_FLASH_IDLE;
> @@ -2560,7 +2563,8 @@ static int atomisp_s_ext_ctrls(struct file *file, void *fh,
> int i, ret = 0;
>
> /* input_lock is not need for the Camera related IOCTLs
> - * The input_lock downgrade the FPS of 3A*/
> + * The input_lock downgrade the FPS of 3A
> + */
> ret = atomisp_camera_s_ext_ctrls(file, fh, c);
> if (ret != -EINVAL)
> return ret;



Thanks,
Mauro

\
 
 \ /
  Last update: 2020-07-22 16:07    [W:0.490 / U:1.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site