From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::442; helo=mail-wr1-x442.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr1-x442.google.com (mail-wr1-x442.google.com [IPv6:2a00:1450:4864:20::442]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E01A421A07A92 for ; Mon, 19 Nov 2018 08:30:22 -0800 (PST) Received: by mail-wr1-x442.google.com with SMTP id j17-v6so32723171wrq.11 for ; Mon, 19 Nov 2018 08:30:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=zvYzxKIiQ4fsp78twP6tJQGlwp5G6PQOJ24e6eWnkv4=; b=PlDk79AZlu6Frv5u82CQ1trfYaWtbdZN5SzXU7GT5sO/DWeJylgTIY/CdT8QDzofb1 mzQXOBEIGTJ6IMnddGMHJmBM0sOgjLDICAI7tUzwFexml+fVKT1XAAwcBc63d3e4c0hT 2yUZrjJp4oEoGSGvJOlAZY2qL/lR/O8KY2GZ4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=zvYzxKIiQ4fsp78twP6tJQGlwp5G6PQOJ24e6eWnkv4=; b=Ztacr5eyGL1Bl9zfnpCmekNB6aeFqRr8nbmTfj8lgafALfmGd9FeJlxmnLcPJ3Bics 367R+u0LfSLJX6ZV0ReTz2h2ShydmDT8FKQKrbKKu6XWg4nH9T1nt2mptrrnmmeLmtdk cbfNVbHXHq4+m5b4vI+gTMJrlPobdWKcrAthhyRIqU77XkVqIAiI8/zMkWXKbFgq3Z63 378M1GAzyG8vfq2mf+v16rwRJwwgUucbGfwjjDrNtqm6iNvO+oeyA55Oa5ikYLBF5mSg E84KQhH/mRuB00lDu1lBvJWpUMvgkKbmoVBpVnM2Jj93WYBevfgqMslw6EkZ/EM/WDbk iqcA== X-Gm-Message-State: AGRZ1gIFw127+RJzbsr1awFqw1SZTazD/KJo2iDquxQjYQqk3+jAKFyK h2VKqUfnoh+2Aep3gTQ1ksyKhQ== X-Google-Smtp-Source: AJdET5drU69itI8WurjaNcnVMcZ+8ph7tLiQah+s10gq3wKmycIEe5Ir/5BQ8h42bZC7WEAKPg2BOQ== X-Received: by 2002:a5d:4586:: with SMTP id p6mr18258349wrq.69.1542645020996; Mon, 19 Nov 2018 08:30:20 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id y9sm8412930wrq.55.2018.11.19.08.30.19 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 19 Nov 2018 08:30:19 -0800 (PST) Date: Mon, 19 Nov 2018 16:30:18 +0000 From: Leif Lindholm To: Chandni Cherukuri Cc: edk2-devel@lists.01.org, ard.biesheuvel@linaro.org, graeme.gregory@linaro.org Message-ID: <20181119163018.6t4vr7k5gp6lwk5i@bivouac.eciton.net> References: <1542022916-24462-1-git-send-email-chandni.cherukuri@arm.com> <1542022916-24462-5-git-send-email-chandni.cherukuri@arm.com> MIME-Version: 1.0 In-Reply-To: <1542022916-24462-5-git-send-email-chandni.cherukuri@arm.com> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH v3 edk2-platforms 4/5] Platform/ARM/Sgi: Add ACPI tables for SGI-Clark.Helios platform X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 19 Nov 2018 16:30:23 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Nov 12, 2018 at 05:11:55PM +0530, Chandni Cherukuri wrote: > Added ACPI tables and GUID name 'gSgiClarkHeliosAcpiTablesFileGuid' > for SGI-Clark.Helios platform. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Signed-off-by: Chandni Cherukuri > --- > Platform/ARM/SgiPkg/SgiPlatform.dec | 1 + > Platform/ARM/SgiPkg/SgiPlatform.dsc | 1 + > Platform/ARM/SgiPkg/SgiPlatform.fdf | 1 + > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/AcpiTables.inf | 58 +++++ > Platform/ARM/SgiPkg/Drivers/PlatformDxe/PlatformDxe.inf | 1 + > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Dbg2.aslc | 90 +++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Dsdt.asl | 262 +++++++++++++++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Fadt.aslc | 87 +++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Gtdt.aslc | 152 +++++++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Iort.aslc | 106 ++++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Madt.aslc | 266 ++++++++++++++++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Mcfg.aslc | 61 +++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Spcr.aslc | 77 ++++++ > Platform/ARM/SgiPkg/AcpiTables/SgiClarkH/Ssdt.asl | 95 +++++++ > 14 files changed, 1258 insertions(+) Oh, right, the "A" wasn't a platform revision, it was "Ares". And this is "Helios". Then please, ignore previous comments - the other one needs to be written out ARES/Ares and this one needs to be written out HELIOS/Helios. At which point we are in dire need of deduplication of these ACPI tables: - DBG2, SPCR and MCFG are already identical between the new platforms, and Sgi575 should be modified to the same format. (This set leaving Sgi575 on ACPI6.1 definitions while the new platforms go to 6.2 should be enough explanation for why we need to refactor this.) - IORT has no functional differences at all for A/H, and 575 differs only in (incorrectly) using __builtin_offsetof instead of the OFFSET_OF macro, and the ACPI version. - SSDT differs only in coding style issues and the platform name (which could come in through an include file). - GTDT and FADT are identical between A/H, and only ACPI version differences compared to 575. - DSDT and MADT, I accept, are different enough that a per-configuration source file makes sense. So please resubmit this as a set that: - Moves Dbg2.aslc, Fadt.aslc, Gtdt.aslc, Iort.aslc, Mcfg.aslc, Spcr.aslc and Ssdt.asl directly under Platform/ARM/SgiPkg/AcpiTables/. Moves AcpiTables.inf to Platform/ARM/SgiPkg/AcpiTables/Sgi575AcpiTables.inf. Updates Platform/ARM/SgiPkg/SgiPlatform.dsc so that Sgi575 still builds. - Updates Sgi575 with whatever is required to pull the platform name in for SSDT. - Considers the COM1 name as mentioned in comment on previous patch. (I say considers: if I've misunderstood something, that one can be ignored.) - Brings in Ares and Helios platforms. (separate patches like now) - Brings in Ares and Helios ACPI tables, adding platform-specific Dsdt.asl and Madt.aslc under SgiClarkAres/SgiClarkHelios. (separate patches like now) With .inf files SgiClarkAresAcpiTables.inf and SgiClarkHeliosAcpiTables.inf. And once this is done, I no longer need to echo exactly the same comments I gave for the preceding patch, because everything will have alrady been addressed. Regards, Leif