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 09/08/2013 11:53 PM, Jean Delvare wrote:
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.

Applied, thanks

Guenter


_______________________________________________
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