media: staging: media: atomisp: Fix alignment and line length issues
authorMartiros Shakhzadyan <vrzh@vrzh.net>
Fri, 23 Apr 2021 16:13:48 +0000 (18:13 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Wed, 19 May 2021 07:51:38 +0000 (09:51 +0200)
Fix alignment style issues and adjacent line length issues in sh_css.c

Signed-off-by: Martiros Shakhzadyan <vrzh@vrzh.net>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/atomisp/pci/sh_css.c

index 346f571..0482bfa 100644 (file)
@@ -239,8 +239,8 @@ ia_css_reset_defaults(struct sh_css *css);
 static void
 sh_css_init_host_sp_control_vars(void);
 
-static int set_num_primary_stages(unsigned int *num,
-       enum ia_css_pipe_version version);
+static int
+set_num_primary_stages(unsigned int *num, enum ia_css_pipe_version version);
 
 static bool
 need_capture_pp(const struct ia_css_pipe *pipe);
@@ -2996,9 +2996,8 @@ static int add_firmwares(
 
                ia_css_pipe_get_firmwares_stage_desc(&stage_desc, binary,
                                                     out, in, vf, fw, binary_mode);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       &extra_stage);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          &extra_stage);
                if (err)
                        return err;
                if (fw->info.isp.sp.enable.output != 0)
@@ -3106,9 +3105,8 @@ static int add_yuv_scaler_stage(
                ia_css_pipe_get_generic_stage_desc(&stage_desc,
                                                   yuv_scaler_binary, out_frames, in_frame, vf_frame);
        }
-       err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc,
-               pre_vf_pp_stage);
+       err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                  pre_vf_pp_stage);
        if (err)
                return err;
        in_frame = (*pre_vf_pp_stage)->args.out_frame[0];
@@ -3166,9 +3164,8 @@ static int add_capture_pp_stage(
                ia_css_pipe_get_generic_stage_desc(&stage_desc,
                                                   capture_pp_binary, out_frames, NULL, vf_frame);
        }
-       err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc,
-               capture_pp_stage);
+       err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                  capture_pp_stage);
        if (err)
                return err;
        err = add_firmwares(me, capture_pp_binary, pipe->output_stage, last_fw,
@@ -3523,9 +3520,8 @@ static int create_host_video_pipeline(struct ia_css_pipe *pipe)
                ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
                                                   out_frames, NULL, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       &copy_stage);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          &copy_stage);
                if (err)
                        goto ERR;
                in_frame = me->stages->args.out_frame[0];
@@ -3552,9 +3548,8 @@ static int create_host_video_pipeline(struct ia_css_pipe *pipe)
                ia_css_pipe_get_generic_stage_desc(&stage_desc, video_binary,
                                                   out_frames, in_frame, vf_frame);
        }
-       err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc,
-               &video_stage);
+       err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                  &video_stage);
        if (err)
                goto ERR;
 
@@ -3619,8 +3614,8 @@ static int create_host_video_pipeline(struct ia_css_pipe *pipe)
                for (i = 0; i < num_yuv_scaler; i++) {
                        tmp_out_frame = is_output_stage[i] ? out_frame : NULL;
 
-                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame, tmp_out_frame,
-                                                  NULL,
+                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame,
+                                                  tmp_out_frame, NULL,
                                                   &yuv_scaler_binary[i],
                                                   &yuv_scaler_stage);
 
@@ -3759,9 +3754,8 @@ create_host_preview_pipeline(struct ia_css_pipe *pipe)
                ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
                                                   out_frames, NULL, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       &copy_stage);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          &copy_stage);
                if (err)
                        goto ERR;
                in_frame = me->stages->args.out_frame[0];
@@ -3787,9 +3781,8 @@ create_host_preview_pipeline(struct ia_css_pipe *pipe)
                ia_css_pipe_get_generic_stage_desc(&stage_desc, preview_binary,
                                                   out_frames, in_frame, NULL);
        }
-       err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc,
-               &preview_stage);
+       err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                  &preview_stage);
        if (err)
                goto ERR;
        /* If we use copy iso preview, the input must be yuv iso raw */
@@ -4122,8 +4115,8 @@ ia_css_pipe_enqueue_buffer(struct ia_css_pipe *pipe,
        }
 
        hmm_store(h_vbuf->vptr,
-                  (void *)(&ddr_buffer),
-                  sizeof(struct sh_css_hmm_buffer));
+                 (void *)(&ddr_buffer),
+                 sizeof(struct sh_css_hmm_buffer));
        if ((buf_type == IA_CSS_BUFFER_TYPE_3A_STATISTICS)
            || (buf_type == IA_CSS_BUFFER_TYPE_DIS_STATISTICS)
            || (buf_type == IA_CSS_BUFFER_TYPE_LACE_STATISTICS)) {
@@ -4284,8 +4277,8 @@ ia_css_pipe_dequeue_buffer(struct ia_css_pipe *pipe,
                }
 
                hmm_load(ddr_buffer_addr,
-                         &ddr_buffer,
-                         sizeof(struct sh_css_hmm_buffer));
+                        &ddr_buffer,
+                        sizeof(struct sh_css_hmm_buffer));
 
                /* if the kernel_ptr is 0 or an invalid, return an error.
                 * do not access the buffer via the kernal_ptr.
@@ -4938,7 +4931,7 @@ sh_css_pipes_stop(struct ia_css_stream *stream)
        for (i = 0; i < stream->num_pipes; i++) {
                /* send the "stop" request to the "ia_css_pipe" instance */
                IA_CSS_LOG("Send the stop-request to the pipe: pipe_id=%d",
-                       stream->pipes[i]->pipeline.pipe_id);
+                          stream->pipes[i]->pipeline.pipe_id);
                err = ia_css_pipeline_request_stop(&stream->pipes[i]->pipeline);
 
                /*
@@ -4987,7 +4980,7 @@ sh_css_pipes_stop(struct ia_css_stream *stream)
 
                /* send the "stop" request to "Copy Pipe" */
                IA_CSS_LOG("Send the stop-request to the pipe: pipe_id=%d",
-                       copy_pipe->pipeline.pipe_id);
+                          copy_pipe->pipeline.pipe_id);
                err = ia_css_pipeline_request_stop(&copy_pipe->pipeline);
        }
 
