Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Vinod Koul <vinod.koul@intel.com>
To: Jie Yang <yang.jie@intel.com>
Cc: alsa-devel@alsa-project.org, liam.r.girdwood@intel.com,
	mengdong.lin@intel.com, broonie@kernel.org,
	jarkko.nikula@intel.com, yao.jin@intel.com,
	pierre-louis.bossart@intel.com
Subject: Re: [PATCH 0/5] Intel SST driver restructure
Date: Fri, 13 Mar 2015 11:58:29 +0530	[thread overview]
Message-ID: <20150313062829.GA32000@intel.com> (raw)
In-Reply-To: <1426170720-8260-1-git-send-email-yang.jie@intel.com>

On Thu, Mar 12, 2015 at 10:31:55PM +0800, Jie Yang wrote:
> In order to prepare for the upstreaming of Skylake drivers and to
> allow reuse of code between different Intel audio DSP products,
> this patch series restructures the Intel directory as follows:
> 
> 1. Create "common" directory. Move common files into this directory.
> 
> 2. Create "boards" directory. Move all machine drivers into this directory.
> 
> 3. Create "haswell" directory. Move all remaining hsw/bdw code into this directory.
> 
> 4. Create "baytrail" directory. Move remaining baytrail driver to this directory.
> 
> 5. Create "atom" directory. Move all MCG platform drivers(including sst/ folder)
> to this directory.
> 
> Todo next step:
> 1. Change and move atom related codes(in atom/sst/) to corresponding and try
> share as many as possible codes(e.g. sst_acpi.c).
> 2. Share more common codes about IPC for different platforms, e.g. The methods for
> queueing and sending/reciving Tx/Rx messages, The methods for mailbox IO, The IPC
> error return codes, etc.
> 
> Jie Yang (5):
>   ASoC: Intel: create common folder and move common files in
>   ASoC: Intel: create haswell folder and move haswell platform files in
>   ASoC: Intel: create boards folder and move sst boards files in
>   ASoC: Intel: create baytrail folder and move baytrail platform files
>     in
>   ASoC: Intel: create atom folder and move atom platform files in
> 
>  sound/soc/intel/Makefile                           | 42 +++-------------------
>  sound/soc/intel/atom/Makefile                      |  7 ++++
>  sound/soc/intel/{ => atom}/sst-atom-controls.c     |  0
>  sound/soc/intel/{ => atom}/sst-atom-controls.h     |  0
>  sound/soc/intel/{ => atom}/sst-mfld-dsp.h          |  0
>  .../intel/{ => atom}/sst-mfld-platform-compress.c  |  0
>  sound/soc/intel/{ => atom}/sst-mfld-platform-pcm.c |  0
>  sound/soc/intel/{ => atom}/sst-mfld-platform.h     |  0
>  sound/soc/intel/{ => atom}/sst/Makefile            |  0
>  sound/soc/intel/{ => atom}/sst/sst.c               |  0
>  sound/soc/intel/{ => atom}/sst/sst.h               |  0
>  sound/soc/intel/{ => atom}/sst/sst_acpi.c          |  0
>  sound/soc/intel/{ => atom}/sst/sst_drv_interface.c |  0
>  sound/soc/intel/{ => atom}/sst/sst_ipc.c           |  0
>  sound/soc/intel/{ => atom}/sst/sst_loader.c        |  0
>  sound/soc/intel/{ => atom}/sst/sst_pci.c           |  0
>  sound/soc/intel/{ => atom}/sst/sst_pvt.c           |  0
>  sound/soc/intel/{ => atom}/sst/sst_stream.c        |  0
For atom parts

Acked-by: Vinod Koul <vinod.koul@intel.com>

-- 
~Vinod

