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

[Nhi Pham via groups.io]

Hi Leif,


________________________________
From: Leif Lindholm <leif at nuviainc.com>
Sent: Friday, December 4, 2020 12:20 AM
To: devel at edk2.groups.io <devel at edk2.groups.io>; Nhi Pham OS <nhi at 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



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#68320): https://edk2.groups.io/g/devel/message/68320
Mute This Topic: https://groups.io/mt/78678099/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20201204/85b55eef/attachment.htm>