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