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

Bin Meng bmeng.cn at gmail.com
Wed Apr 15 16:38:37 CEST 2020


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

Regards,
Bin


More information about the U-Boot mailing list