Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp1189737ybe; Fri, 13 Sep 2019 12:27:40 -0700 (PDT) X-Google-Smtp-Source: APXvYqy6d5wQw+VLQ107AlvrTH7C6Sc0e4v5/vDkm+gVYCyGNMD3KTSkpJHf2iBo96jDLeuMrxuk X-Received: by 2002:a50:d552:: with SMTP id f18mr50194355edj.36.1568402860298; Fri, 13 Sep 2019 12:27:40 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1568402860; cv=none; d=google.com; s=arc-20160816; b=tRFrnUIJyZm96sW8Yj02pt+XVg1B7QbLUTOxv8pyVU3bJQghcpzlcRcnOFZxIqnaUy k6DNGS2WorPZUy6iqJ5+h8vmn1Cjo7uFgEiHEZQuSjo+tUXzeTF5CjXUMe3u/D0HjQxm uAITR1rwttdXreh0tXuqpQ/eFklGVrEkmXYsOEBvCFxN3rsPFARyAywcFQ9xbWr8Odnf 1GKUx6fexu4OYegzZj+Q+mTfFXf7NK7wbCMVHF7qxkoK8Q9/NrXvbL6RNtfVpXxxtgiX vEmzCkofKr6AMKh6kTc2RS2MJn/R6y6JMaPw2RSgTfy6vAG2FTw8Qi7w67DfKuWjjBbr vOYQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-transfer-encoding:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=CpFJ0Or1yxeNvpl7TVaHByalBVgRzhebTRgTdu/Myoo=; b=flrR8fnFshiXGVfawkpKcyTv/0evX3Q4A9NysCqK7dQ00dMk/rTw9Qa3JcG+LJahFX ugoxpJOVza0xd2AL8SDzje+2+kPk1ZBQFikq+9NN4xXJf/wgSXnf9GWZV+Dy8ts9NQ5P agsE60lveZhjyHwj3BYh+yYdvmI4jioYHP/ga8JJTi5LbEM5Eb5zlpwX36AEWiGX+RIj r2OOOYeM8ScFmyCzF3FG+vu/B6697s+Ap0etlPp96cYBuRCrFPff+Vpjc2V1H4U5DD2Z lm+KkVePINNuSz9iqLPalf6E4NOjsx4x9R2JsaPirZPPh4X2I1JuVZZp3Xq/6QG8+zeQ jv/Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@alien8.de header.s=dkim header.b=GQhXiY0R; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=alien8.de Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b56si18227113edb.418.2019.09.13.12.27.16; Fri, 13 Sep 2019 12:27:40 -0700 (PDT) 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; dkim=pass header.i=@alien8.de header.s=dkim header.b=GQhXiY0R; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=alien8.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731094AbfIMRx1 (ORCPT + 99 others); Fri, 13 Sep 2019 13:53:27 -0400 Received: from mail.skyhub.de ([5.9.137.197]:45788 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726822AbfIMRx1 (ORCPT ); Fri, 13 Sep 2019 13:53:27 -0400 Received: from zn.tnic (p200300EC2F0DC5006892875336F1420F.dip0.t-ipconnect.de [IPv6:2003:ec:2f0d:c500:6892:8753:36f1:420f]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.skyhub.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id E97671EC03F6; Fri, 13 Sep 2019 19:53:25 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alien8.de; s=dkim; t=1568397206; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=CpFJ0Or1yxeNvpl7TVaHByalBVgRzhebTRgTdu/Myoo=; b=GQhXiY0RK7UxrkeUSbC6UeOZRWgIOOSG8xqxRO0+tErFfykWm7CKpe96eGDHONqNKhRPFG VQ277y4HQg/4nuP+h8lk3ZMEsR5WyF0cUnDrUrdYp1hUX0KZAqiNUtq8o8h6KrVzvd5BmK kSXh0o+qjLqc35Z+6pDXe49SMYIKzSo= Date: Fri, 13 Sep 2019 19:53:20 +0200 From: Borislav Petkov To: Mauro Carvalho Chehab Cc: Linux Edac Mailing List , Linux Kernel Mailing List , Tony Luck , Qiuxu Zhuo , Robert Richter , James Morse , Khuong Dinh Subject: Re: [PATCH 0/7] Address most issues when building with W=1 Message-ID: <20190913175320.GD4190@zn.tnic> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 13, 2019 at 11:50:25AM -0300, Mauro Carvalho Chehab wrote: > There is a recent discussion at KS ML with regards to use W=1 as default. > > No idea if this will happen or not, but it doesn't hurt cleaning up W=1 > warnings from the EDAC subsystem, specially since it helps to cleanup > a few things. > > This patch series addresses most of such warnings. After this series, > there will be just two W=1 warnings: > > 1) i5100 EDAC driver: > > drivers/edac/i5100_edac.c: In function ‘i5100_read_log’: > drivers/edac/i5100_edac.c:487:11: warning: variable ‘ecc_loc’ set but not used [-Wunused-but-set-variable] > 487 | unsigned ecc_loc = 0; > | ^~~~~~~ > > > The ecc_loc contents is filled from MC data, but it is not used anywere. > The i5100 MC is very old: the affected code was added in 2008. It should > probably be safe to just drop the corresponding data, but, as it may > contain some relevant info, I was a little reticent of doing that. > > 2) Xgene EDAC driver: > > drivers/edac/xgene_edac.c: In function ‘xgene_edac_rb_report’: > drivers/edac/xgene_edac.c:1486:7: warning: variable ‘address’ set but not used [-Wunused-but-set-variable] > 1486 | u32 address; > | ^~~~~~~ > > I suspect that the content of the address field should actually be used on > at least some of the logs. + Khuong Dinh for that. > I may eventually submit patches later to address the above cases, but let's > solve first the other cases, as they all sound trivial enough. > > Mauro Carvalho Chehab (7): > EDAC: i5100_edac: get rid of an unused var > EDAC: i7300_edac: rename a kernel-doc var description > EDAC: i7300_edac: fix a kernel-doc syntax > EDAC: i5400_edac: print type at debug message > EDAC: i5400_edac: get rid of some unused vars > EDAC: sb_edac: get rid of unused vars > EDAC: skx_common: get rid of unused type var > > drivers/edac/i5100_edac.c | 2 -- > drivers/edac/i5400_edac.c | 15 +++------------ > drivers/edac/i7300_edac.c | 4 ++-- > drivers/edac/sb_edac.c | 21 ++++++++------------- > drivers/edac/skx_common.c | 5 +---- > 5 files changed, 14 insertions(+), 33 deletions(-) Looks ok to me at a quick glance, ACK. I've already sent the 5.4 pull request to Linus so you could queue those after -rc1. It's not like they're urgent or so. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette