From patchwork Fri Feb 11 20:23:17 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 1591813 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: bilbo.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=chromium.org header.i=@chromium.org header.a=rsa-sha256 header.s=google header.b=Sv642uiQ; dkim-atps=neutral Authentication-Results: ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de; envelope-from=u-boot-bounces@lists.denx.de; receiver=) Received: from phobos.denx.de (phobos.denx.de [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01]) (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 4JwQ774lfDz9s8q for ; Sat, 12 Feb 2022 07:23:47 +1100 (AEDT) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id F298E83BB0; Fri, 11 Feb 2022 21:23:38 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=chromium.org header.i=@chromium.org header.b="Sv642uiQ"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 33D9283BA0; Fri, 11 Feb 2022 21:23:37 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de X-Spam-Level: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.2 Received: from mail-io1-xd32.google.com (mail-io1-xd32.google.com [IPv6:2607:f8b0:4864:20::d32]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 9E27683A81 for ; Fri, 11 Feb 2022 21:23:33 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=chromium.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=sjg@chromium.org Received: by mail-io1-xd32.google.com with SMTP id d188so12811961iof.7 for ; Fri, 11 Feb 2022 12:23:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=MHoFLZEM9v0bi2n7kdrvoaTOElxbwwXs29Dw3/Su7WA=; b=Sv642uiQE8Fqq/is7dIkTIraf4hIaNJ3vCF6qNSdWiZ13eIdmmBXN0yX4hQgLnJ07l Q/I1BAi+L4/VuKHL3fAKork9dAnKYmcz3THpsJvu0Al0FdTeCAbc8mEODVd2Z8V7SJlH RLWb2z/zRtIj21t9gYBw0gkrxm8DFf+n2AG+w= 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=MHoFLZEM9v0bi2n7kdrvoaTOElxbwwXs29Dw3/Su7WA=; b=6IZZG/9e5Kw76Lx5lt2nN1RrWrRB+kKurzAqyzlKKOfwtcKvvDzpJvZJjd5M0/APIx BrBjCcKtjDWYIrENXubZa8dHkhFnk03hT93xqudQ3JO5bpOFkGB+/m3+12RQkGqAr/pa D6Yu1vszZOhEEboeqc2n4ayf8fbcQVMTKQ+R0Q/b5QPVvuP0CxQsQ6Zbd34HIqZAjPrn dI1SkdFTR2qvhAsv80mNkhK+PcOEVyYkPZfaO6Tqz5JxByt7McaKijJOIE7BTeeofsd+ R6XePBp/PY3MJJBQ9YnOqOP1KF/sNLSKMdoP6jhP2BEIW28bR2ZJOLHw2ZKDlIcu2xBJ HG4Q== X-Gm-Message-State: AOAM533ftm34HetYLtaJ7KSH4Lr11SCjjTVrGMwiQVTpsWvJZvlQRCUg e7Spxr1d8JtcR0PIbjMUm58QDE0NkYNa6g== X-Google-Smtp-Source: ABdhPJx3DeRq6DNmfU20h2fdiTxOtxy4zlmkd7mTIUfdVy9a+SYygTogrWbWBeLKJ9BqZG5qdIL2rg== X-Received: by 2002:a05:6638:24cd:: with SMTP id y13mr1820647jat.45.1644611012260; Fri, 11 Feb 2022 12:23:32 -0800 (PST) Received: from kiwi.bld.corp.google.com (c-67-190-101-114.hsd1.co.comcast.net. [67.190.101.114]) by smtp.gmail.com with ESMTPSA id t2sm9496071ilp.49.2022.02.11.12.23.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 11 Feb 2022 12:23:31 -0800 (PST) From: Simon Glass To: U-Boot Mailing List Cc: Tom Rini , Heinrich Schuchardt , Simon Glass , Alper Nebi Yasak , Arie Haenel , Bin Meng , Bruce Monroe , Daniel Schwierzeck , Heiko Thiery , Igor Opaniuk , Julien Lenoir , =?utf-8?q?Marek_Beh=C3=BAn?= , Masahiro Yamada , Patrice Chotard , Trevor Woerner , Walter Lozano Subject: [PATCH 0/9] Enable the pylint checker in CI Date: Fri, 11 Feb 2022 13:23:17 -0700 Message-Id: <20220211202327.851395-1-sjg@chromium.org> X-Mailer: git-send-email 2.35.1.265.g69c8d7142f-goog MIME-Version: 1.0 X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.5 at phobos.denx.de X-Virus-Status: Clean This series adds a new errors-only pylint check and adds it to the CI systems. It also fixes the current errors in the U-Boot Python code, disabling errors where it seems necessary. A small patch to buildman allows it to build sandbox without any changes to the default config file. Simon Glass (9): patman: Correct pylint errors buildman: Correct pylint errors dtoc: Correct pylint errors binman: Correct pylint errors moveconfig: Correct pylint errors test: Correct pylint errors Makefile: Add a way to check for pylint errors buildman: Update default config to build for sandbox Azure/GitLab CI: Add the pylint checker .azure-pipelines.yml | 22 +++++++++++++++++++++ .gitlab-ci.yml | 16 +++++++++++++++ Makefile | 10 ++++++++-- doc/develop/python_cq.rst | 11 +++++++++++ test/py/tests/test_android/test_avb.py | 2 +- test/py/tests/test_bind.py | 8 ++++---- test/py/tests/vboot_evil.py | 3 ++- test/py/u_boot_console_base.py | 8 ++++++++ tools/binman/cmdline.py | 2 +- tools/binman/control.py | 6 ++++++ tools/binman/elf_test.py | 13 ++++++------ tools/binman/entry.py | 2 ++ tools/binman/entry_test.py | 7 ++----- tools/binman/etype/blob_dtb.py | 3 +++ tools/binman/etype/blob_phase.py | 3 +++ tools/binman/etype/cbfs.py | 3 +++ tools/binman/etype/fdtmap.py | 5 +++++ tools/binman/etype/files.py | 2 ++ tools/binman/etype/section.py | 1 + tools/binman/etype/u_boot_dtb_with_ucode.py | 3 +++ tools/binman/ftest.py | 21 ++------------------ tools/buildman/bsettings.py | 1 + tools/buildman/builder.py | 2 +- tools/buildman/cfgutil.py | 6 +++--- tools/buildman/func_test.py | 6 ------ tools/buildman/kconfiglib.py | 1 + tools/buildman/main.py | 4 ++-- tools/concurrencytest/__init__.py | 0 tools/dtoc/test_dtoc.py | 6 +++--- tools/dtoc/test_fdt.py | 6 +++--- tools/moveconfig.py | 5 +---- tools/patman/checkpatch.py | 4 +++- tools/patman/command.py | 8 +------- tools/patman/commit.py | 2 +- tools/patman/cros_subprocess.py | 18 ++++------------- tools/patman/func_test.py | 10 ++++++++++ tools/patman/patchstream.py | 2 +- tools/patman/series.py | 3 +-- tools/patman/settings.py | 4 ++-- tools/patman/tools.py | 10 +++++----- 40 files changed, 154 insertions(+), 95 deletions(-) create mode 100644 tools/concurrencytest/__init__.py