Re: [PATCH v6 0/2] arm/acpi: simplify aml code and enable SHPC

[Igor Mammedov]

On Mon, 9 Dec 2019 17:51:10 +0000
Peter Maydell <address@hidden> wrote:

> On Mon, 9 Dec 2019 at 06:38, Heyi Guo <address@hidden> wrote:
> >
> > After the introduction of generic PCIe root port and PCIe-PCI bridge, we 
> > will
> > also have SHPC controller on ARM, and we don't support ACPI hot plug, so 
> > just
> > enable SHPC native hot plug.
> >
> > Igor also spotted the store operation outside of bit and/or is not 
> > necessary, so
> > simply the code at first.
> >
> > v6:
> > - Fix "make check" errors by updating tests/data/acpi/virt/DSDT*.
> >
> > v5:
> > - Refine commit message of patch 1/2
> >
> > v4:
> > - Improve the code indention.
> >
> > Cc: Shannon Zhao <address@hidden>
> > Cc: Peter Maydell <address@hidden>
> > Cc: "Michael S. Tsirkin" <address@hidden>
> > Cc: Igor Mammedov <address@hidden>  
> 
> Thanks, applied to target-arm.next. (it's a bit awkward that acpi
> table updates require also updating a bunch of binary test files,
> but I suppose trying to make the golden-reference be some textual
> format would be not very feasible.)

Michael tried document it (commit 30c63d4fbd69)
so that binary blobs would not be required (trusted).

Problem is usually in blobs being un-review-able and
unresolvable merge conflicts, that's why Michael
updates tables manually for all changes in pull req.

But in this case it touches only ARM tables and it's
the first change to go in, so it could just work out.


> 
> thanks
> -- PMM
>