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

[Peter Maydell]

On Tue, 10 Dec 2019 at 13:49, Igor Mammedov <address@hidden> wrote:
>
> 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.

A workflow that requires me to do a blob update when I
apply patches to target-arm.next isn't going to work for me.
If Michael wants the blobs to be handled like that, then all
patches which need to touch them will have to go via his tree.

thanks
-- PMM