On Mon, Oct 28, 2013 at 01:19:29PM -0700, Lisa Nguyen wrote:
> --- a/drivers/staging/media/davinci_vpfe/vpfe_video.c
> +++ b/drivers/staging/media/davinci_vpfe/vpfe_video.c
> @@ -346,7 +346,10 @@ static int vpfe_pipeline_disable(struct vpfe_pipeline
> *pipe)
> }
> mutex_unlock(&mdev->graph_mutex);
>
> - return (ret == 0) ? ret : -ETIMEDOUT ;
> + if (ret == 0)
> + return ret;
> + else
> + return -ETIMEDOUT;
> }
Since everyone has an opinion on this return format my prefered is:
if (ret)
return -ETIMEDOUT;
return 0;
I like a clear "return 0;" instead of a "return ret;" where ret is
always zero.
I also like to keep my error path and my success path as separate as
possible. Handle the errors right away and move them out of the success
path.
regards,
dan carpenter
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
