Received: by 2002:ac0:946b:0:0:0:0:0 with SMTP id j40csp664526imj; Thu, 7 Feb 2019 09:55:00 -0800 (PST) X-Google-Smtp-Source: AHgI3IZDW5sAo+bVmBTj79M5NflJojCLVks8tLyxuPB8FU+LwMeUokBwJFKiPh7wPth2ALXRdH8V X-Received: by 2002:a63:ce08:: with SMTP id y8mr11109763pgf.388.1549562100515; Thu, 07 Feb 2019 09:55:00 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1549562100; cv=none; d=google.com; s=arc-20160816; b=0PwbOV/IizYl0ezZ1frXdvRKls2ibkYX7W+owtBiUeoWDVwZ+x9CmSp8BXC4nPHFL2 uf0MUabpMS7qT4iNWlV7HN7iMe2yQHfNXZurdLomwpxG6LH7b84ZbRGawRdOWU9FD6Se Xwr3NOqV14IVr0CRbyj4RGyI0XmaJCH/M6XZ0+Mby1EwAfi/qeG6/TlbJ5gY7ulBaNjv ovl2AJhBStguOjcDZL66j/qd3s4YnHxoSmWCXFBc/ZstnWQPQD4W2T/2STm3x6oRSjed LAC1nqfiAgCe1B9qTSc3ezPDFUNjwGRDseoafif3/IebLrheI1t3hpdkDwDXhv9Wl9zC jiBg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :references:in-reply-to:from:subject:cc:to:message-id:date; bh=NSFV3U4HTmkqgpXTC6LG5IDGz1ij0pCb2hkJtLCi3+E=; b=nFhryhVKsCABAelxWqQUXclcPEXcJdQgB9HqUNDvfssSzxUzlD4J3Oo0U2MDvj8ECr viBji408997xyecpANh9e7HcU0j16bqbf6ZeKFUtH99xdkKr/Q0AkwswBtcWLdT86fZL UN5EW1mcIHj5jzqqTR80ClFQWk0N0XUO5/lLUoFw7jQkEkulEZ5St2TGPiki8+12vQFN OULL6D2IvQMT9XtbhRIcXn6NHoMAZWW9EnYwkiDxj7AHdcHOvBr+gINqX/Sepu3g7Y8C E8SEh14D1v27sSAv0Ige+xCND3cJaeJhHR72MDSfEX0BGzBgxbcpN/15mOeLduwIZv7K iQAQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t185si2462806pgt.425.2019.02.07.09.54.44; Thu, 07 Feb 2019 09:55:00 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727150AbfBGRx6 (ORCPT + 99 others); Thu, 7 Feb 2019 12:53:58 -0500 Received: from shards.monkeyblade.net ([23.128.96.9]:50070 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726793AbfBGRx5 (ORCPT ); Thu, 7 Feb 2019 12:53:57 -0500 Received: from localhost (unknown [IPv6:2601:601:9f80:35cd::bf5]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: davem-davemloft) by shards.monkeyblade.net (Postfix) with ESMTPSA id 0778C146D59A9; Thu, 7 Feb 2019 09:53:56 -0800 (PST) Date: Thu, 07 Feb 2019 09:53:56 -0800 (PST) Message-Id: <20190207.095356.712805621945637462.davem@davemloft.net> To: yzhai003@ucr.edu Cc: csong@cs.ucr.edu, zhiyunq@cs.ucr.edu, peppe.cavallaro@st.com, alexandre.torgue@st.com, maxime.ripard@free-electrons.com, wens@csie.org, netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] net: stmmac: Variable "val" in function sun8i_dwmac_set_syscon() could be uninitialized From: David Miller In-Reply-To: References: <20190205221559.17545-1-yzhai003@ucr.edu> <20190206.201736.1146228626076916564.davem@davemloft.net> X-Mailer: Mew version 6.8 on Emacs 26.1 Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Thu, 07 Feb 2019 09:53:57 -0800 (PST) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Yizhuo Zhai Date: Wed, 6 Feb 2019 21:52:15 -0800 > Thanks, but why initialization matters here? Is performance the main > concern? Code that is unnecessary is hard to audit. People will ask "why" is it initialized? In what situations is the initialized value of "0" ever used? You are wasting people's time and energy by writing unnecessary code.