leandrof/linux
1
0
Fork 0
forked from Minki/linux
Code Issues Pull Requests Packages Projects Releases Wiki Activity

staging: media: davinci_vpfe: Remove useless cast on void pointer

Browse Source 
void pointers do not need to be cast to other pointer types.

The semantic patch used to find this:

@r@
expression x;
void* e;
type T;
identifier f;
@@

(
  *((T *)e)
|
  ((T *)x)[...]
|
  ((T *)x)->f
|
- (T *)
  e
)

Build tested it.

Signed-off-by: Tapasweni Pathak <tapaswenipathak@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Tapasweni Pathak 2014-10-30 17:02:36 +05:30 committed by Greg Kroah-Hartman
parent 31f8f066f5
commit ad1d2a4fc8
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/staging/media/davinci_vpfe/dm365_ipipeif.c
View File

@ -421,7 +421,7 @@ static int
ipipeif_get_config(struct v4l2_subdev *sd, void __user *arg) ipipeif_get_config(struct v4l2_subdev *sd, void __user *arg)
{ {
struct vpfe_ipipeif_device *ipipeif = v4l2_get_subdevdata(sd); struct vpfe_ipipeif_device *ipipeif = v4l2_get_subdevdata(sd);
struct ipipeif_params *config = (struct ipipeif_params *)arg; struct ipipeif_params *config = arg;
struct device *dev = ipipeif->subdev.v4l2_dev->dev; struct device *dev = ipipeif->subdev.v4l2_dev->dev;
if (!arg) { if (!arg) {
@ -462,7 +462,7 @@ ipipeif_get_config(struct v4l2_subdev *sd, void __user *arg)
static long ipipeif_ioctl(struct v4l2_subdev *sd, static long ipipeif_ioctl(struct v4l2_subdev *sd,
unsigned int cmd, void *arg) unsigned int cmd, void *arg)
{ {
struct ipipeif_params *config = (struct ipipeif_params *)arg; struct ipipeif_params *config = arg;
int ret = -ENOIOCTLCMD; int ret = -ENOIOCTLCMD;
switch (cmd) { switch (cmd) {

6
drivers/staging/media/davinci_vpfe/dm365_resizer.c
View File

@ -149,7 +149,7 @@ configure_resizer_out_params(struct vpfe_resizer_device *resizer, int index,
param->rsz_en[index] = DISABLE; param->rsz_en[index] = DISABLE;
return; return;
} }
output = (struct vpfe_rsz_output_spec *)output_spec; output = output_spec;
param->rsz_en[index] = ENABLE; param->rsz_en[index] = ENABLE;
if (partial) { if (partial) {
param->rsz_rsc_param[index].h_flip = output->h_flip; param->rsz_rsc_param[index].h_flip = output->h_flip;
@ -1218,12 +1218,12 @@ static long resizer_ioctl(struct v4l2_subdev *sd, unsigned int cmd, void *arg)
switch (cmd) { switch (cmd) {
case VIDIOC_VPFE_RSZ_S_CONFIG: case VIDIOC_VPFE_RSZ_S_CONFIG:
user_config = (struct vpfe_rsz_config *)arg; user_config = arg;
ret = resizer_set_configuration(resizer, user_config); ret = resizer_set_configuration(resizer, user_config);
break; break;
case VIDIOC_VPFE_RSZ_G_CONFIG: case VIDIOC_VPFE_RSZ_G_CONFIG:
user_config = (struct vpfe_rsz_config *)arg; user_config = arg;
if (!user_config->config) { if (!user_config->config) {
dev_err(dev, "error in VIDIOC_VPFE_RSZ_G_CONFIG\n"); dev_err(dev, "error in VIDIOC_VPFE_RSZ_G_CONFIG\n");
return -EINVAL; return -EINVAL;