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 3A575C636CC for ; Tue, 7 Feb 2023 13:14:12 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 4F19585C98; Tue, 7 Feb 2023 14:14:09 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=amd.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=amd.com header.i=@amd.com header.b="A0QP6yS5"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 294E5856D2; Tue, 7 Feb 2023 14:14:07 +0100 (CET) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10on20608.outbound.protection.outlook.com [IPv6:2a01:111:f400:7e88::608]) (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 80D3485C38; Tue, 7 Feb 2023 14:14:01 +0100 (CET) Authentication-Results: phobos.denx.de; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: phobos.denx.de; spf=fail smtp.mailfrom=michal.simek@amd.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=jQoWF3dpKJvZnyA9JxxhkjPp+H6GkCenLq9amz8uQGqQcz+Xoa0gE1/kl+IxLxSnmSS/UVXU7/47WLsxp7B/X3HVEJ5JCWN+E4h9DYtZ03cbGAv/aIMBSG0tFSTbFtdsfBP/NcJ16mrtShd26QCyv7yBLatqBJe/iEKVdiMPQYdhfPd12dhwHz8U84f3/ULU024/Cek3lTNDlLuMVORiQ4FAb0KhXXaSF/hQvMDGv3qr7t1Y1Eq6tAXiFEH5t6An0Qtg3MoQE92wdinlsKcTFgk45KZJSHeKARMrnZ0w1EX9plXu5ee65nKiPPsygkultxdzRHeUtQNW7abIwfEZwg== 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=oSnQdCgYVvCBF+vbBNhpxWXSxHqBkja8L+TGcaGEBp4=; b=B0XIiOofaaltqI4sO8vZHtHb09N4ilXEBcdthjqeFrvtL4hyKXPf7ejUH8H616DXm66NA3DssN03JZzFl2QHZomdVNxVUw8x4Z5Rr9a5pS6/Ff42sOtdz2b/M10YIqnD7P7VZDB5sxqEKDBUEQxrBXgyd7DHk1yHxvetndd1rICmYJSCxzPmb5tzc8F5Z3+BARzG/HqORemOelhsYIkERhdfYXjD5PWf0tZoxyx2AluIp2/tH6SoPAETSM69+ZKeGy1jAKvzJNp9UVeXhEtYhoBLX7hyrcBrN7uZuzlrAnULSnX9zfunrmsfLKLWFl70SGl3uICMvcDpl2jJISWmOg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=chromium.org smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=oSnQdCgYVvCBF+vbBNhpxWXSxHqBkja8L+TGcaGEBp4=; b=A0QP6yS5q2WCrnWLeQ/yHim9q2WcQmnPQ/m1C3AwgJoMTArDEMLfav9izE3O3rKhhfB0yOmGZUp5/hn13mjkiryZjYAPzZAVyF0d9D6jNKKgh/AEnfy1ZlY/8Txz6iMi3fPpn5figt2jQx4Obma9y4F2tjxnTnjif4Zl9dJq5hc= Received: from MW4PR03CA0204.namprd03.prod.outlook.com (2603:10b6:303:b8::29) by DM6PR12MB4123.namprd12.prod.outlook.com (2603:10b6:5:21f::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.34; Tue, 7 Feb 2023 13:13:58 +0000 Received: from CO1NAM11FT017.eop-nam11.prod.protection.outlook.com (2603:10b6:303:b8:cafe::93) by MW4PR03CA0204.outlook.office365.com (2603:10b6:303:b8::29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.36 via Frontend Transport; Tue, 7 Feb 2023 13:13:58 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by CO1NAM11FT017.mail.protection.outlook.com (10.13.175.108) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.6064.35 via Frontend Transport; Tue, 7 Feb 2023 13:13:58 +0000 Received: from [10.254.241.50] (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.34; Tue, 7 Feb 2023 07:13:55 -0600 Message-ID: Date: Tue, 7 Feb 2023 14:13:52 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Content-Language: en-US To: Simon Glass , Peter Maydell CC: Tom Rini , , U-Boot Custodians References: <20230201225428.2001161-1-sjg@chromium.org> <20230201225428.2001161-7-sjg@chromium.org> <5919786f-354d-7d7f-d2a7-19f0b7121a0c@amd.com> From: Michal Simek Subject: Re: [PATCH v3 6/8] dm: treewide: Complete migration to new driver model schema In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.180.168.240] X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CO1NAM11FT017:EE_|DM6PR12MB4123:EE_ X-MS-Office365-Filtering-Correlation-Id: 390b53af-e861-435b-b1fa-08db090d2c16 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: YRU+xhMN8MBFk7I7Q9J6BbC8Y7MLRAo5Tq0Vfzh27H+S7lb9e3ns8gzV7sfgeddh6RQvXsjVqaazrk9QKULCJo4h+eeJ4CeWAgSHrVx4zEcwz5k7ORcTyoz2K5k3jCHjzSXToGW23sW/0Rzl/6lCp0UC42Y6Asm6RQtN8dzOpe9HFIu7zopisDM3I/LlxYkDjDv8/wo/dKj/oguJGIjHb/H6KOSmtchWK5xsL5uurpqfjWXGTTsg7kIN8Y8QUeUEzF5Y7QIHMDyNtW2xBrO2ITywfv2tRiCagaR/syTKyUxNESX267OLspRIX16DO/PHv07JsXlWnMkvWje23sjirIeMajgVVlfIJufGVpRoBhf2GwFFFWFzHeMAeuAiZG17OKXv+nepVu0lOHDZTdAJH+m6PrRgwW1jxvdw2jm2k4wtcqSJXUionXhILDTCKS25VMmrxbKA5dVLjBintA/9sojJPtq9Cbosuh2GD6XtGpkmc/35Nc8NnEzCiLo6OzOm0DtjM6JOGEQLDrYF769Ps4hhanKwKM9dStQq+2EbrRazAUOcOKbRGtoxPmWyLdWvB6vVOtZb+aLuoIm7ZP0kAjWFAw9xKGRv6WS13RM09FyBg71ElK0EOMGkDvoRpDlz6msh2SI0nPbpVSnjj94wa6IVkor0k1U0BmE7XuQfZpMrMiPLi9TtuVjeoztnvDykdr8IOqX1EC69UhIUlx7J9O2+RY8Z4S0f+souCEMB5pl7ooTo6RAOPZno0DpY40TJDH/bH0CABG5J0ZWrRp36dmoCXU3Su/2A5mcVz4V0iGs= X-Forefront-Antispam-Report: CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:SATLEXMB04.amd.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230025)(4636009)(376002)(346002)(396003)(136003)(39860400002)(451199018)(40470700004)(46966006)(36840700001)(31686004)(47076005)(81166007)(82310400005)(40460700003)(36756003)(86362001)(356005)(186003)(40480700001)(36860700001)(82740400003)(2616005)(31696002)(83380400001)(16526019)(336012)(426003)(26005)(478600001)(8676002)(966005)(16576012)(6666004)(53546011)(54906003)(110136005)(70206006)(41300700001)(8936002)(70586007)(316002)(4326008)(44832011)(2906002)(5660300002)(36900700001)(43740500002); DIR:OUT; SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Feb 2023 13:13:58.0404 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 390b53af-e861-435b-b1fa-08db090d2c16 X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: CO1NAM11FT017.eop-nam11.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4123 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 Hi, On 2/6/23 18:12, Simon Glass wrote: > +Peter Maydell > > Hi, > > On Mon, 6 Feb 2023 at 07:56, Michal Simek wrote: >> >> >> >> On 2/6/23 15:44, Tom Rini wrote: >>> On Mon, Feb 06, 2023 at 01:22:48PM +0100, Michal Simek wrote: >>>> Hi Simon, >>>> >>>> On 2/1/23 23:54, Simon Glass wrote: >>>>> Update various build and test components to use the new schema. >>>>> >>>>> Signed-off-by: Simon Glass >>>>> --- >>>>> >>>>> (no changes since v1) >>>>> >>>>> drivers/core/ofnode.c | 10 +++++----- >>>>> drivers/video/video-uclass.c | 4 ++-- >>>>> dts/Kconfig | 2 +- >>>>> include/dm/device.h | 2 +- >>>>> include/dm/ofnode.h | 10 +++++----- >>>>> scripts/Makefile.lib | 12 ++++++------ >>>>> test/dm/test-fdt.c | 2 +- >>>>> test/py/tests/test_ofplatdata.py | 8 ++++---- >>>>> tools/binman/binman.rst | 3 +-- >>>>> tools/dtoc/test_fdt.py | 8 ++++---- >>>>> 10 files changed, 30 insertions(+), 31 deletions(-) >>>>> >>>>> diff --git a/drivers/core/ofnode.c b/drivers/core/ofnode.c >>>>> index 4d56b1a7675..5249a60639b 100644 >>>>> --- a/drivers/core/ofnode.c >>>>> +++ b/drivers/core/ofnode.c >>>>> @@ -1265,22 +1265,22 @@ bool ofnode_pre_reloc(ofnode node) >>>>> { >>>>> #if defined(CONFIG_SPL_BUILD) || defined(CONFIG_TPL_BUILD) >>>>> /* for SPL and TPL the remaining nodes after the fdtgrep 1st pass >>>>> - * had property dm-pre-reloc or u-boot,dm-spl/tpl. >>>>> + * had property bootph-all or bootph-pre-sram/bootph-pre-ram. >>>>> * They are removed in final dtb (fdtgrep 2nd pass) >>>>> */ >>>>> return true; >>>>> #else >>>>> - if (ofnode_read_bool(node, "u-boot,dm-pre-reloc")) >>>>> + if (ofnode_read_bool(node, "bootph-all")) >>>>> return true; >>>>> - if (ofnode_read_bool(node, "u-boot,dm-pre-proper")) >>>>> + if (ofnode_read_bool(node, "bootph-some-ram")) >>>>> return true; >>>>> /* >>>>> * In regular builds individual spl and tpl handling both >>>>> * count as handled pre-relocation for later second init. >>>>> */ >>>>> - if (ofnode_read_bool(node, "u-boot,dm-spl") || >>>>> - ofnode_read_bool(node, "u-boot,dm-tpl")) >>>>> + if (ofnode_read_bool(node, "bootph-pre-ram") || >>>>> + ofnode_read_bool(node, "bootph-pre-sram")) >>>>> return true; >>>> >>>> Please correct me if I am wrong but this change will likely break all boards >>>> which didn't migrate to this at this stage. And because targeting early >>>> stages people will be without console. >>>> I think we should have transition period for 1-2 releases to give people >>>> enough time to migrate. It means print big warning that they have to migrate >>>> their DTS. >>> >>> What's the migration case here we're missing? Is it platforms that >>> maintain a dts externally, via tooling / etc, that populate those nodes? >> >> Yes and I expect there will be a lot of DTs around with some changes for >> specific products. >> >> Also for example QEMU is also generating DT based on it's configuration and >> provide it to U-Boot. >> https://gitlab.com/qemu-project/qemu/-/blob/master/hw/arm/xlnx-versal-virt.c#L91 >> When this patch is applied CI loop should fail for Versal. > > I am not sure how it helps us to drag this out. It is a breaking > change, but a drawn-out process is just going to create a lot of > confusion. People should be free to use the schema in Linux .dts files > from now on, but if it is not immediately supported in U-Boot then > they cannot. This is the most important point, after all. Definitely having this sort it out is good way to go and I remember we talked about it in past but none had really time to sort it out till now. New tags are in dt-schema and they can be spread everywhere (In spite of I think that property in options is cleaner solution - and could be still added in future). I just want to say that big cut after a lot of year by one series is not the best thing to do. I won't have any issue with it because simply I will add both u-boot,dm* and bootph* properties to all DTs which are out of mainline and for some time will maintain it but for customers who are having DTB somewhere out of mainline they will get to state that their newer u-boot stops to work without any single line saying that property name has changed. Smart one will start bisecting and will find this commit and they will know what to do with it. Qemu: We use -device loader,addr for quite a long time internally and to fix issue for Versal we just need to save current dtb as was described in qemu thread with -M dumpdtb=out.dtb and save it to uboot-test-hooks to continue to work in CI. Not sure if any other target in CI is affected by this change. Thanks, Michal