alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/3] conf: Add vendor tuples in conf
@ 2016-08-12  7:00 Vinod Koul
  2016-08-22 12:26 ` Takashi Iwai
  0 siblings, 1 reply; 3+ messages in thread
From: Vinod Koul @ 2016-08-12  7:00 UTC (permalink / raw)
  To: alsa-devel; +Cc: liam.r.girdwood, patches.audio, broonie, Vinod Koul

Topology changes in alsa-lib allow driver data for modules to be be passed
in topology conf file using tuples. These changes allow private data to
represented by tuples of type UUID, byte, short, word and strings.

This patch series adds changes to :
 - include tuple and token list in the conf file
   to define module private data.
 - Private data blobs for modules are no longer required. So, remove the
   binary blobs and the tool used to generate the blobs.
 - Then add bxt conf file which is based on tuples

Takashi,
	This should be merged after the kernel changes are merged.

Shreyas NC (3):
  conf: sklrt286: define module private data through tuples in skylake
  conf: sklrt286: Remove tool to generate private data blobs
  conf: bxtrt298: Add topology conf file for bxt

 configure.ac                                       |    2 +-
 src/conf/topology/Makefile.am                      |    2 +-
 src/conf/topology/bxtrt298/Makefile.am             |    4 +
 src/conf/topology/bxtrt298/bxt_i2s.conf            | 3323 ++++++++++++++++++++
 src/conf/topology/sklrt286/Makefile.am             |    3 +-
 src/conf/topology/sklrt286/codec0_in-cpr-1.bin     |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/codec0_in-mi.bin        |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/codec0_out-cpr-4.bin    |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/codec0_out-mo.bin       |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/codec1_out-cpr-5.bin    |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/codec1_out-mo.bin       |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/data/Makefile.am        |    6 -
 src/conf/topology/sklrt286/data/pvt.c              | 1815 -----------
 src/conf/topology/sklrt286/data/pvt_data.c         |   90 -
 src/conf/topology/sklrt286/data/pvt_local.h        |    9 -
 .../topology/sklrt286/data/skl-tplg-interface.h    |  232 --
 .../topology/sklrt286/dmic01_hifi_in-cpr-3.bin     |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/dmic01_hifi_in-mi.bin   |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi1_pt_out-cpr-7.bin  |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi1_pt_out-cpr-8.bin  |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi2_pt_out-cpr-10.bin |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi2_pt_out-cpr-9.bin  |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi3_pt_out-cpr-11.bin |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/hdmi3_pt_out-cpr-12.bin |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/media0_in-cpr-0.bin     |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/media0_in-mi.bin        |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/media0_out-cpr-6.bin    |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/media0_out-mo.bin       |  Bin 4244 -> 0 bytes
 src/conf/topology/sklrt286/skl_i2s.conf            | 2694 +++++++++++++++-
 29 files changed, 5988 insertions(+), 2192 deletions(-)
 create mode 100644 src/conf/topology/bxtrt298/Makefile.am
 create mode 100644 src/conf/topology/bxtrt298/bxt_i2s.conf
 delete mode 100644 src/conf/topology/sklrt286/codec0_in-cpr-1.bin
 delete mode 100644 src/conf/topology/sklrt286/codec0_in-mi.bin
 delete mode 100644 src/conf/topology/sklrt286/codec0_out-cpr-4.bin
 delete mode 100644 src/conf/topology/sklrt286/codec0_out-mo.bin
 delete mode 100644 src/conf/topology/sklrt286/codec1_out-cpr-5.bin
 delete mode 100644 src/conf/topology/sklrt286/codec1_out-mo.bin
 delete mode 100644 src/conf/topology/sklrt286/data/Makefile.am
 delete mode 100644 src/conf/topology/sklrt286/data/pvt.c
 delete mode 100644 src/conf/topology/sklrt286/data/pvt_data.c
 delete mode 100644 src/conf/topology/sklrt286/data/pvt_local.h
 delete mode 100644 src/conf/topology/sklrt286/data/skl-tplg-interface.h
 delete mode 100644 src/conf/topology/sklrt286/dmic01_hifi_in-cpr-3.bin
 delete mode 100644 src/conf/topology/sklrt286/dmic01_hifi_in-mi.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi1_pt_out-cpr-7.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi1_pt_out-cpr-8.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi2_pt_out-cpr-10.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi2_pt_out-cpr-9.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi3_pt_out-cpr-11.bin
 delete mode 100644 src/conf/topology/sklrt286/hdmi3_pt_out-cpr-12.bin
 delete mode 100644 src/conf/topology/sklrt286/media0_in-cpr-0.bin
 delete mode 100644 src/conf/topology/sklrt286/media0_in-mi.bin
 delete mode 100644 src/conf/topology/sklrt286/media0_out-cpr-6.bin
 delete mode 100644 src/conf/topology/sklrt286/media0_out-mo.bin

-- 
1.9.1

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/3] conf: Add vendor tuples in conf
  2016-08-12  7:00 [PATCH 0/3] conf: Add vendor tuples in conf Vinod Koul
@ 2016-08-22 12:26 ` Takashi Iwai
  2016-08-23  3:56   ` Vinod Koul
  0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2016-08-22 12:26 UTC (permalink / raw)
  To: Vinod Koul; +Cc: liam.r.girdwood, patches.audio, alsa-devel, broonie

On Fri, 12 Aug 2016 09:00:14 +0200,
Vinod Koul wrote:
> 
> Topology changes in alsa-lib allow driver data for modules to be be passed
> in topology conf file using tuples. These changes allow private data to
> represented by tuples of type UUID, byte, short, word and strings.
> 
> This patch series adds changes to :
>  - include tuple and token list in the conf file
>    to define module private data.
>  - Private data blobs for modules are no longer required. So, remove the
>    binary blobs and the tool used to generate the blobs.
>  - Then add bxt conf file which is based on tuples
> 
> Takashi,
> 	This should be merged after the kernel changes are merged.
> 
> Shreyas NC (3):
>   conf: sklrt286: define module private data through tuples in skylake
>   conf: sklrt286: Remove tool to generate private data blobs
>   conf: bxtrt298: Add topology conf file for bxt

Did you submit all patches?  I've seen only this cover letter.
In anyway, please resubmit.


thanks,

Takashi

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/3] conf: Add vendor tuples in conf
  2016-08-22 12:26 ` Takashi Iwai
