Re: [edk2-devel] [edk2-platforms][PATCH 1/3] Platform/Ampere: Add Readme.md

["Nhi Pham" <nhi@os.amperecomputing.com>]

[-- Attachment #1: Type: text/plain, Size: 1464 bytes --]

Hi Leif,


________________________________
From: Leif Lindholm <leif@nuviainc.com>
Sent: Friday, December 4, 2020 12:20 AM
To: devel@edk2.groups.io <devel@edk2.groups.io>; Nhi Pham OS <nhi@os.amperecomputing.com>
Subject: Re: [edk2-devel] [edk2-platforms][PATCH 1/3] Platform/Ampere: Add Readme.md

Hi Nhi,

On Thu, Dec 03, 2020 at 07:44:39 -0800, Nhi Pham via groups.io wrote:
> >> +
> >> +```bash
> >> +$ wget https://acpica.org/sites/acpica/files/acpica-unix2-20200110.tar.gz
> >>
> >> +$ tar xzf acpica-unix2-20200110.tar.gz
> >> +$ cd acpica-unix2-20200110
> >> +$ make && sudo make install
> >> +$ iasl -v
> >> +```
> >
> > I could well believe a newer version of IASL than shipped with the
> > abovementioned Linux distros would be required. However, Instructions
> > for how to rebuild/install a newer acpica-tools would be better to
> > have in the top-level Readme.md.
>
> The top level README does not mention the IASL compiler. Do you think we should add it?

Yes, that's an omission - ACPI hasn't seen as much churn recently as
it did for a while, but the underlying problems with iasl's lack of
backwards/forwards compatibility has not been resolved.

Would you be happy to do so?

[Nhi] Yes, I would be willing to take a look at this as a different topic.  At this moment, it appears that only version 20200110 would work with our code,so we will need to state so specifically in our README.md.

/
    Leif


[-- Attachment #2: Type: text/html, Size: 2906 bytes --]