Re: [RESEND PATCH v2 0/7] ASoC: Intel: Skylake: Driver fundaments overhaul

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 23-07-19, 16:58, Cezary Rojewski wrote:
> Skylake driver is divided into two modules:
> - snd_soc_skl
> - snd_soc_skl_ipc
> 
> and nothing would be wrong if not for the fact that both cannot exist
> without one another. IPC module is not some kind of extension, as it is
> the case for snd_hda_ext_core which is separated from snd_hda_core -
> legacy hda interface. It's as much core Skylake module as snd_soc_skl
> is.
> 
> Statement backup by existence of circular dependency between this two.
> To eliminate said problem, struct skl_sst has been created. From that
> momment, Skylake has been plagued by header errors (incomplete sturcts,
> unknown references etc.) whenever something new is to be added or code
> is cleaned up.
> 
> Fix this flawed design by merging snd_soc_skl and snd_soc_skl_ipc.
> Also, do not forget about struct skl_sst redundancy.
> Followup changes address harmful assumptions and false logic which
> driver currently implements e.g.: attempt to take role of master for
> DSP scheduling when in fact entire control takes place in DSP.
> 
> Changes since v1:
> - Rebased onto 5.4

On a lighter note:

Did this series time travel from the future! Did you charge the flux
capacitor to go back?

> 
> Amadeusz Sławiński (2):
>   ASoC: Intel: Skylake: Combine snd_soc_skl_ipc and snd_soc_skl
>   ASoC: Intel: Skylake: Do not disable FW notifications
> 
> Cezary Rojewski (5):
>   ASoC: Intel: Skylake: Merge skl_sst and skl into skl_dev struct
>   ASoC: Intel: Skylake: Remove MCPS available check
>   ASoC: Intel: Skylake: Remove memory available check
>   ASoC: Intel: Skylake: Make MCPS and CPS params obsolete
>   ASoC: Intel: Skylake: Cleanup skl_module_cfg declaration
> 
>  sound/soc/intel/common/sst-ipc.h        |   1 +
>  sound/soc/intel/skylake/Makefile        |  12 +-
>  sound/soc/intel/skylake/bxt-sst.c       |  50 +--
>  sound/soc/intel/skylake/cnl-sst-dsp.h   |   7 +-
>  sound/soc/intel/skylake/cnl-sst.c       |  37 +-
>  sound/soc/intel/skylake/skl-debug.c     |  14 +-
>  sound/soc/intel/skylake/skl-messages.c  | 245 ++++++-------
>  sound/soc/intel/skylake/skl-nhlt.c      |  18 +-
>  sound/soc/intel/skylake/skl-pcm.c       |  74 ++--
>  sound/soc/intel/skylake/skl-ssp-clk.c   |   4 +-
>  sound/soc/intel/skylake/skl-sst-dsp.c   |  10 +-
>  sound/soc/intel/skylake/skl-sst-dsp.h   |  29 +-
>  sound/soc/intel/skylake/skl-sst-ipc.c   |   8 +-
>  sound/soc/intel/skylake/skl-sst-ipc.h   |  52 +--
>  sound/soc/intel/skylake/skl-sst-utils.c |  37 +-
>  sound/soc/intel/skylake/skl-sst.c       |  51 +--
>  sound/soc/intel/skylake/skl-topology.c  | 441 ++++++++----------------
>  sound/soc/intel/skylake/skl-topology.h  |  43 +--
>  sound/soc/intel/skylake/skl.c           |  54 +--
>  sound/soc/intel/skylake/skl.h           | 102 ++++--
>  20 files changed, 546 insertions(+), 743 deletions(-)
> 
> -- 
> 2.17.1
> 
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

-- 
~Vinod
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux