Hi Lisa,

Thanks for the patch.

On Wed, Dec 11, 2013 at 11:39 AM, Lisa Nguyen <l...@xenapiadmin.com> wrote:
> Rewrite the return statement in vpfe_video.c. This will prevent
> the checkpatch.pl script from generating a warning saying
> to remove () from this particular return statement.
>
> Signed-off-by: Lisa Nguyen <l...@xenapiadmin.com>

Acked-by: Lad, Prabhakar <prabhakar.cse...@gmail.com>

Will be queueing it for 3.14.

Regrads,
--Prabhakar Lad

> ---
> Changes since v3:
> - Removed () from return statement per Laurent Pinchart's suggestion
>
>  drivers/staging/media/davinci_vpfe/vpfe_video.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/media/davinci_vpfe/vpfe_video.c 
> b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> index 24d98a6..3b036be 100644
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> @@ -346,7 +346,7 @@ static int vpfe_pipeline_disable(struct vpfe_pipeline 
> *pipe)
>         }
>         mutex_unlock(&mdev->graph_mutex);
>
> -       return (ret == 0) ? ret : -ETIMEDOUT ;
> +       return ret ? -ETIMEDOUT : 0;
>  }
>
>  /*
> --
> 1.7.9.5
>
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to