Smatch complains that it's possible we print "rate" in the debug output when it hasn't been initialized. It should be zero on that path. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> --- v2: fix it a different way diff --git a/drivers/opp/of.c b/drivers/opp/of.c index 7026e9f484ea..6d15f05bfc28 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -287,7 +287,7 @@ static int _opp_add_static_v2(struct opp_table *opp_table, struct device *dev, struct device_node *np) { struct dev_pm_opp *new_opp; - u64 rate; + u64 rate = 0; u32 val; int ret; bool rate_not_available = false; -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html