Re: [PATCH] hwmon: (ina2xx) Remove casting the return value which is a void pointer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 09 Sep 2013 14:29:32 +0900, Jingoo Han wrote:
> Casting the return value which is a void pointer is redundant.
> The conversion from void pointer to any other pointer type is
> guaranteed by the C programming language.
> 
> Signed-off-by: Jingoo Han <jg1.han@xxxxxxxxxxx>
> ---
>  drivers/hwmon/ina2xx.c |    3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/hwmon/ina2xx.c b/drivers/hwmon/ina2xx.c
> index 18c0623..70a39a8 100644
> --- a/drivers/hwmon/ina2xx.c
> +++ b/drivers/hwmon/ina2xx.c
> @@ -233,8 +233,7 @@ static int ina2xx_probe(struct i2c_client *client,
>  		return -ENOMEM;
>  
>  	if (dev_get_platdata(&client->dev)) {
> -		pdata =
> -		  (struct ina2xx_platform_data *)dev_get_platdata(&client->dev);
> +		pdata = dev_get_platdata(&client->dev);
>  		shunt = pdata->shunt_uohms;
>  	} else if (!of_property_read_u32(client->dev.of_node,
>  				"shunt-resistor", &val)) {

Acked-by: Jean Delvare <khali@xxxxxxxxxxxx>

Guenter, I'll let you pick it.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux