| Message ID | 1483097651-7172-1-git-send-email-augustocaringi@gmail.com (mailing list archive) |
|---|---|
| State | Mainlined |
| Delegated to: | Rafael Wysocki |
| Headers | show |
On Friday, December 30, 2016 11:34:08 AM Augusto Mecking Caringi wrote: > This patch fixes the following gcc warning: > > drivers/base/power/domain.c: In function ‘genpd_runtime_resume’: > drivers/base/power/domain.c:642:14: warning: ‘time_start’ may be used > uninitialized in this function [-Wmaybe-uninitialized] > elapsed_ns = ktime_to_ns(ktime_sub(ktime_get(), time_start) > > The same problem (in another function in this same file) was fixed in > commit d33d5a6c88fc (avoid spurious "may be used uninitialized" warning) > > Signed-off-by: Augusto Mecking Caringi <augustocaringi@gmail.com> > --- > drivers/base/power/domain.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index a5e1262..2997026 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -626,6 +626,7 @@ static int genpd_runtime_resume(struct device *dev) > > out: > /* Measure resume latency. */ > + time_start = 0; > if (timed && runtime_pm) > time_start = ktime_get(); Applied. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index a5e1262..2997026 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -626,6 +626,7 @@ static int genpd_runtime_resume(struct device *dev) out: /* Measure resume latency. */ + time_start = 0; if (timed && runtime_pm) time_start = ktime_get();
This patch fixes the following gcc warning: drivers/base/power/domain.c: In function ‘genpd_runtime_resume’: drivers/base/power/domain.c:642:14: warning: ‘time_start’ may be used uninitialized in this function [-Wmaybe-uninitialized] elapsed_ns = ktime_to_ns(ktime_sub(ktime_get(), time_start) The same problem (in another function in this same file) was fixed in commit d33d5a6c88fc (avoid spurious "may be used uninitialized" warning) Signed-off-by: Augusto Mecking Caringi <augustocaringi@gmail.com> --- drivers/base/power/domain.c | 1 + 1 file changed, 1 insertion(+)