@@ -5814,7 +5807,7 @@ static bool need_capt_ldc(
 }
 
 static int set_num_primary_stages(unsigned int *num,
-       enum ia_css_pipe_version version)
+                                 enum ia_css_pipe_version version)
 {
        int err = 0;
 
@@ -6014,10 +6007,13 @@ static int load_primary_binaries(
                        capt_pp_in_info = &prim_out_info;
 
                ia_css_pipe_get_capturepp_binarydesc(pipe,
-                                                       &capture_pp_descr, capt_pp_in_info,
-                                                       &capt_pp_out_info, &vf_info);
+                                                    &capture_pp_descr,
+                                                    capt_pp_in_info,
+                                                    &capt_pp_out_info,
+                                                    &vf_info);
+
                err = ia_css_binary_find(&capture_pp_descr,
-                                           &mycs->capture_pp_binary);
+                                        &mycs->capture_pp_binary);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -6027,11 +6023,12 @@ static int load_primary_binaries(
                        struct ia_css_binary_descr capt_ldc_descr;
 
                        ia_css_pipe_get_ldc_binarydesc(pipe,
-                                                       &capt_ldc_descr, &prim_out_info,
-                                                       &capt_ldc_out_info);
+                                                      &capt_ldc_descr,
+                                                      &prim_out_info,
+                                                      &capt_ldc_out_info);
 
                        err = ia_css_binary_find(&capt_ldc_descr,
-                                                   &mycs->capture_ldc_binary);
+                                                &mycs->capture_ldc_binary);
                        if (err) {
                                IA_CSS_LEAVE_ERR_PRIVATE(err);
                                return err;
@@ -6048,8 +6045,9 @@ static int load_primary_binaries(
                if (pipe->enable_viewfinder[IA_CSS_PIPE_OUTPUT_STAGE_0] &&
                    (i == mycs->num_primary_stage - 1))
                        local_vf_info = &vf_info;
-               ia_css_pipe_get_primary_binarydesc(pipe, &prim_descr[i], &prim_in_info,
-                                                   &prim_out_info, local_vf_info, i);
+               ia_css_pipe_get_primary_binarydesc(pipe, &prim_descr[i],
+                                                  &prim_in_info, &prim_out_info,
+                                                  local_vf_info, i);
                err = ia_css_binary_find(&prim_descr[i], &mycs->primary_binary[i]);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
@@ -6101,8 +6099,8 @@ static int load_primary_binaries(
        /* ISP Copy */
        if (need_isp_copy_binary) {
                err = load_copy_binary(pipe,
-                                       &mycs->copy_binary,
-                                       &mycs->primary_binary[0]);
+                                      &mycs->copy_binary,
+                                      &mycs->primary_binary[0]);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -6211,7 +6209,7 @@ static int load_advanced_binaries(
 
        assert(pipe);
        assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE ||
-               pipe->mode == IA_CSS_PIPE_ID_COPY);
+              pipe->mode == IA_CSS_PIPE_ID_COPY);
        if (pipe->pipe_settings.capture.pre_isp_binary.info)
                return 0;
        pipe_out_info = &pipe->output_info[0];
@@ -6224,17 +6222,18 @@ static int load_advanced_binaries(
        need_pp = need_capture_pp(pipe);
 
        ia_css_frame_info_set_format(&vf_info,
-                                       IA_CSS_FRAME_FORMAT_YUV_LINE);
+                                    IA_CSS_FRAME_FORMAT_YUV_LINE);
 
        /* we build up the pipeline starting at the end */
        /* Capture post-processing */
        if (need_pp) {
                struct ia_css_binary_descr capture_pp_descr;
 
-               ia_css_pipe_get_capturepp_binarydesc(pipe,
-                                                       &capture_pp_descr, &post_out_info, pipe_out_info, &vf_info);
+               ia_css_pipe_get_capturepp_binarydesc(pipe, &capture_pp_descr,
+                                                    &post_out_info,
+                                                    pipe_out_info, &vf_info);
                err = ia_css_binary_find(&capture_pp_descr,
-                                           &pipe->pipe_settings.capture.capture_pp_binary);
+                                        &pipe->pipe_settings.capture.capture_pp_binary);
                if (err)
                        return err;
        } else {
@@ -6245,10 +6244,11 @@ static int load_advanced_binaries(
        {
                struct ia_css_binary_descr post_gdc_descr;
 
-               ia_css_pipe_get_post_gdc_binarydesc(pipe,
-                                                   &post_gdc_descr, &post_in_info, &post_out_info, &vf_info);
+               ia_css_pipe_get_post_gdc_binarydesc(pipe, &post_gdc_descr,
+                                                   &post_in_info,
+                                                   &post_out_info, &vf_info);
                err = ia_css_binary_find(&post_gdc_descr,
-                                           &pipe->pipe_settings.capture.post_isp_binary);
+                                        &pipe->pipe_settings.capture.post_isp_binary);
                if (err)
                        return err;
        }
@@ -6258,9 +6258,9 @@ static int load_advanced_binaries(
                struct ia_css_binary_descr gdc_descr;
 
                ia_css_pipe_get_gdc_binarydesc(pipe, &gdc_descr, &gdc_in_info,
-                                               &pipe->pipe_settings.capture.post_isp_binary.in_frame_info);
+                                              &pipe->pipe_settings.capture.post_isp_binary.in_frame_info);
                err = ia_css_binary_find(&gdc_descr,
-                                           &pipe->pipe_settings.capture.anr_gdc_binary);
+                                        &pipe->pipe_settings.capture.anr_gdc_binary);
                if (err)
                        return err;
        }
@@ -6272,9 +6272,9 @@ static int load_advanced_binaries(
                struct ia_css_binary_descr pre_gdc_descr;
 
                ia_css_pipe_get_pre_gdc_binarydesc(pipe, &pre_gdc_descr, &pre_in_info,
-                                                   &pipe->pipe_settings.capture.anr_gdc_binary.in_frame_info);
+                                                  &pipe->pipe_settings.capture.anr_gdc_binary.in_frame_info);
                err = ia_css_binary_find(&pre_gdc_descr,
-                                           &pipe->pipe_settings.capture.pre_isp_binary);
+                                        &pipe->pipe_settings.capture.pre_isp_binary);
                if (err)
                        return err;
        }
@@ -6296,7 +6296,7 @@ static int load_advanced_binaries(
                ia_css_pipe_get_vfpp_binarydesc(pipe,
                                                &vf_pp_descr, vf_pp_in_info, pipe_vf_out_info);
                err = ia_css_binary_find(&vf_pp_descr,
-                                           &pipe->pipe_settings.capture.vf_pp_binary);
+                                        &pipe->pipe_settings.capture.vf_pp_binary);
                if (err)
                        return err;
        }
@@ -6308,8 +6308,8 @@ static int load_advanced_binaries(
 #endif
        if (need_isp_copy)
                load_copy_binary(pipe,
-                                   &pipe->pipe_settings.capture.copy_binary,
-                                   &pipe->pipe_settings.capture.pre_isp_binary);
+                                &pipe->pipe_settings.capture.copy_binary,
+                                &pipe->pipe_settings.capture.pre_isp_binary);
 
        return err;
 }
@@ -6324,7 +6324,7 @@ static int load_bayer_isp_binaries(
        IA_CSS_ENTER_PRIVATE("");
        assert(pipe);
        assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE ||
-               pipe->mode == IA_CSS_PIPE_ID_COPY);
+              pipe->mode == IA_CSS_PIPE_ID_COPY);
        pipe_out_info = &pipe->output_info[0];
 
        if (pipe->pipe_settings.capture.pre_isp_binary.info)
@@ -6335,11 +6335,11 @@ static int load_bayer_isp_binaries(
                return err;
 
        ia_css_pipe_get_pre_de_binarydesc(pipe, &pre_de_descr,
-                                           &pre_isp_in_info,
-                                           pipe_out_info);
+                                         &pre_isp_in_info,
+                                         pipe_out_info);
 
        err = ia_css_binary_find(&pre_de_descr,
-                                   &pipe->pipe_settings.capture.pre_isp_binary);
+                                &pipe->pipe_settings.capture.pre_isp_binary);
 
        return err;
 }
@@ -6358,7 +6358,7 @@ static int load_low_light_binaries(
        IA_CSS_ENTER_PRIVATE("");
        assert(pipe);
        assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE ||
-               pipe->mode == IA_CSS_PIPE_ID_COPY);
+              pipe->mode == IA_CSS_PIPE_ID_COPY);
 
        if (pipe->pipe_settings.capture.pre_isp_binary.info)
                return 0;
@@ -6373,17 +6373,18 @@ static int load_low_light_binaries(
        need_pp = need_capture_pp(pipe);
 
        ia_css_frame_info_set_format(&vf_info,
-                                       IA_CSS_FRAME_FORMAT_YUV_LINE);
+                                    IA_CSS_FRAME_FORMAT_YUV_LINE);
 
        /* we build up the pipeline starting at the end */
        /* Capture post-processing */
        if (need_pp) {
                struct ia_css_binary_descr capture_pp_descr;
 
-               ia_css_pipe_get_capturepp_binarydesc(pipe,
-                                                       &capture_pp_descr, &post_out_info, pipe_out_info, &vf_info);
+               ia_css_pipe_get_capturepp_binarydesc(pipe, &capture_pp_descr,
+                                                    &post_out_info,
+                                                    pipe_out_info, &vf_info);
                err = ia_css_binary_find(&capture_pp_descr,
-                                           &pipe->pipe_settings.capture.capture_pp_binary);
+                                        &pipe->pipe_settings.capture.capture_pp_binary);
                if (err)
                        return err;
        } else {
@@ -6397,7 +6398,7 @@ static int load_low_light_binaries(
                ia_css_pipe_get_post_anr_binarydesc(pipe,
                                                    &post_anr_descr, &post_in_info, &post_out_info, &vf_info);
                err = ia_css_binary_find(&post_anr_descr,
-                                           &pipe->pipe_settings.capture.post_isp_binary);
+                                        &pipe->pipe_settings.capture.post_isp_binary);
                if (err)
                        return err;
        }
