From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Mon, 6 Jul 2020 13:21:51 -0600
Subject: [PATCH v1 07/43] dm: acpi: Add support for the NHLT table
In-Reply-To: <CAEUhbmU8T6Q21nW2M=Ok4LRrJ87thGSb1zc-V6cOcjqGo0CACg@mail.gmail.com>
References: <20200615035738.248710-1-sjg@chromium.org>
 <20200615035738.248710-5-sjg@chromium.org>
 <CAEUhbmXSeq_CHXM6heqpfQVTLr_PHv-evubTznC1Vrb-ymGEuw@mail.gmail.com>
 <CAPnjgZ1ST3jAmesPLKW1-=C+Ajn1Lg_Fh14ijZ4_6s_TMKJcGA@mail.gmail.com>
 <CAEUhbmW9BEkYm8BS+1rFGeNnZwaN74hOC7Ba9hQD-53ZfXCQYg@mail.gmail.com>
 <CAPnjgZ0N5z+98i3Aku-_PrZPPpjCw2Wmutv4oLpi0Bv_54ZyQQ@mail.gmail.com>
 <CAEUhbmU8T6Q21nW2M=Ok4LRrJ87thGSb1zc-V6cOcjqGo0CACg@mail.gmail.com>
Message-ID: <CAPnjgZ2C32k1oRuhjwe+gyPGn4w2QSgDbUqHqkQ8+kk0iDvuLw@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Bin,

On Thu, 2 Jul 2020 at 22:33, Bin Meng <bmeng.cn@gmail.com> wrote:
>
> Hi Simon,
>
> On Fri, Jul 3, 2020 at 11:50 AM Simon Glass <sjg@chromium.org> wrote:
> >
> > Hi Bin,
> >
> > On Thu, 2 Jul 2020 at 18:54, Bin Meng <bmeng.cn@gmail.com> wrote:
> > >
> > > Hi Simon,
> > >
> > > On Fri, Jul 3, 2020 at 8:46 AM Simon Glass <sjg@chromium.org> wrote:
> > > >
> > > > Hi Bin,
> > > >
> > > > On Mon, 29 Jun 2020 at 20:49, Bin Meng <bmeng.cn@gmail.com> wrote:
> > > > >
> > > > > Hi Simon,
> > > > >
> > > > > On Mon, Jun 15, 2020 at 11:57 AM Simon Glass <sjg@chromium.org> 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 <sjg@chromium.org>
> > > > > > ---
> > > > > >
> > > > > >  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.
> > >
> > > Will there be only one device that sets up the NHLT info?
> >
> > WIth coral it is two devices. I'm not sure of the maximum, but I
> > suppose it depends on the audio codecs present.
>
> Could we make this method to be provided by the codec device, instead
> of a generic ACPI core method?

The codec device does implement this. See the drivers where they
actually implement the NHLT method.

This is definitely an ACPI-specific thing, so I think we need core
support for iterating through drivers that want to provide this info.

Regards,
Simon