Hi Dan,
On 04/10/2015 10:30 AM, Dan Carpenter wrote:
Static checkers complain that "timers[i].delay_on" is an unsigned long
but we're writing to only 32 bits of it. The code works on 32 bit
systems and little endian 64 bit systems so it doesn't cause a problem
in practise but it's still better to silence the warning.
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
---
v2: use a temporary variable
diff --git a/drivers/leds/leds-netxbig.c b/drivers/leds/leds-netxbig.c
index 028686f2..6cb4537 100644
--- a/drivers/leds/leds-netxbig.c
+++ b/drivers/leds/leds-netxbig.c
@@ -444,12 +444,17 @@ static int netxbig_leds_get_of_pdata(struct device *dev,
if (!timers)
return -ENOMEM;
for (i = 0; i < num_timers; i++) {
+ u32 delay_on = 0;
+ u32 delay_off = 0;
These variables don't need initialization, as they are assigned
a new value in of_property_read_u32_index anyway.
of_property_read_u32_index(np, "timers", 3 * i,
&timers[i].mode);
of_property_read_u32_index(np, "timers", 3 * i + 1,
- (u32 *) &timers[i].delay_on);
+ &delay_on);
of_property_read_u32_index(np, "timers", 3 * i + 2,
- (u32 *) &timers[i].delay_off);
+ &delay_off);
+ timers[i].delay_on = delay_on;
+ timers[i].delay_off = delay_off;
}
pdata->timer = timers;
pdata->num_timer = num_timers;
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Best Regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line "unsubscribe linux-leds" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
