On Tue, 9 Mar 2010 17:45:57 -0600, H Hartley Sweeten wrote:
> The private_data member of struct file is a void *, there is no need
> to cast it.
>
> Signed-off-by: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx>
> Cc: Jean Delvare <khali@xxxxxxxxxxxx>
> Cc: Ben Dooks <ben-linux@xxxxxxxxx>
>
> ---
>
> diff --git a/drivers/i2c/i2c-dev.c b/drivers/i2c/i2c-dev.c
> index f4110aa..c507204 100644
> --- a/drivers/i2c/i2c-dev.c
> +++ b/drivers/i2c/i2c-dev.c
> @@ -138,7 +138,7 @@ static ssize_t i2cdev_read (struct file *file, char __user *buf, size_t count,
> char *tmp;
> int ret;
>
> - struct i2c_client *client = (struct i2c_client *)file->private_data;
> + struct i2c_client *client = file->private_data;
>
> if (count > 8192)
> count = 8192;
> @@ -162,7 +162,7 @@ static ssize_t i2cdev_write (struct file *file, const char __user *buf, size_t c
> {
> int ret;
> char *tmp;
> - struct i2c_client *client = (struct i2c_client *)file->private_data;
> + struct i2c_client *client = file->private_data;
>
> if (count > 8192)
> count = 8192;
> @@ -369,7 +369,7 @@ static noinline int i2cdev_ioctl_smbus(struct i2c_client *client,
>
> static long i2cdev_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
> {
> - struct i2c_client *client = (struct i2c_client *)file->private_data;
> + struct i2c_client *client = file->private_data;
> unsigned long funcs;
>
> dev_dbg(&client->adapter->dev, "ioctl, cmd=0x%02x, arg=0x%02lx\n",
Applied, thanks.
--
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
