Commit 4b72aa8a authored by Rehas Sachdeva's avatar Rehas Sachdeva Committed by Greg Kroah-Hartman

staging: media: davinci_vpfe: Merge assignment with return

Instead of storing the return value into a variable and then returning it, we
can club the two into a single return statement. This change was made using
the following semantic patch by Coccinelle:

@@
local idexpression ret;
expression e;
@@

-ret =
+return
     e;
-return ret;
Signed-off-by: default avatarRehas Sachdeva <aquannie@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5173abfc
...@@ -146,9 +146,8 @@ enum v4l2_field vpfe_isif_get_fid(struct vpfe_device *vpfe_dev) ...@@ -146,9 +146,8 @@ enum v4l2_field vpfe_isif_get_fid(struct vpfe_device *vpfe_dev)
u32 field_status; u32 field_status;
field_status = isif_read(isif->isif_cfg.base_addr, MODESET); field_status = isif_read(isif->isif_cfg.base_addr, MODESET);
field_status = (field_status >> DM365_ISIF_MDFS_OFFSET) & return (field_status >> DM365_ISIF_MDFS_OFFSET) &
DM365_ISIF_MDFS_MASK; DM365_ISIF_MDFS_MASK;
return field_status;
} }
static int static int
...@@ -594,8 +593,7 @@ isif_validate_raw_params(struct vpfe_isif_raw_config *params) ...@@ -594,8 +593,7 @@ isif_validate_raw_params(struct vpfe_isif_raw_config *params)
ret = isif_validate_dfc_params(&params->dfc); ret = isif_validate_dfc_params(&params->dfc);
if (ret) if (ret)
return ret; return ret;
ret = isif_validate_bclamp_params(&params->bclamp); return isif_validate_bclamp_params(&params->bclamp);
return ret;
} }
static int isif_set_params(struct v4l2_subdev *sd, void *params) static int isif_set_params(struct v4l2_subdev *sd, void *params)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment