From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nathan Chancellor Subject: Re: [PATCH] pinctrl: aspeed: Wrap -Woverride-init with cc-option Date: Mon, 10 Dec 2018 17:25:25 -0700 Message-ID: <20181211002525.GA8468@flashbox> References: <20181211000506.7559-1-natechancellor@gmail.com> <1544487006.3301683.1605247720.567E9685@webmail.messagingengine.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Joel Stanley Cc: Andrew Jeffery , Linus Walleij , linux-gpio@vger.kernel.org, Linux ARM , linux-aspeed@lists.ozlabs.org, Linux Kernel Mailing List List-Id: linux-gpio@vger.kernel.org On Tue, Dec 11, 2018 at 10:46:26AM +1030, Joel Stanley wrote: > On Tue, 11 Dec 2018 at 10:40, Andrew Jeffery wrote: > > > > > > > > On Tue, 11 Dec 2018, at 10:35, Nathan Chancellor wrote: > > > Clang does not support this option: > > > > > > warning: unknown warning option '-Woverride-init'; did you mean > > > '-Woverride-module'? [-Wunknown-warning-option] > > > 1 warning generated. > > > > > > Signed-off-by: Nathan Chancellor > > > > Acked-by: Andrew Jeffery > > Thanks Nathan. > > Andrew explained to me that this warning helps find bugs when > developing the complicated (some would say crazy) set of macros that > this pinctrl driver uses. It would be good to have this warning > available in clang. Do we have a clang issue opened for that? > > Acked-by: Joel Stanley > > Cheers, > > Joel Hi Joel, I just opened https://github.com/ClangBuiltLinux/linux/issues/293. I searched the LLVM issue tracker and haven't see any reference of this. I can file an issue but if you are a little more familiar with this warning, you are welcome to do it instead. Thanks for the quick response, Nathan