From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AIpwx49JV7CtLXkE2dYPL1OaOIGO/QeVRemjwuQUNB9hdkTfvtV8FRJy07njz3YnrtuE89dgmyEl ARC-Seal: i=1; a=rsa-sha256; t=1524406131; cv=none; d=google.com; s=arc-20160816; b=FEOzzT8IWGYoqSy3pT/p9mvoTBM6DM7iW+0oMlShb9kCkAyEv9KsucFmwg1DDZaOUy 6Zie3DX+259wyTLqOMVhGEkppEp7mKtFdzeJnOfqUQ/77V9MxGEDyTdS0ROwYUlpvV99 t1Bnv17ZBP2hRjWQE+xoDZIxb4mPnbDX5ekYEGwDFNAnv0/HAm9XjjrGoo9I/fR3/t+T DsVLBQP35AohzJ2rIPR27CVAWxWeDNZY/5zS67tQu4aF6jJtcjJzk6qcnYh1pcgE39s3 qq2FZlC1PLS20H855+mG/vIBEFfxhuXEQi6HZ1h0nuiPchXhxtMysdfZMsF5MLK600ev pNVg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:user-agent:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=Wxm9rKNBgqbju1mj0B5+5k7HT98e6JESTezPE9VawgI=; b=gMelUXHmwPigAAjPs0JEp2iupSthBGDD/8jJOrnJDq0eFSuuC7csdrKdsODq+G2dRL FLo5hQzWq8Eot9nfcI4tleyabauzmKRyBV1H+8c0PXWGXv/ZPOAAeUAcodiOveEuCnvs URL5a6b4U7FUPWKIhuK30XFwPWkXrJ83FMPrsWRKNqDfToieZzTvUMyrYnsxvS6m/W5u ld8Iw1X3/6WJwZMPAnAa1A5FcezyU32OSWzSj+jYdPnrnArVsuWaaWBsL3Pbpe5cIzth 6MqnG8dyz3kfhr0IMMAOxlES537fDsg97C04fyzOaFRywBSqC5IYizmvncJl3Pot73y8 hMpw== ARC-Authentication-Results: i=1; mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.61.202 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org Authentication-Results: mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.61.202 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Andi Kleen , Arnd Bergmann , Geert Uytterhoeven , Stephen Boyd Subject: [PATCH 4.14 107/164] clk: fix false-positive Wmaybe-uninitialized warning Date: Sun, 22 Apr 2018 15:52:54 +0200 Message-Id: <20180422135139.797232331@linuxfoundation.org> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180422135135.400265110@linuxfoundation.org> References: <20180422135135.400265110@linuxfoundation.org> User-Agent: quilt/0.65 X-stable: review MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-LABELS: =?utf-8?b?IlxcU2VudCI=?= X-GMAIL-THRID: =?utf-8?q?1598455168511704983?= X-GMAIL-MSGID: =?utf-8?q?1598455683516765937?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: 4.14-stable review patch. If anyone has any objections, please let me know. ------------------ From: Arnd Bergmann commit ce33f284935e08229046b30635e6aadcbab02b53 upstream. When we build this driver with on x86-32, gcc produces a false-positive warning: drivers/clk/renesas/clk-sh73a0.c: In function 'sh73a0_cpg_clocks_init': drivers/clk/renesas/clk-sh73a0.c:155:10: error: 'parent_name' may be used uninitialized in this function [-Werror=maybe-uninitialized] return clk_register_fixed_factor(NULL, name, parent_name, 0, We can work around that warning by adding a fake initialization, I tried and failed to come up with any better workaround. This is currently one of few remaining warnings for a 4.14.y randconfig build, so it would be good to also have it backported at least to that version. Older versions have more randconfig warnings, so we might not care. I had not noticed this earlier, because one patch in my randconfig test tree removes the '-ffreestanding' option on x86-32, and that avoids the warning. The -ffreestanding flag was originally global but moved into arch/i386 by Andi Kleen in commit 6edfba1b33c7 ("[PATCH] x86_64: Don't define string functions to builtin") as a 'temporary workaround'. Like many temporary hacks, this turned out to be rather long-lived, from all I can tell we still need a simple fix to asm/string_32.h before it can be removed, but I'm not sure about how to best do that. Cc: stable@vger.kernel.org Cc: Andi Kleen Signed-off-by: Arnd Bergmann Acked-by: Geert Uytterhoeven Signed-off-by: Stephen Boyd Signed-off-by: Greg Kroah-Hartman --- drivers/clk/renesas/clk-sh73a0.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) --- a/drivers/clk/renesas/clk-sh73a0.c +++ b/drivers/clk/renesas/clk-sh73a0.c @@ -46,7 +46,7 @@ struct div4_clk { unsigned int shift; }; -static struct div4_clk div4_clks[] = { +static const struct div4_clk div4_clks[] = { { "zg", "pll0", CPG_FRQCRA, 16 }, { "m3", "pll1", CPG_FRQCRA, 12 }, { "b", "pll1", CPG_FRQCRA, 8 }, @@ -79,7 +79,7 @@ sh73a0_cpg_register_clock(struct device_ { const struct clk_div_table *table = NULL; unsigned int shift, reg, width; - const char *parent_name; + const char *parent_name = NULL; unsigned int mult = 1; unsigned int div = 1; @@ -135,7 +135,7 @@ sh73a0_cpg_register_clock(struct device_ shift = 24; width = 5; } else { - struct div4_clk *c; + const struct div4_clk *c; for (c = div4_clks; c->name; c++) { if (!strcmp(name, c->name)) {