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

[Marek Szyprowski]

Hi Kuninor

On 2019-06-25 02:51, Kuninori Morimoto wrote:
> 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 at renesas.com>
>> 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.

The same issue:

# aplay /usr/share/sounds/alsa/Front_Center.wav
aplay: main:788: audio open error: Invalid argument
#


> 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...

Reverting this patch fixes the issue, so it is definitely related somehow.


Best regards
-- 
Marek Szyprowski, PhD
Samsung R&D Institute Poland