Re: [alsa-devel] [PATCH resend 25/47] ASoC: samsung: snow: don't select unnecessary Platform

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

 



Hi Marek

Thank you for your report.

> > ALSA SoC is now supporting "no Platform". Sound card doesn't need to
> > select "CPU component" as "Platform" anymore if it doesn't need
> > special Platform.
> > This patch removes such settings.
> >
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
> 
> This patch causes regression on Samsung Exynos-based boards: Snow, 
> Peach-Pi and Peach-Pit (all that use snow machine driver) since linux 
> next-20190620. ASoC device is properly registered, but it is not usable:
> 
> [    3.203006] samsung-i2s 3830000.i2s-sec: DMA channels sourced from 
> device 3830000.i2s
> [    3.213440] snow-audio sound: Failed parsing codec node
> [    3.423241] snow-audio sound: multicodec <-> samsung-i2s mapping ok
> [    3.429776] max98095 7-0011: ASoC: mux External MIC has no paths
> [    3.434513] max98095 7-0011: ASoC: mux Linein Mux has no paths
> [    4.043488] ALSA device list:
> [    4.047171]   #0: Snow-I2S-MAX98095
> 
> # speaker-test -l1
> 
> speaker-test 1.1.3
> 
> Playback device is default
> Stream parameters are 48000Hz, S16_LE, 1 channels
> Using 16 octaves of pink noise
> Playback open error: -22,Invalid argument

Hmm.. strange...
Can you test normal aplay here ? I'm not familiar with speaker-test.

I have no idea so far, but one capability is that if your system
used / based dummy platform, and if your CPU driver didn't use/have
snd_soc_set_runtime_hwparams() / snd_pcm_lib_ioctl,
it might be happen, but this patch seems not related to it...


Thank you for your help !!
Best regards
---
Kuninori Morimoto



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux for Synopsys ARC Processors]    
  • [Linux on Unisoc (RDA Micro) SoCs]     [Linux Actions SoC]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  •   Powered by Linux