@@ -6407,9 +6408,9 @@ static int load_low_light_binaries(
                struct ia_css_binary_descr anr_descr;
 
                ia_css_pipe_get_anr_binarydesc(pipe, &anr_descr, &anr_in_info,
-                                               &pipe->pipe_settings.capture.post_isp_binary.in_frame_info);
+                                              &pipe->pipe_settings.capture.post_isp_binary.in_frame_info);
                err = ia_css_binary_find(&anr_descr,
-                                           &pipe->pipe_settings.capture.anr_gdc_binary);
+                                        &pipe->pipe_settings.capture.anr_gdc_binary);
                if (err)
                        return err;
        }
@@ -6421,9 +6422,9 @@ static int load_low_light_binaries(
                struct ia_css_binary_descr pre_anr_descr;
 
                ia_css_pipe_get_pre_anr_binarydesc(pipe, &pre_anr_descr, &pre_in_info,
-                                                   &pipe->pipe_settings.capture.anr_gdc_binary.in_frame_info);
+                                                  &pipe->pipe_settings.capture.anr_gdc_binary.in_frame_info);
                err = ia_css_binary_find(&pre_anr_descr,
-                                           &pipe->pipe_settings.capture.pre_isp_binary);
+                                        &pipe->pipe_settings.capture.pre_isp_binary);
                if (err)
                        return err;
        }
@@ -6442,10 +6443,10 @@ static int load_low_light_binaries(
        {
                struct ia_css_binary_descr vf_pp_descr;
 
-               ia_css_pipe_get_vfpp_binarydesc(pipe,
-                                               &vf_pp_descr, vf_pp_in_info, pipe_vf_out_info);
+               ia_css_pipe_get_vfpp_binarydesc(pipe, &vf_pp_descr,
+                                               vf_pp_in_info, pipe_vf_out_info);
                err = ia_css_binary_find(&vf_pp_descr,
-                                           &pipe->pipe_settings.capture.vf_pp_binary);
+                                        &pipe->pipe_settings.capture.vf_pp_binary);
                if (err)
                        return err;
        }
@@ -6457,8 +6458,8 @@ static int load_low_light_binaries(
 #endif
        if (need_isp_copy)
                err = load_copy_binary(pipe,
-                                       &pipe->pipe_settings.capture.copy_binary,
-                                       &pipe->pipe_settings.capture.pre_isp_binary);
+                                      &pipe->pipe_settings.capture.copy_binary,
+                                      &pipe->pipe_settings.capture.pre_isp_binary);
 
        return err;
 }
@@ -6492,7 +6493,7 @@ static int load_capture_binaries(
        IA_CSS_ENTER_PRIVATE("");
        assert(pipe);
        assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE ||
-               pipe->mode == IA_CSS_PIPE_ID_COPY);
+              pipe->mode == IA_CSS_PIPE_ID_COPY);
 
        if (pipe->pipe_settings.capture.primary_binary[0].info) {
                IA_CSS_LEAVE_ERR_PRIVATE(0);
@@ -6587,7 +6588,7 @@ unload_capture_binaries(struct ia_css_pipe *pipe)
 
 static bool
 need_downscaling(const struct ia_css_resolution in_res,
-                   const struct ia_css_resolution out_res)
+                const struct ia_css_resolution out_res)
 {
        if (in_res.width > out_res.width || in_res.height > out_res.height)
                return true;
@@ -6658,9 +6659,9 @@ static int ia_css_pipe_create_cas_scaler_desc_single_output(
        descr->num_output_stage = 1;
 
        hor_ds_factor = CEIL_DIV(cas_scaler_in_info->res.width,
-                                   cas_scaler_out_info->res.width);
+                                cas_scaler_out_info->res.width);
        ver_ds_factor = CEIL_DIV(cas_scaler_in_info->res.height,
-                                   cas_scaler_out_info->res.height);
+                                cas_scaler_out_info->res.height);
        /* use the same horizontal and vertical downscaling factor for simplicity */
        assert(hor_ds_factor == ver_ds_factor);
 
@@ -6670,31 +6671,36 @@ static int ia_css_pipe_create_cas_scaler_desc_single_output(
                i *= max_scale_factor_per_stage;
        }
 
-       descr->in_info = kmalloc(descr->num_stage * sizeof(struct ia_css_frame_info),
-                                   GFP_KERNEL);
+       descr->in_info = kmalloc(descr->num_stage *
+                                sizeof(struct ia_css_frame_info),
+                                GFP_KERNEL);
        if (!descr->in_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->internal_out_info = kmalloc(descr->num_stage * sizeof(
-                                               struct ia_css_frame_info), GFP_KERNEL);
+       descr->internal_out_info = kmalloc(descr->num_stage *
+                                          sizeof(struct ia_css_frame_info),
+                                          GFP_KERNEL);
        if (!descr->internal_out_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->out_info = kmalloc(descr->num_stage * sizeof(struct ia_css_frame_info),
-                                   GFP_KERNEL);
+       descr->out_info = kmalloc(descr->num_stage *
+                                 sizeof(struct ia_css_frame_info),
+                                 GFP_KERNEL);
        if (!descr->out_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->vf_info = kmalloc(descr->num_stage * sizeof(struct ia_css_frame_info),
-                                   GFP_KERNEL);
+       descr->vf_info = kmalloc(descr->num_stage *
+                                sizeof(struct ia_css_frame_info),
+                                GFP_KERNEL);
        if (!descr->vf_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->is_output_stage = kmalloc(descr->num_stage * sizeof(bool), GFP_KERNEL);
+       descr->is_output_stage = kmalloc(descr->num_stage * sizeof(bool),
+                                        GFP_KERNEL);
        if (!descr->is_output_stage) {
                err = -ENOMEM;
                goto ERR;
@@ -6738,9 +6744,9 @@ static int ia_css_pipe_create_cas_scaler_desc_single_output(
                                max_scale_factor_per_stage;
                        descr->internal_out_info[i].format = IA_CSS_FRAME_FORMAT_YUV420;
                        ia_css_frame_info_init(&descr->internal_out_info[i],
-                                               tmp_in_info.res.width / max_scale_factor_per_stage,
-                                               tmp_in_info.res.height / max_scale_factor_per_stage,
-                                               IA_CSS_FRAME_FORMAT_YUV420, 0);
+                                              tmp_in_info.res.width / max_scale_factor_per_stage,
+                                              tmp_in_info.res.height / max_scale_factor_per_stage,
+                                              IA_CSS_FRAME_FORMAT_YUV420, 0);
                        descr->out_info[i].res.width = 0;
                        descr->out_info[i].res.height = 0;
                        descr->vf_info[i].res.width = 0;
@@ -6816,30 +6822,35 @@ static int ia_css_pipe_create_cas_scaler_desc(
        descr->num_stage = num_stages;
 
        descr->in_info = kmalloc_array(descr->num_stage,
-                                       sizeof(struct ia_css_frame_info), GFP_KERNEL);
+                                      sizeof(struct ia_css_frame_info),
+                                      GFP_KERNEL);
        if (!descr->in_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->internal_out_info = kmalloc(descr->num_stage * sizeof(
-                                               struct ia_css_frame_info), GFP_KERNEL);
+       descr->internal_out_info = kmalloc(descr->num_stage *
+                                          sizeof(struct ia_css_frame_info),
+                                          GFP_KERNEL);
        if (!descr->internal_out_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->out_info = kmalloc(descr->num_stage * sizeof(struct ia_css_frame_info),
-                                   GFP_KERNEL);
+       descr->out_info = kmalloc(descr->num_stage *
+                                 sizeof(struct ia_css_frame_info),
+                                 GFP_KERNEL);
        if (!descr->out_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->vf_info = kmalloc(descr->num_stage * sizeof(struct ia_css_frame_info),
-                                   GFP_KERNEL);
+       descr->vf_info = kmalloc(descr->num_stage *
+                                sizeof(struct ia_css_frame_info),
+                                GFP_KERNEL);
        if (!descr->vf_info) {
                err = -ENOMEM;
                goto ERR;
        }
-       descr->is_output_stage = kmalloc(descr->num_stage * sizeof(bool), GFP_KERNEL);
+       descr->is_output_stage = kmalloc(descr->num_stage * sizeof(bool),
+                                        GFP_KERNEL);
        if (!descr->is_output_stage) {
                err = -ENOMEM;
                goto ERR;
@@ -6849,7 +6860,7 @@ static int ia_css_pipe_create_cas_scaler_desc(
                if (out_info[i]) {
                        if (i > 0) {
                                assert((out_info[i - 1]->res.width >= out_info[i]->res.width) &&
-                                       (out_info[i - 1]->res.height >= out_info[i]->res.height));
+                                      (out_info[i - 1]->res.height >= out_info[i]->res.height));
                        }
                }
        }
@@ -6897,9 +6908,9 @@ static int ia_css_pipe_create_cas_scaler_desc(
                                max_scale_factor_per_stage;
                        descr->internal_out_info[i].format = IA_CSS_FRAME_FORMAT_YUV420;
                        ia_css_frame_info_init(&descr->internal_out_info[i],
-                                               tmp_in_info.res.width / max_scale_factor_per_stage,
-                                               tmp_in_info.res.height / max_scale_factor_per_stage,
-                                               IA_CSS_FRAME_FORMAT_YUV420, 0);
+                                              tmp_in_info.res.width / max_scale_factor_per_stage,
+                                              tmp_in_info.res.height / max_scale_factor_per_stage,
+                                              IA_CSS_FRAME_FORMAT_YUV420, 0);
                        descr->out_info[i].res.width = 0;
                        descr->out_info[i].res.height = 0;
                        descr->vf_info[i].res.width = 0;
@@ -6978,13 +6989,14 @@ load_yuvpp_binaries(struct ia_css_pipe *pipe)
                struct ia_css_binary_descr yuv_scaler_descr;
 
                err = ia_css_pipe_create_cas_scaler_desc(pipe,
-                       &cas_scaler_descr);
+                                                        &cas_scaler_descr);
                if (err)
                        goto ERR;
                mycs->num_output = cas_scaler_descr.num_output_stage;
                mycs->num_yuv_scaler = cas_scaler_descr.num_stage;
                mycs->yuv_scaler_binary = kzalloc(cas_scaler_descr.num_stage *
-                                                   sizeof(struct ia_css_binary), GFP_KERNEL);
+                                                 sizeof(struct ia_css_binary),
+                                                 GFP_KERNEL);
                if (!mycs->yuv_scaler_binary) {
                        err = -ENOMEM;
                        goto ERR;
@@ -6998,12 +7010,13 @@ load_yuvpp_binaries(struct ia_css_pipe *pipe)
                for (i = 0; i < cas_scaler_descr.num_stage; i++) {
                        mycs->is_output_stage[i] = cas_scaler_descr.is_output_stage[i];
                        ia_css_pipe_get_yuvscaler_binarydesc(pipe,
-                                                               &yuv_scaler_descr, &cas_scaler_descr.in_info[i],
-                                                               &cas_scaler_descr.out_info[i],
-                                                               &cas_scaler_descr.internal_out_info[i],
-                                                               &cas_scaler_descr.vf_info[i]);
+                                                            &yuv_scaler_descr,
+                                                            &cas_scaler_descr.in_info[i],
+                                                            &cas_scaler_descr.out_info[i],
+                                                            &cas_scaler_descr.internal_out_info[i],
+                                                            &cas_scaler_descr.vf_info[i]);
                        err = ia_css_binary_find(&yuv_scaler_descr,
-                                                   &mycs->yuv_scaler_binary[i]);
+                                                &mycs->yuv_scaler_binary[i]);
                        if (err)
                                goto ERR;
                }
@@ -7043,8 +7056,8 @@ load_yuvpp_binaries(struct ia_css_pipe *pipe)
 
        if (need_isp_copy_binary) {
                err = load_copy_binary(pipe,
-                                       &mycs->copy_binary,
-                                       next_binary);
+                                      &mycs->copy_binary,
+                                      next_binary);
 
                if (err)
                        goto ERR;
@@ -7089,8 +7102,9 @@ load_yuvpp_binaries(struct ia_css_pipe *pipe)
 
                mycs->num_vf_pp = 1;
        }
-       mycs->vf_pp_binary = kzalloc(mycs->num_vf_pp * sizeof(struct ia_css_binary),
-                                       GFP_KERNEL);
+       mycs->vf_pp_binary = kzalloc(mycs->num_vf_pp *
+                                    sizeof(struct ia_css_binary),
+                                    GFP_KERNEL);
        if (!mycs->vf_pp_binary) {
                err = -ENOMEM;
                goto ERR;
@@ -7419,18 +7433,26 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
 #endif
 
                if (need_scaler) {
-                       ia_css_pipe_util_set_output_frames(bin_out_frame, 0, NULL);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                           bin_out_frame, in_frame_local, NULL);
+                       ia_css_pipe_util_set_output_frames(bin_out_frame,
+                                                          0, NULL);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          copy_binary,
+                                                          bin_out_frame,
+                                                          in_frame_local,
+                                                          NULL);
                } else {
-                       ia_css_pipe_util_set_output_frames(bin_out_frame, 0, out_frame[0]);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                           bin_out_frame, in_frame_local, NULL);
+                       ia_css_pipe_util_set_output_frames(bin_out_frame,
+                                                          0, out_frame[0]);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          copy_binary,
+                                                          bin_out_frame,
+                                                          in_frame_local,
+                                                          NULL);
                }
 
                err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       &copy_stage);
+                                                          &stage_desc,
+                                                          &copy_stage);
 
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
@@ -7462,10 +7484,11 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
                                tmp_vf_frame = NULL;
                        }
 
-                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame, tmp_out_frame,
-                                                   NULL,
-                                                   &yuv_scaler_binary[i],
-                                                   &yuv_scaler_stage);
+                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame,
+                                                  tmp_out_frame,
+                                                  NULL,
+                                                  &yuv_scaler_binary[i],
+                                                  &yuv_scaler_stage);
 
                        if (err) {
                                IA_CSS_LEAVE_ERR_PRIVATE(err);
@@ -7476,8 +7499,10 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
                        if (pipe->pipe_settings.yuvpp.is_output_stage[i]) {
                                if (tmp_vf_frame && (tmp_vf_frame->info.res.width != 0)) {
                                        in_frame = yuv_scaler_stage->args.out_vf_frame;
-                                       err = add_vf_pp_stage(pipe, in_frame, tmp_vf_frame, &vf_pp_binary[j],
-                                                               &vf_pp_stage);
+                                       err = add_vf_pp_stage(pipe, in_frame,
+                                                             tmp_vf_frame,
+                                                             &vf_pp_binary[j],
+                                                             &vf_pp_stage);
 
                                        if (err) {
                                                IA_CSS_LEAVE_ERR_PRIVATE(err);
@@ -7490,8 +7515,8 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
        } else if (copy_stage) {
                if (vf_frame[0] && vf_frame[0]->info.res.width != 0) {
                        in_frame = copy_stage->args.out_vf_frame;
-                       err = add_vf_pp_stage(pipe, in_frame, vf_frame[0], &vf_pp_binary[0],
-                                               &vf_pp_stage);
+                       err = add_vf_pp_stage(pipe, in_frame, vf_frame[0],
+                                             &vf_pp_binary[0], &vf_pp_stage);
                }
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
@@ -7499,7 +7524,8 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
                }
        }
 
-       ia_css_pipeline_finalize_stages(&pipe->pipeline, pipe->stream->config.continuous);
+       ia_css_pipeline_finalize_stages(&pipe->pipeline,
+                                       pipe->stream->config.continuous);
 
        IA_CSS_LEAVE_ERR_PRIVATE(0);
 
@@ -7508,8 +7534,8 @@ create_host_yuvpp_pipeline(struct ia_css_pipe *pipe)
 
 static int
 create_host_copy_pipeline(struct ia_css_pipe *pipe,
-                           unsigned int max_input_width,
-                           struct ia_css_frame *out_frame)
+                         unsigned int max_input_width,
+                         struct ia_css_frame *out_frame)
 {
        struct ia_css_pipeline *me;
        int err = 0;
@@ -7528,12 +7554,8 @@ create_host_copy_pipeline(struct ia_css_pipe *pipe,
 
        if (copy_on_sp(pipe) &&
            pipe->stream->config.input_config.format == ATOMISP_INPUT_FORMAT_BINARY_8) {
-               ia_css_frame_info_init(
-                   &out_frame->info,
-                   JPEG_BYTES,
-                   1,
-                   IA_CSS_FRAME_FORMAT_BINARY_8,
-                   0);
+               ia_css_frame_info_init(&out_frame->info, JPEG_BYTES, 1,
+                                      IA_CSS_FRAME_FORMAT_BINARY_8, 0);
        } else if (out_frame->info.format == IA_CSS_FRAME_FORMAT_RAW) {
                out_frame->info.raw_bit_depth =
                ia_css_pipe_util_pipe_input_format_bpp(pipe);
@@ -7544,12 +7566,12 @@ create_host_copy_pipeline(struct ia_css_pipe *pipe,
        pipe->mode  = IA_CSS_PIPE_ID_COPY;
 
        ia_css_pipe_get_sp_func_stage_desc(&stage_desc, out_frame,
-                                           IA_CSS_PIPELINE_RAW_COPY, max_input_width);
-       err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc,
-               NULL);
+                                          IA_CSS_PIPELINE_RAW_COPY,
+                                          max_input_width);
+       err = ia_css_pipeline_create_and_add_stage(me, &stage_desc, NULL);
 
-       ia_css_pipeline_finalize_stages(&pipe->pipeline, pipe->stream->config.continuous);
+       ia_css_pipeline_finalize_stages(&pipe->pipeline,
+                                       pipe->stream->config.continuous);
 
        ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE_PRIVATE,
                            "create_host_copy_pipeline() leave:\n");
@@ -7588,9 +7610,10 @@ create_host_isyscopy_capture_pipeline(struct ia_css_pipe *pipe)
        me->pipe_id = IA_CSS_PIPE_ID_CAPTURE;
        pipe->mode  = IA_CSS_PIPE_ID_CAPTURE;
        ia_css_pipe_get_sp_func_stage_desc(&stage_desc, out_frame,
-                                           IA_CSS_PIPELINE_ISYS_COPY, max_input_width);
+                                          IA_CSS_PIPELINE_ISYS_COPY,
+                                          max_input_width);
        err = ia_css_pipeline_create_and_add_stage(me,
-               &stage_desc, &out_stage);
+                                                  &stage_desc, &out_stage);
        if (err)
                return err;
 
@@ -7642,7 +7665,8 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
        IA_CSS_ENTER_PRIVATE("");
        assert(pipe);
        assert(pipe->stream);
-       assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE || pipe->mode == IA_CSS_PIPE_ID_COPY);
+       assert(pipe->mode == IA_CSS_PIPE_ID_CAPTURE ||
+              pipe->mode == IA_CSS_PIPE_ID_COPY);
 
        me = &pipe->pipeline;
        mode = pipe->config.default_capture_config.mode;
@@ -7732,26 +7756,37 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
                        ia_css_pipe_util_set_output_frames(out_frames, 0, out_frame);
 #if defined(ISP2401)
                        if (!continuous) {
-                               ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                                   out_frames, in_frame, NULL);
+                               ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                                  copy_binary,
+                                                                  out_frames,
+                                                                  in_frame,
+                                                                  NULL);
                        } else {
                                in_frame = pipe->stream->last_pipe->continuous_frames[0];
-                               ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                                   out_frames, in_frame, NULL);
+                               ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                                  copy_binary,
+                                                                  out_frames,
+                                                                  in_frame,
+                                                                  NULL);
                        }
 #else
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                           out_frames, NULL, NULL);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          copy_binary,
+                                                          out_frames,
+                                                          NULL, NULL);
 #endif
                } else {
-                       ia_css_pipe_util_set_output_frames(out_frames, 0, in_frame);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, copy_binary,
-                                                           out_frames, NULL, NULL);
+                       ia_css_pipe_util_set_output_frames(out_frames, 0,
+                                                          in_frame);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          copy_binary,
+                                                          out_frames,
+                                                          NULL, NULL);
                }
 
                err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       &current_stage);
+                                                          &stage_desc,
+                                                          &current_stage);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7788,11 +7823,14 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
                            * Proper investigation should be done to come up with the clean
                            * solution.
                            * */
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, primary_binary[i],
-                                                           out_frames, local_in_frame, NULL);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          primary_binary[i],
+                                                          out_frames,
+                                                          local_in_frame,
+                                                          NULL);
                        err = ia_css_pipeline_create_and_add_stage(me,
-                               &stage_desc,
-                               &current_stage);
+                                                                  &stage_desc,
+                                                                  &current_stage);
                        if (err) {
                                IA_CSS_LEAVE_ERR_PRIVATE(err);
                                return err;
@@ -7808,18 +7846,18 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
                    mode == IA_CSS_CAPTURE_MODE_LOW_LIGHT) {
                ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, pre_isp_binary,
-                                                   out_frames, in_frame, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc, NULL);
+                                                  out_frames, in_frame, NULL);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          NULL);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
                }
                ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, anr_gdc_binary,
-                                                   out_frames, NULL, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc, NULL);
+                                                  out_frames, NULL, NULL);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          NULL);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7827,16 +7865,21 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
 
                if (need_pp) {
                        ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, post_isp_binary,
-                                                           out_frames, NULL, NULL);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          post_isp_binary,
+                                                          out_frames,
+                                                          NULL, NULL);
                } else {
-                       ia_css_pipe_util_set_output_frames(out_frames, 0, out_frame);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, post_isp_binary,
-                                                           out_frames, NULL, NULL);
+                       ia_css_pipe_util_set_output_frames(out_frames, 0,
+                                                          out_frame);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          post_isp_binary,
+                                                          out_frames,
+                                                          NULL, NULL);
                }
 
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc, &current_stage);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          &current_stage);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7844,10 +7887,9 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
        } else if (mode == IA_CSS_CAPTURE_MODE_BAYER) {
                ia_css_pipe_util_set_output_frames(out_frames, 0, out_frame);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, pre_isp_binary,
-                                                   out_frames, in_frame, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       NULL);
+                                                  out_frames, in_frame, NULL);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          NULL);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7862,31 +7904,34 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
 
                if (need_ldc) {
                        ia_css_pipe_util_set_output_frames(out_frames, 0, NULL);
-                       ia_css_pipe_get_generic_stage_desc(&stage_desc, capture_ldc_binary,
-                                                           out_frames, local_in_frame, NULL);
+                       ia_css_pipe_get_generic_stage_desc(&stage_desc,
+                                                          capture_ldc_binary,
+                                                          out_frames,
+                                                          local_in_frame,
+                                                          NULL);
                        err = ia_css_pipeline_create_and_add_stage(me,
-                               &stage_desc,
-                               &current_stage);
+                                                                  &stage_desc,
+                                                                  &current_stage);
                        local_in_frame = current_stage->args.out_frame[0];
                }
                err = add_capture_pp_stage(pipe, me, local_in_frame,
-                                           need_yuv_pp ? NULL : out_frame,
+                                          need_yuv_pp ? NULL : out_frame,
 #else
        /* ldc and capture_pp not supported in same pipeline */
        if (need_ldc && current_stage) {
                in_frame = current_stage->args.out_frame[0];
                ia_css_pipe_util_set_output_frames(out_frames, 0, out_frame);
                ia_css_pipe_get_generic_stage_desc(&stage_desc, capture_ldc_binary,
-                                                   out_frames, in_frame, NULL);
-               err = ia_css_pipeline_create_and_add_stage(me,
-                       &stage_desc,
-                       NULL);
+                                                  out_frames, in_frame, NULL);
+               err = ia_css_pipeline_create_and_add_stage(me, &stage_desc,
+                                                          NULL);
        } else if (need_pp && current_stage) {
                in_frame = current_stage->args.out_frame[0];
-               err = add_capture_pp_stage(pipe, me, in_frame, need_yuv_pp ? NULL : out_frame,
+               err = add_capture_pp_stage(pipe, me, in_frame,
+                                          need_yuv_pp ? NULL : out_frame,
 #endif
-                                           capture_pp_binary,
-                                           &current_stage);
+                                          capture_pp_binary,
+                                          &current_stage);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7903,10 +7948,10 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
                        else
                                tmp_out_frame = NULL;
 
-                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame, tmp_out_frame,
-                                                   NULL,
-                                                   &yuv_scaler_binary[i],
-                                                   &yuv_scaler_stage);
+                       err = add_yuv_scaler_stage(pipe, me, tmp_in_frame,
+                                                  tmp_out_frame, NULL,
+                                                  &yuv_scaler_binary[i],
+                                                  &yuv_scaler_stage);
                        if (err) {
                                IA_CSS_LEAVE_ERR_PRIVATE(err);
                                return err;
@@ -7928,7 +7973,7 @@ create_host_regular_capture_pipeline(struct ia_css_pipe *pipe)
        if (mode != IA_CSS_CAPTURE_MODE_RAW && mode != IA_CSS_CAPTURE_MODE_BAYER && current_stage && vf_frame) {
                in_frame = current_stage->args.out_vf_frame;
                err = add_vf_pp_stage(pipe, in_frame, vf_frame, vf_pp_binary,
-                                       &current_stage);
+                                     &current_stage);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        return err;
@@ -7980,7 +8025,7 @@ static int capture_start(
        me = &pipe->pipeline;
 
        if ((pipe->config.default_capture_config.mode == IA_CSS_CAPTURE_MODE_RAW   ||
-               pipe->config.default_capture_config.mode == IA_CSS_CAPTURE_MODE_BAYER) &&
+            pipe->config.default_capture_config.mode == IA_CSS_CAPTURE_MODE_BAYER) &&
            (pipe->config.mode != IA_CSS_PIPE_MODE_COPY)) {
                if (copy_on_sp(pipe)) {
                        err = start_copy_on_sp(pipe, &me->out_frame[0]);
@@ -8024,7 +8069,7 @@ static int capture_start(
        if (pipe->config.mode == IA_CSS_PIPE_MODE_COPY &&
            pipe->stream->reconfigure_css_rx) {
                ia_css_isys_rx_configure(&pipe->stream->csi_rx_config,
-                                           pipe->stream->config.mode);
+                                        pipe->stream->config.mode);
                pipe->stream->reconfigure_css_rx = false;
        }
 #endif
@@ -8035,8 +8080,8 @@ static int capture_start(
 
 static int
 sh_css_pipe_get_output_frame_info(struct ia_css_pipe *pipe,
-                                   struct ia_css_frame_info *info,
-                                   unsigned int idx)
+                                 struct ia_css_frame_info *info,
+                                 unsigned int idx)
 {
        assert(pipe);
        assert(info);
@@ -8054,7 +8099,7 @@ sh_css_pipe_get_output_frame_info(struct ia_css_pipe *pipe,
                    IA_CSS_FRAME_FORMAT_BINARY_8,
                    0);
        } else if (info->format == IA_CSS_FRAME_FORMAT_RAW ||
-                   info->format == IA_CSS_FRAME_FORMAT_RAW_PACKED) {
+                  info->format == IA_CSS_FRAME_FORMAT_RAW_PACKED) {
                info->raw_bit_depth =
                ia_css_pipe_util_pipe_input_format_bpp(pipe);
        }
@@ -8066,9 +8111,9 @@ sh_css_pipe_get_output_frame_info(struct ia_css_pipe *pipe,
 
 void
 ia_css_stream_send_input_frame(const struct ia_css_stream *stream,
-                               const unsigned short *data,
-                               unsigned int width,
-                               unsigned int height)
+                              const unsigned short *data,
+                              unsigned int width,
+                              unsigned int height)
 {
        assert(stream);
 
@@ -8092,22 +8137,22 @@ ia_css_stream_start_input_frame(const struct ia_css_stream *stream)
 
 void
 ia_css_stream_send_input_line(const struct ia_css_stream *stream,
-                               const unsigned short *data,
-                               unsigned int width,
-                               const unsigned short *data2,
-                               unsigned int width2)
+                             const unsigned short *data,
+                             unsigned int width,
+                             const unsigned short *data2,
+                             unsigned int width2)
 {
        assert(stream);
 
        ia_css_inputfifo_send_line(stream->config.channel_id,
-                                   data, width, data2, width2);
+                                  data, width, data2, width2);
 }
 
 void
 ia_css_stream_send_input_embedded_line(const struct ia_css_stream *stream,
-                                       enum atomisp_input_format format,
-                                       const unsigned short *data,
-                                       unsigned int width)
+                                      enum atomisp_input_format format,
+                                      const unsigned short *data,
+                                      unsigned int width)
 {
        assert(stream);
        if (!data || width == 0)
@@ -8228,7 +8273,7 @@ acc_unload_extension(struct ia_css_fw_info *firmware)
 /* Load firmware for extension */
 static int
 ia_css_pipe_load_extension(struct ia_css_pipe *pipe,
-                           struct ia_css_fw_info *firmware)
+                          struct ia_css_fw_info *firmware)
 {
        int err = 0;
 
@@ -8252,7 +8297,7 @@ ia_css_pipe_load_extension(struct ia_css_pipe *pipe,
 /* Unload firmware for extension */
 static void
 ia_css_pipe_unload_extension(struct ia_css_pipe *pipe,
-                               struct ia_css_fw_info *firmware)
+                            struct ia_css_fw_info *firmware)
 {
        IA_CSS_ENTER_PRIVATE("fw = %p pipe = %p", firmware, pipe);
 
@@ -8294,7 +8339,7 @@ ia_css_pipeline_uses_params(struct ia_css_pipeline *me)
 
 static int
 sh_css_pipeline_add_acc_stage(struct ia_css_pipeline *pipeline,
-                               const void *acc_fw)
+                             const void *acc_fw)
 {
        struct ia_css_fw_info *fw = (struct ia_css_fw_info *)acc_fw;
        /* In QoS case, load_extension already called, so skipping */
@@ -8312,8 +8357,8 @@ sh_css_pipeline_add_acc_stage(struct ia_css_pipeline *pipeline,
 
                ia_css_pipe_get_acc_stage_desc(&stage_desc, NULL, fw);
                err = ia_css_pipeline_create_and_add_stage(pipeline,
-                       &stage_desc,
-                       NULL);
+                                                          &stage_desc,
+                                                          NULL);
        }
 
        ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE,
@@ -8326,7 +8371,7 @@ sh_css_pipeline_add_acc_stage(struct ia_css_pipeline *pipeline,
     * Refer to "sh_css_internal.h" for details.
     */
 int ia_css_stream_capture_frame(struct ia_css_stream *stream,
-       unsigned int exp_id)
+                               unsigned int exp_id)
 {
        struct sh_css_tag_descr tag_descr;
        u32 encoded_tag_descr;
@@ -8485,22 +8530,22 @@ sh_css_init_host_sp_control_vars(void)
        (void)HIVE_ADDR_host_sp_com;
 
        sp_dmem_store_uint32(SP0_ID,
-                               (unsigned int)sp_address_of(ia_css_ispctrl_sp_isp_started),
-                               (uint32_t)(0));
+                            (unsigned int)sp_address_of(ia_css_ispctrl_sp_isp_started),
+                            (uint32_t)(0));
 
        sp_dmem_store_uint32(SP0_ID,
-                               (unsigned int)sp_address_of(host_sp_queues_initialized),
-                               (uint32_t)(0));
+                            (unsigned int)sp_address_of(host_sp_queues_initialized),
+                            (uint32_t)(0));
        sp_dmem_store_uint32(SP0_ID,
-                               (unsigned int)sp_address_of(sp_sleep_mode),
-                               (uint32_t)(0));
+                            (unsigned int)sp_address_of(sp_sleep_mode),
+                            (uint32_t)(0));
        sp_dmem_store_uint32(SP0_ID,
-                               (unsigned int)sp_address_of(ia_css_dmaproxy_sp_invalidate_tlb),
-                               (uint32_t)(false));
+                            (unsigned int)sp_address_of(ia_css_dmaproxy_sp_invalidate_tlb),
+                            (uint32_t)(false));
 #ifndef ISP2401
        sp_dmem_store_uint32(SP0_ID,
-                               (unsigned int)sp_address_of(sp_stop_copy_preview),
-                               my_css.stop_copy_preview ? (uint32_t)(1) : (uint32_t)(0));
+                            (unsigned int)sp_address_of(sp_stop_copy_preview),
+                            my_css.stop_copy_preview ? (uint32_t)(1) : (uint32_t)(0));
 #endif
        store_sp_array_uint(host_sp_com, o, host2sp_cmd_ready);
 
