Re: [PATCH] staging: media: atomisp: remove unnecessary parentheses in return statement

[Sakari Ailus]

Hi Teng,

On Tue, Mar 17, 2026 at 04:56:36PM +0100, Teng Liu wrote:
> Remove unnecessary parentheses around the return value in
> ia_css_pipeline_get_pipe_io_status(). return is not a function,
> so parentheses are not required.
> 
> This addresses the following checkpatch error:
>   ERROR: return is not a function, parentheses are not required
> 
> Signed-off-by: Teng Liu <27rabbitlt@xxxxxxxxx>
> ---
>  .../staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c   | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c b/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c
> index 0470871f8..fba567de4 100644
> --- a/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c
> +++ b/drivers/staging/media/atomisp/pci/runtime/pipeline/src/pipeline.c
> @@ -446,7 +446,7 @@ bool ia_css_pipeline_has_stopped(struct ia_css_pipeline *pipeline)
>  
>  struct sh_css_sp_pipeline_io_status *ia_css_pipeline_get_pipe_io_status(void)
>  {
> -	return(&sh_css_sp_group.pipe_io_status);
> +	return &sh_css_sp_group.pipe_io_status;

Is it the only such case in the driver?

>  }
>  
>  bool ia_css_pipeline_is_mapped(unsigned int key)

-- 
Regards,

Sakari Ailus