From patchwork Tue Jan 6 22:03:40 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 1325 Return-Path: X-Original-To: u-boot-concept@u-boot.org Delivered-To: u-boot-concept@u-boot.org DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737164; bh=VyRVPgNfRhr7TZacLRp6OSGT9B3KgXYUPbKs3mxcY68=; h=From:To:Date:In-Reply-To:References:CC:Subject:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=VxCtXQ8CjHbSTZDsawN9uXbDQ4SmI+8mjNBaE8dWdCjVMs0MjFarfepdXKUm92Nix jY4TOspgCxT6scetTKZCtNkWwK40MVFVV5ohZxD5rygrmU3FWJH/Kl08Fr1BDVuioc BBANiJFb3zqvMpK1JiQSNziF/FR2jiRkallu8M/xhvDAhJDURqP23AhJ5ZCfIig0oc U3EioIWsiTTb2auU8YuD84Ivt6FWDPo8csKrtQWEJGeC38FUbIXQu8GtNzFbXwichQ lKpOEzrDR/Kz5zPNtNjHGq3uvaorcMJAEfKrsJQR6U9u+CkpjJ04B1o206i57jnoV5 6O9WwbQn3EgdA== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 203DC69172 for ; Tue, 6 Jan 2026 15:06:04 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10024) with ESMTP id qK7hVQ-_fAKH for ; Tue, 6 Jan 2026 15:06:04 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737164; bh=VyRVPgNfRhr7TZacLRp6OSGT9B3KgXYUPbKs3mxcY68=; h=From:To:Date:In-Reply-To:References:CC:Subject:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=VxCtXQ8CjHbSTZDsawN9uXbDQ4SmI+8mjNBaE8dWdCjVMs0MjFarfepdXKUm92Nix jY4TOspgCxT6scetTKZCtNkWwK40MVFVV5ohZxD5rygrmU3FWJH/Kl08Fr1BDVuioc BBANiJFb3zqvMpK1JiQSNziF/FR2jiRkallu8M/xhvDAhJDURqP23AhJ5ZCfIig0oc U3EioIWsiTTb2auU8YuD84Ivt6FWDPo8csKrtQWEJGeC38FUbIXQu8GtNzFbXwichQ lKpOEzrDR/Kz5zPNtNjHGq3uvaorcMJAEfKrsJQR6U9u+CkpjJ04B1o206i57jnoV5 6O9WwbQn3EgdA== Received: from mail.u-boot.org (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 095B05FF82 for ; Tue, 6 Jan 2026 15:06:04 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737161; bh=3GS2m/RRBkT7ddylt5QzGmcJX4nD79Qqz65VuPZKlhY=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=cn35aPRlt6ShY38FHgYJV8yeaUgsY8OwqMM9hkC3jSVhKNnhrBIMTFOgdJrKRFMGZ wvC5x9pSEk6WTLvrgEh0OmufMEy+AYzkIUUbogsMpN//UsM0fUb+9MkFvYulaYvjVj NseJfWSu0tcTdFYkQrARzg3gAEMWcbxqQgLhU4yupFqXGphi0rJj1dr5ZSfz0kk6Mu gTX/0bl52KLjT09a7+rUZ6uXlVbegAdSqs96Vs2VALc2ZcUv7rtbO7zzbOJ5jhrkhq iKMVp80F22YYQR5Q9ZRzQNrEH0kIvf3BwjJwr24rm7WGj8Hl17H9aVZ9YgkGqZM1U2 7cTlPiM5qMHqQ== Received: from localhost (localhost [127.0.0.1]) by mail.u-boot.org (Postfix) with ESMTP id 778F95FF82; Tue, 6 Jan 2026 15:06:01 -0700 (MST) X-Virus-Scanned: Debian amavis at Received: from mail.u-boot.org ([127.0.0.1]) by localhost (mail.u-boot.org [127.0.0.1]) (amavis, port 10026) with ESMTP id 83UD-vS-X6xc; Tue, 6 Jan 2026 15:06:01 -0700 (MST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=u-boot.org; s=default; t=1767737157; bh=rnBg13+Sdn1NfVdoXNhMJCtcUG0M+jAMD3hHUw0rgJo=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=hC13RJI5xTfMD2x3fFIpHcNNMNYnNs3EdTEAYp7CwJ95ia8UYZLbffv8TUBlLmDfQ mHvQ2JxFtBDkMMhq1LijonHcCfR06s8ssP9WPAZjZS/WMaLYh6bvDF9Cuv+0Zj2/YS I18dBbESbOmPV6T8ulpwJn8cJFGvILgUOJ/giZPqbcDtzeOrMNyH9yFJcONi+UkQ9Z PULs1XNrJvHcKm5ynhHyZFv4pPH9gdcNWYzagudpDjlQQNiZM4eut00XespXbA7TlA XsI/fnCEPFFtwj5lcFPithCaWz4NPm5znCP+sQzO+QYFPtX2Bd+oWy945CE9AIeY9R BI56jQ3F0v36A== Received: from u-boot.org (unknown [73.34.74.121]) by mail.u-boot.org (Postfix) with ESMTPSA id 4CCA3690CB; Tue, 6 Jan 2026 15:05:57 -0700 (MST) From: Simon Glass To: U-Boot Concept Date: Tue, 6 Jan 2026 15:03:40 -0700 Message-ID: <20260106220403.2915630-23-sjg@u-boot.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260106220403.2915630-1-sjg@u-boot.org> References: <20260106220403.2915630-1-sjg@u-boot.org> MIME-Version: 1.0 Message-ID-Hash: OJXXUP3MLGVYYTOIGB6NBYIQ3CUN2TTQ X-Message-ID-Hash: OJXXUP3MLGVYYTOIGB6NBYIQ3CUN2TTQ X-MailFrom: sjg@u-boot.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: Simon Glass , Claude X-Mailman-Version: 3.3.10 Precedence: list Subject: [Concept] [PATCH 22/29] buildman: Split out arch config summary display List-Id: Discussion and patches related to U-Boot Concept Archived-At: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: Simon Glass Extract the 'for arch in arch_list' loop that prints configuration summaries into _print_arch_config_summary() to reduce complexity. Co-developed-by: Claude Signed-off-by: Simon Glass --- tools/buildman/builder.py | 46 +++++++++++++++++++++++++-------------- 1 file changed, 30 insertions(+), 16 deletions(-) diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py index 723d65bc76d..7217cccc687 100644 --- a/tools/buildman/builder.py +++ b/tools/buildman/builder.py @@ -1588,6 +1588,33 @@ class Builder: all_config_minus, all_config_change) return '\n'.join(lines) + def _print_arch_config_summary(self, arch, arch_config_plus, + arch_config_minus, arch_config_change): + """Print configuration summary for a single architecture + + Args: + arch (str): Architecture name + arch_config_plus (dict): Dict of added configs by arch/filename + arch_config_minus (dict): Dict of removed configs by arch/filename + arch_config_change (dict): Dict of changed configs by arch/filename + """ + lines = [] + all_plus = {} + all_minus = {} + all_change = {} + for name in self.config_filenames: + all_plus.update(arch_config_plus[arch][name]) + all_minus.update(arch_config_minus[arch][name]) + all_change.update(arch_config_change[arch][name]) + self._add_config(lines, name, + arch_config_plus[arch][name], + arch_config_minus[arch][name], + arch_config_change[arch][name]) + self._add_config(lines, 'all', all_plus, all_minus, all_change) + if lines: + tprint(f'{arch}:') + self._output_config_info(lines) + def _show_config_changes(self, board_selected, board_dict, config): """Show changes in configuration @@ -1636,22 +1663,9 @@ class Builder: lines_by_target[lines] = [target] for arch in arch_list: - lines = [] - all_plus = {} - all_minus = {} - all_change = {} - for name in self.config_filenames: - all_plus.update(arch_config_plus[arch][name]) - all_minus.update(arch_config_minus[arch][name]) - all_change.update(arch_config_change[arch][name]) - self._add_config(lines, name, - arch_config_plus[arch][name], - arch_config_minus[arch][name], - arch_config_change[arch][name]) - self._add_config(lines, 'all', all_plus, all_minus, all_change) - if lines: - tprint(f'{arch}:') - self._output_config_info(lines) + self._print_arch_config_summary(arch, arch_config_plus, + arch_config_minus, + arch_config_change) for lines, targets in lines_by_target.items(): if not lines: