1. 28 Jun, 2019 22 commits
    • Kuninori Morimoto's avatar
      ASoC: kirkwood: armada-370-db: consider CPU-Platform possibility · bb6630d1
      Kuninori Morimoto authored
      commit 717f1633 ("ASoC: kirkwood: armada-370-db: don't select
      unnecessary Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 717f1633 ("ASoC: kirkwood: armada-370-db: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      bb6630d1
    • Kuninori Morimoto's avatar
      ASoC: fsl: imx-audmix: consider CPU-Platform possibility · 9213866a
      Kuninori Morimoto authored
      commit d8893261 ("ASoC: fsl: imx-audmix: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit d8893261 ("ASoC: fsl: imx-audmix: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      9213866a
    • Kuninori Morimoto's avatar
      ASoC: fsl: imx-spdif: consider CPU-Platform possibility · 9e8f5299
      Kuninori Morimoto authored
      commit 014f07ca ("ASoC: fsl: imx-spdif: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 014f07ca ("ASoC: fsl: imx-spdif: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      9e8f5299
    • Kuninori Morimoto's avatar
      ASoC: fsl: imx-sgtl5000: consider CPU-Platform possibility · 2b1407c9
      Kuninori Morimoto authored
      commit 82bf78ca ("ASoC: fsl: imx-sgtl5000: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 82bf78ca ("ASoC: fsl: imx-sgtl5000: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      2b1407c9
    • Kuninori Morimoto's avatar
      ASoC: fsl: imx-es8328: consider CPU-Platform possibility · 8bebc822
      Kuninori Morimoto authored
      commit 577cf50d ("ASoC: fsl: imx-es8328: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 577cf50d ("ASoC: fsl: imx-es8328: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      8bebc822
    • Kuninori Morimoto's avatar
      ASoC: fsl: fsl-asoc-card: consider CPU-Platform possibility · 9998d3e2
      Kuninori Morimoto authored
      commit e57a4c2f ("ASoC: fsl: fsl-asoc-card: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit e57a4c2f ("ASoC: fsl: fsl-asoc-card: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      9998d3e2
    • Kuninori Morimoto's avatar
      ASoC: fsl: eukrea-tlv320: consider CPU-Platform possibility · 68707c37
      Kuninori Morimoto authored
      commit 2058ea1c ("ASoC: fsl: eukrea-tlv320: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 2058ea1c ("ASoC: fsl: eukrea-tlv320: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      68707c37
    • Kuninori Morimoto's avatar
      ASoC: atmel: tse850-pcm5142: consider CPU-Platform possibility · 6910bb93
      Kuninori Morimoto authored
      commit 655368df ("ASoC: atmel: tse850-pcm5142: don't select
      unnecessary Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 655368df ("ASoC: atmel: tse850-pcm5142: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      6910bb93
    • Kuninori Morimoto's avatar
      ASoC: atmel: sam9x5_wm8731: consider CPU-Platform possibility · e1be6aa0
      Kuninori Morimoto authored
      commit ced5b080 ("ASoC: atmel: sam9x5_wm8731: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit ced5b080 ("ASoC: atmel: sam9x5_wm8731: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      e1be6aa0
    • Kuninori Morimoto's avatar
      ASoC: atmel: sam9g20_wm8731: consider CPU-Platform possibility · 0efd1d7c
      Kuninori Morimoto authored
      commit bfc7938e ("ASoC: atmel: sam9g20_wm8731: don't select
      unnecessary Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit bfc7938e ("ASoC: atmel: sam9g20_wm8731: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      0efd1d7c
    • Kuninori Morimoto's avatar
      ASoC: amtel: mikroe-proto: consider CPU-Platform possibility · 716407ea
      Kuninori Morimoto authored
      commit 318ebbe8 ("ASoC: atmel: mikroe-proto: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 318ebbe8 ("ASoC: atmel: mikroe-proto: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      716407ea
    • Kuninori Morimoto's avatar
      ASoC: atmel: atmel_wm8904: consider CPU-Platform possibility unnecessary Platform" · 16589b77
      Kuninori Morimoto authored
      commit 3609750e ("ASoC: atmel: atmel_wm8904: don't select
      unnecessary Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 3609750e ("ASoC: atmel: atmel_wm8904: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      16589b77
    • Kuninori Morimoto's avatar
      ASoC: atmel: atmel-pdmic: consider CPU-Platform possibility · ea2d1868
      Kuninori Morimoto authored
      commit 7baf32e1 ("ASoC: atmel: atmel-pdmic: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 7baf32e1 ("ASoC: atmel: atmel-pdmic: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      ea2d1868
    • Kuninori Morimoto's avatar
      ASoC: atmel: atmel-classd: consider CPU-Platform possibility · ca1c67d1
      Kuninori Morimoto authored
      commit 02602401 ("ASoC: atmel: atmel-classd: don't select
      unnecessary Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 02602401 ("ASoC: atmel: atmel-classd: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      ca1c67d1
    • Kuninori Morimoto's avatar
      ASoC: vc4: vc4_htmi: consider CPU-Platform possibility · 8a90efd1
      Kuninori Morimoto authored
      commit 6c6de1c9 ("ASoC: vc4: vc4_hdmi: don't select unnecessary
      Platform")
      
      Current ALSA SoC avoid to add duplicate component to rtd,
      and this driver was selecting CPU component as Platform component.
      Thus, above patch removed Platform settings from this driver,
      because it assumed these are same component.
      
      But, some CPU driver is using generic DMAEngine, in such case, both
      CPU component and Platform component will have same of_node/name.
      In other words, there are some components which are different but
      have same of_node/name.
      
      In such case, Card driver definitely need to select Platform even
      though it is same as CPU.
      It is depends on CPU driver, but is difficult to know it from Card driver.
      This patch reverts above patch.
      
      Fixes: commit 6c6de1c9 ("ASoC: vc4: vc4_hdmi: don't select unnecessary Platform")
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      8a90efd1
    • Colin Ian King's avatar
      ASoC: topology: fix memory leaks on sm, se and sbe · 1ad741d0
      Colin Ian King authored
      Currently when a kstrdup fails the error exit paths don't free
      the allocations for sm, se and sbe.  This can be fixed by assigning
      kc[i].private_value to these before doing the ksrtdup so that the error
      exit path will be able to free these objects.
      
      Addresses-Coverity: ("Resource leak")
      Fixes: 9f90af3a ("ASoC: topology: Consolidate and fix asoc_tplg_dapm_widget_*_create flow")
      Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      1ad741d0
    • Kuninori Morimoto's avatar
      ASoC: atmel: atmel-pcm-dma.c: use devm_snd_dmaengine_pcm_register() · 8af26493
      Kuninori Morimoto authored
      We have devm_xxx version of snd_dmaengine_pcm_register,
      let's use it.
      
      This patch also removes related empty functions
      Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Reviewed-by: default avatarCodrin Ciubotariu <codrin.ciubotariu@microchip.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      8af26493
    • Jerome Brunet's avatar
      ASoC: meson: axg-card: remove useless check on codec · 8fc22fa4
      Jerome Brunet authored
      While checking cpus before dereferencing the pointer is required, it is
      not necessary for codecs. 'codec' can't possibly be NULL in the loop
      Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
      Signed-off-by: default avatarJerome Brunet <jbrunet@baylibre.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      8fc22fa4
    • Jerome Brunet's avatar
      ASoC: soc-core: support dai_link with platforms_num != 1 · 34614739
      Jerome Brunet authored
      Add support platforms_num != 1 in dai_link. Initially, the main purpose of
      this change was to make the platform optional in the dai_link, instead of
      inserting the dummy platform driver.
      
      This particular case had just been solved by Kuninori Morimoto with
      commit 1d768989 ("ASoC: soc-core: allow no Platform on dai_link").
      
      However, this change may still be useful for those who need multiple
      platform components on a single dai_link (it solves one of the FIXME
      note in soc-core)
      Acked-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
      Signed-off-by: default avatarJerome Brunet <jbrunet@baylibre.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      34614739
    • Jerome Brunet's avatar
      ASoC: soc-core: defer card registration if codec component is missing · af18b13f
      Jerome Brunet authored
      Like cpus and platforms, defer sound card initialization if the codec
      component is missing when initializing the dai_link
      Signed-off-by: default avatarJerome Brunet <jbrunet@baylibre.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      af18b13f
    • Codrin Ciubotariu's avatar
      ASoC: codecs: ad193x: Reset used registers at probe · 8af6b229
      Codrin Ciubotariu authored
      Since the ad193x codecs have no software reset, we have to reinitialize the
      registers after a hardware reset to assure no previous values are kept.
      Signed-off-by: default avatarCodrin Ciubotariu <codrin.ciubotariu@microchip.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      8af6b229
    • Codrin Ciubotariu's avatar
      ASoC: codecs: ad193x: Group register initialization at probe · bc0a5f43
      Codrin Ciubotariu authored
      Create a structure with the register initialization values at probe and
      use it to initialize all the registers at once.
      Signed-off-by: default avatarCodrin Ciubotariu <codrin.ciubotariu@microchip.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      bc0a5f43
  2. 26 Jun, 2019 5 commits
  3. 25 Jun, 2019 13 commits