[PATCH v6 12/29] dm: core: Add basic ACPI support

[Simon Glass]

Hi Bin,

On Wed, 15 Apr 2020 at 08:38, Bin Meng <bmeng.cn at gmail.com> wrote:
>
> Hi Simon,
>
> On Wed, Apr 15, 2020 at 10:31 PM Bin Meng <bmeng.cn at gmail.com> wrote:
> >
> > Hi Simon,
> >
> > On Fri, Apr 10, 2020 at 3:36 AM Simon Glass <sjg at chromium.org> wrote:
> > >
> > > On Thu, 9 Apr 2020 at 11:40, Andy Shevchenko
> > > <andriy.shevchenko at linux.intel.com> wrote:
> > > >
> > > > On Thu, Apr 09, 2020 at 10:27:38AM -0600, Simon Glass wrote:
> > > > > ACPI (Advanced Configuration and Power Interface) is a standard for
> > > > > specifying information about a platform. It is a little like device
> > > > > tree but the bindings are part of the specification and it supports an
> > > > > interpreted bytecode language.
> > > > >
> > > > > Driver model does not use ACPI for U-Boot's configuration, but it is
> > > > > convenient to have it support generation of ACPI tables for passing to
> > > > > Linux, etc.
> > > > >
> > > > > As a starting point, add an optional set of ACPI operations to each
> > > > > device. Initially only a single operation is available, to obtain the
> > > > > ACPI name for the device. More operations are added later.
> > > > >
> > > > > Enable ACPI for sandbox to ensure build coverage and so that we can add
> > > > > tests.
> > > > >
> > > >
> > > > Both looks good to me now, thanks!
> > > >
> > > > > Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
> > > > > Reviewed-by: Wolfgang Wallner <wolfgang.wallner at br-automation.com>
> > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > ---
> > > > >
> > > > > Changes in v6:
> > > > > - Use ACPI_NAME_LEN in acpi_copy_name since we add the nul anyway
> > > > >
> > > > > Changes in v5:
> > > > > - Capitalise ACPI_OPS_PTR since it includes a comma
> > > > >
> > > > > Changes in v4: None
> > > > > Changes in v3:
> > > > > - Drop 'Intel' from 'Intel ACPI'
> > > > > - Reword commit message to drop the bit about ACPI being complicated
> > > > > - Compute ACPI_NAME_MAX from ACPI_NAME_LEN
> > > > > - Rename acpi_return_name() to acpi_copy_name()
> > > > > - Use strncpy() instead of strcpy() in acpi_copy_name()
> > > > >
> > > > > Changes in v2:
> > > > > - Move LOGC_ACPI definition to this patch
> > > > >
> > > > >  configs/tools-only_defconfig |  1 +
> > > > >  drivers/core/Kconfig         |  9 +++++
> > > > >  drivers/core/Makefile        |  1 +
> > > > >  drivers/core/acpi.c          | 33 ++++++++++++++++
> > > > >  include/dm/acpi.h            | 73 ++++++++++++++++++++++++++++++++++++
> > > > >  include/dm/device.h          |  5 +++
> > > > >  include/log.h                |  2 +
> > > > >  7 files changed, 124 insertions(+)
> > > > >  create mode 100644 drivers/core/acpi.c
> > > > >  create mode 100644 include/dm/acpi.h
> > >
> > > Andy let me add your tag.
> > >
> > > Reviewed-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
> >
> > Do you know what happened to this series?
> >
> > I only see 2 patches in this series showing up on the patchwork.
> > http://patchwork.ozlabs.org/user/todo/uboot/?series=169449
>
> Or show I replace the 2 patches tagged as v6 in the v5 sereis?
> http://patchwork.ozlabs.org/user/todo/uboot/?series=169328

Yes that's right. I didn't send the whole thing again as there were
just those two minor changes.

Regards,
Simon