>  sound/soc/intel/baytrail/Makefile                  |  4 +++
>  sound/soc/intel/{ => baytrail}/sst-baytrail-dsp.c  |  4 +--
>  sound/soc/intel/{ => baytrail}/sst-baytrail-ipc.c  |  4 +--
>  sound/soc/intel/{ => baytrail}/sst-baytrail-ipc.h  |  0
>  sound/soc/intel/{ => baytrail}/sst-baytrail-pcm.c  |  4 +--
>  sound/soc/intel/boards/Makefile                    | 15 ++++++++
>  sound/soc/intel/{ => boards}/broadwell.c           |  6 ++--
>  sound/soc/intel/{ => boards}/byt-max98090.c        |  2 +-
>  sound/soc/intel/{ => boards}/byt-rt5640.c          |  4 +--
>  .../{bytcr_dpcm_rt5640.c => boards/bytcr_rt5640.c} |  0
>  sound/soc/intel/{ => boards}/cht_bsw_rt5645.c      |  0
>  sound/soc/intel/{ => boards}/cht_bsw_rt5672.c      |  0
>  sound/soc/intel/{ => boards}/haswell.c             |  6 ++--
>  sound/soc/intel/{ => boards}/mfld_machine.c        |  0
>  sound/soc/intel/common/Makefile                    |  6 ++++
>  sound/soc/intel/{ => common}/sst-acpi.c            |  0
>  sound/soc/intel/{ => common}/sst-dsp-priv.h        |  0
>  sound/soc/intel/{ => common}/sst-dsp.c             |  0
>  sound/soc/intel/{ => common}/sst-dsp.h             |  0
>  sound/soc/intel/{ => common}/sst-firmware.c        |  0
>  sound/soc/intel/haswell/Makefile                   |  4 +++
>  sound/soc/intel/{ => haswell}/sst-haswell-dsp.c    |  6 ++--
>  sound/soc/intel/{ => haswell}/sst-haswell-ipc.c    |  4 +--
>  sound/soc/intel/{ => haswell}/sst-haswell-ipc.h    |  0
>  sound/soc/intel/{ => haswell}/sst-haswell-pcm.c    |  6 ++--
>  43 files changed, 64 insertions(+), 60 deletions(-)
>  create mode 100644 sound/soc/intel/atom/Makefile
>  rename sound/soc/intel/{ => atom}/sst-atom-controls.c (100%)
>  rename sound/soc/intel/{ => atom}/sst-atom-controls.h (100%)
>  rename sound/soc/intel/{ => atom}/sst-mfld-dsp.h (100%)
>  rename sound/soc/intel/{ => atom}/sst-mfld-platform-compress.c (100%)
>  rename sound/soc/intel/{ => atom}/sst-mfld-platform-pcm.c (100%)
>  rename sound/soc/intel/{ => atom}/sst-mfld-platform.h (100%)
>  rename sound/soc/intel/{ => atom}/sst/Makefile (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst.h (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_acpi.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_drv_interface.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_ipc.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_loader.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_pci.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_pvt.c (100%)
>  rename sound/soc/intel/{ => atom}/sst/sst_stream.c (100%)
>  create mode 100644 sound/soc/intel/baytrail/Makefile
>  rename sound/soc/intel/{ => baytrail}/sst-baytrail-dsp.c (99%)
>  rename sound/soc/intel/{ => baytrail}/sst-baytrail-ipc.c (99%)
>  rename sound/soc/intel/{ => baytrail}/sst-baytrail-ipc.h (100%)
>  rename sound/soc/intel/{ => baytrail}/sst-baytrail-pcm.c (99%)
>  create mode 100644 sound/soc/intel/boards/Makefile
>  rename sound/soc/intel/{ => boards}/broadwell.c (98%)
>  rename sound/soc/intel/{ => boards}/byt-max98090.c (99%)
>  rename sound/soc/intel/{ => boards}/byt-rt5640.c (98%)
>  rename sound/soc/intel/{bytcr_dpcm_rt5640.c => boards/bytcr_rt5640.c} (100%)
>  rename sound/soc/intel/{ => boards}/cht_bsw_rt5645.c (100%)
>  rename sound/soc/intel/{ => boards}/cht_bsw_rt5672.c (100%)
>  rename sound/soc/intel/{ => boards}/haswell.c (98%)
>  rename sound/soc/intel/{ => boards}/mfld_machine.c (100%)
>  create mode 100644 sound/soc/intel/common/Makefile
>  rename sound/soc/intel/{ => common}/sst-acpi.c (100%)
>  rename sound/soc/intel/{ => common}/sst-dsp-priv.h (100%)
>  rename sound/soc/intel/{ => common}/sst-dsp.c (100%)
>  rename sound/soc/intel/{ => common}/sst-dsp.h (100%)
>  rename sound/soc/intel/{ => common}/sst-firmware.c (100%)
>  create mode 100644 sound/soc/intel/haswell/Makefile
>  rename sound/soc/intel/{ => haswell}/sst-haswell-dsp.c (99%)
>  rename sound/soc/intel/{ => haswell}/sst-haswell-ipc.c (99%)
>  rename sound/soc/intel/{ => haswell}/sst-haswell-ipc.h (100%)
>  rename sound/soc/intel/{ => haswell}/sst-haswell-pcm.c (99%)
> 
> -- 
> 1.9.1
> 

-- 

  parent reply	other threads:[~2015-03-13  6:32 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-12 14:31 [PATCH 0/5] Intel SST driver restructure Jie Yang
2015-03-12 14:31 ` [PATCH 1/5] ASoC: Intel: create common folder and move common files in Jie Yang
2015-03-12 14:31 ` [PATCH 2/5] ASoC: Intel: create haswell folder and move haswell platform " Jie Yang
2015-03-12 14:31 ` [PATCH 3/5] ASoC: Intel: create boards folder and move sst boards " Jie Yang
2015-03-12 14:31 ` [PATCH 4/5] ASoC: Intel: create baytrail folder and move baytrail platform " Jie Yang
2015-03-12 14:32 ` [PATCH 5/5] ASoC: Intel: create atom folder and move atom " Jie Yang
2015-03-12 19:54 ` [PATCH 0/5] Intel SST driver restructure Mark Brown
2015-03-13  1:28   ` Jie, Yang
2015-03-13 14:32     ` Antonio Ospite
2015-03-13  6:32   ` Vinod Koul
2015-03-13  6:28 ` Vinod Koul [this message]
2015-03-13  8:19   ` Jarkko Nikula
2015-03-13 14:13     ` Jie, Yang
2015-03-13 23:34       ` Mark Brown
2015-03-14  6:04         ` Jie, Yang
2015-04-01  1:53           ` Jie, Yang
2015-04-01  7:08             ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150313062829.GA32000@intel.com \
    --to=vinod.koul@intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=jarkko.nikula@intel.com \
    --cc=liam.r.girdwood@intel.com \
    --cc=mengdong.lin@intel.com \
    --cc=pierre-louis.bossart@intel.com \
    --cc=yang.jie@intel.com \
    --cc=yao.jin@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox