From patchwork Sat Sep 19 19:02:56 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Lunn X-Patchwork-Id: 1367704 Return-Path: X-Original-To: patchwork-incoming-netdev@ozlabs.org Delivered-To: patchwork-incoming-netdev@ozlabs.org Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=vger.kernel.org (client-ip=23.128.96.18; helo=vger.kernel.org; envelope-from=netdev-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=lunn.ch Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by ozlabs.org (Postfix) with ESMTP id 4Bv0Tv3yT9z9sSf for ; Sun, 20 Sep 2020 05:03:31 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726554AbgISTDZ (ORCPT ); Sat, 19 Sep 2020 15:03:25 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:45370 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726481AbgISTDZ (ORCPT ); Sat, 19 Sep 2020 15:03:25 -0400 Received: from andrew by vps0.lunn.ch with local (Exim 4.94) (envelope-from ) id 1kJi8j-00FPaf-LS; Sat, 19 Sep 2020 21:03:17 +0200 From: Andrew Lunn To: netdev Cc: David Miller , Jakub Kicinski , Masahiro Yamada , Michal Marek , linux-kbuild@vger.kernel.org, clang-built-linux@googlegroups.com, Andrew Lunn Subject: [PATCH RFC/RFT 0/2] W=1 by default for Ethernet PHY subsystem Date: Sat, 19 Sep 2020 21:02:56 +0200 Message-Id: <20200919190258.3673246-1-andrew@lunn.ch> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org There is a movement to make the code base compile clean with W=1. Some subsystems are already clean. In order to keep them clean, we need developers to build new code with W=1 by default in these subsystems. This patchset refactors the core Makefile warning code to allow the additional warnings W=1 adds available to any Makefile. The Ethernet PHY subsystem Makefiles then make use of this to make W=1 the default for this subsystem. RFT since i've only tested with x86 and arm with a modern gcc. Is the code really clean for older compilers? For clang? Andrew Lunn (2): scripts: Makefile.extrawarn: Add W=1 warnings to a symbol net: phylib: Enable W=1 by default drivers/net/mdio/Makefile | 3 +++ drivers/net/pcs/Makefile | 3 +++ drivers/net/phy/Makefile | 3 +++ scripts/Makefile.extrawarn | 33 ++++++++++++++++++--------------- 4 files changed, 27 insertions(+), 15 deletions(-)