From patchwork Thu Oct 28 14:19:25 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Lukas Bulwahn X-Patchwork-Id: 1547521 Return-Path: X-Original-To: incoming-imx@patchwork.ozlabs.org Delivered-To: patchwork-incoming-imx@bilbo.ozlabs.org Authentication-Results: bilbo.ozlabs.org; dkim=pass (2048-bit key; secure) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=yxkVWVWP; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20210112 header.b=MZZW77BM; dkim-atps=neutral Authentication-Results: ozlabs.org; spf=none (no SPF record) smtp.mailfrom=lists.infradead.org (client-ip=2607:7c80:54:e::133; helo=bombadil.infradead.org; envelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org; receiver=) Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:e::133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by bilbo.ozlabs.org (Postfix) with ESMTPS id 4Hg74n27VFz9sS8 for ; Fri, 29 Oct 2021 01:20:25 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=1TCjn/tfTuKZprmjsgK+8T7BX7bbIxgvA32CNhzqWX4=; b=yxkVWVWPYepOas HQuw5CtVq0S0zR5xzwqasTjsovFccAh5KpE/tDtCcypENpeP0ajqKMyd7LRHI2WBtWrACv9LdXuVk oaweFOK83NhzFUD3nod/mpJIbAWSMuEA7Kvcqxaocj1eQgWp43STRj7NNGD9KZOHllHzdq1agCN4W 6XvHuSmoSkf7WWmVMRXDHfej3GtG68DykSA7ufNVEKRBwagzMQ3vLWJRPx8AxDz2E+ey66sZUO213 zckly7xzJBaZ1PhswDIMtj9lyehUxn8egPDGjo8yGJGFbM5/+DF+45c8z1j11bTt69a/wktti8ezb SCsuRsT+PneGQxWYPRVg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mg6G5-0086E0-C3; Thu, 28 Oct 2021 14:19:57 +0000 Received: from mail-wm1-x32b.google.com ([2a00:1450:4864:20::32b]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mg6G1-0086Cu-1K for linux-arm-kernel@lists.infradead.org; Thu, 28 Oct 2021 14:19:54 +0000 Received: by mail-wm1-x32b.google.com with SMTP id j35-20020a05600c1c2300b0032caeca81b7so4789058wms.0 for ; Thu, 28 Oct 2021 07:19:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=w9dCLGXOlRDJmVPN1HJBPCtrX5Y3dq1CeosXfwTmTIE=; b=MZZW77BM7jF65T9itFpM6n7U5dNOUA1+IJ2d8FO0oyqoTXflUpdwut867AadkBdvOf MdACiu8DDmGMF+ElH9PG5i8Vech57XSQftI0CPVl/H/Lw/Vm1dOXi+WEMpTh9jUyQiki 4IGL04gs8riaamW4lCTxL2bq0QDsXJGQKdGw7lKFv4+GZlRw4q7G23uXPqU+6333/Jht FAL+5sTiLuJhnGZUX+lOTCNtQeJwivdpuzCv9KhQTKrVTHEgBBCUCzuAh7vcvujktWcX FFZ08vnnBqfrGGnGqgBZTDNN5sWGI3Xa1pIT5tRBF+AQbPK+usl8k/O8qrjp212PEG6D MhLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=w9dCLGXOlRDJmVPN1HJBPCtrX5Y3dq1CeosXfwTmTIE=; b=JkKyhIAYY3foCy5xcS3lvQNa9OxVVsZjp7yZt/3808Ut3xBfTikLS9iYTo2hxxfOdF JpsJ2CWkqWICVPjFG71s98flDRgsaC5LEzPGQ75gYXSkFDpKPZf+iLnGSU+F/MiiB2EM KGdCaX+0APeiBoESaaWOAyUu5O77Vp6KmS0groaDfYJGAJAEb1KVz5kEzLqdLX8GHRv0 krcB5z0WGjNNLuLQ9t1NP7N3DuvT3O7hxYaYkH5e9ZIMUvlc/ecafZSlnfkMvwHfRSex ZZ2RdCDNIM/gHZ6bNn9X1ZSJofXlbNMecMwdQ7uObpx0f4NPb6+wJxkWpOjRyfVH1Ma9 xD+w== X-Gm-Message-State: AOAM5322qYAzuExs0+tYz3PO3H3kzFbfMMyoghqJS5slg5xZBhKo0eYE axJ6wZEtFVNMgdJuVuOOve+UVegmAY4bug== X-Google-Smtp-Source: ABdhPJzg6/fblA82QiNOpf6te3qmuIz0aMLwxzyCLkqjiM6kfgiHZAu7QfphRHWjzrEyKfVdfQhyfQ== X-Received: by 2002:a05:600c:2288:: with SMTP id 8mr13025376wmf.40.1635430790515; Thu, 28 Oct 2021 07:19:50 -0700 (PDT) Received: from localhost.localdomain (i5C74E249.versanet.de. [92.116.226.73]) by smtp.gmail.com with ESMTPSA id m2sm6284546wml.15.2021.10.28.07.19.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Oct 2021 07:19:50 -0700 (PDT) From: Lukas Bulwahn To: Russell King , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Sekhar Nori , Bartosz Golaszewski , Linus Walleij , Imre Kaloz , Krzysztof Halasa , Avi Fishman , Tomer Maimon , Tali Perry , Patrick Venture , Nancy Yuen , Benjamin Fair , Dinh Nguyen , Arnd Bergmann , linux-arm-kernel@lists.infradead.org, openbmc@lists.ozlabs.org Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Lukas Bulwahn Subject: [PATCH 00/13] Kconfig symbol clean-up on ./arch/arm{64} Date: Thu, 28 Oct 2021 16:19:25 +0200 Message-Id: <20211028141938.3530-1-lukas.bulwahn@gmail.com> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211028_071953_120890_9ABC3655 X-CRM114-Status: GOOD ( 21.91 ) X-Spam-Score: -0.2 (/) X-Spam-Report: Spam detection software, running on the system "bombadil.infradead.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: Dear arm maintainers, The script ./scripts/checkkconfigsymbols.py warns on invalid references to Kconfig symbols (often: minor typos, name confusions or outdated references). This is a patch series addressing the issues reported by ./scripts/checkkconfigsymbols.py in the ./arch/arm{64}/ directories, quickly filtered down with: Content analysis details: (-0.2 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2a00:1450:4864:20:0:0:0:32b listed in] [list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider [lukas.bulwahn[at]gmail.com] -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org Dear arm maintainers, The script ./scripts/checkkconfigsymbols.py warns on invalid references to Kconfig symbols (often: minor typos, name confusions or outdated references). This is a patch series addressing the issues reported by ./scripts/checkkconfigsymbols.py in the ./arch/arm{64}/ directories, quickly filtered down with: ./scripts/checkkconfigsymbols.py | grep "arch/arm" -B 1 -A 2 without considering kernel configs, i.e., after removing arch/arm{64}/configs in the working tree. It addresses some issues I considered to be "true positives"; so, issues that should be addressed and cleaned up. The patches on updating the comments might arguably be considered of lower value by itself, but as the other patches show, checkkconfigsymbols does detect some relevant issues in the kernel tree otherwise being unnoticed. So, it might be worth to clean up the kernel tree to have checkkconfigsymbols produce a rather short list of issues and then continuously follow and check what checkkconfigsymbols reports. Some work from checkkconfigsymbols on arch/arm/ is deferred; the other remaining issues that checkkconfigsymbols reports are (as far as I understand it now) "false positives", i.e., the tool misinterprets the actual content) or "intentional", e.g., they refer to historic configs for the purpose of documentation (e.g., in changelogs) or are in various kernel configs that are not continuously updated to match the current set of kernel configs. You can see below a grouped list of remaining reports. Each patch in this series can be discussed and applied individually if needed. They are sent in one patch series, as they all orginate from the investigation guided by the same tool and hence share similar topics and resolutions. Please pick this series of minor clean-up patches on ./arch/arm{64}/. It applies cleanly on next-20211027. Best regards, Lukas Remaining reports from ./scripts/checkkconfigsymbols.py pointing to files in ./arch/arm{64}/ (excluding configs), grouped by the rationale why it is not addressed in this patch series: - Reference in historic comment: ARM_NR_BANKS Referencing files: arch/arm/mach-exynos/exynos.c CPU Referencing files: arch/arm/lib/backtrace.S, arch/arm/lib/csumpartialcopyuser.S, arch/arm/mach-rpc/ecard-loader.S, arch/arm/mach-rpc/io-acorn.S, arch/arm/nwfpe/fpmodule.h CPU_ARM92{0,2,5,6}_CPU_IDLE Referencing files: arch/arm/mm/proc-arm92{0,2,5,6}.S KVM_INDIRECT_VECTORS Referencing files: arch/arm64/kvm/hyp/nvhe/host.S - Use of CONFIG_* environment variable beyond the definition in Kconfig scripts AS_DMB_ISHLD Referencing files: arch/arm64/include/asm/vdso/compat_barrier.h, arch/arm64/kernel/vdso32/Makefile CC_HAS_K_CONSTRAINT Referencing files: arch/arm64/Makefile, arch/arm64/include/asm/atomic_ll_sc.h SHELL Ignore CONFIG_SHELL. Default variable in Kbuild build system. - Parsing Mistake (Incomplete parsing heuristics) by checkkconfigsymbols.py: ASC1, ASC2, SBC, UART Referencing files: arch/arm/Kconfig.debug - Future work to send clean-up patches: IRDA{_MODULE} Referencing files: arch/arm/mach-pxa/balloon3.c, arch/arm/mach-pxa/palm27x.c, arch/arm/mach-pxa/palm27x.h, arch/arm/mach-pxa/palmtc.c, include/linux/atalk.h, include/linux/netdevice.h PXA_FICP{_MODULE} Referencing files: arch/arm/mach-pxa/spitz.c DEBUG_LL_SER3 Referencing files: arch/arm/boot/compressed/head.S already discussed in 2014: https://lore.kernel.org/all/1400055127.31197.1.camel@x220/ Lukas Bulwahn (13): arm: debug: remove obsolete debug code for DEBUG_ZTE_ZX arm: debug: reuse the config DEBUG_OMAP2UART{1,2} for OMAP{3,4,5} arm: Kconfig.debug: drop reference to removed ARCH_MSM arm: drop an obsolete ifdef with the removed config PCI_HOST_ITE8152 arm: davinci: remove reference to obsolete BLK_DEV_PALMCHIP_BK3710 arm: ixp4xx: remove dead configs CPU_IXP43X and CPU_IXP46X arm: imx: remove dead left-over from i.MX{27,31,35} removal arm: imx: rename DEBUG_IMX21_IMX27_UART to DEBUG_IMX27_UART arm: milbeaut: remove select of non-existing PINCTRL_MILBEAUT arm: nomadik: drop selecting obsolete CLKSRC_NOMADIK_MTU_SCHED_CLOCK arm: npcm: drop selecting non-existing ARM_ERRATA_794072 arm: socfpga: always select PL310_ERRATA_753970 arm: pgtable: refer to intended CONFIG_ARM_LPAE in comment arch/arm/Kconfig.debug | 28 ++++++++++------------- arch/arm/include/debug/imx-uart.h | 18 +++++++-------- arch/arm/include/debug/pl01x.S | 7 ------ arch/arm/kernel/bios32.c | 2 -- arch/arm/mach-davinci/board-dm644x-evm.c | 3 +-- arch/arm/mach-davinci/board-dm646x-evm.c | 3 +-- arch/arm/mach-davinci/board-neuros-osd2.c | 3 +-- arch/arm/mach-imx/Kconfig | 12 ---------- arch/arm/mach-imx/Makefile | 2 -- arch/arm/mach-ixp4xx/Kconfig | 13 ----------- arch/arm/mach-milbeaut/Kconfig | 1 - arch/arm/mach-nomadik/Kconfig | 1 - arch/arm/mach-npcm/Kconfig | 1 - arch/arm/mach-socfpga/Kconfig | 2 +- arch/arm/mm/pgd.c | 2 +- 15 files changed, 26 insertions(+), 72 deletions(-) Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Arnd Bergmann Reviewed-by: Linus Walleij