@@ -8527,8 +8572,7 @@ void ia_css_pipe_config_defaults(struct ia_css_pipe_config *pipe_config)
 }
 
 void
-ia_css_pipe_extra_config_defaults(struct ia_css_pipe_extra_config
-                                   *extra_config)
+ia_css_pipe_extra_config_defaults(struct ia_css_pipe_extra_config *extra_config)
 {
        if (!extra_config) {
                IA_CSS_ERROR("NULL input parameter");
@@ -8602,8 +8646,8 @@ int ia_css_pipe_create(const struct ia_css_pipe_config *config,
 
 int
 ia_css_pipe_create_extra(const struct ia_css_pipe_config *config,
-                           const struct ia_css_pipe_extra_config *extra_config,
-                           struct ia_css_pipe **pipe)
+                        const struct ia_css_pipe_extra_config *extra_config,
+                        struct ia_css_pipe **pipe)
 {
        int err = -EINVAL;
        struct ia_css_pipe *internal_pipe = NULL;
@@ -8671,7 +8715,7 @@ ia_css_pipe_create_extra(const struct ia_css_pipe_config *config,
 
        /* YUV downscaling */
        if ((internal_pipe->config.vf_pp_in_res.width ||
-               internal_pipe->config.capt_pp_in_res.width)) {
+            internal_pipe->config.capt_pp_in_res.width)) {
                enum ia_css_frame_format format;
 
                if (internal_pipe->config.vf_pp_in_res.width) {
@@ -8747,7 +8791,7 @@ ia_css_pipe_create_extra(const struct ia_css_pipe_config *config,
        }
        if (internal_pipe->config.acc_extension) {
                err = ia_css_pipe_load_extension(internal_pipe,
-                                                   internal_pipe->config.acc_extension);
+                                                internal_pipe->config.acc_extension);
                if (err) {
                        IA_CSS_LEAVE_ERR_PRIVATE(err);
                        kvfree(internal_pipe);
@@ -8765,7 +8809,7 @@ ia_css_pipe_create_extra(const struct ia_css_pipe_config *config,
 
 int
 ia_css_pipe_get_info(const struct ia_css_pipe *pipe,
-                       struct ia_css_pipe_info *pipe_info)
+                    struct ia_css_pipe_info *pipe_info)
 {
        ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE,
                            "ia_css_pipe_get_info()\n");
@@ -8801,8 +8845,8 @@ bool ia_css_pipe_has_dvs_stats(struct ia_css_pipe_info *pipe_info)
 
 int
 ia_css_pipe_override_frame_format(struct ia_css_pipe *pipe,
-                                   int pin_index,
-                                   enum ia_css_frame_format new_format)
+                                 int pin_index,
+                                 enum ia_css_frame_format new_format)
 {
        int err = 0;
 
@@ -8882,10 +8926,8 @@ ia_css_stream_configure_rx(struct ia_css_stream *stream)
 #endif
 
 static struct ia_css_pipe *
-find_pipe(struct ia_css_pipe *pipes[],
-           unsigned int num_pipes,
-           enum ia_css_pipe_mode mode,
-           bool copy_pipe)
+find_pipe(struct ia_css_pipe *pipes[], unsigned int num_pipes,
+         enum ia_css_pipe_mode mode, bool copy_pipe)
 {
        unsigned int i;
 
@@ -8954,7 +8996,7 @@ ia_css_acc_stream_create(struct ia_css_stream *stream)
 
 static int
 metadata_info_init(const struct ia_css_metadata_config *mdc,
-                   struct ia_css_metadata_info *md)
+                  struct ia_css_metadata_info *md)
 {
        /* Either both width and height should be set or neither */
        if ((mdc->resolution.height > 0) ^ (mdc->resolution.width > 0))
@@ -8982,7 +9024,7 @@ static int check_pipe_resolutions(const struct ia_css_pipe *pipe)
        }
 
        if (ia_css_util_check_res(pipe->config.input_effective_res.width,
-                                   pipe->config.input_effective_res.height) != 0) {
+                                 pipe->config.input_effective_res.height) != 0) {
                IA_CSS_ERROR("effective resolution not supported");
                err = -EINVAL;
                goto EXIT;
@@ -8990,7 +9032,7 @@ static int check_pipe_resolutions(const struct ia_css_pipe *pipe)
        if (!ia_css_util_resolution_is_zero(
                pipe->stream->config.input_config.input_res)) {
                if (!ia_css_util_res_leq(pipe->config.input_effective_res,
-                                           pipe->stream->config.input_config.input_res)) {
+                                        pipe->stream->config.input_config.input_res)) {
                        IA_CSS_ERROR("effective resolution is larger than input resolution");
                        err = -EINVAL;
                        goto EXIT;
@@ -9013,9 +9055,9 @@ EXIT:
 
 int
 ia_css_stream_create(const struct ia_css_stream_config *stream_config,
-                       int num_pipes,
-                       struct ia_css_pipe *pipes[],
-                       struct ia_css_stream **stream)
+                    int num_pipes,
+                    struct ia_css_pipe *pipes[],
+                    struct ia_css_stream **stream)
 {
        struct ia_css_pipe *curr_pipe;
        struct ia_css_stream *curr_stream = NULL;
@@ -9176,11 +9218,11 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
        case IA_CSS_INPUT_MODE_TPG:
 #if !defined(ISP2401)
                IA_CSS_LOG("tpg_configuration: x_mask=%d, y_mask=%d, x_delta=%d, y_delta=%d, xy_mask=%d",
-                           curr_stream->config.source.tpg.x_mask,
-                           curr_stream->config.source.tpg.y_mask,
-                           curr_stream->config.source.tpg.x_delta,
-                           curr_stream->config.source.tpg.y_delta,
-                           curr_stream->config.source.tpg.xy_mask);
+                          curr_stream->config.source.tpg.x_mask,
+                          curr_stream->config.source.tpg.y_mask,
+                          curr_stream->config.source.tpg.x_delta,
+                          curr_stream->config.source.tpg.y_delta,
+                          curr_stream->config.source.tpg.xy_mask);
 
                sh_css_sp_configure_tpg(
                    curr_stream->config.source.tpg.x_mask,
@@ -9205,9 +9247,8 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
        }
 
 #ifdef ISP2401
-       err = aspect_ratio_crop_init(curr_stream,
-                                       pipes,
-                                       &aspect_ratio_crop_enabled);
+       err = aspect_ratio_crop_init(curr_stream, pipes,
+                                    &aspect_ratio_crop_enabled);
        if (err) {
                IA_CSS_LEAVE_ERR(err);
                goto ERR;
@@ -9244,8 +9285,8 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
                        curr_pipe->config.input_effective_res = effective_res;
                }
                IA_CSS_LOG("effective_res=%dx%d",
-                           effective_res.width,
-                           effective_res.height);
+                          effective_res.width,
+                          effective_res.height);
        }
 
        if (IS_ISP2401) {
@@ -9273,13 +9314,13 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
        if (!spcopyonly) {
                sensor_binning_changed =
                    sh_css_params_set_binning_factor(curr_stream,
-                                                       curr_stream->config.sensor_binning_factor);
+                                                    curr_stream->config.sensor_binning_factor);
        } else {
                sensor_binning_changed = false;
        }
 
        IA_CSS_LOG("sensor_binning=%d, changed=%d",
-                   curr_stream->config.sensor_binning_factor, sensor_binning_changed);
+                  curr_stream->config.sensor_binning_factor, sensor_binning_changed);
        /* loop over pipes */
        IA_CSS_LOG("num_pipes=%d", num_pipes);
        curr_stream->cont_capt = false;
@@ -9303,17 +9344,18 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
 
                /* Create copy pipe here, since it may not be exposed to the driver */
                preview_pipe = find_pipe(pipes, num_pipes,
-                                           IA_CSS_PIPE_MODE_PREVIEW, false);
+                                        IA_CSS_PIPE_MODE_PREVIEW, false);
                video_pipe = find_pipe(pipes, num_pipes,
-                                       IA_CSS_PIPE_MODE_VIDEO, false);
-               acc_pipe = find_pipe(pipes, num_pipes,
-                                       IA_CSS_PIPE_MODE_ACC, false);
+                                      IA_CSS_PIPE_MODE_VIDEO, false);
+               acc_pipe = find_pipe(pipes, num_pipes, IA_CSS_PIPE_MODE_ACC,
+                                    false);
                if (acc_pipe && num_pipes == 2 && curr_stream->cont_capt)
                        curr_stream->cont_capt =
                            false; /* preview + QoS case will not need cont_capt switch */
                if (curr_stream->cont_capt) {
                        capture_pipe = find_pipe(pipes, num_pipes,
-                                                   IA_CSS_PIPE_MODE_CAPTURE, false);
+                                                IA_CSS_PIPE_MODE_CAPTURE,
+                                                false);
                        if (!capture_pipe) {
                                err = -EINVAL;
                                goto ERR;
@@ -9395,10 +9437,12 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
                if (!spcopyonly) {
                        if (!IS_ISP2401)
                                err = sh_css_pipe_get_shading_info(curr_pipe,
-                                                                   &pipe_info->shading_info, NULL);
+                                                                  &pipe_info->shading_info,
+                                                                  NULL);
                        else
                                err = sh_css_pipe_get_shading_info(curr_pipe,
-                                                                   &pipe_info->shading_info, &curr_pipe->config);
+                                                                  &pipe_info->shading_info,
+                                                                  &curr_pipe->config);
 
                        if (err)
                                goto ERR;
@@ -9408,7 +9452,8 @@ ia_css_stream_create(const struct ia_css_stream_config *stream_config,
                                goto ERR;
                        for (j = 0; j < IA_CSS_PIPE_MAX_OUTPUT_STAGE; j++) {
                                sh_css_pipe_get_viewfinder_frame_info(curr_pipe,
-                                                                       &pipe_info->vf_output_info[j], j);
+                                                                     &pipe_info->vf_output_info[j],
+                                                                     j);
                                if (err)
                                        goto ERR;
                        }
@@ -9593,7 +9638,7 @@ ia_css_stream_destroy(struct ia_css_stream *stream)
 
 int
 ia_css_stream_get_info(const struct ia_css_stream *stream,
-                       struct ia_css_stream_info *stream_info)
+                      struct ia_css_stream_info *stream_info)
 {
        ia_css_debug_dtrace(IA_CSS_DEBUG_TRACE, "ia_css_stream_get_info: enter/exit\n");
        assert(stream);
@@ -9881,7 +9926,7 @@ ia_css_stream_get_3a_binary(const struct ia_css_stream *stream)
 
 int
 ia_css_stream_set_output_padded_width(struct ia_css_stream *stream,
-                                       unsigned int output_padded_width)
+                                     unsigned int output_padded_width)
 {
        struct ia_css_pipe *pipe;
 
@@ -10358,7 +10403,7 @@ ia_css_unlock_raw_frame(struct ia_css_stream *stream, uint32_t exp_id)
     */
 int
 ia_css_pipe_set_qos_ext_state(struct ia_css_pipe *pipe, uint32_t fw_handle,
-                               bool enable)
+                             bool enable)
 {
        unsigned int thread_id;
        struct ia_css_pipeline_stage *stage;
@@ -10404,7 +10449,7 @@ ia_css_pipe_set_qos_ext_state(struct ia_css_pipe *pipe, uint32_t fw_handle,
     */
 int
 ia_css_pipe_get_qos_ext_state(struct ia_css_pipe *pipe, uint32_t fw_handle,
-                               bool *enable)
+                             bool *enable)
 {
        struct ia_css_pipeline_stage *stage;
        unsigned int thread_id;
@@ -10440,9 +10485,9 @@ ia_css_pipe_get_qos_ext_state(struct ia_css_pipe *pipe, uint32_t fw_handle,
 /* ISP2401 */
 int
 ia_css_pipe_update_qos_ext_mapped_arg(struct ia_css_pipe *pipe,
-                                       u32 fw_handle,
-                                       struct ia_css_isp_param_css_segments *css_seg,
-                                       struct ia_css_isp_param_isp_segments *isp_seg)
+                                     u32 fw_handle,
+                                     struct ia_css_isp_param_css_segments *css_seg,
+                                     struct ia_css_isp_param_isp_segments *isp_seg)
 {
        unsigned int HIVE_ADDR_sp_group;
        static struct sh_css_sp_group sp_group;
@@ -10477,7 +10522,7 @@ ia_css_pipe_update_qos_ext_mapped_arg(struct ia_css_pipe *pipe,
                if (!err) {
                        /* Get the Extension State */
                        enabled = (SH_CSS_QOS_STAGE_IS_ENABLED(&sh_css_sp_group.pipe[thread_id],
-                                                               stage->stage_num)) ? true : false;
+                                                              stage->stage_num)) ? true : false;
                        /* Update mapped arg only when extension stage is not enabled */
                        if (enabled) {
                                IA_CSS_ERROR("Leaving: cannot update when stage is enabled.");
@@ -10487,13 +10532,14 @@ ia_css_pipe_update_qos_ext_mapped_arg(struct ia_css_pipe *pipe,
 
                                HIVE_ADDR_sp_group = fw->info.sp.group;
                                sp_dmem_load(SP0_ID,
-                                               (unsigned int)sp_address_of(sp_group),
-                                               &sp_group, sizeof(struct sh_css_sp_group));
+                                            (unsigned int)sp_address_of(sp_group),
+                                            &sp_group,
+                                            sizeof(struct sh_css_sp_group));
                                hmm_load(sp_group.pipe[thread_id].sp_stage_addr[stage_num],
-                                           &sp_stage, sizeof(struct sh_css_sp_stage));
+                                        &sp_stage, sizeof(struct sh_css_sp_stage));
 
                                hmm_load(sp_stage.isp_stage_addr,
-                                           &isp_stage, sizeof(struct sh_css_isp_stage));
+                                        &isp_stage, sizeof(struct sh_css_isp_stage));
 
                                for (mem = 0; mem < N_IA_CSS_ISP_MEMORIES; mem++) {
                                        isp_stage.mem_initializers.params[IA_CSS_PARAM_CLASS_PARAM][mem].address =
@@ -10509,7 +10555,8 @@ ia_css_pipe_update_qos_ext_mapped_arg(struct ia_css_pipe *pipe,
                                }
 
                                hmm_store(sp_stage.isp_stage_addr,
-                                           &isp_stage, sizeof(struct sh_css_isp_stage));
+                                         &isp_stage,
+                                         sizeof(struct sh_css_isp_stage));
                        }
                }
        }
