From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751951AbdAFBdC convert rfc822-to-8bit (ORCPT ); Thu, 5 Jan 2017 20:33:02 -0500 Received: from cloudserver094114.home.net.pl ([79.96.170.134]:43993 "EHLO cloudserver094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751803AbdAFBc4 (ORCPT ); Thu, 5 Jan 2017 20:32:56 -0500 From: "Rafael J. Wysocki" To: Augusto Mecking Caringi Cc: torvalds@linux-foundation.org, Kevin Hilman , Ulf Hansson , Len Brown , Pavel Machek , Greg Kroah-Hartman , linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] PM / domains: Fix 'may be used uninitialized' build warning Date: Fri, 06 Jan 2017 02:28:42 +0100 Message-ID: <1783360.Wg7bKzXGUa@aspire.rjw.lan> User-Agent: KMail/4.14.10 (Linux/4.10.0-rc1+; KDE/4.14.9; x86_64; ; ) In-Reply-To: <1483097651-7172-1-git-send-email-augustocaringi@gmail.com> References: <1483097651-7172-1-git-send-email-augustocaringi@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 > --- > 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