From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Glass Date: Thu, 2 Jul 2020 18:46:03 -0600 Subject: [PATCH v1 07/43] dm: acpi: Add support for the NHLT table In-Reply-To: References: <20200615035738.248710-1-sjg@chromium.org> <20200615035738.248710-5-sjg@chromium.org> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Bin, On Mon, 29 Jun 2020 at 20:49, Bin Meng wrote: > > Hi Simon, > > On Mon, Jun 15, 2020 at 11:57 AM Simon Glass wrote: > > > > The Intel Non-High-Definition-Audio Link Table (NHLT) table describes the > > audio codecs and connections in a system. Various devices can contribute > > information to produce the table. > > > > Add core support for this, based on a structure which is built up through > > calls to the driver. > > > > Signed-off-by: Simon Glass > > --- > > > > drivers/core/acpi.c | 15 +++++++++++++++ > > include/dm/acpi.h | 26 ++++++++++++++++++++++++++ > > 2 files changed, 41 insertions(+) > > > > diff --git a/drivers/core/acpi.c b/drivers/core/acpi.c > > index ea304a3067..a5053fec6f 100644 > > --- a/drivers/core/acpi.c > > +++ b/drivers/core/acpi.c > > @@ -31,6 +31,7 @@ enum method_t { > > METHOD_WRITE_TABLES, > > METHOD_FILL_SSDT, > > METHOD_INJECT_DSDT, > > + METHOD_SETUP_NHLT, > > Do we really need to promote this to be an ACPI core method? Can we > reuse the SSDT/DSDT one? I don't think so. Those two are for a particular purpose. In fact NHLT is generated while doing SSDT I think. The idea is that drivers that want to contribute to NHLT can do so. But we cannot use the SSDT mechanism since each driver contributes only a part of the info, and we need something else to bring it all together. Regards, Simon