@@ -10520,8 +10567,8 @@ ia_css_pipe_update_qos_ext_mapped_arg(struct ia_css_pipe *pipe,
 #ifdef ISP2401
 static int
 aspect_ratio_crop_init(struct ia_css_stream *curr_stream,
-                       struct ia_css_pipe *pipes[],
-                       bool *do_crop_status)
+                      struct ia_css_pipe *pipes[],
+                      bool *do_crop_status)
 {
        int err = 0;
        int i;
@@ -10567,7 +10614,7 @@ aspect_ratio_crop_check(bool enabled, struct ia_css_pipe *curr_pipe)
 
 static int
 aspect_ratio_crop(struct ia_css_pipe *curr_pipe,
-                   struct ia_css_resolution *effective_res)
+                 struct ia_css_resolution *effective_res)
 {
        int err = 0;
        struct ia_css_resolution crop_res;
@@ -10627,7 +10674,7 @@ aspect_ratio_crop(struct ia_css_pipe *curr_pipe,
        case IA_CSS_PIPE_MODE_YUVPP:
        default:
                IA_CSS_ERROR("aspect ratio cropping invalid args: mode[%d]\n",
-                               curr_pipe->config.mode);
+                            curr_pipe->config.mode);
                assert(0);
                break;
        }
@@ -10691,7 +10738,7 @@ static struct sh_css_hmm_buffer_record
 
        assert(h_vbuf);
        assert((type > IA_CSS_BUFFER_TYPE_INVALID) &&
-               (type < IA_CSS_NUM_DYNAMIC_BUFFER_TYPE));
+              (type < IA_CSS_NUM_DYNAMIC_BUFFER_TYPE));
        assert(kernel_ptr != 0);
 
        buffer_record = &hmm_buffer_record[0];