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 734D7C00140 for ; Fri, 5 Aug 2022 15:51:45 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id AB3D98487F; Fri, 5 Aug 2022 17:51:43 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=kernel.org 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=kernel.org header.i=@kernel.org header.b="ieZXWMs9"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 7052D84884; Fri, 5 Aug 2022 17:51:42 +0200 (CEST) Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id DC6A58487A for ; Fri, 5 Aug 2022 17:51:39 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=kernel.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=pali@kernel.org Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 7C8DF6171E; Fri, 5 Aug 2022 15:51:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8CCFCC433D7; Fri, 5 Aug 2022 15:51:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1659714697; bh=MhkcsgzcFolUNH2+WYdMc3fb9ckvX5EeoIHnObIgpNk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ieZXWMs9zhDD/9iqq3ORrXVR9zf3JUJu8Alg3GNbBmhEa8N8c3Cg0EOvHSHkxCiBj Ix9vMpRG31RWShxLlut1ci2B8sbc6SWdohOFaqLFSQrYHQsDGebZJX4gawVkFthF3Q ItJN8+qXA+QnFnUhrkfHGsbG1eNRDF+bpVl1442g624O8t9Zso5A2v5qfaEm9OR+rE oyQDVtnJ0XZRXLPIRCFqSgGsBaO9xpcI9PtqmjBSrIsB37P6VaRSm0R7JL3LQ+XqPg lp2+/CrNcneND4TZBxnbXPnAPhviuYgVRTutoMU5EnF5bMWltki8jCUJf/Eeg29G1q CgiuDxTbbpNTA== Received: by pali.im (Postfix) id 3219D82D; Fri, 5 Aug 2022 17:51:35 +0200 (CEST) Date: Fri, 5 Aug 2022 17:51:35 +0200 From: Pali =?utf-8?B?Um9ow6Fy?= To: Tom Rini Cc: u-boot@lists.denx.de, kabel@kernel.org Subject: Re: Broken commit de47ff536363289f92f85ed1e4901724d238432d Message-ID: <20220805155135.r5dctspg2oohzdev@pali> References: <20220802091338.f4g45ldhc7qbg6hm@pali> <20220802105826.GB1146598@bill-the-cat> <20220803160013.4oh6x4glh2thtcog@pali> <20220803161318.GS1146598@bill-the-cat> <20220805142124.6swsha6aj62f33e3@pali> <20220805144731.GV1146598@bill-the-cat> <20220805145935.xumynhjbih3nekjq@pali> <20220805150340.GX1146598@bill-the-cat> <20220805151259.hdftfa3utfiq4i3w@pali> <20220805154400.GA1146598@bill-the-cat> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220805154400.GA1146598@bill-the-cat> User-Agent: NeoMutt/20180716 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.6 at phobos.denx.de X-Virus-Status: Clean On Friday 05 August 2022 11:44:00 Tom Rini wrote: > On Fri, Aug 05, 2022 at 05:12:59PM +0200, Pali Rohár wrote: > > On Friday 05 August 2022 11:03:40 Tom Rini wrote: > > > On Fri, Aug 05, 2022 at 04:59:35PM +0200, Pali Rohár wrote: > > > > On Friday 05 August 2022 10:47:31 Tom Rini wrote: > > > > > On Fri, Aug 05, 2022 at 04:21:24PM +0200, Pali Rohár wrote: > > > > > > On Wednesday 03 August 2022 12:13:18 Tom Rini wrote: > > > > > > > On Wed, Aug 03, 2022 at 06:00:13PM +0200, Pali Rohár wrote: > > > > > > > > On Tuesday 02 August 2022 06:58:26 Tom Rini wrote: > > > > > > > > > On Tue, Aug 02, 2022 at 11:13:38AM +0200, Pali Rohár wrote: > > > > > > > > > > > > > > > > > > > Hello Tom! > > > > > > > > > > > > > > > > > > > > Your commit de47ff536363289f92f85ed1e4901724d238432d ("Convert > > > > > > > > > > CONFIG_SYS_MPC85XX_NO_RESETVEC to Kconfig") seems to be broken. > > > > > > > > > > > > > > > > > > I thought I had managed to mirror the TPL/SPL/full usage that was there > > > > > > > > > prior, but apparently some got missed. > > > > > > > > > > > > > > > > Yea, conversion to Kconfig seems that was incorrect. > > > > > > > > > > > > > > As the config files were just unclear, but you seem to understand what > > > > > > > it's supposed to be, a patch to clean it up would be most appreciated, > > > > > > > thanks. > > > > > > > > > > > > > > -- > > > > > > > Tom > > > > > > > > > > > > Broken is also commit d433c74eecdce1e4952ef4e8c712a9289c0dfcc2. Seems > > > > > > that all kconfig migration changes done after that commit are broken. > > > > > > > > > > > > I really do not have energy to investigate what and how was broken due > > > > > > to incorrect kconfig migration. > > > > > > > > > > > > > > > > > > I did simple test. Applied following change: > > > > > > > > > > > > diff --git a/include/configs/p1_p2_rdb_pc.h b/include/configs/p1_p2_rdb_pc.h > > > > > > index a6523753d5ca..489f24df0ab1 100644 > > > > > > --- a/include/configs/p1_p2_rdb_pc.h > > > > > > +++ b/include/configs/p1_p2_rdb_pc.h > > > > > > @@ -624,3 +624,7 @@ __stringify(__PCIE_RST_CMD)"\0" > > > > > > "bootm $norbootaddr - $norfdtaddr" > > > > > > > > > > > > #endif /* __CONFIG_H */ > > > > > > + > > > > > > +#ifdef CONFIG_SDCARD > > > > > > +#error > > > > > > +#endif > > > > > > > > > > > > And then called: > > > > > > > > > > > > make CROSS_COMPILE=powerpc-linux-gnuspe- P2020RDB-PC_defconfig u-boot.bin > > > > > > > > > > > > And it failed, even when this defconfig file is not SD card builds. > > > > > > > > > > Where is PBL in that case even then? > > > > > > > > P2020 (and older) are pre-PBL boards, they do not support NXP PBL > > > > header. > > > > > > Ah, OK, then it should just be removing TARGET_P2020RDB from the choice > > > on "Freescale PBL load location". > > > > > > -- > > > Tom > > > > I just do not understand. > > > > P10** and P20** do not support NXP PBL. They support only pre-PBL and > > for SD card pre-PBL support I added option FSL_PREPBL_ESDHC_BOOT_SECTOR. > > > > But CONFIG_SDCARD is automatically set when SYS_EXTRA_OPTIONS contains > > "SDCARD" string and CONFIG_SDCARD is used then also in P10** and P20** > > SD-card version of SPL to load proper U-Boot. > > So CONFIG_SDCARD was over-loaded? That's very frustrating. That's what > needs to be corrected then. > > -- > Tom But it was correct, no? CONFIG_SDCARD ensures that U-Boot is compiled in mode in which can be booted from SD card. Or what do you have in mind as purpose of this symbol? The issue is that your Kconfig migration changes enabled CONFIG_SDCARD also when building (parallel) NOR version of U-Boot.