From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753375AbcAGWwA (ORCPT ); Thu, 7 Jan 2016 17:52:00 -0500 Received: from mail-pa0-f47.google.com ([209.85.220.47]:34107 "EHLO mail-pa0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752817AbcAGWv6 (ORCPT ); Thu, 7 Jan 2016 17:51:58 -0500 From: Daniel Axtens To: Arnd Bergmann , linuxppc-dev@lists.ozlabs.org Cc: Brian Norris , Michael Ellerman , Michael Neuling , Anton Blanchard , linux-kernel@vger.kernel.org, Michal Marek , Ian Munsie Subject: Re: Build failure: -Wno-unused-const-variable DNE on old GCC In-Reply-To: <10821353.mAlCTdTQBC@wuerfel> References: <20160107185406.GA65420@google.com> <10821353.mAlCTdTQBC@wuerfel> User-Agent: Notmuch/0.21 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-pc-linux-gnu) Date: Fri, 08 Jan 2016 09:51:35 +1100 Message-ID: <87oacx6m14.fsf@gamma.ozlabs.ibm.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Hi Arnd, Thanks for your patch. Acked-by: Daniel Axtens > Alternatively, remove the -Werror. We occasionally get people that add this > flag to a Makefile, but it tends to cause more trouble whenever a new > gcc version arrives. Speaking up as the person who added -Werror to cxl, I'd really rather it stayed. There are a number of reasons I think this. Here's the first three that came to mind. - cxl is powerpc specific (and always will be for deep seated hardware reasons), and is handled through the powerpc tree. arch/powerpc compiles with -Werror, and as part of the powerpc ecosystem, cxl should too. - It forces cxl developers to a higher standard. cxl has already had more than it's fair share of incredibly difficult to debug issues, so any way we can reduce the risk of errors going in makes our lives (and our end-users lives) better. - I am (and I'm quite confident the other cxl people are) quite happy to send patches to fix build-breaking issues such as this. Indeed, I would have, except you sent it during the Australian night :) If it's really super-duper important we can consider putting it behind a config guard, but I'd really rather not. Regards, Daniel --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 Comment: GPGTools - https://gpgtools.org iQIcBAEBCgAGBQJWjuv5AAoJEPC3R3P2I92FkEUQAKJlmQyzcyuOit3ezRC5c7iC dAbOw8D/5Z29bN3GV1IQe5PykF3jKC/u8EVZXTt08xOHyoUW2LxZ2mWpySX/Oc1d JBzTIJ5lBwMbV1ilwmSfXHFqqeeoPLEjuQKjCZGaPNclUUZWPJmequGIYdtvwqDD T9Kvv6ly2OMF2Cc41kjbXdiGwjgtbxHjoYFCq+saezlc1dPbWl+JB50Du9rkL6HN VmU6IHiTA7EIzBHcjTw7FXFfVPCDjKndLymw81WFa6541NEdEej3tYxKeLgHSkd9 9xz89J2hbD4UQ9jQyDLEPA0jJ5XODtI4rJyV//LxL/HEVqXl4hyPtpNoSx7izsHY zfSbZvtDer/Yx0EIRSSQfbiR2rfzHks1+bbP/Li8kzyhvKVf9VNT5Q+ncmDPdtk6 9RZc/MpDR5qJQ+Joq3tiLTsbJxctObQt/aNTmf0FWZLZiU3eu4KwXEHCDcWLLz4z YqxztssilW7WumMGf13N3WwB5bxitm4il51UzzdiWSPvVW+1FVbfydgNP/1VUWJm kuYQHjJy9mA5TZMM5kWev3LEGQXwKYwj5bXd5B7tpz3wcD+HsFy64T/3OQTEupy+ VVf/PNQWztrXZj2qkTCCB+2uY3+ebIq6zMuVCZ2SJrFgWUGgjoMr/RYqTwNIPsbj aPZpFUeroq+sKwEQ6dPq =4hn7 -----END PGP SIGNATURE----- --=-=-=--