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 6DA1DC77B78 for ; Thu, 4 May 2023 07:45:42 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id EE7CB847DD; Thu, 4 May 2023 09:45:39 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=siemens.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=siemens.com header.i=@siemens.com header.b="eKpvWZ5q"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 9667B8475A; Thu, 4 May 2023 09:45:37 +0200 (CEST) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on20617.outbound.protection.outlook.com [IPv6:2a01:111:f400:7e1a::617]) (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 749DC847E3 for ; Thu, 4 May 2023 09:45:34 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=reject dis=none) header.from=siemens.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=jan.kiszka@siemens.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=T0FQEDDLRFFL9jlkcxfbjE0zbQapwnjXjahuussK/5vQK/BDaBhxiAe4KVbaad34absbJy/NclKNrSp7MKsmn1Cz+seHAAeTUQntXB0eMzmdQXhkvl4acDP3Eb3HUSVW3tn/bnhldzkzk6+pxFTPmbkpw40NPXNRsZHf8Z6bIUQh40p7jiZPtiDl/xzxN/T//rG/SxhmrI/pMIClSmQzo60GzIEEPnE7vhjFGtQIKB6W3hTglZDKKetyaswMhLdvJZQehjSIgQfiS/aT6ZXa9ah2N1zLHy/LMRgJUnHscZM/DEqgxEf2F3uL5Fqwb9uARq6mjQX3y8UPmqt/N+gyBw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=YWZOPIG6OFU0114eBNiBFQe/lJT1jw1aK3c/EuwAkUQ=; b=IIRumUiNXS999/cwn+FoJva8PdpPdT6myMnZEo0BMnA4P+uEKg5fzYcZZJY/4GsUKn8utF8ej/ZKKyBzigaBKPLkUKjoNie7y9QDHZIeeUcWRt/Xrt8AjAEqBJBabUfPRas44lTx5w3FqH2aYxQP1UNDJTReRuUs8AMx5cyiZ81MIocK+LZzoBGHplvQf/8kdef2vLlKEBTJz5ziCIPDph/QYaG+57TuHLJgJ4INdz9hCuirY6q+qFH8vyqQCE1DK/hra0i0PWtpShJIrx+WQl5A5an3dsdQGqDz0tGCo6E0dA7rNE2hdb4+G8nIAsbPxw/XODwSINrMmXuQN0tcrA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=siemens.com; dmarc=pass action=none header.from=siemens.com; dkim=pass header.d=siemens.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=siemens.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=YWZOPIG6OFU0114eBNiBFQe/lJT1jw1aK3c/EuwAkUQ=; b=eKpvWZ5qb4BM6hJaki9LHUfkpwRxOkRf6RAMoTQY5x13GSwr9faomPLx2tLhhJKSreuHWFUlR4zBK4g3vC6YZHHY4YbNPi3/8F7VwHjh4v5jjMZjDRFo7+24ce85Y/3xvDqziIBVaoJiTy7dWT67toK6eJPaYt7uMr2M61Egki1n6NVfPmbjP6dBXK///VlSwEeCTFqyPpakM5nkQE3khX0MTxLxqa+9Wrjfdu8da+xfxsDtTWa2OilnigjnW2REVU/vKIoN/SdPBLYkLyuIlVu/6owVR/U4tZKcBU9AS+G9R1uiHsVcnT/B49B4IRixU8B8NsQP5ztPIx7p6LqLJw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=siemens.com; Received: from AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:588::19) by GV2PR10MB6720.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:150:b6::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.36; Thu, 4 May 2023 07:45:31 +0000 Received: from AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM ([fe80::f5ab:5f59:3b47:fc2e]) by AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM ([fe80::f5ab:5f59:3b47:fc2e%4]) with mapi id 15.20.6340.036; Thu, 4 May 2023 07:45:31 +0000 Message-ID: <323d5661-e31b-e4d9-f5eb-9ef5977e9f05@siemens.com> Date: Thu, 4 May 2023 09:45:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.1 Subject: Re: [PATCH v3 00/19] Migration to using binman for bootloader Content-Language: en-US To: Neha Malcom Francis Cc: u-boot@lists.denx.de, sjg@chromium.org, afd@ti.com, vigneshr@ti.com, rogerq@kernel.org, alpernebiyasak@gmail.com, nm@ti.com, bb@ti.com, u-kumar1@ti.com, Tom Rini References: <20230421123203.1315330-1-n-francis@ti.com> <20230426223750.GA643785@bill-the-cat> From: Jan Kiszka In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: AM0PR04CA0131.eurprd04.prod.outlook.com (2603:10a6:208:55::36) To AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:20b:588::19) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AS4PR10MB6181:EE_|GV2PR10MB6720:EE_ X-MS-Office365-Filtering-Correlation-Id: 2a5b6661-463f-4649-9dd3-08db4c738933 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: gvghACUsgO7NYLD3aS4BVEs+Cda0i2la56o1X6MXqgFEwEZw+rGGxmtmEroVoedDbk+LIx+HL8fX236qHfI7JyqnDWOiaiyJg4qjEExf1xsNZWz5btxOBAxYpkZEVymZK3wvz99rPeZNaazZhEBZSDUN5UX7ktBxVE93Tfq2RarvkDtnntPUOYSL9NK38mQN97ymWke97FhIuGyVuOCBvB2V4zk7E4dj1CvjJOErCrKxkaEsyWm3Jtqttug8hMVWFYSAnQ9Y4nnbRNeHQIJauOcNM6mXMrHV75mwIToLdh1a77njk1Cob0oH7TdB9m20Vx1gUINJEedlmRMMIUioqD+3mD2kILsrb0bc7gHKJpr/zWTBG7FZeOlDBbN+fFshfC8v4J+X29sO7bzToTn++aXNzvMIXBp7vEaU4HK6epv7+yvGbtENPe+7xUh7POpALMaiH5AtqeJy0SsKPUS4Jyy9nVNr78oheGg1GJQ1w6Z2QRqoqkXndkh3rmPNZeT3LeuLaTNrlcuFG+QAdqqpsjHJ31fyZTuEkHT6KAtUfq6BLOZ0oj5zAcyXqUV9oCVyZ+LhbZxf3W8+kbzkuS9mQj8Z9ZXRk7w60mo/59wTutSyBrTlCtrmKfuYYQY2bi95zXtwOWWyMp7ImRJrEzT8RmSUui7RKe+0i1EsSpdIyEtJS3WlnRUpLZMJRiX+q0WK X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230028)(4636009)(346002)(136003)(396003)(376002)(39860400002)(366004)(451199021)(5660300002)(7416002)(966005)(8936002)(8676002)(6666004)(31696002)(86362001)(6916009)(36756003)(6486002)(316002)(4326008)(82960400001)(41300700001)(66476007)(66556008)(66946007)(478600001)(38100700002)(83380400001)(2616005)(2906002)(186003)(31686004)(53546011)(26005)(6512007)(6506007)(44832011)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ckR3UnNpeURlZVlBcDQrd0I0WUpqZ0lkaGpMRmVqbHFWNVovQXBCanR6MG5N?= =?utf-8?B?Smp1VVI2NDdWZ25iZzZ0RUZtWGZwUVRPRWNyUmZxQTI2dS8rY3BNYkp4OXJ6?= =?utf-8?B?UHRqT3V2WjV2S29SV1JzbDY2aVFmczVzbW1tUUlKY2VWeUlmQjNDTUVQRTRr?= =?utf-8?B?L1ZNeWF3VHVIQnVaYVlReHB5dG9taStrZzFKV25SUjZPMlpEdVE4TmFjcFRw?= =?utf-8?B?LzZVZVhKckRZbTZseEExZWROL1hlNWVsK2E0SGROU1F0R0tkMDJlc3ZKYldk?= =?utf-8?B?WHpzdjBRelE2bG1VWnpwUld5aW1FNjFHeFlqa2E3ZGQ3eE16VmpqTGttZm1Y?= =?utf-8?B?WU43YjdQSlNZKytMOW1rNWJSN0M0cWVpRG1FZjlaNlprY25WTjBNamlSU0lG?= =?utf-8?B?cytDTGRQU3NFK1ptQWtMaktmK3lIeEtTeE9XNUdDRHdqL0NpR1VOaVM0TUtD?= =?utf-8?B?b29ockx1YXZLRzBUM1pWVmhkZk9CUVdxbGFBTEZEUk1tczYxWVBFR2VPNWVp?= =?utf-8?B?ZlczWjZoSlBWWitxc3dKWVVlZWdBNVFCSlhBNDdUTzJzcFZseDNyUjk3VGxH?= =?utf-8?B?Nzk4NmF4RDVuTUZyYmdoUE5zbWRoS2ZNci9HcDdsWUV5MFE2YTdGT0NnSWFn?= =?utf-8?B?WHNOR3NNMHlrdENxblZtc2xRY3VmRGFBMU9HR2hjSE1DaEh5WGFuK0w0c3Qw?= =?utf-8?B?aktudHl3dXlPUUUzQnIvQXIwWFBSa2hFTDFySW5rMDB5eTV0Z01xMHRmTnRB?= =?utf-8?B?aXRHRkEvUTJzaUhkQjZ0d0FPOU9raHFIaDlEQkIyemRMSzJHYVh4czQ3cVQr?= =?utf-8?B?YldkRFdUOGRZK2ZDWUtRK05qcStOLzVaMUVaMVJZRWZFWGVOTUR6SHRveUNY?= =?utf-8?B?UGpVbE5FTnkwOEU3TzRSNjlKV1QxMzdkSEhZNHdMcGkvcXplMVFnTk9LZkpa?= =?utf-8?B?MHlxcjhXYyt2WVJXVzRtbW12SUpqcTRaMzRDVlR1eE9qaWtoUUppTUY0ZVY5?= =?utf-8?B?dnVYd21CVEtTN3gxRFNHM25ZWXNXOU5senNFc0Q5UEVEdG9PMWFVcXVaMDND?= =?utf-8?B?ZW1oaXYvTUQ0ak91eDEycmVjVXB3V2JCTmg1NFhHeEgwZEJYWHpGQjdUTXUw?= =?utf-8?B?WU5JMHJyS1huOFU0Q3VQc0o2L2p2clZub1Z1MUhadjJ1eFBORlhnZnBRa0Nk?= =?utf-8?B?c084QlpJQ1BkS1lOTURUcU5oN1paZVRXaERVeCs4bkJ1NVVVajh2RnRVSFJG?= =?utf-8?B?VWZtN09wWVM5RVp1TTVvSml5aGQ2c3Y0bmtieU1DMnhTampGNzYvUjJnT1lx?= =?utf-8?B?QVBxV3JwNXpaQkl5Z1dEdGRiOFUxbE1WbFhjTlU4MGxOMVlyekRjQWkveFVC?= =?utf-8?B?eVlpdFdicUs2dStMU1FmREd1bkV5SHBjeS9rRklQczM3VUhzdlk1bjZyREtF?= =?utf-8?B?TGNoenJEWmpLTDZVNzMxZjdNa1RhalNPbG9FQUczeHhLVVNFZTF5MnJsR2tP?= =?utf-8?B?YmZzZk5zUTFsQ1ZDUGRFMldJVE56ZDQ1WXhsd1hUQVl4UDVPV3BNSVhHWFh2?= =?utf-8?B?dWlWWkhNWTYyNjhQWkpLT2sxYUFTc3BnNURJeFZBS2x2VzNKenlacjd4bzE0?= =?utf-8?B?MkYwb29aVklmZ2RnczRlMjRFKzFNaEtTSnVzekVmQkRHNE5tZklwNlBGSnVj?= =?utf-8?B?aHVHSnFuZVNMd3pvUWJicCtpY2J3S0tmVk5PUXBSaDFBekhrTC8vRXBWUFho?= =?utf-8?B?Qk5SWTg5ekIvZW9OcWpIV2FnZHo3V1pNSDJoOTNjWHZ5Y0VLbldoWmpYejk1?= =?utf-8?B?Z2FBWXg0a01BcFlGZjVwbThGNVRPaGl3NnhwdS94TXJHQW5yaUVZYUF4REZL?= =?utf-8?B?UjBCRXZEUU5mTzAxRTlleU8rMzRrS0ExWVRRQjhPZEQvUVdWVlRURzI2QnBP?= =?utf-8?B?YkJVWEgrWnJmKzdXeElvM2wxeFJwL0ZuWktmK2cxUUhPYWVEQXlwUU01Y3Bj?= =?utf-8?B?SUFLSDIyalJHSW8zUHZyWHdwc3JtNmVpM3BIcThmZ1NTTnc1dDc1ZXord3Y5?= =?utf-8?B?WFZWQUdNRmx1S0ZsSGhZN05CUThla2hBYktGTEl6bno3Q3ZTdzBkakc3WE1p?= =?utf-8?Q?4YDq/uuF5awmjuBNnxFNLLTLx?= X-OriginatorOrg: siemens.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2a5b6661-463f-4649-9dd3-08db4c738933 X-MS-Exchange-CrossTenant-AuthSource: AS4PR10MB6181.EURPRD10.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 May 2023 07:45:31.1045 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 38ae3bcd-9579-4fd4-adda-b42e1495d55a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: D60LY8HzBwT4Djz610G4SRAYsdiS++PEwsntARSQPeIUszJ63SeZRMk9IMysAbb/2CsVfohL+0g3PYZKH88B7Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV2PR10MB6720 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 On 04.05.23 08:13, Neha Malcom Francis wrote: > Hi Jan > > On 04/05/23 10:13, Neha Malcom Francis wrote: >> Hi Jan, >> >> On 03/05/23 22:04, Jan Kiszka wrote: >>> On 03.05.23 14:56, Neha Malcom Francis wrote: >>>> 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. >>> >>> Where do I have to look at? Is there a git repo with that experiment >>> somewhere? >>> >>> Jan >>> >> >> There's no experiment yet, I will send one today; but I do not have >> complete understanding of the booting; whether the tispl.bin (which I >> assume is the only boot component that is affecting iot2050 boot since >> k3_fit_atf.sh is no longer there) has any concept of signing? Is >> core-secdev-k3 ever used? >> > > I have a tree posted here [2] that builds flash.bin with no error for > me. Please confirm whether your build flow does the same and also let me > know if the binary actually boots. > > [2] > https://github.com/nehamalcom/u-boot/tree/migration-to-binman-cicd-iot2050 > FWIW, dependencies, signing procedure etc. are all documented in u-boot/doc/board/siemens/iot2050.rst. I will try to have a look at that patch as well. Jan -- Siemens AG, Technology Competence Center Embedded Linux