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 X-Spam-Level: X-Spam-Status: No, score=-12.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2F08DC432BE for ; Tue, 31 Aug 2021 03:16:09 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 59DDD60E90 for ; Tue, 31 Aug 2021 03:16:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 59DDD60E90 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.denx.de Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 32F2C833F4; Tue, 31 Aug 2021 05:16:06 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="aY07NsDv"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id DBBA2833CF; Tue, 31 Aug 2021 05:16:03 +0200 (CEST) Received: from mail-ed1-x531.google.com (mail-ed1-x531.google.com [IPv6:2a00:1450:4864:20::531]) (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 1891D833CF for ; Tue, 31 Aug 2021 05:15:58 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=daniel.schwierzeck@gmail.com Received: by mail-ed1-x531.google.com with SMTP id n11so24591324edv.11 for ; Mon, 30 Aug 2021 20:15:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:subject:from:to:cc:date:in-reply-to:references :user-agent:mime-version:content-transfer-encoding; bh=fbttfVAoXKSQTarMHMXPBxQSBnGjTQUilojyQO6Gdao=; b=aY07NsDvtgJXZpnf44niZRfSuc35dcUU8FeciC+3Bi+LhQXwTjnRw5/qJWvttqJMkX z095ngXVRe9AFAVtOnsGnBTEE8qCOW97Y85oOQsxsG0cYKxhYAgoRe5t7KLpfp0m1Ifm hWIIBN1Zd/e0WZYMbRp5Aj550IdzNYL4fx6K7kjXgWB8Kp0aXMUCoB6mjjmlSNHfW6d0 srT/D9bzl7dynl/kX7cS/9S6gRmjaYmWydjtxuvcESWRwJCyHGVy2+cdoNh/I+eZdtMz 2QBdt+wLt2Z/lsLl49hmaAr8732oavTPnr1OUlT5WSlmpJeYrsdJizBVOwh01CQQgL7O jXMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=fbttfVAoXKSQTarMHMXPBxQSBnGjTQUilojyQO6Gdao=; b=cGPxuObZxv6N+Q3MAVbiCEPA0VBpjzKXdaRWvoPPeUjCxtSdcrnDnYVA5QHx0nPE+C GBFnqkX9zXrpr4qvJUngGerr1mQT7B6xW/3VS1LP3h0uMS/9hThL2Gp6aIMorlwVlME2 CHz+6pB4bvhBvv/q7VGaSo0kMtlDpPalVOPgC7+WQraCbtXZmBUMbDl8xH+Qq3F4ychZ opg0CeZDUzE8q2FPwHRZwH1msE8+AUMYAhfyHO8w5UKFSE5SDmb7cRZAA7OhwoxTmwNz b4CkPwv8Po6bY6egHje0KTl1BUs9yRRC9n+XZeH1AmKaZ38xN5m2yU1v7QZ2hfAVdMk5 TtUg== X-Gm-Message-State: AOAM532VHfT5HgKXSZwO+JeqEaQYRBw37o+CJR/ekBKc0ruSRG6oQt6X 6CjS7uuoqwl/rX4SXFwxvcM= X-Google-Smtp-Source: ABdhPJyxQnCpfbndJHnVhxFL1j7mhOtJJjTLTDHSeVcxKTRwLeSgFFSyhG9+Jl9ub5P3cF4a8r/WGw== X-Received: by 2002:aa7:d610:: with SMTP id c16mr19146100edr.286.1630379757662; Mon, 30 Aug 2021 20:15:57 -0700 (PDT) Received: from unifi.lan.schwierd.dedyn.io (i5E860708.versanet.de. [94.134.7.8]) by smtp.googlemail.com with ESMTPSA id o12sm8455198edv.19.2021.08.30.20.15.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Aug 2021 20:15:56 -0700 (PDT) Message-ID: <18689a049708b4294ae0e1db7c5cbbeebf08edbb.camel@gmail.com> Subject: Re: [PATCH] Convert CONFIG_SKIP_LOWLEVEL_INIT et al to Kconfig From: Daniel Schwierzeck To: Tom Rini Cc: u-boot@lists.denx.de Date: Tue, 31 Aug 2021 05:15:56 +0200 In-Reply-To: <20210831024836.GZ858@bill-the-cat> References: <20210828011830.13238-1-trini@konsulko.com> <787c16cf4e453d07f8947b26c635a746b7678480.camel@gmail.com> <20210831024836.GZ858@bill-the-cat> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.36.5-0ubuntu1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.34 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.2 at phobos.denx.de X-Virus-Status: Clean Am Montag, dem 30.08.2021 um 22:48 -0400 schrieb Tom Rini: > On Tue, Aug 31, 2021 at 04:45:10AM +0200, Daniel Schwierzeck wrote: > > Am Freitag, dem 27.08.2021 um 21:18 -0400 schrieb Tom Rini: > > > This converts the following to Kconfig: > > > CONFIG_SKIP_LOWLEVEL_INIT > > > CONFIG_SKIP_LOWLEVEL_INIT_ONLY > > > > > > In order to do this, we need to introduce SPL and TPL variants of > > > these > > > options so that we can clearly disable these options only in SPL > > > in > > > some > > > cases, and both instances in other cases. > > > > > > Signed-off-by: Tom Rini > > > --- > > > README | 16 ------ > > > arch/Kconfig | 57 > > > +++++++++++++++++++ > > > arch/arm/cpu/arm1136/start.S | 8 +-- > > > arch/arm/cpu/arm720t/start.S | 10 ++-- > > > arch/arm/cpu/arm920t/start.S | 8 +-- > > > arch/arm/cpu/arm926ejs/start.S | 8 +-- > > > arch/arm/cpu/arm946es/start.S | 6 +- > > > arch/arm/cpu/armv7/Makefile | 2 +- > > > arch/arm/cpu/armv7/start.S | 8 +-- > > > arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 6 ++ > > > arch/arm/cpu/pxa/start.S | 6 +- > > > arch/arm/cpu/sa1100/start.S | 4 +- > > > arch/arm/include/asm/arch-am33xx/chilisom.h | 2 +- > > > arch/arm/mach-at91/arm920t/lowlevel_init.S | 4 +- > > > arch/arm/mach-imx/mx7/soc.c | 2 +- > > > arch/arm/mach-imx/syscounter.c | 2 +- > > > arch/arm/mach-mvebu/include/mach/config.h | 4 -- > > > arch/arm/mach-omap2/Kconfig | 1 + > > > arch/arm/mach-omap2/am33xx/Makefile | 2 +- > > > arch/arm/mach-omap2/am33xx/board.c | 6 +- > > > arch/arm/mach-omap2/am33xx/chilisom.c | 4 +- > > > arch/arm/mach-omap2/omap3/board.c | 4 +- > > > arch/arm/mach-omap2/omap3/lowlevel_init.S | 4 +- > > > arch/arm/mach-orion5x/Makefile | 2 +- > > > arch/arm/mach-rockchip/Kconfig | 1 + > > > arch/arm/mach-tegra/Kconfig | 1 + > > > arch/mips/cpu/start.S | 4 +- > > > arch/mips/mach-mtmips/mt7628/lowlevel_init.S | 4 +- > > > arch/nds32/cpu/n1213/ae3xx/lowlevel_init.S | 4 +- > > > arch/nds32/cpu/n1213/ag101/lowlevel_init.S | 4 +- > > > arch/nds32/cpu/n1213/start.S | 2 +- > > > board/bosch/guardian/Makefile | 2 +- > > > board/bosch/guardian/board.c | 2 +- > > > board/eets/pdu001/Makefile | 2 +- > > > board/eets/pdu001/board.c | 4 +- > > > board/freescale/ls1021aqds/README | 5 +- > > > board/freescale/ls1021atwr/README | 5 +- > > > board/grinn/chiliboard/board.c | 6 +- > > > board/qca/ap152/ap152.c | 2 +- > > > board/tcl/sl50/Makefile | 2 +- > > > board/tcl/sl50/board.c | 2 +- > > > board/ti/am335x/Makefile | 2 +- > > > board/ti/am335x/board.c | 2 +- > > > board/ti/am43xx/Makefile | 2 +- > > > board/ti/am43xx/board.c | 2 +- > > > board/tplink/wdr4300/wdr4300.c | 2 +- > > > board/vscom/baltos/Makefile | 2 +- > > > configs/SBx81LIFKW_defconfig | 1 + > > > configs/SBx81LIFXCAT_defconfig | 1 + > > > configs/adp-ae3xx_defconfig | 1 + > > > configs/adp-ag101p_defconfig | 1 + > > > configs/am43xx_evm_defconfig | 1 + > > > configs/am43xx_evm_rtconly_defconfig | 1 + > > > configs/am43xx_evm_usbhost_boot_defconfig | 1 + > > > configs/am43xx_hs_evm_defconfig | 1 + > > > configs/am64x_evm_a53_defconfig | 1 + > > > configs/am65x_evm_a53_defconfig | 1 + > > > configs/am65x_hs_evm_a53_defconfig | 1 + > > > configs/arndale_defconfig | 2 + > > > configs/aspenite_defconfig | 1 + > > > configs/at91sam9260ek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9260ek_dataflash_cs1_defconfig | 1 + > > > configs/at91sam9260ek_nandflash_defconfig | 1 + > > > configs/at91sam9261ek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9261ek_dataflash_cs3_defconfig | 1 + > > > configs/at91sam9261ek_nandflash_defconfig | 1 + > > > configs/at91sam9263ek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9263ek_dataflash_defconfig | 1 + > > > configs/at91sam9263ek_nandflash_defconfig | 1 + > > > configs/at91sam9263ek_norflash_defconfig | 1 + > > > configs/at91sam9g10ek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9g10ek_dataflash_cs3_defconfig | 1 + > > > configs/at91sam9g10ek_nandflash_defconfig | 1 + > > > configs/at91sam9g20ek_2mmc_defconfig | 1 + > > > .../at91sam9g20ek_2mmc_nandflash_defconfig | 1 + > > > configs/at91sam9g20ek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9g20ek_dataflash_cs1_defconfig | 1 + > > > configs/at91sam9g20ek_nandflash_defconfig | 1 + > > > configs/at91sam9m10g45ek_mmc_defconfig | 1 + > > > configs/at91sam9m10g45ek_nandflash_defconfig | 1 + > > > configs/at91sam9n12ek_mmc_defconfig | 1 + > > > configs/at91sam9n12ek_nandflash_defconfig | 1 + > > > configs/at91sam9n12ek_spiflash_defconfig | 1 + > > > configs/at91sam9rlek_dataflash_defconfig | 1 + > > > configs/at91sam9rlek_mmc_defconfig | 1 + > > > configs/at91sam9rlek_nandflash_defconfig | 1 + > > > configs/at91sam9x5ek_dataflash_defconfig | 1 + > > > configs/at91sam9x5ek_mmc_defconfig | 1 + > > > configs/at91sam9x5ek_nandflash_defconfig | 1 + > > > configs/at91sam9x5ek_spiflash_defconfig | 1 + > > > configs/at91sam9xeek_dataflash_cs0_defconfig | 1 + > > > configs/at91sam9xeek_dataflash_cs1_defconfig | 1 + > > > configs/at91sam9xeek_nandflash_defconfig | 1 + > > > configs/axm_defconfig | 2 + > > > configs/bcm7260_defconfig | 1 + > > > configs/bcm7445_defconfig | 1 + > > > configs/bcm963158_ram_defconfig | 1 + > > > configs/bcm968360bg_ram_defconfig | 1 + > > > configs/bcm968380gerg_ram_defconfig | 3 + > > > configs/bcm968580xref_ram_defconfig | 1 + > > > configs/bk4r1_defconfig | 1 + > > > configs/chromebit_mickey_defconfig | 1 + > > > configs/chromebook_bob_defconfig | 1 + > > > configs/chromebook_jerry_defconfig | 1 + > > > configs/chromebook_minnie_defconfig | 1 + > > > configs/chromebook_speedy_defconfig | 1 + > > > configs/ci20_mmc_defconfig | 3 + > > > configs/clearfog_defconfig | 1 + > > > configs/cm_t43_defconfig | 1 + > > > configs/colibri_vf_defconfig | 1 + > > > configs/comtrend_ar5315u_ram_defconfig | 3 + > > > configs/comtrend_ar5387un_ram_defconfig | 3 + > > > configs/comtrend_ct5361_ram_defconfig | 3 + > > > configs/comtrend_vr3032u_ram_defconfig | 3 + > > > configs/comtrend_wap5813n_ram_defconfig | 3 + > > > configs/controlcenterdc_defconfig | 2 + > > > configs/corvus_defconfig | 2 + > > > configs/d2net_v2_defconfig | 1 + > > > configs/da850evm_defconfig | 2 + > > > configs/da850evm_direct_nor_defconfig | 1 + > > > configs/da850evm_nand_defconfig | 2 + > > > configs/db-88f6720_defconfig | 1 + > > > configs/db-88f6820-amc_defconfig | 1 + > > > configs/db-88f6820-gp_defconfig | 1 + > > > configs/devkit3250_defconfig | 1 + > > > configs/dns325_defconfig | 1 + > > > configs/dockstar_defconfig | 1 + > > > configs/dreamplug_defconfig | 1 + > > > configs/ds109_defconfig | 1 + > > > configs/ea-lpc3250devkitv2_defconfig | 1 + > > > configs/elgin-rv1108_defconfig | 1 + > > > configs/ethernut5_defconfig | 1 + > > > configs/evb-px30_defconfig | 1 + > > > configs/evb-px5_defconfig | 1 + > > > configs/evb-rk3036_defconfig | 2 + > > > configs/evb-rk3128_defconfig | 1 + > > > configs/evb-rk3229_defconfig | 3 + > > > configs/evb-rk3288_defconfig | 2 + > > > configs/evb-rk3308_defconfig | 1 + > > > configs/evb-rk3328_defconfig | 1 + > > > configs/evb-rk3399_defconfig | 1 + > > > configs/evb-rk3568_defconfig | 1 + > > > configs/evb-rv1108_defconfig | 1 + > > > configs/ficus-rk3399_defconfig | 1 + > > > configs/firefly-px30_defconfig | 1 + > > > configs/firefly-rk3288_defconfig | 2 + > > > configs/firefly-rk3399_defconfig | 1 + > > > .../gardena-smart-gateway-at91sam_defconfig | 2 + > > > .../gardena-smart-gateway-mt7688_defconfig | 3 + > > > configs/geekbox_defconfig | 1 + > > > configs/goflexhome_defconfig | 1 + > > > configs/gurnard_defconfig | 1 + > > > configs/guruplug_defconfig | 1 + > > > configs/helios4_defconfig | 1 + > > > configs/highbank_defconfig | 1 + > > > configs/huawei_hg556a_ram_defconfig | 3 + > > > configs/ib62x0_defconfig | 1 + > > > configs/iconnect_defconfig | 1 + > > > configs/imgtec_xilfpga_defconfig | 3 + > > > configs/inetspace_v2_defconfig | 1 + > > > configs/k2e_evm_defconfig | 2 + > > > configs/k2e_hs_evm_defconfig | 1 + > > > configs/k2g_evm_defconfig | 2 + > > > configs/k2g_hs_evm_defconfig | 1 + > > > configs/k2hk_evm_defconfig | 2 + > > > configs/k2hk_hs_evm_defconfig | 1 + > > > configs/k2l_evm_defconfig | 2 + > > > configs/k2l_hs_evm_defconfig | 1 + > > > configs/khadas-edge-captain-rk3399_defconfig | 1 + > > > configs/khadas-edge-rk3399_defconfig | 1 + > > > configs/khadas-edge-v-rk3399_defconfig | 1 + > > > configs/km_kirkwood_128m16_defconfig | 1 + > > > configs/km_kirkwood_defconfig | 1 + > > > configs/km_kirkwood_pci_defconfig | 1 + > > > configs/kmcoge5un_defconfig | 1 + > > > configs/kmnusa_defconfig | 1 + > > > configs/kmsuse2_defconfig | 1 + > > > configs/kylin-rk3036_defconfig | 2 + > > > configs/leez-rk3399_defconfig | 1 + > > > configs/legoev3_defconfig | 1 + > > > configs/linkit-smart-7688_defconfig | 3 + > > > configs/lion-rk3368_defconfig | 1 + > > > configs/ls1021aqds_ddr4_nor_defconfig | 1 + > > > configs/ls1021aqds_ddr4_nor_lpuart_defconfig | 1 + > > > configs/ls1021aqds_nand_defconfig | 2 + > > > configs/ls1021aqds_nor_SECURE_BOOT_defconfig | 1 + > > > configs/ls1021aqds_nor_defconfig | 1 + > > > configs/ls1021aqds_nor_lpuart_defconfig | 1 + > > > configs/ls1021aqds_qspi_defconfig | 1 + > > > configs/ls1021aqds_sdcard_ifc_defconfig | 2 + > > > configs/ls1021aqds_sdcard_qspi_defconfig | 2 + > > > configs/ls1021atwr_nor_SECURE_BOOT_defconfig | 1 + > > > configs/ls1021atwr_nor_defconfig | 1 + > > > configs/ls1021atwr_nor_lpuart_defconfig | 1 + > > > configs/ls1021atwr_qspi_defconfig | 1 + > > > ...s1021atwr_sdcard_ifc_SECURE_BOOT_defconfig | 2 + > > > configs/ls1021atwr_sdcard_ifc_defconfig | 2 + > > > configs/ls1021atwr_sdcard_qspi_defconfig | 2 + > > > configs/ls1028aqds_tfa_SECURE_BOOT_defconfig | 1 + > > > configs/ls1028aqds_tfa_defconfig | 1 + > > > configs/ls1028aqds_tfa_lpuart_defconfig | 1 + > > > configs/ls1028ardb_tfa_SECURE_BOOT_defconfig | 1 + > > > configs/ls1028ardb_tfa_defconfig | 1 + > > > configs/lschlv2_defconfig | 1 + > > > configs/lsxhl_defconfig | 1 + > > > configs/lx2160aqds_tfa_SECURE_BOOT_defconfig | 1 + > > > configs/lx2160aqds_tfa_defconfig | 1 + > > > configs/lx2160ardb_tfa_SECURE_BOOT_defconfig | 1 + > > > configs/lx2160ardb_tfa_defconfig | 1 + > > > configs/lx2160ardb_tfa_stmm_defconfig | 1 + > > > configs/lx2162aqds_tfa_SECURE_BOOT_defconfig | 1 + > > > configs/lx2162aqds_tfa_defconfig | 1 + > > > .../lx2162aqds_tfa_verified_boot_defconfig | 1 + > > > configs/meesc_dataflash_defconfig | 1 + > > > configs/meesc_defconfig | 1 + > > > configs/miqi-rk3288_defconfig | 1 + > > > configs/mt7620_mt7530_rfb_defconfig | 3 + > > > configs/mt7620_rfb_defconfig | 3 + > > > configs/mt7628_rfb_defconfig | 3 + > > > configs/nanopc-t4-rk3399_defconfig | 1 + > > > configs/nanopi-m4-2gb-rk3399_defconfig | 1 + > > > configs/nanopi-m4-rk3399_defconfig | 1 + > > > configs/nanopi-m4b-rk3399_defconfig | 1 + > > > configs/nanopi-neo4-rk3399_defconfig | 1 + > > > configs/nanopi-r2s-rk3328_defconfig | 1 + > > > configs/nanopi-r4s-rk3399_defconfig | 1 + > > > configs/nas220_defconfig | 1 + > > > configs/net2big_v2_defconfig | 1 + > > > configs/netgear_cg3100d_ram_defconfig | 3 + > > > configs/netgear_dgnd3700v2_ram_defconfig | 3 + > > > configs/netspace_lite_v2_defconfig | 1 + > > > configs/netspace_max_v2_defconfig | 1 + > > > configs/netspace_mini_v2_defconfig | 1 + > > > configs/netspace_v2_defconfig | 1 + > > > configs/nokia_rx51_defconfig | 1 + > > > configs/nsa310s_defconfig | 1 + > > > configs/odroid-go2_defconfig | 1 + > > > configs/odroid-xu3_defconfig | 1 + > > > configs/odroid_defconfig | 1 + > > > configs/omapl138_lcdk_defconfig | 2 + > > > configs/openrd_base_defconfig | 1 + > > > configs/openrd_client_defconfig | 1 + > > > configs/openrd_ultimate_defconfig | 1 + > > > configs/orangepi-rk3399_defconfig | 1 + > > > configs/origen_defconfig | 2 + > > > configs/pcm052_defconfig | 1 + > > > configs/peach-pi_defconfig | 2 + > > > configs/peach-pit_defconfig | 2 + > > > configs/pg_wcom_expu1_defconfig | 1 + > > > configs/pg_wcom_seli8_defconfig | 1 + > > > configs/phycore-rk3288_defconfig | 1 + > > > configs/pinebook-pro-rk3399_defconfig | 1 + > > > configs/pm9g45_defconfig | 1 + > > > configs/pogo_e02_defconfig | 1 + > > > configs/popmetal-rk3288_defconfig | 1 + > > > configs/puma-rk3399_defconfig | 1 + > > > configs/px30-core-ctouch2-px30_defconfig | 1 + > > > configs/px30-core-edimm2.2-px30_defconfig | 1 + > > > configs/roc-cc-rk3308_defconfig | 1 + > > > configs/roc-cc-rk3328_defconfig | 1 + > > > configs/roc-pc-mezzanine-rk3399_defconfig | 1 + > > > configs/roc-pc-rk3399_defconfig | 1 + > > > configs/rock-pi-4-rk3399_defconfig | 1 + > > > configs/rock-pi-4c-rk3399_defconfig | 1 + > > > configs/rock-pi-e-rk3328_defconfig | 1 + > > > configs/rock-pi-n10-rk3399pro_defconfig | 1 + > > > configs/rock-pi-n8-rk3288_defconfig | 2 + > > > configs/rock2_defconfig | 1 + > > > configs/rock64-rk3328_defconfig | 1 + > > > configs/rock960-rk3399_defconfig | 1 + > > > configs/rock_defconfig | 2 + > > > configs/rockpro64-rk3399_defconfig | 1 + > > > configs/rpi_2_defconfig | 1 + > > > configs/rpi_3_32b_defconfig | 1 + > > > configs/s5pc210_universal_defconfig | 1 + > > > configs/sagem_f@st1704_ram_defconfig | 3 + > > > configs/sam9x60ek_mmc_defconfig | 1 + > > > configs/sam9x60ek_nandflash_defconfig | 1 + > > > configs/sam9x60ek_qspiflash_defconfig | 1 + > > > configs/sama5d27_giantboard_defconfig | 1 + > > > configs/sama5d27_som1_ek_mmc1_defconfig | 1 + > > > configs/sama5d27_som1_ek_mmc_defconfig | 1 + > > > configs/sama5d27_som1_ek_qspiflash_defconfig | 1 + > > > configs/sama5d27_wlsom1_ek_mmc_defconfig | 1 + > > > .../sama5d27_wlsom1_ek_qspiflash_defconfig | 1 + > > > configs/sama5d2_icp_mmc_defconfig | 1 + > > > configs/sama5d2_ptc_ek_mmc_defconfig | 1 + > > > configs/sama5d2_ptc_ek_nandflash_defconfig | 1 + > > > configs/sama5d2_xplained_emmc_defconfig | 1 + > > > configs/sama5d2_xplained_mmc_defconfig | 1 + > > > configs/sama5d2_xplained_qspiflash_defconfig | 1 + > > > configs/sama5d2_xplained_spiflash_defconfig | 1 + > > > configs/sama5d36ek_cmp_mmc_defconfig | 1 + > > > configs/sama5d36ek_cmp_nandflash_defconfig | 1 + > > > configs/sama5d36ek_cmp_spiflash_defconfig | 1 + > > > configs/sama5d3_xplained_mmc_defconfig | 1 + > > > configs/sama5d3_xplained_nandflash_defconfig | 1 + > > > configs/sama5d3xek_mmc_defconfig | 1 + > > > configs/sama5d3xek_nandflash_defconfig | 1 + > > > configs/sama5d3xek_spiflash_defconfig | 1 + > > > configs/sama5d4_xplained_mmc_defconfig | 1 + > > > configs/sama5d4_xplained_nandflash_defconfig | 1 + > > > configs/sama5d4_xplained_spiflash_defconfig | 1 + > > > configs/sama5d4ek_mmc_defconfig | 1 + > > > configs/sama5d4ek_nandflash_defconfig | 1 + > > > configs/sama5d4ek_spiflash_defconfig | 1 + > > > configs/sfr_nb4-ser_ram_defconfig | 3 + > > > configs/sheep-rk3368_defconfig | 1 + > > > configs/sheevaplug_defconfig | 1 + > > > configs/smartweb_defconfig | 2 + > > > configs/smdk5250_defconfig | 2 + > > > configs/smdk5420_defconfig | 2 + > > > configs/smdkv310_defconfig | 2 + > > > configs/snapper9260_defconfig | 1 + > > > configs/snapper9g20_defconfig | 1 + > > > configs/snow_defconfig | 2 + > > > configs/spring_defconfig | 2 + > > > configs/stemmy_defconfig | 1 + > > > configs/stih410-b2260_defconfig | 1 + > > > configs/taurus_defconfig | 2 + > > > configs/ti816x_evm_defconfig | 1 + > > > configs/tinker-rk3288_defconfig | 2 + > > > configs/tinker-s-rk3288_defconfig | 2 + > > > configs/trats2_defconfig | 1 + > > > configs/trats_defconfig | 1 + > > > configs/turris_omnia_defconfig | 1 + > > > configs/usb_a9263_dataflash_defconfig | 1 + > > > configs/vf610twr_defconfig | 1 + > > > configs/vf610twr_nand_defconfig | 1 + > > > configs/vinco_defconfig | 1 + > > > configs/vocore2_defconfig | 3 + > > > configs/vyasa-rk3288_defconfig | 2 + > > > configs/work_92105_defconfig | 1 + > > > configs/x530_defconfig | 1 + > > > configs/xilinx_zynqmp_r5_defconfig | 1 + > > > configs/zynq_cse_nand_defconfig | 2 + > > > configs/zynq_cse_nor_defconfig | 2 + > > > configs/zynq_cse_qspi_defconfig | 2 + > > > doc/board/AndesTech/adp-ag101p.rst | 4 +- > > > include/configs/SBx81LIFKW.h | 1 - > > > include/configs/SBx81LIFXCAT.h | 1 - > > > include/configs/adp-ae3xx.h | 2 - > > > include/configs/adp-ag101p.h | 2 - > > > include/configs/am43xx_evm.h | 8 --- > > > include/configs/am64x_evm.h | 4 -- > > > include/configs/am65x_evm.h | 4 -- > > > include/configs/aspenite.h | 1 - > > > include/configs/at91-sama5_common.h | 4 -- > > > include/configs/at91sam9260ek.h | 1 - > > > include/configs/at91sam9261ek.h | 2 - > > > include/configs/at91sam9263ek.h | 1 - > > > include/configs/at91sam9m10g45ek.h | 1 - > > > include/configs/at91sam9n12ek.h | 1 - > > > include/configs/at91sam9rlek.h | 2 - > > > include/configs/at91sam9x5ek.h | 1 - > > > include/configs/bcmstb.h | 1 - > > > include/configs/bk4r1.h | 2 - > > > include/configs/bmips_bcm3380.h | 1 - > > > include/configs/bmips_bcm6318.h | 1 - > > > include/configs/bmips_bcm63268.h | 1 - > > > include/configs/bmips_bcm6328.h | 1 - > > > include/configs/bmips_bcm6338.h | 1 - > > > include/configs/bmips_bcm6348.h | 1 - > > > include/configs/bmips_bcm6358.h | 1 - > > > include/configs/bmips_bcm6362.h | 1 - > > > include/configs/bmips_bcm6368.h | 1 - > > > include/configs/bmips_bcm6838.h | 1 - > > > include/configs/broadcom_bcm963158.h | 2 - > > > include/configs/broadcom_bcm968360bg.h | 2 - > > > include/configs/broadcom_bcm968580xref.h | 2 - > > > include/configs/bur_am335x_common.h | 3 - > > > include/configs/ci20.h | 2 - > > > include/configs/cm_t43.h | 3 - > > > include/configs/colibri_vf.h | 2 - > > > include/configs/controlcenterdc.h | 2 - > > > include/configs/corvus.h | 1 - > > > include/configs/da850evm.h | 1 - > > > include/configs/devkit3250.h | 4 -- > > > include/configs/dns325.h | 1 - > > > include/configs/dockstar.h | 1 - > > > include/configs/ea-lpc3250devkitv2.h | 1 - > > > include/configs/ethernut5.h | 1 - > > > include/configs/exynos-common.h | 2 - > > > .../configs/gardena-smart-gateway-at91sam.h | 5 -- > > > .../configs/gardena-smart-gateway-mt7688.h | 3 - > > > include/configs/goflexhome.h | 1 - > > > include/configs/highbank.h | 1 - > > > include/configs/ib62x0.h | 1 - > > > include/configs/iconnect.h | 1 - > > > include/configs/imgtec_xilfpga.h | 1 - > > > include/configs/integrator-common.h | 1 - > > > include/configs/km/km_arm.h | 2 - > > > include/configs/km/pg-wcom-ls102xa.h | 2 - > > > include/configs/lacie_kw.h | 1 - > > > include/configs/legoev3.h | 1 - > > > include/configs/linkit-smart-7688.h | 3 - > > > include/configs/ls1012a_common.h | 2 - > > > include/configs/ls1021aqds.h | 2 - > > > include/configs/ls1021atwr.h | 1 - > > > include/configs/ls1028a_common.h | 2 - > > > include/configs/ls1043a_common.h | 2 - > > > include/configs/ls1043aqds.h | 2 - > > > include/configs/ls1046a_common.h | 2 - > > > include/configs/ls1046aqds.h | 2 - > > > include/configs/ls1088a_common.h | 2 - > > > include/configs/ls2080a_common.h | 2 - > > > include/configs/lsxl.h | 2 - > > > include/configs/lx2160a_common.h | 2 - > > > include/configs/meesc.h | 1 - > > > include/configs/mt7620.h | 3 - > > > include/configs/mt7628.h | 3 - > > > include/configs/mv-plug-common.h | 1 - > > > include/configs/mx7_common.h | 5 -- > > > include/configs/nas220.h | 1 - > > > include/configs/nokia_rx51.h | 2 - > > > include/configs/nsa310s.h | 1 - > > > include/configs/omapl138_lcdk.h | 1 - > > > include/configs/openrd.h | 1 - > > > include/configs/pcm052.h | 2 - > > > include/configs/pm9261.h | 2 - > > > include/configs/pm9263.h | 1 - > > > include/configs/pm9g45.h | 1 - > > > include/configs/pogo_e02.h | 1 - > > > include/configs/px30_common.h | 1 - > > > include/configs/rk3036_common.h | 1 - > > > include/configs/rk3128_common.h | 1 - > > > include/configs/rk3188_common.h | 1 - > > > include/configs/rk322x_common.h | 1 - > > > include/configs/rk3288_common.h | 1 - > > > include/configs/rk3308_common.h | 1 - > > > include/configs/rk3328_common.h | 1 - > > > include/configs/rk3368_common.h | 1 - > > > include/configs/rk3399_common.h | 1 - > > > include/configs/rk3568_common.h | 1 - > > > include/configs/rpi.h | 4 -- > > > include/configs/rv1108_common.h | 1 - > > > include/configs/sam9x60ek.h | 1 - > > > include/configs/siemens-am33x-common.h | 3 - > > > include/configs/smartweb.h | 1 - > > > include/configs/smdkv310.h | 1 - > > > include/configs/snapper9260.h | 1 - > > > include/configs/snapper9g45.h | 1 - > > > include/configs/stemmy.h | 1 - > > > include/configs/stih410-b2260.h | 2 - > > > include/configs/taurus.h | 2 - > > > include/configs/tegra-common-post.h | 3 - > > > include/configs/ti814x_evm.h | 3 - > > > include/configs/ti816x_evm.h | 3 - > > > include/configs/ti_am335x_common.h | 3 - > > > include/configs/ti_armv7_keystone2.h | 1 - > > > include/configs/usb_a9263.h | 2 - > > > include/configs/vf610twr.h | 2 - > > > include/configs/vocore2.h | 3 - > > > include/configs/work_92105.h | 4 -- > > > include/configs/xilinx_zynqmp_r5.h | 2 - > > > include/configs/zynq_cse.h | 2 - > > > scripts/config_whitelist.txt | 2 - > > > 457 files changed, 514 insertions(+), 311 deletions(-) > > > > > > > [...] > > > > > +++ b/configs/ci20_mmc_defconfig > > > @@ -1,4 +1,5 @@ > > > CONFIG_MIPS=y > > > +CONFIG_SKIP_LOWLEVEL_INIT=y > > > CONFIG_SYS_TEXT_BASE=0x80010000 > > > CONFIG_SPL_GPIO=y > > > CONFIG_SPL_LIBGENERIC_SUPPORT=y > > > @@ -11,6 +12,8 @@ CONFIG_SPL_TEXT_BASE=0xf4000a00 > > > CONFIG_SPL_MMC_SUPPORT=y > > > CONFIG_SPL=y > > > CONFIG_ARCH_JZ47XX=y > > > +CONFIG_MIPS_CACHE_SETUP=y > > > +CONFIG_MIPS_CACHE_DISABLE=y > > > CONFIG_FIT=y > > > CONFIG_USE_BOOTARGS=y > > > CONFIG_BOOTARGS="console=ttyS4,115200 rw rootwait > > > root=/dev/mmcblk0p1" > > > > > > > this doesn't look correct for this SPL enabled MIPS board and some > > others as well. Problem is the Kconfig dependency to > > CONFIG_SKIP_LOWLEVEL_INIT: > > > > config MIPS_CACHE_SETUP > > bool "Allow generic start code to initialize and setup caches" > > default n if SKIP_LOWLEVEL_INIT > > default y > > > > config MIPS_CACHE_DISABLE > > bool "Allow generic start code to initially disable caches" > > default n if SKIP_LOWLEVEL_INIT > > default y > > > > Normally both options have to be y in SPL along with > > SKIP_LOWLEVEL_INIT=n. But in U-Boot proper both options have to be > > n > > along with SKIP_LOWLEVEL_INIT=y. I guess we need to introduce > > SPL_MIPS_CACHE_SETUP and SPL_MIPS_CACHE_DISABLE or is there a > > better > > way to deal with dependencies to SKIP_LOWLEVEL_INIT and > > SPL_SKIP_LOWLEVEL_INIT? > > So, size-wise, everything is as it was before this patch is applied. > The previous logic wasn't doing anything (you can't do logic with > symbols not in Kconfig). Everything should function exactly as well > as > before. And yes, given how things build today, you will need SPL_xxx > symbols to control the SPL build, and for code that's common to both, > use CONFIG_IS_ENABLED(xxx) to evaluate the correct setting for the > build. > hm, then commit 57bfb1aed6a87b19ca7e2a76df83ea387621e2e9 didn't work as intended and both options were incorrectly always y since that. Then your patch is correct. I'll look into fixing those options. -- - Daniel