[PATCH v3 02/57] x86: acpi: Add base asl files for common x86 devices

[Bin Meng]

Hi Wolfgang,

On Tue, Sep 22, 2020 at 10:47 PM Wolfgang Wallner
<wolfgang.wallner at br-automation.com> wrote:
>
> Hi Bin,
>
> -----"Bin Meng" <bmeng.cn at gmail.com> schrieb: -----
> > Betreff: Re: [PATCH v3 02/57] x86: acpi: Add base asl files for common x86 devices
> >
> > Hi Wolfgang,
> >
> > On Tue, Sep 22, 2020 at 10:19 PM Wolfgang Wallner
> > <wolfgang.wallner at br-automation.com> wrote:
> > >
> > > Hi Simon,
> > >
> > > -----"Simon Glass" <sjg at chromium.org> schrieb: -----
> > > > Betreff: Re: [PATCH v3 02/57] x86: acpi: Add base asl files for common x86 devices
> > > >
> > > > Hi Wolfgang,
> > > >
> > > > On Mon, 21 Sep 2020 at 07:50, Wolfgang Wallner
> > > > <wolfgang.wallner at br-automation.com> wrote:
> > > > >
> > > > > Hi Simon,
> > > > >
> > > > > -----"Simon Glass" <sjg at chromium.org> schrieb: -----
> > > > > > Betreff: [PATCH v3 02/57] x86: acpi: Add base asl files for common x86 devices
> > > > > >
> > > > > > Add common x86 ASL files, taken from coreboot.
> > > > > >
> > > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > > ---
> > > > > >
> > > > > > (no changes since v1)
> > > > > >
> > > > > >  arch/x86/include/asm/acpi/chromeos.asl    | 108 +++++++++++++++++
> > > > > >  arch/x86/include/asm/acpi/cpu.asl         |  25 ++++
> > > > > >  arch/x86/include/asm/acpi/cros_gnvs.asl   |  29 +++++
> > > > > >  arch/x86/include/asm/acpi/lpc.asl         | 141 ++++++++++++++++++++++
> > > > > >  arch/x86/include/asm/acpi/pci_osc.asl     |  21 ++++
> > > > > >  arch/x86/include/asm/acpi/pcr.asl         |  80 ++++++++++++
> > > > > >  arch/x86/include/asm/acpi/ramoops.asl     |  32 +++++
> > > > > >  arch/x86/include/asm/acpi/sleepstates.asl |  12 +-
> > > > > >  8 files changed, 443 insertions(+), 5 deletions(-)
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/chromeos.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/cpu.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/cros_gnvs.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/lpc.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/pci_osc.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/pcr.asl
> > > > > >  create mode 100644 arch/x86/include/asm/acpi/ramoops.asl
> > > > >
> > > > > There are recent (~June 2020) commits in coreboot which convert most ASL files
> > > > > to ASL 2.0. The ASL 2.0 syntax is much easier to read IMHO, and I think it
> > > > > would be good to update this patch to take advantage of the recent coreboot
> > > > > developments. See e.g. the commit at [1].
> > > >
> > > > Yes that looks nicer, but I have everything working and tested now so
> > > > don't want to go back and unpick things. This seems like something we
> > > > can pick up once we finally have this in the tree. The good thing is
> > > > that this is just a syntax change as I understand it.
> > >
> > > Fair enough. Yes, it should only be a syntax change.
> > >
> > > Regardingn testing: I haven't tested individual patches of part D, but I have
> > > tested the complete series. After applying the fix for the DSDT length my
> > > ApolloLake board has booted reliably with ACPI enabled.
> > >
> >
> > Will you add the "Tested-by" tag for this series?
>
> I lack experience in testing complete series, which is why I was not sure how
> this should be handled. If testing a complete series at once is enough, without
> testing the individual patches, then this series is:
>
> Tested-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
> Tested on a custom ApolloLake board.

Thank you. But I will leave Simon to pick up the tag in the next
version whenever it makes sense.

>
> > BTW: any plan to upstream your board support?
>
> Yes, that would be one of the long term goals.

Great!

Regards,
Bin