From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Tue, 09 Apr 2024 12:43:02 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1ru8wQ-004Kbk-2g for lore@lore.pengutronix.de; Tue, 09 Apr 2024 12:43:02 +0200 Received: from localhost ([127.0.0.1] helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1ru8wQ-0003ZB-Jp; Tue, 09 Apr 2024 12:43:02 +0200 Received: from mail-db8eur05on2093.outbound.protection.outlook.com ([40.107.20.93] helo=EUR05-DB8-obe.outbound.protection.outlook.com) by metis.whiteo.stw.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1ru8wF-0003Xt-PM; Tue, 09 Apr 2024 12:42:52 +0200 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kVbMgXotX3zWHypYr7usERwC8bjyfZs48IOmVsHMY43A3xWsP4VI8HJO2lwjD2IMxcACUxoiODoSpoYmPcVZYjMMpHiFMMbKzqIIkc5JpaNFnHR3xSGHZ3IhhfvTGve3HtxW9P323Icen1OwIRSLItG9Trfu+V8A7HvT6d3bxS/l8iWdeoejfWrVgB7bNtygE2RNKGbCH8odu/p5dMj0qp/76I51tVbWsinlj5Dtl80DnuvqygPE3H+6jN0gbr29jITpo3ndTYsK36m73vky2k4BoO8xmrSV0YwA1bwG+MWex230BxnUCVQifh4fmUlXfWHyoOVbwpzR1/5nDVB8YQ== 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=b8UJJCkyXEez76q3gI/bXunaaxPYbh3iWjw6MeUcI64=; b=UU6URFylz/o2n1gAKCxbKvoU6ndwqOEcHyBsWVkz7Pmia049MbqhLtmbf4DUtbiVhm9T4wRdx46PqxooFBjdUpMYqGRQoJOMr9k09Fx+uRYuOeg48nk/q2fGhbaMW8x5YDO0puLhUcl/rY2oZE5QHo0PzzhHxptUB4SqiWWmfXPw44MeCkyB1UuiIpdwm7oIj7WeKzadrsXvMcAWqtF4HYqVLqQHsEHzQxGuKndPmbhRhHyFiJJdzyXQShsG1PHRJhP+zKQxcmtDYfBgN3ZKdPdrTGF8pz/k8hUhke3YqWXpLvT5Yxl6uExT8K0el3Sdwm/C2WtTa8itIgoGBoS9+w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=t2data.com; dmarc=pass action=none header.from=t2data.com; dkim=pass header.d=t2data.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=t2datacom.onmicrosoft.com; s=selector1-t2datacom-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=b8UJJCkyXEez76q3gI/bXunaaxPYbh3iWjw6MeUcI64=; b=KoUDaMZntuMtX3KHckjTjlPY9/EUqEYGSvBxSJ+QHvIqMnWbz+rZdjoh3HtWDcPDAVjFCLODTXdhiAZgtuo7yDNvUHqtjDM37TUsynpNYGDnipujcQKKsGCYzq83sGjzlHpLo1S0B9ERBIgxj6LHGbMM21FEaLaH+YgkZihKqWk= Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM (2603:10a6:10:334::22) by AS4P251MB0893.EURP251.PROD.OUTLOOK.COM (2603:10a6:20b:575::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7409.55; Tue, 9 Apr 2024 10:42:47 +0000 Received: from DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::ba8c:3db:3a02:7ce4]) by DB9P251MB0618.EURP251.PROD.OUTLOOK.COM ([fe80::ba8c:3db:3a02:7ce4%6]) with mapi id 15.20.7409.042; Tue, 9 Apr 2024 10:42:47 +0000 Message-ID: Date: Tue, 9 Apr 2024 12:39:50 +0200 User-Agent: Mozilla Thunderbird To: Ahmad Fatoum References: <20240409090816.2343063-1-a.fatoum@pengutronix.de> <90b7b9b6-92cd-444f-a8b9-b2552e794dc8@pengutronix.de> Content-Language: en-US From: Christian Melki In-Reply-To: <90b7b9b6-92cd-444f-a8b9-b2552e794dc8@pengutronix.de> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-ClientProxiedBy: GV3P280CA0006.SWEP280.PROD.OUTLOOK.COM (2603:10a6:150:b::6) To DB9P251MB0618.EURP251.PROD.OUTLOOK.COM (2603:10a6:10:334::22) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DB9P251MB0618:EE_|AS4P251MB0893:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: DIF+GA7qOR149X6rLTyZMmosS8fmp+7rQnv6Far8KPc6DQUVfyRid8+XUkz46qqfk0sk1J4BBCB1Z4uginknIrLohiEi6Ie6kGxangfIVo6d2lJcvMm5zgIzUnzjxAughlqlQteQV01cjkG7ia8CU0C1bcDEyO6eEKCggQNlHVAHXEM3NdLOUyM7zbR/HLjwnye2pfa9Tes19lRW1wi6Ig2Y1wFiojIULKBE530TFZskdR9Qap4qxjm9ozwY4fAOEjlIKjXMT6MT3nmQfQm1BjcfkZcACtieFymZUvBpt3kxvb7QsxzrfAMCcoi9ZyytMnNrHNPTjY7CSbCxDGCgZRNl3/95S4hJgw1MdGKXfGfFFpP9i56ZTribfHVjXUcNcNdMTtUgDSzwjP6G32raGv+my49jPm7LmXAvvXOt9p1bOFmk2TQATwWvgnI6myZQacHlrGiGqgL35xhmSSL9XGF1xwZbGQzliYNWTW6IIZBC+ljWiZXWs/0tZKP5LF5dYt/hGMhAgzJlue6xGr76DJLI3EEMO2avSD2gyiGH6oSKz8jadgMU7YlaJC0L3kRstKOMgJT3xVE+2hJVdAdrXQffAm7f/ORu9zEQ8lCwMsg= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB9P251MB0618.EURP251.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(13230031)(1800799015)(376005)(366007); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MGNzblRNN2dCSC9XZUJDY1J2b2VYdTlIWlFSOEZHb3o2ZUZEcXVDSHUxdTdh?= =?utf-8?B?QmM0YmhJQzNCR0FNSVowdWhycnhtcll6N1hqUFFOdXJtNTVnNFVXWHN3MjNL?= =?utf-8?B?Qks5SEo4U1VJdURmNjArQzFLL1VoS0Z0dTZQUjI1WlRpQzVMc0MvSkxBMEMr?= =?utf-8?B?UmNiVDlydXcxMEl3cUFPNlgrbHdrem1VbjZTWU81dUNSNUx3MTZWM3JxSWdS?= =?utf-8?B?VFQ3UHhwS2RHWW9zUFh2UFdGOWRmckFZYXRGdEw0QW1IRzBteTFvbEI0WXRi?= =?utf-8?B?Ty9JZGVKUXNtdi9TclJIQXV0Sm5meStDN1hyaG8rTEtYVUlIYVdmZEFFbWlU?= =?utf-8?B?dVBqRkJLb3A1bzZoWkRZTXZEOUYxVnlkQklPUkFQUHkxOXdyN1h1UUcwWURu?= =?utf-8?B?cE42MGJYMzJEbmM1YWpvaUdlejR5V0VWU2crRjc2aEFkRGRCSmFxYTdiL0Jy?= =?utf-8?B?ZHh0N2hxTVp2SWRFMXdHMjFQWWIvOWpYdjhRWHRGaE95N3o4QUxvTVRYQjJo?= =?utf-8?B?bzZtLzlXcjR3SVRPcFpUcUZlQ1pEa1JjTjMxMGN0ZnFhaUVyR0dEV2hZbUZS?= =?utf-8?B?MmQyNDRyVVVFelBKQndINzViRWl0aTJsc3F3V2lFMEN0K0pSa01tcXlBNTIr?= =?utf-8?B?ZW1GS05KckVnYUFXSHdLei9lK0JJMmIraWVMb0tVaE14ZzZJUXM0WTBuNS9O?= =?utf-8?B?TTFnN3UxN3UrQVF5Y0Z4RHU4ZkxBVGI1djJqcXIvV1BSRkZuWTZSalVqRlpw?= =?utf-8?B?ZUMvTDBqUVNKb1NSVDEwYkJZSEgyNWVzNnhCazZSQ3Y3WThmRnBLY2ZobUVk?= =?utf-8?B?VitCbnFuQmI3SHFVclU2NmZuQnJmQk5scm9SQU9QTGM4NnkzaUxKOWtrdzYy?= =?utf-8?B?ZWJIT1NxUHNNZ3V2WUR1b2ZkWWNHTFRjQmRrNXR5YVBwcGcwb1RLTUdJMjhi?= =?utf-8?B?eHB3QnRDb0x3ZDNpMGttamd3NEFPbEFEUW95OHFnVSsvNUZCRThJTjNFaTBN?= =?utf-8?B?RGYxcXBvUlI0Q1FVbkNzTlNHdjRmMm5nNTV0aFFPbWw3b3JkWHFsN211Wi9m?= =?utf-8?B?REJtcGVKaFIxTE9GcFNpaWVObkZYOWIzZVY2c1lTZE9YcUpXY3JOVFpTbzNT?= =?utf-8?B?dHZtaDJuczhreSt2aklmdXZPY3lTU3VJR3NHVkNOMm5qbmxqQWp4WTJ4Mnlh?= =?utf-8?B?WmJFNjJ5TFJ2RENGWXdxc0JVeVphTk9YaHJtNWVQYVludGNsQzBUUFNGNVU0?= =?utf-8?B?N1VGWWVLZHZ4cHRmekxSdG9IemJjb0ZvVjdzeHFhQTBpc1kvZksyZGtSejlF?= =?utf-8?B?a3NBM1BjSmFqZ1E4em9idjdjeWpSSnk0Vk16RWFOZzFjdUpGMCthS0QwcG1l?= =?utf-8?B?dVBoVlJRMEV5a0pzeWNKcktzNC9sOGpJNjdPNXBib3ZXK1VCKzBNdVlONGJ0?= =?utf-8?B?bXVVTUtQekxhTFZvNTdzYXN2NVgvQXBEZWNMZlJNbzFYdkpGMUs3eDh4aVBh?= =?utf-8?B?dFhRaGdYWXJmNXFuOUdPMk03VHJlS0IvRlBlL0NKRGYxbmd5MG50VlpQQkVq?= =?utf-8?B?bkV0NEhES0lTYjQvSWxlV3N1SWlqeGI4M0tCR0FMbkxvQUtOZTRkMDZyRlFn?= =?utf-8?B?LzRMV244WmIzUFVkaFpCREorTEx3cDhwZTdzc0JDTWdCd1JaQjU1S1FqUWpk?= =?utf-8?B?OEJOczBmbVBMWlErRDVnK2NnSkdSQk1jSTJWcENoZmpaYVpXeEZLY2xJVTQr?= =?utf-8?B?OS9TMzZCVmRNeDErT2c5SGpyYzlqTEZMQjhJalJ0Z0lpWVdtajdmT0tRZ2xZ?= =?utf-8?B?cHNUZmlvWnQrOG9jdjgxT0FKSlEyRnUvRFdDZmd1ak9nUFRRTjdhRklabGhG?= =?utf-8?B?VEJMblI4N1pJQmZDTExXS3JScnpRcmVzUG55eGJTbTJvRldLNU1OSVBkKzdQ?= =?utf-8?B?NFdKbWgyRGVIUkdsSm8yVkJJL0ZwN0RSQ0pKYnpKeFByN05GVFU0VnlxQllB?= =?utf-8?B?RmFxVEJOdkpOK3puS3VhT2VzWFZBbUZmVVFkdkNmSlQvOTVNZ3FIbXdYS1pZ?= =?utf-8?B?cXR4Zk9PRzRzMVdyUXBXdGlLRTJoZ2UvNkRQamxqWEVnK0lIY0l2cXk4WGdG?= =?utf-8?B?UXJnaVpiOGpDZXR4MCt3NU8wQnZaRXNJdTQ5VHJEL0lPcnZ1SmZNZTZ4NXU2?= =?utf-8?B?L3c9PQ==?= X-OriginatorOrg: t2data.com X-MS-Exchange-CrossTenant-Network-Message-Id: ac53a379-aba1-49b3-82c5-08dc5881cbb1 X-MS-Exchange-CrossTenant-AuthSource: DB9P251MB0618.EURP251.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Apr 2024 10:42:47.2421 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 27928da5-aacd-4ba1-9566-c748a6863e6c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: ueBscmv5GrkXnAbjbUyqgrjpJCpghQd1VoBa+tYdcTeOL1H7sQeFm9T47Fz8ZxntBXSLl78eUvAMA6oZlWWYLvdSahJZ8vtYlwRqHTxMn7s= X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS4P251MB0893 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.whiteo.stw.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-2.1 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [PATCH] tf-a: stop defining AS and other variables in cross make env X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false On 4/9/24 12:37 PM, Ahmad Fatoum wrote: > Hello Christian, > > On 09.04.24 12:23, Christian Melki wrote: >> >> >> On 4/9/24 11:08 AM, Ahmad Fatoum wrote: >>> The MAKE_ENV incantation results in PTXdist defining variables like AS >>> and LD that point at the respective tools in the cross toolchain. >>> >>> This was unneeded as we already supply the TF-A built with a >>> CROSS_COMPILE option, which is used to derive the correct tools, but it >>> didn't hurt. >>> >>> This will expectedly change with TF-A v2.11, which should merge commit >>> cc277de81692 ("build: refactor toolchain detection"), currently sitting >>> in the integration branch. >>> >>> This changes the default value of AS to be that of the C compiler and >>> leads to various issues when overriding it with an assembler as was >>> possible before, e.g.: >>> >>>    - hang during tool discovery, because `as -v` keeps waiting for input, >>>      unlike gcc -v >>> >>>    - Assembler errors during build, because GCC-specific options (e.g. >>>      -DMACRO=VAL) are now passed to as(1). >>> >>> While TF-A changing the semantics of the AS option is arguably >>> strange[2], let's take the easy way out and stop setting AS, so the PTXdist >>> rule can build both old and new TF-A without problems. >>> >>> [1]: https://github.com/ARM-software/arm-trusted-firmware/commit/cc277de8169 >>> [2]: https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/27163 >>> >>> Signed-off-by: Ahmad Fatoum >>> --- >>>   rules/tf-a.make | 2 -- >>>   1 file changed, 2 deletions(-) >>> >>> diff --git a/rules/tf-a.make b/rules/tf-a.make >>> index 54fe7d1e37d8..b8c477476c74 100644 >>> --- a/rules/tf-a.make >>> +++ b/rules/tf-a.make >>> @@ -74,8 +74,6 @@ TF_A_CONF_TOOL    := NO >>>   # Compile >>>   # ---------------------------------------------------------------------------- >>>   -TF_A_MAKE_ENV    := $(CROSS_ENV) >>> - >> >> Perhaps not in this config, but isn't the make env used for other things? >> Like TLS stuff? > > Do you have a project, where you have TLS stuff and could test this? > > Anyhow, v2.11 would break the recipe, so this trades breaking all recipes > for breaking only some recipes. Once we have breakage reports for those, > we can start setting only the relevant variables in TF_A_MAKE_ENV. > > Cheers, > Ahmad > I have one with an older release, not actively rebuilding it. And I'm fine with the changes, just highlighting that it's maybe not an entirely useless variable. So please, go ahead with the changes. >> >> regards, >> Christian >> >>>   $(STATEDIR)/tf-a.compile: >>>       @$(call targetinfo) >>> >> >