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 46029C54EBD for ; Mon, 9 Jan 2023 14:38:11 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id DC003850CC; Mon, 9 Jan 2023 15:38:08 +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="WpELDAAV"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 4998484474; Mon, 9 Jan 2023 15:38:07 +0100 (CET) Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12on2047.outbound.protection.outlook.com [40.107.243.47]) (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 D047D850CC for ; Mon, 9 Jan 2023 15:38:03 +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=CegpTnzdBoc9PxQ6FXgWi1sN1rY0j321play27wJ6wJEqNN07/3yAMsgUCkMcjFJUrtFIi8shmVj42xJLWvzNszL+Pmx9eRLIcma7bXwnhW34Go7hRPJ51YegS+rLTLc9KHDeKg6PZuHzYN1oKUwsP8TtmNL1S74J9ZLSeujvC0FEJK0nIRqWGGsJLbkKkSWHN0Svppc2tsjzqzGjRYMyJ24kqmzqWGOSLBs4ZgK+Hla1KMwmzIUVqbcRgn6eTLZlN/UJBUMHmCWUUMeGS9jZStdCZ/Z1Sg1FhHMItw3fweQicVOPVksMB9xzkzuk5JQRf3XHxejg6V6oEr34GV/5w== 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=Ola21LsYZmSZ00/fWDw6+nW3sGYmx4pvhwjgceAVgqQ=; b=I50RqxyIjDyDNHEA1PEmJaAE4Rnom8P9xcY5qnGvB0kv1jm8BY5O7cqW6p/xGIGNSV7JEjBmB0S2CwmfVQCcs3bxZ0fZNOW4fejug91RCasDMJEyWwp//CDy/JAnfOi3F9kLZStbZsJ4c+kDw20DVPNtzqHoloTf4KhJJZznCGU41yVFp7uPwo2LyRhJjkJefIWn6WM/QxyqI0rA3SvY0Zv8W7XHsibMGTYJGsp3Il1haZL0hItXavnW2pwNbRxQsQt50geCLE3GMBmCFb0mKF0gUYQxRE+GWPZ8wIMW/sRm+zjdTDEuop4Y6d8sxj/SD155OZnqkeYegQDlz0QIqw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=konsulko.com 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=Ola21LsYZmSZ00/fWDw6+nW3sGYmx4pvhwjgceAVgqQ=; b=WpELDAAVhTGOHn+Hz4xy/I1C8mVg/QR+xZ9BKc77ZsGAN9appNGn+Sv1fvhij3tmJ2oDrX6/qybTQKEg0EuHDM8hkfE3glABa0N5IkHdUC65vtWywtE3Z0S4J0pzR+lwJXLArFQga56Ef9gGcU1Ei61P7fW5cXnhIOZcd5DaCP4= Received: from MN2PR10CA0029.namprd10.prod.outlook.com (2603:10b6:208:120::42) by DS0PR12MB6536.namprd12.prod.outlook.com (2603:10b6:8:d3::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5986.18; Mon, 9 Jan 2023 14:38:00 +0000 Received: from BL02EPF00010209.namprd05.prod.outlook.com (2603:10b6:208:120:cafe::1) by MN2PR10CA0029.outlook.office365.com (2603:10b6:208:120::42) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5986.18 via Frontend Transport; Mon, 9 Jan 2023 14:38:00 +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 BL02EPF00010209.mail.protection.outlook.com (10.167.241.198) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.5986.15 via Frontend Transport; Mon, 9 Jan 2023 14:37:54 +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; Mon, 9 Jan 2023 08:37:22 -0600 Message-ID: <725130e5-9de2-6c90-a903-f5669db7001b@amd.com> Date: Mon, 9 Jan 2023 15:37:20 +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: Tom Rini CC: Simon Glass , Luca Ceresoli , U-Boot Mailing List , Roger Quadros , Alper Nebi Yasak , Peter Geis , Jerome Forissier , huang lin , Jeffy Chen , Kever Yang , Philipp Tomsich , Ivan Mikhaylov , Quentin Schulz , Philippe Reynes References: <20230107210721.478062-1-sjg@chromium.org> <20230107210721.478062-15-sjg@chromium.org> <20230108134132.GY3787616@bill-the-cat> <20230108160605.GB3787616@bill-the-cat> <20230108162438.GC3787616@bill-the-cat> <128e89b7-db5b-a842-6990-19c52de9da22@amd.com> <20230109141215.GI3787616@bill-the-cat> From: Michal Simek Subject: Re: [PATCH v9 14/14] treewide: Disable USE_SPL_FIT_GENERATOR by default In-Reply-To: <20230109141215.GI3787616@bill-the-cat> 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: BL02EPF00010209:EE_|DS0PR12MB6536:EE_ X-MS-Office365-Filtering-Correlation-Id: a3c9124e-f151-411e-2cbd-08daf24f1b22 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +S2uSD4ZT9XezBCFm5w+LUfQ+4K7cVxJHcj2yvz+dsMkkYzMMEp7zUiwIktQKJTGIH1pYPzdgwlhUGdOWGhNmbyW4WtxB7c78k2LexkXdLhTdVrGPYbp/glEts3QE442Whgt3UgbSijN9zNy+fFy3ilCZk3MyxY9x2fpCoT/tUgh//KklEo7qweEGd4XJdN0jcsiyVXvT+AgPNKcAl9G9SvJ4TsFFXUlAu0EoZcuqELIeFnQhyOSqNETmKTiWOuW942iVRQQld3g3MCITsymdC70fy/yExEn5DrtGdKSqI+EmeJQZS3UbGZUOXB9ooCXspkV9cPX5V3tAgROQYzspbr4De6fqwl6LgpjY8SB8Gd82X3pJ7kWk5XiaIsjJgN0l0VQmgZyXchEssAWjNIEN046CqcaA9oQChONlag/742jDhNux4NRfb0VRFhRDXlo3qjLPq217tsDiq5djVq9tcO4b1gXjpTZoyQd31AF/H3w4Z/wqrlTqTCGZ/vAfPsltQA7jhUrosfOlfuuUi+fnePG1IBgiuh9c5pRBWdht+HUjBztfPj5pIs0DTnz8vWS+TuiljOnhZqoO3/QU4886xeUb5DkHvE+KJE4SH0e4A1lKa/Nvv3h/dG3DXKzbGMvw58EcURJyWPif44PCNasMZKstDqJuMW2qvPVy12KL4IcsNX4ACOevQSiAidHH5sJpLndAZ+Az0jn3WR7Q4uU4RXEDL0axvNMnAI7ofBzvz45pDXr2ku0g1rmZq8OqY+UFEks/+EkT+SPrLIWPf0t7Q== 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:(13230022)(4636009)(39860400002)(136003)(396003)(376002)(346002)(451199015)(40470700004)(46966006)(36840700001)(53546011)(2906002)(478600001)(186003)(16526019)(26005)(31686004)(47076005)(54906003)(16576012)(316002)(2616005)(6916009)(70586007)(36756003)(70206006)(8676002)(4326008)(336012)(40460700003)(426003)(41300700001)(40480700001)(44832011)(5660300002)(82740400003)(83380400001)(8936002)(7416002)(36860700001)(82310400005)(86362001)(356005)(81166007)(31696002)(43740500002)(36900700001); DIR:OUT; SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Jan 2023 14:37:54.7752 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: a3c9124e-f151-411e-2cbd-08daf24f1b22 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: BL02EPF00010209.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR12MB6536 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 1/9/23 15:12, Tom Rini wrote: > On Mon, Jan 09, 2023 at 12:07:05PM +0100, Michal Simek wrote: >> Hi, >> >> On 1/8/23 20:36, Simon Glass wrote: >>> Hi Tom, >>> >>> On Sun, 8 Jan 2023 at 09:24, Tom Rini wrote: >>>> >>>> On Sun, Jan 08, 2023 at 09:20:09AM -0700, Simon Glass wrote: >>>>> Hi Tom, >>>>> >>>>> On Sun, 8 Jan 2023 at 09:06, Tom Rini wrote: >>>>>> >>>>>> On Sun, Jan 08, 2023 at 08:48:37AM -0700, Simon Glass wrote: >>>>>>> Hi Tom, >>>>>>> >>>>>>> On Sun, 8 Jan 2023 at 06:41, Tom Rini wrote: >>>>>>>> >>>>>>>> On Sat, Jan 07, 2023 at 02:07:21PM -0700, Simon Glass wrote: >>>>>>>> >>>>>>>>> This option is deprecated and only used by two boards. Enable it for just >>>>>>>>> those two boards, so others don't accidentally enable it. >>>>>>>>> >>>>>>>>> Signed-off-by: Simon Glass >>>>>>>> [snip] >>>>>>>>> diff --git a/boot/Kconfig b/boot/Kconfig >>>>>>>>> index 55f06761ef8..7ab0dd14211 100644 >>>>>>>>> --- a/boot/Kconfig >>>>>>>>> +++ b/boot/Kconfig >>>>>>>>> @@ -282,12 +282,13 @@ config SPL_FIT_SOURCE >>>>>>>>> config USE_SPL_FIT_GENERATOR >>>>>>>>> bool "Use a script to generate the .its script" >>>>>>>>> depends on SPL_FIT >>>>>>>>> - default y if SPL_FIT && ARCH_ZYNQMP >>>>>>>>> + help >>>>>>>>> + This is deprecated. Please do not use it. Use binman instead. >>>>>>>> >>>>>>>> Lets remove the text around bool so it can't be enabled, and move to >>>>>>>> select'ing it from the two boards that need it. Michal, Luca, what's >>>>>>>> needed to move your two platforms >>>>>>>> (avnet_ultrazedev_cc_v1_0_ultrazedev_som_v1_0 and xilinx_zynqmp_virt off >>>>>>>> of this very legacy option, given that other xilinx platforms have >>>>>>>> already migrated to binman ? >>>>>>> >>>>>>> That's a good idea, but these two boards do not have TARGET Kconfig >>>>>>> options so it is not possible without adding some Kconfig specific to >>>>>>> those boards, then defining it in the defconfig files. >>>>>>> >>>>>>> We already have the legacy warning. >>>>>> >>>>>> Yes, but I swear these are new legacy users as when we started trying to >>>>>> kill off this option it was just a few i.MX platforms lagging behind. >>>>>> >>>>>> Maybe make ARCH_ZYNQMP select DEPRECATED, USE_SPL_FIT_GENERATOR depend >>>>>> on DEPRECATED and add "DERECATED" to the end of the text line? Or maybe >>>>>> Michal or Luca will speak up soon and migrate these over quickly so we >>>>>> can just nuke this. >>>>> >>>>> Isn't that just more tortuous? I can disable SPL_LOAD_FIT on these two >>>>> boards since they don't appear in CI. Then they can convert them when >>>>> ready. >>>>> >>>>> That way we can drop the option now, if that is your goal. >>>> >>>> I thought xilinx_zynqmp_virt was in CI, but I see I'm mistaken. I still >>>> don't want to break platforms outright, and since it's Sunday right now >>>> afterall, we should let Michal and Luca a chance to catch up and chime >>>> in. I hope it's either going to be a quick conversion or expose >>>> something missing and needed in binman, as to why these still haven't >>>> been converted. >>>> >>> >>> OK let's hold off on this patch for now. It is just a clean-up anyway. >> >> First of all. ZynqMP is not wired in CI simply because we are missing some >> bits and pieces in upstream qemu to run it. If we can wire it with Xilinx >> version we can do it. I have asked to fix it our qemu team but they have >> never done it. > > Ah yes, I remember you saying that now. I guess it comes down to how > hard it would be to patch that support in to 6.1.0 (or, move us up to a > newer release and patch on top of that) in tools/docker/Dockerfile. We > special-case the nokia_rx51 support in QEMU because it's so old, so I'd > rather avoid that for another platform if we can. Issue is with firmware interface which is going over mailbox to microblaze unit which runs power management firmware. It means it not single qemu instance which Xilinx uses. There are 2. One microblaze and second ARM and IIRC communication is done via socket. I don't think this will get to Qemu anytime soon. But on the other hand in past we normally run arm core without any firmware on Microblaze. We would have to disable communication between ARM and Microblaze and work with fixed clock. After it we should be able to run zynqmp under CI. I have already asked my colleagues to fix drivers which can't work with ZYNQMP_FIRMWARE off. Also I have seen one missing Kconfig dependency. And then next step would be to identify some registers which Qemu doesn't model but u-boot is reading. I don't think they are modeled now. IIRC we still have internal task to get ZynqMP CI in our tracking system that's why we will get there at some point. Thanks, Michal