From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mx.groups.io with SMTP id smtpd.web11.24251.1593329660813234729 for ; Sun, 28 Jun 2020 00:34:21 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@intel.onmicrosoft.com header.s=selector2-intel-onmicrosoft-com header.b=y/i1QYYj; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: bob.c.feng@intel.com) IronPort-SDR: 0WBPaydJaDAmNtsmfQCnsQGwOwlW5jjyYIb7AuGTmhS4BeuKdMk8SN6Wb51sJm/aaaQaACG+ag RrfQZzDIOgqQ== X-IronPort-AV: E=McAfee;i="6000,8403,9665"; a="147343283" X-IronPort-AV: E=Sophos;i="5.75,290,1589266800"; d="scan'208";a="147343283" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga006.jf.intel.com ([10.7.209.51]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Jun 2020 00:34:19 -0700 IronPort-SDR: htE/7FpvQacHFNm8L0quefEUf8CxHzqJrmhyUvT1OXctVBFbHT0hryPzswUH24H/gc+P0Vzlv6 XUpo+fPQxo7Q== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,290,1589266800"; d="scan'208";a="280559901" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by orsmga006.jf.intel.com with ESMTP; 28 Jun 2020 00:34:19 -0700 Received: from FMSEDG002.ED.cps.intel.com (10.1.192.134) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 28 Jun 2020 00:34:16 -0700 Received: from NAM10-BN7-obe.outbound.protection.outlook.com (104.47.70.109) by edgegateway.intel.com (192.55.55.69) with Microsoft SMTP Server (TLS) id 14.3.439.0; Sun, 28 Jun 2020 00:34:16 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=XeLqF+wNAXYDXVORHmoSOq8pZ+QXcQ7aCB3bcQY71C0GP6Z09ReyMQxA/0uF9Ns1groybiOE9LlPa3xUEithR6iQthR/XRnevAVFCCHCnzv835ElzeXZMel5YBQp5eAPFexPmDqGo0yjrNCxG7GAgL6a/u8LHKpCJe9pRr2de9TGaOvdNtkm19HEUST6VV1tCK9x9mEHXqBf/hhmBdkqUoU4MxvhO/ZufmJKBRmvC30YHPkqMzCmmgYaWsDWcrjOml6PaFkYtmlg1uY23jMGHa6R0lNsC2mYhrk8J7g4SY2u6Y6fsv2SAvSwzCkqcQiw412D+MJ2IKcwUWZd33rWzg== 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-SenderADCheck; bh=k4dvNbyx0m1GHxcZ3G1qFPjnIxMw3dkoPHv//LD0JwY=; b=AD2ycaQOwDxdNrgC11YpU9OfJJWI/Ij7pn5fgSwogKM15shUylHzxCiEH157lgoRaWJBsVIzCYSiK0XqFjwTggaVK0qnBeT2ja9UpD3nDTYHwfKoCUPV9QDWFfl5vX1kqpUqGPyUUGNoHs8LknbanYxvyQRlwfw3QXeRlGGM3a3Sj3Ni3P+u1Oo3roskqxc2/LhVNp5Vf87Lzd3bJoJdFJvL8zZo0CFuodom+3Xf0+7gAS3o3TYIJ3ujNf9VsRcSoMUK2kFpF+B9JWPcVlkXlCEEjEo/OpjRtRFylP36Vrbe8mi4/duh+Oxs61lxvRk98TngRMWqVPCXF4u2kGurDQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=k4dvNbyx0m1GHxcZ3G1qFPjnIxMw3dkoPHv//LD0JwY=; b=y/i1QYYjh/A8vAyJaHEXeTA7q60N0ZaFm1d/oTZTW8gc7hMBeQ9Y+T7hYiKzRLyGInwuNQtroVptN9Ilxd8cNSA0FP0ukUiwKIoPsUyjUhN2PyJVOu8Z5RtUHTAyeJFjbCX4dXkdAsFU5LUs3xVmXnYc6ECumBqJ18WB0+gzzwQ= Received: from BN6PR11MB0068.namprd11.prod.outlook.com (2603:10b6:405:69::17) by BN6PR11MB4034.namprd11.prod.outlook.com (2603:10b6:405:7d::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3131.20; Sun, 28 Jun 2020 07:34:15 +0000 Received: from BN6PR11MB0068.namprd11.prod.outlook.com ([fe80::6870:82d3:5bde:4153]) by BN6PR11MB0068.namprd11.prod.outlook.com ([fe80::6870:82d3:5bde:4153%6]) with mapi id 15.20.3131.026; Sun, 28 Jun 2020 07:34:15 +0000 From: "Bob Feng" To: "devel@edk2.groups.io" , "pierre.gondois@arm.com" CC: Sami Mujawar , Tomas Pilar , "Gao, Liming" , nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ file Thread-Topic: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ file Thread-Index: AQHWSgcjA7iQ/v72okSjnvFg4182Xajn4DtwgAEyUgCABJT+4A== Date: Sun, 28 Jun 2020 07:34:15 +0000 Message-ID: References: <20200624090837.124392-1-pierre.gondois@arm.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: edk2.groups.io; dkim=none (message not signed) header.d=none;edk2.groups.io; dmarc=none action=none header.from=intel.com; x-originating-ip: [192.102.204.45] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: c1ad7180-2da4-4ed7-3a9b-08d81b35a8fb x-ms-traffictypediagnostic: BN6PR11MB4034: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-forefront-prvs: 0448A97BF2 x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: PtP7IvxXWpsuT4VYhZidi4pZf8FFyCmWP1k+7E45neD+xY+1AbWvCdvyw45NbxLi9HL92ZyL7eWsMmN1f4h5RkZAuN8FcaIOMMIgUlEguAxXOynGNpmsCznO2SaDc3+3jxTrjPJfxt1WQb3DXCjk+oLWa+TPUO2INW5X6/Xbm3TVDSaIlS1FRFlkG7/2pU2VucJ+XvME/ESknbTlxhS6IAU6ulhTCnINtViF6Ij2t19skBB4Etd/twLol2kpoZfxSDlBJfqMqC3kv+esedCXy/hwRU3f/Fc+/s3frXYEnpA4CCdd/tMKkOEdaAx7Milqr2U0BTNL6YLy1uT9XgbDwwi2h7kGk31lgNYgumar9+VDT2l1OTzo/845P1KJOEFBPFWLiaV+i52DPq22ng9K5A== x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BN6PR11MB0068.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(396003)(136003)(366004)(39860400002)(376002)(346002)(5660300002)(4326008)(71200400001)(9686003)(55016002)(316002)(33656002)(110136005)(66556008)(64756008)(66946007)(83380400001)(6506007)(966005)(8936002)(54906003)(53546011)(7696005)(76116006)(186003)(478600001)(2906002)(66476007)(66446008)(86362001)(8676002)(52536014)(26005);DIR:OUT;SFP:1102; x-ms-exchange-antispam-messagedata: tGLznNf9aa4T/GY8JMFJeJOsIMKaw6JERMcrM/lATshu1vGj5fm2HBoM//u54zEijzyBlQ551X1gTyX361UfXowk8MPJ7ogOMIZhLPPN9sNUEva0MUU7Zjuzsdb5SiDtuZU3NStOoW/7WCZZ6MnRSUNMkpdv7vho/dOZH9v6qOI36rvc0fCKp29WyHIkN2Vxc/GX7xAjXmLFGYQhm8Pao3k3v3txPczylrlRt19cHhSKkKY/+6cL7cTYMl2C4s/1qiTC4OAfKTCMupyb4ox7lit1IL61JPkNqIOH9KyW7Qy4CmIO66IUY4Ny2fMZa2LwKIcffhJQrLhRMOUsm5P2EYg538dUIm4MIoekC6zhRBh1WQxljduRzSZrohUhGicw3s2fHgZTz9WDlgZd8vcxt9fcP+lPUXAXzyQuV7ZEYHOfWV9UijepX/mxmVQOTeiV2YYqIFV7Q6Pyp2Ri29eK7EpG4PIlP3uq7VGCuF1xo2ebGhTrITWTaWXjWfLI5sAs MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN6PR11MB0068.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c1ad7180-2da4-4ed7-3a9b-08d81b35a8fb X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Jun 2020 07:34:15.3139 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: GAYX7n4hYlXYlT5yD2hvzujdFM0PaiILBUgj9ppUFuM2AyNGuOicj62LMOMKYVndvzWWmlxTqPxytrtzAQv3Vw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB4034 Return-Path: bob.c.feng@intel.com X-OriginatorOrg: intel.com Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Pierre, I met a problem when I push your patch set. If I change the CRLF to a unix format EOL, the patch can't pass PatchCheck= .py. If I don't change the CRLF, The build on Linux will fail. So I can't make the patch set pass the CI. Could you share me how did you do to make your PR 729 pass? Thanks, Bob -----Original Message----- From: devel@edk2.groups.io On Behalf Of PierreGondo= is Sent: Thursday, June 25, 2020 5:31 PM To: devel@edk2.groups.io; Feng, Bob C Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hello Bob, I believe the line endings of the BaseTools/BinWrappers/PosixLike/AmlToC s= cript have been modified to CRLF when I sent the patch. I created a pull re= quest from the linked github branch noted in the patches. It is available a= t https://github.com/PierreARM/edk2/commits/803_Compile_AML_bytecode_array_= into_OBJ_file_v3 . The pull request is available here (to show the result o= f the CI tests) https://github.com/tianocore/edk2/pull/729 . Do you want a v4 or is it possible to pull the patches from the github rep= ository? Sorry for the inconvenience. Regards, Pierre -----Original Message----- From: devel@edk2.groups.io On Behalf Of Bob Feng vi= a groups.io Sent: 24 June 2020 16:16 To: devel@edk2.groups.io; Pierre Gondois Cc: Sami Mujawar ; Tomas Pilar = ; Gao, Liming ; nd Subject: Re: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into O= BJ file Hi Pierre, There are some build failed in OpenCI. Would you please check it? https://github.com/tianocore/edk2/pull/727 Thanks, Bob -----Original Message----- From: devel@edk2.groups.io On Behalf Of PierreGondo= is Sent: Wednesday, June 24, 2020 5:09 PM To: devel@edk2.groups.io Cc: Pierre Gondois ; sami.mujawar@arm.com; tomas.p= ilar@arm.com; Feng, Bob C ; Gao, Liming ; nd@arm.com Subject: [edk2-devel] [PATCH v3 0/4] Compile AML bytecode array into OBJ f= ile Following the BZ at https://bugzilla.tianocore.org/show_bug.cgi?id=3D2425 This patch serie is a another way to solve the dependency of C files over = ASL files. With this new method, the dependency is resolved at the linking = stage. The last method to solve this dependency was to add the possibility to mod= ify INF files to depict such a dependency. This method was not accepted. Th= e discussion is available at https://edk2.groups.io/g/devel/topic/72655342#= 56658 The last patch modifying the INF specification and INF parsing are availab= le at: https://edk2.groups.io/g/devel/topic/72655342#56658 https://edk2.groups.io/g/devel/topic/72656060#56662 Pierre Gondois (4): BaseTools: Generate multiple rules when multiple output files BaseTools: Rename AmlToHex script to AmlToC BaseTools: Compile AML bytecode arrays into .obj file BaseTools: Fix string concatenation BaseTools/BinWrappers/PosixLike/{AmlToHex =3D> AmlToC} | 28= +++---- BaseTools/BinWrappers/WindowsLike/{AmlToHex.bat =3D> AmlToC.bat} | 0 BaseTools/Conf/build_rule.template | 15 += ++- BaseTools/Source/Python/{AmlToHex/AmlToHex.py =3D> AmlToC/AmlToC.py} | 82= ++++++++------------ BaseTools/Source/Python/AutoGen/BuildEngine.py | 2 += - BaseTools/Source/Python/AutoGen/GenMake.py | 6 += + BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | 38 += ++++---- 7 files changed, 89 insertions(+), 82 deletions(-) rename BaseTools/BinW= rappers/PosixLike/{AmlToHex =3D> AmlToC} (97%) rename BaseTools/BinWrapper= s/WindowsLike/{AmlToHex.bat =3D> AmlToC.bat} (100%) rename BaseTools/Sourc= e/Python/{AmlToHex/AmlToHex.py =3D> AmlToC/AmlToC.py} (52%) -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'