From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 31C93C61DA4 for ; Sat, 11 Mar 2023 16:55:42 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 2688485FC2; Sat, 11 Mar 2023 17:55:40 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com 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=konsulko.com header.i=@konsulko.com header.b="H9pvST1z"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 46FB085FD4; Sat, 11 Mar 2023 16:29:05 +0100 (CET) Received: from mail-qt1-x835.google.com (mail-qt1-x835.google.com [IPv6:2607:f8b0:4864:20::835]) (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 7CCEE85FD0 for ; Sat, 11 Mar 2023 16:29:00 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-qt1-x835.google.com with SMTP id l13so8972979qtv.3 for ; Sat, 11 Mar 2023 07:29:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; t=1678548539; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=0fw0ZL+8FWhyUdb7i/CrM3t5hMSuuqvCTutz+3zm15g=; b=H9pvST1z4XWvG9VfPZ+vEopTi43GypU+IG31WxACa00UuE+1ZsSWKBWx3q1r7tlJhD Kvla5hqanUuJp+sswXVRh1YClHO6DVN1qYvwGnFLZphurI+/j/Zlj7OzVc3MiJV/lpZD Vo18czrDUhbMMmXmN4uDXp/ptLIDqrseXwN8o= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678548539; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=0fw0ZL+8FWhyUdb7i/CrM3t5hMSuuqvCTutz+3zm15g=; b=DuIBaw/2NDqiQ83JnoFv5SyWUKkCV+HZkafl+Tn3zXL4doxjQLuo20B5O9Wu8QEpHP ntVokPCJnaaiyTjQavXmyP2/ubarPZCixE5kgYX59I/sZxmlgfStEsQwc8dS+0/eTRw7 doq7KGIZe2G4uiZWhJEEs14/qT4eCqGgCQLm6Rf3iwimwDpXoT2EA8jz2lU1+24IS1g1 JEFAccC1oop3T2lvlug2UhpysCd0c8rpqj77RouA2YKqoItWFNdx/m7bXWkcslN3Xdd9 y/oZisbiib4VIUsh8TJqWJAAHoRmanFTRbVlj9FkDSaBxhIYPEeBvr9pI0SFeAtCzSGf kgLg== X-Gm-Message-State: AO0yUKXeapK3/1/Y+lSGoQrkUDUwq2HGskhQwNMqxJCi/2Tt8bkixZ2Z Cs7ON4/WNPjmCwzNFMEqI73X6w== X-Google-Smtp-Source: AK7set8BRClg4N9G2VXmJhoKUtaycoz8uWqIXPMtqYv/CYfgHxgbM9S56Rmss521ZtkwuYOFv02kGA== X-Received: by 2002:a05:622a:281:b0:3b9:bc8c:c215 with SMTP id z1-20020a05622a028100b003b9bc8cc215mr8268441qtw.32.1678548539058; Sat, 11 Mar 2023 07:28:59 -0800 (PST) Received: from bill-the-cat (2603-6081-7b00-6400-0000-0000-0000-036b.res6.spectrum.com. [2603:6081:7b00:6400::36b]) by smtp.gmail.com with ESMTPSA id r200-20020a3744d1000000b0074305413c73sm1952382qka.95.2023.03.11.07.28.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 11 Mar 2023 07:28:58 -0800 (PST) Date: Sat, 11 Mar 2023 10:28:54 -0500 From: Tom Rini To: Simon Glass Cc: U-Boot Mailing List , AKASHI Takahiro , Alper Nebi Yasak , Andrew Scull , Balamanikandan Gunasundar , Baruch Siach , Bin Meng , Brandon Maier , Dario Binacchi , Dzmitry Sankouski , Fabio Estevam , Heiko Schocher , Heinrich Schuchardt , Ilias Apalodimas , Jaehoon Chung , Jeffy Chen , Jim Liu , Joe Hershberger , Johan Jonker , John Keeping , Kautuk Consul , Kever Yang , Klaus Goger , Leo Yu-Chi Liang , Lukasz Majewski , Marek =?iso-8859-1?Q?Beh=FAn?= , Marek Vasut , Mario Six , Michael Trimarchi , Michal Simek , Michal Suchanek , Nate Drude , Pali =?iso-8859-1?Q?Roh=E1r?= , Pavel Herrmann , Peng Fan , Philipp Tomsich , Philippe Reynes , Quentin Schulz , Rasmus Villemoes , Robert Marko , Samuel Dionne-Riel , Samuel Holland , Sean Anderson , Stefan Roese , Stefano Babic , Tim Harvey , Wolfgang Denk , Ye Li , huang lin Subject: Re: [PATCH v5 00/44] More tidy-ups of Kconfig options Message-ID: <20230311152854.GM3041508@bill-the-cat> References: <20230222163425.2043934-1-sjg@chromium.org> <20230303234322.GV968799@bill-the-cat> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="XH596LqFC1icpvJl" Content-Disposition: inline In-Reply-To: X-Clacks-Overhead: GNU Terry Pratchett X-Mailman-Approved-At: Sat, 11 Mar 2023 17:55:38 +0100 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.8 at phobos.denx.de X-Virus-Status: Clean --XH596LqFC1icpvJl Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Mar 06, 2023 at 11:20:42AM -0700, Simon Glass wrote: > Hi Tom, >=20 > On Fri, 3 Mar 2023 at 16:43, Tom Rini wrote: > > > > On Wed, Feb 22, 2023 at 09:33:41AM -0700, Simon Glass wrote: > > > > > This series was split out of the old 'split config' splc series. It > > > contains clean-up patches which do not depend on split config. > > > > > > This is available at u-boot-dm/spld-working > > > > > > The size changes look pretty good: https://paste.debian.net/1271742/ > > > > > > The remaining patches will move into a new 'splg' series (G for Good). > > > > > > [1] https://patchwork.ozlabs.org/project/uboot/list/?series=3D341504&= state=3D* > > > > > > Changes in v5: > > > - Fix reply typo > > > - Change approach and expand notes after more investigation > > > - Drop FSL_ISBC_KEY_EXT patch as it changes the size > > > - Drop PHY_CADENCE_SIERRA patch as it changes the size > > > > > > Changes in v4: > > > - Avoid use of def_bool > > > - Modify to get rid of def_bool > > > - Adjust Kconfig ordering > > > - Just fix the typo > > > - Reduce and rename commit > > > - Reduce and rename commit > > > - Fix 'wanderboard' typo > > > - Reduce and rename commit > > > > > > Changes in v3: > > > - Add new patch to disable QFW bootmeth in SPL > > > - Move the option down to the non-SPL part of drivers/Makefile > > > - Correct 'VPL' typo > > > - Use a consistent format for the comment > > > - Fix a transitory build error with sandbox_spl > > > - Add a new patch to disallow commands in SPL > > > > > > Changes in v2: > > > - Rebase to previous series > > > > > > Simon Glass (44): > > > mtd: Drop unused kb9202_nand driver > > > mtd: Drop unused CONFIG_ONENAND_U_BOOT > > > sh4: Drop unused twl6030 driver > > > moveconfig: Update to detect / correct missing SPL Kconfigs > > > bootstd: Disable QFW bootmeth in SPL > > > Correct SPL uses of ARCH_MVEBU > > > Correct SPL uses of DISPLAY_AER_FULL > > > Correct SPL uses of MULTIPLEXER > > > Correct SPL use of PG_WCOM_UBOOT_UPDATE_SUPPORTED > > > Correct SPL uses of PHY_FIXED > > > boot: Add Kconfigs for BOOTMETH_VBE_REQUEST > > > Correct SPL use of DM_RNG > > > lib: Add a Kconfig for SPL_BZIP2 > > > moveconfig: Various minor improvements > > > sandbox: Expand size for VPL image > > > event: Add Kconfig options for SPL > > > bootstd: Correct 'VPL' typo > > > env: Avoid checking ENV_IS_IN when env disabled > > > env: Allow VPL environment to be nowhere > > > lib: Add VPL options for SHA1 and SHA256 > > > x86: Use string functions for all 32-bit builds > > > lib: Fix build condition for tiny-printf > > > sandbox: Tidy up RTC options > > > sandbox: Use the generic VPL option to enable VPL > > > sandbox: Tidy up I2C options > > > fixdep: Add support for VPL > > > fixdep: Refactor to make testing easier > > > fixdep: Add some tests for parse_config_line() > > > test: Add SPL versions of the TEST_KCONFIG options > > > lib: Add an SPL config for LIB_UUID > > > test: Tidy up sandbox handling in test-main > > > x86: Fix up use of X86_32BIT_INIT and X86_64 options > > > Add VPL options for BLOBLIST > > > rockchip: Avoid checking environment without ENV_SUPPORT > > > freescale: Drop old pre-DM_ETH code > > > imx: Use SATA instead of CMD_SATA > > > net: Add an SPL config for atheros > > > freescale: Fix odd use of ESDHCI_QUIRK_BROKEN_TIMEOUT_VALUE > > > serial: Support ns16550 driver in TPL > > > dm: Add a TPL symbol for simple-bus > > > x86: coral: Add missing TPL options > > > power: wandboard: Add a missing CONFIG > > > venice: Simplify conditions for network init > > > command: Don't allow commands in SPL > > > > I've applied almost all of this series. I've left the moveconfig.py > > stuff for now as I'm not sure it's useful until split config. I've > > followed up about the X86_32BIT_INIT patch as I do want to see that > > boot-tested, and I've left the ns16650 TPL patch as both that feel like > > "split config makes this an issue" rather than an issue we have today > > and fwiw, the dependencies are wrong in that there's no TPL for omap2 > > which I didn't reply directly to, I only noticed as reviewing the series > > locally before merging. >=20 > OK, thanks for that. Do you think it is worth picking up the fixdep > ones also? The VPL thing is a bug and the others are for tests. >=20 > I will see if I can get some time by next week to do another spin. How > can we get more eyes on the Kconfig-language proposal? Sorry, I should have chimed in sooner. I was thinking perhaps the next step would be to re-sync our Kconfig support with 6.2 or whatever, and see what the reaction is to the refactoring patches. --=20 Tom --XH596LqFC1icpvJl Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmQMniwACgkQFHw5/5Y0 tyw3BgwAtFT4yGrzTDiHYW/cF6G3mj3J0w4vF3fEU/AmMEZpV9QkkZXoGz5fbfnP 0ODJ3sbDdvKkqMLg71hovVcLOWQpT6cGsCsOXK4WlTxoKCCBXxkObfQ9IB5OHYyA v/95FhqkzSDIL5hlVHP5Q20yCTXVPTllI4NrSOemyKgluTzw9c9QR7UtCO4iMr0u SpF8Mxgp7GbO29c0f+hpNHRMJ4JRak/3TioOusadpi9tiqR8azINvjcbMC6Jwkro gQwJqmqN/4osxcEn12DORzUxepH6x/N4nB9fcheMsXv3/qmqy/Q9WEz9pNJQT74R zm5Zq9fxXGBTjPslVpOMGp1TadLQ7BqLPIga5Wkh2fowTViaeNs6ofUaBLkPCk3T R4orwjcHLZrPa/2GlVGbl6yCTRAnoTjgk0rpqvNuOcfeyG0Eb69aXPPe/KCwR5xS 90yUREmHwoF7dw/XMl43C1o2XrcfWEF/i16otQ+w4P5T4jVK4fLvMteBlk2xHpUm aE/8ol0N =4y/H -----END PGP SIGNATURE----- --XH596LqFC1icpvJl--