@ 2016-08-23  3:56   ` Vinod Koul
  0 siblings, 0 replies; 3+ messages in thread
From: Vinod Koul @ 2016-08-23  3:56 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: liam.r.girdwood, patches.audio, alsa-devel, broonie

On Mon, Aug 22, 2016 at 02:26:22PM +0200, Takashi Iwai wrote:
> On Fri, 12 Aug 2016 09:00:14 +0200,
> Vinod Koul wrote:
> > 
> > Topology changes in alsa-lib allow driver data for modules to be be passed
> > in topology conf file using tuples. These changes allow private data to
> > represented by tuples of type UUID, byte, short, word and strings.
> > 
> > This patch series adds changes to :
> >  - include tuple and token list in the conf file
> >    to define module private data.
> >  - Private data blobs for modules are no longer required. So, remove the
> >    binary blobs and the tool used to generate the blobs.
> >  - Then add bxt conf file which is based on tuples
> > 
> > Takashi,
> > 	This should be merged after the kernel changes are merged.
> > 
> > Shreyas NC (3):
> >   conf: sklrt286: define module private data through tuples in skylake
> >   conf: sklrt286: Remove tool to generate private data blobs
> >   conf: bxtrt298: Add topology conf file for bxt
> 
> Did you submit all patches?  I've seen only this cover letter.
> In anyway, please resubmit.

Ah not sure, how. I will send again now

-- 
~Vinod

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-08-23  3:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-12  7:00 [PATCH 0/3] conf: Add vendor tuples in conf Vinod Koul
2016-08-22 12:26 ` Takashi Iwai
2016-08-23  3:56   ` Vinod Koul

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).