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 8D02BC7EE2A for ; Wed, 3 May 2023 12:56:43 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 7613F82A2C; Wed, 3 May 2023 14:56:41 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.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=ti.com header.i=@ti.com header.b="dUHtEjaE"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 1D54F8477B; Wed, 3 May 2023 14:56:40 +0200 (CEST) Received: from fllv0016.ext.ti.com (fllv0016.ext.ti.com [198.47.19.142]) (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 A060081DCF for ; Wed, 3 May 2023 14:56:36 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=ti.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=n-francis@ti.com Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 343CuRLr130835; Wed, 3 May 2023 07:56:27 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1683118587; bh=Got2o+DJUffI/iGRUlF7DcUy3M10VKoXX5DDZyyOAow=; h=Date:Subject:From:To:CC:References:In-Reply-To; b=dUHtEjaENd0tFLJu9V6WOaBu280Z0RXdL87jWwuqEycS6/Pk6f4DEuosEXzZAh786 UXqvspMo3icQRIIY0jGSkyxpsK93FnoRuHin67wvJwk+OzMyBGby+QVliXCROiSYcl 1dAAHKG1HjeGzwTSPt+bfPM2WF5XfgdeM6HjdvZg= Received: from DLEE104.ent.ti.com (dlee104.ent.ti.com [157.170.170.34]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 343CuRZL078531 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 3 May 2023 07:56:27 -0500 Received: from DLEE101.ent.ti.com (157.170.170.31) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 3 May 2023 07:56:26 -0500 Received: from lelv0326.itg.ti.com (10.180.67.84) by DLEE101.ent.ti.com (157.170.170.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 3 May 2023 07:56:26 -0500 Received: from [172.24.145.195] (ileaxei01-snat.itg.ti.com [10.180.69.5]) by lelv0326.itg.ti.com (8.15.2/8.15.2) with ESMTP id 343CuNeR018070; Wed, 3 May 2023 07:56:24 -0500 Message-ID: Date: Wed, 3 May 2023 18:26:22 +0530 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.0 Subject: Re: [PATCH v3 00/19] Migration to using binman for bootloader Content-Language: en-US From: Neha Malcom Francis To: Jan Kiszka CC: , , , , , , , , , Tom Rini References: <20230421123203.1315330-1-n-francis@ti.com> <20230426223750.GA643785@bill-the-cat> In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 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 Hi Jan, On 03/05/23 12:57, Neha Malcom Francis wrote: > Hi Tom > > On 27/04/23 04:07, Tom Rini wrote: >> On Fri, Apr 21, 2023 at 06:01:44PM +0530, Neha Malcom Francis wrote: >> >>> This series aims to eliminate the use of additional custom repositories >>> such as k3-image-gen (K3 Image Generation) repo and core-secdev-k3 (K3 >>> Security Development Tools) that was plumbed into the U-Boot build flow >>> to generate boot images for TI K3 platform devices. And instead, we move >>> towards using binman that aligns better with the community standard >>> build >>> flow. >>> >>> This series uses binman for all K3 platforms supported on U-Boot >>> currently; >>> both HS (High Security, both SE and FS) and GP (General Purpose) >>> devices. >>> >>> Background on using k3-image-gen: >>>     * TI K3 devices require a SYSFW (System Firmware) image consisting >>>     of a signed system firmware image and board configuration binaries, >>>     this is needed to bring up system firmware during U-Boot R5 SPL >>>     startup. >>>     * Board configuration data contain board-specific information >>>     such as resource management, power management and security. >>> >>> Background on using core-secdev-k3: >>>     * Contains resources to sign x509 certificates for HS devices >>> >>> Series intends to use binman to take over the packaging and signing for >>> the R5 bootloader images tiboot3.bin (and sysfw.itb, for non-combined >>> boot flow) instead of k3-image-gen. >>> >>> Series also packages the A72/A53 bootloader images (tispl.bin and >>> u-boot.img) using ATF, OPTEE and DM (Device Manager) >> >> So, next up is fixing this in CI. After taking Andrew's patch to fix the >> typedef issue, and after my patches to ensure we can get >> pyyaml/jsonschema for python, there's problems still: > > > Thanks for checking this! Couple things: > >> Over at https://source.denx.de/u-boot/u-boot/-/jobs/617966: >> binman: Filename 'spl/dts/k3-am68-sk-base-board.dtb' not found in input >> path (.,/builds/u-boot/u-boot,board/ti/j721s2,arch/arm/dts) >> (cwd='/tmp/.bm-work/j721s2_hs_evm_a72') > > 1. This is dependent on the patch merging J721S2 HS and GP configs [1]. > However it has been reverted on -next, seen in the same thread. > >> >> And then: >> https://source.denx.de/u-boot/u-boot/-/jobs/617965#L1328 >> Error: arch/arm/dts/k3-am62a-sk-binman.dtsi:167.1-8 syntax error >> I've fixed this, minor but serious change. > > 2. Regarding iot2050, build fails since it uses > arch/arm/mach-k3/config.mk which is now entirely binman based. Will try > moving iot2050 to binman as well. I'll need some help with this, might need to know the bootloader flow to make a clean migration. > > [1] https://lore.kernel.org/all/20230224050749.13145-1-m-chawdhry@ti.com/ > -- Thanking You Neha Malcom Francis